@probelabs/visor 0.1.37 → 0.1.39

package/dist/index.js

@@ -93923,9 +93923,6 @@ class AIReviewService {
                                 category: 'logic',
                             },
                         ],
-                        suggestions: [
-                            'Configure API keys in your GitHub repository secrets or environment variables',
-                        ],
                         debug: debugInfo,
                     };
                 }
@@ -93962,7 +93959,6 @@ class AIReviewService {
                             category: 'logic',
                         },
                     ],
-                    suggestions: ['Check AI service configuration and API key validity'],
                     debug: debugInfo,
                 };
             }
@@ -94034,9 +94030,6 @@ class AIReviewService {
                             category: 'logic',
                         },
                     ],
-                    suggestions: [
-                        'Check session reuse configuration and ensure parent check completed successfully',
-                    ],
                     debug: debugInfo,
                 };
             }
@@ -94577,23 +94570,18 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
             // Handle plain schema or no schema - no JSON parsing, return response as-is
             if (_schema === 'plain' || !_schema) {
                 log(`📋 ${_schema === 'plain' ? 'Plain' : 'No'} schema detected - returning raw response without JSON parsing`);
-                // Strip common XML wrapper tags that AI might add when given XML-formatted prompts
-                let cleanedResponse = response.trim();
-                // Remove <result>...</result> tags if present
-                cleanedResponse = cleanedResponse
-                    .replace(/^<result>\s*/i, '')
-                    .replace(/\s*<\/result>$/i, '');
-                // Remove <response>...</response> tags if present
-                cleanedResponse = cleanedResponse
-                    .replace(/^<response>\s*/i, '')
-                    .replace(/\s*<\/response>$/i, '');
-                // Remove <answer>...</answer> tags if present
-                cleanedResponse = cleanedResponse
-                    .replace(/^<answer>\s*/i, '')
-                    .replace(/\s*<\/answer>$/i, '');
+                // For plain schema, return the raw response as an issue
                 return {
-                    issues: [],
-                    suggestions: [cleanedResponse.trim()],
+                    issues: [
+                        {
+                            file: 'AI_RESPONSE',
+                            line: 1,
+                            ruleId: 'ai/raw_response',
+                            message: response,
+                            severity: 'info',
+                            category: 'documentation',
+                        },
+                    ],
                     debug: debugInfo,
                 };
             }
@@ -94615,9 +94603,6 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
                         console.error('🚫 AI refused to analyze - returning empty result');
                         return {
                             issues: [],
-                            suggestions: [
-                                'AI was unable to analyze this code. Please check the content or try again.',
-                            ],
                         };
                     }
                     // Try to extract JSON using improved method with proper bracket matching
@@ -94634,24 +94619,33 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
                             // Check if response is plain text and doesn't contain structured data
                             if (!response.includes('{') && !response.includes('}')) {
                                 log('🔧 Plain text response detected, creating structured fallback...');
-                                const isNoChanges = response.toLowerCase().includes('no') &&
-                                    (response.toLowerCase().includes('changes') ||
-                                        response.toLowerCase().includes('code'));
                                 reviewData = {
-                                    issues: [],
-                                    suggestions: isNoChanges
-                                        ? ['No code changes detected in this analysis']
-                                        : [
-                                            `AI response: ${response.substring(0, 200)}${response.length > 200 ? '...' : ''}`,
-                                        ],
+                                    issues: [
+                                        {
+                                            file: 'AI_RESPONSE',
+                                            line: 1,
+                                            ruleId: 'ai/raw_response',
+                                            message: response,
+                                            severity: 'info',
+                                            category: 'documentation',
+                                        },
+                                    ],
                                 };
                             }
                             else {
-                                // Fallback: treat the entire response as a suggestion
+                                // Fallback: treat the entire response as an issue
                                 log('🔧 Creating fallback response from non-JSON content...');
                                 reviewData = {
-                                    issues: [],
-                                    suggestions: [response.trim()],
+                                    issues: [
+                                        {
+                                            file: 'AI_RESPONSE',
+                                            line: 1,
+                                            ruleId: 'ai/raw_response',
+                                            message: response,
+                                            severity: 'info',
+                                            category: 'documentation',
+                                        },
+                                    ],
                                 };
                             }
                         }
@@ -94660,8 +94654,16 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
                         // No JSON found at all - treat as plain text response
                         log('🔧 No JSON found in response, treating as plain text...');
                         reviewData = {
-                            issues: [],
-                            suggestions: [response.trim()],
+                            issues: [
+                                {
+                                    file: 'AI_RESPONSE',
+                                    line: 1,
+                                    ruleId: 'ai/raw_response',
+                                    message: response,
+                                    severity: 'info',
+                                    category: 'documentation',
+                                },
+                            ],
                         };
                     }
                 }
@@ -94671,7 +94673,6 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
             log(`📊 Overall score: ${0}`);
             log(`📋 Total issues: ${reviewData.issues?.length || 0}`);
             log(`🚨 Critical issues: ${reviewData.issues?.filter((i) => i.severity === 'critical').length || 0}`);
-            log(`💡 Suggestions count: ${Array.isArray(reviewData.suggestions) ? reviewData.suggestions.length : 0}`);
             log(`💬 Comments count: ${Array.isArray(reviewData.issues) ? reviewData.issues.length : 0}`);
             // Process issues from the simplified format
             const processedIssues = Array.isArray(reviewData.issues)
@@ -94693,7 +94694,6 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
             // Validate and convert to ReviewSummary format
             const result = {
                 issues: processedIssues,
-                suggestions: Array.isArray(reviewData.suggestions) ? reviewData.suggestions : [],
             };
             // Log issue counts
             const criticalCount = (result.issues || []).filter(i => i.severity === 'critical').length;
@@ -94976,6 +94976,40 @@ class CheckExecutionEngine {
             provider.setWebhookContext(this.webhookContext.webhookData);
         }
     }
