@probelabs/visor 0.1.33 → 0.1.35

package/dist/index.js

@@ -80893,45 +80893,168 @@ class AIReviewService {
      */
     async buildCustomPrompt(prInfo, customInstructions, _schema) {
         const prContext = this.formatPRContext(prInfo);
+        const isIssue = prInfo.isIssue === true;
+        if (isIssue) {
+            // Issue context - no code analysis needed
+            return `<review_request>
+  <instructions>
+${customInstructions}
+  </instructions>
+
+  <context>
+${prContext}
+  </context>
+
+  <rules>
+    <rule>Understand the issue context and requirements from the XML data structure</rule>
+    <rule>Provide helpful, actionable guidance based on the issue details</rule>
+    <rule>Be constructive and supportive in your analysis</rule>
+    <rule>Consider project conventions and patterns when making recommendations</rule>
+    <rule>Suggest practical solutions or next steps that address the specific concern</rule>
+    <rule>Focus on addressing the specific concern raised in the issue</rule>
+    <rule>Reference relevant XML elements like metadata, description, labels, assignees when providing context</rule>
+  </rules>
+</review_request>`;
+        }
+        // PR context - structured XML format
         const analysisType = prInfo.isIncremental ? 'INCREMENTAL' : 'FULL';
-        return `You are a senior code reviewer. 
+        return `<review_request>
+  <analysis_type>${analysisType}</analysis_type>
 
-ANALYSIS TYPE: ${analysisType}
-${analysisType === 'INCREMENTAL'
-            ? '- You are analyzing a NEW COMMIT added to an existing PR. Focus on the <commit_diff> section for changes made in this specific commit.'
-            : '- You are analyzing the COMPLETE PR. Review all changes in the <full_diff> section.'}
+  <analysis_focus>
+    ${analysisType === 'INCREMENTAL'
+            ? 'You are analyzing a NEW COMMIT added to an existing PR. Focus on the changes in the commit_diff section for this specific commit.'
+            : 'You are analyzing the COMPLETE PR. Review all changes in the full_diff section.'}
+  </analysis_focus>
 
-REVIEW INSTRUCTIONS:
+  <instructions>
 ${customInstructions}
+  </instructions>
 
-Analyze the following structured pull request data:
-
+  <context>
 ${prContext}
-
-XML Data Structure Guide:
-- <pull_request>: Root element containing all PR information
-- <metadata>: PR metadata (number, title, author, branches, statistics)
-- <description>: PR description text if provided
-- <full_diff>: Complete unified diff of all changes (for FULL analysis)
-- <commit_diff>: Diff of only the latest commit (for INCREMENTAL analysis)
-- <files_summary>: List of all files changed with statistics
-
-IMPORTANT RULES:
-1. Only analyze code that appears with + (additions) or - (deletions) in the diff
-2. Ignore unchanged code unless it's directly relevant to understanding a change
-3. Line numbers in your response should match the actual file line numbers
-4. Focus on real issues, not nitpicks
-5. Provide actionable, specific feedback
-6. For INCREMENTAL analysis, ONLY review changes in <commit_diff>
-7. For FULL analysis, review all changes in <full_diff>`;
+  </context>
+
+  <rules>
+    <rule>Only analyze code that appears with + (additions) or - (deletions) in the diff sections</rule>
+    <rule>Ignore unchanged code unless directly relevant to understanding a change</rule>
+    <rule>Line numbers in your response should match actual file line numbers from the diff</rule>
+    <rule>Focus on real issues, not nitpicks or cosmetic concerns</rule>
+    <rule>Provide actionable, specific feedback with clear remediation steps</rule>
+    <rule>For INCREMENTAL analysis, ONLY review changes in commit_diff section</rule>
+    <rule>For FULL analysis, review all changes in full_diff section</rule>
+    <rule>Reference specific XML elements like files_summary, metadata when providing context</rule>
+  </rules>
+</review_request>`;
     }
     // REMOVED: Built-in prompts - only use custom prompts from .visor.yaml
     // REMOVED: getFocusInstructions - only use custom prompts from .visor.yaml
     /**
-     * Format PR context for the AI using XML structure
+     * Format PR or Issue context for the AI using XML structure
      */
     formatPRContext(prInfo) {
+        // Check if this is an issue (not a PR)
+        const isIssue = prInfo.isIssue === true;
+        if (isIssue) {
+            // Format as issue context
+            let context = `<issue>
+  <!-- Core issue metadata including identification, status, and timeline information -->
+  <metadata>
+    <number>${prInfo.number}</number>
+    <title>${this.escapeXml(prInfo.title)}</title>
+    <author>${prInfo.author}</author>
+    <state>${prInfo.eventContext?.issue?.state || 'open'}</state>
+    <created_at>${prInfo.eventContext?.issue?.created_at || ''}</created_at>
+    <updated_at>${prInfo.eventContext?.issue?.updated_at || ''}</updated_at>
+    <comments_count>${prInfo.eventContext?.issue?.comments || 0}</comments_count>
+  </metadata>`;
+            // Add issue body/description if available
+            if (prInfo.body) {
+                context += `
+  <!-- Full issue description and body text provided by the issue author -->
+  <description>
+${this.escapeXml(prInfo.body)}
+  </description>`;
+            }
+            // Add labels if available
+            const labels = prInfo.eventContext?.issue?.labels;
+            if (labels && labels.length > 0) {
+                context += `
+  <!-- Applied labels for issue categorization and organization -->
+  <labels>`;
+                labels.forEach((label) => {
+                    context += `
+    <label>${this.escapeXml(label.name || label)}</label>`;
+                });
+                context += `
+  </labels>`;
+            }
+            // Add assignees if available
+            const assignees = prInfo.eventContext?.issue?.assignees;
+            if (assignees && assignees.length > 0) {
+                context += `
+  <!-- Users assigned to work on this issue -->
+  <assignees>`;
+                assignees.forEach((assignee) => {
+                    context += `
+    <assignee>${this.escapeXml(assignee.login || assignee)}</assignee>`;
+                });
+                context += `
+  </assignees>`;
+            }
+            // Add milestone if available
+            const milestone = prInfo.eventContext?.issue?.milestone;
+            if (milestone) {
+                context += `
+  <!-- Associated project milestone information -->
+  <milestone>
+    <title>${this.escapeXml(milestone.title || '')}</title>
+    <state>${milestone.state || 'open'}</state>
+    <due_on>${milestone.due_on || ''}</due_on>
+  </milestone>`;
+            }
+            // Add current/triggering comment if this is a comment event
+            const triggeringComment = prInfo.eventContext?.comment;
+            if (triggeringComment) {
+                context += `
+  <!-- The comment that triggered this analysis -->
+  <triggering_comment>
+    <author>${this.escapeXml(triggeringComment.user?.login || 'unknown')}</author>
+    <created_at>${triggeringComment.created_at || ''}</created_at>
+    <body>${this.escapeXml(triggeringComment.body || '')}</body>
+  </triggering_comment>`;
+            }
+            // Add comment history (excluding the current comment if it exists)
+            const issueComments = prInfo.comments;
+            if (issueComments && issueComments.length > 0) {
+                // Filter out the triggering comment from history if present
+                const historicalComments = triggeringComment
+                    ? issueComments.filter((c) => c.id !== triggeringComment.id)
+                    : issueComments;
+                if (historicalComments.length > 0) {
+                    context += `
+  <!-- Previous comments in chronological order (excluding triggering comment) -->
+  <comment_history>`;
+                    historicalComments.forEach((comment) => {
+                        context += `
+    <comment>
+      <author>${this.escapeXml(comment.author || 'unknown')}</author>
+      <created_at>${comment.createdAt || ''}</created_at>
+      <body>${this.escapeXml(comment.body || '')}</body>
+    </comment>`;
+                    });
+                    context += `
+  </comment_history>`;
+                }
+            }
+            // Close the issue tag
+            context += `
+</issue>`;
+            return context;
+        }
+        // Original PR context formatting
         let context = `<pull_request>
+  <!-- Core pull request metadata including identification, branches, and change statistics -->
   <metadata>
     <number>${prInfo.number}</number>
     <title>${this.escapeXml(prInfo.title)}</title>
@@ -80945,6 +81068,7 @@ IMPORTANT RULES:
         // Add PR description if available
         if (prInfo.body) {
             context += `
+  <!-- Full pull request description provided by the author -->
   <description>
 ${this.escapeXml(prInfo.body)}
   </description>`;
@@ -80952,6 +81076,7 @@ ${this.escapeXml(prInfo.body)}
         // Add full diff if available (for complete PR review)
         if (prInfo.fullDiff) {
             context += `
+  <!-- Complete unified diff showing all changes in the pull request -->
   <full_diff>
 ${this.escapeXml(prInfo.fullDiff)}
   </full_diff>`;
@@ -80960,14 +81085,15 @@ ${this.escapeXml(prInfo.fullDiff)}
         if (prInfo.isIncremental) {
             if (prInfo.commitDiff && prInfo.commitDiff.length > 0) {
                 context += `
+  <!-- Diff of only the latest commit for incremental analysis -->
   <commit_diff>
 ${this.escapeXml(prInfo.commitDiff)}
   </commit_diff>`;
             }
             else {
                 context += `
+  <!-- Commit diff could not be retrieved - falling back to full diff analysis -->
   <commit_diff>
-<!-- Commit diff could not be retrieved - falling back to full diff analysis -->
 ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
   </commit_diff>`;
             }
@@ -80975,10 +81101,11 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
         // Add file summary for context
         if (prInfo.files.length > 0) {
             context += `
+  <!-- Summary of all files changed with statistics -->
   <files_summary>`;
-            prInfo.files.forEach((file, index) => {
+            prInfo.files.forEach(file => {
                 context += `
-    <file index="${index + 1}">
+    <file>
       <filename>${this.escapeXml(file.filename)}</filename>
       <status>${file.status}</status>
       <additions>${file.additions}</additions>
@@ -80988,6 +81115,40 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ''}
             context += `
   </files_summary>`;
         }
