@probelabs/visor 0.1.93 → 0.1.94

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

@@ -2178,7 +2178,7 @@ var PRReviewer = class {
   async reviewPR(owner, repo, prNumber, prInfo, options = {}) {
     const { debug = false, config, checks } = options;
     if (config && checks && checks.length > 0) {
-      const { CheckExecutionEngine: CheckExecutionEngine2 } = await import("./check-execution-engine-RORGGGGP.mjs");
+      const { CheckExecutionEngine: CheckExecutionEngine2 } = await import("./check-execution-engine-YBRPVUWD.mjs");
       const engine = new CheckExecutionEngine2();
       const { results } = await engine.executeGroupedChecks(
         prInfo,
@@ -2197,14 +2197,66 @@ var PRReviewer = class {
       "No configuration provided. Please create a .visor.yaml file with check definitions. Built-in prompts have been removed - all checks must be explicitly configured."
     );
   }
+  /**
+   * Helper to check if a schema definition has a "text" field in its properties
+   */
+  async schemaHasTextField(schema) {
+    try {
+      let schemaObj;
+      if (typeof schema === "object") {
+        schemaObj = schema;
+      } else {
+        const fs5 = __require("fs").promises;
+        const path5 = __require("path");
+        const sanitizedSchemaName = schema.replace(/[^a-zA-Z0-9-]/g, "");
+        if (!sanitizedSchemaName || sanitizedSchemaName !== schema) {
+          return false;
+        }
+        const schemaPath = path5.join(process.cwd(), "output", sanitizedSchemaName, "schema.json");
+        try {
+          const schemaContent = await fs5.readFile(schemaPath, "utf-8");
+          schemaObj = JSON.parse(schemaContent);
+        } catch {
+          return false;
+        }
+      }
+      const properties = schemaObj.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 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);
+      }
+      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;
       }
@@ -4167,7 +4219,8 @@ var LogCheckProvider = class extends CheckProvider {
       dependencyResults,
       includePrContext,
       includeDependencies,
-      includeMetadata
+      includeMetadata,
+      config.__outputHistory
     );
     const renderedMessage = await this.liquid.parseAndRender(message, templateContext);
     const logOutput = this.formatLogOutput(
@@ -4188,7 +4241,7 @@ var LogCheckProvider = class extends CheckProvider {
       logOutput
     };
   }
-  buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true) {
+  buildTemplateContext(prInfo, dependencyResults, _includePrContext = true, _includeDependencies = true, includeMetadata = true, outputHistory) {
     const context = {};
     context.pr = {
       number: prInfo.number,
@@ -4212,6 +4265,7 @@ var LogCheckProvider = class extends CheckProvider {
     if (dependencyResults) {
       const dependencies = {};
       const outputs = {};
+      const history = {};
       context.dependencyCount = dependencyResults.size;
       for (const [checkName, result] of dependencyResults.entries()) {
         dependencies[checkName] = {
@@ -4222,6 +4276,12 @@ var LogCheckProvider = class extends CheckProvider {
         const summary = result;
         outputs[checkName] = summary.output !== void 0 ? summary.output : summary;
       }
+      if (outputHistory) {
+        for (const [checkName, historyArray] of outputHistory) {
+          history[checkName] = historyArray;
+        }
+      }
+      outputs.history = history;
       context.dependencies = dependencies;
       context.outputs = outputs;
     }
@@ -5093,7 +5153,10 @@ var CommandCheckProvider = class extends CheckProvider {
       },
       files: prInfo.files,
       fileCount: prInfo.files.length,
-      outputs: this.buildOutputContext(dependencyResults),
+      outputs: this.buildOutputContext(
+        dependencyResults,
+        config.__outputHistory
+      ),
       env: this.getSafeEnvironmentVariables()
     };
     logger.debug(
@@ -5819,16 +5882,23 @@ ${stderrOutput}` : `Command execution failed: ${errorMessage}`;
       };
     }
   }
-  buildOutputContext(dependencyResults) {
+  buildOutputContext(dependencyResults, outputHistory) {
     if (!dependencyResults) {
       return {};
     }
     const outputs = {};
+    const history = {};
     for (const [checkName, result] of dependencyResults) {
       const summary = result;
       const value = summary.output !== void 0 ? summary.output : summary;
       outputs[checkName] = this.makeJsonSmart(value);
     }
+    if (outputHistory) {
+      for (const [checkName, historyArray] of outputHistory) {
+        history[checkName] = historyArray.map((val) => this.makeJsonSmart(val));
+      }
+    }
+    outputs.history = history;
     return outputs;
   }
   /**
@@ -6377,7 +6447,12 @@ var MemoryCheckProvider = class extends CheckProvider {
     const key = config.key;
     const namespace = config.namespace;
     const memoryStore = MemoryStore.getInstance();
-    const templateContext = this.buildTemplateContext(prInfo, dependencyResults, memoryStore);
+    const templateContext = this.buildTemplateContext(
+      prInfo,
+      dependencyResults,
+      memoryStore,
+      config.__outputHistory
+    );
     let result;
     try {
       switch (operation) {
@@ -6660,7 +6735,7 @@ var MemoryCheckProvider = class extends CheckProvider {
   /**
    * Build template context for Liquid and JS evaluation
    */
-  buildTemplateContext(prInfo, dependencyResults, memoryStore) {
+  buildTemplateContext(prInfo, dependencyResults, memoryStore, outputHistory) {
     const context = {};
     context.pr = {
       number: prInfo.number,
@@ -6679,14 +6754,21 @@ var MemoryCheckProvider = class extends CheckProvider {
         changes: f.changes
       }))
     };
+    const outputs = {};
+    const history = {};
     if (dependencyResults) {
-      const outputs = {};
       for (const [checkName, result] of dependencyResults.entries()) {
         const summary = result;
         outputs[checkName] = summary.output !== void 0 ? summary.output : summary;
       }
-      context.outputs = outputs;
     }
+    if (outputHistory) {
+      for (const [checkName, historyArray] of outputHistory) {
+        history[checkName] = historyArray;
+      }
+    }
+    outputs.history = history;
+    context.outputs = outputs;
     if (memoryStore) {
       context.memory = {
         get: (key, ns) => memoryStore.get(key, ns),
@@ -8263,6 +8345,8 @@ var CheckExecutionEngine = class _CheckExecutionEngine {
   webhookContext;
   routingSandbox;
   executionStats = /* @__PURE__ */ new Map();
+  // Track history of all outputs for each check (useful for loops and goto)
+  outputHistory = /* @__PURE__ */ new Map();
   // Event override to simulate alternate event (used during routing goto)
   routingEventOverride;
   // Cached GitHub context for context elevation when running in Actions
@@ -8506,6 +8590,8 @@ ${expr}
         transform_js: targetCfg.transform_js,
         env: targetCfg.env,
         forEach: targetCfg.forEach,
+        // Pass output history for loop/goto scenarios
+        __outputHistory: this.outputHistory,
         // Include provider-specific keys (e.g., op/values for github)
         ...targetCfg,
         ai: {
@@ -8573,6 +8659,10 @@ ${expr}
         timestamp: Date.now()
       }));
       const enriched = { ...r, issues: enrichedIssues };
+      const enrichedWithOutput = enriched;
+      if (enrichedWithOutput.output !== void 0) {
+        this.trackOutputHistory(target, enrichedWithOutput.output);
+      }
       resultsMap?.set(target, enriched);
       if (debug) log2(`\u{1F527} Debug: inline executed '${target}', issues: ${enrichedIssues.length}`);
       return enriched;
@@ -9341,6 +9431,8 @@ ${expr}
       ai_model: checkConfig.ai_model || config.ai_model,
       // Pass claude_code config if present
       claude_code: checkConfig.claude_code,
+      // Pass output history for loop/goto scenarios
+      __outputHistory: this.outputHistory,
       // Pass any provider-specific config
       ...checkConfig
     };
@@ -9377,10 +9469,14 @@ ${expr}
       }
     }
     const content = await this.renderCheckContent(checkName, result, checkConfig, prInfo);
+    let group = checkConfig.group || "default";
+    if (config?.output?.pr_comment?.group_by === "check" && !checkConfig.group) {
+      group = checkName;
+    }
     return {
       checkName,
       content,
-      group: checkConfig.group || "default",
+      group,
       output: result.output,
       debug: result.debug,
       issues: result.issues
@@ -9519,16 +9615,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] = [];
       }
@@ -10429,6 +10528,10 @@ ${expr}
                   itemResult.issues || [],
                   itemResult.output
                 );
+                const itemOutput = itemResult.output;
+                if (itemOutput !== void 0) {
+                  this.trackOutputHistory(checkName, itemOutput);
+                }
                 const descendantSet = (() => {
                   const visited = /* @__PURE__ */ new Set();
                   const stack = [checkName];
@@ -11059,6 +11162,10 @@ ${error.stack || ""}` : String(error);
             ]);
           } catch {
           }
+          const reviewResultWithOutput = reviewResult;
+          if (reviewResultWithOutput.output !== void 0) {
+            this.trackOutputHistory(checkName, reviewResultWithOutput.output);
+          }
           results.set(checkName, reviewResult);
         } else {
           const errorSummary = {
@@ -12302,6 +12409,16 @@ ${result.value.result.debug.rawResponse}`;
       stats.outputsProduced = (stats.outputsProduced || 0) + 1;
     }
   }
+  /**
+   * Track output in history for loop/goto scenarios
+   */
+  trackOutputHistory(checkName, output) {
+    if (output === void 0) return;
+    if (!this.outputHistory.has(checkName)) {
+      this.outputHistory.set(checkName, []);
+    }
+    this.outputHistory.get(checkName).push(output);
+  }
   /**
    * Record that a check was skipped
    */
@@ -12494,4 +12611,4 @@ ${result.value.result.debug.rawResponse}`;
 export {
   CheckExecutionEngine
 };
-//# sourceMappingURL=chunk-Z47UECAT.mjs.map
+//# sourceMappingURL=chunk-DQRFOQAP.mjs.map