Skip to content

Preserve thread metadata when rollout files are missing - #35644

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/51c03a78434b0aeeb5a2dc685b97243b6ad958bf
Jul 27, 2026
Merged

Preserve thread metadata when rollout files are missing#35644
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/51c03a78434b0aeeb5a2dc685b97243b6ad958bf

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 27, 2026

Copy link
Copy Markdown

Preserve thread metadata when rollout files are missing

What changed

  • Skip threads whose rollout files cannot be found while continuing through
    database pages until the requested number of valid threads is collected.
  • Retain the skipped threads and their rollout paths in the state database
    instead of deleting their metadata during listing.

Testing

  • Update the state-backed listing test to verify that a missing rollout path
    does not consume a page slot or remove the stored thread metadata.

## What changed

- Skip threads whose rollout files cannot be found while continuing through
  database pages until the requested number of valid threads is collected.
- Retain the skipped threads and their rollout paths in the state database
  instead of deleting their metadata during listing.

## Testing

- Update the state-backed listing test to verify that a missing rollout path
  does not consume a page slot or remove the stored thread metadata.

GitOrigin-RevId: 51c03a78434b0aeeb5a2dc685b97243b6ad958bf
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/51c03a78434b0aeeb5a2dc685b97243b6ad958bf branch from d44210d to e8f0f64 Compare July 27, 2026 16:55
@copyberry
copyberry Bot merged commit e8f0f64 into main Jul 27, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/51c03a78434b0aeeb5a2dc685b97243b6ad958bf branch July 27, 2026 16:56
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 27, 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.

6 participants