codesift-mcp 0.9.5 → 0.9.10

package/dist/tools/review-diff/checks/complexity.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"complexity.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/complexity.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAI9D;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,KAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACzB,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,YAAY,CAAC,CAAC;QACzC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC,CAAC;QAElE,MAAM,QAAQ,GAAoB,MAAM,CAAC,SAAS;aAC/C,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,qBAAqB,GAAG,EAAE,CAAC;aACxE,GAAG,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;YACZ,KAAK,EAAE,YAAY;YACnB,QAAQ,EAAE,MAAe;YACzB,OAAO,EAAE,qBAAqB,EAAE,CAAC,IAAI,QAAQ,EAAE,CAAC,IAAI,4BAA4B,EAAE,CAAC,qBAAqB,MAAM,EAAE,GAAG;YACnH,IAAI,EAAE,EAAE,CAAC,IAAI;YACb,IAAI,EAAE,EAAE,CAAC,UAAU;YACnB,MAAM,EAAE,EAAE,CAAC,IAAI;SAChB,CAAC,CAAC,CAAC;QAEN,OAAO;YACL,KAAK,EAAE,YAAY;YACnB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;YAC7C,QAAQ;YACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,sCAAsC;gBAC1D,CAAC,CAAC,sCAAsC;SAC3C,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO;YACL,KAAK,EAAE,YAAY;YACnB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,UAAU,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE;SACtE,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/tools/review-diff/checks/coupling.d.ts

@@ -0,0 +1,7 @@
+import type { CheckResult } from "../types.js";
+/**
+ * Coupling gaps check: uses shared computeCoChangePairs from coupling-tools.ts,
+ * then flags coupled files that are missing from the diff.
+ */
+export declare function checkCouplingGaps(repoRoot: string, changedFiles: string[]): Promise<CheckResult>;
+//# sourceMappingURL=coupling.d.ts.map

package/dist/tools/review-diff/checks/coupling.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"coupling.d.ts","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/coupling.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,WAAW,EAAiB,MAAM,aAAa,CAAC;AAK9D;;;GAGG;AACH,wBAAsB,iBAAiB,CACrC,QAAQ,EAAE,MAAM,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAgCtB"}

package/dist/tools/review-diff/checks/coupling.js

@@ -0,0 +1,52 @@
+import { computeCoChangePairs } from "../../coupling-tools.js";
+const MIN_JACCARD = 0.5;
+/**
+ * Coupling gaps check: uses shared computeCoChangePairs from coupling-tools.ts,
+ * then flags coupled files that are missing from the diff.
+ */
+export async function checkCouplingGaps(repoRoot, changedFiles) {
+    const start = Date.now();
+    try {
+        const { pairs } = computeCoChangePairs(repoRoot, {
+            since_days: 180,
+            min_support: 3,
+        });
+        const changedSet = new Set(changedFiles);
+        const findings = pairs.flatMap((pair) => couplingGapFinding(pair, changedSet) ?? []);
+        return {
+            check: "coupling",
+            status: findings.length > 0 ? "warn" : "pass",
+            findings,
+            duration_ms: Date.now() - start,
+            summary: findings.length > 0
+                ? `${findings.length} coupling gap(s) detected`
+                : "No coupling gaps detected",
+        };
+    }
+    catch (err) {
+        return {
+            check: "coupling",
+            status: "error",
+            findings: [],
+            duration_ms: Date.now() - start,
+            summary: `Error: ${err instanceof Error ? err.message : String(err)}`,
+        };
+    }
+}
+function couplingGapFinding(pair, changedSet) {
+    if (pair.jaccard < MIN_JACCARD)
+        return null;
+    const aInDiff = changedSet.has(pair.file_a);
+    const bInDiff = changedSet.has(pair.file_b);
+    if (aInDiff === bInDiff)
+        return null;
+    const present = aInDiff ? pair.file_a : pair.file_b;
+    const missing = aInDiff ? pair.file_b : pair.file_a;
+    return {
+        check: "coupling",
+        severity: "warn",
+        message: `"${present}" is frequently co-changed with "${missing}" (Jaccard ${pair.jaccard.toFixed(2)}, ${pair.co_commits} co-commits) but "${missing}" is not in this diff`,
+        file: present,
+    };
+}
+//# sourceMappingURL=coupling.js.map

package/dist/tools/review-diff/checks/coupling.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"coupling.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/coupling.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAC;AAG/D,MAAM,WAAW,GAAG,GAAG,CAAC;AAGxB;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,QAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IAEzB,IAAI,CAAC;QACH,MAAM,EAAE,KAAK,EAAE,GAAG,oBAAoB,CAAC,QAAQ,EAAE;YAC/C,UAAU,EAAE,GAAG;YACf,WAAW,EAAE,CAAC;SACf,CAAC,CAAC;QAEH,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,YAAY,CAAC,CAAC;QACzC,MAAM,QAAQ,GAAG,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CACtC,kBAAkB,CAAC,IAAI,EAAE,UAAU,CAAC,IAAI,EAAE,CAC3C,CAAC;QAEF,OAAO;YACL,KAAK,EAAE,UAAU;YACjB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;YAC7C,QAAQ;YACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,2BAA2B;gBAC/C,CAAC,CAAC,2BAA2B;SAChC,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO;YACL,KAAK,EAAE,UAAU;YACjB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,UAAU,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE;SACtE,CAAC;IACJ,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CACzB,IAAkB,EAClB,UAAuB;IAEvB,IAAI,IAAI,CAAC,OAAO,GAAG,WAAW;QAAE,OAAO,IAAI,CAAC;IAE5C,MAAM,OAAO,GAAG,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC5C,MAAM,OAAO,GAAG,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC5C,IAAI,OAAO,KAAK,OAAO;QAAE,OAAO,IAAI,CAAC;IAErC,MAAM,OAAO,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC;IACpD,MAAM,OAAO,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC;IAEpD,OAAO;QACL,KAAK,EAAE,UAAU;QACjB,QAAQ,EAAE,MAAM;QAChB,OAAO,EAAE,IAAI,OAAO,oCAAoC,OAAO,cAAc,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,UAAU,qBAAqB,OAAO,uBAAuB;QAC3K,IAAI,EAAE,OAAO;KACd,CAAC;AACJ,CAAC"}

