@probelabs/visor 0.1.38 → 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;
@@ -95395,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
@@ -95514,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
@@ -95534,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();
@@ -95597,7 +95588,6 @@ class CheckExecutionEngine {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
         // Build dependency graph
@@ -95614,7 +95604,6 @@ class CheckExecutionEngine {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
         // Log execution plan
@@ -95668,7 +95657,6 @@ class CheckExecutionEngine {
                                 error: null,
                                 result: {
                                     issues: [],
-                                    suggestions: [`Check '${checkName}' was skipped - condition not met`],
                                 },
                             };
                         }
@@ -95781,7 +95769,6 @@ class CheckExecutionEngine {
                                 replacement: undefined,
                             },
                         ],
-                        suggestions: [],
                     };
                     results.set(checkName, errorSummary);
                     // Check if we should stop execution due to fail-fast
@@ -95879,7 +95866,6 @@ class CheckExecutionEngine {
                             error: null,
                             result: {
                                 issues: [],
-                                suggestions: [`Check '${checkName}' was skipped - condition not met`],
                             },
                         };
                     }
@@ -96001,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);
@@ -96035,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;
@@ -96086,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}`);
@@ -96130,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}`);
@@ -96167,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;
@@ -96273,7 +96238,6 @@ class CheckExecutionEngine {
         }
         return {
             issues: filteredIssues,
-            suggestions: aggregatedSuggestions,
             debug: aggregatedDebug,
         };
     }
@@ -96384,7 +96348,6 @@ class CheckExecutionEngine {
                         replacement: undefined,
                     },
                 ],
-                suggestions: [`Error: ${errorMessage}`],
             },
             executionTime,
             timestamp,
@@ -96599,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`);
             }
@@ -96875,7 +96838,6 @@ async function main() {
                     issues: Object.values(groupedResults)
                         .flatMap((r) => r.map((check) => check.issues || []).flat())
                         .flat(),
-                    suggestions: [],
                 },
                 executionTime: 0,
                 timestamp: new Date().toISOString(),
@@ -96891,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(),
@@ -96907,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(),
@@ -98331,7 +98291,6 @@ class FailureConditionEvaluator {
             // Required output property (empty for if conditions)
             output: {
                 issues: [],
-                suggestions: [],
             },
             // Utility metadata
             metadata: {
@@ -98473,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 || {
@@ -98584,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 => ({
@@ -98600,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
@@ -100743,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;
@@ -100791,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;
@@ -101137,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({
@@ -101210,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
@@ -101507,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`;
@@ -102196,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
@@ -103008,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
@@ -103034,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: [],
             };
         }
     }
@@ -103422,7 +103349,6 @@ class HttpCheckProvider extends check_provider_interface_1.CheckProvider {
             : [];
         return {
             issues,
-            suggestions: Array.isArray(response.suggestions) ? response.suggestions : [],
         };
     }
     createErrorResult(url, error) {
@@ -103441,7 +103367,6 @@ class HttpCheckProvider extends check_provider_interface_1.CheckProvider {
                     replacement: undefined,
                 },
             ],
-            suggestions: [`Webhook ${url} failed: ${errorMessage}`],
         };
     }
     validateSeverity(severity) {
@@ -103602,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) {
@@ -103626,7 +103549,6 @@ class HttpClientProvider extends check_provider_interface_1.CheckProvider {
                         category: 'logic',
                     },
                 ],
-                suggestions: [],
             };
         }
     }
