bmad-plus 0.20.0 → 0.21.0

package/tools/cli/lib/review.js

@@ -14,7 +14,7 @@ const path = require('node:path');
 const crypto = require('node:crypto');
 const { spawnSync } = require('node:child_process');
 const { matchesAny } = require('./glob');
-const { redactFields } = require('./redact');
+const { redact, redactFields } = require('./redact');
 const rules = require('./review-rules');
 
 const SCOPE_SCHEMA = 'bmad-plus/review-scope/1';
@@ -39,6 +39,17 @@ const DISPOSITIONS = ['confirmed', 'refuted', 'unresolved'];
 const CONFIDENCE = ['high', 'medium', 'low'];
 const OUTCOMES = ['completed', 'failed', 'waived'];
 
+/**
+ * Why a review session ended. Only `completed` lets the gate reach a verdict; every other
+ * reason keeps it incomplete, whatever the coverage says.
+ */
+const STOP_REASONS = ['completed', 'budget', 'time-limit', 'failure-streak', 'interrupted'];
+const ATTEMPT_OUTCOMES = ['completed', 'failed'];
+/** Consecutive failed attempts after which a unit is abandoned rather than retried again. */
+const STRIKES = 3;
+const RUN_KEYS = ['stop', 'detail', 'passes', 'attempts', 'tokens', 'durationMs'];
+const ATTEMPT_KEYS = ['unit', 'group', 'outcome', 'reason'];
+
 /** Files whose content must never reach a review packet, whatever an include pattern says. */
 const SECRET_PATTERNS = [
   '**/.env',
@@ -195,6 +206,14 @@ function planUnits(selected, limits = UNIT_LIMITS) {
   }));
 }
 
