@rigour-labs/cli 6.13.0-rc.4 → 6.13.0-rc.5

package/dist/commands/check.js

@@ -8,6 +8,7 @@ import { randomUUID } from 'crypto';
 import { EXIT_PASS, EXIT_FAIL, EXIT_CONFIG_ERROR, EXIT_INTERNAL_ERROR, exitCodeFor } from './exit-codes.js';
 import { deepProvider } from './deep-provider.js';
 import { UsageError } from './review-config.js';
+import { scoreNote } from './score-note.js';
 // Helper to log events for Rigour Studio
 async function logStudioEvent(cwd, event) {
     try {
@@ -80,6 +81,9 @@ export async function checkCommand(cwd, files = [], options = {}) {
                         console.log(trendColor(`\nScore Trend: ${scoresStr} (${trend.direction} ${arrow})`));
                     }
                     console.log(chalk.dim(`\nFinished in ${result.checkMs}ms (cached) | Score: ${result.report.stats.score ?? '?'}/100`));
+                    const cachedNote = scoreNote(result.report.stats.score);
+                    if (cachedNote)
+                        console.log(chalk.dim(cachedNote));
                     process.exit(result.report.status === 'PASS' ? EXIT_PASS : EXIT_FAIL);
                 }
             }
@@ -236,6 +240,9 @@ export async function checkCommand(cwd, files = [], options = {}) {
             footerParts.push(`Score: ${report.stats.score}/100`);
         }
         console.log(chalk.dim('\n' + footerParts.join(' | ')));
+        const note = scoreNote(report.stats.score);
+        if (note)
+            console.log(chalk.dim(note));
         process.exit(exitCodeFor(report));
     }
     catch (error) {

package/dist/commands/review-human.js

@@ -69,43 +69,51 @@ function fileFindingsByCheck(result) {
     }
     return countsByCheck(byCheck);
 }
-/** One line each, in plain words, for what was seen and never blocks. */
+/**
+ * One line each, in plain words, for what was seen and never blocks. With --notes, each list is printed under its own
+ * line, so nothing listed sits under a "Not shown" line and reads as the issue it left out.
+ */
 function printQuietLines(result, notes) {
-    const lines = [];
+    const groups = [];
+    const item = (f) => `${f.files?.[0] || '?'}:${f.line ?? '?'}  ${f.title}`;
     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)'}`);
+        groups.push({ line: `Also seen, never blocking: ${seen} note${seen === 1 ? '' : 's'}${notes ? '' : ' (rigour review --notes)'}`, items: result.advisory.map(item) });
     const wholeFile = fileFindingsByCheck(result);
+    // The count above includes these: with notes on a line too, say they are part of it.
     if (wholeFile)
-        lines.push(`About a changed file as a whole: ${wholeFile}${notes ? ' (listed below)' : ', shown with --notes'}.`);
+        groups.push({ line: `${result.advisory.length ? 'Of these, about' : 'About'} a changed file as a whole: ${wholeFile}${notes ? ' (listed below)' : ', shown with --notes'}.`, items: result.fileFindings.map(item) });
+    if (result.hints.length)
+        groups.push({ line: `To confirm by hand: ${result.hints.length} hint${result.hints.length === 1 ? '' : 's'}${notes ? '' : ' (rigour review --notes)'}`, items: result.hints });
     // 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.preexistingByCheck)} (review.show_preexisting: true lists them).`);
