Skip to content

Conversation

@autologie
Copy link
Contributor

Summary

Tested sending many attachment files. Server responded with status 413, which is expected, but the chat UI wasn't able to present any feedback because streamRequest function didn't call onError callback function. Also adding more user-friendly messages.

Related Linear tickets, Github issues, and Community forum posts

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)

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

@bundlemon
Copy link

bundlemon bot commented Dec 5, 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
11.43MB (+146.82KB +1.27%) -
**/*.css
233.4KB (+12.63KB +5.72%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Dec 5, 2025
@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

❌ Patch coverage is 37.50000% with 30 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...nd/editor-ui/src/features/ai/chatHub/chat.store.ts 14.81% 23 Missing ⚠️
...ackages/frontend/@n8n/rest-api-client/src/utils.ts 77.77% 4 Missing ⚠️
...end/editor-ui/src/features/ai/chatHub/ChatView.vue 0.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@currents-bot
Copy link

currents-bot bot commented Dec 5, 2025

E2E Tests: n8n tests passed after 12m 0.5s

🟢 573 · 🔴 0 · ⚪️ 37 · 🟣 1

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: 0d65f4d

  • Spec files: 122

  • Overall tests: 610

  • Duration: 12m 0.5s

  • Parallelization: 9

Groups

GroupId Results Spec Files Progress
multi-main:ui:isolated 🟢 54 · 🔴 0 · ⚪️ 0 8 / 8
multi-main:ui 🟢 519 · 🔴 0 · ⚪️ 37 · 🟣 1 114 / 114


This message was posted automatically by currents.dev | Integration Settings

@autologie autologie requested a review from mutdmour December 5, 2025 16:10
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.

2 participants