@probelabs/visor 0.1.125 → 0.1.127

package/dist/sdk/{chunk-A4PGHURG.mjs → chunk-TS6BUNAI.mjs}

@@ -16,11 +16,11 @@ import {
   WorkflowRegistry,
   init_dependency_resolver,
   init_workflow_registry
-} from "./chunk-J6EVEXC2.mjs";
+} from "./chunk-VW2GBXQT.mjs";
 import {
   config_exports,
   init_config
-} from "./chunk-RTKJXNZS.mjs";
+} from "./chunk-XWJPT5KQ.mjs";
 import {
   getPromptStateManager,
   init_prompt_state
@@ -32,16 +32,11 @@ import {
   init_routing,
   init_snapshot_store,
   snapshot_store_exports
-} from "./chunk-PXFIALUH.mjs";
+} from "./chunk-EORMDOZU.mjs";
 import {
   FailureConditionEvaluator,
   init_failure_condition_evaluator
-} from "./chunk-SWEEZ5D5.mjs";
-import {
-  compileAndRun,
-  createSecureSandbox,
-  init_sandbox
-} from "./chunk-BOVFH3LI.mjs";
+} from "./chunk-MPS4HVQI.mjs";
 import {
   addEvent,
   emitNdjsonFallback,
@@ -59,13 +54,18 @@ import {
 import {
   createExtendedLiquid,
   init_liquid_extensions
-} from "./chunk-7GUAFV6L.mjs";
+} from "./chunk-BHOKBQPB.mjs";
 import {
   createPermissionHelpers,
   detectLocalMode,
   init_author_permissions,
   resolveAssociationFromEvent
 } from "./chunk-CNX7V5JK.mjs";
+import {
+  compileAndRun,
+  createSecureSandbox,
+  init_sandbox
+} from "./chunk-BOVFH3LI.mjs";
 import {
   MemoryStore,
   init_memory_store,
@@ -1534,6 +1534,9 @@ ${"=".repeat(60)}
             // Use systemPrompt (native in rc168+) with fallback to customPrompt for backward compat
             systemPrompt: systemPrompt || this.config.systemPrompt || this.config.customPrompt
           };
+          if (this.config.maxIterations !== void 0) {
+            options.maxIterations = this.config.maxIterations;
+          }
           let traceFilePath = "";
           let telemetryConfig = null;
           let probeFileTracer = null;
@@ -2023,9 +2026,12 @@ ${"=".repeat(60)}
               const errMsg = parseErr instanceof Error ? parseErr.message : String(parseErr);
               log(`\u{1F50D} Direct JSON parsing failed: ${errMsg}`);
               if (response.toLowerCase().includes("i cannot") || response.toLowerCase().includes("unable to")) {
-                console.error("\u{1F6AB} AI refused to analyze - returning empty result");
+                console.error("\u{1F6AB} AI refused to analyze - returning refusal as output");
+                const trimmed2 = response.trim();
                 return {
-                  issues: []
+                  issues: [],
+                  output: trimmed2 ? { text: trimmed2 } : {},
+                  debug: debugInfo
                 };
               }
               log("\u{1F527} Treating response as plain text (no JSON extraction)");
@@ -2844,9 +2850,7 @@ var init_custom_tool_executor = __esm({
        * Apply JavaScript transform to output
        */
       async applyJavaScriptTransform(transformJs, output, context2) {
-        if (!this.sandbox) {
-          this.sandbox = createSecureSandbox();
-        }
+        this.sandbox = createSecureSandbox();
         const code = `
       const output = ${JSON.stringify(output)};
       const context = ${JSON.stringify(context2)};
@@ -2977,7 +2981,7 @@ async function executeWorkflowAsTool(workflowId, args, context2, argsOverrides)
     ...args,
     ...argsOverrides
   };
-  const { WorkflowCheckProvider: WorkflowCheckProvider2 } = await import("./workflow-check-provider-YUNNF4KC.mjs");
+  const { WorkflowCheckProvider: WorkflowCheckProvider2 } = await import("./workflow-check-provider-3IWBAZP7.mjs");
   const provider = new WorkflowCheckProvider2();
   const checkConfig = {
     type: "workflow",
@@ -4097,6 +4101,10 @@ ${preview}`);
           if (aiAny2.timeout !== void 0) {
             aiConfig.timeout = aiAny2.timeout;
           }
+          if (aiAny2.max_iterations !== void 0 || aiAny2.maxIterations !== void 0) {
+            const raw = aiAny2.max_iterations ?? aiAny2.maxIterations;
+            aiConfig.maxIterations = Number(raw);
+          }
           if (aiAny2.provider !== void 0) {
             aiConfig.provider = aiAny2.provider;
           }
@@ -4231,6 +4239,9 @@ ${preview}`);
         if (config.ai_provider !== void 0) {
           aiConfig.provider = config.ai_provider;
         }
+        if (config.ai_max_iterations !== void 0 && aiConfig.maxIterations === void 0) {
+          aiConfig.maxIterations = config.ai_max_iterations;
+        }
         const customPrompt = config.prompt;
         if (!customPrompt) {
           throw new Error(
@@ -4633,6 +4644,7 @@ ${processedPrompt}` : processedPrompt;
           "ai.model",
           "ai.apiKey",
           "ai.timeout",
+          "ai.max_iterations",
           "ai.mcpServers",
           "ai.enableDelegate",
           // legacy persona/prompt keys supported in config
@@ -4640,6 +4652,7 @@ ${processedPrompt}` : processedPrompt;
           "ai_prompt_type",
           "ai_custom_prompt",
           "ai_system_prompt",
+          "ai_max_iterations",
           // new provider resilience and tools toggles
           "ai.retry",
           "ai.fallback",
@@ -5326,9 +5339,7 @@ var init_http_client_provider = __esm({
           }
           if (transformJs) {
             try {
-              if (!this.sandbox) {
-                this.sandbox = this.createSecureSandbox();
-              }
+              this.sandbox = this.createSecureSandbox();
               const jsScope = {
                 output: data,
                 pr: templateContext.pr,
@@ -7179,23 +7190,6 @@ ${bodyWithReturn}
                   { injectLog: false, wrapFunction: false }
                 );
               }
-              try {
-                if (finalOutput && typeof finalOutput === "object" && !Array.isArray(finalOutput) && (finalOutput.error === void 0 || finalOutput.issues === void 0)) {
-                  const vm = await import("vm");
-                  const vmContext = vm.createContext({ scope: jsContext });
-                  const vmCode = `
-                (function(){
-                  const output = scope.output; const pr = scope.pr; const files = scope.files; const outputs = scope.outputs; const env = scope.env; const log = ()=>{};
-${bodyWithReturn}
-                })()
-              `;
-                  const vmResult = vm.runInContext(vmCode, vmContext, { timeout: 1e3 });
-                  if (vmResult && typeof vmResult === "object") {
-                    finalOutput = vmResult;
-                  }
-                }
-              } catch {
-              }
               let finalSnapshot = null;
               try {
                 if (finalOutput && typeof finalOutput === "object" && !Array.isArray(finalOutput)) {
@@ -7682,6 +7676,11 @@ ${bodyWithReturn}
               isTimeout = true;
             }
           }
+          if (!isTimeout && error instanceof Error) {
+            if (/timed out/i.test(error.message)) {
+              isTimeout = true;
+            }
+          }
           let stderrOutput = "";
           if (error && typeof error === "object") {
             const execError = error;
@@ -8443,9 +8442,7 @@ var init_memory_check_provider = __esm({
        * Evaluate JavaScript expression in context using SandboxJS for secure execution
        */
       evaluateJavaScript(expression, context2) {
-        if (!this.sandbox) {
-          this.sandbox = this.createSecureSandbox();
-        }
+        this.sandbox = this.createSecureSandbox();
         try {
           const scope = { ...context2 };
           return compileAndRun(this.sandbox, `return (${expression});`, scope, {
@@ -8678,9 +8675,7 @@ var init_mcp_check_provider = __esm({
           }
           if (cfg.transform_js) {
             try {
-              if (!this.sandbox) {
-                this.sandbox = this.createSecureSandbox();
-              }
+              this.sandbox = this.createSecureSandbox();
               const scope = {
                 output: finalOutput,
                 pr: templateContext.pr,
@@ -10273,6 +10268,53 @@ var init_worktree_manager = __esm({
           const metadata2 = await this.loadMetadata(worktreePath);
           if (metadata2) {
             if (metadata2.ref === ref) {
+              try {
+                const bareRepoPath2 = metadata2.bare_repo_path || await this.getOrCreateBareRepo(
+                  repository,
+                  repoUrl,
+                  options.token,
+                  options.fetchDepth,
+                  options.cloneTimeoutMs
+                );
+                const fetched2 = await this.fetchRef(bareRepoPath2, ref);
+                if (fetched2) {
+                  const latestCommit = await this.getCommitShaForRef(bareRepoPath2, ref);
+                  if (latestCommit && latestCommit !== metadata2.commit) {
+                    logger.info(
+                      `Worktree ref ${ref} advanced (${metadata2.commit} -> ${latestCommit}), updating...`
+                    );
+                    const checkoutCmd = `git -C ${this.escapeShellArg(worktreePath)} checkout --detach ${this.escapeShellArg(latestCommit)}`;
+                    const checkoutResult = await this.executeGitCommand(checkoutCmd, {
+                      timeout: 6e4
+                    });
+                    if (checkoutResult.exitCode !== 0) {
+                      throw new Error(`Failed to checkout updated ref: ${checkoutResult.stderr}`);
+                    }
+                    const updatedMetadata = {
+                      ...metadata2,
+                      commit: latestCommit,
+                      created_at: (/* @__PURE__ */ new Date()).toISOString()
+                    };
+                    await this.saveMetadata(worktreePath, updatedMetadata);
+                    if (options.clean) {
+                      logger.debug(`Cleaning updated worktree`);
+                      await this.cleanWorktree(worktreePath);
+                    }
+                    this.activeWorktrees.set(worktreeId, updatedMetadata);
+                    return {
+                      id: worktreeId,
+                      path: worktreePath,
+                      ref: updatedMetadata.ref,
+                      commit: updatedMetadata.commit,
+                      metadata: updatedMetadata,
+                      locked: false
+                    };
+                  }
+                }
+              } catch (error) {
+                const errorMessage = error instanceof Error ? error.message : String(error);
+                logger.warn(`Failed to refresh worktree, will reuse existing: ${errorMessage}`);
+              }
               if (options.clean) {
                 logger.debug(`Cleaning existing worktree`);
                 await this.cleanWorktree(worktreePath);
@@ -10298,8 +10340,11 @@ var init_worktree_manager = __esm({
                   options.fetchDepth,
                   options.cloneTimeoutMs
                 );
-                await this.fetchRef(bareRepoPath2, ref);
+                const fetched2 = await this.fetchRef(bareRepoPath2, ref);
                 const newCommit = await this.getCommitShaForRef(bareRepoPath2, ref);
+                if (!fetched2) {
+                  logger.warn(`Using cached ref ${ref} for update; fetch failed`);
+                }
                 const checkoutCmd = `git -C ${this.escapeShellArg(worktreePath)} checkout --detach ${this.escapeShellArg(newCommit)}`;
                 const checkoutResult = await this.executeGitCommand(checkoutCmd, { timeout: 6e4 });
                 if (checkoutResult.exitCode !== 0) {
@@ -10337,8 +10382,11 @@ var init_worktree_manager = __esm({
             await fsp.rm(worktreePath, { recursive: true, force: true });
           }
         }
-        await this.fetchRef(bareRepoPath, ref);
+        const fetched = await this.fetchRef(bareRepoPath, ref);
         const commit = await this.getCommitShaForRef(bareRepoPath, ref);
+        if (!fetched) {
+          logger.warn(`Using cached ref ${ref}; fetch failed`);
+        }
         await this.pruneWorktrees(bareRepoPath);
         logger.info(`Creating worktree for ${repository}@${ref} (${commit})`);
         const createCmd = `git -C ${this.escapeShellArg(
@@ -10392,8 +10440,13 @@ var init_worktree_manager = __esm({
       async fetchRef(bareRepoPath, ref) {
         this.validateRef(ref);
         logger.debug(`Fetching ref: ${ref}`);
-        const fetchCmd = `git -C ${this.escapeShellArg(bareRepoPath)} fetch origin ${this.escapeShellArg(ref + ":" + ref)} 2>&1 || true`;
-        await this.executeGitCommand(fetchCmd, { timeout: 6e4 });
+        const fetchCmd = `git -C ${this.escapeShellArg(bareRepoPath)} fetch origin ${this.escapeShellArg(ref + ":" + ref)} 2>&1`;
+        const result = await this.executeGitCommand(fetchCmd, { timeout: 6e4 });
+        if (result.exitCode !== 0) {
+          logger.warn(`Failed to fetch ref ${ref}: ${result.stderr}`);
+          return false;
+        }
+        return true;
       }
       /**
        * Clean worktree (reset and remove untracked files)
@@ -11170,7 +11223,6 @@ var init_check_provider_registry = __esm({
 });
 
 // src/workflow-executor.ts
-import { Liquid } from "liquidjs";
 var WorkflowExecutor;
 var init_workflow_executor = __esm({
   "src/workflow-executor.ts"() {
@@ -11179,11 +11231,12 @@ var init_workflow_executor = __esm({
     init_dependency_resolver();
     init_logger();
     init_sandbox();
+    init_liquid_extensions();
     WorkflowExecutor = class {
       providerRegistry = null;
       liquid;
       constructor() {
-        this.liquid = new Liquid();
+        this.liquid = createExtendedLiquid();
       }
       /**
        * Lazy-load the provider registry to avoid circular dependency during initialization
@@ -12623,7 +12676,7 @@ var init_dependency_gating = __esm({
 // src/state-machine/dispatch/template-renderer.ts
 async function renderTemplateContent(checkId, checkConfig, reviewSummary) {
   try {
-    const { createExtendedLiquid: createExtendedLiquid2 } = await import("./liquid-extensions-DFDEBMUI.mjs");
+    const { createExtendedLiquid: createExtendedLiquid2 } = await import("./liquid-extensions-I7O7KMHF.mjs");
     const fs8 = await import("fs/promises");
     const path9 = await import("path");
     const schemaRaw = checkConfig.schema || "plain";
@@ -12700,7 +12753,7 @@ function hasFatalIssues(result) {
   if (!result.issues) return false;
   return result.issues.some((issue) => {
     const ruleId = issue.ruleId || "";
-    return ruleId.endsWith("/error") || ruleId.includes("/execution_error") || ruleId.endsWith("_fail_if");
+    return ruleId.endsWith("/error") || ruleId.includes("/execution_error") || ruleId.includes("timeout") || ruleId.endsWith("_fail_if");
   });
 }
 function updateStats(results, state, isForEachIteration = false) {
@@ -14837,6 +14890,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
         const ruleId = issue.ruleId || "";
         return ruleId.endsWith("/error") || // System errors
         ruleId.includes("/execution_error") || // Command failures
+        ruleId.includes("timeout") || // Timeouts
         ruleId.endsWith("_fail_if");
       });
       if (iterationHasFatalIssues && output !== void 0 && output !== null && typeof output === "object") {
@@ -14952,7 +15006,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
             allIssues.push(failIssue);
             const nowHasFatalIssues = enrichedResult.issues.some((issue) => {
               const ruleId = issue.ruleId || "";
-              return ruleId.endsWith("/error") || ruleId.includes("/execution_error") || ruleId.endsWith("_fail_if");
+              return ruleId.endsWith("/error") || ruleId.includes("/execution_error") || ruleId.includes("timeout") || ruleId.endsWith("_fail_if");
             });
             if (nowHasFatalIssues && output !== void 0 && output !== null && typeof output === "object" && !output.__failed) {
               output = { ...output, __failed: true };
@@ -15194,7 +15248,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
           }
         }
         try {
-          const { evaluateTransitions } = await import("./routing-7FXPULTO.mjs");
+          const { evaluateTransitions } = await import("./routing-QHTGDIXF.mjs");
           const transTarget = await evaluateTransitions(
             onFinish.transitions,
             forEachParent,
@@ -15254,7 +15308,7 @@ async function executeCheckWithForEachItems2(checkId, forEachParent, forEachItem
             `[LevelDispatch] Error evaluating on_finish transitions for ${forEachParent}: ${e instanceof Error ? e.message : String(e)}`
           );
         }
-        const { evaluateGoto: evaluateGoto2 } = await import("./routing-7FXPULTO.mjs");
+        const { evaluateGoto: evaluateGoto2 } = await import("./routing-QHTGDIXF.mjs");
         if (context2.debug) {
           logger.info(
             `[LevelDispatch] Evaluating on_finish.goto_js for forEach parent: ${forEachParent}`
@@ -16329,6 +16383,7 @@ function hasFatalIssues2(result) {
     const ruleId = issue.ruleId || "";
     return ruleId.endsWith("/error") || // System errors
     ruleId.includes("/execution_error") || // Command failures
+    ruleId.includes("timeout") || // Timeouts
     ruleId.endsWith("_fail_if") && ruleId !== "global_fail_if";
   });
 }
@@ -16372,6 +16427,7 @@ function updateStats2(results, state, isForEachIteration = false) {
       const ruleId = issue.ruleId || "";
       return ruleId.endsWith("/error") || // System errors, exceptions
       ruleId.includes("/execution_error") || // Command failures
+      ruleId.includes("timeout") || // Timeouts
       ruleId.endsWith("_fail_if") && ruleId !== "global_fail_if";
     });
     if (error) {
@@ -16416,7 +16472,7 @@ function updateStats2(results, state, isForEachIteration = false) {
 }
 async function renderTemplateContent2(checkId, checkConfig, reviewSummary) {
   try {
-    const { createExtendedLiquid: createExtendedLiquid2 } = await import("./liquid-extensions-DFDEBMUI.mjs");
+    const { createExtendedLiquid: createExtendedLiquid2 } = await import("./liquid-extensions-I7O7KMHF.mjs");
     const fs8 = await import("fs/promises");
     const path9 = await import("path");
     const schemaRaw = checkConfig.schema || "plain";
@@ -17047,7 +17103,6 @@ var init_runner = __esm({
 });
 
 // src/providers/workflow-check-provider.ts
-import { Liquid as Liquid2 } from "liquidjs";
 var WorkflowCheckProvider;
 var init_workflow_check_provider = __esm({
   "src/providers/workflow-check-provider.ts"() {
@@ -17057,6 +17112,7 @@ var init_workflow_check_provider = __esm({
     init_logger();
     init_sandbox();
     init_human_id();
+    init_liquid_extensions();
     WorkflowCheckProvider = class extends CheckProvider {
       registry;
       executor;
@@ -17065,7 +17121,7 @@ var init_workflow_check_provider = __esm({
         super();
         this.registry = WorkflowRegistry.getInstance();
         this.executor = new WorkflowExecutor();
-        this.liquid = new Liquid2();
+        this.liquid = createExtendedLiquid();
       }
       getName() {
         return "workflow";
@@ -17490,7 +17546,6 @@ var init_workflow_check_provider = __esm({
         if (!workflow.outputs) {
           return outputs;
         }
-        const sandbox = createSecureSandbox();
         const flat = {};
         try {
           for (const arr of Object.values(groupedResults || {})) {
@@ -17509,6 +17564,7 @@ var init_workflow_check_provider = __esm({
         );
         for (const output of workflow.outputs) {
           if (output.value_js) {
+            const sandbox = createSecureSandbox();
             outputs[output.name] = compileAndRun(
               sandbox,
               output.value_js,
@@ -17606,26 +17662,25 @@ var init_workflow_check_provider = __esm({
         if (rawData.imports && Array.isArray(rawData.imports)) {
           const configDir = path9.dirname(resolved);
           for (const source of rawData.imports) {
-            try {
-              const results = await this.registry.import(source, {
-                basePath: configDir,
-                validate: true
-              });
-              for (const result of results) {
-                if (!result.valid && result.errors) {
-                  const errors = result.errors.map((e) => `  ${e.path}: ${e.message}`).join("\n");
-                  throw new Error(`Failed to import workflow from '${source}':
-${errors}`);
+            const results = await this.registry.import(source, {
+              basePath: configDir,
+              validate: true
+            });
+            for (const result of results) {
+              if (!result.valid && result.errors) {
+                const isAlreadyExists = result.errors.every(
+                  (e) => e.message.includes("already exists")
+                );
+                if (isAlreadyExists) {
+                  logger.debug(`Workflow from '${source}' already imported, skipping`);
+                  continue;
                 }
-              }
-              logger.info(`Imported workflows from: ${source}`);
-            } catch (err) {
-              if (err.message?.includes("already exists")) {
-                logger.debug(`Workflow from '${source}' already imported, skipping`);
-              } else {
-                throw err;
+                const errors = result.errors.map((e) => `  ${e.path}: ${e.message}`).join("\n");
+                throw new Error(`Failed to import workflow from '${source}':
+${errors}`);
               }
             }
+            logger.info(`Imported workflows from: ${source}`);
           }
         }
         const { ConfigManager } = (init_config(), __toCommonJS(config_exports));
@@ -17664,4 +17719,4 @@ export {
   StateMachineRunner,
   init_runner
 };
-//# sourceMappingURL=chunk-A4PGHURG.mjs.map
+//# sourceMappingURL=chunk-TS6BUNAI.mjs.map