Skip to content

Show Codex MCP screenshots in desktop and web chat - #1134

Open
PengXiaoyi wants to merge 1 commit into
spacering-net:mainfrom
PengXiaoyi:fix/codex-mcp-screenshot-preview
Open

PengXiaoyi wants to merge 1 commit into
spacering-net:mainfrom
PengXiaoyi:fix/codex-mcp-screenshot-preview

Conversation

@PengXiaoyi

Copy link
Copy Markdown

What changed

  • Forward image blocks from Codex ACP's MCP tool results through Codeg's existing image preview field.
  • Read inline images from the original Codex function output when restoring history. The persisted tool event can contain a truncated text preview instead of the image.

Validation

  • Installed the patched macOS app locally and opened its Web Service. A real Computer Use screenshot of Calculator appeared in both clients.
  • Refreshed Web and reopened the execution group. The screenshot was still available.
  • Imported the Chrome/Baidu conversation that reproduced [Bug] Missing screenshot previews for Codex Computer Use MCP results #1129 and opened the original screenshot in both desktop and Web.
  • Passed the Rust regression tests for live tool updates and history, the macOS build, the Web build, and preview tests.

Refs #1129

Change-Id: I5d1b10ebe99571ec0fc07638a8d2b3376671d972
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