Skip to content

Conversation

Cadiac
Copy link
Contributor

@Cadiac Cadiac commented Oct 21, 2025

Summary

Save AI message chunks as they're being streamed from execution engine to the frontend by wrapping the res object into a helper that captures the chunks. Construct the partial message and save it on cancel - we might want to save our full message saving to use this mechanism instead of parsing it from executions later on too 🤔

But for now the goal was just to save the partial messages.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/CHA-34/saving-partial-messages

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Oct 21, 2025
Copy link

bundlemon bot commented Oct 21, 2025

BundleMon

Unchanged files (2)
Status Path Size Limits
WASM Dependencies
tree-sitter-bash.wasm
181.26KB -
WASM Dependencies
tree-sitter.wasm
74.47KB -

No change in files bundle size

Groups updated (2)
Status Path Size Limits
**/*.js
5.84MB (+430.83KB +7.76%) -
**/*.css
214.82KB (+1.56KB +0.73%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@Cadiac Cadiac marked this pull request as ready for review October 21, 2025 14:37
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

❌ Patch coverage is 7.69231% with 24 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ckages/cli/src/modules/chat-hub/stream-capturer.ts 5.88% 16 Missing ⚠️
...kages/cli/src/modules/chat-hub/chat-hub.service.ts 11.11% 8 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant