@lannguyensi/harness 0.64.0 → 1.0.0

package/dist/cli/index.js

@@ -1,8 +1,5 @@
 import { spawnSync } from "node:child_process";
-import * as os from "node:os";
-import * as path from "node:path";
 import { Command } from "commander";
-import { escapeForDisplay } from "../io/display-path.js";
 import { HermeticSpawnViolationError } from "../runtime/hermetic-spawn-guard.js";
 // Production version probe for `harness doctor`: synchronous --version
 // invocation with a 5s timeout. Tests inject their own probe; the CLI
@@ -23,67 +20,15 @@ export function defaultVersionProbe(cmd) {
         return null;
     }
 }
-import { add } from "./add/index.js";
-import { adopt } from "./adopt/index.js";
-import { apply, CODEX_CONFIG_BASENAME, CodexInstallRefusalError, DRIFT_HINT_MESSAGE, formatNextSteps, formatRuntimeLine, OPENCODE_CONFIG_BASENAME, SETTINGS_BASENAME, } from "./apply/index.js";
-import { packAdd, packList, packRemove, packReseed, packUpgrade } from "./pack/index.js";
-import { runPackHookPreToolUseCli } from "./pack/hook-pre-tool-use.js";
-import { runPackHookPostToolUseCli } from "./pack/hook-post-tool-use.js";
-import { runPackHookTrackActiveClaimCli } from "./pack/hook-track-active-claim.js";
-import { runPackHookStayInScopeCli } from "./pack/hook-stay-in-scope.js";
-import { runPackHookSubagentStartCli } from "./pack/hook-subagent-start.js";
-import { runPackHookSubagentStopCli } from "./pack/hook-subagent-stop.js";
-import { runPackHookCodexPreToolUseCli } from "./pack/hook-codex-pre-tool-use.js";
-import { runPackHookCodexPostToolUseCli } from "./pack/hook-codex-post-tool-use.js";
-import { runPackHookCodexStopCli } from "./pack/hook-codex-stop.js";
-import { runPackHookCodexUserPromptSubmitCli } from "./pack/hook-codex-user-prompt-submit.js";
-import { isRuntime, KNOWN_RUNTIMES } from "../policy-packs/index.js";
-import { approveBranchProtection } from "./approve/branch-protection.js";
-import { approveRisk } from "./approve/risk.js";
-import { approveUnderstanding } from "./approve/understanding.js";
-import { readPipedStdin } from "./approve/stdin-report.js";
-import { runRecordDogfood, runRecordReview, runRecordReviewSubagent, } from "./record/index.js";
-import { describe, isPillar } from "./describe.js";
-import { diff as diffRun } from "./diff/index.js";
-import { diffSinceApply } from "./diff/since-apply.js";
-import { exportManifest } from "./export.js";
-import { doctor, isDoctorTarget, KNOWN_DOCTOR_TARGETS } from "./doctor/index.js";
-import { format as formatDoctor } from "./doctor/format.js";
-import { deleteRogueLedgers, scanForRogueLedgers, } from "./doctor/rogue-ledger.js";
-import { EX_FAIL, EX_USAGE, HarnessExitError } from "./exit-codes.js";
-import { explain } from "./explain.js";
-import { explainAction } from "./explain-action.js";
-import { explainPolicy } from "./explain-policy.js";
-import { testRisk } from "./test-risk.js";
-import { resolveEnv } from "./resolve-env.js";
-import { detect as detectInit } from "./init/detect.js";
-import { init, isTemplate, KNOWN_TEMPLATES } from "./init/index.js";
-import { runInteractive } from "./init/interactive.js";
-import { isListCategory, list } from "./list.js";
-import { runInterceptCli } from "./policy/intercept.js";
-import { runSessionStartPreflight } from "./session-start/index.js";
-import { writePendingApproval } from "../runtime/pending-approval.js";
-import { runSessionStartBranchCheck } from "./session-start/branch-check.js";
-import { runSessionStartToolchainParity } from "./session-start/toolchain-parity.js";
-import { runSessionStartStaleBaseCheck } from "./session-start/stale-base-check.js";
-import { addCwdOption, addIdentityOptions, addLedgerTimeoutOption, applyCliOptions, } from "./session-start/shared-options.js";
-import { runPackHookBranchProtectionCli } from "./pack/hook-branch-protection.js";
-import { runPackHookSolutionAcceptanceCli } from "./pack/hook-solution-acceptance.js";
-import { runPackHookSolutionAcceptanceWriteguardCli } from "./pack/hook-solution-acceptance-writeguard.js";
-import { runPackHookPostMergeGateRecordCli } from "./pack/hook-post-merge-gate-record.js";
-import { runPackHookPostMergeGateCli } from "./pack/hook-post-merge-gate.js";
-import { runPackHookRuntimeRealityCli } from "./pack/hook-runtime-reality.js";
-import { gateDisable, GateDisableError } from "./gate/disable.js";
-import { gateEnable, GateEnableError } from "./gate/enable.js";
-import { DEFAULT_RETENTION_DAYS, gc } from "./gc/index.js";
-import { formatSmokeReport, runSmoke, splitCommaList, } from "./smoke/index.js";
-import { formatReport, validate } from "./validate/index.js";
+import { EX_USAGE, HarnessExitError } from "./exit-codes.js";
 import { VERSION } from "../version.js";
 import { registerRemove } from "./remove/register.js";
-import { registerDelegate } from "./delegate/register.js";
-import { registerAuditGroup } from "./register-audit-group.js";
 import { registerUninstall } from "./uninstall/register.js";
 import { registerOperatorLifecycle } from "./register-operator-lifecycle.js";
+import { registerInspectGroup } from "./register-inspect-group.js";
+import { registerSetupGroup } from "./register-setup-group.js";
+import { registerPackGroup } from "./register-pack-group.js";
+import { registerGateGroup } from "./register-gate-group.js";
 export function buildProgram(opts = {}) {
     const stdout = opts.stdout ?? ((s) => process.stdout.write(s));
     const stderr = opts.stderr ?? ((s) => process.stderr.write(s));
@@ -109,2080 +54,13 @@ export function buildProgram(opts = {}) {
         const code = err.exitCode === 1 ? EX_USAGE : (err.exitCode ?? EX_USAGE);
         throw new HarnessExitError("", code);
     });