+    /**
+     * Filter checks based on tag filter configuration
+     */
+    filterChecksByTags(checks, config, tagFilter) {
+        if (!tagFilter || (!tagFilter.include && !tagFilter.exclude)) {
+            return checks;
+        }
+        const logFn = this.config?.output?.pr_comment ? console.error : console.log;
+        return checks.filter(checkName => {
+            const checkConfig = config?.checks?.[checkName];
+            if (!checkConfig) {
+                // If no config for this check, include it by default
+                return true;
+            }
+            const checkTags = checkConfig.tags || [];
+            // Check exclude tags first (if any exclude tag matches, skip the check)
+            if (tagFilter.exclude && tagFilter.exclude.length > 0) {
+                const hasExcludedTag = tagFilter.exclude.some(tag => checkTags.includes(tag));
+                if (hasExcludedTag) {
+                    logFn(`⏭️ Skipping check '${checkName}' - has excluded tag`);
+                    return false;
+                }
+            }
+            // Check include tags (if specified, at least one must match)
+            if (tagFilter.include && tagFilter.include.length > 0) {
+                const hasIncludedTag = tagFilter.include.some(tag => checkTags.includes(tag));
+                if (!hasIncludedTag) {
+                    logFn(`⏭️ Skipping check '${checkName}' - does not have required tags`);
+                    return false;
+                }
+            }
+            return true;
+        });
+    }
     /**
      * Execute checks on the local repository
      */
@@ -95005,13 +95039,23 @@ class CheckExecutionEngine {
             }
             // Convert to PRInfo format for compatibility with existing reviewer
             const prInfo = this.gitAnalyzer.toPRInfo(repositoryInfo);
+            // Apply tag filtering if specified
+            const filteredChecks = this.filterChecksByTags(options.checks, options.config, options.tagFilter || options.config?.tag_filter);
+            if (filteredChecks.length === 0) {
+                logFn('⚠️ No checks match the tag filter criteria');
+                // Complete GitHub checks with no checks message if they were initialized
+                if (this.checkRunMap) {
+                    await this.completeGitHubChecksWithError('No checks match the tag filter criteria');
+                }
+                return this.createErrorResult(repositoryInfo, 'No checks match the tag filter criteria', startTime, timestamp, options.checks);
+            }
             // Update GitHub checks to in-progress status
             if (this.checkRunMap) {
                 await this.updateGitHubChecksInProgress(options);
             }
             // Execute checks using the existing PRReviewer
-            logFn(`🤖 Executing checks: ${options.checks.join(', ')}`);
-            const reviewSummary = await this.executeReviewChecks(prInfo, options.checks, options.timeout, options.config, options.outputFormat, options.debug, options.maxParallelism, options.failFast);
+            logFn(`🤖 Executing checks: ${filteredChecks.join(', ')}`);
+            const reviewSummary = await this.executeReviewChecks(prInfo, filteredChecks, options.timeout, options.config, options.outputFormat, options.debug, options.maxParallelism, options.failFast);
             // Complete GitHub checks with results
             if (this.checkRunMap) {
                 await this.completeGitHubChecksWithResults(reviewSummary, options);
@@ -95035,7 +95079,7 @@ class CheckExecutionEngine {
                 reviewSummary,
                 executionTime,
                 timestamp,
-                checksExecuted: options.checks,
+                checksExecuted: filteredChecks,
                 debug: debugInfo,
             };
         }