+        // Add current/triggering comment if this is a comment event
+        const triggeringComment = prInfo.eventContext?.comment;
+        if (triggeringComment) {
+            context += `
+  <!-- The comment that triggered this analysis -->
+  <triggering_comment>
+    <author>${this.escapeXml(triggeringComment.user?.login || 'unknown')}</author>
+    <created_at>${triggeringComment.created_at || ''}</created_at>
+    <body>${this.escapeXml(triggeringComment.body || '')}</body>
+  </triggering_comment>`;
+        }
+        // Add comment history (excluding the current comment if it exists)
+        const prComments = prInfo.comments;
+        if (prComments && prComments.length > 0) {
+            // Filter out the triggering comment from history if present
+            const historicalComments = triggeringComment
+                ? prComments.filter((c) => c.id !== triggeringComment.id)
+                : prComments;
+            if (historicalComments.length > 0) {
+                context += `
+  <!-- Previous PR comments in chronological order (excluding triggering comment) -->
+  <comment_history>`;
+                historicalComments.forEach((comment) => {
+                    context += `
+    <comment>
+      <author>${this.escapeXml(comment.author || 'unknown')}</author>
+      <created_at>${comment.createdAt || ''}</created_at>
+      <body>${this.escapeXml(comment.body || '')}</body>
+    </comment>`;
+                });
+                context += `
+  </comment_history>`;
+            }
+        }
         context += `
 </pull_request>`;
         return context;
@@ -81563,6 +81724,7 @@ const check_provider_registry_1 = __nccwpck_require__(57140);
 const dependency_resolver_1 = __nccwpck_require__(98645);
 const failure_condition_evaluator_1 = __nccwpck_require__(54429);
 const github_check_service_1 = __nccwpck_require__(21367);
+const issue_filter_1 = __nccwpck_require__(36879);
 /**
  * Filter environment variables to only include safe ones for sandbox evaluation
  */
@@ -81598,8 +81760,11 @@ class CheckExecutionEngine {
     githubCheckService;
     checkRunMap;
     githubContext;
+    workingDirectory;
+    config;
     constructor(workingDirectory) {
-        this.gitAnalyzer = new git_repository_analyzer_1.GitRepositoryAnalyzer(workingDirectory);
+        this.workingDirectory = workingDirectory || process.cwd();
+        this.gitAnalyzer = new git_repository_analyzer_1.GitRepositoryAnalyzer(this.workingDirectory);
         this.providerRegistry = check_provider_registry_1.CheckProviderRegistry.getInstance();
         this.failureEvaluator = new failure_condition_evaluator_1.FailureConditionEvaluator();
         // Create a mock Octokit instance for local analysis
@@ -81743,6 +81908,8 @@ class CheckExecutionEngine {
      * Execute review checks using parallel execution for multiple AI checks
      */
     async executeReviewChecks(prInfo, checks, timeout, config, outputFormat, debug, maxParallelism, failFast) {
+        // Store config for use in filtering
+        this.config = config;
         // Determine where to send log messages based on output format
         const logFn = outputFormat === 'json' || outputFormat === 'sarif' ? console.error : console.log;
         logFn(`🔧 Debug: executeReviewChecks called with checks: ${JSON.stringify(checks)}`);
@@ -82583,6 +82750,10 @@ class CheckExecutionEngine {
         // 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;
+        const issueFilter = new issue_filter_1.IssueFilter(suppressionEnabled);
+        const filteredIssues = issueFilter.filterIssues(aggregatedIssues, this.workingDirectory);
         // Collect debug information when debug mode is enabled
         let aggregatedDebug;
         if (debug) {
@@ -82621,7 +82792,7 @@ class CheckExecutionEngine {
             }
         }
         return {
-            issues: aggregatedIssues,
+            issues: filteredIssues,
             suggestions: aggregatedSuggestions,
             debug: aggregatedDebug,
         };
@@ -82709,6 +82880,10 @@ class CheckExecutionEngine {
             : `🔍 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;
+        const issueFilter = new issue_filter_1.IssueFilter(suppressionEnabled);
+        const filteredIssues = issueFilter.filterIssues(aggregatedIssues, this.workingDirectory);
         // Collect debug information when debug mode is enabled
         let aggregatedDebug;
         if (debug) {
@@ -82804,7 +82979,7 @@ class CheckExecutionEngine {
             }
         }
         return {
-            issues: aggregatedIssues,
+            issues: filteredIssues,
             suggestions: aggregatedSuggestions,
             debug: aggregatedDebug,
         };
@@ -84023,13 +84198,31 @@ class ConfigManager {
      */
     loadBundledDefaultConfig() {
         try {
-            // Find the package root by looking for package.json
-            const packageRoot = this.findPackageRoot();
-            if (!packageRoot) {
-                return null;
+            // Try different paths to find the bundled default config
+            const possiblePaths = [
+                // When running as GitHub Action (bundled in dist/)
+                path.join(__dirname, 'defaults', '.visor.yaml'),
+                // When running from source
+                path.join(__dirname, '..', 'defaults', '.visor.yaml'),
+                // Try via package root
+                this.findPackageRoot() ? path.join(this.findPackageRoot(), 'defaults', '.visor.yaml') : '',
+                // GitHub Action environment variable
+                process.env.GITHUB_ACTION_PATH
+                    ? path.join(process.env.GITHUB_ACTION_PATH, 'defaults', '.visor.yaml')
+                    : '',
+                process.env.GITHUB_ACTION_PATH
+                    ? path.join(process.env.GITHUB_ACTION_PATH, 'dist', 'defaults', '.visor.yaml')
+                    : '',
+            ].filter(p => p); // Remove empty paths
+            let bundledConfigPath;
+            for (const possiblePath of possiblePaths) {
+                if (fs.existsSync(possiblePath)) {
+                    bundledConfigPath = possiblePath;
+                    break;
+                }
             }
-            const bundledConfigPath = path.join(packageRoot, 'defaults', '.visor.yaml');
-            if (fs.existsSync(bundledConfigPath)) {
+            if (bundledConfigPath && fs.existsSync(bundledConfigPath)) {
+                console.log(`📦 Loading bundled default configuration from ${bundledConfigPath}`);
                 const configContent = fs.readFileSync(bundledConfigPath, 'utf8');
                 const parsedConfig = yaml.load(configContent);
                 if (!parsedConfig || typeof parsedConfig !== 'object') {
@@ -85983,7 +86176,6 @@ exports.run = run;
 const rest_1 = __nccwpck_require__(47432);
 const auth_app_1 = __nccwpck_require__(76479);
 const core_1 = __nccwpck_require__(37484);
-const path = __importStar(__nccwpck_require__(16928));
 const commands_1 = __nccwpck_require__(99153);
 const pr_analyzer_1 = __nccwpck_require__(80100);
 const reviewer_1 = __nccwpck_require__(532);
@@ -86147,12 +86339,12 @@ async function run() {
         }
         catch {
             // Fall back to bundled default config
-            try {
-                const defaultConfigPath = path.join(process.env.GITHUB_ACTION_PATH || __dirname, 'defaults', '.visor.yaml');
-                config = await configManager.loadConfig(defaultConfigPath);
-                console.log('📋 Using bundled default configuration from defaults/.visor.yaml');
+            const bundledConfig = configManager.loadBundledDefaultConfig();
+            if (bundledConfig) {
+                config = bundledConfig;
+                console.log('📋 Using bundled default configuration');
             }
-            catch {
+            else {
                 // Ultimate fallback if even defaults/.visor.yaml can't be loaded
                 config = {
                     version: '1.0',
@@ -86349,6 +86541,18 @@ async function handleIssueEvent(octokit, owner, repo, context, inputs, config, c
         isIssue: true, // Flag to indicate this is an issue, not a PR
         eventContext: context.event, // Pass the full event context for templates
     };
+    // Fetch comment history for issues
+    try {
+        console.log(`💬 Fetching comment history for issue #${issue.number}`);
+        const analyzer = new pr_analyzer_1.PRAnalyzer(octokit);
+        const comments = await analyzer.fetchPRComments(owner, repo, issue.number);
+        prInfo.comments = comments;
+        console.log(`✅ Retrieved ${comments.length} comments for issue`);
+    }
+    catch (error) {
+        console.warn(`⚠️ Could not fetch issue comments: ${error instanceof Error ? error.message : 'Unknown error'}`);
+        prInfo.comments = [];
+    }
     // Run the checks using CheckExecutionEngine
     const { CheckExecutionEngine } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(80299)));
     const engine = new CheckExecutionEngine();
@@ -86520,7 +86724,7 @@ async function handleIssueComment(octokit, owner, repo, context, inputs, actionC
                 if (isPullRequest) {
                     // It's a PR comment - fetch the PR diff
                     prInfo = await analyzer.fetchPRDiff(owner, repo, prNumber, undefined, 'issue_comment');
-                    // Add event context for templates
+                    // Add event context for templates and XML generation
                     prInfo.eventContext = context.event;
                 }
                 else {
@@ -86538,8 +86742,19 @@ async function handleIssueComment(octokit, owner, repo, context, inputs, actionC
                         fullDiff: '',
                         eventType: 'issue_comment',
                         isIssue: true, // Flag to indicate this is an issue, not a PR
-                        eventContext: context.event, // Pass the full event context for templates
+                        eventContext: context.event, // Pass the full event context
                     };
+                    // Fetch comment history for the issue
+                    try {
+                        console.log(`💬 Fetching comment history for issue #${issue.number}`);
+                        const comments = await analyzer.fetchPRComments(owner, repo, issue.number);
+                        prInfo.comments = comments;
+                        console.log(`✅ Retrieved ${comments.length} comments for issue`);
+                    }
+                    catch (error) {
+                        console.warn(`⚠️ Could not fetch issue comments: ${error instanceof Error ? error.message : 'Unknown error'}`);
+                        prInfo.comments = [];
+                    }
                 }
                 // Extract common arguments
                 const focus = command.args?.find(arg => arg.startsWith('--focus='))?.split('=')[1];
@@ -86619,7 +86834,7 @@ async function handlePullRequestWithConfig(octokit, owner, repo, inputs, config,
     let prInfo;
     try {
         prInfo = await analyzer.fetchPRDiff(owner, repo, prNumber, undefined, eventType);
-        // Add event context for templates
+        // Add event context for templates and XML generation
         prInfo.eventContext = context.event;
     }
     catch (error) {
@@ -87094,6 +87309,159 @@ if (process.env.NODE_ENV !== 'test' && process.env.JEST_WORKER_ID === undefined)
 }
 
 
+/***/ }),
+
+/***/ 36879:
+/***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.IssueFilter = void 0;
+const fs = __importStar(__nccwpck_require__(79896));
+const path = __importStar(__nccwpck_require__(16928));
+/**
+ * Filter for suppressing Visor issues based on special comments in code
+ */
+class IssueFilter {
+    fileCache = new Map();
+    suppressionEnabled;
+    constructor(suppressionEnabled = true) {
+        this.suppressionEnabled = suppressionEnabled;
+    }
+    /**
+     * Filter out issues that have suppression comments
+     * @param issues Array of issues to filter
+     * @param workingDir Working directory for resolving file paths
+     * @returns Filtered array of issues with suppressed ones removed
+     */
+    filterIssues(issues, workingDir = process.cwd()) {
+        if (!this.suppressionEnabled || !issues || issues.length === 0) {
+            return issues;
+        }
+        const filteredIssues = [];
+        const suppressedCount = {};
+        for (const issue of issues) {
+            if (this.shouldSuppressIssue(issue, workingDir)) {
+                // Track suppressed issues for logging
+                suppressedCount[issue.file] = (suppressedCount[issue.file] || 0) + 1;
+            }
+            else {
+                filteredIssues.push(issue);
+            }
+        }
+        // Log suppression summary if any issues were suppressed
+        const totalSuppressed = Object.values(suppressedCount).reduce((sum, count) => sum + count, 0);
+        if (totalSuppressed > 0) {
+            console.log(`🔇 Suppressed ${totalSuppressed} issue(s) via visor-disable comments:`);
+            for (const [file, count] of Object.entries(suppressedCount)) {
+                console.log(`   - ${file}: ${count} issue(s)`);
+            }
+        }
+        return filteredIssues;
+    }
+    /**
+     * Check if an issue should be suppressed based on comments in the file
+     */
+    shouldSuppressIssue(issue, workingDir) {
+        // Skip system-level issues or issues without file/line info
+        if (!issue.file || issue.file === 'system' || issue.file === 'webhook' || issue.line === 0) {
+            return false;
+        }
+        const lines = this.getFileLines(issue.file, workingDir);
+        if (!lines || lines.length === 0) {
+            return false;
+        }
+        // Check for file-level suppression (visor-disable-file in first 5 lines)
+        const firstFiveLines = lines.slice(0, 5).join('\n').toLowerCase();
+        if (firstFiveLines.includes('visor-disable-file')) {
+            return true;
+        }
+        // Check for line-level suppression (visor-disable within ±2 lines)
+        const lineIndex = issue.line - 1; // Convert to 0-based index
+        const startLine = Math.max(0, lineIndex - 2);
+        const endLine = Math.min(lines.length - 1, lineIndex + 2);
+        for (let i = startLine; i <= endLine; i++) {
+            if (lines[i].toLowerCase().includes('visor-disable')) {
+                return true;
+            }
+        }
+        return false;
+    }
+    /**
+     * Get file lines from cache or read from disk
+     */
+    getFileLines(filePath, workingDir) {
+        // Check cache first
+        if (this.fileCache.has(filePath)) {
+            return this.fileCache.get(filePath);
+        }
+        try {
+            // Resolve the file path
+            const resolvedPath = path.isAbsolute(filePath) ? filePath : path.join(workingDir, filePath);
+            if (!fs.existsSync(resolvedPath)) {
+                // Try without working directory if the file doesn't exist
+                if (fs.existsSync(filePath)) {
+                    const content = fs.readFileSync(filePath, 'utf8');
+                    const lines = content.split('\n');
+                    this.fileCache.set(filePath, lines);
+                    return lines;
+                }
+                return null;
+            }
+            const content = fs.readFileSync(resolvedPath, 'utf8');
+            const lines = content.split('\n');
+            this.fileCache.set(filePath, lines);
+            return lines;
+        }
+        catch {
+            // Silently skip files that can't be read
+            return null;
+        }
+    }
+    /**
+     * Clear the file cache (useful for testing or long-running processes)
+     */
+    clearCache() {
+        this.fileCache.clear();
+    }
+}
+exports.IssueFilter = IssueFilter;
+
+
 /***/ }),
 
 /***/ 25508:
@@ -87840,6 +88208,17 @@ class PRAnalyzer {
             fullDiff: this.generateFullDiff(validFiles),
             eventType,
         };
+        // Fetch comment history for better context
+        try {
+            console.log(`💬 Fetching comment history for PR #${prInfo.number}`);
+            const comments = await this.fetchPRComments(owner, repo, prInfo.number);
+            prInfo.comments = comments;
+            console.log(`✅ Retrieved ${comments.length} comments`);
+        }
+        catch (error) {
+            console.warn(`⚠️ Could not fetch comments: ${error instanceof Error ? error.message : 'Unknown error'}`);
+            prInfo.comments = [];
+        }
         // Add commit diff for incremental analysis
         if (commitSha) {
             console.log(`🔧 Fetching incremental diff for commit: ${commitSha}`);
@@ -87942,6 +88321,7 @@ exports.AICheckProvider = void 0;
 const check_provider_interface_1 = __nccwpck_require__(14131);
 const ai_review_service_1 = __nccwpck_require__(51796);
 const env_resolver_1 = __nccwpck_require__(58749);
+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));
@@ -88311,15 +88691,24 @@ class AICheckProvider extends check_provider_interface_1.CheckProvider {
         console.error(`🔧 Debug: AICheckProvider full config: ${JSON.stringify(config, null, 2)}`);
         try {
             console.error(`🔧 Debug: AICheckProvider passing checkName: ${config.checkName} to service`);
+            let result;
             // Check if we should use session reuse
             if (sessionInfo?.reuseSession && sessionInfo.parentSessionId) {
                 console.error(`🔄 Debug: Using session reuse with parent session: ${sessionInfo.parentSessionId}`);
-                return await service.executeReviewWithSessionReuse(prInfo, processedPrompt, sessionInfo.parentSessionId, schema, config.checkName);
+                result = await service.executeReviewWithSessionReuse(prInfo, processedPrompt, sessionInfo.parentSessionId, schema, config.checkName);
             }
             else {
                 console.error(`🆕 Debug: Creating new AI session for check: ${config.checkName}`);
-                return await service.executeReview(prInfo, processedPrompt, schema, config.checkName, config.sessionId);
+                result = await service.executeReview(prInfo, processedPrompt, schema, config.checkName, config.sessionId);
             }
+            // Apply issue suppression filtering
+            const suppressionEnabled = config.suppressionEnabled !== false;
+            const issueFilter = new issue_filter_1.IssueFilter(suppressionEnabled);
+            const filteredIssues = issueFilter.filterIssues(result.issues || [], process.cwd());
+            return {
+                ...result,
+                issues: filteredIssues,
+            };
         }
         catch (error) {
             const errorMessage = error instanceof Error ? error.message : String(error);
@@ -88607,6 +88996,7 @@ exports.NoopCheckProvider = NoopCheckProvider;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ToolCheckProvider = void 0;
 const check_provider_interface_1 = __nccwpck_require__(14131);
+const issue_filter_1 = __nccwpck_require__(36879);
 const child_process_1 = __nccwpck_require__(35317);
 const liquidjs_1 = __nccwpck_require__(48694);
 /**
@@ -88692,8 +89082,12 @@ class ToolCheckProvider extends check_provider_interface_1.CheckProvider {
             suggestion: undefined,
             replacement: undefined,
         }));
+        // Apply issue suppression filtering
+        const suppressionEnabled = config.suppressionEnabled !== false;
+        const issueFilter = new issue_filter_1.IssueFilter(suppressionEnabled);
+        const filteredIssues = issueFilter.filterIssues(issues, process.cwd());
         return {
-            issues,
+            issues: filteredIssues,
             suggestions: this.generateSuggestions(comments, renderedCommand),
         };
     }
@@ -88788,6 +89182,7 @@ exports.ToolCheckProvider = ToolCheckProvider;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.WebhookCheckProvider = void 0;
 const check_provider_interface_1 = __nccwpck_require__(14131);
+const issue_filter_1 = __nccwpck_require__(36879);
 /**
  * Check provider that sends PR info to a webhook for external analysis
  */
@@ -88848,7 +89243,15 @@ class WebhookCheckProvider extends check_provider_interface_1.CheckProvider {
             // Send webhook request
             const response = await this.sendWebhookRequest(url, method, headers, payload, timeout);
             // Parse webhook response
-            return this.parseWebhookResponse(response, url);
+            const result = this.parseWebhookResponse(response, url);
+            // Apply issue suppression filtering
+            const suppressionEnabled = config.suppressionEnabled !== false;
+            const issueFilter = new issue_filter_1.IssueFilter(suppressionEnabled);
+            const filteredIssues = issueFilter.filterIssues(result.issues || [], process.cwd());
+            return {
+                ...result,
+                issues: filteredIssues,
+            };
         }
         catch (error) {
             return this.createErrorResult(url, error);
@@ -89088,9 +89491,19 @@ class PRReviewer {
                 prNumber,
                 commitSha: options.commitSha,
             });
-            const commentId = options.commentId
-                ? `${options.commentId}-${groupName}`
-                : `visor-review-${groupName}`;
+            // Generate comment ID - use unique ID for "dynamic" group
+            let commentId;
+            if (groupName === 'dynamic') {
+                // Dynamic group creates a new comment each time with timestamp-based ID
+                const timestamp = Date.now();
+                commentId = `visor-dynamic-${timestamp}`;
+            }
+            else {
+                // Regular groups use static IDs that get updated
+                commentId = options.commentId
+                    ? `${options.commentId}-${groupName}`
+                    : `visor-review-${groupName}`;
+            }
             await this.commentManager.updateOrCreateComment(owner, repo, prNumber, comment, {
                 commentId,
                 triggeredBy: options.triggeredBy || 'unknown',
@@ -89536,15 +89949,31 @@ class ConfigLoader {
      * Load bundled default configuration
      */
     async fetchDefaultConfig() {
-        // First try to find the bundled default config
-        let packageRoot = this.findPackageRoot();
-        if (!packageRoot) {
-            // Fallback: try relative to current file
-            packageRoot = path.resolve(__dirname, '..', '..');
-        }
-        const defaultConfigPath = path.join(packageRoot, 'defaults', '.visor.yaml');
-        if (fs.existsSync(defaultConfigPath)) {
-            console.log(`📦 Loading bundled default configuration`);
+        // Try different paths to find the bundled default config
+        const possiblePaths = [
+            // When running as GitHub Action (bundled in dist/)
+            path.join(__dirname, 'defaults', '.visor.yaml'),
+            // When running from source
+            path.join(__dirname, '..', '..', 'defaults', '.visor.yaml'),
+            // Try via package root
+            this.findPackageRoot() ? path.join(this.findPackageRoot(), 'defaults', '.visor.yaml') : '',
+            // GitHub Action environment variable
+            process.env.GITHUB_ACTION_PATH
+                ? path.join(process.env.GITHUB_ACTION_PATH, 'defaults', '.visor.yaml')
+                : '',
+            process.env.GITHUB_ACTION_PATH
+                ? path.join(process.env.GITHUB_ACTION_PATH, 'dist', 'defaults', '.visor.yaml')
+                : '',
+        ].filter(p => p); // Remove empty paths
+        let defaultConfigPath;
+        for (const possiblePath of possiblePaths) {
+            if (fs.existsSync(possiblePath)) {
+                defaultConfigPath = possiblePath;
+                break;
+            }
+        }
+        if (defaultConfigPath && fs.existsSync(defaultConfigPath)) {
+            console.log(`📦 Loading bundled default configuration from ${defaultConfigPath}`);
             const content = fs.readFileSync(defaultConfigPath, 'utf8');
             const config = yaml.load(content);
             if (!config || typeof config !== 'object') {
@@ -110291,6 +110720,103 @@ ${decodedContent}
         };
       }
     }
+    let proactiveFixesApplied = false;
+    const { diagrams: currentDiagrams } = extractMermaidFromMarkdown(fixedResponse);
+    for (let diagramIndex = currentDiagrams.length - 1; diagramIndex >= 0; diagramIndex--) {
+      const diagram = currentDiagrams[diagramIndex];
+      const originalContent = diagram.content;
+      const lines = originalContent.split("\n");
+      let wasFixed = false;
+      const fixedLines = lines.map((line) => {
+        const trimmedLine = line.trim();
+        let modifiedLine = line;
+        if (trimmedLine.match(/\[[^\]]*\]/)) {
+          modifiedLine = modifiedLine.replace(/\[([^\]]*)\]/g, (match, content) => {
+            if (content.trim().startsWith('"') && content.trim().endsWith('"')) {
+              return match;
+            }
+            const needsQuoting = /[()'"<>&]/.test(content) || // Core problematic characters
+            content.includes("e.g.") || content.includes("i.e.") || content.includes("src/") || content.includes("defaults/") || content.includes(".ts") || content.includes(".js") || content.includes(".yaml") || content.includes(".json") || content.includes(".md") || content.includes(".html") || content.includes(".css");
+            if (needsQuoting) {
+              wasFixed = true;
+              const safeContent = content.replace(/"/g, "'");
+              return `["${safeContent}"]`;
+            }
+            return match;
+          });
+        }
+        if (trimmedLine.match(/\{[^{}]*\}/)) {
+          modifiedLine = modifiedLine.replace(/\{([^{}]*)\}/g, (match, content) => {
+            if (content.trim().startsWith('"') && content.trim().endsWith('"')) {
+              return match;
+            }
+            const needsQuoting = /[()'"<>&]/.test(content) || // Core problematic characters
+            content.includes("e.g.") || content.includes("i.e.") || content.includes("src/") || content.includes("defaults/") || content.includes(".ts") || content.includes(".js") || content.includes(".yaml") || content.includes(".json") || content.includes(".md") || content.includes(".html") || content.includes(".css");
+            if (needsQuoting) {
+              wasFixed = true;
+              const safeContent = content.replace(/"/g, "'");
+              return `{"${safeContent}"}`;
+            }
+            return match;
+          });
+        }
+        return modifiedLine;
+      });
+      if (wasFixed) {
+        const fixedContent = fixedLines.join("\n");
+        const attributesStr = diagram.attributes ? ` ${diagram.attributes}` : "";
+        const newCodeBlock = `\`\`\`mermaid${attributesStr}
+${fixedContent}
+\`\`\``;
+        fixedResponse = fixedResponse.slice(0, diagram.startIndex) + newCodeBlock + fixedResponse.slice(diagram.endIndex);
+        fixingResults.push({
+          diagramIndex,
+          wasFixed: true,
+          originalContent,
+          fixedContent,
+          originalError: "Proactive node label quoting",
+          fixMethod: "node_label_quote_wrapping",
+          fixedWithProactiveQuoting: true
+        });
+        proactiveFixesApplied = true;
+        if (debug) {
+          console.log(`[DEBUG] Mermaid validation: Proactively fixed diagram ${diagramIndex + 1} with node label quoting`);
+          console.log(`[DEBUG] Mermaid validation: Applied automatic quoting to special characters`);
+        }
+      }
+    }
+    if (proactiveFixesApplied) {
+      const revalidation = await validateMermaidResponse(fixedResponse);
+      if (revalidation.isValid) {
+        const totalTime2 = Date.now() - startTime;
+        if (debug) {
+          console.log(`[DEBUG] Mermaid validation: All diagrams fixed with proactive quoting in ${totalTime2}ms, no AI needed`);
+          console.log(`[DEBUG] Mermaid validation: Applied ${fixingResults.length} proactive fixes`);
+        }
+        if (tracer) {
+          tracer.recordMermaidValidationEvent("proactive_fix_completed", {
+            "mermaid_validation.success": true,
+            "mermaid_validation.fix_method": "node_label_quote_wrapping",
+            "mermaid_validation.diagrams_fixed": fixingResults.length,
+            "mermaid_validation.duration_ms": totalTime2
+          });
+        }
+        return {
+          ...revalidation,
+          wasFixed: true,
+          originalResponse: response,
+          fixedResponse,
+          fixingResults,
+          performanceMetrics: {
+            totalTimeMs: totalTime2,
+            aiFixingTimeMs: 0,
+            finalValidationTimeMs: 0,
+            diagramsProcessed: fixingResults.length,
+            diagramsFixed: fixingResults.length
+          }
+        };
+      }
+    }
     let subgraphFixesApplied = false;
     const { diagrams: postHtmlDiagrams } = extractMermaidFromMarkdown(fixedResponse);
     const postHtmlValidation = await validateMermaidResponse(fixedResponse);
@@ -110384,16 +110910,21 @@ ${fixedContent}
     const postSubgraphValidation = await validateMermaidResponse(fixedResponse);
     const stillInvalidAfterSubgraph = postSubgraphValidation.diagrams.map((result, index) => ({ ...result, originalIndex: index })).filter((result) => !result.isValid).reverse();
     for (const invalidDiagram of stillInvalidAfterSubgraph) {
-      if (invalidDiagram.error && (invalidDiagram.error.includes("Parentheses in node label") || invalidDiagram.error.includes("Complex expression in diamond node"))) {
+      if (invalidDiagram.error && (invalidDiagram.error.includes("Parentheses in node label") || invalidDiagram.error.includes("Complex expression in diamond node") || invalidDiagram.error.includes("Single quotes in node label"))) {
         const originalContent = invalidDiagram.content;
         const lines = originalContent.split("\n");
         let wasFixed = false;
         const fixedLines = lines.map((line) => {
           const trimmedLine = line.trim();
           let modifiedLine = line;
-          if (trimmedLine.match(/\[[^\]"]*\([^\]"]*\]/)) {
-            modifiedLine = modifiedLine.replace(/\[([^\]"]*\([^\]"]*)\]/g, (match, content) => {
-              if (!content.trim().startsWith('"') || !content.trim().endsWith('"')) {
+          if (trimmedLine.match(/\[[^\]]*\]/)) {
+            modifiedLine = modifiedLine.replace(/\[([^\]]*)\]/g, (match, content) => {
+              if (content.trim().startsWith('"') && content.trim().endsWith('"')) {
+                return match;
+              }
+              const needsQuoting = /[()'"<>&]/.test(content) || // Core problematic characters
+              content.includes("e.g.") || content.includes("i.e.") || content.includes("src/") || content.includes("defaults/") || content.includes(".ts") || content.includes(".js") || content.includes(".yaml") || content.includes(".json");
+              if (needsQuoting) {
                 wasFixed = true;
                 const safeContent = content.replace(/"/g, "'");
                 return `["${safeContent}"]`;
@@ -110401,9 +110932,14 @@ ${fixedContent}
               return match;
             });
           }
-          if (trimmedLine.match(/\{[^{}"]*\([^{}"]*\}/)) {
-            modifiedLine = modifiedLine.replace(/\{([^{}"]*\([^{}"]*)\}/g, (match, content) => {
-              if (!content.trim().startsWith('"') || !content.trim().endsWith('"')) {
+          if (trimmedLine.match(/\{[^{}]*\}/)) {
+            modifiedLine = modifiedLine.replace(/\{([^{}]*)\}/g, (match, content) => {
+              if (content.trim().startsWith('"') && content.trim().endsWith('"')) {
+                return match;
+              }
+              const needsQuoting = /[()'"<>&]/.test(content) || // Core problematic characters
+              content.includes("e.g.") || content.includes("i.e.") || content.includes("src/") || content.includes("defaults/") || content.includes(".ts") || content.includes(".js") || content.includes(".yaml") || content.includes(".json");
+              if (needsQuoting) {
                 wasFixed = true;
                 const safeContent = content.replace(/"/g, "'");
                 return `{"${safeContent}"}`;
@@ -110837,6 +111373,7 @@ var init_ProbeAgent = __esm({
        * @param {boolean} [options.debug] - Enable debug mode
        * @param {boolean} [options.outline] - Enable outline-xml format for search results
        * @param {number} [options.maxResponseTokens] - Maximum tokens for AI responses
+       * @param {boolean} [options.disableMermaidValidation=false] - Disable automatic mermaid diagram validation and fixing
        */
       constructor(options = {}) {
         this.sessionId = options.sessionId || (0, import_crypto5.randomUUID)();
@@ -110848,6 +111385,7 @@ var init_ProbeAgent = __esm({
         this.tracer = options.tracer || null;
         this.outline = !!options.outline;
         this.maxResponseTokens = options.maxResponseTokens || parseInt(process.env.MAX_RESPONSE_TOKENS || "0", 10) || null;
+        this.disableMermaidValidation = !!options.disableMermaidValidation;
         this.allowedFolders = options.path ? [options.path] : [process.cwd()];
         this.clientApiProvider = options.provider || null;
         this.clientApiKey = null;
@@ -111541,56 +112079,60 @@ Convert your previous response content into actual JSON data that follows this s
                 _schemaFormatted: true
               });
               finalResult = cleanSchemaResponse(finalResult);
-              try {
-                if (this.debug) {
-                  console.log(`[DEBUG] Mermaid validation: Starting enhanced mermaid validation...`);
-                }
-                if (this.tracer) {
-                  this.tracer.recordMermaidValidationEvent("schema_processing_started", {
-                    "mermaid_validation.context": "schema_processing",
-                    "mermaid_validation.response_length": finalResult.length
-                  });
-                }
-                const mermaidValidation = await validateAndFixMermaidResponse(finalResult, {
-                  debug: this.debug,
-                  path: this.allowedFolders[0],
-                  provider: this.clientApiProvider,
-                  model: this.model,
-                  tracer: this.tracer
-                });
-                if (mermaidValidation.wasFixed) {
-                  finalResult = mermaidValidation.fixedResponse;
+              if (!this.disableMermaidValidation) {
+                try {
                   if (this.debug) {
-                    console.log(`[DEBUG] Mermaid validation: Diagrams successfully fixed`);
-                    if (mermaidValidation.performanceMetrics) {
-                      const metrics = mermaidValidation.performanceMetrics;
-                      console.log(`[DEBUG] Mermaid validation: Performance - total: ${metrics.totalTimeMs}ms, AI fixing: ${metrics.aiFixingTimeMs}ms`);
-                      console.log(`[DEBUG] Mermaid validation: Results - ${metrics.diagramsFixed}/${metrics.diagramsProcessed} diagrams fixed`);
+                    console.log(`[DEBUG] Mermaid validation: Starting enhanced mermaid validation...`);
+                  }
+                  if (this.tracer) {
+                    this.tracer.recordMermaidValidationEvent("schema_processing_started", {
+                      "mermaid_validation.context": "schema_processing",
+                      "mermaid_validation.response_length": finalResult.length
+                    });
+                  }
+                  const mermaidValidation = await validateAndFixMermaidResponse(finalResult, {
+                    debug: this.debug,
+                    path: this.allowedFolders[0],
+                    provider: this.clientApiProvider,
+                    model: this.model,
+                    tracer: this.tracer
+                  });
+                  if (mermaidValidation.wasFixed) {
+                    finalResult = mermaidValidation.fixedResponse;
+                    if (this.debug) {
+                      console.log(`[DEBUG] Mermaid validation: Diagrams successfully fixed`);
+                      if (mermaidValidation.performanceMetrics) {
+                        const metrics = mermaidValidation.performanceMetrics;
+                        console.log(`[DEBUG] Mermaid validation: Performance - total: ${metrics.totalTimeMs}ms, AI fixing: ${metrics.aiFixingTimeMs}ms`);
+                        console.log(`[DEBUG] Mermaid validation: Results - ${metrics.diagramsFixed}/${metrics.diagramsProcessed} diagrams fixed`);
+                      }
+                      if (mermaidValidation.fixingResults) {
+                        mermaidValidation.fixingResults.forEach((fixResult, index) => {
+                          if (fixResult.wasFixed) {
+                            const method = fixResult.fixedWithHtmlDecoding ? "HTML entity decoding" : "AI correction";
+                            const time = fixResult.aiFixingTimeMs ? ` in ${fixResult.aiFixingTimeMs}ms` : "";
+                            console.log(`[DEBUG] Mermaid validation: Fixed diagram ${fixResult.diagramIndex + 1} with ${method}${time}`);
+                            console.log(`[DEBUG] Mermaid validation: Original error: ${fixResult.originalError}`);
+                          } else {
+                            console.log(`[DEBUG] Mermaid validation: Failed to fix diagram ${fixResult.diagramIndex + 1}: ${fixResult.fixingError}`);
+                          }
+                        });
+                      }
                     }
-                    if (mermaidValidation.fixingResults) {
-                      mermaidValidation.fixingResults.forEach((fixResult, index) => {
-                        if (fixResult.wasFixed) {
-                          const method = fixResult.fixedWithHtmlDecoding ? "HTML entity decoding" : "AI correction";
-                          const time = fixResult.aiFixingTimeMs ? ` in ${fixResult.aiFixingTimeMs}ms` : "";
-                          console.log(`[DEBUG] Mermaid validation: Fixed diagram ${fixResult.diagramIndex + 1} with ${method}${time}`);
-                          console.log(`[DEBUG] Mermaid validation: Original error: ${fixResult.originalError}`);
-                        } else {
-                          console.log(`[DEBUG] Mermaid validation: Failed to fix diagram ${fixResult.diagramIndex + 1}: ${fixResult.fixingError}`);
-                        }
-                      });
+                  } else if (this.debug) {
+                    console.log(`[DEBUG] Mermaid validation: No fixes needed or fixes unsuccessful`);
+                    if (mermaidValidation.diagrams?.length > 0) {
+                      console.log(`[DEBUG] Mermaid validation: Found ${mermaidValidation.diagrams.length} diagrams, all valid: ${mermaidValidation.isValid}`);
                     }
                   }
-                } else if (this.debug) {
-                  console.log(`[DEBUG] Mermaid validation: No fixes needed or fixes unsuccessful`);
-                  if (mermaidValidation.diagrams?.length > 0) {
-                    console.log(`[DEBUG] Mermaid validation: Found ${mermaidValidation.diagrams.length} diagrams, all valid: ${mermaidValidation.isValid}`);
+                } catch (error) {
+                  if (this.debug) {
+                    console.log(`[DEBUG] Mermaid validation: Process failed with error: ${error.message}`);
+                    console.log(`[DEBUG] Mermaid validation: Stack trace: ${error.stack}`);
                   }
                 }
-              } catch (error) {
-                if (this.debug) {
-                  console.log(`[DEBUG] Mermaid validation: Process failed with error: ${error.message}`);
-                  console.log(`[DEBUG] Mermaid validation: Stack trace: ${error.stack}`);
-                }
+              } else if (this.debug) {
+                console.log(`[DEBUG] Mermaid validation: Skipped due to disableMermaidValidation option`);
               }
               if (isJsonSchema(options.schema)) {
                 if (this.debug) {
@@ -111695,26 +112237,30 @@ Convert your previous response content into actual JSON data that follows this s
           } else if (completionAttempted && options.schema) {
             try {
               finalResult = cleanSchemaResponse(finalResult);
-              if (this.debug) {
-                console.log(`[DEBUG] Mermaid validation: Validating attempt_completion result...`);
-              }
-              const mermaidValidation = await validateAndFixMermaidResponse(finalResult, {
-                debug: this.debug,
-                path: this.allowedFolders[0],
-                provider: this.clientApiProvider,
-                model: this.model,
-                tracer: this.tracer
-              });
-              if (mermaidValidation.wasFixed) {
-                finalResult = mermaidValidation.fixedResponse;
+              if (!this.disableMermaidValidation) {
                 if (this.debug) {
-                  console.log(`[DEBUG] Mermaid validation: attempt_completion diagrams fixed`);
-                  if (mermaidValidation.performanceMetrics) {
-                    console.log(`[DEBUG] Mermaid validation: Fixed in ${mermaidValidation.performanceMetrics.totalTimeMs}ms`);
+                  console.log(`[DEBUG] Mermaid validation: Validating attempt_completion result...`);
+                }
+                const mermaidValidation = await validateAndFixMermaidResponse(finalResult, {
+                  debug: this.debug,
+                  path: this.allowedFolders[0],
+                  provider: this.clientApiProvider,
+                  model: this.model,
+                  tracer: this.tracer
+                });
+                if (mermaidValidation.wasFixed) {
+                  finalResult = mermaidValidation.fixedResponse;
+                  if (this.debug) {
+                    console.log(`[DEBUG] Mermaid validation: attempt_completion diagrams fixed`);
+                    if (mermaidValidation.performanceMetrics) {
+                      console.log(`[DEBUG] Mermaid validation: Fixed in ${mermaidValidation.performanceMetrics.totalTimeMs}ms`);
+                    }
                   }
+                } else if (this.debug) {
+                  console.log(`[DEBUG] Mermaid validation: attempt_completion result validation completed (no fixes needed)`);
                 }
               } else if (this.debug) {
-                console.log(`[DEBUG] Mermaid validation: attempt_completion result validation completed (no fixes needed)`);
+                console.log(`[DEBUG] Mermaid validation: Skipped for attempt_completion result due to disableMermaidValidation option`);
               }
               if (isJsonSchema(options.schema)) {
                 if (this.debug) {