@probelabs/visor 0.1.93 → 0.1.95

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();
@@ -2424,14 +2530,66 @@ var init_reviewer = __esm({
           "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 fs12 = require("fs").promises;
+            const path13 = require("path");
+            const sanitizedSchemaName = schema.replace(/[^a-zA-Z0-9-]/g, "");
+            if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
+              return false;
+            }
+            const schemaPath = path13.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+            try {
+              const schemaContent = await fs12.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;
           }
@@ -2564,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`,
             ``,
@@ -2604,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 {
@@ -2824,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);
           }
@@ -3304,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;
@@ -3377,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");
@@ -5127,7 +5285,8 @@ var init_log_check_provider = __esm({
           dependencyResults,
           includePrContext,
           includeDependencies,
-          includeMetadata
+          includeMetadata,
+          config.__outputHistory
         );
         const renderedMessage = await this.liquid.parseAndRender(message, templateContext);
         const logOutput = this.formatLogOutput(
@@ -5148,7 +5307,7 @@ var init_log_check_provider = __esm({
           logOutput
         };
       }
-      buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true) {
+      buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true, outputHistory) {
         const context2 = {};
         context2.pr = {
           number: prInfo.number,
@@ -5172,6 +5331,7 @@ var init_log_check_provider = __esm({
         if (dependencyResults) {
           const dependencies = {};
           const outputs = {};
+          const history = {};
           context2.dependencyCount = dependencyResults.size;
           for (const [checkName, result] of dependencyResults.entries()) {
             dependencies[checkName] = {
@@ -5182,6 +5342,12 @@ var init_log_check_provider = __esm({
             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;
           context2.dependencies = dependencies;
           context2.outputs = outputs;
         }
@@ -6084,7 +6250,10 @@ var init_command_check_provider = __esm({
           },
           files: prInfo.files,
           fileCount: prInfo.files.length,
-          outputs: this.buildOutputContext(dependencyResults),
+          outputs: this.buildOutputContext(
+            dependencyResults,
+            config.__outputHistory
+          ),
           env: this.getSafeEnvironmentVariables()
         };
         logger.debug(
@@ -6810,16 +6979,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;
       }
       /**
@@ -7377,7 +7553,12 @@ var init_memory_check_provider = __esm({
         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) {
@@ -7660,7 +7841,7 @@ var init_memory_check_provider = __esm({
       /**
        * Build template context for Liquid and JS evaluation
        */
-      buildTemplateContext(prInfo, dependencyResults, memoryStore) {
+      buildTemplateContext(prInfo, dependencyResults, memoryStore, outputHistory) {
         const context2 = {};
         context2.pr = {
           number: prInfo.number,
@@ -7679,14 +7860,21 @@ var init_memory_check_provider = __esm({
             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;
           }
-          context2.outputs = outputs;
         }
+        if (outputHistory) {
+          for (const [checkName, historyArray] of outputHistory) {
+            history[checkName] = historyArray;
+          }
+        }
+        outputs.history = history;
+        context2.outputs = outputs;
         if (memoryStore) {
           context2.memory = {
             get: (key, ns) => memoryStore.get(key, ns),
@@ -9224,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() {
@@ -9254,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 {
@@ -9263,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();
@@ -9750,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(
@@ -9768,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 || "",
@@ -9812,7 +9995,6 @@ var init_failure_condition_evaluator = __esm({
             memory: memoryAccessor,
             // Legacy compatibility variables
             issues,
-            suggestions,
             metadata,
             criticalIssues,
             errorIssues,
@@ -9841,7 +10023,6 @@ var init_failure_condition_evaluator = __esm({
             hasIssue,
             countIssues,
             hasFileMatching,
-            hasSuggestion,
             hasIssueWith,
             hasFileWith,
             // Permission helpers
@@ -10111,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;
@@ -10351,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");
       }
       /**
@@ -10573,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 {
@@ -10608,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;
   }
 });
@@ -10681,6 +10859,8 @@ var init_check_execution_engine = __esm({
       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
@@ -10924,6 +11104,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: {
@@ -10991,6 +11173,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;
@@ -11759,6 +11945,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
         };
@@ -11795,10 +11983,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
@@ -11937,16 +12129,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] = [];
           }
@@ -11964,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");
         }
@@ -11974,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("..")) {
@@ -11988,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;
@@ -12040,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,
@@ -12074,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;
@@ -12847,6 +13042,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];
@@ -13477,6 +13676,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 = {
@@ -14720,6 +14923,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
        */
@@ -16225,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 = {}) {
@@ -16312,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;
     }
@@ -16322,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}`);
@@ -16334,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);
@@ -16414,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) {
@@ -16511,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}`
@@ -16538,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"));
@@ -16550,7 +16763,7 @@ var ConfigLoader = class {
         } catch {
         }
       }
-      currentDir = path10.dirname(currentDir);
+      currentDir = path11.dirname(currentDir);
     }
     return null;
   }
@@ -16573,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",
@@ -16581,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"];
   /**
@@ -16602,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
@@ -16651,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;
     }
@@ -16668,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);
@@ -16722,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;
@@ -16766,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"));
@@ -16777,7 +16994,7 @@ var ConfigManager = class {
         } catch {
         }
       }
-      currentDir = path11.dirname(currentDir);
+      currentDir = path12.dirname(currentDir);
     }
     return null;
   }
@@ -16823,8 +17040,10 @@ var ConfigManager = class {
   }
   /**
    * Validate configuration against schema
+   * @param config The config to validate
+   * @param strict If true, treat warnings as errors (default: false)
    */
-  validateConfig(config) {
+  validateConfig(config, strict = false) {
     const errors = [];
     const warnings = [];
     this.validateWithAjvSchema(config, errors, warnings);
@@ -16932,10 +17151,13 @@ var ConfigManager = class {
     if (config.tag_filter) {
       this.validateTagFilter(config.tag_filter, errors);
     }
+    if (strict && warnings.length > 0) {
+      errors.push(...warnings);
+    }
     if (errors.length > 0) {
       throw new Error(errors[0].message);
     }
-    if (warnings.length > 0) {
+    if (!strict && warnings.length > 0) {
       for (const w of warnings) {
         logger.warn(`\u26A0\uFE0F  Config warning [${w.field}]: ${w.message}`);
       }
@@ -17157,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 });
@@ -17391,9 +17613,25 @@ var __ajvValidate = null;
 var __ajvErrors = null;
 
 // src/sdk.ts
-async function loadConfig(configPath) {
+async function loadConfig(configOrPath, options) {
   const cm = new ConfigManager();
-  if (configPath) return cm.loadConfig(configPath);
+  if (typeof configOrPath === "object" && configOrPath !== null) {
+    cm.validateConfig(configOrPath, options?.strict ?? false);
+    const defaultConfig = {
+      version: "1.0",
+      checks: {},
+      max_parallelism: 3,
+      fail_fast: false
+    };
+    return {
+      ...defaultConfig,
+      ...configOrPath,
+      checks: configOrPath.checks || {}
+    };
+  }
+  if (typeof configOrPath === "string") {
+    return cm.loadConfig(configOrPath);
+  }
   return cm.findAndLoadConfig();
 }
 function resolveChecks(checkIds, config) {
@@ -17419,7 +17657,15 @@ function resolveChecks(checkIds, config) {
 }
 async function runChecks(opts = {}) {
   const cm = new ConfigManager();
-  const config = opts.config ? opts.config : opts.configPath ? await cm.loadConfig(opts.configPath) : await cm.findAndLoadConfig();
+  let config;
+  if (opts.config) {
+    cm.validateConfig(opts.config, opts.strictValidation ?? false);
+    config = opts.config;
+  } else if (opts.configPath) {
+    config = await cm.loadConfig(opts.configPath);
+  } else {
+    config = await cm.findAndLoadConfig();
+  }
   const checks = opts.checks && opts.checks.length > 0 ? resolveChecks(opts.checks, config) : Object.keys(config.checks || {});
   const engine = new CheckExecutionEngine(opts.cwd);
   const result = await engine.executeChecks({