@@ -95237,7 +95281,7 @@ class CheckExecutionEngine {
     /**
      * Execute review checks and return grouped results for new architecture
      */
-    async executeGroupedChecks(prInfo, checks, timeout, config, outputFormat, debug, maxParallelism, failFast) {
+    async executeGroupedChecks(prInfo, checks, timeout, config, outputFormat, debug, maxParallelism, failFast, tagFilter) {
         // Determine where to send log messages based on output format
         const logFn = outputFormat === 'json' || outputFormat === 'sarif' ? console.error : console.log;
         // Only output debug messages if debug mode is enabled
@@ -95250,8 +95294,18 @@ class CheckExecutionEngine {
         if (filteredChecks.length !== checks.length && debug) {
             logFn(`🔧 Debug: Event filtering reduced checks from ${checks.length} to ${filteredChecks.length}: ${JSON.stringify(filteredChecks)}`);
         }
+        // Apply tag filtering if specified
+        const tagFilteredChecks = this.filterChecksByTags(filteredChecks, config, tagFilter || config?.tag_filter);
+        if (tagFilteredChecks.length !== filteredChecks.length && debug) {
+            logFn(`🔧 Debug: Tag filtering reduced checks from ${filteredChecks.length} to ${tagFilteredChecks.length}: ${JSON.stringify(tagFilteredChecks)}`);
+        }
         // Use filtered checks for execution
-        checks = filteredChecks;
+        checks = tagFilteredChecks;
+        // Check if we have any checks left after filtering
+        if (checks.length === 0) {
+            logFn('⚠️ No checks remain after tag filtering');
+            return {};
+        }
         if (!config?.checks) {
             throw new Error('Config with check definitions required for grouped execution');
         }
@@ -95341,12 +95395,9 @@ class CheckExecutionEngine {
                 continue;
             // Extract issues for this check
             const checkIssues = (reviewSummary.issues || []).filter(issue => issue.ruleId?.startsWith(`${checkName}/`));
-            // Extract suggestions for this check
-            const checkSuggestions = (reviewSummary.suggestions || []).filter(suggestion => suggestion.startsWith(`[${checkName}]`));
             // Create a mini ReviewSummary for this check
             const checkSummary = {
                 issues: checkIssues,
-                suggestions: checkSuggestions,
                 debug: reviewSummary.debug,
             };
             // Render content for this check
@@ -95460,12 +95511,7 @@ class CheckExecutionEngine {
         }
         else if (schema === 'plain') {
             // Plain schema - return raw content directly
-            // Strip [checkName] prefixes from suggestions before joining
-            const cleanedSuggestions = (reviewSummary.suggestions || []).map(suggestion => {
-                // Remove [checkName] prefix if present
-                return suggestion.replace(/^\[[^\]]+\]\s*/, '');
-            });
-            return (reviewSummary.issues?.[0]?.message || '') + (cleanedSuggestions.join('\n\n') || '');
+            return reviewSummary.issues?.[0]?.message || '';
         }
         else {
             // Use built-in schema template
@@ -95480,7 +95526,6 @@ class CheckExecutionEngine {
         const templateData = {
             issues: reviewSummary.issues || [],
             checkName: checkName,
-            suggestions: reviewSummary.suggestions || [],
         };
         const rendered = await liquid.parseAndRender(templateContent, templateData);
         return rendered.trim();
@@ -95543,7 +95588,6 @@ class CheckExecutionEngine {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
         // Build dependency graph
@@ -95560,7 +95604,6 @@ class CheckExecutionEngine {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
         // Log execution plan
@@ -95614,7 +95657,6 @@ class CheckExecutionEngine {
                                 error: null,
                                 result: {
                                     issues: [],
-                                    suggestions: [`Check '${checkName}' was skipped - condition not met`],
                                 },
                             };
                         }
@@ -95727,7 +95769,6 @@ class CheckExecutionEngine {
                                 replacement: undefined,
                             },
                         ],
-                        suggestions: [],
                     };
                     results.set(checkName, errorSummary);
                     // Check if we should stop execution due to fail-fast
@@ -95825,7 +95866,6 @@ class CheckExecutionEngine {
                             error: null,
                             result: {
                                 issues: [],
-                                suggestions: [`Check '${checkName}' was skipped - condition not met`],
                             },
                         };
                     }
@@ -95947,7 +95987,6 @@ class CheckExecutionEngine {
      */
     aggregateDependencyAwareResults(results, dependencyGraph, debug, stoppedEarly) {
         const aggregatedIssues = [];
-        const aggregatedSuggestions = [];
         const debugInfo = [];
         // Add execution plan info
         const stats = dependency_resolver_1.DependencyResolver.getExecutionStats(dependencyGraph);
@@ -95981,13 +96020,8 @@ class CheckExecutionEngine {
                 }
                 // Issues are already prefixed and enriched with group/schema info
                 aggregatedIssues.push(...(result.issues || []));
-                // Add suggestions with check name prefix
-                const prefixedSuggestions = (result.suggestions || []).map(suggestion => `[${checkName}] ${suggestion}`);
-                aggregatedSuggestions.push(...prefixedSuggestions);
             }
         }
-        // Add summary information
-        aggregatedSuggestions.unshift(...debugInfo);
         console.error(`🔧 Debug: Aggregated ${aggregatedIssues.length} issues from ${results.size} dependency-aware checks`);
         // Apply issue suppression filtering
         const suppressionEnabled = this.config?.output?.suppressionEnabled !== false;
@@ -96032,25 +96066,20 @@ class CheckExecutionEngine {
         }
         return {
             issues: filteredIssues,
-            suggestions: aggregatedSuggestions,
             debug: aggregatedDebug,
         };
     }
     /**
      * Aggregate results from parallel check execution (legacy method)
      */
-    aggregateParallelResults(results, checkNames, debug, stoppedEarly) {
+    aggregateParallelResults(results, checkNames, debug, _stoppedEarly) {
         const aggregatedIssues = [];
-        const aggregatedSuggestions = [];
         const debugInfo = [];
-        let successfulChecks = 0;
-        let failedChecks = 0;
         results.forEach((result, index) => {
             const checkName = checkNames[index];
             if (result.status === 'fulfilled') {
                 const checkResult = result.value;
                 if (checkResult.error) {
-                    failedChecks++;
                     const log = console.error;
                     log(`🔧 Debug: Check ${checkName} failed: ${checkResult.error}`);
                     debugInfo.push(`❌ Check "${checkName}" failed: ${checkResult.error}`);
@@ -96076,18 +96105,13 @@ class CheckExecutionEngine {
                     });
                 }
                 else if (checkResult.result) {
-                    successfulChecks++;
                     console.error(`🔧 Debug: Check ${checkName} succeeded with ${(checkResult.result.issues || []).length} issues`);
                     debugInfo.push(`✅ Check "${checkName}" completed: ${(checkResult.result.issues || []).length} issues found`);
                     // Issues are already prefixed and enriched with group/schema info
                     aggregatedIssues.push(...(checkResult.result.issues || []));
-                    // Add suggestions with check name prefix
-                    const prefixedSuggestions = (checkResult.result.suggestions || []).map(suggestion => `[${checkName}] ${suggestion}`);
-                    aggregatedSuggestions.push(...prefixedSuggestions);
                 }
             }
             else {
-                failedChecks++;
                 const errorMessage = result.reason instanceof Error ? result.reason.message : String(result.reason);
                 const log = console.error;
                 log(`🔧 Debug: Check ${checkName} promise rejected: ${errorMessage}`);
@@ -96113,11 +96137,6 @@ class CheckExecutionEngine {
                 });
             }
         });
-        // Add summary information
-        debugInfo.unshift(stoppedEarly
-            ? `🛑 Parallel execution stopped early (fail-fast): ${successfulChecks} successful, ${failedChecks} failed`
-            : `🔍 Parallel execution completed: ${successfulChecks} successful, ${failedChecks} failed`);
-        aggregatedSuggestions.unshift(...debugInfo);
         console.error(`🔧 Debug: Aggregated ${aggregatedIssues.length} issues from ${results.length} checks`);
         // Apply issue suppression filtering
         const suppressionEnabled = this.config?.output?.suppressionEnabled !== false;
@@ -96219,7 +96238,6 @@ class CheckExecutionEngine {
         }
         return {
             issues: filteredIssues,
-            suggestions: aggregatedSuggestions,
             debug: aggregatedDebug,
         };
     }
@@ -96330,7 +96348,6 @@ class CheckExecutionEngine {
                         replacement: undefined,
                     },
                 ],
-                suggestions: [`Error: ${errorMessage}`],
             },
             executionTime,
             timestamp,
