@rigour-labs/cli 6.12.3-rc.1 → 6.12.3-rc.3

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,14 +51,23 @@ 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) {
+/** Findings left out of the verdict, by check, most first (`: ast-analysis 12, file-size 1`), as `checked.preexisting` or `checked.outsideChange` gives them in JSON. */
+function leftOutByCheck(counts) {
+    const parts = countsByCheck(new Map(Object.entries(counts ?? {})));
+    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 counts of [result.preexistingByCheck, result.outsideChangeByCheck])
-        for (const [check, n] of Object.entries(counts ?? {}))
-            byCheck.set(check, (byCheck.get(check) ?? 0) + n);
-    const parts = [...byCheck].sort((a, b) => b[1] - a[1] || a[0].localeCompare(b[0])).map(([check, n]) => `${check} ${n}`);
-    return parts.length ? `: ${parts.join(', ')}` : '';
+    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) {
@@ -66,9 +75,16 @@ function printQuietLines(result, notes) {
     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 before = result.preexisting + result.excludedOutsideChangedLines;
+    const wholeFile = fileFindingsByCheck(result);
+    if (wholeFile)
+        lines.push(`About a changed file as a whole: ${wholeFile}${notes ? ' (listed below)' : ', shown with --notes'}.`);
+    // Two kinds, each with its own pointer: show_preexisting lists only the first.
+    const before = result.preexisting;
     if (before)
-        lines.push(`Not shown: ${before} issue${before === 1 ? '' : 's'} the code already had before this change${leftOutByCheck(result)} (review.show_preexisting: true lists them).`);
+        lines.push(`Not shown: ${before} issue${before === 1 ? '' : 's'} the code already had before this change${leftOutByCheck(result.preexistingByCheck)} (review.show_preexisting: true lists them).`);
+    const outside = result.excludedOutsideChangedLines;
+    if (outside)
+        lines.push(`Not shown: ${outside} issue${outside === 1 ? '' : 's'} on lines this change did not touch${leftOutByCheck(result.outsideChangeByCheck)}.`);
     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.3-rc.1",
+  "version": "6.12.3-rc.3",
   "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.3-rc.1"
+    "@rigour-labs/core": "6.12.3-rc.3"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",
     "@types/inquirer": "9.0.7",
     "@types/node": "^25.0.3",
-    "@rigour-labs/studio": "6.12.3-rc.1"
+    "@rigour-labs/studio": "6.12.3-rc.3"
   },
   "scripts": {
     "build": "tsc && pnpm bundle-studio",