-    program
-        .command("describe")
-        .description("Print the effective merged manifest")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides for this project name")
-        .option("--pillar <pillar>", "filter output to one section: grounding | tools | memory | hooks | policies | workflows | review_templates")
-        .option("--json", "emit JSON instead of YAML")
-        .action((options) => {
-        let pillar;
-        if (options.pillar !== undefined) {
-            if (!isPillar(options.pillar)) {
-                throw new HarnessExitError(`unknown pillar "${options.pillar}"; expected one of grounding, tools, memory, hooks, policies, workflows, review_templates`, EX_USAGE);
-            }
-            pillar = options.pillar;
-        }
-        const result = describe({
-            configPath: options.config,
-            project: options.project,
-            pillar,
-            json: options.json,
-        });
-        stdout(result.output);
-        if (!result.output.endsWith("\n"))
-            stdout("\n");
-    });
-    program
-        .command("validate")
-        .description("Lint the manifest + referenced assets")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides for this project name")
-        .option("--strict", "promote warnings to errors")
-        .option("--check-lock", "surface harness.lock asset-content drift as warnings (or errors with --strict)")
-        .option("--json", "emit a structured JSON report ({ diagnostics, errorCount, warningCount }) instead of prose")
-        .action((options) => {
-        const result = validate({
-            configPath: options.config,
-            project: options.project,
-            strict: options.strict,
-            checkLock: options.checkLock,
-        });
-        if (options.json) {
-            // JSON goes to stdout regardless of outcome so pipelines can
-            // always parse it; the exit code still carries pass/fail.
-            stdout(`${JSON.stringify({
-                diagnostics: result.diagnostics,
-                errorCount: result.errorCount,
-                warningCount: result.warningCount,
-            }, null, 2)}\n`);
-        }
-        else {
-            const report = formatReport(result);
-            if (result.diagnostics.length > 0)
-                stderr(report);
-            else
-                stdout(report);
-        }
-        if (result.errorCount > 0) {
-            throw new HarnessExitError("", EX_FAIL);
-        }
-    });
-    program
-        .command("doctor")
-        .description("Health summary across all pillars")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides for this project name")
-        .option("--shallow", "skip MCP probes (CLI --version probes still run); report manifest-reference state only")
-        .option("--target <runtime>", `additionally evaluate the harness-side adapter health for a runtime (allowed: ${KNOWN_DOCTOR_TARGETS.join(", ")})`)
-        .option("--recent-sessions <n>", "understanding-gate auto-approval doctor window: how many of the newest .approvals/ session markers to scan (default: 20)")
-        .option("--json", "emit a structured JSON DoctorReport instead of prose (--rm-rogue-ledgers is ignored with --json)")
-        .option("--rm-rogue-ledgers", "after reporting, delete each rogue evidence-ledger directory found (prompts per hit; combine with --yes to skip prompts)")
-        .option("--yes", "with --rm-rogue-ledgers, skip per-hit confirmation prompts")
-        .action(async (options) => {
-        let target;
-        if (options.target !== undefined) {
-            if (!isDoctorTarget(options.target)) {
-                stderr(`unknown --target ${JSON.stringify(options.target)}; expected one of ${KNOWN_DOCTOR_TARGETS.join(", ")}\n`);
-                throw new HarnessExitError("", EX_USAGE);
-            }
-            target = options.target;
-        }
-        let recentSessions;
-        if (options.recentSessions !== undefined) {
-            const parsed = Number(options.recentSessions);
-            if (!Number.isInteger(parsed) || parsed < 1) {
-                stderr(`--recent-sessions must be an integer >= 1, got ${JSON.stringify(options.recentSessions)}\n`);
-                throw new HarnessExitError("", EX_USAGE);
-            }
-            recentSessions = parsed;
-        }
-        const report = await doctor({
-            configPath: options.config,
-            project: options.project,
-            shallow: options.shallow,
-            versionProbe: defaultVersionProbe,
-            ...(target !== undefined ? { target } : {}),
-            ...(recentSessions !== undefined ? { recentSessions } : {}),
-            ...(opts.rogueLedgerScanOptions !== undefined
-                ? { rogueLedgerScanOptions: opts.rogueLedgerScanOptions }
-                : {}),
-        });
-        // Non-zero exit when the report carries errors (task a07b379a):
-        // callers previously had no way to gate CI/scripts on doctor
-        // health, since this action always fell through to exit 0
-        // regardless of report.errorCount. Warnings alone still exit 0.
-        // Checked before every return below (json / no --rm-rogue-ledgers /
-        // no hits found / after the delete+rescan flow) so the exit code
-        // is consistent across all output modes.
-        const failIfErrors = () => {
-            if (report.errorCount > 0) {
-                throw new HarnessExitError("", EX_FAIL);
-            }
-        };
-        if (options.json) {
-            stdout(`${JSON.stringify(report, null, 2)}\n`);
-            failIfErrors();
-            return;
-        }
-        stdout(formatDoctor(report));
-        if (!options.rmRogueLedgers) {
-            failIfErrors();
-            return;
-        }
-        const hits = report.rogueLedgerDbs;
-        if (hits.length === 0) {
-            stdout("no rogue evidence-ledger DBs found; nothing to delete\n");
-            failIfErrors();
-            return;
-        }
-        const result = await deleteRogueLedgers(hits, { yes: options.yes });
-        for (const hit of result.deleted) {
-            stdout(`deleted: ${hit.rogueDir}\n`);
-        }
-        for (const hit of result.skipped) {
-            stdout(`skipped: ${hit.rogueDir}\n`);
-        }
-        // Re-scan and print clean delta so the operator can confirm the
-        // on-disk state after deletion. Uses the same scan options as the
-        // initial scan (injectable via RunOptions.rogueLedgerScanOptions for
-        // tests; production falls back to os.homedir() / process.cwd()).
-        const afterScan = scanForRogueLedgers({
-            homeDir: opts.rogueLedgerScanOptions?.homeDir ?? os.homedir(),
-            cwd: opts.rogueLedgerScanOptions?.cwd ?? process.cwd(),
-            ...(opts.rogueLedgerScanOptions?.fsInterface !== undefined
-                ? { fsInterface: opts.rogueLedgerScanOptions.fsInterface }
-                : {}),
-        });
-        stdout(`rogue evidence-ledger DBs remaining: ${afterScan.length}\n`);
-        failIfErrors();
-    });
-    program
-        .command("list <category>")
-        .description("Flat denormalised listing per category: mcp / cli / skills / memories / hooks / policies / workflows")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--filter <substr>", "case-insensitive substring filter on name (or path for memories)")
-        .option("--json", "emit JSON array instead of an aligned text table")
-        .action((category, options) => {
-        if (!isListCategory(category)) {
-            throw new HarnessExitError(`unknown list category "${category}"; expected one of mcp, cli, skills, memories, hooks, policies, workflows`, EX_USAGE);
-        }
-        const result = list(category, {
-            configPath: options.config,
-            project: options.project,
-            filter: options.filter,
-            json: options.json,
-        });
-        stdout(result.output);
-    });
-    program
-        .command("diff")
-        .description("Diff the manifest against a git ref (--since <ref>) or against the last " +
-        "applied state (--since-apply). --memory-detail expands per-memory-dir " +
-        "Merkle drift back to per-file changes.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--since <ref>", "git ref to diff against")
-        .option("--since-apply", "diff against harness.generated/.last-apply")
-        .option("--memory-detail", "expand per-memory-dir drift to per-file changes")
-        .option("--json", "emit structured JSON output")
-        .action((options) => {
-        if (options.since && options.sinceApply) {
-            throw new HarnessExitError("--since <ref> and --since-apply are mutually exclusive", EX_USAGE);
-        }
-        if (options.sinceApply) {
-            const r = diffSinceApply({
-                ...(options.config !== undefined ? { configPath: options.config } : {}),
-                ...(options.memoryDetail ? { memoryDetail: true } : {}),
-            });
-            if (options.json) {
-                stdout(`${JSON.stringify(r.json, null, 2)}\n`);
-            }
-            else if (!r.hasDrift) {
-                stdout("no drift since last apply\n");
-            }
-            else {
-                stdout(r.output);
-            }
-            for (const w of r.warnings) {
-                stderr(`warning: ${w}\n`);
-            }
-            if (r.hasDrift)
-                throw new HarnessExitError("", EX_FAIL);
-            return;
-        }
-        const result = diffRun({
-            configPath: options.config,
-            project: options.project,
-            since: options.since,
-        });
-        // Override-layer diagnostics go to stderr so stdout stays a clean
-        // diff for piping (task b2660f9e).
-        for (const w of result.warnings)
-            stderr(`harness diff: ${w}\n`);
-        stdout(result.output);
-    });
-    program
-        .command("init")
-        .description("Bootstrap a starter harness.yaml from a template")
-        .option("--template <name>", `template to instantiate: ${KNOWN_TEMPLATES.join(" | ")} (default: minimal)`)
-        .option("--force", "overwrite an existing manifest")
-        .option("--config <path>", "manifest path to write (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--probe", "skip writing, print a JSON snapshot of detected runtimes (Claude Code, Codex), the existing ~/.harness/harness.yaml (or legacy ~/.claude/harness.yaml), and MCP servers wired in settings.json. Read-only.")
-        .option("--interactive", "run the guided wizard (detect environment, pick profile, preview + write). Mutually exclusive with --probe / --template.")
-        .action(async (options) => {
-        if (options.probe && options.interactive) {
-            throw new HarnessExitError("--probe and --interactive are mutually exclusive", EX_USAGE);
-        }
-        if (options.probe) {
-            if (options.template !== undefined || options.force || options.config !== undefined) {
-                throw new HarnessExitError("--probe is read-only; pass it without --template / --force / --config", EX_USAGE);
-            }
-            const result = await detectInit();
-            stdout(`${JSON.stringify(result, null, 2)}\n`);
-            return;
-        }
-        if (options.interactive) {
-            if (options.template !== undefined || options.config !== undefined) {
-                throw new HarnessExitError("--interactive owns its own template + path choices; do not combine with --template / --config", EX_USAGE);
-            }
-            // Thread --force into the wizard's forceOverwrite: without it
-            // an existing manifest makes the wizard re-prompt, and
-            // `harness init --interactive --force` looked like it should
-            // overwrite non-interactively but the flag never reached
-            // runInteractive (harness/418cebd4).
-            const r = await runInteractive({
-                stdout,
-                stderr,
-                ...(options.force ? { forceOverwrite: true } : {}),
-            });
-            if (r.aborted) {
-                return;
-            }
-            if (r.validateClean === false) {
-                throw new HarnessExitError(`manifest written but failed harness validate; see stderr for diagnostics`, EX_FAIL);
-            }
-            if (r.binResolutionClean === false) {
-                throw new HarnessExitError(`manifest written but one or more declared MCP/CLI binaries do not resolve on PATH; see stderr for diagnostics and remediation hints`, EX_FAIL);
-            }
-            return;
-        }
-        if (options.template !== undefined && !isTemplate(options.template)) {
-            throw new HarnessExitError(`unknown template "${options.template}"; expected one of ${KNOWN_TEMPLATES.join(", ")}`, EX_USAGE);
-        }
-        const result = await init({
-            template: options.template,
-            force: options.force,
-            configPath: options.config,
-        });
-        if (result.stderr)
-            stderr(result.stderr);
-        stdout(result.stdout);
-    });
-    const addCmd = program
-        .command("add")
-        .description("Insert a new entry into harness.yaml (managed mutation)");
-    function addCommonOptions(c) {
-        return c
-            .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-            .option("--dry-run", "print the unified diff and exit without writing");
-    }
-    function parseBlocking(s) {
-        if (s === "false")
-            return false;
-        if (s === "soft" || s === "hard")
-            return s;
-        throw new HarnessExitError(`invalid --blocking value "${s}"; expected one of false, soft, hard`, EX_USAGE);
-    }
-    function parseIntFlag(s, label) {
-        const n = Number.parseInt(s, 10);
-        if (!Number.isFinite(n) || String(n) !== s.trim()) {
-            throw new HarnessExitError(`invalid ${label} value "${s}"; expected an integer`, EX_USAGE);
-        }
-        return n;
-    }
-    async function runAdd(action, opts) {
-        const result = await add(action, { configPath: opts.config, dryRun: opts.dryRun });
-        for (const w of result.warnings) {
-            stderr(`warning: ${w}\n`);
-        }
-        if (opts.dryRun) {
-            stdout(result.diff);
-            return;
-        }
-        stdout(`added ${result.type} ${JSON.stringify(result.name)} to ${result.path}\n`);
-    }
-    addCommonOptions(addCmd
-        .command("mcp <name>")
-        .description("Add an MCP server entry under tools.mcp[]")
-        .option("--command <cmd>", "argv-style command; comma-separated for multi-token")
-        .option("--health-verb <v>", "MCP verb to invoke for liveness")
-        .option("--health-timeout-ms <n>", "verb timeout in ms (default 5000 when --health-verb is set)")
-        .option("--enabled <bool>", "true|false (default true)")).action(async (name, options) => {
-        const command = options.command
-            ? options.command.includes(",")
-                ? options.command.split(",").map((s) => s.trim())
-                : options.command
-            : "";
-        if (!command) {
-            throw new HarnessExitError("harness add mcp: --command is required", EX_USAGE);
-        }
-        const entry = {
-            name,
-            command,
-        };
-        if (options.healthVerb !== undefined) {
-            const timeoutMs = options.healthTimeoutMs
-                ? parseIntFlag(options.healthTimeoutMs, "--health-timeout-ms")
-                : 5000;
-            entry.health = {
-                verb: options.healthVerb,
-                timeout_ms: timeoutMs,
-            };
-        }
-        if (options.enabled !== undefined) {
-            if (options.enabled !== "true" && options.enabled !== "false") {
-                throw new HarnessExitError(`invalid --enabled value "${options.enabled}"; expected true or false`, EX_USAGE);
-            }
-            entry.enabled = options.enabled === "true";
-        }
-        await runAdd({ type: "mcp", entry: entry }, { config: options.config, dryRun: options.dryRun });
-    });
-    addCommonOptions(addCmd
-        .command("cli <name>")
-        .description("Add a CLI tool entry under tools.cli[]")
-        .requiredOption("--binary <b>", "binary name on PATH or absolute path")
-        .option("--required", "validate fails if the binary is missing")
-        .option("--min-version <v>", "minimum semver")).action(async (name, options) => {
-        const entry = {
-            name,
-            binary: options.binary,
-        };
-        if (options.required)
-            entry.required = true;
-        if (options.minVersion !== undefined)
-            entry.min_version = options.minVersion;
-        await runAdd({ type: "cli", entry }, { config: options.config, dryRun: options.dryRun });
-    });
-    addCommonOptions(addCmd
-        .command("skill <name>")
-        .description("Enable a skill by name under tools.skills.enabled[]")).action(async (name, options) => {
-        await runAdd({ type: "skill", entry: name }, { config: options.config, dryRun: options.dryRun });
-    });
-    addCommonOptions(addCmd
-        .command("hook <name>")
-        .description("Add a hook entry under hooks[]")
-        .requiredOption("--event <e>", "runtime event (e.g. SessionStart, PreToolUse)")
-        .requiredOption("--command <c>", "shell command (executable path or script with args)")
-        .option("--match <r>", "tool-name regex filter (PreToolUse / PostToolUse only)")
-        .option("--blocking <m>", "false | soft | hard (default false)")
-        .option("--budget-ms <n>", "timeout in ms (default 30000)")).action(async (name, options) => {
-        const entry = {
-            name,
-            event: options.event,
-            command: options.command,
-            blocking: options.blocking !== undefined ? parseBlocking(options.blocking) : false,
-        };
-        if (options.match !== undefined)
-            entry.match = options.match;
-        if (options.budgetMs !== undefined) {
-            entry.budget_ms = parseIntFlag(options.budgetMs, "--budget-ms");
-        }
-        await runAdd({ type: "hook", entry }, { config: options.config, dryRun: options.dryRun });
-    });
-    program
-        .command("export")
-        .description("Emit the effective merged manifest as a single self-contained YAML " +
-        "(or JSON). --sanitize redacts /home/<user>/ paths and env values whose " +
-        "key looks credential-shaped.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides for this project name")
-        .option("--sanitize", "redact /home/<user>/ paths and credential-shaped env values")
-        .option("--json", "emit JSON instead of YAML")
-        .option("-o, --output <file>", "write to <file> atomically instead of stdout")
-        .action((options) => {
-        const result = exportManifest({
-            configPath: options.config,
-            project: options.project,
-            sanitize: options.sanitize,
-            json: options.json,
-            outputPath: options.output,
-        });
-        if (result.wroteTo === null) {
-            stdout(result.output);
-            if (!result.output.endsWith("\n"))
-                stdout("\n");
-        }
-        else {
-            stderr(`wrote ${result.wroteTo}\n`);
-        }
-    });
-    program
-        .command("adopt <file>")
-        .description("Capture hand-edits into the manifest. Hooks are diffed against <file> " +
-        "(today: ~/.claude/settings.json). MCP servers are ALWAYS diffed " +
-        "against the effective Claude Code registry (top-level mcpServers in " +
-        "~/.claude.json, respecting CLAUDE_CONFIG_DIR) regardless of <file> — " +
-        "Claude Code does not read <file>'s mcpServers block at runtime. " +
-        "Prompts y/N before writing.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--yes", "skip the confirmation prompt (for non-interactive use)")
-        .action(async (file, options) => {
-        const result = await adopt(file, { configPath: options.config, yes: options.yes });
-        if (result.deadSettingsMcpNames.length > 0) {
-            stderr(`⚠ ${file} has a dead \`mcpServers\` block (${result.deadSettingsMcpNames.join(", ")}); ` +
-                "Claude Code does not read this file for MCP registration — adopt diffed against the " +
-                "effective registry instead. Safe to remove by hand.\n");
-        }
-        if (result.registryReadError !== undefined) {
-            stderr(`⚠ could not read the Claude Code MCP registry: ${result.registryReadError}; ` +
-                "treated as empty for MCP drift purposes.\n");
-        }
-        if (result.outcome === "no-drift") {
-            stdout(`nothing to adopt (no drift between ${file} and ${result.manifestPath})\n`);
-            return;
-        }
-        if (result.outcome === "declined") {
-            stdout(`adoption declined; ${result.manifestPath} unchanged\n`);
-            return;
-        }
-        const parts = [];
-        if (result.hookDriftCount > 0) {
-            parts.push(`${result.hookDriftCount} hook${result.hookDriftCount === 1 ? "" : "s"}` +
-                ` (${result.adoptedNames.join(", ")})`);
-        }
-        if (result.mcpDriftCount > 0) {
-            const mcpFrag = `${result.mcpDriftCount} MCP server${result.mcpDriftCount === 1 ? "" : "s"}` +
-                ` (${result.adoptedMcpNames.join(", ")})`;
-            parts.push(result.replacedMcpNames.length > 0
-                ? `${mcpFrag}; replaced existing manifest entry for: ${result.replacedMcpNames.join(", ")}`
-                : mcpFrag);
-        }
-        stdout(`adopted ${parts.join(" + ")} from ${result.settingsPath} into ${result.manifestPath}\n`);
-    });
-    program
-        .command("apply")
-        .description("Regenerate harness.generated/ outputs (settings.json + MEMORY.md index) " +
-        "from the manifest. Refuses to overwrite hand-edits without --overwrite-drift; " +
-        "use `harness adopt <file>` to capture them back into the manifest instead.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides for this project name")
-        .option("--dry-run", "print the would-be diff + restart hints; do not write")
-        .option("--overwrite-drift", "discard any on-disk hand-edits to harness.generated/ files (prompts for `yes`; pair with --yes for non-interactive runs)")
-        .option("--yes", "with --overwrite-drift, skip the confirmation prompt (for non-interactive use)")
-        .option("--strict-lock", "refuse with exit 1 (no write) when harness.lock asset drift is detected; dry-run wins")
-        .option("--target <path>", "additionally write the generated settings.json to <path> (e.g. .claude/settings.local.json); " +
-        "claude-code only: without --runtime it implies claude-code, with --runtime codex or opencode it is refused")
-        .option("--merge", "with --target, 3-way merge into an existing target file (replace owned keys, preserve others)")
-        .option("--force", "with --target, overwrite an existing target file (no merge)")
-        .option("--runtime <runtime>", `policy-pack adapter runtime (${KNOWN_RUNTIMES.join(" | ")}; default: claude-code with --target, ` +
-        "otherwise the runtime of the last apply (inferred from its files for an older record), " +
-        "or claude-code when there is none; --install follows the same default). " +
-        "Selects which adapter shape policy-pack hooks expand into and which artefacts apply writes. " +
-        "`codex` emits harness.generated/codex/config.toml in place of settings.json. " +
-        "`opencode` emits harness.generated/opencode/opencode.json (MCP servers only; not auto-installed).")
-        .option("--install", "with --runtime codex, merge the generated hook block into ~/.codex/config.toml")
-        .option("--codex-config <path>", "with --runtime codex --install, override the Codex config path (default ~/.codex/config.toml)")
-        .option("--quiet", "suppress the post-apply Next-steps hint")
-        .option("--json", "emit a structured JSON summary instead of prose (implies --quiet)")
-        .action(async (options) => {
-        let runtime;
-        if (options.runtime !== undefined) {
-            if (!isRuntime(options.runtime)) {
-                stderr(`unknown --runtime ${JSON.stringify(options.runtime)}; expected one of ${KNOWN_RUNTIMES.join(", ")}\n`);
-                throw new HarnessExitError("", EX_USAGE);
-            }
-            runtime = options.runtime;
-        }
-        // --json is documented as implying --quiet. Normalize early so any
-        // future fall-through path (or new prose branch) honors it without
-        // depending on the JSON early-return below as the only chokepoint.
-        if (options.json)
-            options.quiet = true;
-        // `apply()` throws `CodexInstallRefusalError` (rather than
-        // returning a refuse `outcome`, the way `target-exists-refuse` /
-        // `drift-refuse` do) for a codex-install refusal, because the
-        // refusal is detected deep inside `planCodexConfigInstall`, before
-        // an `ApplyResult` can be assembled. Catch it here so `--json`
-        // still gets a structured error on stdout the same way every
-        // other refusal does, instead of only the plain-text message
-        // `run()`'s top-level catch would otherwise print.
-        let result;
-        try {
-            result = await apply({
-                ...(options.config !== undefined ? { configPath: options.config } : {}),
-                ...(options.project !== undefined ? { project: options.project } : {}),
-                ...(options.dryRun ? { dryRun: true } : {}),
-                ...(options.overwriteDrift ? { overwriteDrift: true } : {}),
-                ...(options.yes ? { yes: true } : {}),
-                ...(options.strictLock ? { strictLock: true } : {}),
-                ...(options.target !== undefined ? { target: options.target } : {}),
-                ...(options.merge ? { merge: true } : {}),
-                ...(options.force ? { force: true } : {}),
-                ...(runtime !== undefined ? { runtime } : {}),
-                ...(options.install ? { installCodex: true } : {}),
-                ...(options.codexConfig !== undefined
-                    ? { codexConfigPath: options.codexConfig }
-                    : {}),
-            });
-        }
-        catch (err) {
-            if (err instanceof CodexInstallRefusalError && options.json) {
-                stdout(`${JSON.stringify({ outcome: "codex-install-refuse", configPath: err.configPath, error: err.message }, null, 2)}\n`);
-                throw new HarnessExitError("", EX_FAIL);
-            }
-            throw err;
-        }
-        if (options.json) {
-            // Machine-readable: one JSON object on stdout regardless of
-            // outcome. Refusals still set the non-zero exit below; consumers
-            // should check both `outcome` in the JSON and the process exit.
-            stdout(`${JSON.stringify(result, null, 2)}\n`);
-            if (result.outcome === "target-exists-refuse" ||
-                result.outcome === "lock-drift-refuse" ||
-                result.outcome === "drift-refuse") {
-                throw new HarnessExitError("", EX_FAIL);
-            }
-            return;
-        }
-        if (result.outcome === "target-exists-refuse") {
-            stderr(`target ${result.targetPath} exists; pass --merge to merge into it, or --force to overwrite\n`);
-            throw new HarnessExitError("", EX_FAIL);
-        }
-        if (result.outcome === "lock-drift-refuse") {
-            for (const d of result.lockDrift) {
-                if (d.reason === "missing") {
-                    stderr(`asset drift detected: ${d.entry.path} missing since last apply\n`);
-                }
-                else {
-                    stderr(`asset drift detected: ${d.entry.path} changed since last apply\n`);
-                }
-            }
-            stderr("--strict-lock: refusing to overwrite the lock; re-run without --strict-lock to acknowledge, or revert the upstream asset edit\n");
-            throw new HarnessExitError("", EX_FAIL);
-        }
-        if (result.outcome === "drift-refuse") {
-            for (const f of result.files) {
-                if (f.diff) {
-                    stderr(`drift detected in ${f.path}:\n`);
-                    stderr(f.diff);
-                    if (!f.diff.endsWith("\n"))
-                        stderr("\n");
-                }
-            }
-            stderr(`${DRIFT_HINT_MESSAGE}\n`);
-            throw new HarnessExitError("", EX_FAIL);
-        }
-        if (result.outcome === "drift-discarded") {
-            stdout("overwrite-drift declined; nothing written\n");
-            return;
-        }
-        const changedFiles = result.files.filter((f) => f.changed);
-        // Name a reused or switched runtime before the file list, in the
-        // dry-run and the real apply alike (agent-tasks b9e6d63c).
-        const runtimeLine = formatRuntimeLine(result);
-        if (runtimeLine !== null)
-            stdout(`${runtimeLine}\n`);
-        if (result.outcome === "no-changes") {
-            stdout("no changes\n");
-        }
-        else if (result.outcome === "would-apply") {
-            stdout(`would apply ${changedFiles.length} file(s):\n`);
-            for (const f of changedFiles) {
-                stdout(`  ${f.path}\n`);
-            }
-            if (result.codexConfigInstall?.changed) {
-                stdout(`  ${result.codexConfigInstall.configPath} (Codex install)\n`);
-                for (const id of result.codexConfigInstall.removedHookIds) {
-                    stdout(`    removing hook block: ${id}\n`);
-                }
-                for (const section of result.codexConfigInstall.foreignSectionsPreserved) {
-                    stdout(`    preserving foreign section: ${section}\n`);
-                }
-            }
-        }
-        else {
-            stdout(`applied ${changedFiles.length} file(s):\n`);
-            for (const f of changedFiles) {
-                stdout(`  ${f.path}\n`);
-            }
-            if (result.targetPath && (result.targetWritten || result.targetInSync)) {
-                if (result.targetWritten) {
-                    if (result.targetMergeSummary) {
-                        stdout(`${result.targetMergeSummary}\n`);
-                    }
-                    else {
-                        stdout(`wrote target: ${result.targetPath}\n`);
-                    }
-                }
-                else {
-                    // Idempotent re-apply: the merge was a no-op because the
-                    // target already held the merged content. Report it as
-                    // success, not silence.
-                    stdout(`target already in sync: ${result.targetPath}\n`);
-                }
-            }
-            if (result.codexConfigInstall?.written) {
-                stdout(`${result.codexConfigInstall.summary}\n`);
-                for (const id of result.codexConfigInstall.removedHookIds) {
-                    stdout(`  removed hook block: ${id}\n`);
-                }
-                for (const section of result.codexConfigInstall.foreignSectionsPreserved) {
-                    stdout(`  preserved foreign section: ${section}\n`);
-                }
-                if (result.codexConfigInstall.backupPath) {
-                    stdout(`backup written to ${result.codexConfigInstall.backupPath}\n`);
-                }
-            }
-            stdout(`harness.lock written to ${result.lockPath}\n`);
-            if (!options.quiet) {
-                // `formatNextSteps` collapses to the single "wired into ..."
-                // line when it receives `targetPath`. Pass it whenever the
-                // target ended this run in sync — written this run, OR
-                // already byte-identical (`targetInSync`). Gating on
-                // `targetWritten` alone misclassified an idempotent
-                // re-apply as "nothing is wired" and looped the operator
-                // through redundant apply commands.
-                // `anyChanged` softens the no-target lede when the generated
-                // manifest is already up to date (avoids over-claiming
-                // "nothing is wired" against operators who wired a target
-                // on a previous run).
-                const generatedSettingsPath = path.join(result.generatedDir, SETTINGS_BASENAME);
-                const codexConfigPath = path.join(result.generatedDir, CODEX_CONFIG_BASENAME);
-                const opencodeConfigPath = path.join(result.generatedDir, OPENCODE_CONFIG_BASENAME);
-                const anyChanged = result.files.some((f) => f.changed);
-                stdout(formatNextSteps({
-                    generatedSettingsPath,
-                    codexConfigPath,
-                    opencodeConfigPath,
-                    anyChanged,
-                    runtime: result.runtime,
-                    ...((result.targetWritten || result.targetInSync) && result.targetPath
-                        ? { targetPath: result.targetPath }
-                        : {}),
-                    // Pass the --install outcome through whenever one was
-                    // requested this run (written or a no-op), so the codex
-                    // lede reflects reality instead of always claiming
-                    // "nothing is installed yet".
-                    ...(result.codexConfigInstall !== undefined
-                        ? {
-                            codexInstall: {
-                                configPath: result.codexConfigInstall.configPath,
-                                written: result.codexConfigInstall.written,
-                                ...(result.codexConfigInstall.backupPath !== undefined
-                                    ? { backupPath: result.codexConfigInstall.backupPath }
-                                    : {}),
-                            },
-                        }
-                        : {}),
-                }));
-            }
-        }
-        for (const d of result.lockDrift) {
-            if (d.reason === "missing") {
-                stderr(`asset drift detected: ${d.entry.path} missing since last apply\n`);
-            }
-            else {
-                stderr(`asset drift detected: ${d.entry.path} changed since last apply\n`);
-            }
-        }
-        for (const w of result.warnings) {
-            stderr(`warning: ${w}\n`);
-        }
-        for (const h of result.restartHints) {
-            stderr(`restart hint: ${h}\n`);
-        }
-    });
+    registerInspectGroup(program, { stdout, stderr }, opts);
+    registerSetupGroup(program, { stdout, stderr });
     registerRemove(program, { stdout, stderr });
