fix(test): skill-e2e-autoplan-dual-voice was shipped broken

The test shipped on main in v0.18.4.0 used wrong option names and
wrong result fields throughout. It could not have passed in any
environment:

Broken API calls:
- `workdir` → should be `workingDirectory`
  The fixture setup (git init, copy autoplan + plan-*-review dirs,
  write TEST_PLAN.md) was completely ignored. claude -p spawned with
  undefined cwd instead of the tmp workdir.
- `timeoutMs: 300_000` → should be `timeout: 300_000`
  Fell back to default 120s. Explains the observed ~170s failure
  (test harness overhead + retry startup).
- `name: 'autoplan-dual-voice'` → should be `testName: 'autoplan-dual-voice'`
  No per-test run directory was created.
- `evalCollector` → not a recognized `runSkillTest` option at all.

Broken result access:
- `result.stdout + result.stderr` → SkillTestResult has neither
  field. `out` was literally "undefinedundefined" every time.
- Every regex match fired false. All 3 assertions (claudeVoiceFired,
  codex-or-unavailable, reachedPhase1) failed on every attempt.
- `logCost(result)` → signature is `logCost(label, result)`.
- `recordE2E('autoplan-dual-voice', result)` → signature is
  `recordE2E(evalCollector, name, suite, result, extra)`.

Fixes:
- Renamed all 4 broken options in the runSkillTest call.
- Changed assertion source to `result.output` plus JSON-serialized
  `result.transcript` (broader net for voice fingerprints in tool
  inputs/outputs).
- Widened regex alternatives: codex voice now matches "CODEX SAYS"
  and "codex-plan-review"; Claude voice now matches subagent_type;
  unavailable matches CODEX_NOT_AVAILABLE.
- Added Agent + Skill + Edit + Grep + Glob to allowedTools. Without
  Agent, /autoplan can't spawn subagents and never reaches Phase 1.
- Raised maxTurns 15 → 30 (autoplan is a long multi-phase skill).
- Fixed logCost + recordE2E signatures, passing `passed:` flag into
  recordE2E per the neighboring context-save pattern.

test/skill-e2e-autoplan-dual-voice.test.ts

@@ -70,31 +70,41 @@ Add a new /greet skill that prints a welcome message.
       // If Codex is unavailable on the test machine, the skill should print
       // [codex-unavailable] and still complete the Claude subagent half.
       const result = await runSkillTest({
-        name: 'autoplan-dual-voice',
-        workdir: workDir,
+        testName: 'autoplan-dual-voice',
+        workingDirectory: workDir,
         prompt: `/autoplan ${planPath}`,
-        timeoutMs: 300_000, // 5 min
-        evalCollector,
+        timeout: 300_000, // 5 min
+        // /autoplan spawns subagents and calls codex via Bash; it needs the
+        // full tool set to get past Phase 1. Bash+Read+Write alone wasn't
+        // enough — the skill stalled trying to invoke Agent/Skill.
+        allowedTools: ['Bash', 'Read', 'Write', 'Edit', 'Grep', 'Glob', 'Agent', 'Skill'],
+        maxTurns: 30,
+        runId,
       });
 
       // Accept EITHER outcome as success:
       //   (a) Both voices produced output (ideal case)
       //   (b) Codex unavailable + Claude voice produced output (graceful degrade)
-      const out = result.stdout + result.stderr;
-      const claudeVoiceFired = /Claude\s+(CEO|subagent)|claude-subagent/i.test(out);
-      const codexVoiceFired = /codex\s+(exec|review|CEO\s+voice)|\[via:codex\]/i.test(out);
-      const codexUnavailable = /\[codex-unavailable\]|AUTH_FAILED|codex_cli_missing/i.test(out);
+      // Session runner returns `output` (final assistant message). Search
+      // transcript tool-call inputs/outputs as a broader net for voice fingerprints.
+      const transcriptText = JSON.stringify(result.transcript || []);
+      const out = (result.output ?? '') + '\n' + transcriptText;
+      const claudeVoiceFired = /Claude\s+(CEO|subagent)|claude-subagent|Agent\s*\(|subagent_type/i.test(out);
+      const codexVoiceFired = /codex\s+(exec|review)|CODEX SAYS|\[via:codex\]|codex-plan-review/i.test(out);
+      const codexUnavailable = /\[codex-unavailable\]|AUTH_FAILED|CODEX_NOT_AVAILABLE|codex_cli_missing|Codex CLI not found/i.test(out);
 
       expect(claudeVoiceFired).toBe(true);
       expect(codexVoiceFired || codexUnavailable).toBe(true);
 
       // Hang protection: if the skill reached Phase 1 at all, our hardening worked.
       // If it didn't, this is a regression from the pre-wave stdin-deadlock era.
-      const reachedPhase1 = /Phase 1|CEO\s+Review|Strategy\s*&\s*Scope/i.test(out);
+      const reachedPhase1 = /Phase 1|CEO\s+Review|Strategy\s*&\s*Scope|plan-ceo-review/i.test(out);
       expect(reachedPhase1).toBe(true);
 
-      logCost(result);
-      recordE2E('autoplan-dual-voice', result);
+      logCost('autoplan-dual-voice', result);
+      recordE2E(evalCollector, 'autoplan-dual-voice', 'Autoplan dual-voice E2E', result, {
+        passed: claudeVoiceFired && (codexVoiceFired || codexUnavailable) && reachedPhase1,
+      });
     },
     330_000, // per-test timeout slightly > spawn timeout so cleanup can run
   );