@probelabs/visor 0.1.93 → 0.1.95

package/dist/sdk/{chunk-Z47UECAT.mjs → chunk-Q4S5A5TO.mjs}

@@ -217,6 +217,30 @@ var init_session_registry = __esm({
 // src/github-comments.ts
 init_logger();
 import { v4 as uuidv4 } from "uuid";
+
+// src/footer.ts
+function generateFooter(options = {}) {
+  const { includeMetadata, includeSeparator = true } = options;
+  const parts = [];
+  if (includeSeparator) {
+    parts.push("---");
+    parts.push("");
+  }
+  parts.push(
+    "*Powered by [Visor](https://probelabs.com/visor) from [Probelabs](https://probelabs.com)*"
+  );
+  if (includeMetadata) {
+    const { lastUpdated, triggeredBy, commitSha } = includeMetadata;
+    const commitInfo = commitSha ? ` | Commit: ${commitSha.substring(0, 7)}` : "";
+    parts.push("");
+    parts.push(`*Last updated: ${lastUpdated} | Triggered by: ${triggeredBy}${commitInfo}*`);
+  }
+  parts.push("");
+  parts.push("\u{1F4A1} **TIP:** You can chat with Visor using `/visor ask <your question>`");
+  return parts.join("\n");
+}
+
+// src/github-comments.ts
 var CommentManager = class {
   octokit;
   retryConfig;
@@ -318,15 +342,17 @@ var CommentManager = class {
    */
   formatCommentWithMetadata(content, metadata) {
     const { commentId, lastUpdated, triggeredBy, commitSha } = metadata;
-    const commitInfo = commitSha ? ` | Commit: ${commitSha.substring(0, 7)}` : "";
+    const footer = generateFooter({
+      includeMetadata: {
+        lastUpdated,
+        triggeredBy,
+        commitSha
+      }
+    });
     return `<!-- visor-comment-id:${commentId} -->
 ${content}
 
----
-
-*Powered by [Visor](https://probelabs.com/visor) from [Probelabs](https://probelabs.com)*
-
-*Last updated: ${lastUpdated} | Triggered by: ${triggeredBy}${commitInfo}*
+${footer}
 <!-- /visor-comment-id:${commentId} -->`;
   }
   /**
@@ -508,6 +534,64 @@ init_session_registry();
 init_logger();
 init_tracer_init();
 import { ProbeAgent } from "@probelabs/probe";
+
+// src/utils/diff-processor.ts
+import { extract } from "@probelabs/probe";
+import * as path from "path";
+async function processDiffWithOutline(diffContent) {
+  if (!diffContent || diffContent.trim().length === 0) {
+    return diffContent;
+  }
+  try {
+    const originalProbePath = process.env.PROBE_PATH;
+    const fs5 = __require("fs");
+    const possiblePaths = [
+      // Relative to current working directory (most common in production)
+      path.join(process.cwd(), "node_modules/@probelabs/probe/bin/probe-binary"),
+      // Relative to __dirname (for unbundled development)
+      path.join(__dirname, "../..", "node_modules/@probelabs/probe/bin/probe-binary"),
+      // Relative to dist directory (for bundled CLI)
+      path.join(__dirname, "node_modules/@probelabs/probe/bin/probe-binary")
+    ];
+    let probeBinaryPath;
+    for (const candidatePath of possiblePaths) {
+      if (fs5.existsSync(candidatePath)) {
+        probeBinaryPath = candidatePath;
+        break;
+      }
+    }
+    if (!probeBinaryPath) {
+      if (process.env.DEBUG === "1" || process.env.VERBOSE === "1") {
+        console.error("Probe binary not found. Tried:", possiblePaths);
+      }
+      return diffContent;
+    }
+    process.env.PROBE_PATH = probeBinaryPath;
+    const extractPromise = extract({
+      content: diffContent,
+      format: "outline-diff",
+      allowTests: true
+      // Allow test files and test code blocks in extraction results
+    });
+    const timeoutPromise = new Promise((_, reject) => {
+      setTimeout(() => reject(new Error("Extract timeout after 30s")), 3e4);
+    });
+    const result = await Promise.race([extractPromise, timeoutPromise]);
+    if (originalProbePath !== void 0) {
+      process.env.PROBE_PATH = originalProbePath;
+    } else {
+      delete process.env.PROBE_PATH;
+    }
+    return typeof result === "string" ? result : JSON.stringify(result);
+  } catch (error) {
+    if (process.env.DEBUG === "1" || process.env.VERBOSE === "1") {
+      console.error("Failed to process diff with outline-diff format:", error);
+    }
+    return diffContent;
+  }
+}
+
+// src/ai-review-service.ts
 function log(...args) {
   logger.debug(args.join(" "));
 }
@@ -549,7 +633,7 @@ var AIReviewService = class {
   /**
    * Execute AI review using probe agent
    */
-  async executeReview(prInfo, customPrompt, schema, _checkName, sessionId) {
+  async executeReview(prInfo, customPrompt, schema, checkName, sessionId) {
     const startTime = Date.now();
     const timestamp = (/* @__PURE__ */ new Date()).toISOString();
     const prompt = await this.buildCustomPrompt(prInfo, customPrompt, schema);
@@ -606,7 +690,7 @@ var AIReviewService = class {
         prompt,
         schema,
         debugInfo,
-        _checkName,
+        checkName,
         sessionId
       );
       const processingTime = Date.now() - startTime;
@@ -766,9 +850,9 @@ var AIReviewService = class {
    */
   async buildCustomPrompt(prInfo, customInstructions, schema, options) {
     const skipPRContext = options?.skipPRContext === true;
-    const prContext = skipPRContext ? "" : this.formatPRContext(prInfo);
-    const isIssue = prInfo.isIssue === true;
     const isCodeReviewSchema = schema === "code-review";
+    const prContext = skipPRContext ? "" : await this.formatPRContext(prInfo, isCodeReviewSchema);
+    const isIssue = prInfo.isIssue === true;
     if (isIssue) {
       if (skipPRContext) {
         return `<instructions>
@@ -857,7 +941,7 @@ ${prContext}
   /**
    * Format PR or Issue context for the AI using XML structure
    */
-  formatPRContext(prInfo) {
+  async formatPRContext(prInfo, isCodeReviewSchema) {
     const prContextInfo = prInfo;
     const isIssue = prContextInfo.isIssue === true;
     const isPRContext = prContextInfo.isPRContext === true;
@@ -939,7 +1023,12 @@ ${this.escapeXml(prInfo.body)}
       }
       const issueComments = prInfo.comments;
       if (issueComments && issueComments.length > 0) {
-        const historicalComments = triggeringComment2 ? issueComments.filter((c) => c.id !== triggeringComment2.id) : issueComments;
+        let historicalComments = triggeringComment2 ? issueComments.filter((c) => c.id !== triggeringComment2.id) : issueComments;
+        if (isCodeReviewSchema) {
+          historicalComments = historicalComments.filter(
+            (c) => !c.body || !c.body.includes("visor-comment-id:pr-review-")
+          );
+        }
         if (historicalComments.length > 0) {
           context2 += `
   <!-- Previous comments in chronological order (excluding triggering comment) -->
@@ -981,24 +1070,27 @@ ${this.escapeXml(prInfo.body)}
     }
     if (includeCodeContext) {
       if (prInfo.fullDiff) {
+        const processedFullDiff = await processDiffWithOutline(prInfo.fullDiff);
         context += `
-  <!-- Complete unified diff showing all changes in the pull request -->
+  <!-- Complete unified diff showing all changes in the pull request (processed with outline-diff) -->
   <full_diff>
-${this.escapeXml(prInfo.fullDiff)}
+${this.escapeXml(processedFullDiff)}
   </full_diff>`;
       }
       if (prInfo.isIncremental) {
         if (prInfo.commitDiff && prInfo.commitDiff.length > 0) {
+          const processedCommitDiff = await processDiffWithOutline(prInfo.commitDiff);
           context += `
-  <!-- Diff of only the latest commit for incremental analysis -->
+  <!-- Diff of only the latest commit for incremental analysis (processed with outline-diff) -->
   <commit_diff>
-${this.escapeXml(prInfo.commitDiff)}
+${this.escapeXml(processedCommitDiff)}
   </commit_diff>`;
         } else {
+          const processedFallbackDiff = prInfo.fullDiff ? await processDiffWithOutline(prInfo.fullDiff) : "";
           context += `
-  <!-- Commit diff could not be retrieved - falling back to full diff analysis -->
+  <!-- Commit diff could not be retrieved - falling back to full diff analysis (processed with outline-diff) -->
   <commit_diff>
-${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ""}
+${this.escapeXml(processedFallbackDiff)}
   </commit_diff>`;
         }
       }
@@ -1034,7 +1126,12 @@ ${prInfo.fullDiff ? this.escapeXml(prInfo.fullDiff) : ""}
     }
     const prComments = prInfo.comments;
     if (prComments && prComments.length > 0) {
-      const historicalComments = triggeringComment ? prComments.filter((c) => c.id !== triggeringComment.id) : prComments;
+      let historicalComments = triggeringComment ? prComments.filter((c) => c.id !== triggeringComment.id) : prComments;
+      if (isCodeReviewSchema) {
+        historicalComments = historicalComments.filter(
+          (c) => !c.body || !c.body.includes("visor-comment-id:pr-review-")
+        );
+      }
       if (historicalComments.length > 0) {
         context += `
   <!-- Previous PR comments in chronological order (excluding triggering comment) -->
@@ -1105,7 +1202,7 @@ ${schemaString}`);
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
           const provider = this.config.provider || "auto";
           const model = this.config.model || "default";
@@ -1219,16 +1316,16 @@ ${"=".repeat(60)}
 `;
           readableVersion += `${"=".repeat(60)}
 `;
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
           if (!fs5.existsSync(debugArtifactsDir)) {
             fs5.mkdirSync(debugArtifactsDir, { recursive: true });
           }
-          const debugFile = path5.join(
+          const debugFile = path6.join(
             debugArtifactsDir,
             `prompt-${_checkName || "unknown"}-${timestamp}.json`
           );
           fs5.writeFileSync(debugFile, debugJson, "utf-8");
-          const readableFile = path5.join(
+          const readableFile = path6.join(
             debugArtifactsDir,
             `prompt-${_checkName || "unknown"}-${timestamp}.txt`
           );
@@ -1265,7 +1362,7 @@ ${"=".repeat(60)}
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
           const agentAny2 = agent;
           let fullHistory = [];
@@ -1276,8 +1373,8 @@ ${"=".repeat(60)}
           } else if (agentAny2._messages) {
             fullHistory = agentAny2._messages;
           }
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
-          const sessionFile = path5.join(
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
+          const sessionFile = path6.join(
             debugArtifactsDir,
             `session-${_checkName || "unknown"}-${timestamp}.json`
           );
@@ -1292,7 +1389,7 @@ ${"=".repeat(60)}
             latestResponse: response
           };
           fs5.writeFileSync(sessionFile, JSON.stringify(sessionData, null, 2), "utf-8");
-          const sessionTxtFile = path5.join(
+          const sessionTxtFile = path6.join(
             debugArtifactsDir,
             `session-${_checkName || "unknown"}-${timestamp}.txt`
           );
@@ -1336,10 +1433,10 @@ ${"=".repeat(60)}
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
-          const responseFile = path5.join(
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
+          const responseFile = path6.join(
             debugArtifactsDir,
             `response-${_checkName || "unknown"}-${timestamp}.txt`
           );
@@ -1511,7 +1608,7 @@ ${schemaString}`);
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const os = __require("os");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
           const debugData = {
@@ -1585,21 +1682,21 @@ ${"=".repeat(60)}
           readableVersion += `${"=".repeat(60)}
 `;
           const tempDir = os.tmpdir();
-          const promptFile = path5.join(tempDir, `visor-prompt-${timestamp}.txt`);
+          const promptFile = path6.join(tempDir, `visor-prompt-${timestamp}.txt`);
           fs5.writeFileSync(promptFile, prompt, "utf-8");
           log(`
 \u{1F4BE} Prompt saved to: ${promptFile}`);
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
           try {
             if (!fs5.existsSync(debugArtifactsDir)) {
               fs5.mkdirSync(debugArtifactsDir, { recursive: true });
             }
-            const debugFile = path5.join(
+            const debugFile = path6.join(
               debugArtifactsDir,
               `prompt-${_checkName || "unknown"}-${timestamp}.json`
             );
             fs5.writeFileSync(debugFile, debugJson, "utf-8");
-            const readableFile = path5.join(
+            const readableFile = path6.join(
               debugArtifactsDir,
               `prompt-${_checkName || "unknown"}-${timestamp}.txt`
             );
@@ -1652,7 +1749,7 @@ $ ${cliCommand}
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
           const agentAny = agent;
           let fullHistory = [];
@@ -1663,8 +1760,8 @@ $ ${cliCommand}
           } else if (agentAny._messages) {
             fullHistory = agentAny._messages;
           }
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
-          const sessionFile = path5.join(
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
+          const sessionFile = path6.join(
             debugArtifactsDir,
             `session-${_checkName || "unknown"}-${timestamp}.json`
           );
@@ -1679,7 +1776,7 @@ $ ${cliCommand}
             latestResponse: response
           };
           fs5.writeFileSync(sessionFile, JSON.stringify(sessionData, null, 2), "utf-8");
-          const sessionTxtFile = path5.join(
+          const sessionTxtFile = path6.join(
             debugArtifactsDir,
             `session-${_checkName || "unknown"}-${timestamp}.txt`
           );
@@ -1723,10 +1820,10 @@ ${"=".repeat(60)}
       if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
         try {
           const fs5 = __require("fs");
-          const path5 = __require("path");
+          const path6 = __require("path");
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path5.join(process.cwd(), "debug-artifacts");
-          const responseFile = path5.join(
+          const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path6.join(process.cwd(), "debug-artifacts");
+          const responseFile = path6.join(
             debugArtifactsDir,
             `response-${_checkName || "unknown"}-${timestamp}.txt`
           );
@@ -1816,7 +1913,7 @@ ${"=".repeat(60)}
    */
   async loadSchemaContent(schema) {
     const fs5 = __require("fs").promises;
-    const path5 = __require("path");
+    const path6 = __require("path");
     if (typeof schema === "object" && schema !== null) {
       log("\u{1F4CB} Using inline schema object from configuration");
       return JSON.stringify(schema);
@@ -1829,12 +1926,12 @@ ${"=".repeat(60)}
       }
     } catch {
     }
-    if ((schema.startsWith("./") || schema.includes(".json")) && !path5.isAbsolute(schema)) {
+    if ((schema.startsWith("./") || schema.includes(".json")) && !path6.isAbsolute(schema)) {
       if (schema.includes("..") || schema.includes("\0")) {
         throw new Error("Invalid schema path: path traversal not allowed");
       }
       try {
-        const schemaPath2 = path5.resolve(process.cwd(), schema);
+        const schemaPath2 = path6.resolve(process.cwd(), schema);
         log(`\u{1F4CB} Loading custom schema from file: ${schemaPath2}`);
         const schemaContent = await fs5.readFile(schemaPath2, "utf-8");
         return schemaContent.trim();
@@ -1848,7 +1945,7 @@ ${"=".repeat(60)}
     if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
       throw new Error("Invalid schema name");
     }
-    const schemaPath = path5.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+    const schemaPath = path6.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
     try {
       const schemaContent = await fs5.readFile(schemaPath, "utf-8");
       return schemaContent.trim();
@@ -2178,7 +2275,7 @@ var PRReviewer = class {
   async reviewPR(owner, repo, prNumber, prInfo, options = {}) {
     const { debug = false, config, checks } = options;
     if (config && checks && checks.length > 0) {
-      const { CheckExecutionEngine: CheckExecutionEngine2 } = await import("./check-execution-engine-RORGGGGP.mjs");
+      const { CheckExecutionEngine: CheckExecutionEngine2 } = await import("./check-execution-engine-NMPXJ7FQ.mjs");
       const engine = new CheckExecutionEngine2();
       const { results } = await engine.executeGroupedChecks(
         prInfo,
@@ -2197,14 +2294,66 @@ var PRReviewer = class {
       "No configuration provided. Please create a .visor.yaml file with check definitions. Built-in prompts have been removed - all checks must be explicitly configured."
     );
   }
+  /**
+   * Helper to check if a schema is comment-generating
+   * Comment-generating schemas include:
+   * - Built-in schemas: code-review, overview, plain, text
+   * - Custom schemas with a "text" field in properties
+   */
+  async isCommentGeneratingSchema(schema) {
+    try {
+      if (typeof schema === "string") {
+        if (["code-review", "overview", "plain", "text"].includes(schema)) {
+          return true;
+        }
+        const fs5 = __require("fs").promises;
+        const path6 = __require("path");
+        const sanitizedSchemaName = schema.replace(/[^a-zA-Z0-9-]/g, "");
+        if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
+          return false;
+        }
+        const schemaPath = path6.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+        try {
+          const schemaContent = await fs5.readFile(schemaPath, "utf-8");
+          const schemaObj = JSON.parse(schemaContent);
+          const properties = schemaObj.properties;
+          return !!(properties && "text" in properties);
+        } catch {
+          return false;
+        }
+      } else {
+        const properties = schema.properties;
+        return !!(properties && "text" in properties);
+      }
+    } catch {
+      return false;
+    }
+  }
+  /**
+   * Filter check results to only include those that should post GitHub comments
+   */
+  async filterCommentGeneratingChecks(checkResults, config) {
+    const filtered = [];
+    for (const r of checkResults) {
+      const cfg = config.checks?.[r.checkName];
+      const type = cfg?.type || "ai";
+      const schema = cfg?.schema;
+      let shouldPostComment = false;
+      const isAICheck = type === "ai" || type === "claude-code";
+      if (!schema || schema === "") {
+        shouldPostComment = isAICheck;
+      } else {
+        shouldPostComment = await this.isCommentGeneratingSchema(schema);
+      }
+      if (shouldPostComment) {
+        filtered.push(r);
+      }
+    }
+    return filtered;
+  }
   async postReviewComment(owner, repo, prNumber, groupedResults, options = {}) {
     for (const [groupName, checkResults] of Object.entries(groupedResults)) {
-      const filteredResults = options.config ? checkResults.filter((r) => {
-        const cfg = options.config.checks?.[r.checkName];
-        const t = cfg?.type || "";
-        const isGitHubOps = t === "github" || r.group === "github" || t === "noop" || t === "command";
-        return !isGitHubOps;
-      }) : checkResults;
+      const filteredResults = options.config ? await this.filterCommentGeneratingChecks(checkResults, options.config) : checkResults;
       if (!filteredResults || filteredResults.length === 0) {
         continue;
       }
@@ -2337,14 +2486,14 @@ var PRReviewer = class {
   saveDebugArtifact(debug) {
     try {
       const fs5 = __require("fs");
-      const path5 = __require("path");
-      const debugDir = path5.join(process.cwd(), "debug-artifacts");
+      const path6 = __require("path");
+      const debugDir = path6.join(process.cwd(), "debug-artifacts");
       if (!fs5.existsSync(debugDir)) {
         fs5.mkdirSync(debugDir, { recursive: true });
       }
       const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
       const filename = `visor-debug-${timestamp}.md`;
-      const filepath = path5.join(debugDir, filename);
+      const filepath = path6.join(debugDir, filename);
       const content = [
         `# Visor Debug Information`,
         ``,
@@ -2376,7 +2525,7 @@ var PRReviewer = class {
 
 // src/git-repository-analyzer.ts
 import { simpleGit } from "simple-git";
-import * as path from "path";
+import * as path2 from "path";
 import * as fs from "fs";
 var MAX_PATCH_SIZE = 50 * 1024;
 var GitRepositoryAnalyzer = class {
@@ -2591,7 +2740,7 @@ ${file.patch}`).join("\n\n");
           console.error(`\u23ED\uFE0F  Skipping excluded file: ${file}`);
           continue;
         }
-        const filePath = path.join(this.cwd, file);
+        const filePath = path2.join(this.cwd, file);
         const fileChange = await this.analyzeFileChange(file, status2, filePath, includeContext);
         changes.push(fileChange);
       }
@@ -3052,7 +3201,7 @@ var EnvironmentResolver = class {
 
 // src/issue-filter.ts
 import * as fs2 from "fs";
-import * as path2 from "path";
+import * as path3 from "path";
 var IssueFilter = class {
   fileCache = /* @__PURE__ */ new Map();
   suppressionEnabled;
@@ -3120,7 +3269,7 @@ var IssueFilter = class {
       return this.fileCache.get(filePath);
     }
     try {
-      const resolvedPath = path2.isAbsolute(filePath) ? filePath : path2.join(workingDir, filePath);
+      const resolvedPath = path3.isAbsolute(filePath) ? filePath : path3.join(workingDir, filePath);
       if (!fs2.existsSync(resolvedPath)) {
         if (fs2.existsSync(filePath)) {
           const content2 = fs2.readFileSync(filePath, "utf8");
@@ -3148,7 +3297,7 @@ var IssueFilter = class {
 
 // src/providers/ai-check-provider.ts
 import fs3 from "fs/promises";
-import path3 from "path";
+import path4 from "path";
 var AICheckProvider = class extends CheckProvider {
   aiReviewService;
   liquidEngine;
@@ -3265,7 +3414,7 @@ var AICheckProvider = class extends CheckProvider {
     const hasFileExtension = /\.[a-zA-Z0-9]{1,10}$/i.test(str);
     const hasPathSeparators = /[\/\\]/.test(str);
     const isRelativePath = /^\.{1,2}\//.test(str);
-    const isAbsolutePath = path3.isAbsolute(str);
+    const isAbsolutePath = path4.isAbsolute(str);
     const hasTypicalFileChars = /^[a-zA-Z0-9._\-\/\\:~]+$/.test(str);
     if (!(hasFileExtension || isRelativePath || isAbsolutePath || hasPathSeparators)) {
       return false;
@@ -3275,10 +3424,10 @@ var AICheckProvider = class extends CheckProvider {
     }
     try {
       let resolvedPath;
-      if (path3.isAbsolute(str)) {
-        resolvedPath = path3.normalize(str);
+      if (path4.isAbsolute(str)) {
+        resolvedPath = path4.normalize(str);
       } else {
-        resolvedPath = path3.resolve(process.cwd(), str);
+        resolvedPath = path4.resolve(process.cwd(), str);
       }
       const fs5 = __require("fs").promises;
       try {
@@ -3299,14 +3448,14 @@ var AICheckProvider = class extends CheckProvider {
       throw new Error("Prompt file must have .liquid extension");
     }
     let resolvedPath;
-    if (path3.isAbsolute(promptPath)) {
+    if (path4.isAbsolute(promptPath)) {
       resolvedPath = promptPath;
     } else {
-      resolvedPath = path3.resolve(process.cwd(), promptPath);
+      resolvedPath = path4.resolve(process.cwd(), promptPath);
     }
-    if (!path3.isAbsolute(promptPath)) {
-      const normalizedPath = path3.normalize(resolvedPath);
-      const currentDir = path3.resolve(process.cwd());
+    if (!path4.isAbsolute(promptPath)) {
+      const normalizedPath = path4.normalize(resolvedPath);
+      const currentDir = path4.resolve(process.cwd());
       if (!normalizedPath.startsWith(currentDir)) {
         throw new Error("Invalid prompt file path: path traversal detected");
       }
@@ -4167,7 +4316,8 @@ var LogCheckProvider = class extends CheckProvider {
       dependencyResults,
       includePrContext,
       includeDependencies,
-      includeMetadata
+      includeMetadata,
+      config.__outputHistory
     );
     const renderedMessage = await this.liquid.parseAndRender(message, templateContext);
     const logOutput = this.formatLogOutput(
@@ -4188,7 +4338,7 @@ var LogCheckProvider = class extends CheckProvider {
       logOutput
     };
   }
-  buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true) {
+  buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true, outputHistory) {
     const context = {};
     context.pr = {
       number: prInfo.number,
@@ -4212,6 +4362,7 @@ var LogCheckProvider = class extends CheckProvider {
     if (dependencyResults) {
       const dependencies = {};
       const outputs = {};
+      const history = {};
       context.dependencyCount = dependencyResults.size;
       for (const [checkName, result] of dependencyResults.entries()) {
         dependencies[checkName] = {
@@ -4222,6 +4373,12 @@ var LogCheckProvider = class extends CheckProvider {
         const summary = result;
         outputs[checkName] = summary.output !== void 0 ? summary.output : summary;
       }
+      if (outputHistory) {
+        for (const [checkName, historyArray] of outputHistory) {
+          history[checkName] = historyArray;
+        }
+      }
+      outputs.history = history;
       context.dependencies = dependencies;
       context.outputs = outputs;
     }
@@ -4581,7 +4738,7 @@ ${cfg.value_js}
 
 // src/providers/claude-code-check-provider.ts
 import fs4 from "fs/promises";
-import path4 from "path";
+import path5 from "path";
 
 // src/providers/claude-code-types.ts
 async function safeImport(moduleName) {
@@ -4742,7 +4899,7 @@ var ClaudeCodeCheckProvider = class extends CheckProvider {
     const hasFileExtension = /\.[a-zA-Z0-9]{1,10}$/i.test(str);
     const hasPathSeparators = /[\/\\]/.test(str);
     const isRelativePath = /^\.{1,2}\//.test(str);
-    const isAbsolutePath = path4.isAbsolute(str);
+    const isAbsolutePath = path5.isAbsolute(str);
     const hasTypicalFileChars = /^[a-zA-Z0-9._\-\/\\:~]+$/.test(str);
     if (!(hasFileExtension || isRelativePath || isAbsolutePath || hasPathSeparators)) {
       return false;
@@ -4752,10 +4909,10 @@ var ClaudeCodeCheckProvider = class extends CheckProvider {
     }
     try {
       let resolvedPath;
-      if (path4.isAbsolute(str)) {
-        resolvedPath = path4.normalize(str);
+      if (path5.isAbsolute(str)) {
+        resolvedPath = path5.normalize(str);
       } else {
-        resolvedPath = path4.resolve(process.cwd(), str);
+        resolvedPath = path5.resolve(process.cwd(), str);
       }
       try {
         const stat = await fs4.stat(resolvedPath);
@@ -4775,14 +4932,14 @@ var ClaudeCodeCheckProvider = class extends CheckProvider {
       throw new Error("Prompt file must have .liquid extension");
     }
     let resolvedPath;
-    if (path4.isAbsolute(promptPath)) {
+    if (path5.isAbsolute(promptPath)) {
       resolvedPath = promptPath;
     } else {
-      resolvedPath = path4.resolve(process.cwd(), promptPath);
+      resolvedPath = path5.resolve(process.cwd(), promptPath);
     }
-    if (!path4.isAbsolute(promptPath)) {
-      const normalizedPath = path4.normalize(resolvedPath);
-      const currentDir = path4.resolve(process.cwd());
+    if (!path5.isAbsolute(promptPath)) {
+      const normalizedPath = path5.normalize(resolvedPath);
+      const currentDir = path5.resolve(process.cwd());
       if (!normalizedPath.startsWith(currentDir)) {
         throw new Error("Invalid prompt file path: path traversal detected");
       }
@@ -5093,7 +5250,10 @@ var CommandCheckProvider = class extends CheckProvider {
       },
       files: prInfo.files,
       fileCount: prInfo.files.length,
-      outputs: this.buildOutputContext(dependencyResults),
+      outputs: this.buildOutputContext(
+        dependencyResults,
+        config.__outputHistory
+      ),
       env: this.getSafeEnvironmentVariables()
     };
     logger.debug(
@@ -5819,16 +5979,23 @@ ${stderrOutput}` : `Command execution failed: ${errorMessage}`;
       };
     }
   }
-  buildOutputContext(dependencyResults) {
+  buildOutputContext(dependencyResults, outputHistory) {
     if (!dependencyResults) {
       return {};
     }
     const outputs = {};
+    const history = {};
     for (const [checkName, result] of dependencyResults) {
       const summary = result;
       const value = summary.output !== void 0 ? summary.output : summary;
       outputs[checkName] = this.makeJsonSmart(value);
     }
+    if (outputHistory) {
+      for (const [checkName, historyArray] of outputHistory) {
+        history[checkName] = historyArray.map((val) => this.makeJsonSmart(val));
+      }
+    }
+    outputs.history = history;
     return outputs;
   }
   /**
@@ -6377,7 +6544,12 @@ var MemoryCheckProvider = class extends CheckProvider {
     const key = config.key;
     const namespace = config.namespace;
     const memoryStore = MemoryStore.getInstance();
-    const templateContext = this.buildTemplateContext(prInfo, dependencyResults, memoryStore);
+    const templateContext = this.buildTemplateContext(
+      prInfo,
+      dependencyResults,
+      memoryStore,
+      config.__outputHistory
+    );
     let result;
     try {
       switch (operation) {
@@ -6660,7 +6832,7 @@ var MemoryCheckProvider = class extends CheckProvider {
   /**
    * Build template context for Liquid and JS evaluation
    */
-  buildTemplateContext(prInfo, dependencyResults, memoryStore) {
+  buildTemplateContext(prInfo, dependencyResults, memoryStore, outputHistory) {
     const context = {};
     context.pr = {
       number: prInfo.number,
@@ -6679,14 +6851,21 @@ var MemoryCheckProvider = class extends CheckProvider {
         changes: f.changes
       }))
     };
+    const outputs = {};
+    const history = {};
     if (dependencyResults) {
-      const outputs = {};
       for (const [checkName, result] of dependencyResults.entries()) {
         const summary = result;
         outputs[checkName] = summary.output !== void 0 ? summary.output : summary;
       }
-      context.outputs = outputs;
     }
+    if (outputHistory) {
+      for (const [checkName, historyArray] of outputHistory) {
+        history[checkName] = historyArray;
+      }
+    }
+    outputs.history = history;
+    context.outputs = outputs;
     if (memoryStore) {
       context.memory = {
         get: (key, ns) => memoryStore.get(key, ns),
@@ -7424,10 +7603,6 @@ var FailureConditionEvaluator = class _FailureConditionEvaluator {
         if (!Array.isArray(issues2)) return false;
         return issues2.some((issue) => issue.file?.includes(pattern));
       };
-      const hasSuggestion = (suggestions2, text) => {
-        if (!Array.isArray(suggestions2)) return false;
-        return suggestions2.some((s) => s.toLowerCase().includes(text.toLowerCase()));
-      };
       const hasIssueWith = hasIssue;
       const hasFileWith = hasFileMatching;
       const permissionHelpers = createPermissionHelpers(
@@ -7442,7 +7617,6 @@ var FailureConditionEvaluator = class _FailureConditionEvaluator {
       const isFirstTimer = permissionHelpers.isFirstTimer;
       const output = context.output || {};
       const issues = output.issues || [];
-      const suggestions = [];
       const metadata = context.metadata || {
         checkName: context.checkName || "",
         schema: context.schema || "",
@@ -7486,7 +7660,6 @@ var FailureConditionEvaluator = class _FailureConditionEvaluator {
         memory: memoryAccessor,
         // Legacy compatibility variables
         issues,
-        suggestions,
         metadata,
         criticalIssues,
         errorIssues,
@@ -7515,7 +7688,6 @@ var FailureConditionEvaluator = class _FailureConditionEvaluator {
         hasIssue,
         countIssues,
         hasFileMatching,
-        hasSuggestion,
         hasIssueWith,
         hasFileWith,
         // Permission helpers
@@ -8019,11 +8191,7 @@ Please check your configuration and try again.`
       });
     }
     sections.push("");
-    sections.push("---");
-    sections.push("");
-    sections.push(
-      "*Powered by [Visor](https://probelabs.com/visor) from [Probelabs](https://probelabs.com)*"
-    );
+    sections.push(generateFooter());
     return sections.join("\n");
   }
   /**
@@ -8263,6 +8431,8 @@ var CheckExecutionEngine = class _CheckExecutionEngine {
   webhookContext;
   routingSandbox;
   executionStats = /* @__PURE__ */ new Map();
+  // Track history of all outputs for each check (useful for loops and goto)
+  outputHistory = /* @__PURE__ */ new Map();
   // Event override to simulate alternate event (used during routing goto)
   routingEventOverride;
   // Cached GitHub context for context elevation when running in Actions
@@ -8506,6 +8676,8 @@ ${expr}
         transform_js: targetCfg.transform_js,
         env: targetCfg.env,
         forEach: targetCfg.forEach,
+        // Pass output history for loop/goto scenarios
+        __outputHistory: this.outputHistory,
         // Include provider-specific keys (e.g., op/values for github)
         ...targetCfg,
         ai: {
@@ -8573,6 +8745,10 @@ ${expr}
         timestamp: Date.now()
       }));
       const enriched = { ...r, issues: enrichedIssues };
+      const enrichedWithOutput = enriched;
+      if (enrichedWithOutput.output !== void 0) {
+        this.trackOutputHistory(target, enrichedWithOutput.output);
+      }
       resultsMap?.set(target, enriched);
       if (debug) log2(`\u{1F527} Debug: inline executed '${target}', issues: ${enrichedIssues.length}`);
       return enriched;
@@ -9341,6 +9517,8 @@ ${expr}
       ai_model: checkConfig.ai_model || config.ai_model,
       // Pass claude_code config if present
       claude_code: checkConfig.claude_code,
+      // Pass output history for loop/goto scenarios
+      __outputHistory: this.outputHistory,
       // Pass any provider-specific config
       ...checkConfig
     };
@@ -9377,10 +9555,14 @@ ${expr}
       }
     }
     const content = await this.renderCheckContent(checkName, result, checkConfig, prInfo);
+    let group = checkConfig.group || "default";
+    if (config?.output?.pr_comment?.group_by === "check" && !checkConfig.group) {
+      group = checkName;
+    }
     return {
       checkName,
       content,
-      group: checkConfig.group || "default",
+      group,
       output: result.output,
       debug: result.debug,
       issues: result.issues
@@ -9519,16 +9701,19 @@ ${expr}
           }
         ];
       }
+      let group = checkConfig.group || "default";
+      if (config?.output?.pr_comment?.group_by === "check" && !checkConfig.group) {
+        group = checkName;
+      }
       const checkResult = {
         checkName,
         content,
-        group: checkConfig.group || "default",
+        group,
         output: checkSummary.output,
         debug: reviewSummary.debug,
         issues: issuesForCheck
         // Include structured issues + rendering error if any
       };
-      const group = checkResult.group;
       if (!groupedResults[group]) {
         groupedResults[group] = [];
       }
@@ -9546,7 +9731,7 @@ ${expr}
    * - Enforcing .liquid file extension
    */
   async validateTemplatePath(templatePath) {
-    const path5 = await import("path");
+    const path6 = await import("path");
     if (!templatePath || typeof templatePath !== "string" || templatePath.trim() === "") {
       throw new Error("Template path must be a non-empty string");
     }
@@ -9556,7 +9741,7 @@ ${expr}
     if (!templatePath.endsWith(".liquid")) {
       throw new Error("Template file must have .liquid extension");
     }
-    if (path5.isAbsolute(templatePath)) {
+    if (path6.isAbsolute(templatePath)) {
       throw new Error("Template path must be relative to project directory");
     }
     if (templatePath.includes("..")) {
@@ -9570,14 +9755,14 @@ ${expr}
     if (!projectRoot || typeof projectRoot !== "string") {
       throw new Error("Unable to determine project root directory");
     }
-    const resolvedPath = path5.resolve(projectRoot, templatePath);
-    const resolvedProjectRoot = path5.resolve(projectRoot);
+    const resolvedPath = path6.resolve(projectRoot, templatePath);
+    const resolvedProjectRoot = path6.resolve(projectRoot);
     if (!resolvedPath || !resolvedProjectRoot || resolvedPath === "" || resolvedProjectRoot === "") {
       throw new Error(
         `Unable to resolve template path: projectRoot="${projectRoot}", templatePath="${templatePath}", resolvedPath="${resolvedPath}", resolvedProjectRoot="${resolvedProjectRoot}"`
       );
     }
-    if (!resolvedPath.startsWith(resolvedProjectRoot + path5.sep) && resolvedPath !== resolvedProjectRoot) {
+    if (!resolvedPath.startsWith(resolvedProjectRoot + path6.sep) && resolvedPath !== resolvedProjectRoot) {
       throw new Error("Template path escapes project directory");
     }
     return resolvedPath;
@@ -9622,7 +9807,7 @@ ${expr}
     }
     const { createExtendedLiquid: createExtendedLiquid2 } = await import("./liquid-extensions-GMEGEGC3.mjs");
     const fs5 = await import("fs/promises");
-    const path5 = await import("path");
+    const path6 = await import("path");
     const liquid = createExtendedLiquid2({
       trimTagLeft: false,
       trimTagRight: false,
@@ -9656,7 +9841,7 @@ ${expr}
       if (!sanitizedSchema) {
         throw new Error("Invalid schema name");
       }
-      const templatePath = path5.join(__dirname, `../output/${sanitizedSchema}/template.liquid`);
+      const templatePath = path6.join(__dirname, `../output/${sanitizedSchema}/template.liquid`);
       templateContent = await fs5.readFile(templatePath, "utf-8");
       if (sanitizedSchema === "issue-assistant") {
         enrichAssistantContext = true;
@@ -10429,6 +10614,10 @@ ${expr}
                   itemResult.issues || [],
                   itemResult.output
                 );
+                const itemOutput = itemResult.output;
+                if (itemOutput !== void 0) {
+                  this.trackOutputHistory(checkName, itemOutput);
+                }
                 const descendantSet = (() => {
                   const visited = /* @__PURE__ */ new Set();
                   const stack = [checkName];
@@ -11059,6 +11248,10 @@ ${error.stack || ""}` : String(error);
             ]);
           } catch {
           }
+          const reviewResultWithOutput = reviewResult;
+          if (reviewResultWithOutput.output !== void 0) {
+            this.trackOutputHistory(checkName, reviewResultWithOutput.output);
+          }
           results.set(checkName, reviewResult);
         } else {
           const errorSummary = {
@@ -12302,6 +12495,16 @@ ${result.value.result.debug.rawResponse}`;
       stats.outputsProduced = (stats.outputsProduced || 0) + 1;
     }
   }
+  /**
+   * Track output in history for loop/goto scenarios
+   */
+  trackOutputHistory(checkName, output) {
+    if (output === void 0) return;
+    if (!this.outputHistory.has(checkName)) {
+      this.outputHistory.set(checkName, []);
+    }
+    this.outputHistory.get(checkName).push(output);
+  }
   /**
    * Record that a check was skipped
    */
@@ -12494,4 +12697,4 @@ ${result.value.result.debug.rawResponse}`;
 export {
   CheckExecutionEngine
 };
-//# sourceMappingURL=chunk-Z47UECAT.mjs.map
+//# sourceMappingURL=chunk-Q4S5A5TO.mjs.map