Skip to content

Add turn_id to Codex skill invocation analytics#21122

Merged
edwardysun3 merged 2 commits intomainfrom
codex/skill-invocation-turn-id
May 5, 2026
Merged

Add turn_id to Codex skill invocation analytics#21122
edwardysun3 merged 2 commits intomainfrom
codex/skill-invocation-turn-id

Conversation

@edwardysun3
Copy link
Copy Markdown
Contributor

Summary

  • add turn_id to the Codex skill invocation analytics payload
  • populate it from the existing turn-level TrackEventsContext
  • update the reducer test expectation so serialized skill_invocation events include the turn id

Why

chatgpt_skill_resource_invocation / fact_skill_product_events already have backend/schema support for turn_id, but Codex skill invocation events were only sending thread_id. This caused Codex Desktop skill invocation rows to keep turn_id null and prevented reliable turn-level joins with artifact/file events.

Validation

  • cargo test -p codex-analytics reducer_ingests_skill_invoked_fact

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

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

@edwardysun3 edwardysun3 marked this pull request as ready for review May 5, 2026 01:43
@edwardysun3
Copy link
Copy Markdown
Contributor Author

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

github-actions Bot added a commit that referenced this pull request May 5, 2026
@edwardysun3 edwardysun3 merged commit 7e71d02 into main May 5, 2026
26 checks passed
@edwardysun3 edwardysun3 deleted the codex/skill-invocation-turn-id branch May 5, 2026 04:11
@github-actions github-actions Bot locked and limited conversation to collaborators May 5, 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