n8n
n8n copied to clipboard
feat(editor): Implement AI Assistant chat UI
Summary
This PR introduces initial version of the Generate next step AI Assistant. It will be used as an experiment at first but should become a full-fledged feature soon.
Review notes
Please take a look at this Linear comment before reviewing.
How to test
See here
Related tickets and issues
ADO-2190
Review / Merge checklist
- [ ] PR title and summary are descriptive. Remember, the title automatically goes into the changelog. Use
(no-changelog)
otherwise. (conventions) - [ ] Docs updated or follow-up ticket created.
- [ ] Tests included.
A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests.
3 flaky tests on run #4881 ↗︎
![]() |
![]() |
![]() |
![]() |
![]() |
Details:
🌳 🖥️ browsers:node18.12.0-chrome107 🤖 MiloradFilipovic 🗃️ e2e/* | |||
Project: n8n | Commit: 5ce9d7464c |
||
Status: Passed | Duration: 04:30 💡 | ||
Started: May 7, 2024 11:52 AM | Ended: May 7, 2024 11:56 AM |
5-ndv.cy.ts • 2 flaky tests
Test | Artifacts | |
---|---|---|
NDV > should not retrieve remote options when required params throw errors |
Screenshots
Video
|
|
NDV > Stop listening for trigger event from NDV |
Screenshots
Video
|
24-ndv-paired-item.cy.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
NDV > resolves expression with default item when input node is not parent, while still pairing items |
Test Replay
Screenshots
Video
|
Review all test suite changes for PR #9300 ↗︎
LGTM, I'm just wondering, did you test the new changes with the Chat Trigger node?
@OlegIvaniv thanks!
From what I see, looks good (left is the current master
, right is this branch) but was hoping you can give it a test ride also since you have more context here
LGTM, I'm just wondering, did you test the new changes with the Chat Trigger node?
@OlegIvaniv thanks! From what I see, looks good (left is the current
master
, right is this branch) but was hoping you can give it a test ride also since you have more context here
I did some testing on my local instance with your changes, and everything seems to be working as expected. Thanks!
:white_check_mark: All Cypress E2E specs passed
Got released with [email protected]