From 66ee123fa7004a35e904d7595723f10962507cbe Mon Sep 17 00:00:00 2001 From: garrytan Date: Fri, 2 Oct 2026 23:45:22 +0000 Subject: [PATCH] fix(outside-voice): preflight mode list no longer looks exhaustive (s02 M10) The preflight also prints under_codex and under_current_harness; not_authed exists only on the Codex path. 'every other mode skips them' covers all non-ready modes. No test pins the old line. expected red until the regeneration commit --- scripts/resolvers/outside-voice-steps.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/resolvers/outside-voice-steps.ts b/scripts/resolvers/outside-voice-steps.ts index 0fe18e417..de0dca52d 100644 --- a/scripts/resolvers/outside-voice-steps.ts +++ b/scripts/resolvers/outside-voice-steps.ts @@ -254,7 +254,7 @@ echo "DIFF_SIZE: $DIFF_TOTAL" ${outsideVoicePreflight(ctx, { disabledBehavior: 'codex-only' })} \`CODEX_MODE: disabled\` means skip the ${outsideVoiceFor(ctx).label} passes ONLY. -\`ready\` runs them; \`not_installed\` / \`not_authed\` skip with the printed reason. +\`ready\` runs them; every other mode skips them with the printed reason. The ${outsideVoiceFor(ctx).nativeLabel} adversarial subagent always runs. **User override:** If the user explicitly requested "full review", "structured review", or "P1 gate", also run the ${outsideVoiceFor(ctx).label} structured review regardless of diff size (still requires \`CODEX_MODE: ready\`).