@rigour-labs/cli 6.9.0-rc.3 → 6.9.0-rc.5

package/dist/cli.js

@@ -273,6 +273,8 @@ program
     .option('--single', 'With --reviewer: one judge for this run, whatever your settings say (a team floor still applies)')
     .option('--panel', 'With --reviewer: a panel of judges for this run; only what a majority confirms blocks')
     .option('--no-panel', 'With --reviewer: no panel for this run (a team that requires one refuses this)')
+    .option('--orchestrator', 'With --reviewer: only the parts of the review this change needs, picked without a model, usually in one pass (experimental)')
+    .option('--no-orchestrator', 'With --reviewer: one judge for the whole review, for this run (a team that requires the orchestrator refuses this)')
     .option('--status', 'What the background reviewer has done for this branch: running, last verdict, open items')
     .option('--all', 'Show every finding, not the first five')
     .option('--notes', 'List the notes that never block')

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,14 @@ 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.)`,
+        `  The model reviewer: ${m.model.share.model} of ${m.model.share.model + m.model.share.checks} finding(s) at first review on ${m.model.share.prs} pull request(s)${m.model.share.rate === null ? '' : ` (${Math.round(m.model.share.rate * 100)}%)`}; $${m.model.costPerPr.totalUsd.toFixed(2)} over ${m.model.costPerPr.prs} pull request(s)${m.model.costPerPr.medianUsd === null ? '' : `, median $${m.model.costPerPr.medianUsd.toFixed(2)}`}.`,
+        `  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/review-reviewer.d.ts

@@ -1,7 +1,7 @@
 import { type Config, type ReviewerResult, type ReviewResult, type RunChoice } from '@rigour-labs/core';
 /** The base a branch review runs against: the one named, else where the branch left main. */
 export declare function reviewerBase(cwd: string, named: string | undefined): string | undefined;
-export declare function reviewerFor(cwd: string, base: string | undefined, config: Config, full: boolean, choice: RunChoice, review: ReviewResult, goal?: boolean): Promise<ReviewerResult>;
+export declare function reviewerFor(cwd: string, base: string | undefined, config: Config, full: boolean, choice: RunChoice, review: ReviewResult, goal?: boolean, orchestrator?: boolean): Promise<ReviewerResult>;
 export declare function printReviewer(result: ReviewerResult, options?: {
     notes?: boolean;
 }): void;

package/dist/commands/review-reviewer.js

@@ -12,10 +12,10 @@ import { branchBase, itemLine, reviewerInputs, reviewStatus, runReviewer } from
 export function reviewerBase(cwd, named) {
     return named ?? branchBase(cwd)?.mainRef.replace(/^refs\/(remotes\/|heads\/)/, '');
 }
-export async function reviewerFor(cwd, base, config, full, choice, review, goal) {
+export async function reviewerFor(cwd, base, config, full, choice, review, goal, orchestrator) {
     if (!base)
         return { outcome: 'unavailable', items: [], unverified: [], resolved: [], answerInReply: [], notes: [], advisory: [], disputed: [], dropped: [], dismissed: [], reason: 'no base to review against: pass --base, or fetch the main branch', reviewers: [], cached: false };
-    return runReviewer(cwd, base, config, undefined, undefined, { trigger: 'review', full, choice, ...(goal !== undefined ? { goal } : {}), ...reviewerInputs(review) });
+    return runReviewer(cwd, base, config, undefined, undefined, { trigger: 'review', full, choice, ...(goal !== undefined ? { goal } : {}), ...(orchestrator !== undefined ? { orchestrator } : {}), ...reviewerInputs(review) });
 }
 /** Verified should-fixes shown in full before the rest fold into a count. */
 const ADVISORY_SHOWN = 5;
@@ -75,7 +75,7 @@ function printMode(result) {
     if (!mode)
         return;
     if (mode.asked !== mode.ran || mode.degraded || mode.escalation) {
-        const why = [mode.degraded, mode.escalation].filter(Boolean).join('; ');
+        const why = [mode.degraded, mode.escalation, mode.specialists?.fallback].filter(Boolean).join('; ');
         console.log(chalk.yellow(`  ${mode.asked} asked (${mode.source}), ${mode.ran} ran${why ? `: ${why}` : ''}`));
     }
     for (const line of mode.refused ?? [])

package/dist/commands/review.d.ts

@@ -23,6 +23,7 @@ export interface ReviewOptions {
     full?: boolean;
     single?: boolean;
     panel?: boolean;
+    orchestrator?: boolean;
     status?: boolean;
     all?: boolean;
     notes?: boolean;

package/dist/commands/review.js

@@ -62,7 +62,7 @@ export async function reviewCommand(cwd, options = {}) {
         if (options.base)
             recordPrCatches(cwd, result.findings);
         const receipt = receiptFor(cwd, diff ?? changeDiff(cwd, source), config, !!options.independent);
-        const reviewer = options.reviewer ? await reviewerFor(cwd, reviewerBase(cwd, options.base), config, !!options.full, { ...(options.single ? { mode: 'single' } : {}), ...(options.panel !== undefined ? { panel: options.panel } : {}) }, result, options.goal) : undefined;
+        const reviewer = options.reviewer ? await reviewerFor(cwd, reviewerBase(cwd, options.base), config, !!options.full, { ...(options.single ? { mode: 'single' } : {}), ...(options.panel !== undefined ? { panel: options.panel } : {}) }, result, options.goal, options.orchestrator) : undefined;
         const checked = options.json ? whatWasChecked(cwd, options, trustedRef, result) : undefined;
         await print(result, options, receipt, reviewer, { cwd, scope: scopeOf(options), commits: commitsOf(cwd, options.base), ms: Date.now() - started }, checked, goalReport(goal, goalDescription, result));
         if (!isDeep && !receipt && !options.ci && !options.json && !options.githubSummary) {

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;
@@ -47,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;
@@ -78,9 +78,7 @@ function learnedFromPoints(points) {
  * 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 === 'reclassified' || 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')
@@ -103,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.3",
+  "version": "6.9.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.9.0-rc.3"
+    "@rigour-labs/core": "6.9.0-rc.5"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",
     "@types/inquirer": "9.0.7",
     "@types/node": "^25.0.3",
-    "@rigour-labs/studio": "6.9.0-rc.3"
+    "@rigour-labs/studio": "6.9.0-rc.5"
   },
   "scripts": {
     "build": "tsc && pnpm bundle-studio",