Skip to content

fix: include attachment text in chat payload#2114

Open
embire2 wants to merge 1 commit intostackblitz-labs:mainfrom
embire2:feat/document-uploads
Open

fix: include attachment text in chat payload#2114
embire2 wants to merge 1 commit intostackblitz-labs:mainfrom
embire2:feat/document-uploads

Conversation

@embire2
Copy link

@embire2 embire2 commented Feb 6, 2026

Summary

  • extract text and PDF attachments and include them in a fenced ATTACHMENTS block in the user message payload
  • keep attachment text aligned with uploads and screenshots
  • enforce size limits and truncate long attachment text
  • add pdfjs-dist for client-side PDF text extraction

Testing

  • pnpm run typecheck
  • pnpm run lint
  • e2e: Playwright smoke (drag/drop a text attachment, send message, assert /api/chat payload includes ATTACHMENTS, filename, and content)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants