Release v1.16.1 — Fix 6 Responses API schema bugs + debug logging#145
Merged
Release v1.16.1 — Fix 6 Responses API schema bugs + debug logging#145
Conversation
commit: |
jpr5
changed the title
jpr5
force-pushed
the
blitz/d5-subagents-fix/integration
branch
from
8882b37 to
dc6cf30
Compare
- item_reference dropped: synthesize assistant message for orphaned function_call_output - annotations missing: add annotations: [] to all output_text content items - item_id missing on reasoning_summary_part.added, .done, and reasoning_summary_text.done - web_search_call action missing type: "search" in streaming events and output prefix - item_reference for assistant text messages not counted in assistantCount - multi-fco assistantCount inflation: backward scan to append to existing assistant messages
27 new tests covering all 6 schema conformance fixes plus e2e integration tests for turnIndex, item_reference handling, and multi-fco scenarios.
Add logger.debug at fixture match point in all 12 remaining handlers (search.ts, rerank.ts, moderation.ts already had it). Logs fixture matched/no-match for debugging fixture resolution issues.
jpr5
force-pushed
the
blitz/d5-subagents-fix/integration
branch
from
dc6cf30 to
6c5d173
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Release v1.16.1 — fixes 6 schema conformance bugs in the Responses API and adds debug logging across all LLM handlers.
Fixed
responsesInputToMessages()now synthesizes an assistant message with a matchingfunction_callwhen afunction_call_outputhas no prior matching call, preventing item_reference loss (turnIndex stuck at 0)annotations: []to all fouroutput_textcontent items for schema conformanceitem_idtoreasoning_summary_part.added,.done, andreasoning_summary_text.doneeventsaction: { query }toaction: { type: "search", query }in both streaming events and output prefixAdded
logger.debug("Fixture matched/No fixture matched")) across all 14 LLM handler filesTest plan
npx vitest run— 2602/2602 pass