@probelabs/visor 0.1.94 → 0.1.96

package/dist/sdk/sdk.js

@@ -136,6 +136,33 @@ var init_logger = __esm({
   }
 });
 
+// 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");
+}
+var init_footer = __esm({
+  "src/footer.ts"() {
+    "use strict";
+  }
+});
+
 // src/github-comments.ts
 var import_uuid, CommentManager;
 var init_github_comments = __esm({
@@ -143,6 +170,7 @@ var init_github_comments = __esm({
     "use strict";
     import_uuid = require("uuid");
     init_logger();
+    init_footer();
     CommentManager = class {
       octokit;
       retryConfig;
@@ -244,15 +272,17 @@ var init_github_comments = __esm({
        */
       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} -->`;
       }
       /**
@@ -718,18 +748,81 @@ var init_session_registry = __esm({
   }
 });
 
+// src/utils/diff-processor.ts
+async function processDiffWithOutline(diffContent) {
+  if (!diffContent || diffContent.trim().length === 0) {
+    return diffContent;
+  }
+  try {
+    const originalProbePath = process.env.PROBE_PATH;
+    const fs12 = require("fs");
+    const possiblePaths = [
+      // Relative to current working directory (most common in production)
+      path2.join(process.cwd(), "node_modules/@probelabs/probe/bin/probe-binary"),
+      // Relative to __dirname (for unbundled development)
+      path2.join(__dirname, "../..", "node_modules/@probelabs/probe/bin/probe-binary"),
+      // Relative to dist directory (for bundled CLI)
+      path2.join(__dirname, "node_modules/@probelabs/probe/bin/probe-binary")
+    ];
+    let probeBinaryPath;
+    for (const candidatePath of possiblePaths) {
+      if (fs12.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 = (0, import_probe2.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;
+  }
+}
+var import_probe2, path2;
+var init_diff_processor = __esm({
+  "src/utils/diff-processor.ts"() {
+    "use strict";
+    import_probe2 = require("@probelabs/probe");
+    path2 = __toESM(require("path"));
+  }
+});
+
 // src/ai-review-service.ts
 function log(...args) {
   logger.debug(args.join(" "));
 }
-var import_probe2, AIReviewService;
+var import_probe3, AIReviewService;
 var init_ai_review_service = __esm({
   "src/ai-review-service.ts"() {
     "use strict";
-    import_probe2 = require("@probelabs/probe");
+    import_probe3 = require("@probelabs/probe");
     init_session_registry();
     init_logger();
     init_tracer_init();
+    init_diff_processor();
     AIReviewService = class {
       config;
       sessionRegistry;
@@ -768,7 +861,7 @@ var init_ai_review_service = __esm({
       /**
        * 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);
@@ -825,7 +918,7 @@ var init_ai_review_service = __esm({
             prompt,
             schema,
             debugInfo,
-            _checkName,
+            checkName,
             sessionId
           );
           const processingTime = Date.now() - startTime;
@@ -985,9 +1078,9 @@ var init_ai_review_service = __esm({
        */
       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>
@@ -1076,7 +1169,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;
@@ -1158,7 +1251,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) {
               context3 += `
   <!-- Previous comments in chronological order (excluding triggering comment) -->
@@ -1200,24 +1298,27 @@ ${this.escapeXml(prInfo.body)}
         }
         if (includeCodeContext) {
           if (prInfo.fullDiff) {
+            const processedFullDiff = await processDiffWithOutline(prInfo.fullDiff);
             context2 += `
-  <!-- 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);
               context2 += `
-  <!-- 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) : "";
               context2 += `
-  <!-- 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>`;
             }
           }
@@ -1253,7 +1354,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) {
             context2 += `
   <!-- Previous PR comments in chronological order (excluding triggering comment) -->
@@ -1324,7 +1430,7 @@ ${schemaString}`);
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
               const provider = this.config.provider || "auto";
               const model = this.config.model || "default";
@@ -1438,16 +1544,16 @@ ${"=".repeat(60)}
 `;
               readableVersion += `${"=".repeat(60)}
 `;
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
               if (!fs12.existsSync(debugArtifactsDir)) {
                 fs12.mkdirSync(debugArtifactsDir, { recursive: true });
               }
-              const debugFile = path12.join(
+              const debugFile = path13.join(
                 debugArtifactsDir,
                 `prompt-${_checkName || "unknown"}-${timestamp}.json`
               );
               fs12.writeFileSync(debugFile, debugJson, "utf-8");
-              const readableFile = path12.join(
+              const readableFile = path13.join(
                 debugArtifactsDir,
                 `prompt-${_checkName || "unknown"}-${timestamp}.txt`
               );
@@ -1484,7 +1590,7 @@ ${"=".repeat(60)}
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
               const agentAny2 = agent;
               let fullHistory = [];
@@ -1495,8 +1601,8 @@ ${"=".repeat(60)}
               } else if (agentAny2._messages) {
                 fullHistory = agentAny2._messages;
               }
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
-              const sessionFile = path12.join(
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
+              const sessionFile = path13.join(
                 debugArtifactsDir,
                 `session-${_checkName || "unknown"}-${timestamp}.json`
               );
@@ -1511,7 +1617,7 @@ ${"=".repeat(60)}
                 latestResponse: response
               };
               fs12.writeFileSync(sessionFile, JSON.stringify(sessionData, null, 2), "utf-8");
-              const sessionTxtFile = path12.join(
+              const sessionTxtFile = path13.join(
                 debugArtifactsDir,
                 `session-${_checkName || "unknown"}-${timestamp}.txt`
               );
@@ -1555,10 +1661,10 @@ ${"=".repeat(60)}
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
-              const responseFile = path12.join(
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
+              const responseFile = path13.join(
                 debugArtifactsDir,
                 `response-${_checkName || "unknown"}-${timestamp}.txt`
               );
@@ -1696,7 +1802,7 @@ ${"=".repeat(60)}
           if (this.config.model) {
             options.model = this.config.model;
           }
-          const agent = new import_probe2.ProbeAgent(options);
+          const agent = new import_probe3.ProbeAgent(options);
           log("\u{1F680} Calling ProbeAgent...");
           let schemaString = void 0;
           let effectiveSchema = typeof schema === "object" ? "custom" : schema;
@@ -1730,7 +1836,7 @@ ${schemaString}`);
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const os = require("os");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
               const debugData = {
@@ -1804,21 +1910,21 @@ ${"=".repeat(60)}
               readableVersion += `${"=".repeat(60)}
 `;
               const tempDir = os.tmpdir();
-              const promptFile = path12.join(tempDir, `visor-prompt-${timestamp}.txt`);
+              const promptFile = path13.join(tempDir, `visor-prompt-${timestamp}.txt`);
               fs12.writeFileSync(promptFile, prompt, "utf-8");
               log(`
 \u{1F4BE} Prompt saved to: ${promptFile}`);
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
               try {
                 if (!fs12.existsSync(debugArtifactsDir)) {
                   fs12.mkdirSync(debugArtifactsDir, { recursive: true });
                 }
-                const debugFile = path12.join(
+                const debugFile = path13.join(
                   debugArtifactsDir,
                   `prompt-${_checkName || "unknown"}-${timestamp}.json`
                 );
                 fs12.writeFileSync(debugFile, debugJson, "utf-8");
-                const readableFile = path12.join(
+                const readableFile = path13.join(
                   debugArtifactsDir,
                   `prompt-${_checkName || "unknown"}-${timestamp}.txt`
                 );
@@ -1871,7 +1977,7 @@ $ ${cliCommand}
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
               const agentAny = agent;
               let fullHistory = [];
@@ -1882,8 +1988,8 @@ $ ${cliCommand}
               } else if (agentAny._messages) {
                 fullHistory = agentAny._messages;
               }
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
-              const sessionFile = path12.join(
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
+              const sessionFile = path13.join(
                 debugArtifactsDir,
                 `session-${_checkName || "unknown"}-${timestamp}.json`
               );
@@ -1898,7 +2004,7 @@ $ ${cliCommand}
                 latestResponse: response
               };
               fs12.writeFileSync(sessionFile, JSON.stringify(sessionData, null, 2), "utf-8");
-              const sessionTxtFile = path12.join(
+              const sessionTxtFile = path13.join(
                 debugArtifactsDir,
                 `session-${_checkName || "unknown"}-${timestamp}.txt`
               );
@@ -1942,10 +2048,10 @@ ${"=".repeat(60)}
           if (process.env.VISOR_DEBUG_AI_SESSIONS === "true") {
             try {
               const fs12 = require("fs");
-              const path12 = require("path");
+              const path13 = require("path");
               const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path12.join(process.cwd(), "debug-artifacts");
-              const responseFile = path12.join(
+              const debugArtifactsDir = process.env.VISOR_DEBUG_ARTIFACTS || path13.join(process.cwd(), "debug-artifacts");
+              const responseFile = path13.join(
                 debugArtifactsDir,
                 `response-${_checkName || "unknown"}-${timestamp}.txt`
               );
@@ -2035,7 +2141,7 @@ ${"=".repeat(60)}
        */
       async loadSchemaContent(schema) {
         const fs12 = require("fs").promises;
-        const path12 = require("path");
+        const path13 = require("path");
         if (typeof schema === "object" && schema !== null) {
           log("\u{1F4CB} Using inline schema object from configuration");
           return JSON.stringify(schema);
@@ -2048,12 +2154,12 @@ ${"=".repeat(60)}
           }
         } catch {
         }
-        if ((schema.startsWith("./") || schema.includes(".json")) && !path12.isAbsolute(schema)) {
+        if ((schema.startsWith("./") || schema.includes(".json")) && !path13.isAbsolute(schema)) {
           if (schema.includes("..") || schema.includes("\0")) {
             throw new Error("Invalid schema path: path traversal not allowed");
           }
           try {
-            const schemaPath2 = path12.resolve(process.cwd(), schema);
+            const schemaPath2 = path13.resolve(process.cwd(), schema);
             log(`\u{1F4CB} Loading custom schema from file: ${schemaPath2}`);
             const schemaContent = await fs12.readFile(schemaPath2, "utf-8");
             return schemaContent.trim();
@@ -2067,7 +2173,7 @@ ${"=".repeat(60)}
         if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
           throw new Error("Invalid schema name");
         }
-        const schemaPath = path12.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+        const schemaPath = path13.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
         try {
           const schemaContent = await fs12.readFile(schemaPath, "utf-8");
           return schemaContent.trim();
@@ -2425,30 +2531,36 @@ var init_reviewer = __esm({
         );
       }
       /**
-       * Helper to check if a schema definition has a "text" field in its properties
+       * 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 schemaHasTextField(schema) {
+      async isCommentGeneratingSchema(schema) {
         try {
-          let schemaObj;
-          if (typeof schema === "object") {
-            schemaObj = schema;
-          } else {
+          if (typeof schema === "string") {
+            if (["code-review", "overview", "plain", "text"].includes(schema)) {
+              return true;
+            }
             const fs12 = require("fs").promises;
-            const path12 = require("path");
+            const path13 = require("path");
             const sanitizedSchemaName = schema.replace(/[^a-zA-Z0-9-]/g, "");
             if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
               return false;
             }
-            const schemaPath = path12.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+            const schemaPath = path13.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
             try {
               const schemaContent = await fs12.readFile(schemaPath, "utf-8");
-              schemaObj = JSON.parse(schemaContent);
+              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);
           }
-          const properties = schemaObj.properties;
-          return !!(properties && "text" in properties);
         } catch {
           return false;
         }
@@ -2466,14 +2578,8 @@ var init_reviewer = __esm({
           const isAICheck = type === "ai" || type === "claude-code";
           if (!schema || schema === "") {
             shouldPostComment = isAICheck;
-          } else if (typeof schema === "string") {
-            if (schema === "text" || schema === "plain") {
-              shouldPostComment = true;
-            } else {
-              shouldPostComment = await this.schemaHasTextField(schema);
-            }
-          } else if (typeof schema === "object") {
-            shouldPostComment = await this.schemaHasTextField(schema);
+          } else {
+            shouldPostComment = await this.isCommentGeneratingSchema(schema);
           }
           if (shouldPostComment) {
             filtered.push(r);
@@ -2616,14 +2722,14 @@ var init_reviewer = __esm({
       saveDebugArtifact(debug) {
         try {
           const fs12 = require("fs");
-          const path12 = require("path");
-          const debugDir = path12.join(process.cwd(), "debug-artifacts");
+          const path13 = require("path");
+          const debugDir = path13.join(process.cwd(), "debug-artifacts");
           if (!fs12.existsSync(debugDir)) {
             fs12.mkdirSync(debugDir, { recursive: true });
           }
           const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
           const filename = `visor-debug-${timestamp}.md`;
-          const filepath = path12.join(debugDir, filename);
+          const filepath = path13.join(debugDir, filename);
           const content = [
             `# Visor Debug Information`,
             ``,
@@ -2656,12 +2762,12 @@ var init_reviewer = __esm({
 });
 
 // src/git-repository-analyzer.ts
-var import_simple_git, path2, fs2, MAX_PATCH_SIZE, GitRepositoryAnalyzer;
+var import_simple_git, path3, fs2, MAX_PATCH_SIZE, GitRepositoryAnalyzer;
 var init_git_repository_analyzer = __esm({
   "src/git-repository-analyzer.ts"() {
     "use strict";
     import_simple_git = require("simple-git");
-    path2 = __toESM(require("path"));
+    path3 = __toESM(require("path"));
     fs2 = __toESM(require("fs"));
     MAX_PATCH_SIZE = 50 * 1024;
     GitRepositoryAnalyzer = class {
@@ -2876,7 +2982,7 @@ ${file.patch}`).join("\n\n");
               console.error(`\u23ED\uFE0F  Skipping excluded file: ${file}`);
               continue;
             }
-            const filePath = path2.join(this.cwd, file);
+            const filePath = path3.join(this.cwd, file);
             const fileChange = await this.analyzeFileChange(file, status2, filePath, includeContext);
             changes.push(fileChange);
           }
@@ -3356,12 +3462,12 @@ var init_env_resolver = __esm({
 });
 
 // src/issue-filter.ts
-var fs3, path3, IssueFilter;
+var fs3, path4, IssueFilter;
 var init_issue_filter = __esm({
   "src/issue-filter.ts"() {
     "use strict";
     fs3 = __toESM(require("fs"));
-    path3 = __toESM(require("path"));
+    path4 = __toESM(require("path"));
     IssueFilter = class {
       fileCache = /* @__PURE__ */ new Map();
       suppressionEnabled;
@@ -3429,7 +3535,7 @@ var init_issue_filter = __esm({
           return this.fileCache.get(filePath);
         }
         try {
-          const resolvedPath = path3.isAbsolute(filePath) ? filePath : path3.join(workingDir, filePath);
+          const resolvedPath = path4.isAbsolute(filePath) ? filePath : path4.join(workingDir, filePath);
           if (!fs3.existsSync(resolvedPath)) {
             if (fs3.existsSync(filePath)) {
               const content2 = fs3.readFileSync(filePath, "utf8");
@@ -9306,7 +9412,7 @@ function resolveTargetPath(outDir) {
   }
   if (CURRENT_FILE) return CURRENT_FILE;
   const ts = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-  CURRENT_FILE = path8.join(outDir, `${ts}.ndjson`);
+  CURRENT_FILE = path9.join(outDir, `${ts}.ndjson`);
   return CURRENT_FILE;
 }
 function isEnabled() {
@@ -9336,7 +9442,7 @@ async function flushNdjson() {
 function emitNdjsonFallback(name, attrs) {
   try {
     if (!isEnabled()) return;
-    const outDir = process.env.VISOR_TRACE_DIR || path8.join(process.cwd(), "output", "traces");
+    const outDir = process.env.VISOR_TRACE_DIR || path9.join(process.cwd(), "output", "traces");
     const line = JSON.stringify({ name, attributes: attrs }) + "\n";
     appendAsync(outDir, line);
   } catch {
@@ -9345,18 +9451,18 @@ function emitNdjsonFallback(name, attrs) {
 function emitNdjsonSpanWithEvents(name, attrs, events) {
   try {
     if (!isEnabled()) return;
-    const outDir = process.env.VISOR_TRACE_DIR || path8.join(process.cwd(), "output", "traces");
+    const outDir = process.env.VISOR_TRACE_DIR || path9.join(process.cwd(), "output", "traces");
     const line = JSON.stringify({ name, attributes: attrs, events }) + "\n";
     appendAsync(outDir, line);
   } catch {
   }
 }
-var fs8, path8, CURRENT_FILE, dirReady, writeChain;
+var fs8, path9, CURRENT_FILE, dirReady, writeChain;
 var init_fallback_ndjson = __esm({
   "src/telemetry/fallback-ndjson.ts"() {
     "use strict";
     fs8 = __toESM(require("fs"));
-    path8 = __toESM(require("path"));
+    path9 = __toESM(require("path"));
     CURRENT_FILE = null;
     dirReady = false;
     writeChain = Promise.resolve();
@@ -9832,10 +9938,6 @@ var init_failure_condition_evaluator = __esm({
             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(
@@ -9850,7 +9952,6 @@ var init_failure_condition_evaluator = __esm({
           const isFirstTimer2 = permissionHelpers.isFirstTimer;
           const output = context2.output || {};
           const issues = output.issues || [];
-          const suggestions = [];
           const metadata = context2.metadata || {
             checkName: context2.checkName || "",
             schema: context2.schema || "",
@@ -9894,7 +9995,6 @@ var init_failure_condition_evaluator = __esm({
             memory: memoryAccessor,
             // Legacy compatibility variables
             issues,
-            suggestions,
             metadata,
             criticalIssues,
             errorIssues,
@@ -9923,7 +10023,6 @@ var init_failure_condition_evaluator = __esm({
             hasIssue,
             countIssues,
             hasFileMatching,
-            hasSuggestion,
             hasIssueWith,
             hasFileWith,
             // Permission helpers
@@ -10193,6 +10292,7 @@ var GitHubCheckService;
 var init_github_check_service = __esm({
   "src/github-check-service.ts"() {
     "use strict";
+    init_footer();
     GitHubCheckService = class {
       octokit;
       maxAnnotations = 50;
@@ -10433,11 +10533,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");
       }
       /**
@@ -10655,12 +10751,12 @@ function emitMermaidFromMarkdown(checkName, markdown, origin) {
         addEvent("diagram.block", { check: checkName, origin, code });
         addDiagramBlock(origin);
         if (process.env.VISOR_TRACE_REPORT === "true") {
-          const outDir = process.env.VISOR_TRACE_DIR || path9.join(process.cwd(), "output", "traces");
+          const outDir = process.env.VISOR_TRACE_DIR || path10.join(process.cwd(), "output", "traces");
           try {
             if (!fs9.existsSync(outDir)) fs9.mkdirSync(outDir, { recursive: true });
             const ts = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-            const jsonPath = path9.join(outDir, `${ts}.trace.json`);
-            const htmlPath = path9.join(outDir, `${ts}.report.html`);
+            const jsonPath = path10.join(outDir, `${ts}.trace.json`);
+            const htmlPath = path10.join(outDir, `${ts}.report.html`);
             let data = { spans: [] };
             if (fs9.existsSync(jsonPath)) {
               try {
@@ -10690,14 +10786,14 @@ function emitMermaidFromMarkdown(checkName, markdown, origin) {
   }
   return count;
 }
-var fs9, path9, MERMAID_RE;
+var fs9, path10, MERMAID_RE;
 var init_mermaid_telemetry = __esm({
   "src/utils/mermaid-telemetry.ts"() {
     "use strict";
     init_trace_helpers();
     init_metrics2();
     fs9 = __toESM(require("fs"));
-    path9 = __toESM(require("path"));
+    path10 = __toESM(require("path"));
     MERMAID_RE = /```mermaid\s*\n([\s\S]*?)\n```/gi;
   }
 });
@@ -12063,7 +12159,7 @@ ${expr}
        * - Enforcing .liquid file extension
        */
       async validateTemplatePath(templatePath) {
-        const path12 = await import("path");
+        const path13 = await import("path");
         if (!templatePath || typeof templatePath !== "string" || templatePath.trim() === "") {
           throw new Error("Template path must be a non-empty string");
         }
@@ -12073,7 +12169,7 @@ ${expr}
         if (!templatePath.endsWith(".liquid")) {
           throw new Error("Template file must have .liquid extension");
         }
-        if (path12.isAbsolute(templatePath)) {
+        if (path13.isAbsolute(templatePath)) {
           throw new Error("Template path must be relative to project directory");
         }
         if (templatePath.includes("..")) {
@@ -12087,14 +12183,14 @@ ${expr}
         if (!projectRoot || typeof projectRoot !== "string") {
           throw new Error("Unable to determine project root directory");
         }
-        const resolvedPath = path12.resolve(projectRoot, templatePath);
-        const resolvedProjectRoot = path12.resolve(projectRoot);
+        const resolvedPath = path13.resolve(projectRoot, templatePath);
+        const resolvedProjectRoot = path13.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 + path12.sep) && resolvedPath !== resolvedProjectRoot) {
+        if (!resolvedPath.startsWith(resolvedProjectRoot + path13.sep) && resolvedPath !== resolvedProjectRoot) {
           throw new Error("Template path escapes project directory");
         }
         return resolvedPath;
@@ -12139,7 +12235,7 @@ ${expr}
         }
         const { createExtendedLiquid: createExtendedLiquid2 } = await Promise.resolve().then(() => (init_liquid_extensions(), liquid_extensions_exports));
         const fs12 = await import("fs/promises");
-        const path12 = await import("path");
+        const path13 = await import("path");
         const liquid = createExtendedLiquid2({
           trimTagLeft: false,
           trimTagRight: false,
@@ -12173,7 +12269,7 @@ ${expr}
           if (!sanitizedSchema) {
             throw new Error("Invalid schema name");
           }
-          const templatePath = path12.join(__dirname, `../output/${sanitizedSchema}/template.liquid`);
+          const templatePath = path13.join(__dirname, `../output/${sanitizedSchema}/template.liquid`);
           templateContent = await fs12.readFile(templatePath, "utf-8");
           if (sanitizedSchema === "issue-assistant") {
             enrichAssistantContext = true;
@@ -16342,13 +16438,13 @@ init_check_execution_engine();
 // src/config.ts
 var yaml2 = __toESM(require("js-yaml"));
 var fs11 = __toESM(require("fs"));
-var path11 = __toESM(require("path"));
+var path12 = __toESM(require("path"));
 init_logger();
 var import_simple_git2 = __toESM(require("simple-git"));
 
 // src/utils/config-loader.ts
 var fs10 = __toESM(require("fs"));
-var path10 = __toESM(require("path"));
+var path11 = __toESM(require("path"));
 var yaml = __toESM(require("js-yaml"));
 var ConfigLoader = class {
   constructor(options = {}) {
@@ -16429,7 +16525,7 @@ var ConfigLoader = class {
         return source.toLowerCase();
       case "local" /* LOCAL */:
         const basePath = this.options.baseDir || process.cwd();
-        return path10.resolve(basePath, source);
+        return path11.resolve(basePath, source);
       default:
         return source;
     }
@@ -16439,7 +16535,7 @@ var ConfigLoader = class {
    */
   async fetchLocalConfig(filePath) {
     const basePath = this.options.baseDir || process.cwd();
-    const resolvedPath = path10.resolve(basePath, filePath);
+    const resolvedPath = path11.resolve(basePath, filePath);
     this.validateLocalPath(resolvedPath);
     if (!fs10.existsSync(resolvedPath)) {
       throw new Error(`Configuration file not found: ${resolvedPath}`);
@@ -16451,7 +16547,7 @@ var ConfigLoader = class {
         throw new Error(`Invalid YAML in configuration file: ${resolvedPath}`);
       }
       const previousBaseDir = this.options.baseDir;
-      this.options.baseDir = path10.dirname(resolvedPath);
+      this.options.baseDir = path11.dirname(resolvedPath);
       try {
         if (config.extends) {
           const processedConfig = await this.processExtends(config);
@@ -16531,14 +16627,14 @@ var ConfigLoader = class {
   async fetchDefaultConfig() {
     const possiblePaths = [
       // When running as GitHub Action (bundled in dist/)
-      path10.join(__dirname, "defaults", ".visor.yaml"),
+      path11.join(__dirname, "defaults", ".visor.yaml"),
       // When running from source
-      path10.join(__dirname, "..", "..", "defaults", ".visor.yaml"),
+      path11.join(__dirname, "..", "..", "defaults", ".visor.yaml"),
       // Try via package root
-      this.findPackageRoot() ? path10.join(this.findPackageRoot(), "defaults", ".visor.yaml") : "",
+      this.findPackageRoot() ? path11.join(this.findPackageRoot(), "defaults", ".visor.yaml") : "",
       // GitHub Action environment variable
-      process.env.GITHUB_ACTION_PATH ? path10.join(process.env.GITHUB_ACTION_PATH, "defaults", ".visor.yaml") : "",
-      process.env.GITHUB_ACTION_PATH ? path10.join(process.env.GITHUB_ACTION_PATH, "dist", "defaults", ".visor.yaml") : ""
+      process.env.GITHUB_ACTION_PATH ? path11.join(process.env.GITHUB_ACTION_PATH, "defaults", ".visor.yaml") : "",
+      process.env.GITHUB_ACTION_PATH ? path11.join(process.env.GITHUB_ACTION_PATH, "dist", "defaults", ".visor.yaml") : ""
     ].filter((p) => p);
     let defaultConfigPath;
     for (const possiblePath of possiblePaths) {
@@ -16628,8 +16724,8 @@ var ConfigLoader = class {
    */
   validateLocalPath(resolvedPath) {
     const projectRoot = this.options.projectRoot || process.cwd();
-    const normalizedPath = path10.normalize(resolvedPath);
-    const normalizedRoot = path10.normalize(projectRoot);
+    const normalizedPath = path11.normalize(resolvedPath);
+    const normalizedRoot = path11.normalize(projectRoot);
     if (!normalizedPath.startsWith(normalizedRoot)) {
       throw new Error(
         `Security error: Path traversal detected. Cannot access files outside project root: ${projectRoot}`
@@ -16655,9 +16751,9 @@ var ConfigLoader = class {
    */
   findPackageRoot() {
     let currentDir = __dirname;
-    const root = path10.parse(currentDir).root;
+    const root = path11.parse(currentDir).root;
     while (currentDir !== root) {
-      const packageJsonPath = path10.join(currentDir, "package.json");
+      const packageJsonPath = path11.join(currentDir, "package.json");
       if (fs10.existsSync(packageJsonPath)) {
         try {
           const packageJson = JSON.parse(fs10.readFileSync(packageJsonPath, "utf8"));
@@ -16667,7 +16763,7 @@ var ConfigLoader = class {
         } catch {
         }
       }
-      currentDir = path10.dirname(currentDir);
+      currentDir = path11.dirname(currentDir);
     }
     return null;
   }
@@ -16690,6 +16786,16 @@ var ConfigLoader = class {
 init_config_merger();
 var import_ajv = __toESM(require("ajv"));
 var import_ajv_formats = __toESM(require("ajv-formats"));
+var VALID_EVENT_TRIGGERS = [
+  "pr_opened",
+  "pr_updated",
+  "pr_closed",
+  "issue_opened",
+  "issue_comment",
+  "manual",
+  "schedule",
+  "webhook_received"
+];
 var ConfigManager = class {
   validCheckTypes = [
     "ai",
@@ -16698,20 +16804,13 @@ var ConfigManager = class {
     "http",
     "http_input",
     "http_client",
+    "memory",
     "noop",
     "log",
+    "memory",
     "github"
   ];
-  validEventTriggers = [
-    "pr_opened",
-    "pr_updated",
-    "pr_closed",
-    "issue_opened",
-    "issue_comment",
-    "manual",
-    "schedule",
-    "webhook_received"
-  ];
+  validEventTriggers = [...VALID_EVENT_TRIGGERS];
   validOutputFormats = ["table", "json", "markdown", "sarif"];
   validGroupByOptions = ["check", "file", "severity", "group"];
   /**
@@ -16719,24 +16818,25 @@ var ConfigManager = class {
    */
   async loadConfig(configPath, options = {}) {
     const { validate = true, mergeDefaults = true, allowedRemotePatterns } = options;
+    const resolvedPath = path12.isAbsolute(configPath) ? configPath : path12.resolve(process.cwd(), configPath);
     try {
-      if (!fs11.existsSync(configPath)) {
-        throw new Error(`Configuration file not found: ${configPath}`);
+      if (!fs11.existsSync(resolvedPath)) {
+        throw new Error(`Configuration file not found: ${resolvedPath}`);
       }
-      const configContent = fs11.readFileSync(configPath, "utf8");
+      const configContent = fs11.readFileSync(resolvedPath, "utf8");
       let parsedConfig;
       try {
         parsedConfig = yaml2.load(configContent);
       } catch (yamlError) {
         const errorMessage = yamlError instanceof Error ? yamlError.message : String(yamlError);
-        throw new Error(`Invalid YAML syntax in ${configPath}: ${errorMessage}`);
+        throw new Error(`Invalid YAML syntax in ${resolvedPath}: ${errorMessage}`);
       }
       if (!parsedConfig || typeof parsedConfig !== "object") {
         throw new Error("Configuration file must contain a valid YAML object");
       }
       if (parsedConfig.extends) {
         const loaderOptions = {
-          baseDir: path11.dirname(configPath),
+          baseDir: path12.dirname(resolvedPath),
           allowRemote: this.isRemoteExtendsAllowed(),
           maxDepth: 10,
           allowedRemotePatterns
@@ -16768,12 +16868,12 @@ var ConfigManager = class {
           throw error;
         }
         if (error.message.includes("ENOENT")) {
-          throw new Error(`Configuration file not found: ${configPath}`);
+          throw new Error(`Configuration file not found: ${resolvedPath}`);
         }
         if (error.message.includes("EPERM")) {
-          throw new Error(`Permission denied reading configuration file: ${configPath}`);
+          throw new Error(`Permission denied reading configuration file: ${resolvedPath}`);
         }
-        throw new Error(`Failed to read configuration file ${configPath}: ${error.message}`);
+        throw new Error(`Failed to read configuration file ${resolvedPath}: ${error.message}`);
       }
       throw error;
     }
@@ -16785,7 +16885,7 @@ var ConfigManager = class {
     const gitRoot = await this.findGitRepositoryRoot();
     const searchDirs = [gitRoot, process.cwd()].filter(Boolean);
     for (const baseDir of searchDirs) {
-      const possiblePaths = [path11.join(baseDir, ".visor.yaml"), path11.join(baseDir, ".visor.yml")];
+      const possiblePaths = [path12.join(baseDir, ".visor.yaml"), path12.join(baseDir, ".visor.yml")];
       for (const configPath of possiblePaths) {
         if (fs11.existsSync(configPath)) {
           return this.loadConfig(configPath, options);
@@ -16839,18 +16939,18 @@ var ConfigManager = class {
       const possiblePaths = [];
       if (typeof __dirname !== "undefined") {
         possiblePaths.push(
-          path11.join(__dirname, "defaults", ".visor.yaml"),
-          path11.join(__dirname, "..", "defaults", ".visor.yaml")
+          path12.join(__dirname, "defaults", ".visor.yaml"),
+          path12.join(__dirname, "..", "defaults", ".visor.yaml")
         );
       }
       const pkgRoot = this.findPackageRoot();
       if (pkgRoot) {
-        possiblePaths.push(path11.join(pkgRoot, "defaults", ".visor.yaml"));
+        possiblePaths.push(path12.join(pkgRoot, "defaults", ".visor.yaml"));
       }
       if (process.env.GITHUB_ACTION_PATH) {
         possiblePaths.push(
-          path11.join(process.env.GITHUB_ACTION_PATH, "defaults", ".visor.yaml"),
-          path11.join(process.env.GITHUB_ACTION_PATH, "dist", "defaults", ".visor.yaml")
+          path12.join(process.env.GITHUB_ACTION_PATH, "defaults", ".visor.yaml"),
+          path12.join(process.env.GITHUB_ACTION_PATH, "dist", "defaults", ".visor.yaml")
         );
       }
       let bundledConfigPath;
@@ -16883,8 +16983,8 @@ var ConfigManager = class {
    */
   findPackageRoot() {
     let currentDir = __dirname;
-    while (currentDir !== path11.dirname(currentDir)) {
-      const packageJsonPath = path11.join(currentDir, "package.json");
+    while (currentDir !== path12.dirname(currentDir)) {
+      const packageJsonPath = path12.join(currentDir, "package.json");
       if (fs11.existsSync(packageJsonPath)) {
         try {
           const packageJson = JSON.parse(fs11.readFileSync(packageJsonPath, "utf8"));
@@ -16894,7 +16994,7 @@ var ConfigManager = class {
         } catch {
         }
       }
-      currentDir = path11.dirname(currentDir);
+      currentDir = path12.dirname(currentDir);
     }
     return null;
   }
@@ -17279,7 +17379,7 @@ var ConfigManager = class {
     try {
       if (!__ajvValidate) {
         try {
-          const jsonPath = path11.resolve(__dirname, "generated", "config-schema.json");
+          const jsonPath = path12.resolve(__dirname, "generated", "config-schema.json");
           const jsonSchema = require(jsonPath);
           if (jsonSchema) {
             const ajv = new import_ajv.default({ allErrors: true, allowUnionTypes: true, strict: false });