@@ -103781,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
@@ -103813,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) {
@@ -103917,9 +103838,14 @@ class LogCheckProvider extends check_provider_interface_1.CheckProvider {
         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: [], // Log provider doesn't generate issues
-            suggestions: [logOutput], // Put formatted log output as the primary suggestion
+            issues: [],
+            // Add log output as custom field
+            logOutput,
         };
     }
     buildTemplateContext(prInfo, dependencyResults, includePrContext = true, includeDependencies = true, includeMetadata = true) {
@@ -103952,9 +103878,8 @@ class LogCheckProvider extends check_provider_interface_1.CheckProvider {
             for (const [checkName, result] of dependencyResults.entries()) {
                 dependencies[checkName] = {
                     issueCount: result.issues?.length || 0,
-                    suggestionCount: result.suggestions?.length || 0,
+                    suggestionCount: 0,
                     issues: result.issues || [],
-                    suggestions: result.suggestions || [],
                 };
             }
             context.dependencies = dependencies;
@@ -104097,7 +104022,6 @@ class NoopCheckProvider extends check_provider_interface_1.CheckProvider {
         // It exists purely for command orchestration and dependency triggering
         return {
             issues: [],
-            suggestions: [],
         };
     }
     getSupportedConfigKeys() {
@@ -104219,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) {
@@ -104274,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'];
     }
@@ -104355,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`;
@@ -104364,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.';
     }

package/dist/output-formatters.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/output-formatters.ts"],"names":[],"mappings":"AACA,OAAO,EACL,aAAa,EAMd,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AAExD,MAAM,WAAW,cAAc;IAC7B,cAAc,EAAE,iBAAiB,CAAC;IAClC,aAAa,EAAE,aAAa,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,KAAK,CAAC,EAAE,SAAS,CAAC;IAClB,iBAAiB,CAAC,EAAE,sBAAsB,EAAE,CAAC;CAC9C;AAED,MAAM,WAAW,SAAS;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,cAAc,CAAC,EAAE,MAAM,EAAE,CAAC;IAC1B,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,cAAc,CAAC,EAAE,KAAK,CAAC;QACrB,SAAS,EAAE,MAAM,CAAC;QAClB,QAAQ,EAAE,MAAM,CAAC;QACjB,KAAK,EAAE,MAAM,CAAC;QACd,cAAc,EAAE,MAAM,CAAC;QACvB,OAAO,EAAE,OAAO,CAAC;KAClB,CAAC,CAAC;CACJ;AAED,MAAM,WAAW,sBAAsB;IACrC,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,gBAAgB,CAAC,EAAE,OAAO,CAAC;CAC5B;AAED,qBAAa,gBAAgB;IAC3B;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IA+L1F;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IAqCzF;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,MAAM,EAAE,cAAc,EAAE,QAAQ,GAAE,sBAA2B,GAAG,MAAM;IAmI3F;;OAEG;IACH,MAAM,CAAC,gBAAgB,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IAiO7F,OAAO,CAAC,MAAM,CAAC,uBAAuB;IAqBtC;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;IAc7B;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAsBpC,OAAO,CAAC,MAAM,CAAC,cAAc;IAU7B,OAAO,CAAC,MAAM,CAAC,kBAAkB;IAUjC,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAU/B,OAAO,CAAC,MAAM,CAAC,YAAY;IAK3B,OAAO,CAAC,MAAM,CAAC,QAAQ;CAmBxB"}
+{"version":3,"file":"","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/output-formatters.ts"],"names":[],"mappings":"AACA,OAAO,EACL,aAAa,EAMd,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AAExD,MAAM,WAAW,cAAc;IAC7B,cAAc,EAAE,iBAAiB,CAAC;IAClC,aAAa,EAAE,aAAa,CAAC;IAC7B,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,KAAK,CAAC,EAAE,SAAS,CAAC;IAClB,iBAAiB,CAAC,EAAE,sBAAsB,EAAE,CAAC;CAC9C;AAED,MAAM,WAAW,SAAS;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,cAAc,CAAC,EAAE,MAAM,EAAE,CAAC;IAC1B,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,cAAc,CAAC,EAAE,KAAK,CAAC;QACrB,SAAS,EAAE,MAAM,CAAC;QAClB,QAAQ,EAAE,MAAM,CAAC;QACjB,KAAK,EAAE,MAAM,CAAC;QACd,cAAc,EAAE,MAAM,CAAC;QACvB,OAAO,EAAE,OAAO,CAAC;KAClB,CAAC,CAAC;CACJ;AAED,MAAM,WAAW,sBAAsB;IACrC,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,gBAAgB,CAAC,EAAE,OAAO,CAAC;CAC5B;AAED,qBAAa,gBAAgB;IAC3B;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IA2K1F;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IAoCzF;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,MAAM,EAAE,cAAc,EAAE,QAAQ,GAAE,sBAA2B,GAAG,MAAM;IAmI3F;;OAEG;IACH,MAAM,CAAC,gBAAgB,CAAC,MAAM,EAAE,cAAc,EAAE,OAAO,GAAE,sBAA2B,GAAG,MAAM;IAuN7F,OAAO,CAAC,MAAM,CAAC,uBAAuB;IAqBtC;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;IAc7B;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAsBpC,OAAO,CAAC,MAAM,CAAC,cAAc;IAU7B,OAAO,CAAC,MAAM,CAAC,kBAAkB;IAUjC,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAU/B,OAAO,CAAC,MAAM,CAAC,YAAY;IAK3B,OAAO,CAAC,MAAM,CAAC,QAAQ;CAmBxB"}

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;AAU5C;;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;IAkDvD;;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;IA+JlC;;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;IA2K/B,sBAAsB,IAAI,MAAM,EAAE;IAmB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IASrC,eAAe,IAAI,MAAM,EAAE;CAO5B"}
+{"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;AAU5C;;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;IAkDvD;;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;IA8JlC;;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;IA2K/B,sBAAsB,IAAI,MAAM,EAAE;IAmB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IASrC,eAAe,IAAI,MAAM,EAAE;CAO5B"}

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

@@ -1 +1 @@
-{"version":3,"file":"claude-code-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/claude-code-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;AAc5C;;GAEG;AACH,qBAAa,8BAA+B,SAAQ,KAAK;;CAOxD;AAED;;GAEG;AACH,qBAAa,4BAA6B,SAAQ,KAAK;;CAOtD;AAED;;;GAGG;AACH,qBAAa,uBAAwB,SAAQ,aAAa;IACxD,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,gBAAgB,CAAiC;;IAOzD,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAwDvD;;OAEG;YACW,0BAA0B;IAyCxC;;OAEG;YACW,aAAa;IAqE3B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiB7B;;OAEG;YACW,aAAa;IAmB3B;;OAEG;YACW,UAAU;IAkExB;;OAEG;YACW,kBAAkB;IA0ChC;;OAEG;YACW,oBAAoB;IA+HlC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAmBzB,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,iBAAiB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,EAC9C,WAAW,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,MAAM,CAAC;QAAC,YAAY,CAAC,EAAE,OAAO,CAAA;KAAE,GACjE,OAAO,CAAC,aAAa,CAAC;YAgBX,iBAAiB;IA8H/B,sBAAsB,IAAI,MAAM,EAAE;IAiB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IA0BrC,eAAe,IAAI,MAAM,EAAE;CAQ5B"}
+{"version":3,"file":"claude-code-check-provider.d.ts","sourceRoot":"","sources":["file:///home/runner/work/visor/visor/src/providers/claude-code-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;AAc5C;;GAEG;AACH,qBAAa,8BAA+B,SAAQ,KAAK;;CAOxD;AAED;;GAEG;AACH,qBAAa,4BAA6B,SAAQ,KAAK;;CAOtD;AAED;;;GAGG;AACH,qBAAa,uBAAwB,SAAQ,aAAa;IACxD,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,gBAAgB,CAAiC;;IAOzD,OAAO,IAAI,MAAM;IAIjB,cAAc,IAAI,MAAM;IAIlB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAwDvD;;OAEG;YACW,0BAA0B;IAyCxC;;OAEG;YACW,aAAa;IAqE3B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiB7B;;OAEG;YACW,aAAa;IAmB3B;;OAEG;YACW,UAAU;IAkExB;;OAEG;YACW,kBAAkB;IA0ChC;;OAEG;YACW,oBAAoB;IA8HlC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAiBzB,OAAO,CACX,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,mBAAmB,EAC3B,iBAAiB,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,aAAa,CAAC,EAC9C,WAAW,CAAC,EAAE;QAAE,eAAe,CAAC,EAAE,MAAM,CAAC;QAAC,YAAY,CAAC,EAAE,OAAO,CAAA;KAAE,GACjE,OAAO,CAAC,aAAa,CAAC;YAgBX,iBAAiB;IA8H/B,sBAAsB,IAAI,MAAM,EAAE;IAiB5B,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IA0BrC,eAAe,IAAI,MAAM,EAAE;CAQ5B"}