+function contentDigest(projectDir, file) {
+  try {
+    return sha256(fs.readFileSync(path.join(projectDir, file)));
+  } catch {
+    return null;
+  }
+}
+
 /**
  * The review scope: what changed between base and head (or the working tree), which files
  * are selected, and why every other changed file is not. Secrets are excluded before any
@@ -234,12 +253,24 @@ function buildScope(projectDir, options = {}) {
     }
   }
   changed.sort((a, b) => a.path.localeCompare(b.path));
+  // The review's own evidence is not part of the change under review: writing findings must
+  // never alter the scope they answer.
+  const evidenceDir = path
+    .relative(projectDir, path.resolve(projectDir, options.outputDir || DEFAULT_DIR))
+    .split(path.sep)
+    .join('/');
+  const ownEvidence = (file) =>
+    evidenceDir !== '' &&
+    !evidenceDir.startsWith('..') &&
+    !path.isAbsolute(evidenceDir) &&
+    (file === evidenceDir || file.startsWith(`${evidenceDir}/`));
+  const candidates = changed.filter((entry) => !ownEvidence(entry.path));
 
   const include = options.include || [];
   const exclude = options.exclude || [];
   const selected = [];
   const excluded = [];
-  for (const entry of changed) {
+  for (const entry of candidates) {
     const file = entry.path;
     let reason = null;
     if (matchesAny(file, SECRET_PATTERNS)) reason = 'secret';
@@ -262,10 +293,16 @@ function buildScope(projectDir, options = {}) {
   const effort = options.effort || 'medium';
   if (!EFFORTS[effort]) throw new Error(`effort must be one of ${Object.keys(EFFORTS).join('|')}`);
   const ruleset = options.ruleset || rules.loadRuleset(projectDir);
-  for (const item of selected) item.rules = rules.rulesFor(ruleset, item.path);
+  for (const item of selected) {
+    item.rules = rules.rulesFor(ruleset, item.path);
+    // A commit fixes the content under review; a working tree does not, so its bytes are sealed.
+    if (options.workspace) item.sha256 = contentDigest(projectDir, item.path);
+  }
   const identity = {
     base,
     head: head || 'WORKTREE',
+    // The ref as given, so a continuation can tell whether it still names the reviewed commit.
+    headRef: options.workspace ? null : options.head || 'HEAD',
     mergeBase,
     workspace: Boolean(options.workspace),
     include,
@@ -273,10 +310,11 @@ function buildScope(projectDir, options = {}) {
     effort,
     rulesSha256: ruleset.sha256,
   };
-  const units = planUnits(selected, options.unitLimits).map((unit) => ({
-    ...unit,
-    rules: [...new Set(unit.paths.flatMap((p) => selected.find((s) => s.path === p).rules))],
-  }));
+  const units = planUnits(selected, options.unitLimits).map((unit) => {
+    const ids = [...new Set(unit.paths.flatMap((p) => selected.find((s) => s.path === p).rules))];
+    // Rule groups let parallel reviewers split one unit by rule family.
+    return { ...unit, rules: ids, groups: rules.groupsOf(ruleset, ids) };
+  });
   const lines = selected.reduce((n, item) => n + item.added + item.removed, 0);
   const plan = reviewPlan(effort, lines, units.length);
   const body = { identity, selected, excluded, units, plan };
@@ -285,7 +323,7 @@ function buildScope(projectDir, options = {}) {
     id: options.id,
     ...body,
     totals: {
-      changed: changed.length,
+      changed: candidates.length,
       selected: selected.length,
       excluded: excluded.length,
       lines,
@@ -442,6 +480,148 @@ const REDACTED_FIELDS = [
 
 // ── Coverage and gate ─────────────────────────────────────────────────────────
 
+const isCount = (value) => Number.isInteger(value) && value >= 0;
+const isRecord = (value) => Boolean(value) && typeof value === 'object' && !Array.isArray(value);
+
+/** What an attempt worked on: a whole unit (`u2`) or one rule group of it (`u2/data`). */
+const attemptKey = (attempt) => (attempt.group ? `${attempt.unit}/${attempt.group}` : attempt.unit);
+const unitOfKey = (key) => key.split('/')[0];
+
+/**
+ * The three-strikes rule over an ordered attempt log: work whose attempts failed three
+ * times in a row is exhausted, and any later attempt of it is a violation. A whole unit
+ * and each of its groups overlap: exhausting one blocks the other.
+ */
+function strikeLedger(attempts) {
+  const streak = new Map();
+  const exhausted = new Set();
+  const violations = [];
+  const blocked = (attempt) =>
+    exhausted.has(attempt.unit) ||
+    (attempt.group
+      ? exhausted.has(attemptKey(attempt))
+      : [...exhausted].some((key) => unitOfKey(key) === attempt.unit));
+  for (const [index, attempt] of attempts.entries()) {
+    const key = attemptKey(attempt);
+    if (blocked(attempt)) {
+      violations.push({ index, unit: key });
+      continue;
+    }
+    const failures = attempt.outcome === 'failed' ? (streak.get(key) || 0) + 1 : 0;
+    streak.set(key, failures);
+    if (failures >= STRIKES) exhausted.add(key);
+  }
+  return { exhausted, violations, units: new Set([...exhausted].map(unitOfKey)) };
+}
+
+const unitIndex = (scope) =>
+  new Map(scope.units.flatMap((unit) => unit.paths.map((file) => [file, unit.id])));
+
+/** Units whose review completed: in one attempt, or in one completed attempt per rule group. */
+function completedUnits(scope, attempts) {
+  const done = new Set(attempts.filter((a) => a.outcome === 'completed').map(attemptKey));
+  return new Set(
+    scope.units
+      .filter(
+        (unit) =>
+          done.has(unit.id) ||
+          (Array.isArray(unit.groups) &&
+            unit.groups.length > 0 &&
+            unit.groups.every((group) => done.has(`${unit.id}/${group.id}`)))
+      )
+      .map((unit) => unit.id)
+  );
+}
+
+/**
+ * The run record of a review: why it stopped, what it observed (passes run, units
+ * attempted, and tokens or duration only when the host reports them — never estimated),
+ * and the ordered log of unit attempts. When the log is not empty it is the whole story:
+ * a file is completed only if an attempt of its unit completed.
+ */
+function validateRun(run, scope, items) {
+  if (!isRecord(run)) return ['run must be an object'];
+  const errors = [];
+  for (const key of Object.keys(run))
+    if (!RUN_KEYS.includes(key)) errors.push(`run: unknown key "${key}"`);
+  if (!STOP_REASONS.includes(run.stop))
+    errors.push(`run: stop "${run.stop}" is not one of ${STOP_REASONS.join('|')}`);
+  if (run.detail !== undefined && typeof run.detail !== 'string')
+    errors.push('run: detail is text');
+  else if (STOP_REASONS.includes(run.stop) && run.stop !== 'completed' && !run.detail?.trim())
+    errors.push(`run: a review stopped by ${run.stop} says why in detail`);
+  if (!isCount(run.passes)) errors.push('run: passes is the number of passes actually run');
+  else if (run.stop === 'completed' && run.passes < 1)
+    errors.push('run: a completed review ran at least one pass');
+  for (const key of ['tokens', 'durationMs'])
+    if (run[key] !== undefined && !isCount(run[key]))
+      errors.push(`run: ${key} is a non-negative integer reported by the host, or absent`);
+  if (!Array.isArray(run.attempts)) {
+    errors.push('run: attempts lists every unit attempt, in order (empty when none was logged)');
+    return errors;
+  }
+  const units = new Map(scope.units.map((unit) => [unit.id, unit]));
+  for (const [index, attempt] of run.attempts.entries()) {
+    const at = `run: attempt ${index + 1}`;
+    if (!isRecord(attempt)) {
+      errors.push(`${at}: must be an object`);
+      continue;
+    }
+    for (const key of Object.keys(attempt))
+      if (!ATTEMPT_KEYS.includes(key)) errors.push(`${at}: unknown key "${key}"`);
+    const unit = units.get(attempt.unit);
+    if (!unit) errors.push(`${at}: unit "${attempt.unit}" is not in the scope`);
+    else if (
+      attempt.group !== undefined &&
+      !(unit.groups || []).some((group) => group.id === attempt.group)
+    )
+      errors.push(`${at}: group "${attempt.group}" is not a rule group of unit ${unit.id}`);
+    if (!ATTEMPT_OUTCOMES.includes(attempt.outcome))
+      errors.push(
+        `${at}: outcome "${attempt.outcome}" is not one of ${ATTEMPT_OUTCOMES.join('|')}`
+      );
+    if (attempt.reason !== undefined && typeof attempt.reason !== 'string')
+      errors.push(`${at}: reason is text`);
+    else if (attempt.outcome === 'failed' && !attempt.reason?.trim())
+      errors.push(`${at}: a failed attempt needs a reason`);
+  }
+  if (errors.length) return errors;
+
+  const { exhausted, violations } = strikeLedger(run.attempts);
+  for (const { index, unit } of violations)
+    errors.push(
+      `run: attempt ${index + 1} retries unit ${unit} after ${STRIKES} consecutive failures — it is abandoned in this review`
+    );
+  if (run.stop === 'failure-streak' && !exhausted.size)
+    errors.push(`run: failure-streak needs a unit with ${STRIKES} consecutive failed attempts`);
+  if (run.attempts.length) {
+    const unitOf = unitIndex(scope);
+    const done = completedUnits(scope, run.attempts);
+    for (const item of items)
+      if (item.outcome === 'completed' && unitOf.has(item.path) && !done.has(unitOf.get(item.path)))
+        errors.push(
+          `${item.path}: completed in coverage, but unit ${unitOf.get(item.path)} was not completed — as a whole or in every rule group`
+        );
+  }
+  return errors;
+}
+
+/** What the review actually consumed, as reported; null when no run was recorded. */
+function observedUsage(scope, run) {
+  if (!isRecord(run)) return null;
+  const attempts = Array.isArray(run.attempts) ? run.attempts.filter(isRecord) : [];
+  return {
+    passes: isCount(run.passes) ? run.passes : null,
+    plannedPasses: scope.plan ? scope.plan.passes : null,
+    units: scope.units.length,
+    unitsAttempted: new Set(attempts.map((a) => a.unit)).size,
+    attempts: attempts.length,
+    failedAttempts: attempts.filter((a) => a.outcome === 'failed').length,
+    ...(isCount(run.tokens) ? { tokens: run.tokens } : {}),
+    ...(isCount(run.durationMs) ? { durationMs: run.durationMs } : {}),
+  };
+}
+
 function validateCoverage(doc, scope) {
   const errors = [];
   if (!doc || doc.schema !== COVERAGE_SCHEMA) errors.push(`schema must be "${COVERAGE_SCHEMA}"`);
@@ -461,12 +641,13 @@ function validateCoverage(doc, scope) {
     )
       errors.push(`${item.path}: a ${item.outcome} file needs a reason`);
   }
