Skip to content

Fix chat image attachments for Gemini and Codex#95

Merged
Zhang-Henry merged 1 commit intomainfrom
fix/chat-image-attachments-multimodal
Mar 26, 2026
Merged

Fix chat image attachments for Gemini and Codex#95
Zhang-Henry merged 1 commit intomainfrom
fix/chat-image-attachments-multimodal

Conversation

@bbsngg
Copy link
Copy Markdown
Contributor

@bbsngg bbsngg commented Mar 25, 2026

Summary

  • forward uploaded chat images to Codex and send them to the Codex SDK as structured local_image inputs
  • switch Gemini chat attachments from plain file-path notes to @path references so Gemini CLI reads them as multimodal inputs
  • harden temporary attachment filenames and add Windows-focused path/file-name tests for shared image handling

Testing

  • npm run typecheck
  • node --test test/image-attachment-files.test.mjs
  • node --check server/claude-sdk.js
  • node --check server/gemini-cli.js
  • node --check server/openai-codex.js

@bbsngg bbsngg requested review from Zhang-Henry and davidliuk March 25, 2026 02:02
@Zhang-Henry Zhang-Henry force-pushed the fix/chat-image-attachments-multimodal branch from 2ea0424 to aa94765 Compare March 26, 2026 06:57
@Zhang-Henry Zhang-Henry merged commit 8b4e08e into main Mar 26, 2026
1 check passed
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