+        groups.push({ line: `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)}.`);
+        groups.push({ line: `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)
-        lines.push(`To confirm by hand: ${result.hints.length} hint${result.hints.length === 1 ? '' : 's'}${notes ? '' : ' (rigour review --notes)'}`);
+        groups.push({ line: '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.dismissed)
-        lines.push(`Dismissed earlier as not a bug: ${result.dismissed}.`);
+        groups.push({ line: `Dismissed earlier as not a bug: ${result.dismissed}.` });
     if (result.muted)
-        lines.push(`Muted: ${result.muted} from checks this repository usually dismisses (rigour precision).`);
-    for (const line of lines)
-        console.log(chalk.dim(line));
+        groups.push({ line: `Muted: ${result.muted} from checks this repository usually dismisses (rigour precision).` });
+    let open = false;
+    for (const group of groups) {
+        console.log(chalk.dim(group.line));
+        const listed = notes ? group.items ?? [] : [];
+        for (const text of listed)
+            console.log(chalk.dim(`    ${text}`));
+        // A blank line closes a list, so the next line is not read as part of it.
+        if (listed.length)
+            console.log('');
+        open = !listed.length;
+    }
     if (result.controlFilesChanged.length)
         console.log(chalk.yellow(`This change edits Rigour's own settings: ${result.controlFilesChanged.join(', ')}`));
     for (const f of result.contextFindings)
         console.log(chalk.yellow(`Nearby, not in your change: ${f.files?.[0] || '?'}:${f.line ?? '?'} ${f.title}`));
-    if (notes) {
-        for (const f of [...result.advisory, ...result.fileFindings])
-            console.log(chalk.dim(`  ${f.files?.[0] || '?'}:${f.line ?? '?'}  ${f.title}`));
-        for (const hint of result.hints)
-            console.log(chalk.dim(`  ${hint}`));
-    }
-    if (lines.length || result.controlFilesChanged.length || result.contextFindings.length)
+    if (open || result.controlFilesChanged.length || result.contextFindings.length)
         console.log('');
 }
 /** Why the review is not a pass although it may have found nothing, and the one step that finishes it. */

package/dist/commands/review.js

@@ -27,6 +27,7 @@ import { loadConfig, UsageError } from './review-config.js';
 import { deepProvider } from './deep-provider.js';
 import { buildCiReviewSummary, renderGithubSummary } from './review-summary.js';
 import { EXIT_PASS, EXIT_FAIL, EXIT_CONFIG_ERROR, EXIT_INTERNAL_ERROR } from './exit-codes.js';
+import { scoreNote } from './score-note.js';
 export async function reviewCommand(cwd, options = {}) {
     const started = Date.now();
     if (options.status)
@@ -229,14 +230,19 @@ function writeJson(result, summary, receipt, reviewer, checked, goal) {
 function printCi(result) {
     const score = result.report?.stats.score;
     const scoreStr = score !== undefined ? ` (${score}/100)` : '';
+    const note = scoreNote(score, true);
     if (result.findings.length === 0) {
         console.log(`${result.status === 'ERROR' ? 'ERROR' : 'PASS'}${scoreStr}`);
+        if (note)
+            console.log(`  ${note}`);
         return;
     }
     console.log(`FAIL: ${result.findings.length} violation(s) on changed lines${scoreStr}`);
     for (const f of result.findings) {
         console.log(`  - [${(f.severity || 'medium').toUpperCase()}] ${f.files?.[0] || ''}:${f.line ?? '?'} ${f.title}`);
     }
+    if (note)
+        console.log(`  ${note}`);
 }
 /** Without a model, point at the risky changed functions a person or their agent should still check. */
 function hintReviewTask(cwd, diff, router) {

package/dist/commands/score-note.d.ts

@@ -0,0 +1,2 @@
+/** Undefined at 100 or with no score; `review` adds that only the findings to fix decide its verdict. */
+export declare function scoreNote(score: number | undefined, review?: boolean): string | undefined;

package/dist/commands/score-note.js

@@ -0,0 +1,12 @@
+/**
+ * What a score below 100 means, in one line: a person seeing "PASS (62/100)" should not have to guess why a note that
+ * never blocks cost points. The weights are read from the ones the score is computed with (SEVERITY_WEIGHTS).
+ */
+import { SEVERITY_WEIGHTS } from '@rigour-labs/core';
+/** Undefined at 100 or with no score; `review` adds that only the findings to fix decide its verdict. */
+export function scoreNote(score, review = false) {
+    if (score === undefined || score >= 100)
+        return undefined;
+    const weights = ['critical', 'high', 'medium', 'low'].map(s => `${s} ${SEVERITY_WEIGHTS[s]}`).join(', ');
+    return `Score ${score}/100 counts every finding in the files checked, notes included (${weights} points each, capped per check)${review ? '; only the findings to fix decide the verdict' : ''}.`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rigour-labs/cli",
-  "version": "6.13.0-rc.4",
+  "version": "6.13.0-rc.5",
   "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.13.0-rc.4"
+    "@rigour-labs/core": "6.13.0-rc.5"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",
     "@types/inquirer": "9.0.7",
     "@types/node": "^25.0.3",
-    "@rigour-labs/studio": "6.13.0-rc.4"
+    "@rigour-labs/studio": "6.13.0-rc.5"
   },
   "scripts": {
     "build": "tsc && pnpm bundle-studio",