@probelabs/visor 0.1.79 → 0.1.81

package/dist/index.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-process.env.VISOR_VERSION = '0.1.79';
+process.env.VISOR_VERSION = '0.1.81';
 process.env.PROBE_VERSION = '0.6.0-rc122';
 /******/ (() => { // webpackBootstrap
 /******/ 	var __webpack_modules__ = ({
@@ -102520,21 +102520,30 @@ class CheckExecutionEngine {
                             dependencyResults.set(depId, depResult);
                         }
                     }
-                    // If any direct dependency failed (errors/critical or */error ruleIds), skip this check
+                    // If any direct dependency failed or was skipped, skip this check
                     const directDeps = checkConfig.depends_on || [];
                     const failedDeps = [];
                     for (const depId of directDeps) {
                         const depRes = results.get(depId);
                         if (!depRes)
                             continue;
-                        // Only treat command provider execution/transform failures as dependency-fatal
-                        const hasFatalCommandFailure = (depRes.issues || []).some(issue => {
+                        // Check if dependency was skipped
+                        const wasSkipped = (depRes.issues || []).some(issue => {
                             const id = issue.ruleId || '';
-                            return (id.endsWith('/command/execution_error') ||
+                            return id.endsWith('/__skipped');
+                        });
+                        // Check for fatal failures: command provider execution/transform failures and forEach iteration errors
+                        const hasFatalFailure = (depRes.issues || []).some(issue => {
+                            const id = issue.ruleId || '';
+                            return (id === 'command/execution_error' ||
+                                id.endsWith('/command/execution_error') ||
+                                id === 'command/transform_js_error' ||
                                 id.endsWith('/command/transform_js_error') ||
-                                id.endsWith('/command/transform_error'));
+                                id === 'command/transform_error' ||
+                                id.endsWith('/command/transform_error') ||
+                                id.endsWith('/forEach/iteration_error'));
                         });
-                        if (hasFatalCommandFailure)
+                        if (wasSkipped || hasFatalFailure)
                             failedDeps.push(depId);
                     }
                     if (failedDeps.length > 0) {
@@ -102606,8 +102615,8 @@ class CheckExecutionEngine {
                         if (debug) {
                             log(`🔄 Debug: Check "${checkName}" depends on forEach check "${forEachParentName}", executing ${forEachItems.length} times`);
                         }
-                        // Log forEach processing start
-                        logger_1.logger.info(`  Processing ${forEachItems.length} items...`);
+                        // Log forEach processing start (non-debug)
+                        logger_1.logger.info(`  forEach: processing ${forEachItems.length} items from "${forEachParentName}"...`);
                         const allIssues = [];
                         const allOutputs = [];
                         const aggregatedContents = [];
@@ -102678,8 +102687,19 @@ class CheckExecutionEngine {
                                 parent: forEachParentName,
                             });
                             // Record iteration completion
+                            // Check if this iteration had fatal errors
+                            const hadFatalError = (itemResult.issues || []).some(issue => {
+                                const id = issue.ruleId || '';
+                                return (id === 'command/execution_error' ||
+                                    id.endsWith('/command/execution_error') ||
+                                    id === 'command/transform_js_error' ||
+                                    id.endsWith('/command/transform_js_error') ||
+                                    id === 'command/transform_error' ||
+                                    id.endsWith('/command/transform_error'));
+                            });
                             const iterationDuration = (Date.now() - iterationStart) / 1000;
-                            this.recordIterationComplete(checkName, iterationStart, true, itemResult.issues || [], itemResult.output);
+                            this.recordIterationComplete(checkName, iterationStart, !hadFatalError, // Success if no fatal errors
+                            itemResult.issues || [], itemResult.output);
                             // Log iteration progress
                             logger_1.logger.info(`  ✔ ${itemIndex + 1}/${forEachItems.length} (${iterationDuration.toFixed(1)}s)`);
                             return { index: itemIndex, itemResult };
@@ -102691,7 +102711,22 @@ class CheckExecutionEngine {
                         const forEachResults = await this.executeWithLimitedParallelism(itemTasks, forEachConcurrency, false);
                         for (const result of forEachResults) {
                             if (result.status === 'rejected') {
-                                throw result.reason;
+                                // Instead of throwing, record the failure and continue with other iterations
+                                const error = result.reason;
+                                const errorMessage = error instanceof Error ? error.message : String(error);
+                                // Create an error issue for this failed iteration
+                                allIssues.push({
+                                    ruleId: `${checkName}/forEach/iteration_error`,
+                                    severity: 'error',
+                                    category: 'logic',
+                                    message: `forEach iteration failed: ${errorMessage}`,
+                                    file: '',
+                                    line: 0,
+                                });
+                                if (debug) {
+                                    log(`🔄 Debug: forEach iteration for check "${checkName}" failed: ${errorMessage}`);
+                                }
+                                continue;
                             }
                             // Skip results from skipped items (those that failed if condition)
                             if (result.value.skipped) {
@@ -102749,7 +102784,18 @@ class CheckExecutionEngine {
                         // Execute with retry/routing semantics
                         finalResult = await this.executeWithRouting(checkName, checkConfig, provider, providerConfig, prInfo, dependencyResults, sessionInfo, config, dependencyGraph, debug, results);
                         // Record normal (non-forEach) execution
-                        this.recordIterationComplete(checkName, checkStartTime, true, finalResult.issues || [], finalResult.output);
+                        // Check if this check had fatal errors
+                        const hadFatalError = (finalResult.issues || []).some(issue => {
+                            const id = issue.ruleId || '';
+                            return (id === 'command/execution_error' ||
+                                id.endsWith('/command/execution_error') ||
+                                id === 'command/transform_js_error' ||
+                                id.endsWith('/command/transform_js_error') ||
+                                id === 'command/transform_error' ||
+                                id.endsWith('/command/transform_error'));
+                        });
+                        this.recordIterationComplete(checkName, checkStartTime, !hadFatalError, // Success if no fatal errors
+                        finalResult.issues || [], finalResult.output);
                         if (checkConfig.forEach) {
                             try {
                                 const finalResultWithOutput = finalResult;
@@ -102830,11 +102876,24 @@ class CheckExecutionEngine {
                 const result = levelResults[i];
                 const checkConfig = config.checks[checkName];
                 if (result.status === 'fulfilled' && result.value.result && !result.value.error) {
-                    // Skip storing results for skipped checks (they should not appear in outputs)
+                    // For skipped checks, store a marker so dependent checks can detect the skip
                     if (result.value.skipped) {
                         if (debug) {
-                            log(`🔧 Debug: Not storing result for skipped check "${checkName}"`);
+                            log(`🔧 Debug: Storing skip marker for skipped check "${checkName}"`);
                         }
+                        // Store a special marker result with a skip issue so dependencies can detect it
+                        results.set(checkName, {
+                            issues: [
+                                {
+                                    ruleId: `${checkName}/__skipped`,
+                                    severity: 'info',
+                                    category: 'logic',
+                                    message: 'Check was skipped',
+                                    file: '',
+                                    line: 0,
+                                },
+                            ],
+                        });
                         continue;
                     }
                     const reviewResult = result.value.result;
@@ -102864,6 +102923,8 @@ class CheckExecutionEngine {
                         else {
                             normalizedOutput = [rawOutput];
                         }
+                        // Log forEach items found (non-debug)
+                        logger_1.logger.info(`  Found ${normalizedOutput.length} items for forEach iteration`);
                         try {
                             const preview = JSON.stringify(normalizedOutput);
                             logger_1.logger.debug(`🔧 Debug: Check "${checkName}" forEach output: ${preview?.slice(0, 200) || '(empty)'}`);
@@ -102928,19 +102989,6 @@ class CheckExecutionEngine {
                 }
             }
         }
-        // Build and log final execution summary
-        const executionStatistics = this.buildExecutionStatistics();
-        // Show detailed summary table (only if logFn outputs to console)
-        // Skip when output format is JSON/SARIF to avoid polluting structured output
-        // Check if logFn is console.log (not a no-op or console.error)
-        if (logFn === console.log) {
-            this.logExecutionSummary(executionStatistics);
-        }
-        // Add warning if execution stopped early
-        if (shouldStopExecution) {
-            logger_1.logger.info('');
-            logger_1.logger.warn(`⚠️  Execution stopped early due to fail-fast`);
-        }
         if (debug) {
             if (shouldStopExecution) {
                 log(`🛑 Execution stopped early due to fail-fast after processing ${results.size} of ${checks.length} checks`);
@@ -102949,7 +102997,7 @@ class CheckExecutionEngine {
                 log(`✅ Dependency-aware execution completed successfully for all ${results.size} checks`);
             }
         }
-        // Cleanup sessions after execution
+        // Cleanup sessions BEFORE printing summary to avoid mixing debug logs with table output
         if (sessionIds.size > 0 && debug) {
             log(`🧹 Cleaning up ${sessionIds.size} AI sessions...`);
             for (const [checkName, sessionId] of sessionIds) {
@@ -102962,6 +103010,19 @@ class CheckExecutionEngine {
                 }
             }
         }
+        // Build and log final execution summary
+        const executionStatistics = this.buildExecutionStatistics();
+        // Show detailed summary table (only if logFn outputs to console)
+        // Skip when output format is JSON/SARIF to avoid polluting structured output
+        // Check if logFn is console.log (not a no-op or console.error)
+        if (logFn === console.log) {
+            this.logExecutionSummary(executionStatistics);
+        }
+        // Add warning if execution stopped early
+        if (shouldStopExecution) {
+            logger_1.logger.info('');
+            logger_1.logger.warn(`⚠️  Execution stopped early due to fail-fast`);
+        }
         // Aggregate all results
         return this.aggregateDependencyAwareResults(results, dependencyGraph, debug, shouldStopExecution);
     }
@@ -103166,7 +103227,9 @@ class CheckExecutionEngine {
                     debugInfo.push(`✅ Check "${checkName}" completed: ${(result.issues || []).length} issues found (level ${executionGroup.level})`);
                 }
                 // Issues are already prefixed and enriched with group/schema info
-                aggregatedIssues.push(...(result.issues || []));
+                // Filter out internal __skipped markers
+                const nonInternalIssues = (result.issues || []).filter(issue => !issue.ruleId?.endsWith('/__skipped'));
+                aggregatedIssues.push(...nonInternalIssues);
                 const resultSummary = result;
                 const resultContent = resultSummary.content;
                 if (typeof resultContent === 'string' && resultContent.trim()) {
@@ -104514,6 +104577,13 @@ async function main() {
         const hasRepositoryError = allResults.some((result) => {
             return result.content.includes('Not a git repository');
         });
+        // Cleanup AI sessions before exit to prevent process hanging
+        const { SessionRegistry } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(46059)));
+        const sessionRegistry = SessionRegistry.getInstance();
+        if (sessionRegistry.getActiveSessionIds().length > 0) {
+            logger_1.logger.debug(`🧹 Cleaning up ${sessionRegistry.getActiveSessionIds().length} active AI sessions...`);
+            sessionRegistry.clearAllSessions();
+        }
         if (criticalCount > 0 || hasRepositoryError) {
             process.exit(1);
         }
@@ -108970,6 +109040,15 @@ async function run() {
             (0, core_1.setFailed)(error instanceof Error ? error.message : 'Unknown error');
         }
     }
+    finally {
+        // Cleanup AI sessions before GitHub Action exits to prevent process hanging
+        const { SessionRegistry } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(46059)));
+        const sessionRegistry = SessionRegistry.getInstance();
+        if (sessionRegistry.getActiveSessionIds().length > 0) {
+            console.log(`🧹 Cleaning up ${sessionRegistry.getActiveSessionIds().length} active AI sessions...`);
+            sessionRegistry.clearAllSessions();
+        }
+    }
 }
 function mapGitHubEventToTrigger(eventName, action) {
     if (!eventName)
@@ -111193,7 +111272,6 @@ const issue_filter_1 = __nccwpck_require__(36879);
 const liquid_extensions_1 = __nccwpck_require__(33042);
 const promises_1 = __importDefault(__nccwpck_require__(91943));
 const path_1 = __importDefault(__nccwpck_require__(16928));
-const claude_code_types_1 = __nccwpck_require__(21710);
 /**
  * AI-powered check provider using probe agent
  */
@@ -111520,53 +111598,6 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
             throw new Error(`Failed to render prompt template: ${error instanceof Error ? error.message : 'Unknown error'}`);
         }
     }
-    /**
-     * Setup MCP tools based on AI configuration
-     */
-    async setupMcpTools(aiConfig) {
-        const tools = [];
-        // Setup custom MCP servers if configured
-        if (aiConfig.mcpServers) {
-            try {
-                // Import MCP SDK for custom server creation using safe import
-                const mcpModule = await (0, claude_code_types_1.safeImport)('@modelcontextprotocol/sdk');
-                if (!mcpModule) {
-                    console.warn('@modelcontextprotocol/sdk package not found. MCP servers disabled.');
-                    return tools;
-                }
-                const createSdkMcpServer = mcpModule.createSdkMcpServer || mcpModule.default?.createSdkMcpServer;
-                if (typeof createSdkMcpServer === 'function') {
-                    for (const [serverName, serverConfig] of Object.entries(aiConfig.mcpServers)) {
-                        try {
-                            // Create MCP server instance
-                            const server = await createSdkMcpServer({
-                                name: serverName,
-                                command: serverConfig.command,
-                                args: serverConfig.args || [],
-                                env: { ...process.env, ...serverConfig.env },
-                            });
-                            // Add server tools to available tools
-                            const serverTools = (await server.listTools());
-                            tools.push(...serverTools.map(tool => ({
-                                name: tool.name,
-                                server: serverName,
-                            })));
-                        }
-                        catch (serverError) {
-                            console.warn(`Failed to setup MCP server ${serverName}: ${serverError instanceof Error ? serverError.message : 'Unknown error'}`);
-                        }
-                    }
-                }
-                else {
-                    console.warn('createSdkMcpServer function not found in @modelcontextprotocol/sdk. MCP servers disabled.');
-                }
-            }
-            catch (error) {
-                console.warn(`Failed to import MCP SDK: ${error instanceof Error ? error.message : 'Unknown error'}. MCP servers disabled.`);
-            }
-        }
-        return tools;
-    }
     async execute(prInfo, config, _dependencyResults, sessionInfo) {
         // Apply environment configuration if present
         if (config.env) {
@@ -111630,15 +111661,12 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
         if (config.ai?.mcpServers) {
             Object.assign(mcpServers, config.ai.mcpServers);
         }
-        // Setup MCP tools if any servers are configured
+        // Pass MCP server config directly to AI service
         if (Object.keys(mcpServers).length > 0) {
-            // Pass raw server config to AI service so it can enable MCP in ProbeAgent
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             aiConfig.mcpServers = mcpServers;
-            // Optional: attempt to enumerate tools for debug visibility (not required for functionality)
-            const mcpConfig = { mcpServers };
-            const mcpTools = await this.setupMcpTools(mcpConfig);
             if (aiConfig.debug) {
-                console.error(`🔧 Debug: AI check MCP configured with ${Object.keys(mcpServers).length} servers; discovered ${mcpTools.length} tools`);
+                console.error(`🔧 Debug: AI check MCP configured with ${Object.keys(mcpServers).length} servers`);
             }
         }
         // Process prompt with Liquid templates and file loading
@@ -112052,59 +112080,6 @@ class ClaudeCodeCheckProvider extends check_provider_interface_1.CheckProvider {
             throw new Error(`Failed to initialize Claude Code SDK: ${error instanceof Error ? error.message : 'Unknown error'}`);
         }
     }
-    /**
-     * Setup MCP tools based on configuration
-     */
-    async setupMcpTools(config) {
-        const tools = [];
-        // Add allowed tools
-        if (config.allowedTools) {
-            for (const toolName of config.allowedTools) {
-                tools.push({ name: toolName });
-            }
-        }
-        // Setup custom MCP servers if configured
-        if (config.mcpServers) {
-            try {
-                // Import MCP SDK for custom server creation using safe import
-                const mcpModule = await (0, claude_code_types_1.safeImport)('@modelcontextprotocol/sdk');
-                if (!mcpModule) {
-                    console.warn('@modelcontextprotocol/sdk package not found. MCP servers disabled.');
-                    return tools;
-                }
-                const createSdkMcpServer = mcpModule.createSdkMcpServer || mcpModule.default?.createSdkMcpServer;
-                if (typeof createSdkMcpServer === 'function') {
-                    for (const [serverName, serverConfig] of Object.entries(config.mcpServers)) {
-                        try {
-                            // Create MCP server instance
-                            const server = await createSdkMcpServer({
-                                name: serverName,
-                                command: serverConfig.command,
-                                args: serverConfig.args || [],
-                                env: { ...process.env, ...serverConfig.env },
-                            });
-                            // Add server tools to available tools
-                            const serverTools = (await server.listTools());
-                            tools.push(...serverTools.map(tool => ({
-                                name: tool.name,
-                                server: serverName,
-                            })));
-                        }
-                        catch (serverError) {
-                            console.warn(`Failed to setup MCP server ${serverName}: ${serverError instanceof Error ? serverError.message : 'Unknown error'}`);
-                        }
-                    }
-                }
-                else {
-                    console.warn('createSdkMcpServer function not found in @modelcontextprotocol/sdk. MCP servers disabled.');
-                }
-            }
-            catch (error) {
-                console.warn(`Failed to import MCP SDK: ${error instanceof Error ? error.message : 'Unknown error'}. MCP servers disabled.`);
-            }
-        }
-        return tools;
-    }
     /**
      * Group files by their file extension for template context
      */
@@ -112369,16 +112344,22 @@ class ClaudeCodeCheckProvider extends check_provider_interface_1.CheckProvider {
         try {
             // Initialize Claude Code client
             const client = await this.initializeClaudeCodeClient();
-            // Setup MCP tools
-            const tools = await this.setupMcpTools(claudeCodeConfig);
-            // Prepare query object
+            // Prepare query object with MCP servers passed directly to SDK
             const query = {
                 query: processedPrompt,
-                tools: tools.length > 0 ? tools : undefined,
                 maxTurns: claudeCodeConfig.maxTurns || 5,
                 systemPrompt: claudeCodeConfig.systemPrompt,
                 subagent: claudeCodeConfig.subagent,
             };
+            // Add allowed tools if specified
+            if (claudeCodeConfig.allowedTools && claudeCodeConfig.allowedTools.length > 0) {
+                query.tools = claudeCodeConfig.allowedTools.map(name => ({ name }));
+            }
+            // Pass MCP servers directly to the SDK - let it handle spawning and tool discovery
+            if (claudeCodeConfig.mcpServers && Object.keys(claudeCodeConfig.mcpServers).length > 0) {
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                query.mcpServers = claudeCodeConfig.mcpServers;
+            }
             // Execute query with Claude Code
             let response;
             if (sessionInfo?.reuseSession && sessionInfo.parentSessionId) {
@@ -112412,7 +112393,6 @@ class ClaudeCodeCheckProvider extends check_provider_interface_1.CheckProvider {
                 sessionId: response.session_id,
                 turnCount: response.turn_count,
                 usage: response.usage,
-                toolsUsed: tools.map(t => t.name),
             };
             // Apply issue suppression filtering
             const suppressionEnabled = config.suppressionEnabled !== false;
@@ -112708,8 +112688,22 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
                 output = parsed;
             }
             catch {
-                // If not JSON, keep as string
-                output = rawOutput;
+                // Try to extract JSON from the end of output (for commands with debug logs)
+                const extracted = this.extractJsonFromEnd(rawOutput);
+                if (extracted) {
+                    try {
+                        output = JSON.parse(extracted);
+                        logger_1.logger.debug(`🔧 Debug: Extracted and parsed JSON from end of output (${extracted.length} chars from ${rawOutput.length} total)`);
+                    }
+                    catch {
+                        // Extraction found something but it's not valid JSON
+                        output = rawOutput;
+                    }
+                }
+                else {
+                    // Not JSON, keep as string
+                    output = rawOutput;
+                }
             }
             // Apply transform if specified (Liquid or JavaScript)
             let finalOutput = output;
@@ -112945,6 +112939,7 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
      *  - If it's a JSON string, expose parsed properties via Proxy (e.g., value.key)
      *  - When coerced to string (toString/valueOf/Symbol.toPrimitive), return the original raw string
      *  - If parsing fails or value is not a string, return the value unchanged
+     *  - Attempts to extract JSON from the end of the output if full parse fails
      */
     makeJsonSmart(value) {
         if (typeof value !== 'string') {
@@ -112952,12 +112947,28 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
         }
         const raw = value;
         let parsed;
+        // First try: parse the entire string as JSON
         try {
             parsed = JSON.parse(raw);
         }
         catch {
-            // Not JSON, return original string
-            return raw;
+            // Second try: extract JSON from the end of the output
+            // Look for { or [ at the start of a line and take everything after it
+            const jsonMatch = this.extractJsonFromEnd(raw);
+            if (jsonMatch) {
+                try {
+                    parsed = JSON.parse(jsonMatch);
+                    logger_1.logger.debug(`🔧 Debug: Extracted JSON from end of output (${jsonMatch.length} chars from ${raw.length} total)`);
+                }
+                catch {
+                    // Not valid JSON even after extraction, return original string
+                    return raw;
+                }
+            }
+            else {
+                // Not JSON, return original string
+                return raw;
+            }
         }
         // Use a boxed string so string methods still work via Proxy fallback
         const boxed = new String(raw);
@@ -113010,6 +113021,30 @@ class CommandCheckProvider extends check_provider_interface_1.CheckProvider {
         };
         return new Proxy(boxed, handler);
     }
+    /**
+     * Extract JSON from the end of a string that may contain logs/debug output
+     * Looks for the last occurrence of { or [ and tries to parse from there
+     */
+    extractJsonFromEnd(text) {
+        // Strategy: Find the last line that starts with { or [
+        // Then try to parse from that point to the end
+        const lines = text.split('\n');
+        // Search backwards for a line starting with { or [
+        for (let i = lines.length - 1; i >= 0; i--) {
+            const trimmed = lines[i].trim();
+            if (trimmed.startsWith('{') || trimmed.startsWith('[')) {
+                // Found potential JSON start - take everything from here to the end
+                const candidate = lines.slice(i).join('\n');
+                // Quick validation: does it look like valid JSON structure?
+                const trimmedCandidate = candidate.trim();
+                if ((trimmedCandidate.startsWith('{') && trimmedCandidate.endsWith('}')) ||
+                    (trimmedCandidate.startsWith('[') && trimmedCandidate.endsWith(']'))) {
+                    return trimmedCandidate;
+                }
+            }
+        }
+        return null;
+    }
     /**
      * Recursively apply JSON-smart wrapper to outputs object values
      */
@@ -114404,7 +114439,11 @@ exports.SessionRegistry = void 0;
 class SessionRegistry {
     static instance;
     sessions = new Map();
-    constructor() { }
+    exitHandlerRegistered = false;
+    constructor() {
+        // Register process exit handlers to cleanup sessions
+        this.registerExitHandlers();
+    }
     /**
      * Get the singleton instance of SessionRegistry
      */
@@ -114437,7 +114476,19 @@ class SessionRegistry {
     unregisterSession(sessionId) {
         if (this.sessions.has(sessionId)) {
             console.error(`🗑️  Unregistering AI session: ${sessionId}`);
+            const agent = this.sessions.get(sessionId);
             this.sessions.delete(sessionId);
+            // Cleanup the ProbeAgent instance to prevent hanging processes
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            if (agent && typeof agent.cleanup === 'function') {
+                try {
+                    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                    agent.cleanup();
+                }
+                catch (error) {
+                    console.error(`⚠️  Warning: Failed to cleanup ProbeAgent: ${error}`);
+                }
+            }
         }
     }
     /**
@@ -114445,6 +114496,19 @@ class SessionRegistry {
      */
     clearAllSessions() {
         console.error(`🧹 Clearing all AI sessions (${this.sessions.size} sessions)`);
+        // Cleanup each ProbeAgent instance before clearing
+        for (const [, agent] of this.sessions.entries()) {
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            if (agent && typeof agent.cleanup === 'function') {
+                try {
+                    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                    agent.cleanup();
+                }
+                catch {
+                    // Silent fail during bulk cleanup
+                }
+            }
+        }
         this.sessions.clear();
     }
     /**
@@ -114459,6 +114523,51 @@ class SessionRegistry {
     hasSession(sessionId) {
         return this.sessions.has(sessionId);
     }
+    /**
+     * Register process exit handlers to cleanup sessions on exit
+     */
+    registerExitHandlers() {
+        if (this.exitHandlerRegistered) {
+            return;
+        }
+        const cleanupAndExit = (signal) => {
+            if (this.sessions.size > 0) {
+                console.error(`\n🧹 [${signal}] Cleaning up ${this.sessions.size} active AI sessions...`);
+                this.clearAllSessions();
+            }
+        };
+        // Handle normal process exit
+        process.on('exit', () => {
+            if (this.sessions.size > 0) {
+                console.error(`🧹 [exit] Cleaning up ${this.sessions.size} active AI sessions...`);
+                // Note: async operations won't complete here, but sync cleanup methods will
+                for (const [, agent] of this.sessions.entries()) {
+                    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                    if (agent && typeof agent.cleanup === 'function') {
+                        try {
+                            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                            agent.cleanup();
+                        }
+                        catch {
+                            // Silent fail on exit
+                        }
+                    }
+                }
+                this.sessions.clear();
+            }
+        });
+        // Handle SIGINT (Ctrl+C)
+        process.on('SIGINT', () => {
+            cleanupAndExit('SIGINT');
+            process.exit(0);
+        });
+        // Handle SIGTERM
+        process.on('SIGTERM', () => {
+            cleanupAndExit('SIGTERM');
+            process.exit(0);
+        });
+        this.exitHandlerRegistered = true;
+    }
 }
 exports.SessionRegistry = SessionRegistry;
 
@@ -228692,7 +228801,7 @@ module.exports = /*#__PURE__*/JSON.parse('{"application/1d-interleaved-parityfec
 /***/ ((module) => {
 
 "use strict";
-module.exports = {"rE":"0.1.79"};
+module.exports = {"rE":"0.1.81"};
 
 /***/ })
 

package/dist/providers/ai-check-provider.d.ts

@@ -35,10 +35,6 @@ export declare class AICheckProvider extends CheckProvider {
      * Render Liquid template in prompt with comprehensive event context
      */
     private renderPromptTemplate;
-    /**
-     * Setup MCP tools based on AI configuration
-     */
-    private setupMcpTools;
     execute(prInfo: PRInfo, config: CheckProviderConfig, _dependencyResults?: Map<string, ReviewSummary>, sessionInfo?: {
         parentSessionId?: string;
         reuseSession?: boolean;

package/dist/providers/ai-check-provider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ai-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/ai-check-provider.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAChF,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAW5C;;GAEG;AACH,qBAAa,eAAgB,SAAQ,aAAa;IAChD,OAAO,CAAC,eAAe,CAAkB;IACzC,OAAO,CAAC,YAAY,CAAS;;IAQ7B,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAoDvD;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAqB1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiB7B;;OAEG;YACW,aAAa;IAmB3B;;OAEG;YACW,UAAU;IAsExB;;OAEG;YACW,kBAAkB;IA0ChC;;OAEG;YACW,oBAAoB;IA0IlC;;OAEG;YACW,aAAa;IA8DrB,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,kBAAkB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,EAC/C,WAAW,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,MAAM,CAAC;QAAC,YAAY,CAAC,EAAE,OAAO,CAAA;KAAE,GACjE,OAAO,CAAC,aAAa,CAAC;YAiBX,iBAAiB;IAuL/B,sBAAsB,IAAI,MAAM,EAAE;IAmB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IAYrC,eAAe,IAAI,MAAM,EAAE;CAQ5B"}
+{"version":3,"file":"ai-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/ai-check-provider.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAChF,OAAO,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACxC,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAS5C;;GAEG;AACH,qBAAa,eAAgB,SAAQ,aAAa;IAChD,OAAO,CAAC,eAAe,CAAkB;IACzC,OAAO,CAAC,YAAY,CAAS;;IAQ7B,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAoDvD;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAqB1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiB7B;;OAEG;YACW,aAAa;IAmB3B;;OAEG;YACW,UAAU;IAsExB;;OAEG;YACW,kBAAkB;IA0ChC;;OAEG;YACW,oBAAoB;IA0I5B,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,kBAAkB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,EAC/C,WAAW,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,MAAM,CAAC;QAAC,YAAY,CAAC,EAAE,OAAO,CAAA;KAAE,GACjE,OAAO,CAAC,aAAa,CAAC;YAiBX,iBAAiB;IAoL/B,sBAAsB,IAAI,MAAM,EAAE;IAmB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IAYrC,eAAe,IAAI,MAAM,EAAE;CAQ5B"}

package/dist/providers/claude-code-check-provider.d.ts

@@ -28,10 +28,6 @@ export declare class ClaudeCodeCheckProvider extends CheckProvider {
      * Initialize Claude Code SDK client
      */
     private initializeClaudeCodeClient;
-    /**
-     * Setup MCP tools based on configuration
-     */
-    private setupMcpTools;
     /**
      * Group files by their file extension for template context
      */