Skip to content

feat(chat-sessions): show user attachments and preview PDFs - #76

Merged
squarezw merged 3 commits into
mainfrom
feat/chat-session-attachments
Sep 24, 2026
Merged

squarezw merged 3 commits into
mainfrom
feat/chat-session-attachments

Conversation

@squarezw

Copy link
Copy Markdown
Owner

Summary

  • Session details and reopened chats now show the files a user uploaded with that turn.
  • PDF preview treats stored display labels such as PDF as the real MIME type from the file extension.
  • Attachment preview streams the object inline, so the browser shows the PDF instead of downloading it.

Test plan

  • Open a historical session that has a PDF attachment and confirm the file name appears under the question.
  • Click the PDF and confirm the preview dialog renders it, rather than "this file type is not supported".
  • Reopen the same session from chat history and confirm the attachment is attached to the user message.
  • Download an OSS file without inline=1 and confirm it still redirects to the signed URL.

Made with Cursor

Jesse and others added 3 commits September 24, 2026 16:43
Session history only stored question and answer, so admins could not see the files a user attached. Read the persisted metadata and render it under the question, including when reopening a chat.

Co-authored-by: Cursor <cursoragent@cursor.com>
Session rows keep the bubble label "PDF" instead of a MIME type, and the signed object URL forces a download. Recognize the label from the extension and stream the file inline.

Co-authored-by: Cursor <cursoragent@cursor.com>
The attachment list already landed in #75. Keep the PDF preview fix on top of that.

Co-authored-by: Cursor <cursoragent@cursor.com>
@squarezw
squarezw merged commit 6150ae5 into main Sep 24, 2026
1 check passed
@squarezw
squarezw deleted the feat/chat-session-attachments branch September 24, 2026 11:22
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.

1 participant