Skip to content

[codex-analytics] emit forked thread id on initialization#26248

Merged
kbazzi merged 4 commits into
mainfrom
kbazzi/codex-forked-from-thread-id-harness
Jun 4, 2026
Merged

[codex-analytics] emit forked thread id on initialization#26248
kbazzi merged 4 commits into
mainfrom
kbazzi/codex-forked-from-thread-id-harness

Conversation

@kbazzi
Copy link
Copy Markdown
Contributor

@kbazzi kbazzi commented Jun 3, 2026

Why

  • Thread initialization analytics do not identify the source thread for forked threads.
  • The session viewer needs this lineage to construct thread trees.
  • Depends on openai/openai#987854. Do not release this change before that backend schema change is deployed.

What Changed

  • Adds optional forked_from_thread_id to codex_thread_initialized.
  • Populates it from the existing thread fork lineage for app-server and in-process subagent initialization paths.
  • Keeps it null for non-forked threads.

Verification

  • just fmt
  • just test -p codex-analytics
  • just test -p codex-app-server thread_fork_tracks_thread_initialized_analytics

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

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

@kbazzi
Copy link
Copy Markdown
Contributor Author

kbazzi commented Jun 3, 2026

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

github-actions Bot added a commit that referenced this pull request Jun 3, 2026
@kbazzi kbazzi marked this pull request as ready for review June 3, 2026 20:35
@kbazzi kbazzi requested a review from a team as a code owner June 3, 2026 20:35
Copy link
Copy Markdown
Collaborator

@owenlin0 owenlin0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@kbazzi kbazzi merged commit 9e41f8d into main Jun 4, 2026
31 checks passed
@kbazzi kbazzi deleted the kbazzi/codex-forked-from-thread-id-harness branch June 4, 2026 18:24
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 4, 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