feat(codex): port native goals to orchestration V2 - #1
Draft
cjavad wants to merge 2 commits into
Draft
Conversation
Thread transfer impact
This comment will update automatically after the next completed run. |
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.
Codex can start a native goal continuation after a turn completes, but orchestration V2 treats the unknown turn as a subagent and loses its output. This ports pingdotgg#7935 to V2, attaches native continuation output to T3 runs, and adds native
/goalcontrols and live status to web/desktop and mobile.Native notifications are buffered without blocking the protocol reader while T3 attaches a run. Resumed threads retain continuation ownership; fast successive turns and user messages racing adoption retain their output.
/goal resumecan reconnect a dormant session. Goal mutations serialize across clients, subscriptions reconnect, and Stop checks native goal status before pausing/interruption. Archive/delete discards unattached native work. Codex remains the goal-state authority. Stop follows the intent of pingdotgg#10447.Validation: the PR's adapter/session-manager suites pass 94 tests and its server typecheck passes. The combined build includes the existing fork, migration, scheduler and runtime patches: 145 focused tests pass, server/web/mobile typechecks pass, production server/web builds pass, and scoped lint has zero errors (42 warnings). It starts against a snapshot of the live 5 GB database and serves its HTML and JavaScript successfully. No live-provider/browser/mobile-device workflow has been exercised; UI evidence remains to attach before upstream review.
Prepared for local use:
t3code-branch:goals-ready-20260909, image363861031077. Reproducible combined source:build/codex-goals-ready-20260909atb42a46ef5. The goal changes are saved in the local patches system. The running container andlatestimage tag were not changed.Based on upstream pingdotgg#7935 at
3e2f3794aee2f61a2333d928d97bb57c566745d8. This PR targetst3code/codex-turn-mapping; the combined build branch retains the newer prepared runtime and all existing fixes.Model/harness: GPT-6 / Codex.