n8n icon indicating copy to clipboard operation
n8n copied to clipboard

feat(editor): Implement AI Assistant chat UI

Open MiloradFilipovic opened this issue 10 months ago • 1 comments

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.

MiloradFilipovic avatar May 03 '24 09:05 MiloradFilipovic

3 flaky tests on run #4881 ↗︎

0 359 12 0 Flakiness 3

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
Flakiness  5-ndv.cy.ts • 2 flaky tests

View Output Video

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
Flakiness  24-ndv-paired-item.cy.ts • 1 flaky test

View Output Video

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 ↗︎

cypress[bot] avatar May 03 '24 13:05 cypress[bot]

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 image

MiloradFilipovic avatar May 07 '24 08:05 MiloradFilipovic

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 image

I did some testing on my local instance with your changes, and everything seems to be working as expected. Thanks!

OlegIvaniv avatar May 07 '24 11:05 OlegIvaniv

:white_check_mark: All Cypress E2E specs passed

github-actions[bot] avatar May 07 '24 11:05 github-actions[bot]

Got released with [email protected]

janober avatar May 08 '24 12:05 janober