Skip to content

feat: qualify native Claude function image results - #16

Merged
SaladDay merged 1 commit into
mainfrom
codex/function-result-images
Sep 22, 2026
Merged

SaladDay merged 1 commit into
mainfrom
codex/function-result-images

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Claude previously rejected every public function image result. This adds successful inline PNG/JPEG results for environment:none, preserving ordered caller content while accepting native image resizing. Failed images, workspace images and remote references remain rejected before batch persistence. Core reuses the common result contract and checks Runtime image support only for image-bearing delivery.

Native receipt validation retains Session/call ownership, success, text, block order/count and image presence. Coverage documents the distinct Codex transport acknowledgement limitation without changing its lifecycle.

Validation:

  • Real Kimi API through independent Core, PostgreSQL and Docker Runtime for Codex and Claude: fixed SDK/raw HTTP, PNG/large PNG/JPEG, failed text, retries/conflicts, cancellation, cold continuation and tenant isolation.
  • All make check components passed (server Core/runtime/database gates and local Core Web gate, including 73 Playwright cases); packaging expectation corrected and failed/remaining gates rerun.
  • Targeted Go race tests, generated OpenAPI and fresh independent GPT-6 Astra high review passed.

This qualifies the documented subset, not complete protocol compatibility.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

@SaladDay
SaladDay merged commit fd23f16 into main Sep 22, 2026
2 of 3 checks 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.

1 participant