+  if (doc && doc.run !== undefined) errors.push(...validateRun(doc.run, scope, doc.items || []));
   return errors;
 }
 
 /**
  * The review verdict. `incomplete` whenever a selected file was not accounted for, failed,
- * or the evidence is invalid; otherwise `findings` or `clean` — the latter only means no
+ * the review stopped before the end, or the evidence is invalid; otherwise `findings` or `clean` — the latter only means no
  * confirmed or unresolved finding within a fully covered scope.
  */
 function reviewGate({ scope, findings, coverage, anchored }) {
@@ -489,6 +670,17 @@ function reviewGate({ scope, findings, coverage, anchored }) {
     );
   if (failed.length)
     reasons.push(`${failed.length} file(s) whose review failed: ${failed.slice(0, 5).join(', ')}`);
+  const run = coverage && isRecord(coverage.run) ? coverage.run : null;
+  // A review that stopped early is never clean, even when every file happens to be listed.
+  if (run && STOP_REASONS.includes(run.stop) && run.stop !== 'completed')
+    reasons.push(
+      `the review stopped before the end (${run.stop}): ${typeof run.detail === 'string' ? run.detail.trim() : ''}`
+    );
+  if (run && Array.isArray(run.attempts)) {
+    const { exhausted } = strikeLedger(run.attempts.filter(isRecord));
+    for (const unit of exhausted)
+      reasons.push(`unit ${unit} abandoned after ${STRIKES} consecutive failed attempts`);
+  }
   const open = ((anchored && anchored.findings) || []).filter((f) => f.disposition !== 'refuted');
   const unlocated = open.filter((f) => f.location.status !== 'located');
   if (unlocated.length)
@@ -504,7 +696,11 @@ function reviewGate({ scope, findings, coverage, anchored }) {
   const status = reasons.length ? 'incomplete' : open.length ? 'findings' : 'clean';
   return {
     status,
-    reasons,
+    // Reasons echo host-written text (a stop detail, an invalid value), and they reach the
+    // terminal, the CI log and the check document alike: redacted once, here.
+    reasons: reasons.map((reason) => redact(reason).text),
+    stop: run && STOP_REASONS.includes(run.stop) ? run.stop : null,
+    usage: observedUsage(scope, run),
     coverage: {
       selected: total,
       completed: done,
@@ -525,6 +721,255 @@ function reviewGate({ scope, findings, coverage, anchored }) {
   };
 }
 
+// ── Check result for CI ───────────────────────────────────────────────────────
+
+const CHECK_SCHEMA = 'bmad-plus/review-check/1';
+const GATE_EXIT = { clean: 0, findings: 1, incomplete: 2 };
+const ANNOTATION_LEVEL = { critical: 'failure', high: 'failure', medium: 'warning', low: 'notice' };
+/** The GitHub check-runs API accepts at most 50 annotations per request. */
+const MAX_ANNOTATIONS = 50;
+const MAX_SUMMARY_ROWS = 50;
+
+const cell = (value) =>
+  String(value)
+    .replace(/[|\\`]/g, '\\$&')
+    .replace(/\s+/g, ' ');
+
+/**
+ * The gate verdict as a stable document a CI step consumes without parsing prose: the
+ * disposition and its exit code, the reasons, the counts, the scope it answers, the open
+ * findings with their anchored lines, and a `github` block shaped like the check-runs API
+ * (`conclusion` and `output` with a Markdown summary and annotations). Free text is passed
+ * through the redaction floor; the document carries no timestamp, so the same evidence
+ * always yields the same bytes.
+ */
+function checkResult({ id, scope, verdict, anchored }) {
+  const rank = (severity) => SEVERITIES.indexOf(severity);
+  const open = ((anchored && anchored.findings) || [])
+    .filter((f) => f.disposition !== 'refuted')
+    .map((f) => ({
+      id: f.id,
+      path: f.path,
+      severity: f.severity,
+      category: f.category,
+      confidence: f.confidence,
+      disposition: f.disposition,
+      ...(f.rule !== undefined ? { rule: f.rule } : {}),
+      anchor: f.location.status,
+      lineStart: f.location.status === 'located' ? f.location.lineStart : null,
+      lineEnd: f.location.status === 'located' ? f.location.lineEnd : null,
+      message: f.content,
+    }))
+    .sort(
+      (a, b) =>
+        rank(a.severity) - rank(b.severity) ||
+        a.path.localeCompare(b.path) ||
+        (a.lineStart || 0) - (b.lineStart || 0) ||
+        String(a.id).localeCompare(String(b.id))
+    );
+  const reasons = verdict.reasons.map((reason) => redact(reason).text);
+  const located = open.filter((f) => f.anchor === 'located');
+  const annotations = located.slice(0, MAX_ANNOTATIONS).map((f) => ({
+    path: f.path,
+    start_line: f.lineStart,
+    end_line: f.lineEnd,
+    annotation_level: ANNOTATION_LEVEL[f.severity],
+    title: `${f.severity} ${f.category} (${f.id})`,
+    message: f.message,
+  }));
+  const { coverage, findings } = verdict;
+  const summary = [
+    `**${verdict.status}**: ${coverage.completed}/${coverage.selected} selected file(s) completed, ${coverage.waived} waived, ${coverage.failed} failed, ${coverage.missing} missing; ${findings.open} open finding(s), ${findings.refuted} refuted.`,
+    `Scope \`${scope.sha256.slice(0, 12)}\` (${scope.identity.effort || 'medium'} effort)${verdict.stop ? `, stopped: ${verdict.stop}` : ''}.`,
+    ...(reasons.length ? ['', ...reasons.map((reason) => `- ${cell(reason)}`)] : []),
+    ...(open.length
+      ? [
+          '',
+          '| Severity | Category | Finding | Location |',
+          '|---|---|---|---|',
+          ...open
+            .slice(0, MAX_SUMMARY_ROWS)
+            .map(
+              (f) =>
+                `| ${f.severity} | ${f.category} | ${cell(f.id)} | ${cell(f.path)}${f.lineStart ? `:${f.lineStart}` : ` (${f.anchor})`} |`
+            ),
+          ...(open.length > MAX_SUMMARY_ROWS
+            ? [`\n${open.length - MAX_SUMMARY_ROWS} more in the check document.`]
+            : []),
+        ]
+      : []),
+    ...(verdict.status === 'clean'
+      ? [
+          '',
+          'Clean means no open finding within a fully covered scope; it does not prove the code correct.',
+        ]
+      : []),
+  ].join('\n');
+  const body = {
+    id,
+    scopeSha256: scope.sha256,
+    identity: {
+      base: scope.identity.base,
+      head: scope.identity.head,
+      mergeBase: scope.identity.mergeBase,
+      workspace: scope.identity.workspace,
+      effort: scope.identity.effort || null,
+      rulesSha256: scope.identity.rulesSha256 || null,
+    },
+    status: verdict.status,
+    exitCode: GATE_EXIT[verdict.status],
+    reasons,
+    stop: verdict.stop,
+    usage: verdict.usage,
+    coverage,
+    findings,
+    open,
+    github: {
+      name: `bmad-plus review ${id}`,
+      conclusion: verdict.status === 'clean' ? 'success' : 'failure',
+      output: {
+        title: `Review ${verdict.status}: ${findings.open} open finding(s), ${coverage.completed}/${coverage.selected} file(s) completed`,
+        summary,
+        annotations,
+      },
+      annotationsOmitted: located.length - annotations.length,
+    },
+  };
+  return { schema: CHECK_SCHEMA, ...body, sha256: sha256(JSON.stringify(body)) };
+}
+
+// ── Continuing an interrupted review ──────────────────────────────────────────
+
+const CONTINUE_SCHEMA = 'bmad-plus/review-continue/1';
+const short = (sha) => String(sha).slice(0, 12);
+
+/**
+ * Why a sealed scope no longer describes the code: its head ref names another commit, the
+ * review rules changed, or the change itself did (for a working tree, any selected byte).
+ * Empty when the scope rebuilt now is exactly the scope that was sealed.
+ */
+function scopeDrift(projectDir, scope, options = {}) {
+  const { identity } = scope;
+  // Since continuation, every scope carries headRef (null for a working tree); an older one,
+  // commit or working tree, lacks the key along with the per-file digests it relies on.
+  if (!('headRef' in identity) || (!identity.workspace && !identity.headRef))
+    return ['the scope was sealed without its head ref, before continuation existed'];
+  const drift = [];
+  if (!identity.workspace) {
+    let now = null;
+    try {
+      now = resolveCommit(projectDir, identity.headRef);
+    } catch {
+      // Reported just below: a ref that no longer resolves has moved too.
+    }
+    if (now !== identity.head)
+      drift.push(
+        now
+          ? `${identity.headRef} moved from ${short(identity.head)} to ${short(now)}`
+          : `${identity.headRef} no longer resolves to a commit`
+      );
+  }
+  let rebuilt;
+  try {
+    rebuilt = buildScope(projectDir, {
+      id: scope.id,
+      base: identity.base,
+      head: identity.workspace ? undefined : identity.headRef,
+      workspace: identity.workspace,
+      include: identity.include,
+      exclude: identity.exclude,
+      effort: identity.effort,
+      ruleset: options.ruleset,
+      outputDir: options.outputDir,
+    });
+  } catch (error) {
+    return [...drift, `the scope cannot be rebuilt: ${error.message}`];
+  }
+  if (rebuilt.sha256 === scope.sha256 || drift.length) return drift;
+  if (rebuilt.identity.rulesSha256 !== identity.rulesSha256) drift.push('the review rules changed');
+  // Rules are compared above; here only what each file is and how it changed.
+  const describe = (list) =>
+    new Map(list.map((item) => [item.path, JSON.stringify({ ...item, rules: undefined })]));
+  const sealed = describe(scope.selected);
+  const current = describe(rebuilt.selected);
+  const moved = [...new Set([...sealed.keys(), ...current.keys()])]
+    .filter((file) => sealed.get(file) !== current.get(file))
+    .sort();
+  if (moved.length)
+    drift.push(
+      `${moved.length} file(s) changed since the scope was sealed: ${moved.slice(0, 5).join(', ')}${moved.length > 5 ? '…' : ''}`
+    );
+  if (!drift.length) drift.push('the scope rebuilt now differs from the sealed one');
+  return drift;
+}
+
+/**
+ * What an interrupted review still owes against its unchanged scope: selected files never
+ * accounted for or whose review failed, grouped back into their units; files of units
+ * abandoned under the three-strikes rule, which this review will not retry; open findings
+ * to requote; and the passes the plan still asks for when the run stopped early.
+ */
+function remainingWork({ scope, coverage, anchored, findingErrors = [] }) {
+  const accounted = new Map(((coverage && coverage.items) || []).map((item) => [item.path, item]));
+  const run = coverage && isRecord(coverage.run) ? coverage.run : null;
+  const attempts = run && Array.isArray(run.attempts) ? run.attempts.filter(isRecord) : [];
+  const { units: exhausted } = strikeLedger(attempts);
+  const done = new Set(attempts.filter((a) => a.outcome === 'completed').map(attemptKey));
+  const unitOf = unitIndex(scope);
+  const files = [];
+  const abandoned = [];
+  for (const item of scope.selected) {
+    const entry = accounted.get(item.path);
+    if (entry && entry.outcome !== 'failed') continue;
+    const record = {
+      path: item.path,
+      unit: unitOf.get(item.path),
+      state: entry ? 'failed' : 'missing',
+    };
+    (exhausted.has(record.unit) ? abandoned : files).push(record);
+  }
+  const pending = new Set(files.map((file) => file.path));
+  const units = scope.units
+    .map((unit) => ({
+      id: unit.id,
+      paths: unit.paths.filter((file) => pending.has(file)),
+      rules: unit.rules,
+      // Rule groups another reviewer already completed are not handed out again.
+      ...(unit.groups
+        ? { groups: unit.groups.filter((group) => !done.has(`${unit.id}/${group.id}`)) }
+        : {}),
+    }))
+    .filter((unit) => unit.paths.length);
+  const requote = ((anchored && anchored.findings) || [])
+    .filter((f) => f.disposition !== 'refuted' && f.location.status !== 'located')
+    .map((f) => ({ id: f.id, path: f.path, status: f.location.status }));
+  const usage = observedUsage(scope, run);
+  const stopped = run && run.stop !== 'completed';
+  const passes =
+    stopped && usage.passes !== null && usage.plannedPasses !== null
+      ? Math.max(0, usage.plannedPasses - usage.passes)
+      : 0;
+  return {
+    schema: CONTINUE_SCHEMA,
+    id: scope.id,
+    scopeSha256: scope.sha256,
+    previous: run ? { stop: STOP_REASONS.includes(run.stop) ? run.stop : null, usage } : null,
+    counts: {
+      files: files.length,
+      units: units.length,
+      abandoned: abandoned.length,
+      requote: requote.length,
+      findingErrors: findingErrors.length,
+      passes,
+    },
+    units,
+    files,
+    abandoned,
+    requote,
+    findingErrors,
+  };
+}
+
 // ── Comparing two reviews ─────────────────────────────────────────────────────
 
 const COMPARE_SCHEMA = 'bmad-plus/review-compare/1';
@@ -602,6 +1047,7 @@ function layout(projectDir, dir = DEFAULT_DIR, id) {
     coverage: path.join(root, 'coverage.json'),
     checklist: path.join(root, 'checklist.md'),
     compare: path.join(root, 'compare.json'),
+    continue: path.join(root, 'continue.json'),
   };
 }
 
@@ -617,6 +1063,8 @@ module.exports = {
   DISPOSITIONS,
   CONFIDENCE,
   OUTCOMES,
+  STOP_REASONS,
+  STRIKES,
   SECRET_PATTERNS,
   GENERATED_PATTERNS,
   parseNumstat,
@@ -626,7 +1074,15 @@ module.exports = {
   locate,
   anchorFindings,
   validateCoverage,
+  validateRun,
+  strikeLedger,
   reviewGate,
+  CHECK_SCHEMA,
+  GATE_EXIT,
+  checkResult,
+  CONTINUE_SCHEMA,
+  scopeDrift,
+  remainingWork,
   reviewPlan,
   findingKey,
   compareReviews,

package/tools/cli/review-rules/index.yaml

@@ -1,36 +1,45 @@
 # Built-in review checklists, applied by path. Every matching rule applies to a file.
 # A project adds, replaces (same id) or disables rules in _bmad/review-rules.yaml.
+# A group is a rule family: parallel reviewers can split one unit by group.
 schema: bmad-plus/review-rules/1
 rules:
   - id: general
     title: Every change
+    group: code
     globs: ['**']
     doc: general.md
   - id: javascript-typescript
     title: JavaScript and TypeScript
+    group: code
     globs: ['**/*.{js,cjs,mjs,jsx,ts,cts,mts,tsx}', '**/*.{vue,svelte}']
     doc: javascript-typescript.md
   - id: python
     title: Python
+    group: code
     globs: ['**/*.{py,pyi}']
     doc: python.md
   - id: sql-and-migrations
     title: SQL, queries and migrations
+    group: data
     globs: ['**/*.sql', '**/{migrations,migrate,db/migrate}/**', '**/*.prisma', '**/schema.rb']
     doc: sql-and-migrations.md
   - id: shell
     title: Shell and PowerShell scripts
+    group: code
     globs: ['**/*.{sh,bash,zsh,ps1,psm1}', '**/{Makefile,makefile}']
     doc: shell.md
   - id: ci-workflows
     title: CI workflows
+    group: delivery
     globs: ['.github/workflows/**/*.{yml,yaml}', '**/.gitlab-ci.yml', '**/{bitbucket-pipelines.yml,azure-pipelines.yml}']
     doc: ci-workflows.md
   - id: containers
     title: Containers and deployment manifests
+    group: delivery
     globs: ['**/{Dockerfile,Containerfile}', '**/*.dockerfile', '**/{docker-,}compose*.{yml,yaml}', '**/{k8s,kubernetes,helm,charts}/**/*.{yml,yaml}']
     doc: containers.md
   - id: configuration
     title: Configuration and dependency manifests
+    group: delivery
     globs: ['**/*.{json,yml,yaml,toml,ini,properties}', '**/.env.example', '**/{package.json,pyproject.toml,requirements*.txt,go.mod,Cargo.toml,composer.json}']
     doc: configuration.md