@probelabs/visor 0.1.93 → 0.1.94

package/dist/sdk/sdk.d.mts

@@ -659,9 +659,18 @@ interface RunOptions extends VisorOptions {
     output?: {
         format?: 'table' | 'json' | 'markdown' | 'sarif';
     };
+    /** Strict mode: treat config warnings (like unknown keys) as errors (default: false) */
+    strictValidation?: boolean;
 }
-/** Load a Visor config from path, or discover defaults if path is omitted. */
-declare function loadConfig(configPath?: string): Promise<VisorConfig>;
+/**
+ * Load and validate a Visor config.
+ * @param configOrPath - Config object, file path, or omit to discover defaults
+ * @param options - Validation options
+ * @returns Validated config with defaults applied
+ */
+declare function loadConfig(configOrPath?: string | Partial<VisorConfig>, options?: {
+    strict?: boolean;
+}): Promise<VisorConfig>;
 /** Expand check IDs by including their dependencies (shallow->deep). */
 declare function resolveChecks(checkIds: string[], config: VisorConfig | undefined): string[];
 /**

package/dist/sdk/sdk.d.ts

@@ -659,9 +659,18 @@ interface RunOptions extends VisorOptions {
     output?: {
         format?: 'table' | 'json' | 'markdown' | 'sarif';
     };
+    /** Strict mode: treat config warnings (like unknown keys) as errors (default: false) */
+    strictValidation?: boolean;
 }
-/** Load a Visor config from path, or discover defaults if path is omitted. */
-declare function loadConfig(configPath?: string): Promise<VisorConfig>;
+/**
+ * Load and validate a Visor config.
+ * @param configOrPath - Config object, file path, or omit to discover defaults
+ * @param options - Validation options
+ * @returns Validated config with defaults applied
+ */
+declare function loadConfig(configOrPath?: string | Partial<VisorConfig>, options?: {
+    strict?: boolean;
+}): Promise<VisorConfig>;
 /** Expand check IDs by including their dependencies (shallow->deep). */
 declare function resolveChecks(checkIds: string[], config: VisorConfig | undefined): string[];
 /**

package/dist/sdk/sdk.js

@@ -2424,14 +2424,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 definition has a "text" field in its properties
+       */
+      async schemaHasTextField(schema) {
+        try {
+          let schemaObj;
+          if (typeof schema === "object") {
+            schemaObj = schema;
+          } else {
+            const fs12 = require("fs").promises;
+            const path12 = 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");
+            try {
+              const schemaContent = await fs12.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;
           }
@@ -5127,7 +5179,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 +5201,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 +5225,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 +5236,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 +6144,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 +6873,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 +7447,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 +7735,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 +7754,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),
@@ -10681,6 +10763,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 +11008,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 +11077,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 +11849,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 +11887,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 +12033,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] = [];
           }
@@ -12847,6 +12946,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 +13580,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 +14827,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
        */
@@ -16823,8 +16940,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 +17051,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}`);
       }
@@ -17391,9 +17513,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 +17557,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({