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

package/dist/commands/review-human.js

@@ -51,13 +51,9 @@ 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);
+/** 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. */
@@ -82,9 +78,13 @@ function printQuietLines(result, 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;
+    // 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.2",
+  "version": "6.12.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.2"
+    "@rigour-labs/core": "6.12.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.2"
+    "@rigour-labs/studio": "6.12.3"
   },
   "scripts": {
     "build": "tsc && pnpm bundle-studio",