Skip to content

codex: use store history for core review forks#20577

Draft
wiltzius-openai wants to merge 1 commit intowiltzius/codex/thread-store-app-server-migrationfrom
wiltzius/codex/thread-store-migration-stack
Draft

codex: use store history for core review forks#20577
wiltzius-openai wants to merge 1 commit intowiltzius/codex/thread-store-app-server-migrationfrom
wiltzius/codex/thread-store-migration-stack

Conversation

@wiltzius-openai
Copy link
Copy Markdown
Contributor

Summary

  • fork loaded parent threads from ThreadStore history in agent control paths
  • migrate guardian review fork history to loaded session ThreadStore history
  • keep explicit rollout-path compatibility fallbacks outside the loaded-thread path

Tests

  • cargo test -p codex-core guardian_review_request_layout_matches_model_visible_request_snapshot
  • cargo test -p codex-core spawn_agent_fork
  • just fmt
  • just fix -p codex-core

Bojun-Vvibe added a commit to Bojun-Vvibe/oss-contributions that referenced this pull request May 1, 2026
- anomalyco/opencode#25255: doom loop scope+filter-order fix (merge-as-is)
- openai/codex#20577: store-history fork migration (merge-after-nits)
- openai/codex#20528: skill-scoped hooks feature (merge-after-nits)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant