Skip to content

fix(messaging): pass through PDF attachments#691

Open
huntharo wants to merge 3 commits into
mainfrom
codex/messaging-pdf-pass-through
Open

fix(messaging): pass through PDF attachments#691
huntharo wants to merge 3 commits into
mainfrom
codex/messaging-pdf-pass-through

Conversation

@huntharo

@huntharo huntharo commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pass PDF attachments from messaging platforms through as bounded file inputs instead of extracting text from them.
  • Preserve Codex compatibility by writing file inputs to a local temp path and sending Codex a file reference/mention rather than an unsupported raw file input variant.
  • Render file parts in desktop transcripts and carry PDF/file metadata through the agent-core message builder.

Replaces closed PR #386.

Verification

  • pnpm test apps/desktop/src/main/tests/codex-client.test.ts apps/desktop/src/main/tests/messaging-attachment-processor.test.ts apps/desktop/src/main/tests/messaging-controller.test.ts
  • pnpm --filter @pwragent/desktop typecheck
  • pnpm lint:boundaries

Notes

@huntharo
huntharo force-pushed the codex/messaging-pdf-pass-through branch from 1320330 to 656e10c Compare June 10, 2026 19:56
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