codesift-mcp 0.9.0 → 0.9.10

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

@@ -0,0 +1,31 @@
+import type { CheckResult, ReviewFinding } from "./types.js";
+/**
+ * Returns the tier (1 | 2 | 3) for a given check name.
+ *
+ * Tier 1 — critical (−20 per finding): secrets, breaking
+ * Tier 2 — important (−5 per finding): coupling, complexity, dead-code,
+ *           blast-radius, bug-patterns
+ * Tier 3 — advisory (−1 per finding, only if no T1/T2): test-gaps, hotspots
+ */
+export declare function findingTier(check: string): 1 | 2 | 3;
+/**
+ * Calculates a 0-100 quality score from findings and check results.
+ *
+ * Deductions:
+ *   - T1 findings: −20 each, floor at 0
+ *   - T2 findings: −5 each, floor at 20 (overridden by T1 floor)
+ *   - T3 findings: −1 each, floor at 50 (only applied when there are no T1/T2 findings)
+ *   - Errored checks: −3 each
+ *   - Final floor: 0
+ */
+export declare function calculateScore(findings: ReviewFinding[], checks: CheckResult[]): number;
+/**
+ * Determines the overall verdict from check statuses.
+ *
+ * - Any "fail" -> "fail"
+ * - Any "warn" (and no "fail") -> "warn"
+ * - Otherwise -> "pass"
+ * - "timeout" and "error" do not affect verdict direction
+ */
+export declare function determineVerdict(checks: CheckResult[]): "pass" | "warn" | "fail";
+//# sourceMappingURL=scoring.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"scoring.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/scoring.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE7D;;;;;;;GAOG;AACH,wBAAgB,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAiBpD;AAED;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAC5B,QAAQ,EAAE,aAAa,EAAE,EACzB,MAAM,EAAE,WAAW,EAAE,GACpB,MAAM,CAiCR;AAED;;;;;;;GAOG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,WAAW,EAAE,GAAG,MAAM,GAAG,MAAM,GAAG,MAAM,CAQhF"}

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

@@ -0,0 +1,82 @@
+/**
+ * Returns the tier (1 | 2 | 3) for a given check name.
+ *
+ * Tier 1 — critical (−20 per finding): secrets, breaking
+ * Tier 2 — important (−5 per finding): coupling, complexity, dead-code,
+ *           blast-radius, bug-patterns
+ * Tier 3 — advisory (−1 per finding, only if no T1/T2): test-gaps, hotspots
+ */
+export function findingTier(check) {
+    switch (check) {
+        case "secrets":
+        case "breaking":
+            return 1;
+        case "coupling":
+        case "complexity":
+        case "dead-code":
+        case "blast-radius":
+        case "bug-patterns":
+        case "astro-hydration":
+            return 2;
+        default:
+            return 3;
+    }
+}
+/**
+ * Calculates a 0-100 quality score from findings and check results.
+ *
+ * Deductions:
+ *   - T1 findings: −20 each, floor at 0
+ *   - T2 findings: −5 each, floor at 20 (overridden by T1 floor)
+ *   - T3 findings: −1 each, floor at 50 (only applied when there are no T1/T2 findings)
+ *   - Errored checks: −3 each
+ *   - Final floor: 0
+ */
+export function calculateScore(findings, checks) {
+    const t1Count = findings.filter((f) => findingTier(f.check) === 1).length;
+    const t2Count = findings.filter((f) => findingTier(f.check) === 2).length;
+    const t3Count = findings.filter((f) => findingTier(f.check) === 3).length;
+    const errorCount = checks.filter((c) => c.status === "error").length;
+    let score = 100;
+    // Tier 1 deductions
+    score -= t1Count * 20;
+    if (score < 0)
+        score = 0;
+    // Tier 2 deductions (floor 20, but T1 can override below 20)
+    const afterT2 = score - t2Count * 5;
+    if (t1Count === 0) {
+        // T2 floor is 20 only when no T1 findings
+        score = Math.max(afterT2, 20);
+    }
+    else {
+        // T1 already applied; T2 can further reduce but overall floor is 0
+        score = Math.max(afterT2, 0);
+    }
+    // Tier 3 deductions (floor 50, only when no T1/T2 findings)
+    if (t1Count === 0 && t2Count === 0) {
+        const afterT3 = score - t3Count * 1;
+        score = Math.max(afterT3, 50);
+    }
+    // Error deductions
+    score -= errorCount * 3;
+    // Final floor
+    return Math.max(score, 0);
+}
+/**
+ * Determines the overall verdict from check statuses.
+ *
+ * - Any "fail" -> "fail"
+ * - Any "warn" (and no "fail") -> "warn"
+ * - Otherwise -> "pass"
+ * - "timeout" and "error" do not affect verdict direction
+ */
+export function determineVerdict(checks) {
+    const hasFail = checks.some((c) => c.status === "fail");
+    if (hasFail)
+        return "fail";
+    const hasWarn = checks.some((c) => c.status === "warn");
+    if (hasWarn)
+        return "warn";
+    return "pass";
+}
+//# sourceMappingURL=scoring.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"scoring.js","sourceRoot":"","sources":["../../../src/tools/review-diff/scoring.ts"],"names":[],"mappings":"AAEA;;;;;;;GAOG;AACH,MAAM,UAAU,WAAW,CAAC,KAAa;IACvC,QAAQ,KAAK,EAAE,CAAC;QACd,KAAK,SAAS,CAAC;QACf,KAAK,UAAU;YACb,OAAO,CAAC,CAAC;QAEX,KAAK,UAAU,CAAC;QAChB,KAAK,YAAY,CAAC;QAClB,KAAK,WAAW,CAAC;QACjB,KAAK,cAAc,CAAC;QACpB,KAAK,cAAc,CAAC;QACpB,KAAK,iBAAiB;YACpB,OAAO,CAAC,CAAC;QAEX;YACE,OAAO,CAAC,CAAC;IACb,CAAC;AACH,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,cAAc,CAC5B,QAAyB,EACzB,MAAqB;IAErB,MAAM,OAAO,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;IAC1E,MAAM,OAAO,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;IAC1E,MAAM,OAAO,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;IAC1E,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,OAAO,CAAC,CAAC,MAAM,CAAC;IAErE,IAAI,KAAK,GAAG,GAAG,CAAC;IAEhB,oBAAoB;IACpB,KAAK,IAAI,OAAO,GAAG,EAAE,CAAC;IACtB,IAAI,KAAK,GAAG,CAAC;QAAE,KAAK,GAAG,CAAC,CAAC;IAEzB,6DAA6D;IAC7D,MAAM,OAAO,GAAG,KAAK,GAAG,OAAO,GAAG,CAAC,CAAC;IACpC,IAAI,OAAO,KAAK,CAAC,EAAE,CAAC;QAClB,0CAA0C;QAC1C,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IAChC,CAAC;SAAM,CAAC;QACN,mEAAmE;QACnE,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC;IAC/B,CAAC;IAED,4DAA4D;IAC5D,IAAI,OAAO,KAAK,CAAC,IAAI,OAAO,KAAK,CAAC,EAAE,CAAC;QACnC,MAAM,OAAO,GAAG,KAAK,GAAG,OAAO,GAAG,CAAC,CAAC;QACpC,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IAChC,CAAC;IAED,mBAAmB;IACnB,KAAK,IAAI,UAAU,GAAG,CAAC,CAAC;IAExB,cAAc;IACd,OAAO,IAAI,CAAC,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC;AAC5B,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,gBAAgB,CAAC,MAAqB;IACpD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC;IACxD,IAAI,OAAO;QAAE,OAAO,MAAM,CAAC;IAE3B,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC;IACxD,IAAI,OAAO;QAAE,OAAO,MAAM,CAAC;IAE3B,OAAO,MAAM,CAAC;AAChB,CAAC"}

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

@@ -0,0 +1,5 @@
+export interface TimeoutSentinel {
+    status: "timeout";
+}
+export declare function withTimeout<T>(promise: Promise<T>, ms: number): Promise<T | TimeoutSentinel>;
+//# sourceMappingURL=timeout.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"timeout.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/timeout.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,SAAS,CAAC;CACnB;AAED,wBAAgB,WAAW,CAAC,CAAC,EAC3B,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC,EACnB,EAAE,EAAE,MAAM,GACT,OAAO,CAAC,CAAC,GAAG,eAAe,CAAC,CAY9B"}

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

@@ -0,0 +1,14 @@
+export function withTimeout(promise, ms) {
+    let timeout;
+    const timeoutPromise = new Promise((resolve) => {
+        timeout = setTimeout(() => resolve({ status: "timeout" }), ms);
+    });
+    return Promise.race([
+        promise,
+        timeoutPromise,
+    ]).finally(() => {
+        if (timeout !== undefined)
+            clearTimeout(timeout);
+    });
+}
+//# sourceMappingURL=timeout.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"timeout.js","sourceRoot":"","sources":["../../../src/tools/review-diff/timeout.ts"],"names":[],"mappings":"AAIA,MAAM,UAAU,WAAW,CACzB,OAAmB,EACnB,EAAU;IAEV,IAAI,OAAkD,CAAC;IACvD,MAAM,cAAc,GAAG,IAAI,OAAO,CAAkB,CAAC,OAAO,EAAE,EAAE;QAC9D,OAAO,GAAG,UAAU,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,OAAO,OAAO,CAAC,IAAI,CAAC;QAClB,OAAO;QACP,cAAc;KACf,CAAC,CAAC,OAAO,CAAC,GAAG,EAAE;QACd,IAAI,OAAO,KAAK,SAAS;YAAE,YAAY,CAAC,OAAO,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;AACL,CAAC"}

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

@@ -0,0 +1,56 @@
+export interface ReviewDiffOptions {
+    repo: string;
+    since?: string;
+    /** End ref — defaults to "HEAD". Use "WORKING" for uncommitted changes. */
+    until?: string;
+    /** Comma-separated check names to run (defaults to all) */
+    checks?: string;
+    /** Token budget for responses (default 8000) */
+    token_budget?: number;
+    /** Glob patterns of files to exclude from review */
+    exclude_patterns?: string[];
+    /** Maximum files to review before capping (default 50) */
+    max_files?: number;
+    /** Per-check timeout in milliseconds (default 30000) */
+    check_timeout_ms?: number;
+}
+export interface ReviewFinding {
+    /** Which check produced this finding */
+    check: string;
+    severity: "error" | "warn" | "info";
+    message: string;
+    file?: string;
+    line?: number;
+    symbol?: string;
+}
+export interface CheckResult {
+    check: string;
+    status: "pass" | "warn" | "fail" | "error" | "timeout";
+    findings: ReviewFinding[];
+    duration_ms: number;
+    /** Human-readable summary line (optional) */
+    summary?: string;
+}
+export interface DiffStats {
+    files_changed: number;
+    files_reviewed: number;
+}
+export interface ReviewMetadata {
+    files_capped?: boolean;
+    index_warning?: string;
+}
+export interface ReviewDiffResult {
+    repo: string;
+    since: string;
+    checks: CheckResult[];
+    findings: ReviewFinding[];
+    /** 0-100 quality score */
+    score: number;
+    verdict: "pass" | "warn" | "fail";
+    duration_ms: number;
+    diff_stats: DiffStats;
+    metadata: ReviewMetadata;
+    /** Structured error (present instead of throwing) */
+    error?: string;
+}
+//# sourceMappingURL=types.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/tools/review-diff/types.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,2EAA2E;IAC3E,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,2DAA2D;IAC3D,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gDAAgD;IAChD,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,oDAAoD;IACpD,gBAAgB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC5B,0DAA0D;IAC1D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,wDAAwD;IACxD,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B;AAED,MAAM,WAAW,aAAa;IAC5B,wCAAwC;IACxC,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAC;IACpC,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,GAAG,SAAS,CAAC;IACvD,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,WAAW,EAAE,MAAM,CAAC;IACpB,6CAA6C;IAC7C,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,SAAS;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,cAAc;IAC7B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,0BAA0B;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,SAAS,CAAC;IACtB,QAAQ,EAAE,cAAc,CAAC;IACzB,qDAAqD;IACrD,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB"}

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

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=types.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../../src/tools/review-diff/types.ts"],"names":[],"mappings":""}

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

@@ -1,149 +1,21 @@
 /**
- * review-diff-tools.ts
- *
- * Types, tier assignment, scoring, verdict, and orchestrator for the review_diff MCP tool.
- * Pure functions + one async orchestrator that fans out sub-checks.
- */
-import type { CodeIndex } from "../types.js";
-export interface ReviewDiffOptions {
-    repo: string;
-    since?: string;
-    /** End ref — defaults to "HEAD". Use "WORKING" for uncommitted changes. */
-    until?: string;
-    /** Comma-separated check names to run (defaults to all) */
-    checks?: string;
-    /** Token budget for responses (default 8000) */
-    token_budget?: number;
-    /** Glob patterns of files to exclude from review */
-    exclude_patterns?: string[];
-    /** Maximum files to review before capping (default 50) */
-    max_files?: number;
-    /** Per-check timeout in milliseconds (default 30000) */
-    check_timeout_ms?: number;
-}
-export interface ReviewFinding {
-    /** Which check produced this finding */
-    check: string;
-    severity: "error" | "warn" | "info";
-    message: string;
-    file?: string;
-    line?: number;
-    symbol?: string;
-}
-export interface CheckResult {
-    check: string;
-    status: "pass" | "warn" | "fail" | "error" | "timeout";
-    findings: ReviewFinding[];
-    duration_ms: number;
-    /** Human-readable summary line (optional) */
-    summary?: string;
-}
-export interface DiffStats {
-    files_changed: number;
-    files_reviewed: number;
-}
-export interface ReviewMetadata {
-    files_capped?: boolean;
-    index_warning?: string;
-}
-export interface ReviewDiffResult {
-    repo: string;
-    since: string;
-    checks: CheckResult[];
-    findings: ReviewFinding[];
-    /** 0-100 quality score */
-    score: number;
-    verdict: "pass" | "warn" | "fail";
-    duration_ms: number;
-    diff_stats: DiffStats;
-    metadata: ReviewMetadata;
-    /** Structured error (present instead of throwing) */
-    error?: string;
-}
-/**
- * Returns the tier (1 | 2 | 3) for a given check name.
- *
- * Tier 1 — critical (−20 per finding): secrets, breaking
- * Tier 2 — important (−5 per finding): coupling, complexity, dead-code,
- *           blast-radius, bug-patterns
- * Tier 3 — advisory (−1 per finding, only if no T1/T2): test-gaps, hotspots
- */
-export declare function findingTier(check: string): 1 | 2 | 3;
-/**
- * Calculates a 0-100 quality score from findings and check results.
- *
- * Deductions:
- *   - T1 findings: −20 each, floor at 0
- *   - T2 findings: −5 each, floor at 20 (overridden by T1 floor)
- *   - T3 findings: −1 each, floor at 50 (only applied when there are no T1/T2 findings)
- *   - Errored checks: −3 each
- *   - Final floor: 0
- */
-export declare function calculateScore(findings: ReviewFinding[], checks: CheckResult[]): number;
-/**
- * Determines the overall verdict from check statuses.
- *
- * - Any "fail" → "fail"
- * - Any "warn" (and no "fail") → "warn"
- * - Otherwise → "pass"
- * - "timeout" and "error" do not affect verdict direction
- */
-export declare function determineVerdict(checks: CheckResult[]): "pass" | "warn" | "fail";
-/**
- * Blast-radius check: run impactAnalysis and map affected_symbols to T2 findings.
- */
-export declare function checkBlastRadius(index: CodeIndex, since: string, until: string): Promise<CheckResult>;
-/**
- * Secrets check: run scanSecrets scoped to changedFiles and map findings to T1.
- */
-export declare function checkSecrets(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
-/**
- * Dead-code check: run findDeadCode scoped to changedFiles and map candidates to T2 findings.
- */
-export declare function checkDeadCode(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
-/**
- * Astro hydration check: run hydrationAuditFromIndex scoped to Astro files in diff.
- * Returns skipped when diff has zero .astro files.
- */
-export declare function checkAstroHydration(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
-export declare function checkBugPatterns(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
-/**
- * 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>;
-/**
- * Complexity delta check: run analyzeComplexity and filter to functions in
- * changedFiles with cyclomatic complexity > 10. Maps to T2 findings.
- */
-export declare function checkComplexityDelta(index: CodeIndex, changedFiles: string[]): Promise<CheckResult>;
-/**
- * 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>;
-/**
- * Breaking changes check: detect exported symbols removed between `since` and
- * current index.  For each changed .ts/.js file, `git show` retrieves the old
- * source and a regex extracts export names.  These are compared against the
- * current index symbols.  Missing exports → T1 "breaking" findings.
- *
- * File-level renames (detected via `git diff --find-renames`) are suppressed
- * because renames naturally lose old export names.
- */
-export declare function checkBreakingChanges(index: CodeIndex, repoRoot: string, changedFiles: string[], since: string, until: string): Promise<CheckResult>;
-/**
- * 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>;
-export declare function reviewDiff(repo: string, opts: ReviewDiffOptions): Promise<ReviewDiffResult>;
+ * Public facade for the review_diff MCP tool.
+ *
+ * The implementation lives under ./review-diff/ so each check adapter can
+ * evolve independently without making this high-churn entry point a conflict
+ * magnet.
+ */
+export type { CheckResult, DiffStats, ReviewDiffOptions, ReviewDiffResult, ReviewFinding, ReviewMetadata, } from "./review-diff/types.js";
+export { calculateScore, determineVerdict, findingTier, } from "./review-diff/scoring.js";
+export { checkBlastRadius } from "./review-diff/checks/blast-radius.js";
+export { checkSecrets } from "./review-diff/checks/secrets.js";
+export { checkDeadCode } from "./review-diff/checks/dead-code.js";
+export { checkAstroHydration } from "./review-diff/checks/astro-hydration.js";
+export { checkBugPatterns } from "./review-diff/checks/bug-patterns.js";
+export { checkHotspots } from "./review-diff/checks/hotspots.js";
+export { checkComplexityDelta } from "./review-diff/checks/complexity.js";
+export { checkCouplingGaps } from "./review-diff/checks/coupling.js";
+export { checkBreakingChanges } from "./review-diff/checks/breaking.js";
+export { checkTestGaps } from "./review-diff/checks/test-gaps.js";
+export { reviewDiff } from "./review-diff/orchestrator.js";
 //# sourceMappingURL=review-diff-tools.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"review-diff-tools.d.ts","sourceRoot":"","sources":["../../src/tools/review-diff-tools.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAiBH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAM7C,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,2EAA2E;IAC3E,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,2DAA2D;IAC3D,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gDAAgD;IAChD,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,oDAAoD;IACpD,gBAAgB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC5B,0DAA0D;IAC1D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,wDAAwD;IACxD,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B;AAED,MAAM,WAAW,aAAa;IAC5B,wCAAwC;IACxC,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAC;IACpC,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,GAAG,SAAS,CAAC;IACvD,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,WAAW,EAAE,MAAM,CAAC;IACpB,6CAA6C;IAC7C,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,SAAS;IACxB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,cAAc;IAC7B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,0BAA0B;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,SAAS,CAAC;IACtB,QAAQ,EAAE,cAAc,CAAC;IACzB,qDAAqD;IACrD,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAMD;;;;;;;GAOG;AACH,wBAAgB,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAiBpD;AAMD;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAC5B,QAAQ,EAAE,aAAa,EAAE,EACzB,MAAM,EAAE,WAAW,EAAE,GACpB,MAAM,CAiCR;AAMD;;;;;;;GAOG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,WAAW,EAAE,GAAG,MAAM,GAAG,MAAM,GAAG,MAAM,CAQhF;AAiDD;;GAEG;AACH,wBAAsB,gBAAgB,CACpC,KAAK,EAAE,SAAS,EAChB,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,MAAM,GACZ,OAAO,CAAC,WAAW,CAAC,CAgCtB;AAED;;GAEG;AACH,wBAAsB,YAAY,CAChC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAqCtB;AAED;;GAEG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAsCtB;AAiED;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAqDtB;AAED,wBAAsB,gBAAgB,CACpC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAiEtB;AAED;;;GAGG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAiCtB;AAED;;;GAGG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAmCtB;AAED;;;GAGG;AACH,wBAAsB,iBAAiB,CACrC,QAAQ,EAAE,MAAM,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAuDtB;AAED;;;;;;;;GAQG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,SAAS,EAChB,QAAQ,EAAE,MAAM,EAChB,YAAY,EAAE,MAAM,EAAE,EACtB,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,MAAM,GACZ,OAAO,CAAC,WAAW,CAAC,CAyGtB;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CACjC,KAAK,EAAE,SAAS,EAChB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,WAAW,CAAC,CAwEtB;AAqDD,wBAAsB,UAAU,CAC9B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,iBAAiB,GACtB,OAAO,CAAC,gBAAgB,CAAC,CAiM3B"}
+{"version":3,"file":"review-diff-tools.d.ts","sourceRoot":"","sources":["../../src/tools/review-diff-tools.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,YAAY,EACV,WAAW,EACX,SAAS,EACT,iBAAiB,EACjB,gBAAgB,EAChB,aAAa,EACb,cAAc,GACf,MAAM,wBAAwB,CAAC;AAEhC,OAAO,EACL,cAAc,EACd,gBAAgB,EAChB,WAAW,GACZ,MAAM,0BAA0B,CAAC;AAElC,OAAO,EAAE,gBAAgB,EAAE,MAAM,sCAAsC,CAAC;AACxE,OAAO,EAAE,YAAY,EAAE,MAAM,iCAAiC,CAAC;AAC/D,OAAO,EAAE,aAAa,EAAE,MAAM,mCAAmC,CAAC;AAClE,OAAO,EAAE,mBAAmB,EAAE,MAAM,yCAAyC,CAAC;AAC9E,OAAO,EAAE,gBAAgB,EAAE,MAAM,sCAAsC,CAAC;AACxE,OAAO,EAAE,aAAa,EAAE,MAAM,kCAAkC,CAAC;AACjE,OAAO,EAAE,oBAAoB,EAAE,MAAM,oCAAoC,CAAC;AAC1E,OAAO,EAAE,iBAAiB,EAAE,MAAM,kCAAkC,CAAC;AACrE,OAAO,EAAE,oBAAoB,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,EAAE,aAAa,EAAE,MAAM,mCAAmC,CAAC;AAClE,OAAO,EAAE,UAAU,EAAE,MAAM,+BAA+B,CAAC"}