Skip to content
This repository was archived by the owner on Oct 1, 2026. It is now read-only.

feat: remove the manual-only Codex dispatch route and resume on the active dispatch route (#62) - #63

Merged
lukemaj merged 1 commit into
mainfrom
feat/terra-62
Sep 23, 2026
Merged

lukemaj merged 1 commit into
mainfrom
feat/terra-62

Conversation

@lukemaj

@lukemaj lukemaj commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Closes #62. Parent #10.

Change

  • Policy 2.6.0 removes the manual-only Codex dispatch route and its stage entry; RUNNER.md, the generated skill reference and tests no longer name it (git grep -i terra is empty).
  • Dispatch persists the Codex route it ran on; resume_luna builds the resume command and records meta.route from that route instead of the first dispatch route.
  • The resume test dispatches through a fake Codex on a second Codex dispatch route, checks the persisted route, restores the stage order and resumes. Removing either the persistence or the lookup fails it (verified by mutation).
  • One version transition: 0.25.1 to 0.26.0.

Proof (exact candidate 72c5b75)

  • python3 scripts/test.py: Ran 516 tests, OK (skipped=1)
  • python3 -m compileall -q runner scripts tests: rc 0
  • python3 -m runner.policy validate: rc 0
  • versionctl release-check: release candidate OK v0.26.0

Review

Independent review by Luna max on Codex (read-only): first pass CHANGES_REQUESTED with one P2 (the resume test seeded the route by hand); fixed; delta review on 72c5b75: APPROVE, no findings.

Taken over from the earlier stale branch under the user's end-to-end authority (2026-09-23) and rebased onto current main.

🤖 Generated with Claude Code

@lukemaj

lukemaj commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

Independent proof (planner, exact SHA e24c990):

  • python3 -m runner.policy validate: exit 0
  • python3 -m unittest discover -s tests: Ran 485 tests in 300.575s, OK
  • git grep -i terra: no matches

Review: logic correct. Codex dispatch now saves luna/max as the job's dispatch route; that route has no concurrency cap, so nothing else changes. Merge awaits human approval.

…ctive dispatch route (#62)

Policy 2.6.0 drops the manual-only dispatch route, its stage entry, and
its mentions in RUNNER.md, the generated skill reference, and tests.
Dispatch persists the Codex route it ran on, and resume builds its command
and records meta.route from that route instead of the first dispatch
route. The resume test dispatches through a fake Codex on a second Codex
route, so removing either the persistence or the lookup fails it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@lukemaj lukemaj changed the title feat: resume on the active dispatch route feat: remove the manual-only Codex dispatch route and resume on the active dispatch route (#62) Sep 23, 2026
@lukemaj
lukemaj merged commit 3a32e84 into main Sep 23, 2026
1 check passed
@lukemaj
lukemaj deleted the feat/terra-62 branch September 23, 2026 17:16
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.

Remove Terra from the policy; Codex resume uses the job's active dispatch route

1 participant