Skip to content

Summarize streamed response item logs - #31792

Merged
jif-oai merged 1 commit into
mainfrom
jif/summarize-stream-event-logs
Jul 9, 2026
Merged

Summarize streamed response item logs#31792
jif-oai merged 1 commit into
mainfrom
jif/summarize-stream-event-logs

Conversation

@jif-oai

@jif-oai jif-oai commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Why

handle_non_tool_response_item logged the complete decoded ResponseItem at DEBUG. Those values can contain assistant text, reasoning content, and tool payloads that are already stored in the durable rollout, making the SQLite copy both redundant and potentially large or sensitive.

What changed

  • Replace the complete item dump with bounded item_type and item_id fields.
  • Keep an event-flow breadcrumb for debugging without duplicating item content.
  • Leave rollout persistence and tool-call telemetry unchanged.

The item-type match is exhaustive so new response item variants must choose an explicit log label.

Related to #28224.

@jif-oai
jif-oai requested a review from a team as a code owner July 9, 2026 13:11
@jif-oai
jif-oai merged commit 538f29e into main Jul 9, 2026
35 checks passed
@jif-oai
jif-oai deleted the jif/summarize-stream-event-logs branch July 9, 2026 13:35
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants