Merged
Conversation
…cking - stream.go: extract id/model from nested response object in response.created/in_progress events (was reading wrong top-level key) - stream.go: capture call_id/name from response.output_item.added so first tool_calls delta includes proper id and function.name header - stream.go: emit full tool call header (id, type, function.name, first args) on first delta; subsequent deltas carry args only - server.go: sseUsageExtractor.processChunk now handles Responses API response.completed events and OpenAI Chat final chunks (no "type" field, usage at top level with prompt_tokens/completion_tokens) - tests: assert id and model fields in streaming chunks; add response.output_item.added to tool call test; cover Responses API and OpenAI Chat usage extraction in sseUsageExtractor tests Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…i-chat stream Both response.created and response.in_progress events triggered the initial role delta emit. Add roleSent guard so only the first of these events emits the chunk; subsequent ones still update id/model. Add TestTransformResponsesAPIToOpenAIChat_NoDuplicateRoleChunk to verify only one role:assistant chunk appears when both events arrive. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…i-chat stream Both response.created and response.in_progress events triggered the initial role delta emit. Add roleSent guard so only the first of these events emits the chunk; subsequent ones still update id/model. Add TestTransformResponsesAPIToOpenAIChat_NoDuplicateRoleChunk to verify only one role:assistant chunk appears when both events arrive. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
No description provided.