@@ -96545,7 +96562,7 @@ class CheckExecutionEngine {
             try {
                 const checkIssues = issuesByCheck.get(checkName) || [];
                 // Evaluate failure conditions for this specific check
-                const failureResults = await this.evaluateFailureConditions(checkName, { issues: checkIssues, suggestions: [] }, options.config);
+                const failureResults = await this.evaluateFailureConditions(checkName, { issues: checkIssues }, options.config);
                 await this.githubCheckService.completeCheckRun(options.githubChecks.owner, options.githubChecks.repo, checkRun.id, checkName, failureResults, checkIssues);
                 console.log(`✅ Completed ${checkName} check with ${checkIssues.length} issues`);
             }
@@ -96799,17 +96816,34 @@ async function main() {
         logFn(`🤖 Executing checks: ${checksToRun.join(', ')}`);
         // Create CheckExecutionEngine for running checks
         const engine = new check_execution_engine_1.CheckExecutionEngine();
+        // Build tag filter from CLI options
+        const tagFilter = options.tags || options.excludeTags
+            ? {
+                include: options.tags,
+                exclude: options.excludeTags,
+            }
+            : undefined;
         // Execute checks with proper parameters (cast to PRInfo)
-        const groupedResults = await engine.executeGroupedChecks(repositoryInfo, checksToRun, options.timeout, config, options.output, options.debug || false);
+        const groupedResults = await engine.executeGroupedChecks(repositoryInfo, checksToRun, options.timeout, config, options.output, options.debug || false, options.maxParallelism, options.failFast, tagFilter);
         // Format output based on format type
         let output;
         if (options.output === 'json') {
             output = JSON.stringify(groupedResults, null, 2);
         }
         else if (options.output === 'sarif') {
-            // For SARIF output, we need to convert to SARIF format
-            // For now, output as JSON until proper SARIF formatting is implemented
-            output = JSON.stringify(groupedResults, null, 2);
+            // Build analysis result and format as SARIF
+            const analysisResult = {
+                repositoryInfo,
+                reviewSummary: {
+                    issues: Object.values(groupedResults)
+                        .flatMap((r) => r.map((check) => check.issues || []).flat())
+                        .flat(),
+                },
+                executionTime: 0,
+                timestamp: new Date().toISOString(),
+                checksExecuted: checksToRun,
+            };
+            output = output_formatters_1.OutputFormatters.formatAsSarif(analysisResult);
         }
         else if (options.output === 'markdown') {
             // Create analysis result for markdown formatting
@@ -96819,7 +96853,6 @@ async function main() {
                     issues: Object.values(groupedResults)
                         .flatMap((r) => r.map((check) => check.issues || []).flat())
                         .flat(),
-                    suggestions: [], // Suggestions are now embedded in issues
                 },
                 executionTime: 0,
                 timestamp: new Date().toISOString(),
@@ -96835,7 +96868,6 @@ async function main() {
                     issues: Object.values(groupedResults)
                         .flatMap((r) => r.map((check) => check.issues || []).flat())
                         .flat(),
-                    suggestions: [], // Suggestions are now embedded in issues
                 },
                 executionTime: 0,
                 timestamp: new Date().toISOString(),
@@ -96958,6 +96990,8 @@ class CLI {
             .option('--max-parallelism <count>', 'Maximum number of checks to run in parallel (default: 3)', value => parseInt(value, 10))
             .option('--debug', 'Enable debug mode for detailed output')
             .option('--fail-fast', 'Stop execution on first failure condition')
+            .option('--tags <tags>', 'Include checks with these tags (comma-separated)')
+            .option('--exclude-tags <tags>', 'Exclude checks with these tags (comma-separated)')
             .option('--no-remote-extends', 'Disable loading configurations from remote URLs')
             .addHelpText('after', this.getExamplesText())
             .exitOverride(); // Prevent automatic process.exit for better error handling
@@ -96993,6 +97027,8 @@ class CLI {
                 .option('--max-parallelism <count>', 'Maximum number of checks to run in parallel (default: 3)', value => parseInt(value, 10))
                 .option('--debug', 'Enable debug mode for detailed output')
                 .option('--fail-fast', 'Stop execution on first failure condition')
+                .option('--tags <tags>', 'Include checks with these tags (comma-separated)')
+                .option('--exclude-tags <tags>', 'Exclude checks with these tags (comma-separated)')
                 .option('--allowed-remote-patterns <patterns>', 'Comma-separated list of allowed URL prefixes for remote config extends (e.g., "https://github.com/,https://raw.githubusercontent.com/")')
                 .allowUnknownOption(false)
                 .allowExcessArguments(false) // Don't allow positional arguments
@@ -97015,6 +97051,15 @@ class CLI {
                     .split(',')
                     .map((p) => p.trim());
             }
+            // Parse tag filters
+            let tags;
+            if (options.tags) {
+                tags = options.tags.split(',').map((t) => t.trim());
+            }
+            let excludeTags;
+            if (options.excludeTags) {
+                excludeTags = options.excludeTags.split(',').map((t) => t.trim());
+            }
             return {
                 checks: uniqueChecks,
                 output: options.output,
@@ -97023,6 +97068,8 @@ class CLI {
                 maxParallelism: options.maxParallelism,
                 debug: options.debug,
                 failFast: options.failFast,
+                tags,
+                excludeTags,
                 allowedRemotePatterns,
                 help: options.help,
                 version: options.version,
@@ -97129,7 +97176,10 @@ Examples:
   visor --check all --timeout 300000 --output json           # 5 minute timeout
   visor --check all --max-parallelism 5 --output json        # Run up to 5 checks in parallel
   visor --check all --debug --output markdown                # Enable debug mode
-  visor --check all --fail-fast --output json                # Stop on first failure`;
+  visor --check all --fail-fast --output json                # Stop on first failure
+  visor --tags local,fast --output table                      # Run checks tagged as 'local' or 'fast'
+  visor --exclude-tags slow,experimental --output json        # Skip checks tagged as 'slow' or 'experimental'
+  visor --tags security --exclude-tags slow                   # Run security checks but skip slow ones`;
     }
     /**
      * Display help
@@ -97270,6 +97320,7 @@ class ConfigManager {
         'http_input',
         'http_client',
         'noop',
+        'log',
     ];
     validEventTriggers = [
         'pr_opened',
@@ -97580,6 +97631,10 @@ class ConfigManager {
                 });
             }
         }
+        // Validate tag_filter if present
+        if (config.tag_filter) {
+            this.validateTagFilter(config.tag_filter, errors);
+        }
         if (errors.length > 0) {
             throw new Error(errors[0].message);
         }
@@ -97694,6 +97749,98 @@ class ConfigManager {
                 }
             }
         }
+        // Validate tags configuration
+        if (checkConfig.tags !== undefined) {
+            if (!Array.isArray(checkConfig.tags)) {
+                errors.push({
+                    field: `checks.${checkName}.tags`,
+                    message: `Invalid tags value for "${checkName}": must be an array of strings`,
+                    value: checkConfig.tags,
+                });
+            }
+            else {
+                // Validate each tag
+                const validTagPattern = /^[a-zA-Z0-9][a-zA-Z0-9-_]*$/;
+                checkConfig.tags.forEach((tag, index) => {
+                    if (typeof tag !== 'string') {
+                        errors.push({
+                            field: `checks.${checkName}.tags[${index}]`,
+                            message: `Invalid tag at index ${index} for "${checkName}": must be a string`,
+                            value: tag,
+                        });
+                    }
+                    else if (!validTagPattern.test(tag)) {
+                        errors.push({
+                            field: `checks.${checkName}.tags[${index}]`,
+                            message: `Invalid tag "${tag}" for "${checkName}": tags must be alphanumeric with hyphens or underscores (start with alphanumeric)`,
+                            value: tag,
+                        });
+                    }
+                });
+            }
+        }
+    }
+    /**
+     * Validate tag filter configuration
+     */
+    validateTagFilter(tagFilter, errors) {
+        const validTagPattern = /^[a-zA-Z0-9][a-zA-Z0-9-_]*$/;
+        // Validate include tags
+        if (tagFilter.include !== undefined) {
+            if (!Array.isArray(tagFilter.include)) {
+                errors.push({
+                    field: 'tag_filter.include',
+                    message: 'tag_filter.include must be an array of strings',
+                    value: tagFilter.include,
+                });
+            }
+            else {
+                tagFilter.include.forEach((tag, index) => {
+                    if (typeof tag !== 'string') {
+                        errors.push({
+                            field: `tag_filter.include[${index}]`,
+                            message: `Invalid tag at index ${index}: must be a string`,
+                            value: tag,
+                        });
+                    }
+                    else if (!validTagPattern.test(tag)) {
+                        errors.push({
+                            field: `tag_filter.include[${index}]`,
+                            message: `Invalid tag "${tag}": tags must be alphanumeric with hyphens or underscores`,
+                            value: tag,
+                        });
+                    }
+                });
+            }
+        }
+        // Validate exclude tags
+        if (tagFilter.exclude !== undefined) {
+            if (!Array.isArray(tagFilter.exclude)) {
+                errors.push({
+                    field: 'tag_filter.exclude',
+                    message: 'tag_filter.exclude must be an array of strings',
+                    value: tagFilter.exclude,
+                });
+            }
+            else {
+                tagFilter.exclude.forEach((tag, index) => {
+                    if (typeof tag !== 'string') {
+                        errors.push({
+                            field: `tag_filter.exclude[${index}]`,
+                            message: `Invalid tag at index ${index}: must be a string`,
+                            value: tag,
+                        });
+                    }
+                    else if (!validTagPattern.test(tag)) {
+                        errors.push({
+                            field: `tag_filter.exclude[${index}]`,
+                            message: `Invalid tag "${tag}": tags must be alphanumeric with hyphens or underscores`,
+                            value: tag,
+                        });
+                    }
+                });
+            }
+        }
     }
     /**
      * Validate HTTP server configuration
@@ -98144,7 +98291,6 @@ class FailureConditionEvaluator {
             // Required output property (empty for if conditions)
             output: {
                 issues: [],
-                suggestions: [],
             },
             // Utility metadata
             metadata: {
@@ -98286,7 +98432,7 @@ class FailureConditionEvaluator {
             // Extract context variables
             const output = context.output || {};
             const issues = output.issues || [];
-            const suggestions = output.suggestions || [];
+            const suggestions = [];
             // Backward compatibility: provide metadata for transition period
             // TODO: Remove after all configurations are updated
             const metadata = context.metadata || {
@@ -98397,7 +98543,7 @@ class FailureConditionEvaluator {
      * Build the evaluation context for expressions
      */
     buildEvaluationContext(checkName, checkSchema, checkGroup, reviewSummary, previousOutputs) {
-        const { issues, suggestions, debug } = reviewSummary;
+        const { issues, debug } = reviewSummary;
         const context = {
             output: {
                 issues: (issues || []).map(issue => ({
@@ -98413,7 +98559,6 @@ class FailureConditionEvaluator {
                     suggestion: issue.suggestion,
                     replacement: issue.replacement,
                 })),
-                suggestions,
                 // Include additional schema-specific data from reviewSummary
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 ...reviewSummary, // Pass through any additional fields
@@ -99612,6 +99757,9 @@ async function run() {
             'max-parallelism': (0, core_1.getInput)('max-parallelism') || undefined,
             'ai-provider': (0, core_1.getInput)('ai-provider') || undefined,
             'ai-model': (0, core_1.getInput)('ai-model') || undefined,
+            // Tag filtering inputs
+            tags: (0, core_1.getInput)('tags') || undefined,
+            'exclude-tags': (0, core_1.getInput)('exclude-tags') || undefined,
             // Legacy inputs for backward compatibility
             'visor-config-path': (0, core_1.getInput)('visor-config-path') || undefined,
             'visor-checks': (0, core_1.getInput)('visor-checks') || undefined,
@@ -100553,7 +100701,6 @@ async function completeIndividualChecks(checkService, owner, repo, checkRunMap,
             // Create a ReviewSummary for this check's issues
             const checkReviewSummary = {
                 issues: checkIssues,
-                suggestions: [],
             };
             // Determine which fail_if to use: check-specific overrides global
             const effectiveFailIf = checkFailIf || globalFailIf;
@@ -100601,7 +100748,6 @@ async function completeCombinedCheck(checkService, owner, repo, checkRunMap, gro
         // Create a combined ReviewSummary with all issues
         const combinedReviewSummary = {
             issues: allIssues,
-            suggestions: [],
         };
         // Evaluate global fail_if for the combined check
         const globalFailIf = config?.fail_if;
@@ -100947,22 +101093,6 @@ class OutputFormatters {
         else {
             output += 'No issues found!\n\n';
         }
-        // Suggestions table
-        if ((result.reviewSummary.suggestions || []).length > 0) {
-            const suggestionsTable = new cli_table3_1.default({
-                head: ['#', 'Suggestion'],
-                colWidths: [5, 70],
-                style: {
-                    head: ['cyan', 'bold'],
-                    border: ['grey'],
-                },
-            });
-            output += 'Suggestions\n';
-            (result.reviewSummary.suggestions || []).forEach((suggestion, index) => {
-                suggestionsTable.push([(index + 1).toString(), this.wrapText(suggestion, 65)]);
-            });
-            output += suggestionsTable.toString() + '\n\n';
-        }
         // Files table (if requested)
         if (options.includeFiles && result.repositoryInfo.files.length > 0) {
             const filesTable = new cli_table3_1.default({
@@ -101020,7 +101150,6 @@ class OutputFormatters {
             issues: options.groupByCategory
                 ? this.groupCommentsByCategory((0, reviewer_1.convertIssuesToComments)(issues || []))
                 : issues || [],
-            suggestions: result.reviewSummary.suggestions || [],
             files: options.includeFiles ? result.repositoryInfo.files : undefined,
             debug: result.debug, // Include debug information when available
             failureConditions: result.failureConditions || [], // Include failure condition results
@@ -101317,14 +101446,6 @@ class OutputFormatters {
             output += `## No Issues Found\n\n`;
             output += `Great job! No issues were detected in the analyzed code.\n\n`;
         }
-        // Suggestions
-        if ((result.reviewSummary.suggestions || []).length > 0) {
-            output += `## Recommendations\n\n`;
-            (result.reviewSummary.suggestions || []).forEach((suggestion, index) => {
-                output += `${index + 1}. ${suggestion}\n`;
-            });
-            output += '\n';
-        }
         // Files (if requested)
         if (options.includeFiles && result.repositoryInfo.files.length > 0) {
             output += `## Files Changed\n\n`;
@@ -101683,6 +101804,7 @@ const issue_filter_1 = __nccwpck_require__(36879);
 const liquidjs_1 = __nccwpck_require__(48694);
 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
  */
@@ -101724,6 +101846,39 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
                 !['google', 'anthropic', 'openai', 'mock'].includes(cfg.ai.provider)) {
                 return false;
             }
+            // Validate mcpServers if present
+            if (cfg.ai.mcpServers) {
+                if (!this.validateMcpServers(cfg.ai.mcpServers)) {
+                    return false;
+                }
+            }
+        }
+        // Validate check-level MCP servers if present
+        if (cfg.ai_mcp_servers) {
+            if (!this.validateMcpServers(cfg.ai_mcp_servers)) {
+                return false;
+            }
+        }
+        return true;
+    }
+    /**
+     * Validate MCP servers configuration
+     */
+    validateMcpServers(mcpServers) {
+        if (typeof mcpServers !== 'object' || mcpServers === null) {
+            return false;
+        }
+        for (const serverConfig of Object.values(mcpServers)) {
+            if (!serverConfig || typeof serverConfig !== 'object') {
+                return false;
+            }
+            const config = serverConfig;
+            if (!config.command || typeof config.command !== 'string') {
+                return false;
+            }
+            if (config.args && !Array.isArray(config.args)) {
+                return false;
+            }
         }
         return true;
     }
@@ -101972,9 +102127,8 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
                         styleIssues: result.issues?.filter(i => i.category === 'style') || [],
                         logicIssues: result.issues?.filter(i => i.category === 'logic') || [],
                         documentationIssues: result.issues?.filter(i => i.category === 'documentation') || [],
-                        // All issues and suggestions
+                        // All issues
                         issues: result.issues || [],
-                        suggestions: result.suggestions || [],
                         // Debug information if available
                         debug: result.debug,
                         // Raw data for advanced use
@@ -101990,6 +102144,53 @@ 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 (createSdkMcpServer) {
+                    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) {
@@ -102038,6 +102239,32 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
         if (!customPrompt) {
             throw new Error(`No prompt defined for check. All checks must have prompts defined in .visor.yaml configuration.`);
         }
+        // Setup MCP tools from multiple configuration levels
+        const mcpServers = {};
+        // 1. Start with global MCP servers (from visor config root)
+        const globalConfig = config; // Cast to access potential global config
+        if (globalConfig.ai_mcp_servers) {
+            Object.assign(mcpServers, globalConfig.ai_mcp_servers);
+        }
+        // 2. Add check-level MCP servers (overrides global)
+        if (config.ai_mcp_servers) {
+            Object.assign(mcpServers, config.ai_mcp_servers);
+        }
+        // 3. Add ai.mcpServers (overrides everything)
+        if (config.ai?.mcpServers) {
+            Object.assign(mcpServers, config.ai.mcpServers);
+        }
+        // Setup MCP tools if any servers are configured
+        if (Object.keys(mcpServers).length > 0) {
+            const mcpConfig = { mcpServers };
+            const mcpTools = await this.setupMcpTools(mcpConfig);
+            if (mcpTools.length > 0) {
+                aiConfig.tools = mcpTools;
+                if (aiConfig.debug) {
+                    console.error(`🔧 Debug: AI check configured with ${mcpTools.length} MCP tools from ${Object.keys(mcpServers).length} servers`);
+                }
+            }
+        }
         // Process prompt with Liquid templates and file loading
         const processedPrompt = await this.processPrompt(customPrompt, prInfo, config.eventContext, _dependencyResults);
         // Create AI service with config - environment variables will be used if aiConfig is empty
@@ -102106,8 +102333,10 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
             'ai.model',
             'ai.apiKey',
             'ai.timeout',
+            'ai.mcpServers',
             'ai_model',
             'ai_provider',
+            'ai_mcp_servers',
             'env',
         ];
     }
@@ -102143,6 +102372,7 @@ const http_check_provider_1 = __nccwpck_require__(31115);
 const http_input_provider_1 = __nccwpck_require__(74423);
 const http_client_provider_1 = __nccwpck_require__(36270);
 const noop_check_provider_1 = __nccwpck_require__(53003);
+const log_check_provider_1 = __nccwpck_require__(24903);
 const claude_code_check_provider_1 = __nccwpck_require__(17985);
 /**
  * Registry for managing check providers
@@ -102174,6 +102404,7 @@ class CheckProviderRegistry {
         this.register(new http_input_provider_1.HttpInputProvider());
         this.register(new http_client_provider_1.HttpClientProvider());
         this.register(new noop_check_provider_1.NoopCheckProvider());
+        this.register(new log_check_provider_1.LogCheckProvider());
         // Try to register ClaudeCodeCheckProvider - it may fail if dependencies are missing
         try {
             this.register(new claude_code_check_provider_1.ClaudeCodeCheckProvider());
@@ -102707,7 +102938,6 @@ class ClaudeCodeCheckProvider extends check_provider_interface_1.CheckProvider {
                         documentationIssues: result.issues?.filter(i => i.category === 'documentation') || [],
                         // All issues and suggestions
                         issues: result.issues || [],
-                        suggestions: result.suggestions || [],
                         // Debug information if available
                         debug: result.debug,
                         // Raw data for advanced use
@@ -102733,14 +102963,12 @@ class ClaudeCodeCheckProvider extends check_provider_interface_1.CheckProvider {
             // Convert to ReviewSummary format
             return {
                 issues: parsed.issues || [],
-                suggestions: parsed.suggestions || [],
             };
         }
         catch {
             // If not JSON, treat as plain text comment
             return {
                 issues: [],
-                suggestions: [],
             };
         }
     }
@@ -103121,7 +103349,6 @@ class HttpCheckProvider extends check_provider_interface_1.CheckProvider {
             : [];
         return {
             issues,
-            suggestions: Array.isArray(response.suggestions) ? response.suggestions : [],
         };
     }
     createErrorResult(url, error) {
@@ -103140,7 +103367,6 @@ class HttpCheckProvider extends check_provider_interface_1.CheckProvider {
                     replacement: undefined,
                 },
             ],
-            suggestions: [`Webhook ${url} failed: ${errorMessage}`],
         };
     }
     validateSeverity(severity) {
@@ -103301,16 +103527,14 @@ class HttpClientProvider extends check_provider_interface_1.CheckProvider {
                                 category: 'logic',
                             },
                         ],
-                        suggestions: [],
                     };
                 }
             }
-            // Return the fetched data as suggestions for dependent checks to access
+            // Return the fetched data as a custom field for dependent checks to access
             return {
                 issues: [],
-                suggestions: [
-                    typeof processedData === 'string' ? processedData : JSON.stringify(processedData),
-                ],
+                // Add custom data field that will be passed through to dependent checks
+                data: processedData,
             };
         }
         catch (error) {
@@ -103325,7 +103549,6 @@ class HttpClientProvider extends check_provider_interface_1.CheckProvider {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
     }
@@ -103480,7 +103703,6 @@ class HttpInputProvider extends check_provider_interface_1.CheckProvider {
         if (!webhookData) {
             return {
                 issues: [],
-                suggestions: [`Waiting for webhook data on endpoint: ${endpoint}`],
             };
         }
         // Apply transformation if specified
@@ -103512,15 +103734,15 @@ class HttpInputProvider extends check_provider_interface_1.CheckProvider {
                             category: 'logic',
                         },
                     ],
-                    suggestions: [],
                 };
             }
         }
-        // Return the processed data as suggestions for dependent checks to access
-        // We encode the data as JSON in a suggestion so it can be accessed via outputs
+        // Return the processed data as a custom field for dependent checks to access
+        // This will be available in outputs for dependent checks
         return {
             issues: [],
-            suggestions: [JSON.stringify(processedData)],
+            // Add custom data field that will be passed through
+            data: processedData,
         };
     }
     getWebhookData(endpoint) {
@@ -103555,6 +103777,209 @@ class HttpInputProvider extends check_provider_interface_1.CheckProvider {
 exports.HttpInputProvider = HttpInputProvider;
 
 
+/***/ }),
+
+/***/ 24903:
+/***/ ((__unused_webpack_module, exports, __nccwpck_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.LogCheckProvider = void 0;
+const check_provider_interface_1 = __nccwpck_require__(14131);
+const liquidjs_1 = __nccwpck_require__(48694);
+/**
+ * Check provider that outputs debugging and logging information.
+ * Useful for troubleshooting check workflows and understanding execution flow.
+ */
+class LogCheckProvider extends check_provider_interface_1.CheckProvider {
+    liquid;
+    constructor() {
+        super();
+        this.liquid = new liquidjs_1.Liquid({
+            strictVariables: false,
+            strictFilters: false,
+        });
+    }
+    getName() {
+        return 'log';
+    }
+    getDescription() {
+        return 'Output debugging and logging information for troubleshooting check workflows';
+    }
+    async validateConfig(config) {
+        if (!config || typeof config !== 'object') {
+            return false;
+        }
+        const cfg = config;
+        // Type must be 'log'
+        if (cfg.type !== 'log') {
+            return false;
+        }
+        // Message is required
+        if (!cfg.message || typeof cfg.message !== 'string') {
+            return false;
+        }
+        // Validate log level if provided
+        if (cfg.level && !['debug', 'info', 'warn', 'error'].includes(cfg.level)) {
+            return false;
+        }
+        return true;
+    }
+    async execute(prInfo, config, dependencyResults, _sessionInfo) {
+        const message = config.message;
+        const level = config.level || 'info';
+        const includePrContext = config.include_pr_context !== false;
+        const includeDependencies = config.include_dependencies !== false;
+        const includeMetadata = config.include_metadata !== false;
+        // Prepare template context
+        const templateContext = this.buildTemplateContext(prInfo, dependencyResults, includePrContext, includeDependencies, includeMetadata);
+        // Render the log message template
+        const renderedMessage = await this.liquid.parseAndRender(message, templateContext);
+        // Build the log output
+        const logOutput = this.formatLogOutput(level, renderedMessage, templateContext, includePrContext, includeDependencies, includeMetadata);
+        // Output to console based on log level
+        const logFn = level === 'error' ? console.error : level === 'warn' ? console.warn : console.log;
+        logFn(logOutput);
+        // Return with the log content as custom data for dependent checks
+        return {
+            issues: [],
+            // Add log output as custom field
+            logOutput,
+        };
+    }
+    buildTemplateContext(prInfo, dependencyResults, includePrContext = true, includeDependencies = true, includeMetadata = true) {
+        const context = {};
+        if (includePrContext) {
+            context.pr = {
+                number: prInfo.number,
+                title: prInfo.title,
+                body: prInfo.body,
+                author: prInfo.author,
+                base: prInfo.base,
+                head: prInfo.head,
+                totalAdditions: prInfo.totalAdditions,
+                totalDeletions: prInfo.totalDeletions,
+                files: prInfo.files.map(f => ({
+                    filename: f.filename,
+                    status: f.status,
+                    additions: f.additions,
+                    deletions: f.deletions,
+                    changes: f.changes,
+                })),
+            };
+            // Add convenience data
+            context.filenames = prInfo.files.map(f => f.filename);
+            context.fileCount = prInfo.files.length;
+        }
+        if (includeDependencies && dependencyResults) {
+            const dependencies = {};
+            context.dependencyCount = dependencyResults.size;
+            for (const [checkName, result] of dependencyResults.entries()) {
+                dependencies[checkName] = {
+                    issueCount: result.issues?.length || 0,
+                    suggestionCount: 0,
+                    issues: result.issues || [],
+                };
+            }
+            context.dependencies = dependencies;
+        }
+        if (includeMetadata) {
+            context.metadata = {
+                timestamp: new Date().toISOString(),
+                executionTime: Date.now(),
+                nodeVersion: process.version,
+                platform: process.platform,
+                workingDirectory: process.cwd(),
+            };
+        }
+        return context;
+    }
+    formatLogOutput(level, message, templateContext, includePrContext, includeDependencies, includeMetadata) {
+        const sections = [];
+        // Log level and message
+        const levelEmoji = this.getLevelEmoji(level);
+        sections.push(`${levelEmoji} **${level.toUpperCase()}**: ${message}`);
+        // PR context section
+        if (includePrContext && templateContext.pr) {
+            const pr = templateContext.pr;
+            sections.push('');
+            sections.push('### PR Context');
+            sections.push(`- **PR #${pr.number}**: ${pr.title}`);
+            sections.push(`- **Author**: ${pr.author}`);
+            sections.push(`- **Base**: ${pr.base} → **Head**: ${pr.head}`);
+            sections.push(`- **Changes**: +${pr.totalAdditions} -${pr.totalDeletions}`);
+            sections.push(`- **Files Modified**: ${templateContext.fileCount}`);
+        }
+        // Dependencies section
+        if (includeDependencies && templateContext.dependencies) {
+            const deps = templateContext.dependencies;
+            sections.push('');
+            sections.push('### Dependency Results');
+            if (Object.keys(deps).length === 0) {
+                sections.push('- No dependency results available');
+            }
+            else {
+                for (const [checkName, result] of Object.entries(deps)) {
+                    sections.push(`- **${checkName}**: ${result.issueCount} issues, ${result.suggestionCount} suggestions`);
+                }
+            }
+        }
+        // Metadata section
+        if (includeMetadata && templateContext.metadata) {
+            const meta = templateContext.metadata;
+            sections.push('');
+            sections.push('### Execution Metadata');
+            sections.push(`- **Timestamp**: ${meta.timestamp}`);
+            sections.push(`- **Node Version**: ${meta.nodeVersion}`);
+            sections.push(`- **Platform**: ${meta.platform}`);
+            sections.push(`- **Working Directory**: ${meta.workingDirectory}`);
+        }
+        return sections.join('\n');
+    }
+    getLevelEmoji(level) {
+        switch (level) {
+            case 'debug':
+                return '🐛';
+            case 'info':
+                return 'ℹ️';
+            case 'warn':
+                return '⚠️';
+            case 'error':
+                return '❌';
+            default:
+                return 'ℹ️';
+        }
+    }
+    getSupportedConfigKeys() {
+        return [
+            'type',
+            'message',
+            'level',
+            'include_pr_context',
+            'include_dependencies',
+            'include_metadata',
+            'group',
+            'command',
+            'depends_on',
+            'on',
+            'if',
+        ];
+    }
+    async isAvailable() {
+        // Log provider is always available
+        return true;
+    }
+    getRequirements() {
+        return [
+            'No external dependencies required',
+            'Used for debugging and logging check execution flow',
+        ];
+    }
+}
+exports.LogCheckProvider = LogCheckProvider;
+
+
 /***/ }),
 
 /***/ 53003:
@@ -103597,7 +104022,6 @@ class NoopCheckProvider extends check_provider_interface_1.CheckProvider {
         // It exists purely for command orchestration and dependency triggering
         return {
             issues: [],
-            suggestions: [],
         };
     }
     getSupportedConfigKeys() {
@@ -103719,7 +104143,6 @@ class ToolCheckProvider extends check_provider_interface_1.CheckProvider {
         const filteredIssues = issueFilter.filterIssues(issues, process.cwd());
         return {
             issues: filteredIssues,
-            suggestions: this.generateSuggestions(comments, renderedCommand),
         };
     }
     async executeCommand(command, stdin) {
@@ -103774,17 +104197,6 @@ class ToolCheckProvider extends check_provider_interface_1.CheckProvider {
         }
         return comments;
     }
-    generateSuggestions(comments, command) {
-        const suggestions = [];
-        if (comments.length > 0) {
-            suggestions.push(`Fix ${comments.length} issues found by ${command}`);
-            const errorCount = comments.filter(c => c.severity === 'error').length;
-            if (errorCount > 0) {
-                suggestions.push(`Priority: Fix ${errorCount} errors before merging`);
-            }
-        }
-        return suggestions;
-    }
     getSupportedConfigKeys() {
         return ['type', 'exec', 'command', 'stdin', 'timeout', 'workingDirectory'];
     }
@@ -103855,7 +104267,7 @@ const ai_review_service_1 = __nccwpck_require__(51796);
 // Test utility function - Convert old ReviewSummary to new GroupedCheckResults format
 // This is for backward compatibility with tests only
 function convertReviewSummaryToGroupedResults(reviewSummary, checkName = 'test-check', groupName = 'default') {
-    // Create a simple content string from issues and suggestions
+    // Create a simple content string from issues
     let content = '';
     if (reviewSummary.issues && reviewSummary.issues.length > 0) {
         content += `## Issues Found (${reviewSummary.issues.length})\n\n`;
@@ -103864,12 +104276,6 @@ function convertReviewSummaryToGroupedResults(reviewSummary, checkName = 'test-c
         });
         content += '\n';
     }
-    if (reviewSummary.suggestions && reviewSummary.suggestions.length > 0) {
-        content += `## Suggestions\n\n`;
-        reviewSummary.suggestions.forEach(suggestion => {
-            content += `- ${suggestion}\n`;
-        });
-    }
     if (!content) {
         content = 'No issues found.';
     }