@probelabs/visor 0.1.81 → 0.1.83

package/dist/sdk/sdk.d.mts

@@ -248,6 +248,8 @@ interface CheckConfig {
     claude_code?: ClaudeCodeConfig;
     /** Environment variables for this check */
     env?: EnvConfig;
+    /** Timeout in seconds for command execution (default: 60) */
+    timeout?: number;
     /** Check IDs that this check depends on (optional) */
     depends_on?: string[];
     /** Group name for comment separation (e.g., "code-review", "pr-overview") - optional */

package/dist/sdk/sdk.d.ts

@@ -248,6 +248,8 @@ interface CheckConfig {
     claude_code?: ClaudeCodeConfig;
     /** Environment variables for this check */
     env?: EnvConfig;
+    /** Timeout in seconds for command execution (default: 60) */
+    timeout?: number;
     /** Check IDs that this check depends on (optional) */
     depends_on?: string[];
     /** Group name for comment separation (e.g., "code-review", "pr-overview") - optional */

package/dist/sdk/sdk.js

@@ -4316,6 +4316,16 @@ return ${returnTarget};
           return result;
         } catch (error) {
           const errorMessage = error instanceof Error ? error.message : "Unknown error";
+          let isTimeout = false;
+          if (error && typeof error === "object") {
+            const execError = error;
+            if (execError.killed && execError.signal === "SIGTERM") {
+              isTimeout = true;
+            }
+            if (execError.code === "ETIMEDOUT") {
+              isTimeout = true;
+            }
+          }
           let stderrOutput = "";
           if (error && typeof error === "object") {
             const execError = error;
@@ -4323,17 +4333,32 @@ return ${returnTarget};
               stderrOutput = execError.stderr.trim();
             }
           }
-          const detailedMessage = stderrOutput ? `Command execution failed: ${errorMessage}
+          let detailedMessage;
+          let ruleId;
+          if (isTimeout) {
+            const timeoutSeconds = config.timeout || 60;
+            detailedMessage = `Command execution timed out after ${timeoutSeconds} seconds`;
+            if (stderrOutput) {
+              detailedMessage += `
+
+Stderr output:
+${stderrOutput}`;
+            }
+            ruleId = "command/timeout";
+          } else {
+            detailedMessage = stderrOutput ? `Command execution failed: ${errorMessage}
 
 Stderr output:
 ${stderrOutput}` : `Command execution failed: ${errorMessage}`;
+            ruleId = "command/execution_error";
+          }
           logger.error(`\u2717 ${detailedMessage}`);
           return {
             issues: [
               {
                 file: "command",
                 line: 0,
-                ruleId: "command/execution_error",
+                ruleId,
                 message: detailedMessage,
                 severity: "error",
                 category: "logic"
@@ -7324,172 +7349,187 @@ ${expr}
               let finalResult;
               if (isForEachDependent && forEachParentName) {
                 this.recordForEachPreview(checkName, forEachItems);
-                if (debug) {
-                  log2(
-                    `\u{1F504} Debug: Check "${checkName}" depends on forEach check "${forEachParentName}", executing ${forEachItems.length} times`
+                if (forEachItems.length === 0) {
+                  if (debug) {
+                    log2(
+                      `\u{1F504} Debug: Skipping check "${checkName}" - forEach check "${forEachParentName}" returned 0 items`
+                    );
+                  }
+                  logger.info(`  forEach: no items from "${forEachParentName}", skipping check...`);
+                  finalResult = {
+                    issues: [],
+                    output: []
+                  };
+                  finalResult.isForEach = true;
+                  finalResult.forEachItems = [];
+                } else {
+                  if (debug) {
+                    log2(
+                      `\u{1F504} Debug: Check "${checkName}" depends on forEach check "${forEachParentName}", executing ${forEachItems.length} times`
+                    );
+                  }
+                  logger.info(
+                    `  forEach: processing ${forEachItems.length} items from "${forEachParentName}"...`
                   );
-                }
-                logger.info(
-                  `  forEach: processing ${forEachItems.length} items from "${forEachParentName}"...`
-                );
-                const allIssues = [];
-                const allOutputs = [];
-                const aggregatedContents = [];
-                const itemTasks = forEachItems.map((item, itemIndex) => async () => {
-                  const forEachDependencyResults = /* @__PURE__ */ new Map();
-                  for (const [depName, depResult] of dependencyResults) {
-                    if (forEachParents.includes(depName)) {
-                      const depForEachResult = depResult;
-                      if (Array.isArray(depForEachResult.output) && depForEachResult.output[itemIndex] !== void 0) {
-                        const modifiedResult = {
-                          issues: [],
-                          output: depForEachResult.output[itemIndex]
-                        };
-                        forEachDependencyResults.set(depName, modifiedResult);
-                        const rawResult = {
-                          issues: [],
-                          output: depForEachResult.output
-                        };
-                        forEachDependencyResults.set(`${depName}-raw`, rawResult);
+                  const allIssues = [];
+                  const allOutputs = [];
+                  const aggregatedContents = [];
+                  const itemTasks = forEachItems.map((item, itemIndex) => async () => {
+                    const forEachDependencyResults = /* @__PURE__ */ new Map();
+                    for (const [depName, depResult] of dependencyResults) {
+                      if (forEachParents.includes(depName)) {
+                        const depForEachResult = depResult;
+                        if (Array.isArray(depForEachResult.output) && depForEachResult.output[itemIndex] !== void 0) {
+                          const modifiedResult = {
+                            issues: [],
+                            output: depForEachResult.output[itemIndex]
+                          };
+                          forEachDependencyResults.set(depName, modifiedResult);
+                          const rawResult = {
+                            issues: [],
+                            output: depForEachResult.output
+                          };
+                          forEachDependencyResults.set(`${depName}-raw`, rawResult);
+                        } else {
+                          forEachDependencyResults.set(depName, depResult);
+                        }
                       } else {
                         forEachDependencyResults.set(depName, depResult);
                       }
-                    } else {
-                      forEachDependencyResults.set(depName, depResult);
                     }
-                  }
-                  if (checkConfig.if) {
-                    const conditionResults = new Map(results);
-                    for (const [depName, depResult] of forEachDependencyResults) {
-                      conditionResults.set(depName, depResult);
+                    if (checkConfig.if) {
+                      const conditionResults = new Map(results);
+                      for (const [depName, depResult] of forEachDependencyResults) {
+                        conditionResults.set(depName, depResult);
+                      }
+                      const shouldRun = await this.evaluateCheckCondition(
+                        checkName,
+                        checkConfig.if,
+                        prInfo,
+                        conditionResults,
+                        debug
+                      );
+                      if (!shouldRun) {
+                        if (debug) {
+                          log2(
+                            `\u{1F504} Debug: Skipping forEach item ${itemIndex + 1} for check "${checkName}" (if condition evaluated to false)`
+                          );
+                        }
+                        return {
+                          index: itemIndex,
+                          itemResult: { issues: [] },
+                          skipped: true
+                        };
+                      }
+                    }
+                    if (debug) {
+                      log2(
+                        `\u{1F504} Debug: Executing check "${checkName}" for item ${itemIndex + 1}/${forEachItems.length}`
+                      );
                     }
-                    const shouldRun = await this.evaluateCheckCondition(
+                    const iterationStart = this.recordIterationStart(checkName);
+                    const itemResult = await this.executeWithRouting(
                       checkName,
-                      checkConfig.if,
+                      checkConfig,
+                      provider,
+                      providerConfig,
                       prInfo,
-                      conditionResults,
-                      debug
+                      forEachDependencyResults,
+                      sessionInfo,
+                      config,
+                      dependencyGraph,
+                      debug,
+                      results,
+                      /*foreachContext*/
+                      {
+                        index: itemIndex,
+                        total: forEachItems.length,
+                        parent: forEachParentName
+                      }
+                    );
+                    const hadFatalError = (itemResult.issues || []).some((issue) => {
+                      const id = issue.ruleId || "";
+                      return id === "command/execution_error" || id.endsWith("/command/execution_error") || id === "command/transform_js_error" || id.endsWith("/command/transform_js_error") || id === "command/transform_error" || id.endsWith("/command/transform_error");
+                    });
+                    const iterationDuration = (Date.now() - iterationStart) / 1e3;
+                    this.recordIterationComplete(
+                      checkName,
+                      iterationStart,
+                      !hadFatalError,
+                      // Success if no fatal errors
+                      itemResult.issues || [],
+                      itemResult.output
+                    );
+                    logger.info(
+                      `  \u2714 ${itemIndex + 1}/${forEachItems.length} (${iterationDuration.toFixed(1)}s)`
+                    );
+                    return { index: itemIndex, itemResult };
+                  });
+                  const forEachConcurrency = Math.max(
+                    1,
+                    Math.min(forEachItems.length, effectiveMaxParallelism)
+                  );
+                  if (debug && forEachConcurrency > 1) {
+                    log2(
+                      `\u{1F504} Debug: Limiting forEach concurrency for check "${checkName}" to ${forEachConcurrency}`
                     );
-                    if (!shouldRun) {
+                  }
+                  const forEachResults = await this.executeWithLimitedParallelism(
+                    itemTasks,
+                    forEachConcurrency,
+                    false
+                  );
+                  for (const result of forEachResults) {
+                    if (result.status === "rejected") {
+                      const error = result.reason;
+                      const errorMessage = error instanceof Error ? error.message : String(error);
+                      allIssues.push({
+                        ruleId: `${checkName}/forEach/iteration_error`,
+                        severity: "error",
+                        category: "logic",
+                        message: `forEach iteration failed: ${errorMessage}`,
+                        file: "",
+                        line: 0
+                      });
                       if (debug) {
                         log2(
-                          `\u{1F504} Debug: Skipping forEach item ${itemIndex + 1} for check "${checkName}" (if condition evaluated to false)`
+                          `\u{1F504} Debug: forEach iteration for check "${checkName}" failed: ${errorMessage}`
                         );
                       }
-                      return {
-                        index: itemIndex,
-                        itemResult: { issues: [] },
-                        skipped: true
-                      };
+                      continue;
                     }
-                  }
-                  if (debug) {
-                    log2(
-                      `\u{1F504} Debug: Executing check "${checkName}" for item ${itemIndex + 1}/${forEachItems.length}`
-                    );
-                  }
-                  const iterationStart = this.recordIterationStart(checkName);
-                  const itemResult = await this.executeWithRouting(
-                    checkName,
-                    checkConfig,
-                    provider,
-                    providerConfig,
-                    prInfo,
-                    forEachDependencyResults,
-                    sessionInfo,
-                    config,
-                    dependencyGraph,
-                    debug,
-                    results,
-                    /*foreachContext*/
-                    {
-                      index: itemIndex,
-                      total: forEachItems.length,
-                      parent: forEachParentName
+                    if (result.value.skipped) {
+                      continue;
                     }
-                  );
-                  const hadFatalError = (itemResult.issues || []).some((issue) => {
-                    const id = issue.ruleId || "";
-                    return id === "command/execution_error" || id.endsWith("/command/execution_error") || id === "command/transform_js_error" || id.endsWith("/command/transform_js_error") || id === "command/transform_error" || id.endsWith("/command/transform_error");
-                  });
-                  const iterationDuration = (Date.now() - iterationStart) / 1e3;
-                  this.recordIterationComplete(
-                    checkName,
-                    iterationStart,
-                    !hadFatalError,
-                    // Success if no fatal errors
-                    itemResult.issues || [],
-                    itemResult.output
-                  );
-                  logger.info(
-                    `  \u2714 ${itemIndex + 1}/${forEachItems.length} (${iterationDuration.toFixed(1)}s)`
-                  );
-                  return { index: itemIndex, itemResult };
-                });
-                const forEachConcurrency = Math.max(
-                  1,
-                  Math.min(forEachItems.length, effectiveMaxParallelism)
-                );
-                if (debug && forEachConcurrency > 1) {
-                  log2(
-                    `\u{1F504} Debug: Limiting forEach concurrency for check "${checkName}" to ${forEachConcurrency}`
-                  );
-                }
-                const forEachResults = await this.executeWithLimitedParallelism(
-                  itemTasks,
-                  forEachConcurrency,
-                  false
-                );
-                for (const result of forEachResults) {
-                  if (result.status === "rejected") {
-                    const error = result.reason;
-                    const errorMessage = error instanceof Error ? error.message : String(error);
-                    allIssues.push({
-                      ruleId: `${checkName}/forEach/iteration_error`,
-                      severity: "error",
-                      category: "logic",
-                      message: `forEach iteration failed: ${errorMessage}`,
-                      file: "",
-                      line: 0
-                    });
-                    if (debug) {
-                      log2(
-                        `\u{1F504} Debug: forEach iteration for check "${checkName}" failed: ${errorMessage}`
-                      );
+                    const { itemResult } = result.value;
+                    if (itemResult.issues) {
+                      allIssues.push(...itemResult.issues);
+                    }
+                    const resultWithOutput = itemResult;
+                    if (resultWithOutput.output !== void 0) {
+                      allOutputs.push(resultWithOutput.output);
+                    }
+                    const itemContent = resultWithOutput.content;
+                    if (typeof itemContent === "string" && itemContent.trim()) {
+                      aggregatedContents.push(itemContent.trim());
                     }
-                    continue;
-                  }
-                  if (result.value.skipped) {
-                    continue;
-                  }
-                  const { itemResult } = result.value;
-                  if (itemResult.issues) {
-                    allIssues.push(...itemResult.issues);
                   }
-                  const resultWithOutput = itemResult;
-                  if (resultWithOutput.output !== void 0) {
-                    allOutputs.push(resultWithOutput.output);
+                  const finalOutput = allOutputs.length > 0 ? allOutputs : void 0;
+                  finalResult = {
+                    issues: allIssues,
+                    ...finalOutput !== void 0 ? { output: finalOutput } : {}
+                  };
+                  if (allOutputs.length > 0) {
+                    finalResult.isForEach = true;
+                    finalResult.forEachItems = allOutputs;
                   }
-                  const itemContent = resultWithOutput.content;
-                  if (typeof itemContent === "string" && itemContent.trim()) {
-                    aggregatedContents.push(itemContent.trim());
+                  if (aggregatedContents.length > 0) {
+                    finalResult.content = aggregatedContents.join("\n");
                   }
+                  log2(
+                    `\u{1F504} Debug: Completed forEach execution for check "${checkName}", total issues: ${allIssues.length}`
+                  );
                 }
-                const finalOutput = allOutputs.length > 0 ? allOutputs : void 0;
-                finalResult = {
-                  issues: allIssues,
-                  ...finalOutput !== void 0 ? { output: finalOutput } : {}
-                };
-                if (allOutputs.length > 0) {
-                  finalResult.isForEach = true;
-                  finalResult.forEachItems = allOutputs;
-                }
-                if (aggregatedContents.length > 0) {
-                  finalResult.content = aggregatedContents.join("\n");
-                }
-                log2(
-                  `\u{1F504} Debug: Completed forEach execution for check "${checkName}", total issues: ${allIssues.length}`
-                );
               } else {
                 if (checkConfig.if) {
                   const shouldRun = await this.evaluateCheckCondition(
@@ -9353,6 +9393,10 @@ var init_config_schema = __esm({
               $ref: "#/definitions/EnvConfig",
               description: "Environment variables for this check"
             },
+            timeout: {
+              type: "number",
+              description: "Timeout in seconds for command execution (default: 60)"
+            },
             depends_on: {
               type: "array",
               items: {