codex: use ThreadStore history for core review forks#20577
Draft
wiltzius-openai wants to merge 1 commit intowiltzius/codex/thread-store-app-server-migrationfrom
Draft
codex: use ThreadStore history for core review forks#20577wiltzius-openai wants to merge 1 commit intowiltzius/codex/thread-store-app-server-migrationfrom
wiltzius-openai wants to merge 1 commit intowiltzius/codex/thread-store-app-server-migrationfrom
Conversation
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)
5f7faf8 to
1da0d76
Compare
7b8b9ab to
3dda86b
Compare
1da0d76 to
3474484
Compare
3dda86b to
0ab9b96
Compare
3474484 to
1b79226
Compare
0ab9b96 to
34ea251
Compare
1b79226 to
cfa324f
Compare
34ea251 to
a645067
Compare
de41539 to
e18d4cb
Compare
a645067 to
d64ec1e
Compare
e18d4cb to
fbfd2b9
Compare
d64ec1e to
4f8e5ae
Compare
fbfd2b9 to
6b96710
Compare
4f8e5ae to
9baeb0e
Compare
6b96710 to
8427230
Compare
9449f34 to
03da85a
Compare
8427230 to
3243fd6
Compare
03da85a to
a231ef7
Compare
3243fd6 to
3bc194d
Compare
a231ef7 to
ec5e41f
Compare
3bc194d to
96a42fc
Compare
ec5e41f to
c29f912
Compare
96a42fc to
dcb713c
Compare
c29f912 to
03fc03f
Compare
dcb713c to
79c7d80
Compare
03fc03f to
df8dd3a
Compare
79c7d80 to
d9ed02c
Compare
df8dd3a to
d63d2d8
Compare
d9ed02c to
be8ad4e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
After app-server no longer depends on rollout paths for thread-identity history reads, the remaining loaded-thread review/fork paths in core should use the same store-backed history boundary. This keeps direct rollout parsing in local-store compatibility code while allowing remote stores to provide history without filesystem access.
What changed
ThreadStorehistory in core agent control pathsVerification
cargo test -p codex-core spawn_agent_fork