Skip to content

fix: convert Read tool image content to ACP format (#322)#323

Merged
benbrandt merged 1 commit intozed-industries:mainfrom
blimmer:fix-322
Feb 18, 2026
Merged

fix: convert Read tool image content to ACP format (#322)#323
benbrandt merged 1 commit intozed-industries:mainfrom
blimmer:fix-322

Conversation

@blimmer
Copy link
Contributor

@blimmer blimmer commented Feb 17, 2026

When the Read tool returns image content (e.g., reading a .png file), the raw Claude SDK image block was passed through unconverted, causing ACP clients to fail deserializing the tool call completion event. Route non-text content through toAcpContentBlock to produce the correct ACP image format.

@cla-bot cla-bot bot added the cla-signed label Feb 17, 2026
When the Read tool returns image content (e.g., reading a .png file),
the raw Claude SDK image block was passed through unconverted, causing
ACP clients to fail deserializing the tool call completion event. Route
non-text content through `toAcpContentBlock` to produce the correct ACP
image format.
Copy link
Member

@benbrandt benbrandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@benbrandt benbrandt merged commit 28b85ae into zed-industries:main Feb 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

toolUpdateFromToolResult sends Claude SDK image format instead of ACP format

2 participants