Skip to content

Preserve code cell response metadata in rollout traces - #55725

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8f49a3174ea487a5d91ebc2a6c97133aacaabfd2
Oct 21, 2026
Merged

Preserve code cell response metadata in rollout traces#55725
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8f49a3174ea487a5d91ebc2a6c97133aacaabfd2

Conversation

@copyberry

@copyberry copyberry Bot commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown

Preserve code cell response metadata in rollout traces

Why

Reducing code cell lifecycle events discarded their response payload references and lost the initial response status when the runtime status changed.

What changed

Store the initial response status and separate raw payload IDs for the initial response and final result on CodeCell. Carry payload references through queued lifecycle events so they survive replay when the source item arrives later.

Testing

Extend lifecycle tests to verify distinct initial and final payload IDs and the preserved initial status for yielded cells that complete and failed cells whose lifecycle events precede their source item.

@cassirer-openai copyberry
Preserve code cell response metadata in rollout traces (#55725)
d571eaf 
## Why

Reducing code cell lifecycle events discarded their response payload references and lost the initial response status when the runtime status changed.

## What changed

Store the initial response status and separate raw payload IDs for the initial response and final result on `CodeCell`. Carry payload references through queued lifecycle events so they survive replay when the source item arrives later.

## Testing

Extend lifecycle tests to verify distinct initial and final payload IDs and the preserved initial status for yielded cells that complete and failed cells whose lifecycle events precede their source item.

GitOrigin-RevId: 8f49a3174ea487a5d91ebc2a6c97133aacaabfd2
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/8f49a3174ea487a5d91ebc2a6c97133aacaabfd2 branch from 8302ba4 to d571eaf Compare October 21, 2026 02:02
@copyberry
copyberry Bot merged commit d571eaf into main Oct 21, 2026
1 check passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-21T02:03:02.059896Z 8302ba4 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/8f49a3174ea487a5d91ebc2a6c97133aacaabfd2 branch October 21, 2026 02:03
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 21, 2026

This branch was previously deployed

1 inactive deployment
issue-triage — d571eaf1 Deployed Oct 21, 2026 by twilight-sparkle-crazy-fan via Identify potential duplicates (open issues fallback) #57396
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@twilight-sparkle-crazy-fan @1202kong @yannan-cr @MMarinhoRM @ingChristopherCV @Mako-2121 @cassirer-openai