Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(core): AI tool creation breaking changes #7246

Closed
wants to merge 352 commits into from

Conversation

janober
Copy link
Member

@janober janober commented Sep 25, 2023

Github issue / Community forum post (link here to close automatically):

OlegIvaniv and others added 30 commits August 25, 2023 18:50
…#7005)

- Pinecone Vector Store Insert/Load
- VectoreStore Retrieval
- Vector Store QA Chain
- Aligned input/output colors
Github issue / Community forum post (link here to close automatically):

---------

Signed-off-by: Oleg Ivaniv <me@olegivaniv.com>
Github issue / Community forum post (link here to close automatically):

---------

Signed-off-by: Oleg Ivaniv <me@olegivaniv.com>
netroy added a commit that referenced this pull request Oct 2, 2023
This extracts all core and editor changes from #7246 and #7137, so that
we can get these changes merged first.

ADO-1120

[DB Tests](https://github.com/n8n-io/n8n/actions/runs/6379749011)
[E2E Tests](https://github.com/n8n-io/n8n/actions/runs/6379751480)
[Workflow Tests](https://github.com/n8n-io/n8n/actions/runs/6379752828)

---------

Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com>
Co-authored-by: Oleg Ivaniv <me@olegivaniv.com>
Co-authored-by: Alex Grozav <alex@grozav.com>
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <aditya@netroy.in>
@netroy netroy changed the base branch from ai-tool-creation to master October 2, 2023 15:54
netroy and others added 4 commits October 2, 2023 18:02
Round-up of copy edits.
Includes making the Retrieval Q&A node names consistent (they were
previously different in the nodes panel and the node) This will be
confusing for users, especially ones working with templates, but also
anyone using the docs.
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Attention: 50 lines in your changes are missing coverage. Please review.

Comparison is base (00a4b8b) 33.29% compared to head (e5f84c3) 33.35%.
Report is 4 commits behind head on master.

❗ Current head e5f84c3 differs from pull request most recent head 18d4c88. Consider uploading reports for the commit 18d4c88 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7246      +/-   ##
==========================================
+ Coverage   33.29%   33.35%   +0.05%     
==========================================
  Files        3376     3376              
  Lines      205691   205723      +32     
  Branches    22136    22146      +10     
==========================================
+ Hits        68485    68610     +125     
+ Misses     136093   135999      -94     
- Partials     1113     1114       +1     
Files Coverage Δ
packages/cli/src/LoadNodesAndCredentials.ts 51.23% <100.00%> (+0.30%) ⬆️
packages/cli/src/constants.ts 100.00% <100.00%> (ø)
packages/editor-ui/src/constants.ts 99.68% <100.00%> (ø)
packages/editor-ui/src/views/NodeView.vue 17.32% <ø> (+<0.01%) ⬆️
packages/nodes-base/nodes/Code/Code.node.ts 85.41% <100.00%> (ø)
packages/nodes-base/nodes/Code/Sandbox.ts 72.00% <ø> (ø)
packages/nodes-base/nodes/Webhook/description.ts 100.00% <ø> (ø)
packages/workflow/src/Interfaces.ts 77.77% <ø> (ø)
...kages/editor-ui/src/components/NodeDetailsView.vue 43.09% <0.00%> (ø)
packages/editor-ui/src/components/OutputPanel.vue 63.90% <33.33%> (ø)
... and 8 more

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@janober
Copy link
Member Author

janober commented Oct 3, 2023

Closed in favor of: #7336

@janober janober closed this Oct 3, 2023
@github-actions github-actions bot mentioned this pull request Oct 4, 2023
krynble added a commit that referenced this pull request Oct 5, 2023
# [1.10.0](https://github.com/n8n-io/n8n/compare/n8n@1.9.0...n8n@1.10.0)
(2023-10-05)


### Bug Fixes

* **Convert to/from binary data Node:** Rename 'Move Binary Data' to
'Convert to/from binary data'
([#7318](#7318))
([5e6c1d4](5e6c1d4))
* **core:** Account for itemless case on restoring binary data ID
([#7305](#7305))
([1691223](1691223))
* **core:** Fix pruning of non-finished executions
([#7333](#7333))
([1b4848a](1b4848a))
* **editor:** Disable email confirmation banner for trialing users
([#7340](#7340))
([6d3d178](6d3d178))
* **editor:** Display value of selected matching column in RMC
([#7298](#7298))
([3aac22b](3aac22b))
* **editor:** Fix canvas endpoint snapping when dragging connection
([#7346](#7346))
([b59b908](b59b908))
* **editor:** Fix disappearing NDV header in code nodes
([#7290](#7290))
([7ebf8f3](7ebf8f3))
* **editor:** Fix RLC not loading when an expression can't resolve
([#7295](#7295))
([ddc26c2](ddc26c2))
* **editor:** Separate cloud endpoint calls
([#7312](#7312))
([04dfcd7](04dfcd7))
* **Jira Software Node:** Get all users in dropdown/RLC
([#7322](#7322))
([3704760](3704760)),
closes [#2670](#2670)
* **Notion Node:** Rename Notion API Key to Internal Integration Token
([#7176](#7176))
([ec2aa38](ec2aa38))
* **Postgres Node:** Node requires comma-separated string even when
using a single parameter through an expression
([#7300](#7300))
([763d451](763d451))
* **Set Node:** Do not stringify null and undefined
([#7313](#7313))
([f0a6687](f0a6687))
* **Typeform Trigger Node:** Change output format for TypeForm trigger
to object instead of array
([#7315](#7315))
([b3fc00e](b3fc00e))


### Features

* **core:** Add "Sent by n8n" attribution
([#7183](#7183))
([8f9fe62](8f9fe62))
* **core:** Add support for building LLM applications
([#7235](#7235))
([00a4b8b](00a4b8b)),
closes [#7246](#7246)
[#7137](#7137)
* Workflow History pruning and prune time settings
([#7343](#7343))
([0adc533](0adc533))

Co-authored-by: krynble <krynble@users.noreply.github.com>
@Platonn
Copy link

Platonn commented Oct 10, 2023

Hi @janober Do you consider adding to n8n the support of streaming responses for better UX of chat applications, in the near future?
In other words: LLMs support responding in streams of tokens. It would be great if n8n Respond to Webhook node could also stream those tokes to the Client, instead of returning the whole response in one batch.

@janober
Copy link
Member Author

janober commented Oct 10, 2023

Probably but there is no ETA right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet