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

package/dist/commands/outcomes.js

@@ -27,6 +27,9 @@ export async function outcomesCommand(cwd, options) {
     for (const o of run.outcomes)
         console.log(outcomeLine(o));
     console.log(chalk.dim(`${run.outcomes.length} pull request(s), ${run.read} read now, the rest settled.${run.stopped ? ` Stopped early: ${run.stopped}.` : ''}`));
+    if (run.metrics)
+        for (const line of metricLines(run.metrics))
+            console.log(chalk.dim(line));
     if (run.lessons?.added)
         console.log(`Lessons: ${run.lessons.added} new piece(s) of evidence, ${run.lessons.suggested.length} to look at in Studio (a later fix changed their lines), ${run.lessons.demoted.length} taken back.`);
     return 0;
@@ -37,3 +40,13 @@ function outcomeLine(o) {
     const after = `${o.followUps.length} later commit(s) on its files${fixes ? `, ${chalk.yellow(`${fixes} fix(es)`)}` : ''}`;
     return `#${o.pr} ${o.mergeSha.slice(0, 9)} merged ${o.mergedAt.slice(0, 10)}: ${ci}, ${after}${o.reverted ? `, ${chalk.red(`reverted by ${o.reverted.sha.slice(0, 9)}`)}` : ''}${o.settled ? '' : chalk.dim(' (window open)')}`;
 }
+/** The numbers, as counts; a percentage only where there are enough records for one (metrics.ts). Reviewed and not reviewed are side by side, never compared. */
+function metricLines(m) {
+    const share = (s) => `${s.count} of ${s.of}${s.rate === null ? '' : ` (${Math.round(s.rate * 100)}%)`}`;
+    return [
+        `Numbers: ${m.records.merged} merged, ${m.records.settled} settled, ${m.records.unsettled} with the window still open.`,
+        `  Settled: CI regressed on the merge ${share(m.settled.ciRegressed)}${m.settled.ciUnknown ? ` (${m.settled.ciUnknown} with no CI to read)` : ''}, reverted ${share(m.settled.reverted)}, fixed later ${share(m.settled.fixedLater)}.`,
+        `  Reviewed by Rigour: ${m.settled.reviewed.prs} pull request(s), ${m.settled.reviewed.fixedLater.count} fixed later. Not reviewed: ${m.settled.notReviewed.prs}, ${m.settled.notReviewed.fixedLater.count} fixed later. (Teams choose what gets reviewed: not a comparison.)`,
+        `  Lessons: ${m.lessons.awaitingDecision} waiting on a person, ${m.lessons.promotedFromEvidence} promoted from evidence, ${m.lessons.dismissed} dismissed, ${m.lessons.takenBack} taken back.`,
+    ];
+}

package/dist/commands/studio-learning.d.ts

@@ -7,7 +7,7 @@
  * lesson's subject prefix). Counts that Rigour cannot know here are null, never 0: PR catches
  * recorded on another machine (CI) never reach this one.
  */
-import { type AgentEvent, type LessonRecord, type ReviewLesson, type Story } from '@rigour-labs/core';
+import { type OutcomeMetrics, type AgentEvent, type LessonRecord, type ReviewLesson, type Story } from '@rigour-labs/core';
 export interface LessonJourney {
     id: string;
     text: string;
@@ -33,6 +33,11 @@ export interface LessonJourney {
         pr: number;
         at: string;
     };
+    /** Back to a candidate when outcomes stopped promoting: why, and the evidence that had promoted it, for a person to promote again or dismiss. */
+    reclassified?: {
+        detail: string;
+        evidence: string[];
+    };
 }
 export interface StudioLearning {
     lessons: LessonJourney[];
@@ -42,6 +47,8 @@ export interface StudioLearning {
         reachedPr: number | null;
     }>;
     prRecorded: boolean;
+    /** What happened after merges this checkout read (core outcomes/metrics.ts); absent before `rigour outcomes` has run. */
+    outcomes?: OutcomeMetrics;
 }
 export declare function buildLearning(input: {
     now: Date;

package/dist/commands/studio-learning.js

@@ -7,7 +7,7 @@
  * lesson's subject prefix). Counts that Rigour cannot know here are null, never 0: PR catches
  * recorded on another machine (CI) never reach this one.
  */
-import { decideLesson, fixLessonPrefix, listKnowledgeLessons, readLessons } from '@rigour-labs/core';
+import { decideLesson, fixLessonPrefix, localOutcomeMetrics, pendingDecision, listKnowledgeLessons, readLessons } from '@rigour-labs/core';
 import { personOf } from './git-identity.js';
 import { checkoutRoots, eventsAcross, storiesAcross } from './studio-checkouts.js';
 const WEEK_MS = 7 * 24 * 60 * 60 * 1000;
@@ -51,7 +51,8 @@ export function buildLearning(input) {
             reachedPr: null,
             ...decisionFor(l),
         })),
-    ].sort((a, b) => b.learnedAt.localeCompare(a.learnedAt))
+        // Lessons back to a candidate when outcomes stopped promoting come first: a person decides each once.
+    ].sort((a, b) => Number(!!b.reclassified) - Number(!!a.reclassified) || b.learnedAt.localeCompare(a.learnedAt))
         // The same lesson learned in several places (this repo and personal lessons from others) shows once.
         .filter((lesson, i, all) => all.findIndex(other => other.text === lesson.text) === i);
     const count = input.weeks ?? WEEKS;
@@ -73,12 +74,13 @@ function learnedFromPoints(points) {
 }
 /**
  * What a person can decide on a review lesson, from the last of its evidence and decisions: taken back by evidence
- * (promote it again, or drop it), or a candidate with a later fix on its lines (promote it, or dismiss the evidence).
+ * (promote it again, or drop it), back to a candidate when outcomes stopped promoting (promote it again, or dismiss),
+ * or a candidate with a later fix on its lines (promote it, or dismiss the evidence).
  */
 function decisionFor(lesson) {
-    if (lesson.state !== 'candidate')
-        return { canDecide: false };
-    const last = lesson.evidence.filter(e => e.kind === 'demoted' || e.kind === 'lines' || e.kind === 'accepted' || e.kind === 'rejected' || e.kind === 'dismissed').at(-1);
+    const last = pendingDecision(lesson);
+    if (last?.kind === 'reclassified')
+        return { canDecide: true, reclassified: { detail: last.detail ?? '', evidence: lesson.evidence.filter(e => e.kind === 'outcome' || e.kind === 'lines').map(e => e.detail ?? '').filter(Boolean) } };
     if (last?.kind === 'demoted')
         return { canDecide: true, takenBack: { detail: last.detail ?? '', prs: lesson.evidence.filter(e => e.kind === 'against').map(e => e.pr), at: last.at ?? '' } };
     if (last?.kind === 'lines')
@@ -99,7 +101,9 @@ export function decideReviewLesson(cwd, body) {
 }
 export async function loadLearning(cwd, now = new Date(), weeks = WEEKS) {
     const roots = checkoutRoots(cwd);
-    return buildLearning({ now, lessons: await listKnowledgeLessons(cwd), reviewLessons: readLessons(cwd), stories: storiesAcross(roots), events: eventsAcross(roots), weeks });
+    const learning = buildLearning({ now, lessons: await listKnowledgeLessons(cwd), reviewLessons: readLessons(cwd), stories: storiesAcross(roots), events: eventsAcross(roots), weeks });
+    const outcomes = localOutcomeMetrics(cwd);
+    return outcomes ? { ...learning, outcomes } : learning;
 }
 /**
  * "Fixed before: Credential header follows redirects (semantic-bugs). The header…" → the defect,

package/package.json

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