package/dist/tools/review-diff/checks/dead-code.d.ts

@@ -0,0 +1,7 @@
+import type { CodeIndex } from "../../../types.js";
+import type { CheckResult } from "../types.js";
+/**
+ * Dead-code check: run findDeadCode scoped to changedFiles and map candidates to T2 findings.
+ */
+export declare function checkDeadCode(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
+//# sourceMappingURL=dead-code.d.ts.map

package/dist/tools/review-diff/checks/dead-code.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"dead-code.d.ts","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/dead-code.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAEnD,OAAO,KAAK,EAAE,WAAW,EAAiB,MAAM,aAAa,CAAC;AAE9D;;GAEG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAkCtB"}

package/dist/tools/review-diff/checks/dead-code.js

@@ -0,0 +1,39 @@
+import { findDeadCode } from "../../symbol-tools.js";
+import { literalChangedFilePattern } from "../file-pattern.js";
+/**
+ * Dead-code check: run findDeadCode scoped to changedFiles and map candidates to T2 findings.
+ */
+export async function checkDeadCode(index, changedFiles) {
+    const start = Date.now();
+    try {
+        const filePattern = literalChangedFilePattern(changedFiles);
+        const result = await findDeadCode(index.repo, { file_pattern: filePattern });
+        const findings = result.candidates.map((c) => ({
+            check: "dead-code",
+            severity: "warn",
+            message: `"${c.name}" appears unused — ${c.reason}`,
+            file: c.file,
+            line: c.start_line,
+            symbol: c.name,
+        }));
+        return {
+            check: "dead-code",
+            status: findings.length > 0 ? "warn" : "pass",
+            findings,
+            duration_ms: Date.now() - start,
+            summary: findings.length > 0
+                ? `${findings.length} dead-code candidate(s) found`
+                : "No dead code detected",
+        };
+    }
+    catch (err) {
+        return {
+            check: "dead-code",
+            status: "error",
+            findings: [],
+            duration_ms: Date.now() - start,
+            summary: `Error: ${err instanceof Error ? err.message : String(err)}`,
+        };
+    }
+}
+//# sourceMappingURL=dead-code.js.map

package/dist/tools/review-diff/checks/dead-code.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"dead-code.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/dead-code.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AAErD,OAAO,EAAE,yBAAyB,EAAE,MAAM,oBAAoB,CAAC;AAG/D;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,KAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACzB,IAAI,CAAC;QACH,MAAM,WAAW,GAAG,yBAAyB,CAAC,YAAY,CAAC,CAAC;QAE5D,MAAM,MAAM,GAAG,MAAM,YAAY,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,YAAY,EAAE,WAAW,EAAE,CAAC,CAAC;QAE7E,MAAM,QAAQ,GAAoB,MAAM,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAC9D,KAAK,EAAE,WAAW;YAClB,QAAQ,EAAE,MAAM;YAChB,OAAO,EAAE,IAAI,CAAC,CAAC,IAAI,sBAAsB,CAAC,CAAC,MAAM,EAAE;YACnD,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,CAAC,UAAU;YAClB,MAAM,EAAE,CAAC,CAAC,IAAI;SACf,CAAC,CAAC,CAAC;QAEJ,OAAO;YACL,KAAK,EAAE,WAAW;YAClB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;YAC7C,QAAQ;YACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,+BAA+B;gBACnD,CAAC,CAAC,uBAAuB;SAC5B,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO;YACL,KAAK,EAAE,WAAW;YAClB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,UAAU,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE;SACtE,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/tools/review-diff/checks/hotspots.d.ts

@@ -0,0 +1,8 @@
+import type { CodeIndex } from "../../../types.js";
+import type { CheckResult } from "../types.js";
+/**
+ * Hotspots check: run analyzeHotspots and filter to files in changedFiles.
+ * Maps high-churn files to T3 advisory findings.
+ */
+export declare function checkHotspots(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
+//# sourceMappingURL=hotspots.d.ts.map

package/dist/tools/review-diff/checks/hotspots.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"hotspots.d.ts","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/hotspots.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AACnD,OAAO,KAAK,EAAE,WAAW,EAAiB,MAAM,aAAa,CAAC;AAE9D;;;GAGG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAiCtB"}

package/dist/tools/review-diff/checks/hotspots.js

@@ -0,0 +1,39 @@
+import { analyzeHotspots } from "../../hotspot-tools.js";
+/**
+ * Hotspots check: run analyzeHotspots and filter to files in changedFiles.
+ * Maps high-churn files to T3 advisory findings.
+ */
+export async function checkHotspots(index, changedFiles) {
+    const start = Date.now();
+    try {
+        const changedSet = new Set(changedFiles);
+        const result = await analyzeHotspots(index.repo);
+        const findings = result.hotspots
+            .filter((h) => changedSet.has(h.file))
+            .map((h) => ({
+            check: "hotspots",
+            severity: "warn",
+            message: `High churn file: ${h.file} — hotspot_score ${h.hotspot_score} (${h.commits} commits, ${h.lines_changed} lines changed)`,
+            file: h.file,
+        }));
+        return {
+            check: "hotspots",
+            status: findings.length > 0 ? "warn" : "pass",
+            findings,
+            duration_ms: Date.now() - start,
+            summary: findings.length > 0
+                ? `${findings.length} hotspot file(s) in diff`
+                : "No hotspot files in diff",
+        };
+    }
+    catch (err) {
+        return {
+            check: "hotspots",
+            status: "error",
+            findings: [],
+            duration_ms: Date.now() - start,
+            summary: `Error: ${err instanceof Error ? err.message : String(err)}`,
+        };
+    }
+}
+//# sourceMappingURL=hotspots.js.map

package/dist/tools/review-diff/checks/hotspots.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"hotspots.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/hotspots.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAIzD;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,KAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACzB,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,YAAY,CAAC,CAAC;QACzC,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAEjD,MAAM,QAAQ,GAAoB,MAAM,CAAC,QAAQ;aAC9C,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;aACrC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACX,KAAK,EAAE,UAAU;YACjB,QAAQ,EAAE,MAAe;YACzB,OAAO,EAAE,oBAAoB,CAAC,CAAC,IAAI,oBAAoB,CAAC,CAAC,aAAa,KAAK,CAAC,CAAC,OAAO,aAAa,CAAC,CAAC,aAAa,iBAAiB;YACjI,IAAI,EAAE,CAAC,CAAC,IAAI;SACb,CAAC,CAAC,CAAC;QAEN,OAAO;YACL,KAAK,EAAE,UAAU;YACjB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;YAC7C,QAAQ;YACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,0BAA0B;gBAC9C,CAAC,CAAC,0BAA0B;SAC/B,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO;YACL,KAAK,EAAE,UAAU;YACjB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,UAAU,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE;SACtE,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/tools/review-diff/checks/secrets.d.ts

@@ -0,0 +1,7 @@
+import type { CodeIndex } from "../../../types.js";
+import type { CheckResult } from "../types.js";
+/**
+ * Secrets check: run scanSecrets scoped to changedFiles and map findings to T1.
+ */
+export declare function checkSecrets(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
+//# sourceMappingURL=secrets.d.ts.map

package/dist/tools/review-diff/checks/secrets.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"secrets.d.ts","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/secrets.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAEnD,OAAO,KAAK,EAAE,WAAW,EAAiB,MAAM,aAAa,CAAC;AAE9D;;GAEG;AACH,wBAAsB,YAAY,CAChC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAiCtB"}

package/dist/tools/review-diff/checks/secrets.js

@@ -0,0 +1,38 @@
+import { scanSecrets } from "../../secret-tools.js";
+import { literalChangedFilePattern } from "../file-pattern.js";
+/**
+ * Secrets check: run scanSecrets scoped to changedFiles and map findings to T1.
+ */
+export async function checkSecrets(index, changedFiles) {
+    const start = Date.now();
+    try {
+        const filePattern = literalChangedFilePattern(changedFiles);
+        const result = await scanSecrets(index.repo, { file_pattern: filePattern, min_confidence: "high" });
+        const findings = result.findings.map((f) => ({
+            check: "secrets",
+            severity: "error",
+            message: `Secret detected: ${f.rule} (${f.severity}) — ${f.masked_secret}`,
+            file: f.file,
+            line: f.line,
+        }));
+        return {
+            check: "secrets",
+            status: findings.length > 0 ? "fail" : "pass",
+            findings,
+            duration_ms: Date.now() - start,
+            summary: findings.length > 0
+                ? `${findings.length} secret(s) detected`
+                : "No secrets detected",
+        };
+    }
+    catch (err) {
+        return {
+            check: "secrets",
+            status: "error",
+            findings: [],
+            duration_ms: Date.now() - start,
+            summary: `Error: ${err instanceof Error ? err.message : String(err)}`,
+        };
+    }
+}
+//# sourceMappingURL=secrets.js.map

package/dist/tools/review-diff/checks/secrets.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"secrets.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/secrets.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAEpD,OAAO,EAAE,yBAAyB,EAAE,MAAM,oBAAoB,CAAC;AAG/D;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,YAAY,CAChC,KAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACzB,IAAI,CAAC;QACH,MAAM,WAAW,GAAG,yBAAyB,CAAC,YAAY,CAAC,CAAC;QAE5D,MAAM,MAAM,GAAG,MAAM,WAAW,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,YAAY,EAAE,WAAW,EAAE,cAAc,EAAE,MAAM,EAAE,CAAC,CAAC;QAEpG,MAAM,QAAQ,GAAoB,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAC5D,KAAK,EAAE,SAAS;YAChB,QAAQ,EAAE,OAAO;YACjB,OAAO,EAAE,oBAAoB,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,QAAQ,OAAO,CAAC,CAAC,aAAa,EAAE;YAC1E,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,CAAC,IAAI;SACb,CAAC,CAAC,CAAC;QAEJ,OAAO;YACL,KAAK,EAAE,SAAS;YAChB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;YAC7C,QAAQ;YACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;gBAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,qBAAqB;gBACzC,CAAC,CAAC,qBAAqB;SAC1B,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO;YACL,KAAK,EAAE,SAAS;YAChB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;YAC/B,OAAO,EAAE,UAAU,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE;SACtE,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/tools/review-diff/checks/test-gaps.d.ts

@@ -0,0 +1,16 @@
+import type { CodeIndex } from "../../../types.js";
+import type { CheckResult } from "../types.js";
+/**
+ * Test-gaps check: for each changed non-test source file, verify that at least
+ * one test file covers it — either by naming convention or by import reference.
+ *
+ * Naming convention candidates:
+ *   foo.ts -> foo.test.ts, foo.spec.ts, __tests__/foo.ts, __tests__/foo.test.ts
+ *
+ * Import graph: search index symbols from test files whose source imports the
+ * source file's base name (without extension).
+ *
+ * If BOTH pathways find 0 tests -> T3 advisory finding.
+ */
+export declare function checkTestGaps(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
+//# sourceMappingURL=test-gaps.d.ts.map

package/dist/tools/review-diff/checks/test-gaps.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"test-gaps.d.ts","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/test-gaps.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AACnD,OAAO,KAAK,EAAE,WAAW,EAAiB,MAAM,aAAa,CAAC;AAI9D;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAwBtB"}

package/dist/tools/review-diff/checks/test-gaps.js

@@ -0,0 +1,77 @@
+import path from "node:path";
+import { isTestFile } from "../../../utils/test-file.js";
+const SOURCE_EXTENSIONS = /\.(tsx?|jsx?)$/;
+/**
+ * Test-gaps check: for each changed non-test source file, verify that at least
+ * one test file covers it — either by naming convention or by import reference.
+ *
+ * Naming convention candidates:
+ *   foo.ts -> foo.test.ts, foo.spec.ts, __tests__/foo.ts, __tests__/foo.test.ts
+ *
+ * Import graph: search index symbols from test files whose source imports the
+ * source file's base name (without extension).
+ *
+ * If BOTH pathways find 0 tests -> T3 advisory finding.
+ */
+export async function checkTestGaps(index, changedFiles) {
+    const start = Date.now();
+    const indexFilePaths = new Set(index.files.map((f) => f.path));
+    const findings = [];
+    for (const sourceFile of sourceFilesFromDiff(changedFiles)) {
+        if (hasTestCoverage(index, indexFilePaths, sourceFile))
+            continue;
+        findings.push({
+            check: "test-gaps",
+            severity: "warn",
+            message: `No test found for "${sourceFile}" — add a test file matching naming convention or import it from a test`,
+            file: sourceFile,
+        });
+    }
+    return {
+        check: "test-gaps",
+        status: findings.length > 0 ? "warn" : "pass",
+        findings,
+        duration_ms: Date.now() - start,
+        summary: findings.length > 0
+            ? `${findings.length} source file(s) with no test coverage found`
+            : "All changed source files have test coverage",
+    };
+}
+function sourceFilesFromDiff(changedFiles) {
+    return changedFiles.filter((f) => SOURCE_EXTENSIONS.test(f) && !isTestFile(f));
+}
+function hasTestCoverage(index, indexFilePaths, sourceFile) {
+    const base = path.basename(sourceFile).replace(SOURCE_EXTENSIONS, "");
+    return hasTestByNaming(indexFilePaths, sourceFile, base) || hasTestByImport(index, base);
+}
+function hasTestByNaming(indexFilePaths, sourceFile, base) {
+    return testCandidates(sourceFile, base).some((candidate) => indexFilePaths.has(candidate));
+}
+function testCandidates(sourceFile, base) {
+    const dir = path.dirname(sourceFile);
+    const testsDir = dir.replace(/^src\//, "tests/");
+    return [
+        path.join(dir, `${base}.test.ts`),
+        path.join(dir, `${base}.spec.ts`),
+        path.join(dir, `${base}.test.tsx`),
+        path.join(dir, `${base}.spec.tsx`),
+        path.join(dir, `${base}.test.js`),
+        path.join(dir, `${base}.spec.js`),
+        path.join(dir, "__tests__", `${base}.ts`),
+        path.join(dir, "__tests__", `${base}.test.ts`),
+        path.join(testsDir, `${base}.test.ts`),
+        path.join(testsDir, `${base}.spec.ts`),
+        path.join(testsDir, `${base}.test.tsx`),
+        path.join(testsDir, `${base}.test.js`),
+    ];
+}
+function hasTestByImport(index, base) {
+    return index.symbols.some((sym) => {
+        if (!isTestFile(sym.file))
+            return false;
+        if (!sym.source)
+            return false;
+        return sym.source.includes(base);
+    });
+}
+//# sourceMappingURL=test-gaps.js.map

package/dist/tools/review-diff/checks/test-gaps.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"test-gaps.js","sourceRoot":"","sources":["../../../../src/tools/review-diff/checks/test-gaps.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,UAAU,EAAE,MAAM,6BAA6B,CAAC;AAIzD,MAAM,iBAAiB,GAAG,gBAAgB,CAAC;AAE3C;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,KAAgB,EAChB,YAAsB;IAEtB,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACzB,MAAM,cAAc,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;IAC/D,MAAM,QAAQ,GAAoB,EAAE,CAAC;IAErC,KAAK,MAAM,UAAU,IAAI,mBAAmB,CAAC,YAAY,CAAC,EAAE,CAAC;QAC3D,IAAI,eAAe,CAAC,KAAK,EAAE,cAAc,EAAE,UAAU,CAAC;YAAE,SAAS;QACjE,QAAQ,CAAC,IAAI,CAAC;YACZ,KAAK,EAAE,WAAW;YAClB,QAAQ,EAAE,MAAM;YAChB,OAAO,EAAE,sBAAsB,UAAU,yEAAyE;YAClH,IAAI,EAAE,UAAU;SACjB,CAAC,CAAC;IACL,CAAC;IAED,OAAO;QACL,KAAK,EAAE,WAAW;QAClB,MAAM,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;QAC7C,QAAQ;QACR,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK;QAC/B,OAAO,EAAE,QAAQ,CAAC,MAAM,GAAG,CAAC;YAC1B,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,6CAA6C;YACjE,CAAC,CAAC,6CAA6C;KAClD,CAAC;AACJ,CAAC;AAED,SAAS,mBAAmB,CAAC,YAAsB;IACjD,OAAO,YAAY,CAAC,MAAM,CACxB,CAAC,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,CACnD,CAAC;AACJ,CAAC;AAED,SAAS,eAAe,CACtB,KAAgB,EAChB,cAA2B,EAC3B,UAAkB;IAElB,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,OAAO,CAAC,iBAAiB,EAAE,EAAE,CAAC,CAAC;IACtE,OAAO,eAAe,CAAC,cAAc,EAAE,UAAU,EAAE,IAAI,CAAC,IAAI,eAAe,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;AAC3F,CAAC;AAED,SAAS,eAAe,CACtB,cAA2B,EAC3B,UAAkB,EAClB,IAAY;IAEZ,OAAO,cAAc,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,EAAE,CACzD,cAAc,CAAC,GAAG,CAAC,SAAS,CAAC,CAC9B,CAAC;AACJ,CAAC;AAED,SAAS,cAAc,CAAC,UAAkB,EAAE,IAAY;IACtD,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IACrC,MAAM,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;IACjD,OAAO;QACL,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,UAAU,CAAC;QACjC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,UAAU,CAAC;QACjC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,WAAW,CAAC;QAClC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,WAAW,CAAC;QAClC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,UAAU,CAAC;QACjC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,UAAU,CAAC;QACjC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,WAAW,EAAE,GAAG,IAAI,KAAK,CAAC;QACzC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,WAAW,EAAE,GAAG,IAAI,UAAU,CAAC;QAC9C,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,IAAI,UAAU,CAAC;QACtC,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,IAAI,UAAU,CAAC;QACtC,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,IAAI,WAAW,CAAC;QACvC,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,IAAI,UAAU,CAAC;KACvC,CAAC;AACJ,CAAC;AAED,SAAS,eAAe,CAAC,KAAgB,EAAE,IAAY;IACrD,OAAO,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE;QAChC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,OAAO,KAAK,CAAC;QACxC,IAAI,CAAC,GAAG,CAAC,MAAM;YAAE,OAAO,KAAK,CAAC;QAC9B,OAAO,GAAG,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;AACL,CAAC"}

package/dist/tools/review-diff/constants.d.ts

@@ -0,0 +1,6 @@
+export declare const ALL_CHECKS: readonly ["secrets", "breaking", "coupling", "complexity", "dead-code", "blast-radius", "bug-patterns", "test-gaps", "hotspots", "astro-hydration"];
+export type CheckName = (typeof ALL_CHECKS)[number];
+export declare const DEFAULT_MAX_FILES = 50;
+export declare const DEFAULT_CHECK_TIMEOUT_MS = 30000;
+export declare const HEAD_TILDE_PATTERN: RegExp;
+//# sourceMappingURL=constants.d.ts.map

package/dist/tools/review-diff/constants.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/constants.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,UAAU,qJAWb,CAAC;AAEX,MAAM,MAAM,SAAS,GAAG,CAAC,OAAO,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC;AAEpD,eAAO,MAAM,iBAAiB,KAAK,CAAC;AACpC,eAAO,MAAM,wBAAwB,QAAS,CAAC;AAC/C,eAAO,MAAM,kBAAkB,QAAe,CAAC"}

package/dist/tools/review-diff/constants.js

@@ -0,0 +1,16 @@
+export const ALL_CHECKS = [
+    "secrets",
+    "breaking",
+    "coupling",
+    "complexity",
+    "dead-code",
+    "blast-radius",
+    "bug-patterns",
+    "test-gaps",
+    "hotspots",
+    "astro-hydration",
+];
+export const DEFAULT_MAX_FILES = 50;
+export const DEFAULT_CHECK_TIMEOUT_MS = 30_000;
+export const HEAD_TILDE_PATTERN = /^HEAD~\d+$/;
+//# sourceMappingURL=constants.js.map

package/dist/tools/review-diff/constants.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"constants.js","sourceRoot":"","sources":["../../../src/tools/review-diff/constants.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,UAAU,GAAG;IACxB,SAAS;IACT,UAAU;IACV,UAAU;IACV,YAAY;IACZ,WAAW;IACX,cAAc;IACd,cAAc;IACd,WAAW;IACX,UAAU;IACV,iBAAiB;CACT,CAAC;AAIX,MAAM,CAAC,MAAM,iBAAiB,GAAG,EAAE,CAAC;AACpC,MAAM,CAAC,MAAM,wBAAwB,GAAG,MAAM,CAAC;AAC/C,MAAM,CAAC,MAAM,kBAAkB,GAAG,YAAY,CAAC"}

package/dist/tools/review-diff/file-pattern.d.ts

@@ -0,0 +1,2 @@
+export declare function literalChangedFilePattern(changedFiles: string[]): string;
+//# sourceMappingURL=file-pattern.d.ts.map

package/dist/tools/review-diff/file-pattern.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-pattern.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/file-pattern.ts"],"names":[],"mappings":"AAAA,wBAAgB,yBAAyB,CAAC,YAAY,EAAE,MAAM,EAAE,GAAG,MAAM,CAQxE"}

package/dist/tools/review-diff/file-pattern.js

@@ -0,0 +1,9 @@
+export function literalChangedFilePattern(changedFiles) {
+    if (changedFiles.length === 1) {
+        return changedFiles[0].replace(/[\\*?\[\]{}(),!+@]/g, "\\$&");
+    }
+    return `{${changedFiles
+        .map((filePath) => filePath.replace(/[\\*?\[\]{}(),!+@]/g, "\\$&"))
+        .join(",")}}`;
+}
+//# sourceMappingURL=file-pattern.js.map

package/dist/tools/review-diff/file-pattern.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-pattern.js","sourceRoot":"","sources":["../../../src/tools/review-diff/file-pattern.ts"],"names":[],"mappings":"AAAA,MAAM,UAAU,yBAAyB,CAAC,YAAsB;IAC9D,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO,YAAY,CAAC,CAAC,CAAE,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;IACjE,CAAC;IAED,OAAO,IAAI,YAAY;SACpB,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;SAClE,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;AAClB,CAAC"}

package/dist/tools/review-diff/orchestrator.d.ts

@@ -0,0 +1,3 @@
+import type { ReviewDiffOptions, ReviewDiffResult } from "./types.js";
+export declare function reviewDiff(repo: string, opts: ReviewDiffOptions): Promise<ReviewDiffResult>;
+//# sourceMappingURL=orchestrator.d.ts.map

package/dist/tools/review-diff/orchestrator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/orchestrator.ts"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAe,iBAAiB,EAAE,gBAAgB,EAAiC,MAAM,YAAY,CAAC;AAoBlH,wBAAsB,UAAU,CAC9B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,iBAAiB,GACtB,OAAO,CAAC,gBAAgB,CAAC,CAiC3B"}

