@rigour-labs/core 6.6.4 → 6.6.6

package/dist/types/index.d.ts

@@ -537,6 +537,30 @@ export declare const GatesSchema: z.ZodObject<{
     }, {
         enabled?: boolean | undefined;
     }>>>;
+    /** What a fix leaves half done: the narrower condition still used elsewhere, a prop wired into some sibling mounts only, an accumulator copied every step (review/partial-fixes.ts, partial-wiring.ts, loop-copies.ts). */
+    change_sweep: z.ZodDefault<z.ZodOptional<z.ZodObject<{
+        enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+    }, "strip", z.ZodTypeAny, {
+        enabled: boolean;
+    }, {
+        enabled?: boolean | undefined;
+    }>>>;
+    /**
+     * What a change made redundant, from the project's own TypeScript (review/typed/redundancy.ts): a null filter beside a
+     * range on the same column, a nullable row type the query filters non-null, an optional member every host supplies, a
+     * property written and never read. Runs at push, in `rigour review` and in a backtest (the program takes seconds to build).
+     * `wire_contracts`: files whose types another service reads, so their members are never write-only here.
+     */
+    redundancy: z.ZodDefault<z.ZodOptional<z.ZodObject<{
+        enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+        wire_contracts: z.ZodDefault<z.ZodOptional<z.ZodArray<z.ZodString, "many">>>;
+    }, "strip", z.ZodTypeAny, {
+        enabled: boolean;
+        wire_contracts: string[];
+    }, {
+        enabled?: boolean | undefined;
+        wire_contracts?: string[] | undefined;
+    }>>>;
     /** A parameter the change adds as optional that only tests omit (review/optional-params.ts). */
     optional_params: z.ZodDefault<z.ZodOptional<z.ZodObject<{
         enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
@@ -1008,6 +1032,13 @@ export declare const GatesSchema: z.ZodObject<{
     query_patterns: {
         enabled: boolean;
     };
+    change_sweep: {
+        enabled: boolean;
+    };
+    redundancy: {
+        enabled: boolean;
+        wire_contracts: string[];
+    };
     optional_params: {
         enabled: boolean;
     };
@@ -1280,6 +1311,13 @@ export declare const GatesSchema: z.ZodObject<{
     query_patterns?: {
         enabled?: boolean | undefined;
     } | undefined;
+    change_sweep?: {
+        enabled?: boolean | undefined;
+    } | undefined;
+    redundancy?: {
+        enabled?: boolean | undefined;
+        wire_contracts?: string[] | undefined;
+    } | undefined;
     optional_params?: {
         enabled?: boolean | undefined;
     } | undefined;
@@ -1925,6 +1963,30 @@ export declare const ConfigSchema: z.ZodObject<{
         }, {
             enabled?: boolean | undefined;
         }>>>;
+        /** What a fix leaves half done: the narrower condition still used elsewhere, a prop wired into some sibling mounts only, an accumulator copied every step (review/partial-fixes.ts, partial-wiring.ts, loop-copies.ts). */
+        change_sweep: z.ZodDefault<z.ZodOptional<z.ZodObject<{
+            enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+        }, "strip", z.ZodTypeAny, {
+            enabled: boolean;
+        }, {
+            enabled?: boolean | undefined;
+        }>>>;
+        /**
+         * What a change made redundant, from the project's own TypeScript (review/typed/redundancy.ts): a null filter beside a
+         * range on the same column, a nullable row type the query filters non-null, an optional member every host supplies, a
+         * property written and never read. Runs at push, in `rigour review` and in a backtest (the program takes seconds to build).
+         * `wire_contracts`: files whose types another service reads, so their members are never write-only here.
+         */
+        redundancy: z.ZodDefault<z.ZodOptional<z.ZodObject<{
+            enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
+            wire_contracts: z.ZodDefault<z.ZodOptional<z.ZodArray<z.ZodString, "many">>>;
+        }, "strip", z.ZodTypeAny, {
+            enabled: boolean;
+            wire_contracts: string[];
+        }, {
+            enabled?: boolean | undefined;
+            wire_contracts?: string[] | undefined;
+        }>>>;
         /** A parameter the change adds as optional that only tests omit (review/optional-params.ts). */
         optional_params: z.ZodDefault<z.ZodOptional<z.ZodObject<{
             enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
@@ -2396,6 +2458,13 @@ export declare const ConfigSchema: z.ZodObject<{
         query_patterns: {
             enabled: boolean;
         };
+        change_sweep: {
+            enabled: boolean;
+        };
+        redundancy: {
+            enabled: boolean;
+            wire_contracts: string[];
+        };
         optional_params: {
             enabled: boolean;
         };
@@ -2668,6 +2737,13 @@ export declare const ConfigSchema: z.ZodObject<{
         query_patterns?: {
             enabled?: boolean | undefined;
         } | undefined;
+        change_sweep?: {
+            enabled?: boolean | undefined;
+        } | undefined;
+        redundancy?: {
+            enabled?: boolean | undefined;
+            wire_contracts?: string[] | undefined;
+        } | undefined;
         optional_params?: {
             enabled?: boolean | undefined;
         } | undefined;
@@ -2754,30 +2830,51 @@ export declare const ConfigSchema: z.ZodObject<{
         show_preexisting: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
         /** The GitHub account whose token fetches the pull request's previous review (`gh auth token --user`). */
         github_account: z.ZodOptional<z.ZodString>;
-        /** The fresh reviewer (review/reviewer.ts): the person's own coding agent CLI, headless. `enabled` runs it at every push; `rigour review --reviewer` runs it on request. */
+        /** The reviewer (review/reviewer.ts): the person's own coding-agent CLIs, headless and read-only. `enabled` runs it at push; `rigour review --reviewer` runs it on request. */
         reviewer: z.ZodDefault<z.ZodOptional<z.ZodObject<{
             enabled: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
-            command: z.ZodDefault<z.ZodOptional<z.ZodString>>;
+            /**
+             * At push, once the deterministic gates pass: `background` (default) starts the model review of the pushed
+             * commit without holding the push; `wait` holds the push for it; `off` reviews only on request. Either way
+             * a model is asked only when the branch has an open, non-draft pull request (someone will read the push).
+             */
+            on_push: z.ZodDefault<z.ZodOptional<z.ZodEnum<["background", "wait", "off"]>>>;
+            /** The adapters, in order: claude, cursor, codex. */
+            reviewers: z.ZodDefault<z.ZodOptional<z.ZodArray<z.ZodString, "many">>>;
+            /** single: the first installed reviewer. cross: prefer a vendor not on the commits' trailers. full: two vendors, verdicts merged. */
+            mode: z.ZodDefault<z.ZodOptional<z.ZodEnum<["single", "cross", "full"]>>>;
+            /** The model for the claude reviewer. */
             model: z.ZodOptional<z.ZodString>;
+            /** A model per reviewer name, e.g. { cursor: "auto" }. */
+            models: z.ZodDefault<z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodString>>>;
             timeout_ms: z.ZodDefault<z.ZodOptional<z.ZodNumber>>;
         }, "strip", z.ZodTypeAny, {
             enabled: boolean;
+            mode: "single" | "cross" | "full";
             timeout_ms: number;
-            command: string;
+            on_push: "off" | "background" | "wait";
+            reviewers: string[];
+            models: Record<string, string>;
             model?: string | undefined;
         }, {
             enabled?: boolean | undefined;
+            mode?: "single" | "cross" | "full" | undefined;
             timeout_ms?: number | undefined;
-            command?: string | undefined;
+            on_push?: "off" | "background" | "wait" | undefined;
+            reviewers?: string[] | undefined;
             model?: string | undefined;
+            models?: Record<string, string> | undefined;
         }>>>;
     }, "strip", z.ZodTypeAny, {
         include_heuristics: boolean;
         show_preexisting: boolean;
         reviewer: {
             enabled: boolean;
+            mode: "single" | "cross" | "full";
             timeout_ms: number;
-            command: string;
+            on_push: "off" | "background" | "wait";
+            reviewers: string[];
+            models: Record<string, string>;
             model?: string | undefined;
         };
         github_account?: string | undefined;
@@ -2787,9 +2884,12 @@ export declare const ConfigSchema: z.ZodObject<{
         github_account?: string | undefined;
         reviewer?: {
             enabled?: boolean | undefined;
+            mode?: "single" | "cross" | "full" | undefined;
             timeout_ms?: number | undefined;
-            command?: string | undefined;
+            on_push?: "off" | "background" | "wait" | undefined;
+            reviewers?: string[] | undefined;
             model?: string | undefined;
+            models?: Record<string, string> | undefined;
         } | undefined;
     }>>>;
     planned: z.ZodDefault<z.ZodOptional<z.ZodArray<z.ZodString, "many">>>;
@@ -3032,6 +3132,13 @@ export declare const ConfigSchema: z.ZodObject<{
         query_patterns: {
             enabled: boolean;
         };
+        change_sweep: {
+            enabled: boolean;
+        };
+        redundancy: {
+            enabled: boolean;
+            wire_contracts: string[];
+        };
         optional_params: {
             enabled: boolean;
         };
@@ -3093,8 +3200,11 @@ export declare const ConfigSchema: z.ZodObject<{
         show_preexisting: boolean;
         reviewer: {
             enabled: boolean;
+            mode: "single" | "cross" | "full";
             timeout_ms: number;
-            command: string;
+            on_push: "off" | "background" | "wait";
+            reviewers: string[];
+            models: Record<string, string>;
             model?: string | undefined;
         };
         github_account?: string | undefined;
@@ -3342,6 +3452,13 @@ export declare const ConfigSchema: z.ZodObject<{
         query_patterns?: {
             enabled?: boolean | undefined;
         } | undefined;
+        change_sweep?: {
+            enabled?: boolean | undefined;
+        } | undefined;
+        redundancy?: {
+            enabled?: boolean | undefined;
+            wire_contracts?: string[] | undefined;
+        } | undefined;
         optional_params?: {
             enabled?: boolean | undefined;
         } | undefined;
@@ -3404,9 +3521,12 @@ export declare const ConfigSchema: z.ZodObject<{
         github_account?: string | undefined;
         reviewer?: {
             enabled?: boolean | undefined;
+            mode?: "single" | "cross" | "full" | undefined;
             timeout_ms?: number | undefined;
-            command?: string | undefined;
+            on_push?: "off" | "background" | "wait" | undefined;
+            reviewers?: string[] | undefined;
             model?: string | undefined;
+            models?: Record<string, string> | undefined;
         } | undefined;
     } | undefined;
     planned?: string[] | undefined;

package/dist/types/index.js

@@ -288,6 +288,18 @@ export const GatesSchema = z.object({
     }).optional().default({}),
     /** Query shapes that cost production: offset paging in a loop, a time window with no upper bound (review/query-patterns.ts). */
     query_patterns: z.object({ enabled: z.boolean().optional().default(true) }).optional().default({}),
+    /** What a fix leaves half done: the narrower condition still used elsewhere, a prop wired into some sibling mounts only, an accumulator copied every step (review/partial-fixes.ts, partial-wiring.ts, loop-copies.ts). */
+    change_sweep: z.object({ enabled: z.boolean().optional().default(true) }).optional().default({}),
+    /**
+     * What a change made redundant, from the project's own TypeScript (review/typed/redundancy.ts): a null filter beside a
+     * range on the same column, a nullable row type the query filters non-null, an optional member every host supplies, a
+     * property written and never read. Runs at push, in `rigour review` and in a backtest (the program takes seconds to build).
+     * `wire_contracts`: files whose types another service reads, so their members are never write-only here.
+     */
+    redundancy: z.object({
+        enabled: z.boolean().optional().default(true),
+        wire_contracts: z.array(z.string()).optional().default([]),
+    }).optional().default({}),
     /** A parameter the change adds as optional that only tests omit (review/optional-params.ts). */
     optional_params: z.object({ enabled: z.boolean().optional().default(true) }).optional().default({}),
     /** A changed function whose body duplicates another in the files the change touched (review/duplicate-functions.ts). */
@@ -413,11 +425,23 @@ export const ConfigSchema = z.object({
         show_preexisting: z.boolean().optional().default(false),
         /** The GitHub account whose token fetches the pull request's previous review (`gh auth token --user`). */
         github_account: z.string().optional(),
-        /** The fresh reviewer (review/reviewer.ts): the person's own coding agent CLI, headless. `enabled` runs it at every push; `rigour review --reviewer` runs it on request. */
+        /** The reviewer (review/reviewer.ts): the person's own coding-agent CLIs, headless and read-only. `enabled` runs it at push; `rigour review --reviewer` runs it on request. */
         reviewer: z.object({
             enabled: z.boolean().optional().default(false),
-            command: z.string().optional().default('claude'),
+            /**
+             * At push, once the deterministic gates pass: `background` (default) starts the model review of the pushed
+             * commit without holding the push; `wait` holds the push for it; `off` reviews only on request. Either way
+             * a model is asked only when the branch has an open, non-draft pull request (someone will read the push).
+             */
+            on_push: z.enum(['background', 'wait', 'off']).optional().default('background'),
+            /** The adapters, in order: claude, cursor, codex. */
+            reviewers: z.array(z.string()).optional().default(['claude']),
+            /** single: the first installed reviewer. cross: prefer a vendor not on the commits' trailers. full: two vendors, verdicts merged. */
+            mode: z.enum(['single', 'cross', 'full']).optional().default('single'),
+            /** The model for the claude reviewer. */
             model: z.string().optional(),
+            /** A model per reviewer name, e.g. { cursor: "auto" }. */
+            models: z.record(z.string()).optional().default({}),
             timeout_ms: z.number().optional().default(15 * 60_000),
         }).optional().default({}),
     }).optional().default({}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rigour-labs/core",
-  "version": "6.6.4",
+  "version": "6.6.6",
   "description": "Rigour's review engine: deterministic gates on changed lines, rules and lessons learned from your team's fixes, and per-check precision from what you fix versus dismiss, across TypeScript, JavaScript, Python, Go, Ruby and C#.",
   "engines": {
     "node": ">=22"
@@ -74,11 +74,11 @@
     "pg": "^8.16.3",
     "sqlite3": "^5.1.7",
     "openai": "^4.104.0",
-    "@rigour-labs/brain-darwin-arm64": "6.6.4",
-    "@rigour-labs/brain-darwin-x64": "6.6.4",
-    "@rigour-labs/brain-linux-x64": "6.6.4",
-    "@rigour-labs/brain-linux-arm64": "6.6.4",
-    "@rigour-labs/brain-win-x64": "6.6.4"
+    "@rigour-labs/brain-darwin-x64": "6.6.6",
+    "@rigour-labs/brain-darwin-arm64": "6.6.6",
+    "@rigour-labs/brain-win-x64": "6.6.6",
+    "@rigour-labs/brain-linux-x64": "6.6.6",
+    "@rigour-labs/brain-linux-arm64": "6.6.6"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",