Skip to content

core: overlap diff root discovery with world state - #30286

Merged
anp-oai merged 1 commit into
mainfrom
codex/overlap-turn-diff-root-discovery
Jun 27, 2026
Merged

anp-oai merged 1 commit into
mainfrom
codex/overlap-turn-diff-root-discovery

Conversation

@anp-oai

@anp-oai anp-oai commented Jun 26, 2026 •

Copy link
Copy Markdown
Contributor

Why

Remote diff-root discovery is independent of world-state construction, but it ran afterward and added filesystem metadata latency before the first model request. Overlap the independent work so thread-cold turns do not pay those waits serially.

What

  • Run record_context_updates_and_set_reference_context_item and turn_diff_display_roots with tokio::join!.
  • Reuse the same resolved display roots when constructing TurnDiffTracker; no cache or behavior lifecycle changes are introduced.

Validation

A synthetic executor-skill benchmark with artificial network delay: thread-cold model-request p50 improved from about 1.79 s to 1.58 s.

@anp-oai
anp-oai marked this pull request as ready for review June 26, 2026 23:07
@anp-oai
anp-oai requested a review from a team as a code owner June 26, 2026 23:07
@anp-oai

anp-oai commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@anp-oai
anp-oai force-pushed the codex/overlap-turn-diff-root-discovery branch from 2176b46 to c7b1ae5 Compare June 26, 2026 23:16
@anp-oai
anp-oai merged commit 3ae0543 into main Jun 27, 2026
62 of 66 checks passed
@anp-oai
anp-oai deleted the codex/overlap-turn-diff-root-discovery branch June 27, 2026 01:07
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 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.

2 participants