Skip to content

docs(readme): shorten 2026-05-16 image-handling-parity news item#158

Merged
ericleepi314 merged 1 commit into
mainfrom
docs/readme-shorten-image-parity-news
May 16, 2026
Merged

docs(readme): shorten 2026-05-16 image-handling-parity news item#158
ericleepi314 merged 1 commit into
mainfrom
docs/readme-shorten-image-parity-news

Conversation

@ericleepi314
Copy link
Copy Markdown
Collaborator

Summary

The 2026-05-16 image-handling-parity news entry I landed in #157 was a paragraph long; every other News bullet is one or two sentences. Trimming it to match the house style: four facts joined by semicolons, no inline rationale.

Before

Read tool ported the TS image pipeline (magic-byte format sniff, bounded byte read, resize/compress to the 3.75 MB / 1568 px envelope, base64 size validation); `@image.png` @-mentions now inline as real `ImageBlock` content (the prior text-mode read shipped mojibake the model latched onto); image `tool_result` list shape preserved end-to-end through `_dispatch_single_tool`; OpenAI-compat / GLM / Minimax / DeepSeek / OpenRouter now translate Anthropic `image` and `document` blocks into `image_url` / `file` shapes (with `tool_use_id` correlation across the tool→user split); `validate_images_for_api` promoted into `BaseProvider._prepare_messages` so every provider validates client-side; BOM-aware text decoding (`utf-16` / `utf-32` / `utf-8-sig`) handles Windows-emitted Unicode files without mojibake.

After

Read tool TS image pipeline (sniff, resize/compress, base64 cap); `@image.png` mentions inline as real `ImageBlock`s instead of mojibake; cross-provider Anthropic `image`/`document` → OpenAI `image_url`/`file` translation; pre-API base64 size validation in `BaseProvider`.

The detailed write-up still lives in the PR descriptions for #149 / #154 / #155 / #156 for anyone who wants it.

Test plan

  • Diff is README-only (+1/-1).

🤖 Generated with Claude Code

Bring it in line with the length of the surrounding entries: four
facts joined by semicolons, no inline rationale or low-level
implementation details. The longer write-up lives in the PR
descriptions for #149/#154/#155/#156.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@ericleepi314 ericleepi314 merged commit e1dbe7d into main May 16, 2026
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