fix-external-worker-compat-test — WorksGood live mirror

Metadata

Status	done
Assigned	`agent-903`
Created	2026-07-26T18:29:23.955790619+00:00
Started	2026-07-26T21:40:03.047390343+00:00
Completed	2026-07-26T22:02:28.066532911+00:00
Tokens	1948333 in / 7697 out

Description

cargo test on current main deterministically fails dispatch::plan::tests::test_agency_external_executor_survives_plan_spawn: it feeds every external handler the CLI-locked claude:opus route, but current handler-first model compatibility correctly reroutes Pi to Claude (and would reject other locked mismatches). Update the test matrix to use handler-compatible exact routes while retaining agency-executor precedence. Implement directly — do not decompose further.

Validation

Failing test reproduced on current main
Updated matrix covers Pi, Claude, Codex, and remaining external handlers with compatible routes
cargo test --lib dispatch::plan::tests::test_agency_external_executor_survives_plan_spawn -- --exact passes
cargo test --lib passes without this failure

## Description
`cargo test` on current main deterministically fails `dispatch::plan::tests::test_agency_external_executor_survives_plan_spawn`: it feeds every external handler the CLI-locked `claude:opus` route, but current handler-first model compatibility correctly reroutes Pi to Claude (and would reject other locked mismatches). Update the test matrix to use handler-compatible exact routes while retaining agency-executor precedence. Implement directly — do not decompose further.

## Validation
- [ ] Failing test reproduced on current main
- [ ] Updated matrix covers Pi, Claude, Codex, and remaining external handlers with compatible routes
- [ ] `cargo test --lib dispatch::plan::tests::test_agency_external_executor_survives_plan_spawn -- --exact` passes
- [ ] `cargo test --lib` passes without this failure

Depends on

done verify-spawn-recovery

Required by

(none)

Log

2026-07-26T18:29:23.937214555+00:00 Task paused
2026-07-26T18:29:24.009903737+00:00 Task published
2026-07-26T21:01:07.923660554+00:00 Spawn preparation deferred before launch permit; rollback is complete and no circuit-breaker charge was recorded. Repair the reported checkout/configuration condition and retry: spawn transaction for agent-901 rolled back (task remains dispatchable; rollback diagnostics: complete): failed to establish isolated-worktree observer baseline: observer state already exists; use open_at for restart reconciliation
2026-07-26T21:40:03.219171121+00:00 Spawned by coordinator --executor pi --model openai-codex:gpt-5.6-sol --isolation required-worktree
2026-07-26T21:40:19.215987887+00:00 Starting direct implementation; reproducing stale exact unit test and inspecting handler-compatible model patterns.
2026-07-26T21:51:35.048706583+00:00 Reproduced exact main failure: Pi + claude:opus is compatibility-rerouted to Claude, defeating the stale expected Pi assertion. Inspecting external executor variants and compatible exact-route conventions.
2026-07-26T21:54:28.348719065+00:00 Updated the agency matrix: matching Claude/Codex routes plus flexible exact OpenRouter inner routes for Pi and every registered external CLI; dispatcher default is now native so the provenance assertion still proves agency precedence. Exact regression test passes.
2026-07-26T21:58:26.038918333+00:00 Validated cargo fmt/check, cargo clippy, cargo build, exact regression test, and cargo test --lib -- --test-threads=1 (3008 passed). Default parallel cargo test --lib no longer has the dispatch failure; it exposed the known unrelated profile HOME race (root test passes in the serial full-lib run).
2026-07-26T22:02:15.725075400+00:00 Committed: 455b1762 — pushed to remote
2026-07-26T22:02:28.066533231+00:00 Task marked as done