package/dist/tools/review-diff/orchestrator.js

@@ -0,0 +1,173 @@
+import picomatch from "picomatch";
+import { changedSymbols } from "../diff-tools.js";
+import { getCodeIndex } from "../index-tools.js";
+import { validateGitRef } from "../../utils/git-validation.js";
+import { ALL_CHECKS, DEFAULT_CHECK_TIMEOUT_MS, DEFAULT_MAX_FILES, HEAD_TILDE_PATTERN, } from "./constants.js";
+import { runCheck } from "./check-runner.js";
+import { calculateScore, determineVerdict } from "./scoring.js";
+import { withTimeout } from "./timeout.js";
+export async function reviewDiff(repo, opts) {
+    const startTime = Date.now();
+    const since = opts.since ?? "HEAD~1";
+    const until = opts.until;
+    const maxFiles = opts.max_files ?? DEFAULT_MAX_FILES;
+    const checkTimeoutMs = opts.check_timeout_ms ?? DEFAULT_CHECK_TIMEOUT_MS;
+    const prepared = await prepareReview(repo, opts, since, until, maxFiles, startTime);
+    if (prepared.status === "early")
+        return prepared.result;
+    const enabledChecks = resolveEnabledChecks(opts.checks);
+    const checkResults = await runEnabledChecks(enabledChecks, repo, prepared.reviewState.changedFiles, prepared.index, since, until ?? "HEAD", checkTimeoutMs);
+    for (const cr of checkResults) {
+        prepared.reviewState.allFindings.push(...cr.findings);
+    }
+    return reviewResult(repo, since, startTime, prepared.reviewState, checkResults);
+}
+async function prepareReview(repo, opts, since, until, maxFiles, startTime) {
+    const refError = validateDiffRefs(since, until);
+    if (refError) {
+        return {
+            status: "early",
+            result: failReviewResult(repo, since, startTime, `invalid_ref: ${refError}`),
+        };
+    }
+    const index = await getCodeIndex(repo);
+    if (!index) {
+        return {
+            status: "early",
+            result: failReviewResult(repo, since, startTime, `Repository not found: ${repo}`),
+        };
+    }
+    const changedFiles = await getFilteredChangedFiles(repo, since, until, opts);
+    if (changedFiles.length === 0) {
+        return {
+            status: "early",
+            result: emptyDiffResult(repo, since, startTime),
+        };
+    }
+    return {
+        status: "ready",
+        index,
+        reviewState: prepareDiffReviewState(changedFiles, maxFiles, since),
+    };
+}
+async function getFilteredChangedFiles(repo, since, until, opts) {
+    const diffResult = await changedSymbols(repo, since, until ?? "HEAD", undefined);
+    return applyExcludePatterns(diffResult.map((f) => f.file), opts.exclude_patterns);
+}
+function validateDiffRefs(since, until) {
+    try {
+        validateGitRef(since);
+        if (until && until !== "WORKING" && until !== "STAGED") {
+            validateGitRef(until);
+        }
+        return null;
+    }
+    catch (err) {
+        return err instanceof Error ? err.message : String(err);
+    }
+}
+function failReviewResult(repo, since, startTime, error) {
+    return earlyReviewResult(repo, since, startTime, 0, "fail", error);
+}
+function emptyDiffResult(repo, since, startTime) {
+    return earlyReviewResult(repo, since, startTime, 100, "pass");
+}
+function earlyReviewResult(repo, since, startTime, score, verdict, error) {
+    const result = {
+        repo,
+        since,
+        checks: [],
+        findings: [],
+        score,
+        verdict,
+        duration_ms: Date.now() - startTime,
+        diff_stats: { files_changed: 0, files_reviewed: 0 },
+        metadata: {},
+    };
+    if (error !== undefined)
+        result.error = error;
+    return result;
+}
+function applyExcludePatterns(changedFiles, excludePatterns) {
+    if (!excludePatterns || excludePatterns.length === 0)
+        return changedFiles;
+    const isExcluded = picomatch(excludePatterns);
+    return changedFiles.filter((f) => !isExcluded(f));
+}
+function prepareDiffReviewState(changedFiles, maxFiles, since) {
+    const allFindings = [];
+    const metadata = {};
+    const totalFilesChanged = changedFiles.length;
+    let filesToReview = changedFiles;
+    if (filesToReview.length > maxFiles) {
+        metadata.files_capped = true;
+        allFindings.push({
+            check: "large-diff",
+            severity: "info",
+            message: `Large diff: ${filesToReview.length} files changed, reviewing first ${maxFiles}. Consider smaller commits.`,
+        });
+        filesToReview = filesToReview.slice(0, maxFiles);
+    }
+    if (!HEAD_TILDE_PATTERN.test(since)) {
+        metadata.index_warning =
+            `Ref "${since}" is not a HEAD~N pattern. Index may not reflect this commit range.`;
+    }
+    return {
+        changedFiles: filesToReview,
+        totalFilesChanged,
+        allFindings,
+        metadata,
+    };
+}
+function reviewResult(repo, since, startTime, reviewState, checkResults) {
+    return {
+        repo,
+        since,
+        checks: checkResults,
+        findings: reviewState.allFindings,
+        score: calculateScore(reviewState.allFindings, checkResults),
+        verdict: determineVerdict(checkResults),
+        duration_ms: Date.now() - startTime,
+        diff_stats: {
+            files_changed: reviewState.totalFilesChanged,
+            files_reviewed: reviewState.changedFiles.length,
+        },
+        metadata: reviewState.metadata,
+    };
+}
+function resolveEnabledChecks(checks) {
+    const requestedChecks = checks
+        ? checks.split(",").map((c) => c.trim())
+        : [...ALL_CHECKS];
+    return requestedChecks.filter((c) => ALL_CHECKS.includes(c));
+}
+async function runEnabledChecks(enabledChecks, repo, changedFiles, index, since, until, checkTimeoutMs) {
+    const checkPromises = enabledChecks.map((checkName) => withTimeout(runCheck(checkName, repo, changedFiles, index, since, until), checkTimeoutMs));
+    const settled = await Promise.allSettled(checkPromises);
+    return settled.map((outcome, i) => checkResultFromSettled(outcome, enabledChecks[i] ?? `check_${i}`, checkTimeoutMs));
+}
+function checkResultFromSettled(outcome, checkName, checkTimeoutMs) {
+    if (!outcome || outcome.status === "rejected") {
+        return {
+            check: checkName,
+            status: "error",
+            findings: [],
+            duration_ms: 0,
+            summary: `Error: ${outcome && outcome.status === "rejected" && outcome.reason instanceof Error ? outcome.reason.message : String(outcome?.status === "rejected" ? outcome.reason : "unknown")}`,
+        };
+    }
+    if (isTimeoutSentinel(outcome.value)) {
+        return {
+            check: checkName,
+            status: "timeout",
+            findings: [],
+            duration_ms: checkTimeoutMs,
+            summary: `Timed out after ${checkTimeoutMs}ms`,
+        };
+    }
+    return outcome.value;
+}
+function isTimeoutSentinel(value) {
+    return value.status === "timeout" && !("findings" in value);
+}
+//# sourceMappingURL=orchestrator.js.map

