@rigour-labs/cli 6.9.0 → 6.10.0-rc.2

package/dist/commands/hooks-check-repos.d.ts

@@ -8,3 +8,8 @@ export declare function groupFilesByRepo(cwd: string, files: string[]): Array<{
 export declare function hookFindings(result: HookCheckerResult): Failure[];
 /** Open what this edit introduced and resolve what it fixed; fixes become stories and lessons. */
 export declare function recordEditCatches(root: string, result: HookCheckerResult, files: string[]): Promise<void>;
+/** Re-checks the per-edit findings another version of the checks opened (with `all`, every open one), with this version's checks (agent-fixes.ts). */
+export declare function recheckEditFindings(root: string, all?: boolean): Promise<{
+    closed: number;
+    kept: number;
+}>;

package/dist/commands/hooks-check-repos.js

@@ -5,7 +5,7 @@
 import { spawnSync } from 'child_process';
 import fs from 'fs';
 import path from 'path';
-import { recordFixLessons, recordReviewOutcome } from '@rigour-labs/core';
+import { recheckOpenFindings, recordFixLessons, recordReviewOutcome, runHookChecker } from '@rigour-labs/core';
 /** Findings about the write or the file's size, not a defect in the code: never stories. */
 const NOT_CODE = new Set(['file-guard', 'agent-scope', 'hook-timeout', 'file-size']);
 /** Files grouped by the git repository that holds them; a file outside any repository stays with `cwd`. */
@@ -34,9 +34,15 @@ export function hookFindings(result) {
 }
 /** Open what this edit introduced and resolve what it fixed; fixes become stories and lessons. */
 export async function recordEditCatches(root, result, files) {
+    // After an upgrade, what the old checks reported on earlier edits is checked again by these: never credited as a fix.
+    await recheckEditFindings(root).catch(() => undefined);
     const capture = recordReviewOutcome(root, hookFindings(result), files, 'edit');
     await recordFixLessons(root, capture.fixes).catch(() => undefined); // learning never blocks an edit
 }
+/** Re-checks the per-edit findings another version of the checks opened (with `all`, every open one), with this version's checks (agent-fixes.ts). */
+export function recheckEditFindings(root, all = false) {
+    return recheckOpenFindings(root, 'edit', async (files) => hookFindings(await runHookChecker({ cwd: root, files: files.map(f => path.join(root, f)) })), all);
+}
 /** The directory's real, long-form path; the path as given when it cannot be resolved (deleted, for example). */
 function realDir(dir) {
     try {

package/dist/commands/review-human.js

@@ -60,6 +60,8 @@ function printQuietLines(result, 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).`);
+    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)'}`);
     if (result.dismissed)

package/dist/commands/review-reviewer.js

@@ -66,7 +66,9 @@ export function printReviewer(result, options = {}) {
     if (result.record && result.recordPath)
         console.log(chalk.dim(`  record: ${result.recordPath} (integrity ${result.record.integrity.slice(0, 16)})`));
     const tokens = result.tokens ? `, ${(result.tokens.input + result.tokens.output).toLocaleString('en-US')} tokens` : '';
-    const cost = `${result.costUsd !== undefined ? `, $${result.costUsd.toFixed(2)}` : ''}${tokens}`;
+    // What every run of this review cost, failed ones included (the verdict's judges alone would under-count it).
+    const spent = result.spentUsd ?? result.costUsd;
+    const cost = `${spent !== undefined ? `, $${spent.toFixed(2)}` : ''}${tokens}`;
     console.log(`  ${result.items.length} open item(s)${result.cached ? chalk.dim(' (cached for this commit)') : cost}\n`);
 }
 /** Which mode ran, against which was asked, and why: a panel never shrinks to one judge silently. */
@@ -100,7 +102,9 @@ export function reviewerJson(result) {
         dismissed: result.dismissed,
         mode: result.mode ?? null,
         panel: result.panel ?? null,
+        // The verdict's judges, as reported; kept for compatibility. spent_usd is every run, and 0 for a cached verdict.
         cost_usd: result.costUsd ?? null,
+        spent_usd: result.spentUsd ?? 0,
         tokens: result.tokens ?? null,
         cached: result.cached,
         previous_review: result.previousReview ?? null,

package/dist/commands/studio.js

@@ -241,6 +241,9 @@ async function handleApiRequest(req, res, url, ctx) {
     if (url.pathname === '/api/week') {
         try {
             const { loadWeek } = await import('./studio-week.js');
+            // What an older version of the checks opened is checked again before it is listed (a no-op once done).
+            const { recheckEditFindings } = await import('./hooks-check-repos.js');
+            await recheckEditFindings(cwd).catch(() => undefined);
             sendJson(res, 200, loadWeek(cwd));
         }
         catch (e) {
@@ -248,6 +251,16 @@ async function handleApiRequest(req, res, url, ctx) {
         }
         return true;
     }
+    if (url.pathname === '/api/recheck' && req.method === 'POST') {
+        try {
+            const { recheckEditFindings } = await import('./hooks-check-repos.js');
+            sendJson(res, 200, await recheckEditFindings(cwd, true));
+        }
+        catch (e) {
+            sendJson(res, 500, { error: e.message });
+        }
+        return true;
+    }
     if (url.pathname === '/api/dismiss' && req.method === 'POST') {
         let body = '';
         req.on('data', (chunk) => (body += chunk));

package/package.json

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