Skip to content

fix(openai): don't double marshel responses api#293

Merged
taciturnaxolotl merged 2 commits into
mainfrom
malformed-json-handling
Jun 24, 2026
Merged

fix(openai): don't double marshel responses api#293
taciturnaxolotl merged 2 commits into
mainfrom
malformed-json-handling

Conversation

@taciturnaxolotl

@taciturnaxolotl taciturnaxolotl commented Jun 22, 2026

Copy link
Copy Markdown
Member

the responses api was being double encoded improperly

@taciturnaxolotl

taciturnaxolotl commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

once this merges we need to properly handle CallWarnings in crush

@andreynering

Copy link
Copy Markdown
Member

You'll need to re-record VCRs for this PR. Let me know if you need assistance with it.

Also, make sure to test in Crush to ensure things keep working. Specially tool calls with arguments.

@taciturnaxolotl taciturnaxolotl force-pushed the malformed-json-handling branch from bc37950 to 54e057b Compare June 23, 2026 21:57
@taciturnaxolotl taciturnaxolotl changed the title fix: sanitize malformed tool call input and fix Responses API double-encoding fix(openai): don't double marshel responses api Jun 23, 2026
@taciturnaxolotl taciturnaxolotl force-pushed the malformed-json-handling branch from aaaf795 to bae5145 Compare June 24, 2026 17:53
@taciturnaxolotl taciturnaxolotl force-pushed the malformed-json-handling branch from d9f5aea to f8d8f4c Compare June 24, 2026 18:08
@taciturnaxolotl taciturnaxolotl force-pushed the malformed-json-handling branch from f8d8f4c to 014ee08 Compare June 24, 2026 18:43
@taciturnaxolotl taciturnaxolotl merged commit 4816e03 into main Jun 24, 2026
11 checks passed
@taciturnaxolotl taciturnaxolotl deleted the malformed-json-handling branch June 24, 2026 18:52
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.

2 participants