Skip to content

fix: backfill tool call data in chat stream conversion#2306

Open
MiaoZang wants to merge 1 commit intoWei-Shaw:mainfrom
MiaoZang:upstream/responses-chat-tool-call-backfill
Open

fix: backfill tool call data in chat stream conversion#2306
MiaoZang wants to merge 1 commit intoWei-Shaw:mainfrom
MiaoZang:upstream/responses-chat-tool-call-backfill

Conversation

@MiaoZang
Copy link
Copy Markdown

@MiaoZang MiaoZang commented May 8, 2026

Summary

  • Backfill function name and arguments when Responses streams emit them in response.function_call_arguments.done or response.output_item.done.
  • Track whether a tool call name/arguments have already been sent to avoid duplicate deltas.

Verification

  • go test ./internal/pkg/apicompat -run 'TestResponsesEventToChatChunks_ToolCall' -count=1
  • git diff --check

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

All contributors have signed the CLA. ✅
Posted by the CLA Assistant Lite bot.

@MiaoZang
Copy link
Copy Markdown
Author

MiaoZang commented May 9, 2026

I have read the CLA Document and I hereby sign the CLA

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