@rigour-labs/cli 6.12.2 → 6.12.3-rc.2

package/dist/commands/review-checked.d.ts

@@ -10,10 +10,12 @@ export interface Checked {
     uncommitted: boolean;
     /** The settings read: a path, `<path> at <commit>` for an independent review, or `defaults`. */
     config: string;
-    /** Every check the run reached, by id: PASS, FAIL, SKIP or ERROR. A check whose findings were all already in the base reads PASS. */
+    /** Every check the run reached, by id: PASS, FAIL, SKIP or ERROR. A check whose findings were all already in the base, or only on lines the change did not touch, reads PASS. */
     checks: Record<string, string>;
     /** By check, how many findings the base already had: left out of the verdict and of `checks`. */
     preexisting: Record<string, number>;
+    /** By check, how many findings sat only on lines the change did not touch: left out of the verdict and of `checks`. */
+    outsideChange: Record<string, number>;
 }
 export declare function whatWasChecked(cwd: string, options: {
     base?: string;

package/dist/commands/review-checked.js

@@ -18,6 +18,7 @@ export function whatWasChecked(cwd, options, trustedRef, result) {
         config: configSource(cwd, options, trustedRef),
         checks: { ...(result.report?.summary ?? {}) },
         preexisting: { ...result.preexistingByCheck },
+        outsideChange: { ...result.outsideChangeByCheck },
     };
 }
 function git(cwd, args) {

package/dist/commands/review-human.js

@@ -10,7 +10,7 @@
 import chalk from 'chalk';
 import fs from 'fs';
 import path from 'path';
-import { findingKey } from '@rigour-labs/core';
+import { findingKey, gateOf } from '@rigour-labs/core';
 import { enabledHere } from './personal.js';
 import { printReceipt } from './review-receipt.js';
 const SHOWN = 5;
@@ -51,15 +51,40 @@ function printFindings(result, all) {
         console.log(chalk.dim(`\n  …and ${count - SHOWN} more: rigour review --all`));
     console.log(chalk.dim('\n  Wrong? Dismiss it once and it never comes back.\n'));
 }
+/** The left-out findings by check, most first (`: ast-analysis 12, file-size 1`), as `checked.preexisting` and `checked.outsideChange` give them in JSON. */
+function leftOutByCheck(result) {
+    const byCheck = new Map();
+    for (const counts of [result.preexistingByCheck, result.outsideChangeByCheck])
+        for (const [check, n] of Object.entries(counts ?? {}))
+            byCheck.set(check, (byCheck.get(check) ?? 0) + n);
+    const parts = countsByCheck(byCheck);
+    return parts ? `: ${parts}` : '';
+}
+/** `ast-analysis 12, file-size 3`: most first, then by name. */
+function countsByCheck(byCheck) {
+    return [...byCheck].sort((a, b) => b[1] - a[1] || a[0].localeCompare(b[0])).map(([check, n]) => `${check} ${n}`).join(', ');
+}
+/** Findings about a changed file as a whole (no line), by the check that gave them: they fail their check without a row of their own. */
+function fileFindingsByCheck(result) {
+    const byCheck = new Map();
+    for (const f of result.fileFindings) {
+        const check = gateOf(f) ?? f.id;
+        byCheck.set(check, (byCheck.get(check) ?? 0) + 1);
+    }
+    return countsByCheck(byCheck);
+}
 /** One line each, in plain words, for what was seen and never blocks. */
 function printQuietLines(result, notes) {
     const lines = [];
     const seen = result.advisory.length + result.fileFindings.length;
     if (seen)
         lines.push(`Also seen, never blocking: ${seen} note${seen === 1 ? '' : 's'}${notes ? '' : ' (rigour review --notes)'}`);
+    const wholeFile = fileFindingsByCheck(result);
+    if (wholeFile)
+        lines.push(`About a changed file as a whole: ${wholeFile}${notes ? ' (listed below)' : ', shown with --notes'}.`);
     const before = result.preexisting + result.excludedOutsideChangedLines;
     if (before)
-        lines.push(`Not shown: ${before} issue${before === 1 ? '' : 's'} the code already had before this change (review.show_preexisting: true lists them).`);
+        lines.push(`Not shown: ${before} issue${before === 1 ? '' : 's'} the code already had before this change${leftOutByCheck(result)} (review.show_preexisting: true lists them).`);
     if (result.baseUnknown)
         lines.push('Compared with no base: HEAD already holds this diff, so findings on its lines were not checked against the code before it. Pass --base to compare.');
     if (result.hints.length)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rigour-labs/cli",
-  "version": "6.12.2",
+  "version": "6.12.3-rc.2",
   "description": "Code review that happens while your agent writes: deterministic checks on the lines it changed, quiet unless it can prove a defect, and learning from your team's fixes and dismissals. Works with Claude Code, Cursor, Codex, Cline and Windsurf.",
   "engines": {
     "node": ">=22.13"
@@ -59,13 +59,13 @@
     "inquirer": "9.2.16",
     "ora": "^8.0.1",
     "yaml": "^2.8.2",
-    "@rigour-labs/core": "6.12.2"
+    "@rigour-labs/core": "6.12.3-rc.2"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",
     "@types/inquirer": "9.0.7",
     "@types/node": "^25.0.3",
-    "@rigour-labs/studio": "6.12.2"
+    "@rigour-labs/studio": "6.12.3-rc.2"
   },
   "scripts": {
     "build": "tsc && pnpm bundle-studio",