@rigour-labs/cli 6.12.3-rc.1 → 6.12.3-rc.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -10,7 +10,7 @@
|
|
|
10
10
|
import chalk from 'chalk';
|
|
11
11
|
import fs from 'fs';
|
|
12
12
|
import path from 'path';
|
|
13
|
-
import { findingKey } from '@rigour-labs/core';
|
|
13
|
+
import { findingKey, gateOf } from '@rigour-labs/core';
|
|
14
14
|
import { enabledHere } from './personal.js';
|
|
15
15
|
import { printReceipt } from './review-receipt.js';
|
|
16
16
|
const SHOWN = 5;
|
|
@@ -57,8 +57,21 @@ function leftOutByCheck(result) {
|
|
|
57
57
|
for (const counts of [result.preexistingByCheck, result.outsideChangeByCheck])
|
|
58
58
|
for (const [check, n] of Object.entries(counts ?? {}))
|
|
59
59
|
byCheck.set(check, (byCheck.get(check) ?? 0) + n);
|
|
60
|
-
const parts =
|
|
61
|
-
return parts
|
|
60
|
+
const parts = countsByCheck(byCheck);
|
|
61
|
+
return parts ? `: ${parts}` : '';
|
|
62
|
+
}
|
|
63
|
+
/** `ast-analysis 12, file-size 3`: most first, then by name. */
|
|
64
|
+
function countsByCheck(byCheck) {
|
|
65
|
+
return [...byCheck].sort((a, b) => b[1] - a[1] || a[0].localeCompare(b[0])).map(([check, n]) => `${check} ${n}`).join(', ');
|
|
66
|
+
}
|
|
67
|
+
/** 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. */
|
|
68
|
+
function fileFindingsByCheck(result) {
|
|
69
|
+
const byCheck = new Map();
|
|
70
|
+
for (const f of result.fileFindings) {
|
|
71
|
+
const check = gateOf(f) ?? f.id;
|
|
72
|
+
byCheck.set(check, (byCheck.get(check) ?? 0) + 1);
|
|
73
|
+
}
|
|
74
|
+
return countsByCheck(byCheck);
|
|
62
75
|
}
|
|
63
76
|
/** One line each, in plain words, for what was seen and never blocks. */
|
|
64
77
|
function printQuietLines(result, notes) {
|
|
@@ -66,6 +79,9 @@ function printQuietLines(result, notes) {
|
|
|
66
79
|
const seen = result.advisory.length + result.fileFindings.length;
|
|
67
80
|
if (seen)
|
|
68
81
|
lines.push(`Also seen, never blocking: ${seen} note${seen === 1 ? '' : 's'}${notes ? '' : ' (rigour review --notes)'}`);
|
|
82
|
+
const wholeFile = fileFindingsByCheck(result);
|
|
83
|
+
if (wholeFile)
|
|
84
|
+
lines.push(`About a changed file as a whole: ${wholeFile}${notes ? ' (listed below)' : ', shown with --notes'}.`);
|
|
69
85
|
const before = result.preexisting + result.excludedOutsideChangedLines;
|
|
70
86
|
if (before)
|
|
71
87
|
lines.push(`Not shown: ${before} issue${before === 1 ? '' : 's'} the code already had before this change${leftOutByCheck(result)} (review.show_preexisting: true lists them).`);
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@rigour-labs/cli",
|
|
3
|
-
"version": "6.12.3-rc.
|
|
3
|
+
"version": "6.12.3-rc.2",
|
|
4
4
|
"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.",
|
|
5
5
|
"engines": {
|
|
6
6
|
"node": ">=22.13"
|
|
@@ -59,13 +59,13 @@
|
|
|
59
59
|
"inquirer": "9.2.16",
|
|
60
60
|
"ora": "^8.0.1",
|
|
61
61
|
"yaml": "^2.8.2",
|
|
62
|
-
"@rigour-labs/core": "6.12.3-rc.
|
|
62
|
+
"@rigour-labs/core": "6.12.3-rc.2"
|
|
63
63
|
},
|
|
64
64
|
"devDependencies": {
|
|
65
65
|
"@types/fs-extra": "^11.0.4",
|
|
66
66
|
"@types/inquirer": "9.0.7",
|
|
67
67
|
"@types/node": "^25.0.3",
|
|
68
|
-
"@rigour-labs/studio": "6.12.3-rc.
|
|
68
|
+
"@rigour-labs/studio": "6.12.3-rc.2"
|
|
69
69
|
},
|
|
70
70
|
"scripts": {
|
|
71
71
|
"build": "tsc && pnpm bundle-studio",
|