migrate-coordinator-agent

Metadata

Description

## Description

Migrate src/commands/service/coordinator_agent.rs (chat-agent supervisor's per-iteration spawn at lines ~319 and ~507) to call workgraph::dispatch::plan_spawn() for executor/model resolution. Replace the requires_native_executor calls with plan_spawn.

File scope: src/commands/service/coordinator_agent.rs only.

Implement directly — do not decompose further.

## Validation
- [ ] Both call sites at coordinator_agent.rs:319 and :507 use plan_spawn
- [ ] requires_native_executor calls in coordinator_agent.rs removed (function may stay defined in coordinator.rs as long as no other site calls it)
- [ ] Provenance log line emitted per supervisor iteration spawn
- [ ] cargo build + cargo test pass

Depends on

• done .assign-migrate-coordinator-agent

Required by

• (none)

Log

• 2026-04-26T20:41:00.025459475+00:00 Lightweight assignment: agent=Careful Programmer (f5143935), exec_mode=full, context_scope=task, reason=Careful Programmer matches implementation needs; Careful tradeoff ensures exhaustive call-site updates and proper validation; proven track record (41 tasks) suitable for focused refactoring.
• 2026-04-26T20:41:00.214838742+00:00 Spawned by coordinator --executor claude --model opus
• 2026-04-26T20:41:10.439725983+00:00 Starting: read plan_spawn signature + coordinator.rs reference site, then locate the two call sites in coordinator_agent.rs
• 2026-04-26T20:43:27.953421160+00:00 Plan: site 1 (CoordinatorAgent::spawn) - replace requires_native_executor with plan_spawn to compute uses_subprocess. Site 2 (agent_thread_main) - migrate the per-iteration spawn in subprocess_coordinator_loop to call plan_spawn for executor/model resolution + provenance log line. Drop requires_native_executor calls + force_native logic in agent_thread_main.
• 2026-04-26T21:03:25.459971500+00:00 Validated: cargo build + cargo test --lib (1956 pass) + cargo test --bin wg (2818 pass; 1 pre-existing provenance_full_lifecycle failure unchanged from baseline). 12 coordinator_agent tests + 4 requires_native_executor tests all pass. Both call sites at lines ~319 (CoordinatorAgent::spawn) and ~507 (agent_thread_main) replaced with plan_spawn. Per-iteration provenance log line emitted from subprocess_coordinator_loop.
• 2026-04-26T21:03:51.815465890+00:00 Committed: 1a7e2b1de — pushed to remote
• 2026-04-26T21:03:56.786064607+00:00 Task marked as done