Skip to content

fix: Fix Codex PDF attachment handling#97

Merged
bbsngg merged 1 commit intomainfrom
fix/codex-pdf-attachments
Mar 25, 2026
Merged

fix: Fix Codex PDF attachment handling#97
bbsngg merged 1 commit intomainfrom
fix/codex-pdf-attachments

Conversation

@bbsngg
Copy link
Copy Markdown
Contributor

@bbsngg bbsngg commented Mar 25, 2026

Summary

  • route chat attachments through a shared file flow and keep Codex PDF uploads in the workspace
  • send Codex only attachment paths instead of PDF text previews, while still showing uploaded files in the user message UI
  • restore PDF attachment passthrough for Claude and Gemini instead of blocking them

Testing

  • npm run typecheck
  • npm run build

@bbsngg bbsngg changed the title Fix Codex PDF attachment handling fix: Fix Codex PDF attachment handling Mar 25, 2026
@bbsngg bbsngg merged commit 3b45227 into main Mar 25, 2026
1 check passed
@bbsngg bbsngg deleted the fix/codex-pdf-attachments branch March 25, 2026 03:24
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