sentinelayer-cli 0.6.2 → 0.8.1

package/src/commands/review.js

@@ -1,704 +1,704 @@
-import fsp from "node:fs/promises";
-import path from "node:path";
-import process from "node:process";
-
-import pc from "picocolors";
-
-import { formatIngestResolutionNotice, resolveCodebaseIngest } from "../ingest/engine.js";
-import { runAiReviewLayer } from "../review/ai-review.js";
-import {
-  formatFindingsMarkdown,
-  runDeterministicReviewPipeline,
-  runLocalReviewScan,
-  writeReviewReport,
-} from "../review/local-review.js";
-import {
-  buildUnifiedReviewReport,
-  exportUnifiedReviewReport,
-  loadUnifiedReviewReport,
-  recordReviewDecision,
-  writeUnifiedReviewArtifacts,
-} from "../review/report.js";
-import {
-  compareUnifiedReports,
-  loadReviewRunContext,
-  writeReviewComparisonArtifact,
-  writeReviewRunContext,
-} from "../review/replay.js";
-
-function shouldEmitJson(options, command) {
-  const local = Boolean(options && options.json);
-  const globalFromCommand =
-    command && command.optsWithGlobals ? Boolean(command.optsWithGlobals().json) : false;
-  return local || globalFromCommand;
-}
-
-function resolveModeFromOptions(options = {}, { defaultMode = "full" } = {}) {
-  const explicitMode = String(options.mode || "").trim().toLowerCase();
-  const diffFlag = Boolean(options.diff);
-  const stagedFlag = Boolean(options.staged);
-
-  if (diffFlag && stagedFlag) {
-    throw new Error("Use only one of --diff or --staged.");
-  }
-
-  let modeFromFlags = "";
-  if (diffFlag) {
-    modeFromFlags = "diff";
-  } else if (stagedFlag) {
-    modeFromFlags = "staged";
-  }
-
-  if (explicitMode && modeFromFlags && explicitMode !== modeFromFlags) {
-    throw new Error(`Conflicting mode selection: --mode ${explicitMode} with --${modeFromFlags}.`);
-  }
-
-  if (explicitMode) {
-    return explicitMode;
-  }
-  if (modeFromFlags) {
-    return modeFromFlags;
-  }
-  return defaultMode;
-}
-
-function resolveTargetPath(targetPathArg, options = {}) {
-  return path.resolve(process.cwd(), String(options.path || targetPathArg || "."));
-}
-
-function printUnifiedSummary(report) {
-  console.log(pc.bold("Unified review report"));
-  console.log(pc.gray(`Run: ${report.runId}`));
-  console.log(pc.gray(`Mode: ${report.mode}`));
-  console.log(
-    `Findings: P0=${report.summary.P0} P1=${report.summary.P1} P2=${report.summary.P2} P3=${report.summary.P3}`
-  );
-  for (const finding of report.findings || []) {
-    const verdict = finding.adjudication?.verdict || "pending";
-    console.log(
-      `- [${finding.severity}] ${finding.findingId} ${finding.file}:${finding.line} (${verdict}) ${finding.message}`
-    );
-  }
-}
-
-async function executeReviewRun({
-  targetPath,
-  mode,
-  outputDir = "",
-  specFile = "",
-  refreshIngest = false,
-  aiConfig = {},
-  replaySourceRunId = "",
-} = {}) {
-  const deterministic = await runDeterministicReviewPipeline({
-    targetPath,
-    mode,
-    outputDir,
-    specFile,
-    refreshIngest,
-  });
-
-  let aiLayer = null;
-  if (aiConfig.enable) {
-    aiLayer = await runAiReviewLayer({
-      targetPath: deterministic.targetPath,
-      mode: deterministic.mode,
-      runId: deterministic.runId,
-      runDirectory: deterministic.artifacts.runDirectory,
-      deterministic,
-      outputDir,
-      provider: aiConfig.provider,
-      model: aiConfig.model,
-      apiKey: aiConfig.apiKey,
-      sessionId: aiConfig.sessionId,
-      maxFindings: aiConfig.aiMaxFindings,
-      maxCostUsd: aiConfig.maxCost,
-      maxOutputTokens: aiConfig.maxTokens,
-      maxRuntimeMs: aiConfig.maxRuntimeMs,
-      maxToolCalls: aiConfig.maxToolCalls,
-      maxNoProgress: aiConfig.maxNoProgress,
-      warningThresholdPercent: aiConfig.warnAtPercent,
-      dryRun: Boolean(aiConfig.aiDryRun),
-      env: process.env,
-    });
-  }
-
-  const unified = await buildUnifiedReviewReport({
-    targetPath: deterministic.targetPath,
-    mode: deterministic.mode,
-    runId: deterministic.runId,
-    deterministic,
-    aiLayer,
-    specFile,
-  });
-  const unifiedArtifacts = await writeUnifiedReviewArtifacts({
-    runDirectory: deterministic.artifacts.runDirectory,
-    report: unified.report,
-    markdown: unified.markdown,
-  });
-
-  const contextWrite = await writeReviewRunContext({
-    runDirectory: deterministic.artifacts.runDirectory,
-    runId: deterministic.runId,
-    targetPath: deterministic.targetPath,
-    mode: deterministic.mode,
-    invocation: {
-      aiEnabled: Boolean(aiConfig.enable),
-      aiDryRun: Boolean(aiConfig.aiDryRun),
-      provider: aiConfig.provider,
-      model: aiConfig.model,
-      sessionId: aiConfig.sessionId,
-      aiMaxFindings: aiConfig.aiMaxFindings,
-      maxCost: aiConfig.maxCost,
-      maxTokens: aiConfig.maxTokens,
-      maxRuntimeMs: aiConfig.maxRuntimeMs,
-      maxToolCalls: aiConfig.maxToolCalls,
-      maxNoProgress: aiConfig.maxNoProgress,
-      warnAtPercent: aiConfig.warnAtPercent,
-      outputDir,
-      specFile,
-      refreshIngest: Boolean(refreshIngest),
-    },
-    replay: {
-      sourceRunId: replaySourceRunId,
-      replayed: Boolean(replaySourceRunId),
-    },
-  });
-
-  const summary = unified.report.summary;
-  const blocking = Boolean(summary.blocking) || Boolean(aiLayer?.budget?.blocking);
-
-  const payload = {
-    command: "review",
-    targetPath: deterministic.targetPath,
-    mode: deterministic.mode,
-    runId: deterministic.runId,
-    runDirectory: deterministic.artifacts.runDirectory,
-    reportPath: deterministic.artifacts.markdownPath,
-    reportJsonPath: deterministic.artifacts.jsonPath,
-    reportUnifiedPath: unifiedArtifacts.reportMarkdownPath,
-    reportUnifiedJsonPath: unifiedArtifacts.reportJsonPath,
-    runContextPath: contextWrite.contextPath,
-    scannedFiles: deterministic.scope.scannedFiles,
-    scopedFiles: deterministic.scope.scannedRelativeFiles,
-    specPath: deterministic.layers.specBinding?.specPath || "",
-    specHashSha256: deterministic.layers.specBinding?.specHashSha256 || "",
-    findingCount: unified.report.findings.length,
-    p0: summary.P0,
-    p1: summary.P1,
-    p2: summary.P2,
-    p3: summary.P3,
-    blocking,
-    deterministicSummary: deterministic.summary,
-    ingestRefresh: deterministic.layers?.ingest?.refresh || null,
-    ai: aiLayer
-      ? {
-          enabled: true,
-          dryRun: aiLayer.dryRun,
-          parser: aiLayer.parser,
-          summary: aiLayer.summary,
-          provider: aiLayer.provider,
-          model: aiLayer.model,
-          findingCount: aiLayer.findings.length,
-          reportPath: aiLayer.artifacts.reportMarkdownPath,
-          reportJsonPath: aiLayer.artifacts.reportJsonPath,
-          promptPath: aiLayer.artifacts.promptPath,
-          usage: aiLayer.usage,
-          pricingFound: aiLayer.pricingFound,
-          budget: aiLayer.budget,
-          cost: aiLayer.cost,
-          telemetry: aiLayer.telemetry,
-        }
-      : {
-          enabled: false,
-        },
-  };
-
-  return {
-    deterministic,
-    aiLayer,
-    unified,
-    unifiedArtifacts,
-    contextWrite,
-    payload,
-    blocking,
-  };
-}
-
-function registerVerdictCommand(review, verdict) {
-  review
-    .command(`${verdict} <findingId>`)
-    .description(`Record HITL verdict '${verdict}' for a unified review finding`)
-    .option("--run-id <id>", "Explicit review run id")
-    .option("--path <path>", "Target workspace path", ".")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--note <text>", "Optional human note for this verdict", "")
-    .option("--actor <id>", "Operator identifier for audit trail", "")
-    .option("--json", "Emit machine-readable output")
-    .action(async (findingId, options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(".", options);
-      const result = await recordReviewDecision({
-        targetPath,
-        runId: options.runId,
-        outputDir: options.outputDir,
-        findingId,
-        verdict,
-        note: options.note,
-        actor: options.actor,
-        env: process.env,
-      });
-
-      const payload = {
-        command: `review ${verdict}`,
-        runId: result.runId,
-        runDirectory: result.runDirectory,
-        findingId: result.findingId,
-        decision: result.decision,
-        reportPath: result.reportMarkdownPath,
-        reportJsonPath: result.reportJsonPath,
-        decisionsPath: result.decisionsPath,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-      } else {
-        console.log(pc.bold(`Recorded verdict '${verdict}'`));
-        console.log(pc.gray(`Run: ${result.runId}`));
-        console.log(pc.gray(`Finding: ${result.findingId}`));
-        console.log(pc.gray(`Report: ${result.reportMarkdownPath}`));
-      }
-    });
-}
-
-export function registerReviewCommand(program) {
-  const review = program
-    .command("review")
-    .description("Run deterministic local review with optional AI reasoning layer")
-    .argument("[targetPath]", "Target workspace path", ".")
-    .option("--path <path>", "Target workspace path override")
-    .option("--diff", "Alias for --mode diff")
-    .option("--staged", "Alias for --mode staged")
-    .option("--ai", "Enable AI reasoning layer over deterministic findings")
-    .option("--ai-dry-run", "Run AI layer in dry-run mode (no provider call)")
-    .option("--provider <name>", "AI provider override (openai|anthropic|google)")
-    .option("--model <id>", "AI model override")
-    .option("--api-key <key>", "Optional explicit API key override")
-    .option("--session-id <id>", "AI cost/telemetry session id override")
-    .option("--ai-max-findings <n>", "Max number of structured AI findings", "20")
-    .option("--max-cost <usd>", "Max AI cost budget for this review session", "1.0")
-    .option("--max-tokens <n>", "Max AI output token budget (0 disables)", "0")
-    .option("--max-runtime-ms <n>", "Max AI runtime budget in ms (0 disables)", "0")
-    .option("--max-tool-calls <n>", "Max AI tool-call budget (0 disables)", "0")
-    .option("--max-no-progress <n>", "Max no-progress streak before stop", "3")
-    .option("--warn-at-percent <n>", "Warning threshold percentage for enabled budgets", "80")
-    .option("--spec <path>", "Spec file path relative to target (defaults to SPEC.md or docs/spec.md)")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--refresh", "Refresh CODEBASE_INGEST before deterministic review")
-    .option("--json", "Emit machine-readable output")
-    .action(async (targetPathArg, options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(targetPathArg, options);
-      const mode = resolveModeFromOptions(options, {
-        defaultMode: "full",
-      });
-
-      const outcome = await executeReviewRun({
-        targetPath,
-        mode,
-        outputDir: options.outputDir,
-        specFile: options.spec,
-        refreshIngest: Boolean(options.refresh),
-        aiConfig: {
-          enable: Boolean(options.ai),
-          aiDryRun: Boolean(options.aiDryRun),
-          provider: options.provider,
-          model: options.model,
-          apiKey: options.apiKey,
-          sessionId: options.sessionId,
-          aiMaxFindings: options.aiMaxFindings,
-          maxCost: options.maxCost,
-          maxTokens: options.maxTokens,
-          maxRuntimeMs: options.maxRuntimeMs,
-          maxToolCalls: options.maxToolCalls,
-          maxNoProgress: options.maxNoProgress,
-          warnAtPercent: options.warnAtPercent,
-        },
-      });
-
-      if (emitJson) {
-        console.log(JSON.stringify(outcome.payload, null, 2));
-      } else {
-        console.log(pc.bold("Deterministic review complete"));
-        console.log(pc.gray(`Run: ${outcome.deterministic.runId}`));
-        console.log(pc.gray(`Mode: ${outcome.deterministic.mode}`));
-        console.log(pc.gray(`Report: ${outcome.deterministic.artifacts.markdownPath}`));
-        console.log(pc.gray(`JSON: ${outcome.deterministic.artifacts.jsonPath}`));
-        console.log(pc.gray(`Unified report: ${outcome.unifiedArtifacts.reportMarkdownPath}`));
-        console.log(pc.gray(`Unified JSON: ${outcome.unifiedArtifacts.reportJsonPath}`));
-        console.log(pc.gray(`Run context: ${outcome.contextWrite.contextPath}`));
-        if (outcome.payload.ingestRefresh?.stale || outcome.payload.ingestRefresh?.refreshed) {
-          const color =
-            outcome.payload.ingestRefresh?.stale && !outcome.payload.ingestRefresh?.refreshed
-              ? pc.yellow
-              : pc.gray;
-          console.log(color(formatIngestResolutionNotice(outcome.payload.ingestRefresh)));
-        }
-        if (outcome.aiLayer) {
-          console.log(
-            pc.gray(
-              `AI: ${outcome.aiLayer.provider}/${outcome.aiLayer.model} findings=${outcome.aiLayer.findings.length} dry_run=${outcome.aiLayer.dryRun ? "yes" : "no"}`
-            )
-          );
-          console.log(pc.gray(`AI report: ${outcome.aiLayer.artifacts.reportMarkdownPath}`));
-        }
-        console.log(`Files scanned: ${outcome.deterministic.scope.scannedFiles}`);
-        console.log(
-          `Findings: P0=${outcome.unified.report.summary.P0} P1=${outcome.unified.report.summary.P1} P2=${outcome.unified.report.summary.P2} P3=${outcome.unified.report.summary.P3}`
-        );
-      }
-
-      if (outcome.blocking) {
-        if (!emitJson) {
-          console.log(pc.red("Blocking findings detected (P0/P1 > 0 or budget stop)."));
-        }
-        process.exitCode = 2;
-      }
-    });
-
-  review
-    .command("show")
-    .description("Show latest or specified unified review report")
-    .option("--run-id <id>", "Explicit review run id")
-    .option("--path <path>", "Target workspace path", ".")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--json", "Emit machine-readable output")
-    .action(async (options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(".", options);
-      const loaded = await loadUnifiedReviewReport({
-        targetPath,
-        runId: options.runId,
-        outputDir: options.outputDir,
-        env: process.env,
-      });
-
-      const payload = {
-        command: "review show",
-        runId: loaded.report.runId,
-        runDirectory: loaded.runDirectory,
-        reportPath: loaded.reportMarkdownPath,
-        reportJsonPath: loaded.reportJsonPath,
-        decisionsPath: loaded.decisionsPath,
-        report: loaded.report,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-        return;
-      }
-
-      console.log(pc.gray(`Report: ${loaded.reportMarkdownPath}`));
-      printUnifiedSummary(loaded.report);
-    });
-
-  review
-    .command("export")
-    .description("Export unified review report in md/json/sarif/github-annotations format")
-    .option("--run-id <id>", "Explicit review run id")
-    .option("--path <path>", "Target workspace path", ".")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--format <format>", "Export format (sarif|json|md|github-annotations)", "md")
-    .option("--output-file <path>", "Optional custom export output path")
-    .option("--json", "Emit machine-readable output")
-    .action(async (options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(".", options);
-      const exported = await exportUnifiedReviewReport({
-        targetPath,
-        runId: options.runId,
-        outputDir: options.outputDir,
-        format: options.format,
-        outputFile: options.outputFile,
-        env: process.env,
-      });
-
-      const payload = {
-        command: "review export",
-        runId: exported.runId,
-        runDirectory: exported.runDirectory,
-        format: exported.format,
-        outputPath: exported.outputPath,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-        return;
-      }
-
-      console.log(pc.bold("Review report exported"));
-      console.log(pc.gray(`Run: ${exported.runId}`));
-      console.log(pc.gray(`Format: ${exported.format}`));
-      console.log(pc.gray(`Output: ${exported.outputPath}`));
-    });
-
-  review
-    .command("replay <runId>")
-    .description("Replay a previous unified review run with captured context and compare drift")
-    .option("--path <path>", "Target workspace path override")
-    .option("--spec <path>", "Spec file path override for replay run")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--refresh", "Refresh CODEBASE_INGEST before replay run")
-    .option("--no-ai", "Disable AI layer for replay run")
-    .option("--ai-dry-run", "Force AI dry-run during replay")
-    .option("--json", "Emit machine-readable output")
-    .action(async (runId, options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const lookupPath = resolveTargetPath(".", options);
-      const source = await loadUnifiedReviewReport({
-        targetPath: lookupPath,
-        runId,
-        outputDir: options.outputDir,
-        env: process.env,
-      });
-
-      let sourceContext = null;
-      try {
-        sourceContext = await loadReviewRunContext(source.runDirectory);
-      } catch {
-        sourceContext = null;
-      }
-
-      const invocation = sourceContext?.context?.invocation || {};
-      const replayTargetPath = options.path
-        ? resolveTargetPath(".", options)
-        : path.resolve(sourceContext?.context?.targetPath || source.report.targetPath || lookupPath);
-      const aiEnabled = options.noAi ? false : Boolean(invocation.aiEnabled);
-
-      const replayOutcome = await executeReviewRun({
-        targetPath: replayTargetPath,
-        mode: sourceContext?.context?.mode || source.report.mode || "full",
-        outputDir: options.outputDir || invocation.outputDir || "",
-        specFile: options.spec || invocation.specFile || "",
-        refreshIngest: Boolean(options.refresh || invocation.refreshIngest),
-        aiConfig: {
-          enable: aiEnabled,
-          aiDryRun: Boolean(options.aiDryRun || invocation.aiDryRun),
-          provider: invocation.provider,
-          model: invocation.model,
-          sessionId: invocation.sessionId,
-          aiMaxFindings: invocation.aiMaxFindings || "20",
-          maxCost: invocation.maxCost || "1.0",
-          maxTokens: invocation.maxTokens || "0",
-          maxRuntimeMs: invocation.maxRuntimeMs || "0",
-          maxToolCalls: invocation.maxToolCalls || "0",
-          maxNoProgress: invocation.maxNoProgress || "3",
-          warnAtPercent: invocation.warnAtPercent || "80",
-        },
-        replaySourceRunId: source.report.runId,
-      });
-
-      const comparison = compareUnifiedReports(source.report, replayOutcome.unified.report);
-      const artifact = await writeReviewComparisonArtifact({
-        runDirectory: replayOutcome.deterministic.artifacts.runDirectory,
-        baseRunId: source.report.runId,
-        candidateRunId: replayOutcome.unified.report.runId,
-        comparison,
-      });
-
-      const payload = {
-        command: "review replay",
-        sourceRunId: source.report.runId,
-        replayRunId: replayOutcome.unified.report.runId,
-        runDirectory: replayOutcome.deterministic.artifacts.runDirectory,
-        sourceReportPath: source.reportMarkdownPath,
-        replayReportPath: replayOutcome.unifiedArtifacts.reportMarkdownPath,
-        replayReportJsonPath: replayOutcome.unifiedArtifacts.reportJsonPath,
-        comparisonPath: artifact.artifactPath,
-        deterministicEquivalent: comparison.deterministicEquivalent,
-        driftDetected: !comparison.deterministicEquivalent,
-        counts: comparison.counts,
-        summaryDelta: comparison.summaryDelta,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-      } else {
-        console.log(pc.bold("Review replay complete"));
-        console.log(pc.gray(`Source run: ${payload.sourceRunId}`));
-        console.log(pc.gray(`Replay run: ${payload.replayRunId}`));
-        console.log(pc.gray(`Comparison: ${payload.comparisonPath}`));
-        console.log(
-          `Drift: ${payload.driftDetected ? "detected" : "none"} (added=${payload.counts.added}, removed=${payload.counts.removed}, severity_changed=${payload.counts.severityChanged})`
-        );
-      }
-    });
-
-  review
-    .command("diff <baseRunId> <candidateRunId>")
-    .description("Compare two unified review runs")
-    .option("--path <path>", "Target workspace path", ".")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--output-file <path>", "Optional custom output path for comparison JSON")
-    .option("--json", "Emit machine-readable output")
-    .action(async (baseRunId, candidateRunId, options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(".", options);
-      const base = await loadUnifiedReviewReport({
-        targetPath,
-        runId: baseRunId,
-        outputDir: options.outputDir,
-        env: process.env,
-      });
-      const candidate = await loadUnifiedReviewReport({
-        targetPath,
-        runId: candidateRunId,
-        outputDir: options.outputDir,
-        env: process.env,
-      });
-
-      const comparison = compareUnifiedReports(base.report, candidate.report);
-      const artifact = await writeReviewComparisonArtifact({
-        runDirectory: candidate.runDirectory,
-        baseRunId: base.report.runId,
-        candidateRunId: candidate.report.runId,
-        comparison,
-      });
-
-      let outputPath = artifact.artifactPath;
-      if (options.outputFile) {
-        outputPath = path.resolve(targetPath, String(options.outputFile));
-        await fsp.mkdir(path.dirname(outputPath), { recursive: true });
-        await fsp.writeFile(outputPath, `${JSON.stringify(artifact.payload, null, 2)}\n`, "utf-8");
-      }
-
-      const payload = {
-        command: "review diff",
-        baseRunId: base.report.runId,
-        candidateRunId: candidate.report.runId,
-        deterministicEquivalent: comparison.deterministicEquivalent,
-        counts: comparison.counts,
-        summaryDelta: comparison.summaryDelta,
-        outputPath,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-      } else {
-        console.log(pc.bold("Review run diff complete"));
-        console.log(pc.gray(`Base: ${payload.baseRunId}`));
-        console.log(pc.gray(`Candidate: ${payload.candidateRunId}`));
-        console.log(pc.gray(`Output: ${payload.outputPath}`));
-        console.log(
-          `Delta: added=${payload.counts.added}, removed=${payload.counts.removed}, severity_changed=${payload.counts.severityChanged}`
-        );
-      }
-    });
-
-  registerVerdictCommand(review, "accept");
-  registerVerdictCommand(review, "reject");
-  registerVerdictCommand(review, "defer");
-
-  review
-    .command("scan")
-    .description("Compatibility mode: run lightweight policy scan")
-    .option("--mode <mode>", "Scan mode: full, diff, or staged", "full")
-    .option("--diff", "Alias for --mode diff")
-    .option("--staged", "Alias for --mode staged")
-    .option("--path <path>", "Target workspace path", ".")
-    .option("--spec <path>", "Spec file path relative to target (defaults to SPEC.md or docs/spec.md)")
-    .option("--output-dir <path>", "Optional artifact output root override")
-    .option("--refresh", "Refresh CODEBASE_INGEST prior to compatibility scan")
-    .option("--json", "Emit machine-readable output")
-    .action(async (options, command) => {
-      const emitJson = shouldEmitJson(options, command);
-      const targetPath = resolveTargetPath(".", options);
-      let ingestRefresh = null;
-      if (options.refresh) {
-        const ingestResolution = await resolveCodebaseIngest({
-          rootPath: targetPath,
-          outputDir: options.outputDir,
-          refresh: true,
-        });
-        ingestRefresh = {
-          outputPath: ingestResolution.outputPath,
-          refreshed: ingestResolution.refreshed,
-          stale: ingestResolution.stale,
-          reasons: ingestResolution.reasons,
-          refreshedBecause: ingestResolution.refreshedBecause,
-          lastCommitAt: ingestResolution.lastCommitAt,
-          contentHash: ingestResolution.fingerprint?.contentHash || "",
-        };
-      }
-      const mode = resolveModeFromOptions(options, {
-        defaultMode: String(options.mode || "full").trim().toLowerCase() || "full",
-      });
-      const scan = await runLocalReviewScan({
-        targetPath,
-        mode,
-        specFile: options.spec,
-      });
-
-      const report = `# Local Review Scan
-
-Generated: ${new Date().toISOString()}
-Target: ${scan.targetPath}
-Mode: ${scan.mode}
-
-Summary:
-- Files scanned: ${scan.scannedFiles}
-- P1 findings: ${scan.p1}
-- P2 findings: ${scan.p2}
-
-Scoped files:
-${scan.scannedRelativeFiles.length > 0 ? scan.scannedRelativeFiles.map((item) => `- ${item}`).join("\n") : "- none"}
-
-Findings:
-${formatFindingsMarkdown(scan.findings)}
-`;
-
-      const reportPath = await writeReviewReport({
-        targetPath: scan.targetPath,
-        mode: scan.mode,
-        outputDir: options.outputDir,
-        reportMarkdown: report,
-      });
-
-      const payload = {
-        command: "review scan",
-        targetPath: scan.targetPath,
-        mode: scan.mode,
-        reportPath,
-        scannedFiles: scan.scannedFiles,
-        scopedFiles: scan.scannedRelativeFiles,
-        specPath: scan.specBinding?.specPath || "",
-        specHashSha256: scan.specBinding?.specHashSha256 || "",
-        p1: scan.p1,
-        p2: scan.p2,
-        blocking: scan.p1 > 0,
-        ingestRefresh,
-      };
-
-      if (emitJson) {
-        console.log(JSON.stringify(payload, null, 2));
-      } else {
-        console.log(pc.bold("Local review scan complete"));
-        console.log(pc.gray(`Mode: ${scan.mode}`));
-        console.log(pc.gray(`Report: ${reportPath}`));
-        if (ingestRefresh) {
-          console.log(pc.gray(formatIngestResolutionNotice(ingestRefresh)));
-        }
-        console.log(`Files scanned: ${scan.scannedFiles}`);
-        console.log(`P1 findings: ${scan.p1}`);
-        console.log(`P2 findings: ${scan.p2}`);
-      }
-
-      if (scan.p1 > 0) {
-        if (!emitJson) {
-          console.log(pc.red("Blocking findings detected (P1 > 0)."));
-        }
-        process.exitCode = 2;
-      }
-    });
-}
-
+import fsp from "node:fs/promises";
+import path from "node:path";
+import process from "node:process";
+
+import pc from "picocolors";
+
+import { formatIngestResolutionNotice, resolveCodebaseIngest } from "../ingest/engine.js";
+import { runAiReviewLayer } from "../review/ai-review.js";
+import {
+  formatFindingsMarkdown,
+  runDeterministicReviewPipeline,
+  runLocalReviewScan,
+  writeReviewReport,
+} from "../review/local-review.js";
+import {
+  buildUnifiedReviewReport,
+  exportUnifiedReviewReport,
+  loadUnifiedReviewReport,
+  recordReviewDecision,
+  writeUnifiedReviewArtifacts,
+} from "../review/report.js";
+import {
+  compareUnifiedReports,
+  loadReviewRunContext,
+  writeReviewComparisonArtifact,
+  writeReviewRunContext,
+} from "../review/replay.js";
+
+function shouldEmitJson(options, command) {
+  const local = Boolean(options && options.json);
+  const globalFromCommand =
+    command && command.optsWithGlobals ? Boolean(command.optsWithGlobals().json) : false;
+  return local || globalFromCommand;
+}
+
+function resolveModeFromOptions(options = {}, { defaultMode = "full" } = {}) {
+  const explicitMode = String(options.mode || "").trim().toLowerCase();
+  const diffFlag = Boolean(options.diff);
+  const stagedFlag = Boolean(options.staged);
+
+  if (diffFlag && stagedFlag) {
+    throw new Error("Use only one of --diff or --staged.");
+  }
+
+  let modeFromFlags = "";
+  if (diffFlag) {
+    modeFromFlags = "diff";
+  } else if (stagedFlag) {
+    modeFromFlags = "staged";
+  }
+
+  if (explicitMode && modeFromFlags && explicitMode !== modeFromFlags) {
+    throw new Error(`Conflicting mode selection: --mode ${explicitMode} with --${modeFromFlags}.`);
+  }
+
+  if (explicitMode) {
+    return explicitMode;
+  }
+  if (modeFromFlags) {
+    return modeFromFlags;
+  }
+  return defaultMode;
+}
+
+function resolveTargetPath(targetPathArg, options = {}) {
+  return path.resolve(process.cwd(), String(options.path || targetPathArg || "."));
+}
+
+function printUnifiedSummary(report) {
+  console.log(pc.bold("Unified review report"));
+  console.log(pc.gray(`Run: ${report.runId}`));
+  console.log(pc.gray(`Mode: ${report.mode}`));
+  console.log(
+    `Findings: P0=${report.summary.P0} P1=${report.summary.P1} P2=${report.summary.P2} P3=${report.summary.P3}`
+  );
+  for (const finding of report.findings || []) {
+    const verdict = finding.adjudication?.verdict || "pending";
+    console.log(
+      `- [${finding.severity}] ${finding.findingId} ${finding.file}:${finding.line} (${verdict}) ${finding.message}`
+    );
+  }
+}
+
+async function executeReviewRun({
+  targetPath,
+  mode,
+  outputDir = "",
+  specFile = "",
+  refreshIngest = false,
+  aiConfig = {},
+  replaySourceRunId = "",
+} = {}) {
+  const deterministic = await runDeterministicReviewPipeline({
+    targetPath,
+    mode,
+    outputDir,
+    specFile,
+    refreshIngest,
+  });
+
+  let aiLayer = null;
+  if (aiConfig.enable) {
+    aiLayer = await runAiReviewLayer({
+      targetPath: deterministic.targetPath,
+      mode: deterministic.mode,
+      runId: deterministic.runId,
+      runDirectory: deterministic.artifacts.runDirectory,
+      deterministic,
+      outputDir,
+      provider: aiConfig.provider,
+      model: aiConfig.model,
+      apiKey: aiConfig.apiKey,
+      sessionId: aiConfig.sessionId,
+      maxFindings: aiConfig.aiMaxFindings,
+      maxCostUsd: aiConfig.maxCost,
+      maxOutputTokens: aiConfig.maxTokens,
+      maxRuntimeMs: aiConfig.maxRuntimeMs,
+      maxToolCalls: aiConfig.maxToolCalls,
+      maxNoProgress: aiConfig.maxNoProgress,
+      warningThresholdPercent: aiConfig.warnAtPercent,
+      dryRun: Boolean(aiConfig.aiDryRun),
+      env: process.env,
+    });
+  }
+
+  const unified = await buildUnifiedReviewReport({
+    targetPath: deterministic.targetPath,
+    mode: deterministic.mode,
+    runId: deterministic.runId,
+    deterministic,
+    aiLayer,
+    specFile,
+  });
+  const unifiedArtifacts = await writeUnifiedReviewArtifacts({
+    runDirectory: deterministic.artifacts.runDirectory,
+    report: unified.report,
+    markdown: unified.markdown,
+  });
+
+  const contextWrite = await writeReviewRunContext({
+    runDirectory: deterministic.artifacts.runDirectory,
+    runId: deterministic.runId,
+    targetPath: deterministic.targetPath,
+    mode: deterministic.mode,
+    invocation: {
+      aiEnabled: Boolean(aiConfig.enable),
+      aiDryRun: Boolean(aiConfig.aiDryRun),
+      provider: aiConfig.provider,
+      model: aiConfig.model,
+      sessionId: aiConfig.sessionId,
+      aiMaxFindings: aiConfig.aiMaxFindings,
+      maxCost: aiConfig.maxCost,
+      maxTokens: aiConfig.maxTokens,
+      maxRuntimeMs: aiConfig.maxRuntimeMs,
+      maxToolCalls: aiConfig.maxToolCalls,
+      maxNoProgress: aiConfig.maxNoProgress,
+      warnAtPercent: aiConfig.warnAtPercent,
+      outputDir,
+      specFile,
+      refreshIngest: Boolean(refreshIngest),
+    },
+    replay: {
+      sourceRunId: replaySourceRunId,
+      replayed: Boolean(replaySourceRunId),
+    },
+  });
+
+  const summary = unified.report.summary;
+  const blocking = Boolean(summary.blocking) || Boolean(aiLayer?.budget?.blocking);
+
+  const payload = {
+    command: "review",
+    targetPath: deterministic.targetPath,
+    mode: deterministic.mode,
+    runId: deterministic.runId,
+    runDirectory: deterministic.artifacts.runDirectory,
+    reportPath: deterministic.artifacts.markdownPath,
+    reportJsonPath: deterministic.artifacts.jsonPath,
+    reportUnifiedPath: unifiedArtifacts.reportMarkdownPath,
+    reportUnifiedJsonPath: unifiedArtifacts.reportJsonPath,
+    runContextPath: contextWrite.contextPath,
+    scannedFiles: deterministic.scope.scannedFiles,
+    scopedFiles: deterministic.scope.scannedRelativeFiles,
+    specPath: deterministic.layers.specBinding?.specPath || "",
+    specHashSha256: deterministic.layers.specBinding?.specHashSha256 || "",
+    findingCount: unified.report.findings.length,
+    p0: summary.P0,
+    p1: summary.P1,
+    p2: summary.P2,
+    p3: summary.P3,
+    blocking,
+    deterministicSummary: deterministic.summary,
+    ingestRefresh: deterministic.layers?.ingest?.refresh || null,
+    ai: aiLayer
+      ? {
+          enabled: true,
+          dryRun: aiLayer.dryRun,
+          parser: aiLayer.parser,
+          summary: aiLayer.summary,
+          provider: aiLayer.provider,
+          model: aiLayer.model,
+          findingCount: aiLayer.findings.length,
+          reportPath: aiLayer.artifacts.reportMarkdownPath,
+          reportJsonPath: aiLayer.artifacts.reportJsonPath,
+          promptPath: aiLayer.artifacts.promptPath,
+          usage: aiLayer.usage,
+          pricingFound: aiLayer.pricingFound,
+          budget: aiLayer.budget,
+          cost: aiLayer.cost,
+          telemetry: aiLayer.telemetry,
+        }
+      : {
+          enabled: false,
+        },
+  };
+
+  return {
+    deterministic,
+    aiLayer,
+    unified,
+    unifiedArtifacts,
+    contextWrite,
+    payload,
+    blocking,
+  };
+}
+
+function registerVerdictCommand(review, verdict) {
+  review
+    .command(`${verdict} <findingId>`)
+    .description(`Record HITL verdict '${verdict}' for a unified review finding`)
+    .option("--run-id <id>", "Explicit review run id")
+    .option("--path <path>", "Target workspace path", ".")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--note <text>", "Optional human note for this verdict", "")
+    .option("--actor <id>", "Operator identifier for audit trail", "")
+    .option("--json", "Emit machine-readable output")
+    .action(async (findingId, options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(".", options);
+      const result = await recordReviewDecision({
+        targetPath,
+        runId: options.runId,
+        outputDir: options.outputDir,
+        findingId,
+        verdict,
+        note: options.note,
+        actor: options.actor,
+        env: process.env,
+      });
+
+      const payload = {
+        command: `review ${verdict}`,
+        runId: result.runId,
+        runDirectory: result.runDirectory,
+        findingId: result.findingId,
+        decision: result.decision,
+        reportPath: result.reportMarkdownPath,
+        reportJsonPath: result.reportJsonPath,
+        decisionsPath: result.decisionsPath,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+      } else {
+        console.log(pc.bold(`Recorded verdict '${verdict}'`));
+        console.log(pc.gray(`Run: ${result.runId}`));
+        console.log(pc.gray(`Finding: ${result.findingId}`));
+        console.log(pc.gray(`Report: ${result.reportMarkdownPath}`));
+      }
+    });
+}
+
+export function registerReviewCommand(program) {
+  const review = program
+    .command("review")
+    .description("Run deterministic local review with optional AI reasoning layer")
+    .argument("[targetPath]", "Target workspace path", ".")
+    .option("--path <path>", "Target workspace path override")
+    .option("--diff", "Alias for --mode diff")
+    .option("--staged", "Alias for --mode staged")
+    .option("--ai", "Enable AI reasoning layer over deterministic findings")
+    .option("--ai-dry-run", "Run AI layer in dry-run mode (no provider call)")
+    .option("--provider <name>", "AI provider override (openai|anthropic|google)")
+    .option("--model <id>", "AI model override")
+    .option("--api-key <key>", "Optional explicit API key override")
+    .option("--session-id <id>", "AI cost/telemetry session id override")
+    .option("--ai-max-findings <n>", "Max number of structured AI findings", "20")
+    .option("--max-cost <usd>", "Max AI cost budget for this review session", "1.0")
+    .option("--max-tokens <n>", "Max AI output token budget (0 disables)", "0")
+    .option("--max-runtime-ms <n>", "Max AI runtime budget in ms (0 disables)", "0")
+    .option("--max-tool-calls <n>", "Max AI tool-call budget (0 disables)", "0")
+    .option("--max-no-progress <n>", "Max no-progress streak before stop", "3")
+    .option("--warn-at-percent <n>", "Warning threshold percentage for enabled budgets", "80")
+    .option("--spec <path>", "Spec file path relative to target (defaults to SPEC.md or docs/spec.md)")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--refresh", "Refresh CODEBASE_INGEST before deterministic review")
+    .option("--json", "Emit machine-readable output")
+    .action(async (targetPathArg, options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(targetPathArg, options);
+      const mode = resolveModeFromOptions(options, {
+        defaultMode: "full",
+      });
+
+      const outcome = await executeReviewRun({
+        targetPath,
+        mode,
+        outputDir: options.outputDir,
+        specFile: options.spec,
+        refreshIngest: Boolean(options.refresh),
+        aiConfig: {
+          enable: Boolean(options.ai),
+          aiDryRun: Boolean(options.aiDryRun),
+          provider: options.provider,
+          model: options.model,
+          apiKey: options.apiKey,
+          sessionId: options.sessionId,
+          aiMaxFindings: options.aiMaxFindings,
+          maxCost: options.maxCost,
+          maxTokens: options.maxTokens,
+          maxRuntimeMs: options.maxRuntimeMs,
+          maxToolCalls: options.maxToolCalls,
+          maxNoProgress: options.maxNoProgress,
+          warnAtPercent: options.warnAtPercent,
+        },
+      });
+
+      if (emitJson) {
+        console.log(JSON.stringify(outcome.payload, null, 2));
+      } else {
+        console.log(pc.bold("Deterministic review complete"));
+        console.log(pc.gray(`Run: ${outcome.deterministic.runId}`));
+        console.log(pc.gray(`Mode: ${outcome.deterministic.mode}`));
+        console.log(pc.gray(`Report: ${outcome.deterministic.artifacts.markdownPath}`));
+        console.log(pc.gray(`JSON: ${outcome.deterministic.artifacts.jsonPath}`));
+        console.log(pc.gray(`Unified report: ${outcome.unifiedArtifacts.reportMarkdownPath}`));
+        console.log(pc.gray(`Unified JSON: ${outcome.unifiedArtifacts.reportJsonPath}`));
+        console.log(pc.gray(`Run context: ${outcome.contextWrite.contextPath}`));
+        if (outcome.payload.ingestRefresh?.stale || outcome.payload.ingestRefresh?.refreshed) {
+          const color =
+            outcome.payload.ingestRefresh?.stale && !outcome.payload.ingestRefresh?.refreshed
+              ? pc.yellow
+              : pc.gray;
+          console.log(color(formatIngestResolutionNotice(outcome.payload.ingestRefresh)));
+        }
+        if (outcome.aiLayer) {
+          console.log(
+            pc.gray(
+              `AI: ${outcome.aiLayer.provider}/${outcome.aiLayer.model} findings=${outcome.aiLayer.findings.length} dry_run=${outcome.aiLayer.dryRun ? "yes" : "no"}`
+            )
+          );
+          console.log(pc.gray(`AI report: ${outcome.aiLayer.artifacts.reportMarkdownPath}`));
+        }
+        console.log(`Files scanned: ${outcome.deterministic.scope.scannedFiles}`);
+        console.log(
+          `Findings: P0=${outcome.unified.report.summary.P0} P1=${outcome.unified.report.summary.P1} P2=${outcome.unified.report.summary.P2} P3=${outcome.unified.report.summary.P3}`
+        );
+      }
+
+      if (outcome.blocking) {
+        if (!emitJson) {
+          console.log(pc.red("Blocking findings detected (P0/P1 > 0 or budget stop)."));
+        }
+        process.exitCode = 2;
+      }
+    });
+
+  review
+    .command("show")
+    .description("Show latest or specified unified review report")
+    .option("--run-id <id>", "Explicit review run id")
+    .option("--path <path>", "Target workspace path", ".")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--json", "Emit machine-readable output")
+    .action(async (options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(".", options);
+      const loaded = await loadUnifiedReviewReport({
+        targetPath,
+        runId: options.runId,
+        outputDir: options.outputDir,
+        env: process.env,
+      });
+
+      const payload = {
+        command: "review show",
+        runId: loaded.report.runId,
+        runDirectory: loaded.runDirectory,
+        reportPath: loaded.reportMarkdownPath,
+        reportJsonPath: loaded.reportJsonPath,
+        decisionsPath: loaded.decisionsPath,
+        report: loaded.report,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+        return;
+      }
+
+      console.log(pc.gray(`Report: ${loaded.reportMarkdownPath}`));
+      printUnifiedSummary(loaded.report);
+    });
+
+  review
+    .command("export")
+    .description("Export unified review report in md/json/sarif/github-annotations format")
+    .option("--run-id <id>", "Explicit review run id")
+    .option("--path <path>", "Target workspace path", ".")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--format <format>", "Export format (sarif|json|md|github-annotations)", "md")
+    .option("--output-file <path>", "Optional custom export output path")
+    .option("--json", "Emit machine-readable output")
+    .action(async (options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(".", options);
+      const exported = await exportUnifiedReviewReport({
+        targetPath,
+        runId: options.runId,
+        outputDir: options.outputDir,
+        format: options.format,
+        outputFile: options.outputFile,
+        env: process.env,
+      });
+
+      const payload = {
+        command: "review export",
+        runId: exported.runId,
+        runDirectory: exported.runDirectory,
+        format: exported.format,
+        outputPath: exported.outputPath,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+        return;
+      }
+
+      console.log(pc.bold("Review report exported"));
+      console.log(pc.gray(`Run: ${exported.runId}`));
+      console.log(pc.gray(`Format: ${exported.format}`));
+      console.log(pc.gray(`Output: ${exported.outputPath}`));
+    });
+
+  review
+    .command("replay <runId>")
+    .description("Replay a previous unified review run with captured context and compare drift")
+    .option("--path <path>", "Target workspace path override")
+    .option("--spec <path>", "Spec file path override for replay run")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--refresh", "Refresh CODEBASE_INGEST before replay run")
+    .option("--no-ai", "Disable AI layer for replay run")
+    .option("--ai-dry-run", "Force AI dry-run during replay")
+    .option("--json", "Emit machine-readable output")
+    .action(async (runId, options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const lookupPath = resolveTargetPath(".", options);
+      const source = await loadUnifiedReviewReport({
+        targetPath: lookupPath,
+        runId,
+        outputDir: options.outputDir,
+        env: process.env,
+      });
+
+      let sourceContext = null;
+      try {
+        sourceContext = await loadReviewRunContext(source.runDirectory);
+      } catch {
+        sourceContext = null;
+      }
+
+      const invocation = sourceContext?.context?.invocation || {};
+      const replayTargetPath = options.path
+        ? resolveTargetPath(".", options)
+        : path.resolve(sourceContext?.context?.targetPath || source.report.targetPath || lookupPath);
+      const aiEnabled = options.noAi ? false : Boolean(invocation.aiEnabled);
+
+      const replayOutcome = await executeReviewRun({
+        targetPath: replayTargetPath,
+        mode: sourceContext?.context?.mode || source.report.mode || "full",
+        outputDir: options.outputDir || invocation.outputDir || "",
+        specFile: options.spec || invocation.specFile || "",
+        refreshIngest: Boolean(options.refresh || invocation.refreshIngest),
+        aiConfig: {
+          enable: aiEnabled,
+          aiDryRun: Boolean(options.aiDryRun || invocation.aiDryRun),
+          provider: invocation.provider,
+          model: invocation.model,
+          sessionId: invocation.sessionId,
+          aiMaxFindings: invocation.aiMaxFindings || "20",
+          maxCost: invocation.maxCost || "1.0",
+          maxTokens: invocation.maxTokens || "0",
+          maxRuntimeMs: invocation.maxRuntimeMs || "0",
+          maxToolCalls: invocation.maxToolCalls || "0",
+          maxNoProgress: invocation.maxNoProgress || "3",
+          warnAtPercent: invocation.warnAtPercent || "80",
+        },
+        replaySourceRunId: source.report.runId,
+      });
+
+      const comparison = compareUnifiedReports(source.report, replayOutcome.unified.report);
+      const artifact = await writeReviewComparisonArtifact({
+        runDirectory: replayOutcome.deterministic.artifacts.runDirectory,
+        baseRunId: source.report.runId,
+        candidateRunId: replayOutcome.unified.report.runId,
+        comparison,
+      });
+
+      const payload = {
+        command: "review replay",
+        sourceRunId: source.report.runId,
+        replayRunId: replayOutcome.unified.report.runId,
+        runDirectory: replayOutcome.deterministic.artifacts.runDirectory,
+        sourceReportPath: source.reportMarkdownPath,
+        replayReportPath: replayOutcome.unifiedArtifacts.reportMarkdownPath,
+        replayReportJsonPath: replayOutcome.unifiedArtifacts.reportJsonPath,
+        comparisonPath: artifact.artifactPath,
+        deterministicEquivalent: comparison.deterministicEquivalent,
+        driftDetected: !comparison.deterministicEquivalent,
+        counts: comparison.counts,
+        summaryDelta: comparison.summaryDelta,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+      } else {
+        console.log(pc.bold("Review replay complete"));
+        console.log(pc.gray(`Source run: ${payload.sourceRunId}`));
+        console.log(pc.gray(`Replay run: ${payload.replayRunId}`));
+        console.log(pc.gray(`Comparison: ${payload.comparisonPath}`));
+        console.log(
+          `Drift: ${payload.driftDetected ? "detected" : "none"} (added=${payload.counts.added}, removed=${payload.counts.removed}, severity_changed=${payload.counts.severityChanged})`
+        );
+      }
+    });
+
+  review
+    .command("diff <baseRunId> <candidateRunId>")
+    .description("Compare two unified review runs")
+    .option("--path <path>", "Target workspace path", ".")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--output-file <path>", "Optional custom output path for comparison JSON")
+    .option("--json", "Emit machine-readable output")
+    .action(async (baseRunId, candidateRunId, options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(".", options);
+      const base = await loadUnifiedReviewReport({
+        targetPath,
+        runId: baseRunId,
+        outputDir: options.outputDir,
+        env: process.env,
+      });
+      const candidate = await loadUnifiedReviewReport({
+        targetPath,
+        runId: candidateRunId,
+        outputDir: options.outputDir,
+        env: process.env,
+      });
+
+      const comparison = compareUnifiedReports(base.report, candidate.report);
+      const artifact = await writeReviewComparisonArtifact({
+        runDirectory: candidate.runDirectory,
+        baseRunId: base.report.runId,
+        candidateRunId: candidate.report.runId,
+        comparison,
+      });
+
+      let outputPath = artifact.artifactPath;
+      if (options.outputFile) {
+        outputPath = path.resolve(targetPath, String(options.outputFile));
+        await fsp.mkdir(path.dirname(outputPath), { recursive: true });
+        await fsp.writeFile(outputPath, `${JSON.stringify(artifact.payload, null, 2)}\n`, "utf-8");
+      }
+
+      const payload = {
+        command: "review diff",
+        baseRunId: base.report.runId,
+        candidateRunId: candidate.report.runId,
+        deterministicEquivalent: comparison.deterministicEquivalent,
+        counts: comparison.counts,
+        summaryDelta: comparison.summaryDelta,
+        outputPath,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+      } else {
+        console.log(pc.bold("Review run diff complete"));
+        console.log(pc.gray(`Base: ${payload.baseRunId}`));
+        console.log(pc.gray(`Candidate: ${payload.candidateRunId}`));
+        console.log(pc.gray(`Output: ${payload.outputPath}`));
+        console.log(
+          `Delta: added=${payload.counts.added}, removed=${payload.counts.removed}, severity_changed=${payload.counts.severityChanged}`
+        );
+      }
+    });
+
+  registerVerdictCommand(review, "accept");
+  registerVerdictCommand(review, "reject");
+  registerVerdictCommand(review, "defer");
+
+  review
+    .command("scan")
+    .description("Compatibility mode: run lightweight policy scan")
+    .option("--mode <mode>", "Scan mode: full, diff, or staged", "full")
+    .option("--diff", "Alias for --mode diff")
+    .option("--staged", "Alias for --mode staged")
+    .option("--path <path>", "Target workspace path", ".")
+    .option("--spec <path>", "Spec file path relative to target (defaults to SPEC.md or docs/spec.md)")
+    .option("--output-dir <path>", "Optional artifact output root override")
+    .option("--refresh", "Refresh CODEBASE_INGEST prior to compatibility scan")
+    .option("--json", "Emit machine-readable output")
+    .action(async (options, command) => {
+      const emitJson = shouldEmitJson(options, command);
+      const targetPath = resolveTargetPath(".", options);
+      let ingestRefresh = null;
+      if (options.refresh) {
+        const ingestResolution = await resolveCodebaseIngest({
+          rootPath: targetPath,
+          outputDir: options.outputDir,
+          refresh: true,
+        });
+        ingestRefresh = {
+          outputPath: ingestResolution.outputPath,
+          refreshed: ingestResolution.refreshed,
+          stale: ingestResolution.stale,
+          reasons: ingestResolution.reasons,
+          refreshedBecause: ingestResolution.refreshedBecause,
+          lastCommitAt: ingestResolution.lastCommitAt,
+          contentHash: ingestResolution.fingerprint?.contentHash || "",
+        };
+      }
+      const mode = resolveModeFromOptions(options, {
+        defaultMode: String(options.mode || "full").trim().toLowerCase() || "full",
+      });
+      const scan = await runLocalReviewScan({
+        targetPath,
+        mode,
+        specFile: options.spec,
+      });
+
+      const report = `# Local Review Scan
+
+Generated: ${new Date().toISOString()}
+Target: ${scan.targetPath}
+Mode: ${scan.mode}
+
+Summary:
+- Files scanned: ${scan.scannedFiles}
+- P1 findings: ${scan.p1}
+- P2 findings: ${scan.p2}
+
+Scoped files:
+${scan.scannedRelativeFiles.length > 0 ? scan.scannedRelativeFiles.map((item) => `- ${item}`).join("\n") : "- none"}
+
+Findings:
+${formatFindingsMarkdown(scan.findings)}
+`;
+
+      const reportPath = await writeReviewReport({
+        targetPath: scan.targetPath,
+        mode: scan.mode,
+        outputDir: options.outputDir,
+        reportMarkdown: report,
+      });
+
+      const payload = {
+        command: "review scan",
+        targetPath: scan.targetPath,
+        mode: scan.mode,
+        reportPath,
+        scannedFiles: scan.scannedFiles,
+        scopedFiles: scan.scannedRelativeFiles,
+        specPath: scan.specBinding?.specPath || "",
+        specHashSha256: scan.specBinding?.specHashSha256 || "",
+        p1: scan.p1,
+        p2: scan.p2,
+        blocking: scan.p1 > 0,
+        ingestRefresh,
+      };
+
+      if (emitJson) {
+        console.log(JSON.stringify(payload, null, 2));
+      } else {
+        console.log(pc.bold("Local review scan complete"));
+        console.log(pc.gray(`Mode: ${scan.mode}`));
+        console.log(pc.gray(`Report: ${reportPath}`));
+        if (ingestRefresh) {
+          console.log(pc.gray(formatIngestResolutionNotice(ingestRefresh)));
+        }
+        console.log(`Files scanned: ${scan.scannedFiles}`);
+        console.log(`P1 findings: ${scan.p1}`);
+        console.log(`P2 findings: ${scan.p2}`);
+      }
+
+      if (scan.p1 > 0) {
+        if (!emitJson) {
+          console.log(pc.red("Blocking findings detected (P1 > 0)."));
+        }
+        process.exitCode = 2;
+      }
+    });
+}
+