package/dist/tools/review-diff/orchestrator.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator.js","sourceRoot":"","sources":["../../../src/tools/review-diff/orchestrator.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,WAAW,CAAC;AAClC,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAClD,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,OAAO,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAE/D,OAAO,EACL,UAAU,EACV,wBAAwB,EACxB,iBAAiB,EACjB,kBAAkB,GAEnB,MAAM,gBAAgB,CAAC;AACxB,OAAO,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAC;AAC7C,OAAO,EAAE,cAAc,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AAChE,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAsB3C,MAAM,CAAC,KAAK,UAAU,UAAU,CAC9B,IAAY,EACZ,IAAuB;IAEvB,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IAC7B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,IAAI,QAAQ,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC;IACzB,MAAM,QAAQ,GAAG,IAAI,CAAC,SAAS,IAAI,iBAAiB,CAAC;IACrD,MAAM,cAAc,GAAG,IAAI,CAAC,gBAAgB,IAAI,wBAAwB,CAAC;IAEzE,MAAM,QAAQ,GAAG,MAAM,aAAa,CAClC,IAAI,EACJ,IAAI,EACJ,KAAK,EACL,KAAK,EACL,QAAQ,EACR,SAAS,CACV,CAAC;IACF,IAAI,QAAQ,CAAC,MAAM,KAAK,OAAO;QAAE,OAAO,QAAQ,CAAC,MAAM,CAAC;IAExD,MAAM,aAAa,GAAG,oBAAoB,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACxD,MAAM,YAAY,GAAG,MAAM,gBAAgB,CACzC,aAAa,EACb,IAAI,EACJ,QAAQ,CAAC,WAAW,CAAC,YAAY,EACjC,QAAQ,CAAC,KAAK,EACd,KAAK,EACL,KAAK,IAAI,MAAM,EACf,cAAc,CACf,CAAC;IAEF,KAAK,MAAM,EAAE,IAAI,YAAY,EAAE,CAAC;QAC9B,QAAQ,CAAC,WAAW,CAAC,WAAW,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,QAAQ,CAAC,CAAC;IACxD,CAAC;IAED,OAAO,YAAY,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,QAAQ,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;AAClF,CAAC;AAED,KAAK,UAAU,aAAa,CAC1B,IAAY,EACZ,IAAuB,EACvB,KAAa,EACb,KAAyB,EACzB,QAAgB,EAChB,SAAiB;IAEjB,MAAM,QAAQ,GAAG,gBAAgB,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;IAChD,IAAI,QAAQ,EAAE,CAAC;QACb,OAAO;YACL,MAAM,EAAE,OAAO;YACf,MAAM,EAAE,gBAAgB,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,gBAAgB,QAAQ,EAAE,CAAC;SAC7E,CAAC;IACJ,CAAC;IAED,MAAM,KAAK,GAAG,MAAM,YAAY,CAAC,IAAI,CAAC,CAAC;IACvC,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,OAAO;YACL,MAAM,EAAE,OAAO;YACf,MAAM,EAAE,gBAAgB,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,yBAAyB,IAAI,EAAE,CAAC;SAClF,CAAC;IACJ,CAAC;IAED,MAAM,YAAY,GAAG,MAAM,uBAAuB,CAAC,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,IAAI,CAAC,CAAC;IAC7E,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO;YACL,MAAM,EAAE,OAAO;YACf,MAAM,EAAE,eAAe,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,CAAC;SAChD,CAAC;IACJ,CAAC;IAED,OAAO;QACL,MAAM,EAAE,OAAO;QACf,KAAK;QACL,WAAW,EAAE,sBAAsB,CAAC,YAAY,EAAE,QAAQ,EAAE,KAAK,CAAC;KACnE,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,uBAAuB,CACpC,IAAY,EACZ,KAAa,EACb,KAAyB,EACzB,IAAuB;IAEvB,MAAM,UAAU,GAAG,MAAM,cAAc,CACrC,IAAI,EACJ,KAAK,EACL,KAAK,IAAI,MAAM,EACf,SAAS,CACV,CAAC;IAEF,OAAO,oBAAoB,CACzB,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,EAC7B,IAAI,CAAC,gBAAgB,CACtB,CAAC;AACJ,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAa,EAAE,KAAyB;IAChE,IAAI,CAAC;QACH,cAAc,CAAC,KAAK,CAAC,CAAC;QACtB,IAAI,KAAK,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,QAAQ,EAAE,CAAC;YACvD,cAAc,CAAC,KAAK,CAAC,CAAC;QACxB,CAAC;QACD,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACtB,OAAO,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IAC1D,CAAC;AACH,CAAC;AAED,SAAS,gBAAgB,CACvB,IAAY,EACZ,KAAa,EACb,SAAiB,EACjB,KAAa;IAEb,OAAO,iBAAiB,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,EAAE,MAAM,EAAE,KAAK,CAAC,CAAC;AACrE,CAAC;AAED,SAAS,eAAe,CACtB,IAAY,EACZ,KAAa,EACb,SAAiB;IAEjB,OAAO,iBAAiB,CAAC,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,GAAG,EAAE,MAAM,CAAC,CAAC;AAChE,CAAC;AAED,SAAS,iBAAiB,CACxB,IAAY,EACZ,KAAa,EACb,SAAiB,EACjB,KAAa,EACb,OAAiC,EACjC,KAAc;IAEd,MAAM,MAAM,GAAqB;QAC/B,IAAI;QACJ,KAAK;QACL,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE,EAAE;QACZ,KAAK;QACL,OAAO;QACP,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS;QACnC,UAAU,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE,cAAc,EAAE,CAAC,EAAE;QACnD,QAAQ,EAAE,EAAE;KACb,CAAC;IACF,IAAI,KAAK,KAAK,SAAS;QAAE,MAAM,CAAC,KAAK,GAAG,KAAK,CAAC;IAC9C,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,oBAAoB,CAC3B,YAAsB,EACtB,eAAqC;IAErC,IAAI,CAAC,eAAe,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,YAAY,CAAC;IAC1E,MAAM,UAAU,GAAG,SAAS,CAAC,eAAe,CAAC,CAAC;IAC9C,OAAO,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;AACpD,CAAC;AAED,SAAS,sBAAsB,CAC7B,YAAsB,EACtB,QAAgB,EAChB,KAAa;IAEb,MAAM,WAAW,GAAoB,EAAE,CAAC;IACxC,MAAM,QAAQ,GAAmB,EAAE,CAAC;IACpC,MAAM,iBAAiB,GAAG,YAAY,CAAC,MAAM,CAAC;IAC9C,IAAI,aAAa,GAAG,YAAY,CAAC;IAEjC,IAAI,aAAa,CAAC,MAAM,GAAG,QAAQ,EAAE,CAAC;QACpC,QAAQ,CAAC,YAAY,GAAG,IAAI,CAAC;QAC7B,WAAW,CAAC,IAAI,CAAC;YACf,KAAK,EAAE,YAAY;YACnB,QAAQ,EAAE,MAAM;YAChB,OAAO,EAAE,eAAe,aAAa,CAAC,MAAM,mCAAmC,QAAQ,6BAA6B;SACrH,CAAC,CAAC;QACH,aAAa,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC;IACnD,CAAC;IAED,IAAI,CAAC,kBAAkB,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;QACpC,QAAQ,CAAC,aAAa;YACpB,QAAQ,KAAK,qEAAqE,CAAC;IACvF,CAAC;IAED,OAAO;QACL,YAAY,EAAE,aAAa;QAC3B,iBAAiB;QACjB,WAAW;QACX,QAAQ;KACT,CAAC;AACJ,CAAC;AAED,SAAS,YAAY,CACnB,IAAY,EACZ,KAAa,EACb,SAAiB,EACjB,WAA4B,EAC5B,YAA2B;IAE3B,OAAO;QACL,IAAI;QACJ,KAAK;QACL,MAAM,EAAE,YAAY;QACpB,QAAQ,EAAE,WAAW,CAAC,WAAW;QACjC,KAAK,EAAE,cAAc,CAAC,WAAW,CAAC,WAAW,EAAE,YAAY,CAAC;QAC5D,OAAO,EAAE,gBAAgB,CAAC,YAAY,CAAC;QACvC,WAAW,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS;QACnC,UAAU,EAAE;YACV,aAAa,EAAE,WAAW,CAAC,iBAAiB;YAC5C,cAAc,EAAE,WAAW,CAAC,YAAY,CAAC,MAAM;SAChD;QACD,QAAQ,EAAE,WAAW,CAAC,QAAQ;KAC/B,CAAC;AACJ,CAAC;AAED,SAAS,oBAAoB,CAAC,MAA0B;IACtD,MAAM,eAAe,GAAG,MAAM;QAC5B,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACxC,CAAC,CAAC,CAAC,GAAG,UAAU,CAAC,CAAC;IAEpB,OAAO,eAAe,CAAC,MAAM,CAAC,CAAC,CAAC,EAAkB,EAAE,CAClD,UAAU,CAAC,QAAQ,CAAC,CAAc,CAAC,CACpC,CAAC;AACJ,CAAC;AAED,KAAK,UAAU,gBAAgB,CAC7B,aAA0B,EAC1B,IAAY,EACZ,YAAsB,EACtB,KAAgB,EAChB,KAAa,EACb,KAAa,EACb,cAAsB;IAEtB,MAAM,aAAa,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC,SAAS,EAAE,EAAE,CACpD,WAAW,CACT,QAAQ,CAAC,SAAS,EAAE,IAAI,EAAE,YAAY,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,CAAC,EAC5D,cAAc,CACf,CACF,CAAC;IAEF,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC;IACxD,OAAO,OAAO,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,CAAC,EAAE,EAAE,CAChC,sBAAsB,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC,CAAC,IAAI,SAAS,CAAC,EAAE,EAAE,cAAc,CAAC,CAClF,CAAC;AACJ,CAAC;AAED,SAAS,sBAAsB,CAC7B,OAAwE,EACxE,SAAiB,EACjB,cAAsB;IAEtB,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,MAAM,KAAK,UAAU,EAAE,CAAC;QAC9C,OAAO;YACL,KAAK,EAAE,SAAS;YAChB,MAAM,EAAE,OAAO;YACf,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,CAAC;YACd,OAAO,EAAE,UAAU,OAAO,IAAI,OAAO,CAAC,MAAM,KAAK,UAAU,IAAI,OAAO,CAAC,MAAM,YAAY,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,KAAK,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS,CAAC,EAAE;SAChM,CAAC;IACJ,CAAC;IAED,IAAI,iBAAiB,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QACrC,OAAO;YACL,KAAK,EAAE,SAAS;YAChB,MAAM,EAAE,SAAS;YACjB,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,cAAc;YAC3B,OAAO,EAAE,mBAAmB,cAAc,IAAI;SAC/C,CAAC;IACJ,CAAC;IAED,OAAO,OAAO,CAAC,KAAK,CAAC;AACvB,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAoC;IAC7D,OAAO,KAAK,CAAC,MAAM,KAAK,SAAS,IAAI,CAAC,CAAC,UAAU,IAAI,KAAK,CAAC,CAAC;AAC9D,CAAC"}