@diff-review-system/drs 1.0.0 → 2.0.0

package/dist/lib/review-orchestrator.d.ts

@@ -0,0 +1,77 @@
+/**
+ * Review orchestrator for local diff reviews
+ *
+ * This module handles local git diff reviews (pre-push analysis).
+ * It uses the shared core logic from review-core.ts.
+ */
+import type { DRSConfig } from './config.js';
+import { type ModelOverrides } from './config.js';
+import { type OpencodeClient } from '../opencode/client.js';
+import { calculateSummary, type ReviewIssue } from './comment-formatter.js';
+import { displayReviewSummary as displaySummary, hasBlockingIssues as checkBlockingIssues } from './review-core.js';
+/**
+ * Source information for a review (platform-agnostic)
+ */
+export interface ReviewSource {
+    /** Human-readable name for logging (e.g., "PR #123", "MR !456", "Local diff") */
+    name: string;
+    /** List of changed file paths */
+    files: string[];
+    /** Optional: files with their diff patches (if available, passed directly to agents) */
+    filesWithDiffs?: Array<{
+        filename: string;
+        patch: string;
+    }>;
+    /** Additional context to pass to review agents */
+    context: Record<string, unknown>;
+    /** Working directory for the review (defaults to process.cwd()) */
+    workingDir?: string;
+    /** Debug mode - print OpenCode configuration */
+    debug?: boolean;
+    /** Whether this is a staged diff (affects git diff command) */
+    staged?: boolean;
+}
+/**
+ * Result of a review execution
+ */
+export interface ReviewResult {
+    /** All issues found by review agents */
+    issues: ReviewIssue[];
+    /** Calculated summary statistics */
+    summary: ReturnType<typeof calculateSummary>;
+    /** Diff-based change summary when available */
+    changeSummary?: import('./change-summary.js').ChangeSummary;
+    /** Number of files actually reviewed (after filtering) */
+    filesReviewed: number;
+}
+/**
+ * Filter files based on ignore patterns in config
+ */
+export declare function filterIgnoredFiles(files: string[], config: DRSConfig): string[];
+export interface ConnectOptions {
+    debug?: boolean;
+    modelOverrides?: ModelOverrides;
+}
+/**
+ * Connect to OpenCode server (or start in-process)
+ */
+export declare function connectToOpenCode(config: DRSConfig, workingDir?: string, options?: ConnectOptions): Promise<OpencodeClient>;
+/**
+ * Execute a code review using OpenCode agents
+ *
+ * This is the core review orchestrator that handles:
+ * - File filtering (ignore patterns)
+ * - OpenCode connection
+ * - Agent execution and streaming
+ * - Issue parsing and collection
+ * - Summary calculation
+ *
+ * Platform-specific logic (GitHub/GitLab/local) should:
+ * 1. Fetch changed files from their source
+ * 2. Call this function with a ReviewSource
+ * 3. Handle posting results to their platform
+ */
+export declare function executeReview(config: DRSConfig, source: ReviewSource): Promise<ReviewResult>;
+export declare const displayReviewSummary: typeof displaySummary;
+export declare const hasBlockingIssues: typeof checkBlockingIssues;
+//# sourceMappingURL=review-orchestrator.d.ts.map

package/dist/lib/review-orchestrator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-orchestrator.d.ts","sourceRoot":"","sources":["../../src/lib/review-orchestrator.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAC7C,OAAO,EAIL,KAAK,cAAc,EACpB,MAAM,aAAa,CAAC;AACrB,OAAO,EAAgC,KAAK,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAC1F,OAAO,EAAE,gBAAgB,EAAE,KAAK,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAC5E,OAAO,EAGL,oBAAoB,IAAI,cAAc,EACtC,iBAAiB,IAAI,mBAAmB,EAEzC,MAAM,kBAAkB,CAAC;AAG1B;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,iFAAiF;IACjF,IAAI,EAAE,MAAM,CAAC;IACb,iCAAiC;IACjC,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,wFAAwF;IACxF,cAAc,CAAC,EAAE,KAAK,CAAC;QAAE,QAAQ,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC5D,kDAAkD;IAClD,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACjC,mEAAmE;IACnE,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,gDAAgD;IAChD,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,+DAA+D;IAC/D,MAAM,CAAC,EAAE,OAAO,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,wCAAwC;IACxC,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,oCAAoC;IACpC,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;IAC7C,+CAA+C;IAC/C,aAAa,CAAC,EAAE,OAAO,qBAAqB,EAAE,aAAa,CAAC;IAC5D,0DAA0D;IAC1D,aAAa,EAAE,MAAM,CAAC;CACvB;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,EAAE,EAAE,MAAM,EAAE,SAAS,GAAG,MAAM,EAAE,CAE/E;AAED,MAAM,WAAW,cAAc;IAC7B,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,cAAc,CAAC,EAAE,cAAc,CAAC;CACjC;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,SAAS,EACjB,UAAU,CAAC,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,cAAc,GACvB,OAAO,CAAC,cAAc,CAAC,CA0BzB;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,aAAa,CACjC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,YAAY,GACnB,OAAO,CAAC,YAAY,CAAC,CAoFvB;AAGD,eAAO,MAAM,oBAAoB,uBAAiB,CAAC;AACnD,eAAO,MAAM,iBAAiB,4BAAsB,CAAC"}

package/dist/lib/review-orchestrator.js

@@ -0,0 +1,124 @@
+/**
+ * Review orchestrator for local diff reviews
+ *
+ * This module handles local git diff reviews (pre-push analysis).
+ * It uses the shared core logic from review-core.ts.
+ */
+import chalk from 'chalk';
+import { shouldIgnoreFile, getModelOverrides, getUnifiedModelOverride, } from './config.js';
+import { createOpencodeClientInstance } from '../opencode/client.js';
+import { calculateSummary } from './comment-formatter.js';
+import { buildBaseInstructions, runReviewPipeline, displayReviewSummary as displaySummary, hasBlockingIssues as checkBlockingIssues, } from './review-core.js';
+import { compressFilesWithDiffs, formatCompressionSummary } from './context-compression.js';
+/**
+ * Filter files based on ignore patterns in config
+ */
+export function filterIgnoredFiles(files, config) {
+    return files.filter((file) => !shouldIgnoreFile(file, config));
+}
+/**
+ * Connect to OpenCode server (or start in-process)
+ */
+export async function connectToOpenCode(config, workingDir, options) {
+    console.log(chalk.gray('Connecting to OpenCode server...\n'));
+    try {
+        // Get model overrides from DRS config
+        const modelOverrides = options?.modelOverrides ?? {
+            ...getModelOverrides(config),
+            ...getUnifiedModelOverride(config),
+        };
+        return await createOpencodeClientInstance({
+            baseUrl: config.opencode.serverUrl || undefined,
+            directory: workingDir || process.cwd(),
+            modelOverrides,
+            provider: config.opencode.provider,
+            config,
+            debug: options?.debug,
+        });
+    }
+    catch (error) {
+        console.error(chalk.red('✗ Failed to connect to OpenCode server'));
+        console.error(chalk.red(`Error: ${error instanceof Error ? error.message : String(error)}\n`));
+        console.log(chalk.yellow('Please ensure OpenCode server is running or check your configuration.\n'));
+        throw error;
+    }
+}
+/**
+ * Execute a code review using OpenCode agents
+ *
+ * This is the core review orchestrator that handles:
+ * - File filtering (ignore patterns)
+ * - OpenCode connection
+ * - Agent execution and streaming
+ * - Issue parsing and collection
+ * - Summary calculation
+ *
+ * Platform-specific logic (GitHub/GitLab/local) should:
+ * 1. Fetch changed files from their source
+ * 2. Call this function with a ReviewSource
+ * 3. Handle posting results to their platform
+ */
+export async function executeReview(config, source) {
+    console.log(chalk.gray(`Found ${source.files.length} changed file(s)\n`));
+    // Filter files based on ignore patterns
+    const filteredFiles = filterIgnoredFiles(source.files, config);
+    const ignoredCount = source.files.length - filteredFiles.length;
+    if (ignoredCount > 0) {
+        console.log(chalk.gray(`Ignoring ${ignoredCount} file(s) based on patterns\n`));
+    }
+    if (filteredFiles.length === 0) {
+        console.log(chalk.yellow('✓ No files to review after filtering\n'));
+        return {
+            issues: [],
+            summary: calculateSummary(0, []),
+            filesReviewed: 0,
+        };
+    }
+    console.log(chalk.gray(`Reviewing ${filteredFiles.length} file(s)\n`));
+    // Connect to OpenCode
+    const opencode = await connectToOpenCode(config, source.workingDir, { debug: source.debug });
+    try {
+        // Build instructions - use provided diffs if available, otherwise fall back to git command
+        const diffCommand = source.staged ? 'git diff --cached -- <file>' : 'git diff -- <file>';
+        // Use provided diffs if available (filtered to match filteredFiles)
+        let filesForInstructions;
+        if (source.filesWithDiffs && source.filesWithDiffs.length > 0) {
+            // Filter to only include files that passed ignore patterns
+            filesForInstructions = source.filesWithDiffs.filter((f) => filteredFiles.includes(f.filename));
+        }
+        else {
+            // No diffs provided - agents will need to run git diff
+            filesForInstructions = filteredFiles.map((f) => ({ filename: f }));
+        }
+        const compression = compressFilesWithDiffs(filesForInstructions, config.contextCompression);
+        const compressionSummary = formatCompressionSummary(compression);
+        if (compressionSummary) {
+            console.log(chalk.yellow('⚠ Diff content trimmed to fit token budget.\n'));
+        }
+        const baseInstructions = buildBaseInstructions(source.name, compression.files, diffCommand, compressionSummary);
+        // Run agents using shared core logic
+        const result = await runReviewPipeline(opencode, config, baseInstructions, source.name, filteredFiles, source.context, source.workingDir || process.cwd(), source.debug || false);
+        return {
+            issues: result.issues,
+            summary: result.summary,
+            changeSummary: result.changeSummary,
+            filesReviewed: result.filesReviewed,
+        };
+    }
+    catch (error) {
+        // Handle "all agents failed" error
+        if (error instanceof Error && error.message === 'All review agents failed') {
+            await opencode.shutdown();
+            process.exit(1);
+        }
+        throw error;
+    }
+    finally {
+        // Always shut down OpenCode client
+        await opencode.shutdown();
+    }
+}
+// Re-export display functions from core for backward compatibility
+export const displayReviewSummary = displaySummary;
+export const hasBlockingIssues = checkBlockingIssues;
+//# sourceMappingURL=review-orchestrator.js.map

package/dist/lib/review-orchestrator.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-orchestrator.js","sourceRoot":"","sources":["../../src/lib/review-orchestrator.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EACL,gBAAgB,EAChB,iBAAiB,EACjB,uBAAuB,GAExB,MAAM,aAAa,CAAC;AACrB,OAAO,EAAE,4BAA4B,EAAuB,MAAM,uBAAuB,CAAC;AAC1F,OAAO,EAAE,gBAAgB,EAAoB,MAAM,wBAAwB,CAAC;AAC5E,OAAO,EACL,qBAAqB,EACrB,iBAAiB,EACjB,oBAAoB,IAAI,cAAc,EACtC,iBAAiB,IAAI,mBAAmB,GAEzC,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,sBAAsB,EAAE,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AAoC5F;;GAEG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAe,EAAE,MAAiB;IACnE,OAAO,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,gBAAgB,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,CAAC;AACjE,CAAC;AAOD;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,MAAiB,EACjB,UAAmB,EACnB,OAAwB;IAExB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,oCAAoC,CAAC,CAAC,CAAC;IAE9D,IAAI,CAAC;QACH,sCAAsC;QACtC,MAAM,cAAc,GAAG,OAAO,EAAE,cAAc,IAAI;YAChD,GAAG,iBAAiB,CAAC,MAAM,CAAC;YAC5B,GAAG,uBAAuB,CAAC,MAAM,CAAC;SACnC,CAAC;QAEF,OAAO,MAAM,4BAA4B,CAAC;YACxC,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,SAAS,IAAI,SAAS;YAC/C,SAAS,EAAE,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE;YACtC,cAAc;YACd,QAAQ,EAAE,MAAM,CAAC,QAAQ,CAAC,QAAQ;YAClC,MAAM;YACN,KAAK,EAAE,OAAO,EAAE,KAAK;SACtB,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,wCAAwC,CAAC,CAAC,CAAC;QACnE,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,UAAU,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC;QAC/F,OAAO,CAAC,GAAG,CACT,KAAK,CAAC,MAAM,CAAC,yEAAyE,CAAC,CACxF,CAAC;QACF,MAAM,KAAK,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,CAAC,KAAK,UAAU,aAAa,CACjC,MAAiB,EACjB,MAAoB;IAEpB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,MAAM,CAAC,KAAK,CAAC,MAAM,oBAAoB,CAAC,CAAC,CAAC;IAE1E,wCAAwC;IACxC,MAAM,aAAa,GAAG,kBAAkB,CAAC,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;IAC/D,MAAM,YAAY,GAAG,MAAM,CAAC,KAAK,CAAC,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC;IAEhE,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC;QACrB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,YAAY,YAAY,8BAA8B,CAAC,CAAC,CAAC;IAClF,CAAC;IAED,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wCAAwC,CAAC,CAAC,CAAC;QACpE,OAAO;YACL,MAAM,EAAE,EAAE;YACV,OAAO,EAAE,gBAAgB,CAAC,CAAC,EAAE,EAAE,CAAC;YAChC,aAAa,EAAE,CAAC;SACjB,CAAC;IACJ,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,aAAa,aAAa,CAAC,MAAM,YAAY,CAAC,CAAC,CAAC;IAEvE,sBAAsB;IACtB,MAAM,QAAQ,GAAG,MAAM,iBAAiB,CAAC,MAAM,EAAE,MAAM,CAAC,UAAU,EAAE,EAAE,KAAK,EAAE,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;IAE7F,IAAI,CAAC;QACH,2FAA2F;QAC3F,MAAM,WAAW,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,6BAA6B,CAAC,CAAC,CAAC,oBAAoB,CAAC;QAEzF,oEAAoE;QACpE,IAAI,oBAAoC,CAAC;QACzC,IAAI,MAAM,CAAC,cAAc,IAAI,MAAM,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC9D,2DAA2D;YAC3D,oBAAoB,GAAG,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CACxD,aAAa,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC,CACnC,CAAC;QACJ,CAAC;aAAM,CAAC;YACN,uDAAuD;YACvD,oBAAoB,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC;QACrE,CAAC;QAED,MAAM,WAAW,GAAG,sBAAsB,CAAC,oBAAoB,EAAE,MAAM,CAAC,kBAAkB,CAAC,CAAC;QAC5F,MAAM,kBAAkB,GAAG,wBAAwB,CAAC,WAAW,CAAC,CAAC;QAEjE,IAAI,kBAAkB,EAAE,CAAC;YACvB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,+CAA+C,CAAC,CAAC,CAAC;QAC7E,CAAC;QAED,MAAM,gBAAgB,GAAG,qBAAqB,CAC5C,MAAM,CAAC,IAAI,EACX,WAAW,CAAC,KAAK,EACjB,WAAW,EACX,kBAAkB,CACnB,CAAC;QAEF,qCAAqC;QACrC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CACpC,QAAQ,EACR,MAAM,EACN,gBAAgB,EAChB,MAAM,CAAC,IAAI,EACX,aAAa,EACb,MAAM,CAAC,OAAO,EACd,MAAM,CAAC,UAAU,IAAI,OAAO,CAAC,GAAG,EAAE,EAClC,MAAM,CAAC,KAAK,IAAI,KAAK,CACtB,CAAC;QAEF,OAAO;YACL,MAAM,EAAE,MAAM,CAAC,MAAM;YACrB,OAAO,EAAE,MAAM,CAAC,OAAO;YACvB,aAAa,EAAE,MAAM,CAAC,aAAa;YACnC,aAAa,EAAE,MAAM,CAAC,aAAa;SACpC,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,mCAAmC;QACnC,IAAI,KAAK,YAAY,KAAK,IAAI,KAAK,CAAC,OAAO,KAAK,0BAA0B,EAAE,CAAC;YAC3E,MAAM,QAAQ,CAAC,QAAQ,EAAE,CAAC;YAC1B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;QACD,MAAM,KAAK,CAAC;IACd,CAAC;YAAS,CAAC;QACT,mCAAmC;QACnC,MAAM,QAAQ,CAAC,QAAQ,EAAE,CAAC;IAC5B,CAAC;AACH,CAAC;AAED,mEAAmE;AACnE,MAAM,CAAC,MAAM,oBAAoB,GAAG,cAAc,CAAC;AACnD,MAAM,CAAC,MAAM,iBAAiB,GAAG,mBAAmB,CAAC"}

package/dist/lib/review-orchestrator.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=review-orchestrator.test.d.ts.map

package/dist/lib/review-orchestrator.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-orchestrator.test.d.ts","sourceRoot":"","sources":["../../src/lib/review-orchestrator.test.ts"],"names":[],"mappings":""}

package/dist/lib/review-orchestrator.test.js

@@ -0,0 +1,413 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { filterIgnoredFiles, connectToOpenCode, executeReview, } from './review-orchestrator.js';
+// Mock dependencies
+vi.mock('./config.js', async () => {
+    const actual = await vi.importActual('./config.js');
+    return {
+        ...actual,
+        shouldIgnoreFile: vi.fn((file, config) => {
+            const patterns = config.review.ignorePatterns || [];
+            return patterns.some((pattern) => {
+                if (pattern.endsWith('/*')) {
+                    const dir = pattern.slice(0, -2);
+                    return file.startsWith(dir + '/');
+                }
+                if (pattern.includes('*')) {
+                    const regex = new RegExp('^' + pattern.replace(/\*/g, '.*').replace(/\?/g, '.') + '$');
+                    return regex.test(file);
+                }
+                return file === pattern;
+            });
+        }),
+        getModelOverrides: vi.fn(() => ({})),
+        getUnifiedModelOverride: vi.fn(() => ({})),
+    };
+});
+// Store mock client instance for verification
+let mockOpencodeClient;
+vi.mock('../opencode/client.js', () => ({
+    createOpencodeClientInstance: vi.fn(async () => {
+        mockOpencodeClient = {
+            createSession: vi.fn(async () => ({ id: 'session-1' })),
+            streamMessages: vi.fn(async function* () {
+                yield {
+                    role: 'assistant',
+                    content: JSON.stringify({
+                        issues: [
+                            {
+                                category: 'QUALITY',
+                                severity: 'MEDIUM',
+                                title: 'Test issue',
+                                file: 'src/app.ts',
+                                line: 10,
+                                problem: 'Test problem',
+                                solution: 'Test solution',
+                            },
+                        ],
+                    }),
+                };
+            }),
+            closeSession: vi.fn(async () => { }),
+            shutdown: vi.fn(async () => { }),
+        };
+        return mockOpencodeClient;
+    }),
+}));
+vi.mock('./review-core.js', () => ({
+    buildBaseInstructions: vi.fn((label) => `Instructions for ${label}`),
+    runReviewPipeline: vi.fn(async (_opencode, _config, _instructions, _label, files) => ({
+        issues: [
+            {
+                category: 'QUALITY',
+                severity: 'MEDIUM',
+                title: 'Test issue',
+                file: files[0] || 'src/app.ts',
+                line: 10,
+                problem: 'Test problem',
+                solution: 'Test solution',
+                agent: 'quality',
+            },
+        ],
+        summary: {
+            filesReviewed: files.length,
+            issuesFound: 1,
+            bySeverity: { CRITICAL: 0, HIGH: 0, MEDIUM: 1, LOW: 0 },
+            byCategory: { SECURITY: 0, QUALITY: 1, STYLE: 0, PERFORMANCE: 0, DOCUMENTATION: 0 },
+        },
+        filesReviewed: files.length,
+        agentResults: [],
+    })),
+    displayReviewSummary: vi.fn(),
+    hasBlockingIssues: vi.fn(() => false),
+}));
+vi.mock('./comment-formatter.js', () => ({
+    calculateSummary: vi.fn((filesReviewed, issues) => ({
+        filesReviewed,
+        issuesFound: issues.length,
+        bySeverity: {
+            CRITICAL: issues.filter((i) => i.severity === 'CRITICAL').length,
+            HIGH: issues.filter((i) => i.severity === 'HIGH').length,
+            MEDIUM: issues.filter((i) => i.severity === 'MEDIUM').length,
+            LOW: issues.filter((i) => i.severity === 'LOW').length,
+        },
+        byCategory: {
+            SECURITY: issues.filter((i) => i.category === 'SECURITY').length,
+            QUALITY: issues.filter((i) => i.category === 'QUALITY').length,
+            STYLE: issues.filter((i) => i.category === 'STYLE').length,
+            PERFORMANCE: issues.filter((i) => i.category === 'PERFORMANCE').length,
+            DOCUMENTATION: issues.filter((i) => i.category === 'DOCUMENTATION').length,
+        },
+    })),
+}));
+vi.mock('./context-compression.js', () => ({
+    compressFilesWithDiffs: vi.fn((files) => ({
+        files,
+        removedFiles: [],
+        removedHunks: 0,
+        originalTokens: 1000,
+        compressedTokens: 1000,
+    })),
+    formatCompressionSummary: vi.fn(() => null),
+}));
+describe('review-orchestrator', () => {
+    beforeEach(() => {
+        vi.spyOn(console, 'log').mockImplementation(() => { });
+        vi.spyOn(console, 'error').mockImplementation(() => { });
+    });
+    describe('filterIgnoredFiles', () => {
+        it('should filter files based on ignore patterns', () => {
+            const config = {
+                review: {
+                    ignorePatterns: ['*.test.ts', 'dist/*', 'node_modules/*'],
+                },
+            };
+            const files = [
+                'src/app.ts',
+                'src/app.test.ts',
+                'dist/bundle.js',
+                'node_modules/package/index.js',
+                'src/utils.ts',
+            ];
+            const result = filterIgnoredFiles(files, config);
+            expect(result).toEqual(['src/app.ts', 'src/utils.ts']);
+        });
+        it('should return all files when no ignore patterns', () => {
+            const config = {
+                opencode: {},
+                gitlab: { url: '', token: '' },
+                github: { token: '' },
+                review: {
+                    ignorePatterns: [],
+                },
+            };
+            const files = ['src/app.ts', 'src/utils.ts', 'README.md'];
+            const result = filterIgnoredFiles(files, config);
+            expect(result).toEqual(files);
+        });
+        it('should handle glob patterns correctly', () => {
+            const config = {
+                review: {
+                    ignorePatterns: ['**/*.spec.ts', 'test/**'],
+                },
+            };
+            const files = ['src/app.ts', 'src/app.spec.ts', 'test/integration.ts', 'lib/utils.ts'];
+            const result = filterIgnoredFiles(files, config);
+            expect(result).toContain('src/app.ts');
+            expect(result).toContain('lib/utils.ts');
+        });
+        it('should handle empty file list', () => {
+            const config = {
+                review: {
+                    ignorePatterns: ['*.test.ts'],
+                },
+            };
+            const result = filterIgnoredFiles([], config);
+            expect(result).toEqual([]);
+        });
+    });
+    describe('connectToOpenCode', () => {
+        it('should connect to OpenCode server successfully', async () => {
+            const config = {
+                opencode: {
+                    serverUrl: 'http://localhost:3000',
+                },
+                review: {},
+            };
+            const client = await connectToOpenCode(config, '/test/dir');
+            expect(client).toBeDefined();
+            // eslint-disable-next-line @typescript-eslint/unbound-method
+            expect(client.createSession).toBeDefined();
+            // eslint-disable-next-line @typescript-eslint/unbound-method
+            expect(client.shutdown).toBeDefined();
+        });
+        it('should handle connection failure', async () => {
+            const { createOpencodeClientInstance } = await import('../opencode/client.js');
+            vi.mocked(createOpencodeClientInstance).mockRejectedValueOnce(new Error('Connection failed'));
+            const config = {
+                opencode: {},
+                review: {},
+            };
+            await expect(connectToOpenCode(config)).rejects.toThrow('Connection failed');
+            expect(console.error).toHaveBeenCalledWith(expect.stringContaining('Failed to connect to OpenCode server'));
+        });
+        it('should pass model overrides to OpenCode client', async () => {
+            const { createOpencodeClientInstance } = await import('../opencode/client.js');
+            const { getModelOverrides, getUnifiedModelOverride } = await import('./config.js');
+            vi.mocked(getModelOverrides).mockReturnValue({
+                'review/security': 'claude-opus-4',
+            });
+            vi.mocked(getUnifiedModelOverride).mockReturnValue({
+                'review/unified-reviewer': 'claude-sonnet-4',
+            });
+            const config = {
+                opencode: {},
+                review: {},
+            };
+            await connectToOpenCode(config, '/test/dir', { debug: true });
+            expect(createOpencodeClientInstance).toHaveBeenCalledWith({
+                baseUrl: undefined,
+                config,
+                directory: '/test/dir',
+                modelOverrides: {
+                    'review/security': 'claude-opus-4',
+                    'review/unified-reviewer': 'claude-sonnet-4',
+                },
+                provider: undefined,
+                debug: true,
+            });
+        });
+        it('should use process.cwd() when no working directory provided', async () => {
+            const { createOpencodeClientInstance } = await import('../opencode/client.js');
+            const config = {
+                opencode: {},
+                review: {},
+            };
+            await connectToOpenCode(config);
+            expect(createOpencodeClientInstance).toHaveBeenCalledWith(expect.objectContaining({
+                config,
+                directory: process.cwd(),
+            }));
+        });
+    });
+    describe('executeReview', () => {
+        let mockConfig;
+        beforeEach(() => {
+            mockConfig = {
+                opencode: {},
+                review: {
+                    agents: ['security', 'quality'],
+                    ignorePatterns: ['*.test.ts'],
+                },
+                contextCompression: {
+                    enabled: false,
+                },
+            };
+        });
+        it('should execute review successfully with files', async () => {
+            const source = {
+                name: 'Local diff',
+                files: ['src/app.ts', 'src/utils.ts'],
+                context: {},
+                workingDir: '/test/dir',
+            };
+            const result = await executeReview(mockConfig, source);
+            expect(result.issues).toBeDefined();
+            expect(result.summary).toBeDefined();
+            expect(result.filesReviewed).toBe(2);
+            expect(result.issues.length).toBeGreaterThan(0);
+        });
+        it('should filter ignored files', async () => {
+            const source = {
+                name: 'Local diff',
+                files: ['src/app.ts', 'src/app.test.ts', 'src/utils.ts'],
+                context: {},
+            };
+            const result = await executeReview(mockConfig, source);
+            // Should have filtered out .test.ts file
+            expect(result.filesReviewed).toBe(2);
+        });
+        it('should return empty result when all files are ignored', async () => {
+            const source = {
+                name: 'Local diff',
+                files: ['test1.test.ts', 'test2.test.ts'],
+                context: {},
+            };
+            const result = await executeReview(mockConfig, source);
+            expect(result.issues).toEqual([]);
+            expect(result.filesReviewed).toBe(0);
+            expect(result.summary.issuesFound).toBe(0);
+        });
+        it('should handle staged diff command', async () => {
+            const { buildBaseInstructions } = await import('./review-core.js');
+            const source = {
+                name: 'Staged changes',
+                files: ['src/app.ts'],
+                context: {},
+                staged: true,
+            };
+            await executeReview(mockConfig, source);
+            expect(buildBaseInstructions).toHaveBeenCalledWith('Staged changes', expect.anything(), 'git diff --cached -- <file>', null);
+        });
+        it('should handle unstaged diff command', async () => {
+            const { buildBaseInstructions } = await import('./review-core.js');
+            const source = {
+                name: 'Unstaged changes',
+                files: ['src/app.ts'],
+                context: {},
+                staged: false,
+            };
+            await executeReview(mockConfig, source);
+            expect(buildBaseInstructions).toHaveBeenCalledWith('Unstaged changes', expect.anything(), 'git diff -- <file>', null);
+        });
+        it('should use provided diffs when available', async () => {
+            const source = {
+                name: 'PR #123',
+                files: ['src/app.ts', 'src/utils.ts'],
+                filesWithDiffs: [
+                    { filename: 'src/app.ts', patch: '+ new code' },
+                    { filename: 'src/utils.ts', patch: '- old code\n+ new code' },
+                ],
+                context: {},
+            };
+            const result = await executeReview(mockConfig, source);
+            expect(result.filesReviewed).toBe(2);
+        });
+        it('should filter filesWithDiffs to match filtered files', async () => {
+            const source = {
+                name: 'PR #123',
+                files: ['src/app.ts', 'src/app.test.ts', 'src/utils.ts'],
+                filesWithDiffs: [
+                    { filename: 'src/app.ts', patch: '+ new code' },
+                    { filename: 'src/app.test.ts', patch: '+ test code' },
+                    { filename: 'src/utils.ts', patch: '+ util code' },
+                ],
+                context: {},
+            };
+            await executeReview(mockConfig, source);
+            // Should filter out .test.ts file from diffs as well
+            const { buildBaseInstructions } = await import('./review-core.js');
+            expect(buildBaseInstructions).toHaveBeenCalled();
+        });
+        it('should handle empty file list', async () => {
+            const source = {
+                name: 'Empty review',
+                files: [],
+                context: {},
+            };
+            const result = await executeReview(mockConfig, source);
+            expect(result.issues).toEqual([]);
+            expect(result.filesReviewed).toBe(0);
+        });
+        it('should pass debug flag to agents', async () => {
+            const source = {
+                name: 'Debug review',
+                files: ['src/app.ts'],
+                context: {},
+                debug: true,
+            };
+            await executeReview(mockConfig, source);
+            const { runReviewPipeline } = await import('./review-core.js');
+            expect(runReviewPipeline).toHaveBeenCalledWith(expect.anything(), mockConfig, expect.anything(), 'Debug review', ['src/app.ts'], {}, process.cwd(), true);
+        });
+        it('should pass additional context to agents', async () => {
+            const source = {
+                name: 'PR #123',
+                files: ['src/app.ts'],
+                context: {
+                    prNumber: 123,
+                    author: 'test-user',
+                },
+            };
+            await executeReview(mockConfig, source);
+            const { runReviewPipeline } = await import('./review-core.js');
+            expect(runReviewPipeline).toHaveBeenCalledWith(expect.anything(), mockConfig, expect.anything(), 'PR #123', ['src/app.ts'], { prNumber: 123, author: 'test-user' }, process.cwd(), false);
+        });
+        it('should shutdown OpenCode client after review', async () => {
+            const source = {
+                name: 'Test review',
+                files: ['src/app.ts'],
+                context: {},
+            };
+            await executeReview(mockConfig, source);
+            // Verify shutdown was called on the mock client
+            expect(mockOpencodeClient.shutdown).toHaveBeenCalled();
+        });
+        it('should shutdown OpenCode client even on error', async () => {
+            const { runReviewPipeline } = await import('./review-core.js');
+            vi.mocked(runReviewPipeline).mockRejectedValueOnce(new Error('Review failed'));
+            const source = {
+                name: 'Test review',
+                files: ['src/app.ts'],
+                context: {},
+            };
+            await expect(executeReview(mockConfig, source)).rejects.toThrow('Review failed');
+            // Verify shutdown was called even on error
+            expect(mockOpencodeClient.shutdown).toHaveBeenCalled();
+        });
+        it('should handle "All review agents failed" error specially', async () => {
+            const { runReviewPipeline } = await import('./review-core.js');
+            vi.mocked(runReviewPipeline).mockRejectedValueOnce(new Error('All review agents failed'));
+            const mockExit = vi.spyOn(process, 'exit').mockImplementation((() => { }));
+            const source = {
+                name: 'Test review',
+                files: ['src/app.ts'],
+                context: {},
+            };
+            await expect(executeReview(mockConfig, source)).rejects.toThrow('All review agents failed');
+            expect(mockExit).toHaveBeenCalledWith(1);
+            mockExit.mockRestore();
+        });
+        it('should log compression warning when context is compressed', async () => {
+            const { formatCompressionSummary } = await import('./context-compression.js');
+            vi.mocked(formatCompressionSummary).mockReturnValueOnce('⚠️  Removed 5 files to fit token budget');
+            const source = {
+                name: 'Large PR',
+                files: ['src/app.ts'],
+                context: {},
+            };
+            await executeReview(mockConfig, source);
+            expect(console.log).toHaveBeenCalledWith(expect.stringContaining('Diff content trimmed to fit token budget'));
+        });
+    });
+});
+//# sourceMappingURL=review-orchestrator.test.js.map