-    // `harness pack` subtree (Phase 6 #3): managed CRUD over policy_packs[].
-    const packCmd = program
-        .command("pack")
-        .description("Manage policy_packs[] entries (add / remove / list / reseed)");
-    packCmd
-        .command("add <name>")
-        .description("Insert a new policy_packs entry. <name> must be a known builtin (see docs/policy-packs/).")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--mode <mode>", "pack-specific config.mode value (e.g. fast_confirm | grill_me | strict)")
-        .option("--source <src>", "pack source (default: builtin)")
-        .option("--description <text>", "operator-facing description")
-        .option("--disabled", "register as enabled: false")
-        .option("--dry-run", "print the unified diff and exit without writing")
-        .action(async (name, options) => {
-        const entry = { name };
-        if (options.source !== undefined)
-            entry.source = options.source;
-        if (options.disabled === true)
-            entry.enabled = false;
-        if (options.description !== undefined)
-            entry.description = options.description;
-        if (options.mode !== undefined)
-            entry.config = { mode: options.mode };
-        const result = await packAdd(entry, {
-            configPath: options.config,
-            dryRun: options.dryRun,
-        });
-        if (options.dryRun) {
-            stdout(result.diff);
-            return;
-        }
-        stdout(`added policy_packs entry ${JSON.stringify(result.name)} to ${result.path}\n`);
-    });
-    packCmd
-        .command("remove <name>")
-        .description("Remove a policy_packs entry. Refuses without --force when applied state " +
-        "is recorded in .last-apply.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--dry-run", "print the unified diff and exit without writing")
-        .option("--force", "remove the manifest entry AND clean up the on-disk pack files + .last-apply state")
-        .action(async (name, options) => {
-        const result = await packRemove(name, {
-            configPath: options.config,
-            dryRun: options.dryRun,
-            force: options.force,
-        });
-        if (result.cleanedFiles.length > 0) {
-            stderr(`(forced cleanup — removed ${result.cleanedFiles.length} pack file(s) and pruned .last-apply)\n`);
-        }
-        if (options.dryRun) {
-            stdout(result.diff);
-            return;
-        }
-        stdout(`removed policy_packs entry ${JSON.stringify(result.name)} from ${result.path}\n`);
-    });
-    // `harness pack reseed <name>` (task 68b9ad9c): pull the shipped
-    // builtin template's config.ux (and config.producers) into an already-
-    // installed manifest. Explicit-only, mirroring add/remove — never
-    // invoked by `apply`, so an upgrade never silently rewrites an
-    // operator's deliberate ux customisation. Paired with the `harness
-    // doctor` divergence warning (src/policy-packs/ux-drift-check.ts).
-    packCmd
-        .command("reseed <name>")
-        .description("Pull the shipped builtin template's config.ux (and config.producers) for <name> into " +
-        "the manifest, preserving other config keys (mode, approval_lifecycle, ...). No-op if " +
-        "already up to date. See `harness doctor` for a divergence warning.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--dry-run", "print the unified diff and exit without writing")
-        .action(async (name, options) => {
-        const result = await packReseed(name, {
-            configPath: options.config,
-            dryRun: options.dryRun,
-        });
-        if (result.fieldsChanged.length === 0) {
-            stdout(`policy_packs entry ${JSON.stringify(result.name)} already matches the shipped template; nothing to reseed.\n`);
-            return;
-        }
-        if (options.dryRun) {
-            stdout(result.diff);
-            return;
-        }
-        stdout(`reseeded ${result.fieldsChanged.map((f) => `config.${f}`).join(", ")} for policy_packs entry ${JSON.stringify(result.name)} in ${result.path}\n`);
-    });
-    // `harness pack upgrade <name>` (task 8f637efd, D-004): text-level
-    // insertion of a pack's missing default config block into an existing
-    // manifest, idempotent, never rewrites content outside the inserted
-    // block. Today's only wired upgrade is `understanding-before-
-    // execution`'s `auto_approve` default; see upgrade.ts's module header
-    // for why this is text-level rather than a Document-API mutation like
-    // `add` / `reseed`.
-    packCmd
-        .command("upgrade <name>")
-        .description("Insert a pack's missing default config block into an existing manifest (today: " +
-        "understanding-before-execution's auto_approve default). Idempotent; " +
-        "refuses on ambiguity rather than guessing where to insert.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--dry-run", "print the unified diff and exit without writing")
-        .action(async (name, options) => {
-        const result = await packUpgrade(name, {
-            configPath: options.config,
-            dryRun: options.dryRun,
-        });
-        if (result.alreadyPresent) {
-            stdout(`policy_packs entry ${JSON.stringify(result.name)} already carries this upgrade; nothing to insert (${result.path} unchanged).\n`);
-            return;
-        }
-        if (options.dryRun) {
-            stdout(result.diff);
-            return;
-        }
-        stdout(`upgraded policy_packs entry ${JSON.stringify(result.name)} in ${result.path}\n`);
-    });
-    // `harness pack hook` runtime sub-tree (Phase 6 #4): wired by the
-    // pack's PreToolUse hook contribution; reads PreToolUse JSON from
-    // stdin, consults the signed approval marker (the persisted report and
-    // the ledger are audit evidence only, task 7402301d), emits Claude Code
-    // deny JSON on block.
-    const packHookCmd = packCmd
-        .command("hook")
-        .description("Pack runtime hook entrypoints (called by Claude Code via settings.json)");
-    packHookCmd
-        .command("pre-tool-use")
-        .description("PreToolUse blocker: read tool-event JSON from stdin, consult the signed approval marker (persisted report and ledger are audit evidence only), emit deny JSON on block")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--reports-dir <path>", "override the persisted-report directory (default: ./.understanding-gate/reports)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        if (options.reportsDir)
-            cliOpts.reportsDir = options.reportsDir;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        await runPackHookPreToolUseCli(cliOpts);
-    });
-    packHookCmd
-        .command("post-tool-use")
-        .description("PostToolUse marker-expiry: read tool-event JSON from stdin, delete the per-session approval marker when the just-completed tool matches config.approval_lifecycle.expire_on_tool_match (agent-tasks/d8ee60ca)")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        await runPackHookPostToolUseCli(cliOpts);
-    });
-    packHookCmd
-        .command("track-active-claim")
-        .description("PostToolUse: read tool-event JSON from stdin, maintain ~/.claude/harness.generated/active-claim on agent-tasks task_start / task_finish / task_abandon / task_merge so `harness approve understanding` can auto-resolve the task id without --task (harness/494fd1e5). task_finish keeps the marker when the resulting status is review and clears it otherwise (task c86e3c4a); task_abandon / task_merge / a non-review task_finish clear it only when the marker already names the same task id, otherwise it is left in place (task c86e3c4a round 2).")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        await runPackHookTrackActiveClaimCli(cliOpts);
-    });
-    packHookCmd
-        .command("stay-in-scope")
-        .description("PostToolUse: read tool-event JSON from stdin and, when explicitly configured, emit a non-blocking reminder and JSONL audit row for configured task-tool payloads. Current manifest configuration controls matching; STAY_IN_SCOPE_DISABLED=1 disables a live hook and STAY_IN_SCOPE_LOG overrides its log path.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        await runPackHookStayInScopeCli(cliOpts);
-    });
-    packHookCmd
-        .command("subagent-start")
-        .description("SubagentStart: read event JSON from stdin, write a signed in-flight record for this (session_id, agent_id) when the parent session currently holds a valid understanding-gate approval (docs/decisions/2026-08-27-ug-auto-mode-approval.md).")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        await runPackHookSubagentStartCli(cliOpts);
-    });
-    packHookCmd
-        .command("subagent-stop")
-        .description("SubagentStop: read event JSON from stdin, remove the in-flight record written for this (session_id, agent_id) by subagent-start.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        await runPackHookSubagentStopCli(cliOpts);
-    });
-    // Phase 6 #6 — Codex adapter sub-commands. Mirror the pre-tool-use
-    // shape; UserPromptSubmit emits developer-context instructions on stdout.
-    packHookCmd
-        .command("codex-pre-tool-use")
-        .description("Codex PreToolUse blocker: read tool-event JSON from stdin, consult the signed approval marker (persisted report and ledger are audit evidence only), exit 2 with stderr reason on block")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--reports-dir <path>", "override the persisted-report directory (default: ./.understanding-gate/reports)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        if (options.reportsDir)
-            cliOpts.reportsDir = options.reportsDir;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        const result = await runPackHookCodexPreToolUseCli(cliOpts);
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("codex-post-tool-use")
-        .description("Codex PostToolUse marker-expiry: read tool-event JSON from stdin, delete the per-session (and, where applicable, per-task) approval marker and expire the persisted report when the just-completed tool matches config.approval_lifecycle.expire_on_tool_match / expire_on_bash_match (task a1348c89, mirrors the Claude `post-tool-use` hook).")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .option("--reports-dir <path>", "override the persisted-report directory (default: ./.understanding-gate/reports)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        if (options.reportsDir)
-            cliOpts.reportsDir = options.reportsDir;
-        await runPackHookCodexPostToolUseCli(cliOpts);
-    });
-    packHookCmd
-        .command("codex-user-prompt-submit")
-        .description("Codex UserPromptSubmit injector: emit developer-context instructions on stdout whenever Codex invokes the event")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        await runPackHookCodexUserPromptSubmitCli(cliOpts);
-    });
-    packHookCmd
-        .command("branch-protection")
-        .description("PreToolUse blocker for the branch-protection pack: read tool-event JSON from stdin, consult the " +
-        "evidence ledger, emit a deny envelope on protected branches unless either a fresh " +
-        "`branch:non-protected` tag (within 5m) or the operator-only override marker " +
-        "(written by `harness approve branch-protection`) is present.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--cwd <path>", "override cwd resolution (default: stdin event.cwd then process.cwd())")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.cwd)
-            cliOpts.cwd = options.cwd;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        const result = await runPackHookBranchProtectionCli(cliOpts);
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("solution-acceptance")
-        .description("PreToolUse completion-gate for the solution-acceptance pack: read tool-event JSON from stdin, " +
-        "and on a task-finishing tool (agent-tasks completion verb or `git push` / `gh pr merge`) emit a " +
-        "deny envelope unless a ready solution-acceptance verdict exists at the current git HEAD for the " +
-        "active-claim task. Fail-closed.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--cwd <path>", "override cwd resolution (default: stdin event.cwd then process.cwd())")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.cwd)
-            cliOpts.cwd = options.cwd;
-        const result = await runPackHookSolutionAcceptanceCli(cliOpts);
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("solution-acceptance-writeguard")
-        .description("PreToolUse anti-forgery write-guard for the solution-acceptance pack: read tool-event JSON from " +
-        "stdin, emit a deny envelope on any agent write into the solution-verdict dir (path-tool file_path " +
-        "inside it, or a non-read-only Bash command that references it). The producer (grounding-mcp) is the " +
-        "only legitimate writer.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--cwd <path>", "override cwd resolution (default: stdin event.cwd then process.cwd())")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.cwd)
-            cliOpts.cwd = options.cwd;
-        const result = await runPackHookSolutionAcceptanceWriteguardCli(cliOpts);
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("post-merge-gate-record")
-        .description("PostToolUse producer for the post-merge-gate pack: read tool-event JSON from stdin and, on a " +
-        "`gh pr merge` Bash call whose tool_output.exit_code is the number 0, record a " +
-        "`post-merge-gate:merged:<repo>:<branch>:<sha>` fact to the evidence ledger. blocking:false; every " +
-        "non-match / failure path is a no-op.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--cwd <path>", "override cwd resolution (default: stdin event.cwd then process.cwd())")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.cwd)
-            cliOpts.cwd = options.cwd;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        await runPackHookPostMergeGateRecordCli(cliOpts);
-    });
-    packHookCmd
-        .command("post-merge-gate")
-        .description("PreToolUse blocker for the post-merge-gate pack: read tool-event JSON from stdin, and on a curated " +
-        "history-mutating Bash command (git commit/add/push/merge/rebase/cherry-pick/revert/reset/stash " +
-        "pop|apply, gh pr create/merge) emit a deny envelope when the current branch tip matches a recorded " +
-        "merged-tip fact. The recovery vocabulary (git switch/checkout/pull/fetch, git branch -d/-D, git stash " +
-        "list/show, any `harness ...` command) always passes as its own command, classified before any manifest " +
-        "or ledger access; chaining one of those with a curated mutation does NOT exempt the mutation. Fails " +
-        "open when the ledger is unreachable.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--cwd <path>", "override cwd resolution (default: stdin event.cwd then process.cwd())")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.cwd)
-            cliOpts.cwd = options.cwd;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        const result = await runPackHookPostMergeGateCli(cliOpts);
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("runtime-reality")
-        .description("PreToolUse drift gate: read tool-event JSON from stdin, run the operator-configured " +
-        "RUNTIME_REALITY_PROBE_CMD to capture actual runtime state, compare against the " +
-        "RUNTIME_REALITY_KEYWORD expectations file, and emit a deny envelope on critical drift. " +
-        "Env-driven (no manifest options); fail-open on any probe / load error.")
-        .action(async () => {
-        const result = await runPackHookRuntimeRealityCli();
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    packHookCmd
-        .command("codex-stop")
-        .description("Codex Stop-equivalent: parse the agent's last message for an Understanding Report and persist it under .understanding-gate/reports/ as approvalStatus:pending. Failure modes resolve to exit 0 (capture must never block the agent's stop path).")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--pack <name>", "pack name to evaluate (default: understanding-before-execution)")
-        .option("--reports-dir <path>", "override the persisted-report directory (default: ./.understanding-gate/reports)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.pack)
-            cliOpts.pack = options.pack;
-        if (options.reportsDir)
-            cliOpts.reportsDir = options.reportsDir;
-        // Intentional: codex-stop's contract is fail-open. The runner
-        // returns exitCode 0 on every code path (including malformed
-        // input, missing session, parser misses); we deliberately do
-        // NOT throw HarnessExitError on a non-zero exit the way the
-        // codex-pre-tool-use sibling does. A future runner change
-        // that introduces a non-zero exit must also revisit this
-        // contract.
-        await runPackHookCodexStopCli(cliOpts);
-    });
-    packCmd
-        .command("list")
-        .description("Print policy_packs entries as a flat table or JSON.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--enabled-only", "skip entries with enabled: false")
-        .option("--json", "emit JSON array instead of an aligned text table")
-        .action((options) => {
-        const result = packList({
-            ...(options.config !== undefined ? { configPath: options.config } : {}),
-            ...(options.project !== undefined ? { project: options.project } : {}),
-            ...(options.enabledOnly === true ? { enabledOnly: true } : {}),
-            ...(options.json === true ? { json: true } : {}),
-        });
-        stdout(result.output);
-    });
-    // `harness approve` (Phase 6 #4): operator-driven approval verbs.
-    // Today only `understanding` is implemented; other packs can plug in
-    // sister sub-commands (e.g. `harness approve preflight`) without
-    // restructuring this surface.
-    const approveCmd = program
-        .command("approve")
-        .description("Operator-driven approval verbs (writes evidence-ledger tags + flips persisted artefacts)");
-    approveCmd
-        .command("understanding")
-        .description("Mark the latest Understanding Report as approved AND write the evidence-ledger tag. " +
-        "Writes the signed approval marker and round-trips the audit records (persisted report, ledger row) " +
-        "so harnessed and solo (@lannguyensi/understanding-gate) stacks stay in sync.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then $CODEX_SESSION_ID, then staged .pending-approval, then newest pending Understanding Report)")
-        .option("--task <ids...>", "agent-tasks task id(s): writes one task-scoped marker per id. Pass several (--task a b c, or --task a,b,c) to pre-approve a whole batch in a single operator action so a multi-task session does not re-prompt for each task it claims (outside approval_lifecycle mode: session the session approval covers only the task claimed when it was granted, and a done, abandoned or merged task expires it) (harness/1ee26e77, harness/0dce3880, harness 5018c0c4)")
-        .option("--reports-dir <path>", "override the persisted-report directory (default: ./.understanding-gate/reports)")
-        .option("--approved-by <actor>", "actor to record on the persisted report (default: harness-approve-cli)")
-        .option("--force", "bypass approve-time report validation (priorArt enforcement on grill_me reports). Writes the marker / ledger / report-flip anyway and stamps the ledger tag with `:forced:<field>` so audit can distinguish forced approvals from clean ones. Emergency-unblock path: default refuses the marker when a grill_me report fails validation. Also overrides the nesting-depth refusal (the marker is then signed with no content binding); never overrides the size refusal (also on the pretty-printed approval rewrite), an unreadable report, or one that is not a regular file.")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.session)
-            cliOpts.session = options.session;
-        // Commander's variadic `<ids...>` yields a string[]; comma-split
-        // and de-dup happen inside approveUnderstanding (dedupeTaskIds).
-        if (options.task && options.task.length > 0)
-            cliOpts.tasks = options.task;
-        if (options.reportsDir)
-            cliOpts.reportsDir = options.reportsDir;
-        if (options.approvedBy)
-            cliOpts.approvedBy = options.approvedBy;
-        if (options.force)
-            cliOpts.force = true;
-        // Report capture (task 61fd36db): the agent attaches the
-        // Understanding Report as a quoted heredoc on stdin. A TTY stdin
-        // means an interactive operator shell with nothing piped — skip
-        // the read entirely so the CLI never sits waiting for input.
-        // Incomplete input (timeout, error, size cap) is refused rather
-        // than captured: a truncated-but-parseable report must never be
-        // persisted and approved as if it were whole.
-        let stdinIncomplete = false;
-        if (!process.stdin.isTTY) {
-            const piped = await readPipedStdin(process.stdin);
-            if (piped.text.trim().length > 0) {
-                if (piped.complete)
-                    cliOpts.reportMarkdown = piped.text;
-                else
-                    stdinIncomplete = true;
-            }
-        }
-        const result = await approveUnderstanding(cliOpts);
-        const lines = [];
-        // Annotate non-explicit session sources so the operator can spot
-        // a wrong id before it lands in the ledger.
-        const sourceNote = result.sessionSource === "pending-approval"
-            ? " (resolved from .pending-approval staged by the gate hook)"
-            : result.sessionSource === "env-claude-code"
-                ? " (from $CLAUDE_CODE_SESSION_ID)"
-                : result.sessionSource === "env-claude"
-                    ? " (from $CLAUDE_SESSION_ID)"
-                    : result.sessionSource === "env-codex"
-                        ? " (from $CODEX_SESSION_ID)"
-                        : result.sessionSource === "newest-report"
-                            ? " (GUESSED from the newest pending Understanding Report)"
-                            : "";
-        lines.push(`session: ${escapeForDisplay(result.sessionId)}${sourceNote}`);
-        if (result.sessionSource === "newest-report") {
-            // Tier-5 is a guess: no --session, no env var, no gate-staged
-            // .pending-approval. It is restricted to `pending` reports
-            // (harness/56f51f2b), but a stale session that left a never-
-            // approved report can still be picked. Name the report file
-            // so the operator can open it and confirm the id before
-            // trusting a marker that may approve the wrong session.
-            lines.push("");
-            lines.push("⚠ WARNING: the session id was GUESSED, not confirmed. There was no");
-            lines.push("  --session flag, no $CLAUDE_CODE_SESSION_ID / $CLAUDE_SESSION_ID /");
-            lines.push("  $CODEX_SESSION_ID, and no");
-            lines.push("  gate-staged .pending-approval, so it was read from the newest pending");
-            lines.push("  Understanding Report:");
-            if (result.newestReportPath) {
-                lines.push(`    ${escapeForDisplay(result.newestReportPath)}`);
-            }
-            lines.push("  If that is not your live session, the marker above approves the wrong");
-            lines.push("  session and the gate stays blocked. Confirm the id matches the running");
-            lines.push("  agent ($CLAUDE_CODE_SESSION_ID / $CLAUDE_SESSION_ID / $CODEX_SESSION_ID); if it differs, re-run");
-            lines.push("  with --session <live-id>.");
-        }
-        if (result.modeWarning) {
-            lines.push(`mode:    ⚠ ${result.modeWarning}`);
-        }
-        if (result.marker.ok) {
-            lines.push(`marker:  ✓ ${escapeForDisplay(result.marker.filePath)} (canonical gate signal)`);
-        }
-        else {
-            lines.push(`marker:  ✗ FAILED (${escapeForDisplay(result.marker.reason)})`);
-            lines.push("  the gate WILL block the next tool call until the marker exists.");
-        }
-        for (const tm of result.taskMarkers) {
-            const sourceNote = tm.source === "active-claim"
-                ? " (auto-resolved from active-claim file)"
-                : "";
-            if (tm.ok) {
-                lines.push(`task:    ✓ ${tm.filePath} (task-scoped, expires when this task ends)${sourceNote}`);
-            }
-            else {
-                lines.push(`task:    ✗ FAILED for task ${tm.taskId}${sourceNote} (${tm.reason})`);
-                lines.push("  the session marker above is still in effect; the task-scoped path is degraded.");
-            }
-        }
-        if (result.taskMarkers.length > 1) {
-            const okCount = result.taskMarkers.filter((t) => t.ok).length;
-            lines.push(`         (${okCount}/${result.taskMarkers.length} task markers written — the batch is pre-approved)`);
-        }
-        if (result.ledger.ok) {
-            lines.push(`ledger:  ✓ wrote ${escapeForDisplay(result.ledger.tag)} (audit only)`);
-        }
-        else {
-            lines.push(`ledger:  ⚠ skipped (${result.ledger.reason ?? "unknown"}) (audit only)`);
-        }
-        if (stdinIncomplete) {
-            lines.push("stdin:   ⚠ piped input arrived incomplete (timeout, stream error, or size cap) — report NOT captured.");
-            lines.push("  the approval itself proceeds below; re-run with the report as a quoted heredoc to persist the audit trail.");
-        }
-        if (result.stdinReport) {
-            if (result.stdinReport.ok) {
-                lines.push(`stdin:   ✓ report captured from stdin → ${result.stdinReport.filePath}`);
-            }
-            else {
-                lines.push(`stdin:   ⚠ report on stdin NOT captured (${result.stdinReport.reason})`);
-                if (result.stdinReport.parseErrorLogPath) {
-                    lines.push(`  raw text + parser reasons kept at ${result.stdinReport.parseErrorLogPath}`);
-                }
-                // Task 5d73d78d review MEDIUM (fix-round-3): this line used to
-                // print unconditionally, but since HIGH-2 a rejected stdin
-                // submission with nothing else to fall back on REFUSES the
-                // approval (marker.ok === false) — "the approval itself
-                // proceeds below" was then simply false. It only still holds
-                // when a genuine earlier same-session report (the HIGH-2
-                // carve-out) governs approval instead, i.e. when the marker
-                // WAS written despite this submission being rejected.
-                if (result.marker.ok) {
-                    lines.push("  the approval itself proceeds below; re-run with a schema-conform report to persist the audit trail.");
-                }
-                else {
-                    lines.push("  the approval is REFUSED — no marker, no ledger tag, no report flip; see `validation` below for why.");
-                }
-            }
-        }
-        if (result.persistedReport.ok) {
-            const prev = result.persistedReport.previousStatus ?? "<missing>";
-            const stampNote = result.persistedReport.sessionIdStamped
-                ? "; stamped sessionId"
-                : "";
-            lines.push(`report:  ✓ ${escapeForDisplay(result.persistedReport.filePath)} (approvalStatus: ${escapeForDisplay(prev)} → approved${stampNote})`);
-            const fb = result.persistedReport.fallbackAdopted;
-            if (fb) {
-                lines.push(`  ⚠ adopted via sessionId-less fallback: created ${escapeForDisplay(fb.createdAt ?? "<unknown>")} (${fb.ageMinutes}m ago).`);
-                lines.push("  The live session's report was never persisted, or an older producer", "  wrote it without a sessionId. Verify this is the report you just read", "  before trusting the approval.");
-            }
-        }
-        else {
-            lines.push(`report:  ⚠ skipped (${result.persistedReport.reason})`);
-        }
-        // Surface the validation outcome explicitly. Three cases:
-        //   - ok: report was loaded and structurally compliant for its mode.
-        //   - failure-enforced: short-circuited, no writes ran. Hard error.
-        //   - failure-forced: writes ran, ledger tag carries `:forced:`.
-        //   - skipped: no report loaded; nothing to validate.
-        const v = result.validation;
-        if ("ok" in v && v.ok) {
-            // mode === null is the legacy / pre-v0.4.0 case the validator
-            // waives by design (the schema bump must not retroactively
-            // reject historical reports). Distinguish it from a positive
-            // check so the operator does not read "passed" and assume the
-            // grill_me priorArt rule actually fired.
-            if (v.mode === null) {
-                lines.push("validation: ⓘ legacy report (no mode field) — priorArt rule waived");
-            }
-            else {
-                lines.push(`validation: ✓ ${escapeForDisplay(v.mode)} report passed structural checks`);
-            }
-        }
-        else if ("ok" in v && v.ok === false) {
-            if (v.enforced) {
-                lines.push(`validation: ✗ ${v.field} FAILED: ${v.reason}`);
-                lines.push("  the marker was NOT written; the gate stays closed.");
-                lines.push("  pass --force to bypass a content check (the ledger tag will be stamped `:forced:<field>` for audit); a report that is oversized, not a regular file or unreadable cannot be forced.");
-            }
-            else {
-                lines.push(`validation: ⚠ ${v.field} failed but --force bypassed (${v.reason})`);
-                lines.push("  the marker IS written; the ledger tag carries a `:forced:` suffix for audit.");
-            }
-        }
-        stdout(`${lines.join("\n")}\n`);
-        // Non-zero exit when validation refused the approval. Throwing
-        // HarnessExitError reaches the CLI entrypoint's catch-all and
-        // surfaces both the lines above and the chosen exit code so the
-        // operator sees the rejection and shell pipelines (`&&`,
-        // CI guards) can react.
-        if ("ok" in v && v.ok === false && v.enforced) {
-            throw new HarnessExitError(`approve refused: ${v.field} validation failed. ` +
-                `Run with --force to bypass a content check (the ledger tag will be stamped \`:forced:${v.field}\` for audit); a report that is oversized, not a regular file or unreadable cannot be forced.`, EX_FAIL);
-        }
-    });
-    approveCmd
-        .command("risk")
-        .description("Grant a Risk Gate require_approval decision (default), or deliberately override " +
-        "a deny-tier block with --force <reason>. The default path writes the " +
-        "risk-approved:${SESSION_ID} ledger tag the production-scoped require_approval " +
-        "policy's requires consults; --scope deletion writes " +
-        "risk-approved:deletion:${SESSION_ID} instead, for the dev-context deletion arm " +
-        "only; --force writes risk-override:${SESSION_ID}:forced:<reason> for the " +
-        "deny-tier policy. Operator action.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then $CODEX_SESSION_ID, then staged .pending-approval)")
-        .option("--scope <scope>", "restrict the ledger tag to one Risk Gate arm instead of the shared production tag; only 'deletion' is recognized (writes risk-approved:deletion:${SESSION_ID} for gate-dev-unsafe-deletion). Omit for the default production-scoped tag. Ignored with --force.")
-        .option("--force <reason>", "operator-deliberate override of a deny-tier Risk Gate block; writes the risk-override tag with the reason stamped into the audit trail")
-        .option("--i-am-the-operator", "acknowledge a scripted / non-TTY --force invocation (otherwise refused)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.session)
-            cliOpts.session = options.session;
-        if (options.scope !== undefined) {
-            if (options.scope !== "deletion") {
-                throw new HarnessExitError(`--scope "${options.scope}" is not recognized; the only supported value is "deletion".`, EX_USAGE);
-            }
-            cliOpts.scope = "deletion";
-        }
-        if (typeof options.force === "string") {
-            const reason = options.force.trim();
-            if (reason.length === 0) {
-                throw new HarnessExitError("--force requires a non-empty <reason>; the reason is recorded in the audit trail.", EX_USAGE);
-            }
-            cliOpts.force = { reason };
-        }
-        if (options.iAmTheOperator)
-            cliOpts.iAmTheOperator = true;
-        const result = await approveRisk(cliOpts);
-        const lines = [];
-        const sourceNote = result.sessionSource === "pending-approval"
-            ? " (resolved from .pending-approval staged by the gate hook)"
-            : result.sessionSource === "env-claude-code"
-                ? " (from $CLAUDE_CODE_SESSION_ID)"
-                : result.sessionSource === "env-claude"
-                    ? " (from $CLAUDE_SESSION_ID)"
-                    : result.sessionSource === "env-codex"
-                        ? " (from $CODEX_SESSION_ID)"
-                        : "";
-        lines.push(`session: ${result.sessionId}${sourceNote}`);
-        if (result.ledger.ok) {
-            lines.push(`ledger:  ✓ wrote ${result.ledger.tag}`);
-            if (result.forced) {
-                lines.push("  deny-tier override recorded; the deny-tier Risk Gate policy now passes for this session.");
-            }
-            else {
-                lines.push("  the Risk Gate require_approval policy now passes for this session.");
-            }
-        }
-        else {
-            lines.push(`ledger:  ✗ FAILED (${result.ledger.reason ?? "unknown"})`);
-            if (result.forced) {
-                lines.push("  the deny-tier override stays unrecorded; the gate keeps blocking.");
-            }
-            else {
-                lines.push("  the require_approval gate stays blocked until the tag is recorded.");
-            }
-        }
-        stdout(`${lines.join("\n")}\n`);
-    });
-    approveCmd
-        .command("branch-protection")
-        .description("Bless a deliberate protected-branch edit for one session. Writes the canonical " +
-        "operator-only approval marker under harness.generated/.approvals/ that the " +
-        "branch-protection blocker consults, plus a best-effort branch-protection-ack " +
-        "ledger row for audit. Operator action: the marker (not the ledger tag) is the " +
-        "trusted override, because the ledger is agent-writable.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then $CODEX_SESSION_ID, then staged .pending-approval)")
-        .option("--reason <text>", "free-form note recorded in the audit ledger tag (why the override fired)")
-        .option("--approved-by <actor>", "actor to record on the marker (default: harness-approve-cli)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.session)
-            cliOpts.session = options.session;
-        if (options.reason)
-            cliOpts.reason = options.reason;
-        if (options.approvedBy)
-            cliOpts.approvedBy = options.approvedBy;
-        const result = await approveBranchProtection(cliOpts);
-        const lines = [];
-        const sourceNote = result.sessionSource === "pending-approval"
-            ? " (resolved from .pending-approval staged by the gate hook)"
-            : result.sessionSource === "env-claude-code"
-                ? " (from $CLAUDE_CODE_SESSION_ID)"
-                : result.sessionSource === "env-claude"
-                    ? " (from $CLAUDE_SESSION_ID)"
-                    : result.sessionSource === "env-codex"
-                        ? " (from $CODEX_SESSION_ID)"
-                        : "";
-        lines.push(`session: ${result.sessionId}${sourceNote}`);
-        if (result.marker.ok) {
-            lines.push(`marker:  ✓ ${result.marker.filePath} (canonical gate signal)`);
-            lines.push("  the branch-protection gate now allows protected-branch edits for this session.");
-        }
-        else {
-            lines.push(`marker:  ✗ FAILED (${result.marker.reason})`);
-            lines.push("  the gate WILL keep blocking the next tool call until the marker exists.");
-        }
-        if (result.ledger.ok) {
-            lines.push(`ledger:  ✓ wrote ${result.ledger.tag} (audit only)`);
-        }
-        else {
-            lines.push(`ledger:  ⚠ skipped (${result.ledger.reason ?? "unknown"}) (audit only)`);
-        }
-        stdout(`${lines.join("\n")}\n`);
-    });
-    registerDelegate(program, { stdout, stderr });
-    const VALID_DECISION_FILTERS = [
-        "allow",
-        "warn",
-        "require_approval",
-        "deny",
-        "warn-degraded",
-        "deny-degraded",
-    ];
-    const isDecisionFilter = (v) => VALID_DECISION_FILTERS.includes(v);
-    program
-        .command("explain [policy]")
-        .description("Print a policy's definition; --trace reads the last recorded evaluation; --last traces the most recent decision in the ledger")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--json", "emit JSON instead of YAML")
-        .option("--trace", "include the full decision trail from the most recent evaluation")
-        .option("--last", "trace the most recent policy decision in the ledger (any policy); mutually exclusive with <policy>")
-        .option("--decision <outcome>", `with --last, restrict to decisions of this outcome (${VALID_DECISION_FILTERS.join(" / ")})`)
-        .option("--session <id>", "grounding session whose audit log to read (default: $CLAUDE_SESSION_ID, then 'default')")
-        .action(async (policyName, options) => {
-        if (options.last && policyName !== undefined) {
-            throw new HarnessExitError("explain: <policy> and --last are mutually exclusive", EX_USAGE);
-        }
-        if (options.decision !== undefined && !options.last) {
-            throw new HarnessExitError("explain: --decision requires --last", EX_USAGE);
-        }
-        if (options.decision !== undefined && !isDecisionFilter(options.decision)) {
-            throw new HarnessExitError(`explain: --decision must be one of ${VALID_DECISION_FILTERS.join(", ")} (got "${options.decision}")`, EX_USAGE);
-        }
-        const explainOpts = {};
-        if (options.config)
-            explainOpts.configPath = options.config;
-        if (options.project)
-            explainOpts.project = options.project;
-        if (options.json)
-            explainOpts.json = options.json;
-        if (options.trace)
-            explainOpts.trace = options.trace;
-        if (options.last)
-            explainOpts.last = options.last;
-        if (options.decision !== undefined && isDecisionFilter(options.decision)) {
-            explainOpts.decisionFilter = options.decision;
-        }
-        if (options.session)
-            explainOpts.sessionId = options.session;
-        const result = await explain(policyName, explainOpts);
-        stdout(result.output);
-    });
-    program
-        .command("explain-action <event.json>")
-        .description("Risk Gate debug verb (Phase 7): read a tool-event JSON file (the Claude Code PreToolUse hook payload shape: " +
-        "{ hook_event_name, tool_name, tool_input, session_id, cwd }) and print the normalized Action Envelope. " +
-        "Inspection surface for the envelope that downstream Risk Gate stages consume; does not evaluate policies.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml); read only for a leading `git switch|checkout`")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--json", "emit the envelope as JSON instead of YAML")
-        .action((eventPath, options) => {
-        const result = explainAction({
-            eventPath,
-            ...(options.config !== undefined && { configPath: options.config }),
-            ...(options.project !== undefined && { project: options.project }),
-            ...(options.json === true && { json: true }),
-        });
-        stdout(result.output);
-        if (!result.output.endsWith("\n"))
-            stdout("\n");
-    });
-    program
-        .command("test-risk <event.json>")
-        .description("Risk Gate debug verb (Phase 7): read a tool-event JSON file, build its Action Envelope, and classify it " +
-        "against the manifest's risk.classifiers[]. Prints the risk profile (severity, categories, reversibility, " +
-        "confidence, reasons). An action no pattern matches reports as unclassified, not as safe.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--json", "emit the risk profile as JSON instead of YAML")
-        .action((eventPath, options) => {
-        const result = testRisk({
-            eventPath,
-            ...(options.config !== undefined && { configPath: options.config }),
-            ...(options.project !== undefined && { project: options.project }),
-            ...(options.json === true && { json: true }),
-        });
-        stdout(result.output);
-        if (!result.output.endsWith("\n"))
-            stdout("\n");
-    });
-    program
-        .command("resolve-env <event.json>")
-        .description("Risk Gate debug verb (Phase 7): read a tool-event JSON file, build its Action Envelope, and resolve its " +
-        "target environment against the manifest's environments.resolvers[] (branch / env-var / kube-context / " +
-        "kube-namespace signals). An action no resolver matches resolves to `unknown`, not to a safe default.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--json", "emit the environment resolution as JSON instead of YAML")
-        .action((eventPath, options) => {
-        const result = resolveEnv({
-            eventPath,
-            ...(options.config !== undefined && { configPath: options.config }),
-            ...(options.project !== undefined && { project: options.project }),
-            ...(options.json === true && { json: true }),
-        });
-        stdout(result.output);
-        if (!result.output.endsWith("\n"))
-            stdout("\n");
-    });
-    program
-        .command("explain-policy <policy>")
-        .description("Risk Gate debug verb (Phase 7): explain whether <policy> would APPLY to a tool event. " +
-        "Reads the event from --event, builds and enriches the Action Envelope, and shows the " +
-        "trigger match, the risk classification, the resolved environment, and a per-clause " +
-        "`when:` breakdown. Evaluates a hypothetical event live and reads nothing from the " +
-        "ledger (use `harness explain <policy> --trace` for the last recorded decision).")
-        .requiredOption("--event <event.json>", "path to the tool-event JSON file")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--json", "emit the explanation as JSON instead of YAML")
-        .action((policyName, options) => {
-        const result = explainPolicy(policyName, {
-            eventPath: options.event,
-            ...(options.config !== undefined && { configPath: options.config }),
-            ...(options.project !== undefined && { project: options.project }),
-            ...(options.json === true && { json: true }),
-        });
-        stdout(result.output);
-        if (!result.output.endsWith("\n"))
-            stdout("\n");
-    });
-    registerAuditGroup(program, { stdout, stderr });
-    program
-        .command("smoke")
-        .description("Drive `claude -p` end-to-end against the apply'd manifest and assert per --expect-* flags. " +
-        "Writes stream.jsonl + stderr.log + settings.json under --output-dir; exits 1 on any expectation miss. " +
-        "Replaces the hand-rolled dogfood recipes under dogfood/phase5/.")
-        .requiredOption("--prompt <text>", "Prompt fed to claude -p")
-        .requiredOption("--output-dir <path>", "Directory for stream.jsonl + stderr.log + settings.json")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--session-id <id>", "session id (default: fresh uuid)")
-        .option("--claude-bin <path>", "claude binary (default: $CLAUDE_BIN, then 'claude' on PATH)")
-        .option("--timeout-ms <n>", "wall-clock budget in milliseconds (default: 60000)")
-        .option("--expect-hook <names>", "comma-separated list of hook names / events that MUST fire (repeatable)", (value, prev = []) => prev.concat(splitCommaList(value)), [])
-        .option("--expect-no-hook <names>", "comma-separated list of hook names / events that MUST NOT fire (repeatable)", (value, prev = []) => prev.concat(splitCommaList(value)), [])
-        .option("--expect-exit <n>", "expected result.is_error: 0 ⇒ false, !=0 ⇒ true")
-        .option("--expect-decision <kind>", "policy decision must be one of allow|deny|warn")
-        .option("--no-delegate", "do not pre-authorize the spawned child (docs/decisions/2026-08-27-ug-auto-mode-approval.md); " +
-        "it then has no delegation binding it to this approved session and falls back to the plain opt-in path")
-        .action(async (options) => {
-        const expectations = {};
-        if (options.expectHook && options.expectHook.length > 0) {
-            expectations.expectHooks = options.expectHook;
-        }
-        if (options.expectNoHook && options.expectNoHook.length > 0) {
-            expectations.expectNoHooks = options.expectNoHook;
-        }
-        if (options.expectExit !== undefined) {
-            const n = Number.parseInt(options.expectExit, 10);
-            if (!Number.isFinite(n)) {
-                throw new HarnessExitError(`harness smoke: --expect-exit must be an integer (got "${options.expectExit}")`, EX_USAGE);
-            }
-            expectations.expectExit = n;
-        }
-        if (options.expectDecision !== undefined) {
-            expectations.expectDecision = options.expectDecision;
-        }
-        const smokeOpts = {
-            prompt: options.prompt,
-            outputDir: options.outputDir,
-            expectations,
-        };
-        // commander's `--no-delegate` negates `options.delegate` (default
-        // true); only an explicit `--no-delegate` flips it to `false`.
-        if (options.delegate === false)
-            smokeOpts.noDelegate = true;
-        if (options.config)
-            smokeOpts.configPath = options.config;
-        if (options.project)
-            smokeOpts.project = options.project;
-        if (options.sessionId)
-            smokeOpts.sessionId = options.sessionId;
-        if (options.claudeBin)
-            smokeOpts.claudeBin = options.claudeBin;
-        if (options.timeoutMs !== undefined) {
-            const n = Number.parseInt(options.timeoutMs, 10);
-            if (!Number.isFinite(n) || n <= 0) {
-                throw new HarnessExitError(`harness smoke: --timeout-ms must be a positive integer`, EX_USAGE);
-            }
-            smokeOpts.timeoutMs = n;
-        }
-        const result = await runSmoke(smokeOpts);
-        stdout(formatSmokeReport(result));
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    });
-    // The action shared by `harness session-start preflight` and its
-    // top-level alias `harness preflight`: the alias delegates to the same
-    // implementation, with the same CLI options.
-    const preflightAction = async (options) => {
-        const cliOpts = {};
-        applyCliOptions(options, cliOpts);
-        if (options.timeout) {
-            const n = Number.parseInt(options.timeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.preflightTimeoutMs = n;
-        }
-        // Opt into the bootstrap-staging side effect from the CLI entry
-        // point only. Library callers (vitest cases) get the no-op default
-        // so they cannot clobber the operator's real pending-approval file.
-        cliOpts.stagePendingApproval = writePendingApproval;
-        await runSessionStartPreflight(cliOpts);
-    };
-    // Top-level alias for `harness session-start preflight`, so the
-    // policy `ux.run:` field can show the short form the agent should
-    // type: `Run: harness preflight`.
-    addLedgerTimeoutOption(addIdentityOptions(program
-        .command("preflight")
-        .description("Alias for `harness session-start preflight`: run agent-preflight against the session cwd " +
-        "and, on a ready:true result, record a `preflight:${REPO}` fact to the evidence ledger. " +
-        "Opt-in `session_start_preflight.setup: true` (default off) passes --setup through; " +
-        "see docs/CLI.md for the trust and scope caveats."), "explicit session id (overrides stdin event + env)").option("--timeout <ms>", "agent-preflight subprocess timeout in milliseconds (default 60000)")).action(preflightAction);
-    // Shared by the three `record` verbs' action handlers below: parse
-    // `--ledger-timeout <ms>` into `cliOpts.ledgerTimeoutMs`, and report a
-    // `RecordResult` (print the recorded fact on success; on failure, throw
-    // with the runner's own exit code and an EMPTY message, since the
-    // runner already wrote the reason to stderr itself — throwing the same
-    // text again would double-print it).
-    //
-    // An unparseable / non-positive value used to fall back to the
-    // default silently (review finding, T-004): an operator who typo'd
-    // `--ledger-timeout 5ooo` got the default timeout with zero
-    // diagnostic. Now a malformed value warns once on stderr before
-    // falling back, same "never a silent gap" convention `resolveBase`'s
-    // own degrade path already uses below.
-    const applyLedgerTimeout = (raw, cliOpts) => {
-        if (!raw)
-            return;
-        const n = Number.parseInt(raw, 10);
-        if (Number.isFinite(n) && n > 0) {
-            cliOpts.ledgerTimeoutMs = n;
-            return;
-        }
-        stderr(`harness record: --ledger-timeout ${JSON.stringify(raw)} is not a positive integer; using the default timeout.\n`);
-    };
-    const reportRecordResult = (result) => {
-        if (result.wrote) {
-            stdout(`recorded ${result.content} for session ${result.sessionId}\n`);
-        }
-        if (result.exitCode !== 0) {
-            throw new HarnessExitError("", result.exitCode);
-        }
-    };
-    // `harness record {review,review-subagent,dogfood}` (task T-001):
-    // evidence-ledger producers for the review-before-merge,
-    // review-subagent-before-pr-create, and dogfood-before-release gate
-    // families (see src/cli/init/templates.ts for the exact policies).
-    // Unlike the `preflight` / `session-start preflight` pair above,
-    // these are NOT hooks: they are invoked deliberately by an agent or
-    // operator, so a failure exits non-zero with a clear stderr message
-    // (written by the runner itself) rather than degrading silently.
-    const recordCmd = program
-        .command("record")
-        .description("Evidence-ledger producers for the review / review-subagent / dogfood gate families. " +
-        "Interactive verbs (not hooks): a failure exits non-zero.");
-    recordCmd
-        .command("review <summary>")
-        .description("Record a review:${PR_NUMBER} + review:${BRANCH} (+ review:${BASE}, + review:${TASK_ID} " +
-        "with --task) fact for the review-before-merge / review-before-merge-bash / " +
-        "review-before-task-merge / review-before-task-finish-automerge gates.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .requiredOption("--pr <number>", "PR number the review:${PR_NUMBER} tag is namespaced by")
-        .option("--base <branch>", "base branch for the review:${BASE} tag. Default: the remote's default branch read from " +
-        "refs/remotes/origin/HEAD (packed-refs fallback included); omitted with a stderr warning " +
-        "when neither resolves. No `gh` shell-out.")
-        .option("--branch <name>", "explicit branch override for review:${BRANCH} (default: current git branch)")
-        .option("--task <id>", "agent-tasks task id for the review:${TASK_ID} tag the task_merge / task_finish(autoMerge) gates read. " +
-        "Optional; pass it and one recorded review satisfies all four merge surfaces.")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then newest Claude Code transcript)")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .action(async (summary, options) => {
-        const cliOpts = { pr: options.pr, summary };
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.base)
-            cliOpts.base = options.base;
-        if (options.branch)
-            cliOpts.branch = options.branch;
-        if (options.task !== undefined)
-            cliOpts.task = options.task;
-        if (options.session)
-            cliOpts.session = options.session;
-        applyLedgerTimeout(options.ledgerTimeout, cliOpts);
-        reportRecordResult(await runRecordReview(cliOpts));
-    });
-    recordCmd
-        .command("review-subagent [summary]")
-        .description("Record a review-subagent:${TASK_ID} + review-subagent:${BRANCH} fact for the " +
-        "review-subagent-before-pr-create / review-subagent-before-pr-create-bash gates. " +
-        "Exactly one of --task or --adhoc is required; --adhoc (work without an agent-tasks " +
-        "task) writes review-subagent:${BRANCH} only.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--task <id>", "agent-tasks task id the review-subagent:${TASK_ID} tag is namespaced by (exactly one of --task or --adhoc)")
-        .option("--adhoc", "record for work without an agent-tasks task: writes review-subagent:${BRANCH} only, no task tag (exactly one of --task or --adhoc)")
-        .requiredOption("--verdict <text>", "reviewer verdict recorded in the fact content")
-        .option("--branch <name>", "explicit branch override for review-subagent:${BRANCH} (default: current git branch)")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then newest Claude Code transcript)")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .action(async (summary, options) => {
-        const cliOpts = {
-            verdict: options.verdict,
-        };
-        if (options.task !== undefined)
-            cliOpts.task = options.task;
-        if (options.adhoc)
-            cliOpts.adhoc = true;
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.branch)
-            cliOpts.branch = options.branch;
-        if (options.session)
-            cliOpts.session = options.session;
-        if (summary)
-            cliOpts.summary = summary;
-        applyLedgerTimeout(options.ledgerTimeout, cliOpts);
-        reportRecordResult(await runRecordReviewSubagent(cliOpts));
-    });
-    recordCmd
-        .command("dogfood <summary>")
-        .description("Record a dogfood:${SESSION_ID} fact for the dogfood-before-release gate.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--session <id>", "explicit session id (default: $CLAUDE_CODE_SESSION_ID, then $CLAUDE_SESSION_ID, then newest Claude Code transcript)")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .action(async (summary, options) => {
-        const cliOpts = { summary };
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.session)
-            cliOpts.session = options.session;
-        applyLedgerTimeout(options.ledgerTimeout, cliOpts);
-        reportRecordResult(await runRecordDogfood(cliOpts));
-    });
-    const sessionStart = program
-        .command("session-start")
-        .description("SessionStart hook entrypoints (called by Claude Code via settings.json)");
-    addLedgerTimeoutOption(addIdentityOptions(sessionStart
-        .command("preflight")
-        .description("SessionStart producer: run agent-preflight against the session cwd and, on a ready:true result, " +
-        "record a `preflight:${REPO}` fact to the evidence ledger so the preflight-before-* policies have a " +
-        "fresh tag to match. Reads SessionStart event JSON from stdin ({ session_id, cwd, hook_event_name }). " +
-        "Opt-in `session_start_preflight.setup: true` (default off) passes --setup through; " +
-        "see docs/CLI.md for the trust and scope caveats. " +
-        "blocking:false \u2014 every failure path logs to stderr and exits 0."), "explicit session id (overrides stdin event + env). Use for manual / scripted invocations " +
-        "where no SessionStart event JSON is piped on stdin. Without it the resolver tries " +
-        "stdin event → $CLAUDE_SESSION_ID → newest Claude Code transcript → 'default' (which logs " +
-        "a loud warning since the literal 'default' session never satisfies a preflight-before-* gate).").option("--timeout <ms>", "agent-preflight subprocess timeout in milliseconds (default 60000)")).action(preflightAction);
-    // The three advisory producers below share one option set and one
-    // action shape: only the description and the runner differ.
-    const addSessionStartProducer = (name, description, run) => {
-        addLedgerTimeoutOption(addCwdOption(addIdentityOptions(sessionStart.command(name).description(description), "explicit session id (overrides stdin event + env)"))).action(async (options) => {
-            const cliOpts = {};
-            applyCliOptions(options, cliOpts);
-            await run(cliOpts);
-        });
-    };
-    addSessionStartProducer("branch-check", "SessionStart producer for the branch-protection pack: read .git/HEAD for the session cwd and, " +
-        "when the branch is NOT in the operator's protected list (default: master, main, develop), " +
-        "record a `branch:non-protected:<branch>` fact to the evidence ledger so the pack's PreToolUse " +
-        "blocker has a fresh tag to satisfy its 5-minute freshness window. Also runnable on demand from " +
-        "the operator's shell. blocking:false \u2014 every failure path logs to stderr and exits 0.", runSessionStartBranchCheck);
-    addSessionStartProducer("toolchain-parity", "SessionStart producer (opt-in via `toolchain_parity.enabled: true`): writes THIS machine's " +
-        "toolchain snapshot (node version, npm globals, OW-Kit version, MCP server names) to " +
-        "`<machine_state_dir>/<profile>.json`, compares it against every OTHER snapshot file already " +
-        "in that directory, and records a `toolchain-parity:ok` / `toolchain-parity:drift:<n>` fact " +
-        "(with a `:unparseable-peer:<n>` suffix whenever a peer file failed to parse as JSON, so an " +
-        "unparseable peer never silently vanishes from the comparison) to the evidence ledger. " +
-        "Purely advisory \u2014 never blocking, and never touches a peer's file. " +
-        "Cross-machine transport of the snapshot files is agent-memory-sync's job, not this command's.", runSessionStartToolchainParity);
-    addSessionStartProducer("stale-base-check", "SessionStart producer (opt-in via `stale_base_check.enabled: true`; task ce3903b0, incident " +
-        "ea8becf5): runs a LIVE `git fetch` of the remote default branch (never trusting the local " +
-        "origin/<default> ref, which can itself be stale \u2014 that is the exact bug this closes) and, when " +
-        "the current branch's base is behind, writes a WARNING to stderr naming how many commits behind, " +
-        "how old the missing work is, and the recovery command. Records a `stale-base:ok` / " +
-        "`stale-base:behind:<n>` fact to the evidence ledger (audit-only \u2014 no gate consumes it). " +
-        "Purely advisory: never blocks, and degrades cleanly (no fact written) when offline, the remote " +
-        "or default branch can't be resolved, or credentials are missing. blocking:false \u2014 every failure " +
-        "path logs to stderr and exits 0.", runSessionStartStaleBaseCheck);
-    // `harness gate` — operator escape hatch for hard-blocking hooks.
-    // Task 8fcddb26: the understanding-before-execution PreToolUse hook can
-    // lock a Claude session out of every Bash call, and the recommended
-    // recovery (`harness approve understanding`) is itself a Bash invocation
-    // and gets caught by the same gate. `gate disable` strips the
-    // offending hook group out of settings.json with a reversible snapshot.
-    const gate = program
-        .command("gate")
-        .description("Operator escape hatch: disable/restore hook groups in ~/.claude/settings.json");
-    gate
-        .command("disable")
-        .description("Remove hook groups from ~/.claude/settings.json whose `matcher` substring-matches " +
-        "`--matcher <pattern>`. Writes a snapshot of the removed groups next to settings.json " +
-        "(`harness.gate-disable.<ts>.json`) and backs up the original to `settings.json.bak.<ts>` " +
-        "so `harness gate enable` can restore them. With no `--matcher` flag, lists the candidate " +
-        "groups without writing.")
-        .option("--matcher <pattern>", "remove groups whose matcher includes this substring")
-        .option("--settings <path>", "override ~/.claude/settings.json")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.matcher)
-            cliOpts.matcher = options.matcher;
-        if (options.settings)
-            cliOpts.settingsPath = options.settings;
-        try {
-            const result = gateDisable(cliOpts);
-            if (result.mode === "list") {
-                if (result.candidates.length === 0) {
-                    stdout(`no hook groups in ${result.settingsPath}; nothing to disable.\n`);
-                    return;
-                }
-                stdout(`candidate hook groups in ${result.settingsPath}:\n`);
-                for (const c of result.candidates) {
-                    const matcherLabel = c.matcher === null ? "(no matcher)" : JSON.stringify(c.matcher);
-                    stdout(`  ${c.event}[${c.index}] matcher=${matcherLabel}: ${c.description}\n`);
-                }
-                stdout(`\nPass --matcher <substring> to remove a group. Removal is reversible via ` +
-                    `\`harness gate enable\`.\n`);
-                return;
-            }
-            stdout(`disabled ${result.removed.length} hook group(s) in ${result.settingsPath}:\n`);
-            for (const r of result.removed) {
-                const matcherLabel = r.group !== null && typeof r.group === "object" && !Array.isArray(r.group)
-                    ? JSON.stringify(r.group["matcher"] ?? null)
-                    : "null";
-                stdout(`  ${r.event}[${r.index}] matcher=${matcherLabel}\n`);
-            }
-            stdout(`backup:   ${result.backupPath}\n`);
-            stdout(`snapshot: ${result.snapshotPath}\n`);
-            stdout(`This is reversible: run \`harness gate enable\` to restore.\n`);
-        }
-        catch (err) {
-            if (err instanceof GateDisableError) {
-                throw new HarnessExitError(err.message, EX_FAIL);
-            }
-            throw err;
-        }
-    });
-    gate
-        .command("enable")
-        .description("Restore hook groups from the newest `harness.gate-disable.*.json` snapshot next to " +
-        "settings.json. Refuses if settings.json has been edited since the snapshot was taken " +
-        "(use `--force` to restore anyway). Idempotent: if settings.json already matches the " +
-        "pre-disable state, exits 0 without writing.")
-        .option("--settings <path>", "override ~/.claude/settings.json")
-        .option("--force", "restore even when settings.json has been edited since the snapshot")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.settings)
-            cliOpts.settingsPath = options.settings;
-        if (options.force)
-            cliOpts.force = true;
-        try {
-            const result = gateEnable(cliOpts);
-            if (result.mode === "no-snapshots") {
-                stdout(`no gate-disable snapshots found next to ${result.settingsPath}; nothing to restore.\n`);
-                return;
-            }
-            if (result.mode === "already-restored") {
-                stdout(`settings.json already matches the pre-disable state; no write needed. ` +
-                    `(snapshot: ${result.snapshotPath})\n`);
-                return;
-            }
-            stdout(`restored ${result.restoredCount} hook group(s) into ${result.settingsPath} from ` +
-                `${result.snapshotPath}.\n`);
-        }
-        catch (err) {
-            if (err instanceof GateEnableError) {
-                throw new HarnessExitError(err.message, EX_FAIL);
-            }
-            throw err;
-        }
-    });
-    program
-        .command("gc")
-        .description("Retention-based cleanup of harness-owned gate state: terminal " +
-        "(approved/expired) understanding-gate reports, parse-error logs, " +
-        "approval markers, expired delegation markers, orphaned delegation " +
-        "adoption ledgers, and stale permission-mode observations older than " +
-        "the retention window, plus stale in-flight subagent records (fixed " +
-        "24h window, independent of --retention-days). Pending reports and " +
-        "anything outside the enumerated harness-owned dirs are never " +
-        "touched (the evidence ledger and solution-acceptance verdict dirs " +
-        "are owned by their producers). Dry-run by default; pass --apply " +
-        "to delete.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--retention-days <n>", `delete artifacts older than this many days (default: ${DEFAULT_RETENTION_DAYS})`)
-        .option("--apply", "delete the listed artifacts (default: dry-run listing only)")
-        .action((options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.apply)
-            cliOpts.apply = true;
-        if (options.retentionDays !== undefined) {
-            const parsed = Number(options.retentionDays);
-            if (!Number.isFinite(parsed) || parsed < 1) {
-                stderr(`--retention-days must be a positive number, got ${JSON.stringify(options.retentionDays)}\n`);
-                throw new HarnessExitError("", EX_USAGE);
-            }
-            cliOpts.retentionDays = parsed;
-        }
-        const result = gc(cliOpts);
-        const sweptDirs = [
-            result.reportsDir,
-            ...(result.parseErrorsDir !== null ? [result.parseErrorsDir] : []),
-            result.approvalsDir,
-            result.delegationsDir,
-            result.adoptionLedgerDir,
-            result.permissionModeObservationsDir,
-            result.inflightRecordsDir,
-        ];
-        if (result.parseErrorsDir === null) {
-            stderr("gc: skipping the parse-errors sweep (reports dir does not have the conventional .understanding-gate/reports shape)\n");
-        }
-        if (result.unparseable.length > 0) {
-            stderr(`gc: ${result.unparseable.length} file(s) could not be parsed and were left in place:\n` +
-                result.unparseable.map((u) => `  [${u.category}] ${escapeForDisplay(u.filePath)} (${u.reason})\n`).join(""));
-        }
-        if (result.candidates.length === 0) {
-            stdout(`gc: nothing older than ${result.retentionDays}d (cutoff ${result.cutoffIso}) under\n` +
-                sweptDirs.map((d) => `  ${d}\n`).join("") +
-                `${result.keptCount} artifact(s) inspected and kept.\n`);
-            return;
-        }
-        const verb = result.applied ? "removing" : "would remove";
-        stdout(`gc: ${verb} ${result.candidates.length} artifact(s) older than ${result.retentionDays}d (cutoff ${result.cutoffIso}); keeping ${result.keptCount}:\n`);
-        for (const c of result.candidates) {
-            stdout(`  [${c.category}] ${escapeForDisplay(c.filePath)} (${c.reason})\n`);
-        }
-        if (!result.applied) {
-            stdout(`\nDry-run; pass --apply to delete.\n`);
-            return;
-        }
-        stdout(`removed ${result.removed.length} file(s).\n`);
-        if (result.failures.length > 0) {
-            for (const f of result.failures) {
-                stderr(`gc: failed to remove ${escapeForDisplay(f.filePath)}: ${escapeForDisplay(f.reason)}\n`);
-            }
-            throw new HarnessExitError(`${result.failures.length} deletion(s) failed`, EX_FAIL);
-        }
-    });
+    registerPackGroup(program, { stdout, stderr });
+    registerGateGroup(program, { stdout, stderr });
     registerUninstall(program, { stdout, stderr });
     registerOperatorLifecycle(program, { stdout, stderr });
-    const policy = program.command("policy").description("Policy runtime verbs");
-    policy
-        .command("intercept")
-        .description("PreToolUse hook entrypoint: read tool-event JSON from stdin, evaluate matching policies, emit Claude Code deny JSON on block. " +
-        "Stdin shape (per Claude Code hook protocol): " +
-        "{ session_id, hook_event_name, tool_name, tool_input, cwd?, transcript_path? }. " +
-        "hook_event_name is required for any policy to match; if missing or unmatched, a one-line diagnostic is written to stderr.")
-        .option("--config <path>", "manifest path (default: ~/.harness/harness.yaml; legacy fallback ~/.claude/harness.yaml)")
-        .option("--project <name>", "apply per-project overrides")
-        .option("--ledger-timeout <ms>", "per-call ledger timeout in milliseconds")
-        .option("--verbose", "emit a stderr diagnostic block for each non-allow decision (also enabled by HARNESS_POLICY_VERBOSE=1)")
-        .option("--hook <name>", "manifest hook name (injected by `harness apply` for the Codex projection so failure logs identify which hook fired)")
-        .action(async (options) => {
-        const cliOpts = {};
-        if (options.config)
-            cliOpts.configPath = options.config;
-        if (options.project)
-            cliOpts.project = options.project;
-        if (options.ledgerTimeout) {
-            const n = Number.parseInt(options.ledgerTimeout, 10);
-            if (Number.isFinite(n) && n > 0)
-                cliOpts.ledgerTimeoutMs = n;
-        }
-        if (options.verbose)
-            cliOpts.verbose = options.verbose;
-        if (options.hook)
-            cliOpts.hookName = options.hook;
-        await runInterceptCli(cliOpts);
-    });
     return program;
 }
 export async function run(opts = {}) {