@diff-review-system/drs 1.1.2 → 2.1.0

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

@@ -1 +1 @@
-{"version":3,"file":"review-orchestrator.d.ts","sourceRoot":"","sources":["../../src/lib/review-orchestrator.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAE7C,OAAO,EAAgC,KAAK,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAE1F,OAAO,EAAE,gBAAgB,EAAE,KAAK,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAG5E;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,iFAAiF;IACjF,IAAI,EAAE,MAAM,CAAC;IACb,iCAAiC;IACjC,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,kDAAkD;IAClD,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC7B,mEAAmE;IACnE,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;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,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;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,SAAS,EACjB,UAAU,CAAC,EAAE,MAAM,GAClB,OAAO,CAAC,cAAc,CAAC,CAoBzB;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,aAAa,CACjC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,YAAY,GACnB,OAAO,CAAC,YAAY,CAAC,CA8IvB;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,YAAY,GAAG,IAAI,CAgB/D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,YAAY,GAAG,OAAO,CAE/D"}
+{"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

@@ -1,9 +1,15 @@
+/**
+ * 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, getAgentNames } from './config.js';
+import { shouldIgnoreFile, getModelOverrides, getUnifiedModelOverride, } from './config.js';
 import { createOpencodeClientInstance } from '../opencode/client.js';
-import { parseReviewIssues } from './issue-parser.js';
 import { calculateSummary } from './comment-formatter.js';
-import { buildReviewPrompt } from './context-loader.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
  */
@@ -13,15 +19,21 @@ export function filterIgnoredFiles(files, config) {
 /**
  * Connect to OpenCode server (or start in-process)
  */
-export async function connectToOpenCode(config, workingDir) {
+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 = getModelOverrides(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) {
@@ -64,120 +76,49 @@ export async function executeReview(config, source) {
     }
     console.log(chalk.gray(`Reviewing ${filteredFiles.length} file(s)\n`));
     // Connect to OpenCode
-    const opencode = await connectToOpenCode(config, source.workingDir);
+    const opencode = await connectToOpenCode(config, source.workingDir, { debug: source.debug });
     try {
-        // Execute review
-        console.log(chalk.gray('Starting code analysis...\n'));
-        const issues = [];
-        const baseInstructions = `Review the following files from ${source.name}:
-
-${filteredFiles.map((f) => `- ${f}`).join('\n')}
-
-**Instructions:**
-1. Use the Read tool to examine each changed file
-2. Analyze the code for issues in your specialty area
-3. Output your findings in this JSON format:
-
-\`\`\`json
-{
-  "issues": [
-    {
-      "category": "SECURITY" | "QUALITY" | "STYLE" | "PERFORMANCE",
-      "severity": "CRITICAL" | "HIGH" | "MEDIUM" | "LOW",
-      "title": "Brief title",
-      "file": "path/to/file.ts",
-      "line": 42,
-      "problem": "Description of the problem",
-      "solution": "How to fix it",
-      "agent": "security" | "quality" | "style" | "performance"
-    }
-  ]
-}
-\`\`\`
-
-Be thorough and identify all issues. Include line numbers when possible.`;
-        const agentNames = getAgentNames(config);
-        const agentPromises = agentNames.map(async (agentType) => {
-            const agentName = `review/${agentType}`;
-            console.log(chalk.gray(`Running ${agentType} review...\n`));
-            try {
-                const reviewPrompt = buildReviewPrompt(agentType, baseInstructions, source.name, filteredFiles);
-                const session = await opencode.createSession({
-                    agent: agentName,
-                    message: reviewPrompt,
-                    context: {
-                        ...source.context,
-                        files: filteredFiles,
-                    },
-                });
-                const agentIssues = [];
-                for await (const message of opencode.streamMessages(session.id)) {
-                    if (message.role === 'assistant') {
-                        const parsedIssues = parseReviewIssues(message.content);
-                        if (parsedIssues.length > 0) {
-                            agentIssues.push(...parsedIssues);
-                            console.log(chalk.green(`✓ [${agentType}] Found ${parsedIssues.length} issue(s)`));
-                        }
-                    }
-                }
-                await opencode.closeSession(session.id);
-                return { agentType, success: true, issues: agentIssues };
-            }
-            catch (error) {
-                console.error(chalk.red(`✗ ${agentType} agent failed: ${error}`));
-                return { agentType, success: false, issues: [] };
-            }
-        });
-        const agentResults = await Promise.all(agentPromises);
-        const successfulAgents = agentResults.filter((r) => r.success);
-        const failedAgents = agentResults.filter((r) => !r.success);
-        if (successfulAgents.length === 0) {
-            console.error(chalk.red('\n✗ All review agents failed!\n'));
-            console.error(chalk.yellow('This usually means:\n' +
-                '  1. Model configuration is incorrect or missing\n' +
-                '  2. API credentials are invalid or missing\n' +
-                '  3. Models are not accessible or timed out\n' +
-                '  4. Agents cannot find files to review\n'));
-            await opencode.shutdown();
-            process.exit(1);
+        // 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));
         }
-        if (failedAgents.length > 0) {
-            console.log(chalk.yellow(`\n⚠️  ${failedAgents.length} of ${agentResults.length} agents failed: ${failedAgents.map((r) => r.agentType).join(', ')}\n`));
+        else {
+            // No diffs provided - agents will need to run git diff
+            filesForInstructions = filteredFiles.map((f) => ({ filename: f }));
         }
-        agentResults.forEach((result) => issues.push(...result.issues));
-        const summary = calculateSummary(filteredFiles.length, issues);
+        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,
-            summary,
-            filesReviewed: filteredFiles.length,
+            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();
     }
 }
-/**
- * Display review summary to terminal (common formatting)
- */
-export function displayReviewSummary(result) {
-    console.log(chalk.bold('\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'));
-    console.log(chalk.bold('📊 Review Summary'));
-    console.log(chalk.bold('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n'));
-    console.log(`  Files reviewed: ${chalk.cyan(result.summary.filesReviewed)}`);
-    console.log(`  Issues found: ${chalk.yellow(result.summary.issuesFound)}`);
-    if (result.summary.issuesFound > 0) {
-        console.log(`    🔴 Critical: ${chalk.red(result.summary.bySeverity.CRITICAL)}`);
-        console.log(`    🟡 High: ${chalk.yellow(result.summary.bySeverity.HIGH)}`);
-        console.log(`    🟠 Medium: ${chalk.hex('#FFA500')(result.summary.bySeverity.MEDIUM)}`);
-        console.log(`    ⚪ Low: ${chalk.gray(result.summary.bySeverity.LOW)}`);
-    }
-    console.log('');
-}
-/**
- * Check if review has blocking issues (CRITICAL or HIGH)
- */
-export function hasBlockingIssues(result) {
-    return result.summary.bySeverity.CRITICAL > 0 || result.summary.bySeverity.HIGH > 0;
-}
+// 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

@@ -1 +1 @@
-{"version":3,"file":"review-orchestrator.js","sourceRoot":"","sources":["../../src/lib/review-orchestrator.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AACjF,OAAO,EAAE,4BAA4B,EAAuB,MAAM,uBAAuB,CAAC;AAC1F,OAAO,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AACtD,OAAO,EAAE,gBAAgB,EAAoB,MAAM,wBAAwB,CAAC;AAC5E,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AA4BxD;;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;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,MAAiB,EACjB,UAAmB;IAEnB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,oCAAoC,CAAC,CAAC,CAAC;IAE9D,IAAI,CAAC;QACH,sCAAsC;QACtC,MAAM,cAAc,GAAG,iBAAiB,CAAC,MAAM,CAAC,CAAC;QAEjD,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;SACf,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,CAAC,CAAC;IAEpE,IAAI,CAAC;QACH,iBAAiB;QACjB,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAC,CAAC;QAEvD,MAAM,MAAM,GAAkB,EAAE,CAAC;QAEjC,MAAM,gBAAgB,GAAG,mCAAmC,MAAM,CAAC,IAAI;;EAEzE,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;;;;;;;;;;;;;;;;;;;;;;;;yEAwB0B,CAAC;QAEtE,MAAM,UAAU,GAAG,aAAa,CAAC,MAAM,CAAC,CAAC;QACzC,MAAM,aAAa,GAAG,UAAU,CAAC,GAAG,CAAC,KAAK,EAAE,SAAS,EAAE,EAAE;YACvD,MAAM,SAAS,GAAG,UAAU,SAAS,EAAE,CAAC;YACxC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,SAAS,cAAc,CAAC,CAAC,CAAC;YAE5D,IAAI,CAAC;gBACH,MAAM,YAAY,GAAG,iBAAiB,CACpC,SAAS,EACT,gBAAgB,EAChB,MAAM,CAAC,IAAI,EACX,aAAa,CACd,CAAC;gBAEF,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,aAAa,CAAC;oBAC3C,KAAK,EAAE,SAAS;oBAChB,OAAO,EAAE,YAAY;oBACrB,OAAO,EAAE;wBACP,GAAG,MAAM,CAAC,OAAO;wBACjB,KAAK,EAAE,aAAa;qBACrB;iBACF,CAAC,CAAC;gBAEH,MAAM,WAAW,GAAkB,EAAE,CAAC;gBAEtC,IAAI,KAAK,EAAE,MAAM,OAAO,IAAI,QAAQ,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC;oBAChE,IAAI,OAAO,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;wBACjC,MAAM,YAAY,GAAG,iBAAiB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;wBACxD,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;4BAC5B,WAAW,CAAC,IAAI,CAAC,GAAG,YAAY,CAAC,CAAC;4BAClC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,SAAS,WAAW,YAAY,CAAC,MAAM,WAAW,CAAC,CAAC,CAAC;wBACrF,CAAC;oBACH,CAAC;gBACH,CAAC;gBAED,MAAM,QAAQ,CAAC,YAAY,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBACxC,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC;YAC3D,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,KAAK,SAAS,kBAAkB,KAAK,EAAE,CAAC,CAAC,CAAC;gBAClE,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;YACnD,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,MAAM,YAAY,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;QAEtD,MAAM,gBAAgB,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC;QAC/D,MAAM,YAAY,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC;QAE5D,IAAI,gBAAgB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAClC,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,iCAAiC,CAAC,CAAC,CAAC;YAC5D,OAAO,CAAC,KAAK,CACX,KAAK,CAAC,MAAM,CACV,uBAAuB;gBACrB,oDAAoD;gBACpD,+CAA+C;gBAC/C,+CAA+C;gBAC/C,2CAA2C,CAC9C,CACF,CAAC;YACF,MAAM,QAAQ,CAAC,QAAQ,EAAE,CAAC;YAC1B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;QAED,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC5B,OAAO,CAAC,GAAG,CACT,KAAK,CAAC,MAAM,CACV,SAAS,YAAY,CAAC,MAAM,OAAO,YAAY,CAAC,MAAM,mBAAmB,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAC7H,CACF,CAAC;QACJ,CAAC;QAED,YAAY,CAAC,OAAO,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC;QAEhE,MAAM,OAAO,GAAG,gBAAgB,CAAC,aAAa,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAE/D,OAAO;YACL,MAAM;YACN,OAAO;YACP,aAAa,EAAE,aAAa,CAAC,MAAM;SACpC,CAAC;IACJ,CAAC;YAAS,CAAC;QACT,mCAAmC;QACnC,MAAM,QAAQ,CAAC,QAAQ,EAAE,CAAC;IAC5B,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,oBAAoB,CAAC,MAAoB;IACvD,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,yCAAyC,CAAC,CAAC,CAAC;IACnE,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,CAAC;IAC7C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,yCAAyC,CAAC,CAAC,CAAC;IAEnE,OAAO,CAAC,GAAG,CAAC,qBAAqB,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC;IAC7E,OAAO,CAAC,GAAG,CAAC,mBAAmB,KAAK,CAAC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC;IAE3E,IAAI,MAAM,CAAC,OAAO,CAAC,WAAW,GAAG,CAAC,EAAE,CAAC;QACnC,OAAO,CAAC,GAAG,CAAC,oBAAoB,KAAK,CAAC,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;QACjF,OAAO,CAAC,GAAG,CAAC,gBAAgB,KAAK,CAAC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC5E,OAAO,CAAC,GAAG,CAAC,kBAAkB,KAAK,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACxF,OAAO,CAAC,GAAG,CAAC,cAAc,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IACzE,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;AAClB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,iBAAiB,CAAC,MAAoB;IACpD,OAAO,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,QAAQ,GAAG,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,IAAI,GAAG,CAAC,CAAC;AACtF,CAAC"}
+{"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