Skip to content

Add trace_id to TurnStartedEvent#23980

Merged
mchen-oai merged 2 commits into
mainfrom
mchen/TurnContextItem-trace_id
May 22, 2026
Merged

Add trace_id to TurnStartedEvent#23980
mchen-oai merged 2 commits into
mainfrom
mchen/TurnContextItem-trace_id

Conversation

@mchen-oai
Copy link
Copy Markdown
Contributor

@mchen-oai mchen-oai commented May 22, 2026

Why

Recent PR removed trace_id from TurnContextItem.

What changed

  • Add to TurnStartedEvent so rollout consumers can correlate turns with telemetry traces.
  • Note that the branch name is out of date because I originally re-added to TurnContextItem, but we decided to move it to TurnStartedEvent.

Verification

  • cargo test -p codex-protocol
  • cargo test -p codex-core --lib regular_turn_emits_turn_started_without_waiting_for_startup_prewarm
  • cargo test -p codex-core --test all emits_warning_when_resumed_model_differs
  • cargo test -p codex-rollout
  • cargo test -p codex-state

@mchen-oai mchen-oai force-pushed the mchen/TurnContextItem-trace_id branch from c0d9578 to 07d7be3 Compare May 22, 2026 02:37
@mchen-oai mchen-oai changed the title Add back trace_id to TurnContextItem Re-add trace_id to TurnContextItem May 22, 2026
@mchen-oai mchen-oai requested review from pakrym-oai and sayan-oai May 22, 2026 02:57
@sayan-oai
Copy link
Copy Markdown
Collaborator

@pakrym-oai removed, will let him stamp

@mchen-oai mchen-oai force-pushed the mchen/TurnContextItem-trace_id branch from 07d7be3 to 688609f Compare May 22, 2026 03:13
@mchen-oai mchen-oai marked this pull request as ready for review May 22, 2026 03:14
@mchen-oai mchen-oai requested a review from a team as a code owner May 22, 2026 03:14
@mchen-oai mchen-oai force-pushed the mchen/TurnContextItem-trace_id branch from 688609f to 9968b1f Compare May 22, 2026 13:51
@mchen-oai mchen-oai changed the title Re-add trace_id to TurnContextItem Add trace_id to TurnStartedEvent May 22, 2026
@mchen-oai mchen-oai force-pushed the mchen/TurnContextItem-trace_id branch from b10873d to 2399586 Compare May 22, 2026 19:46
@mchen-oai mchen-oai merged commit 3c83e57 into main May 22, 2026
31 checks passed
@mchen-oai mchen-oai deleted the mchen/TurnContextItem-trace_id branch May 22, 2026 20:10
@github-actions github-actions Bot locked and limited conversation to collaborators May 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants