opencara 0.23.7 → 0.23.9

package/dist/index.js

@@ -562,6 +562,15 @@ function ensureConfigDir() {
 var DEFAULT_MAX_DIFF_SIZE_KB = 100;
 var DEFAULT_MAX_CONSECUTIVE_ERRORS = 10;
 var DEFAULT_MAX_REPO_SIZE_MB = 100;
+var DEFAULT_COMMAND_TEST_TIMEOUT_MS = 1e4;
+function parseDurationMs(value) {
+  if (typeof value !== "string") return null;
+  const secMatch = value.match(/^(\d+)s$/);
+  if (secMatch) return parseInt(secMatch[1], 10) * 1e3;
+  const minMatch = value.match(/^(\d+)m$/);
+  if (minMatch) return parseInt(minMatch[1], 10) * 6e4;
+  return null;
+}
 var VALID_REPO_MODES = ["public", "private", "whitelist", "blacklist"];
 var REPO_PATTERN = /^[^/]+\/[^/]+$/;
 var REPO_MODE_ALIASES = {
@@ -714,6 +723,20 @@ function parseAgents(data) {
         agent.maxTasksPerDay = v;
       }
     }
+    if (typeof obj.liveness_timeout === "number") {
+      if (obj.liveness_timeout === 0) {
+        agent.livenessTimeout = 0;
+      } else {
+        const v = parsePositiveInt(obj.liveness_timeout);
+        if (v === null) {
+          console.warn(
+            `\u26A0 Config warning: agents[${i}].liveness_timeout must be a non-negative integer (seconds), got ${obj.liveness_timeout}. Value ignored.`
+          );
+        } else {
+          agent.livenessTimeout = v;
+        }
+      }
+    }
     const repoConfig = parseRepoConfig(obj, i);
     if (repoConfig) agent.repos = repoConfig;
     const synthesizeRepoConfig = parseRepoConfig(obj, i, "synthesize_repos");
@@ -783,6 +806,7 @@ function loadConfig() {
     maxRepoSizeMb: DEFAULT_MAX_REPO_SIZE_MB,
     codebaseDir: null,
     codebaseTtl: null,
+    commandTestTimeoutMs: DEFAULT_COMMAND_TEST_TIMEOUT_MS,
     agentCommand: null,
     agents: null,
     usageLimits: {
@@ -841,6 +865,23 @@ function loadConfig() {
     maxRepoSizeMb: overrides.maxRepoSizeMb ?? (typeof data.max_repo_size_mb === "number" ? data.max_repo_size_mb : DEFAULT_MAX_REPO_SIZE_MB),
     codebaseDir: typeof data.codebase_dir === "string" ? data.codebase_dir : null,
     codebaseTtl: typeof data.codebase_ttl === "string" ? data.codebase_ttl : null,
+    commandTestTimeoutMs: (() => {
+      if (data.command_test_timeout === void 0) return DEFAULT_COMMAND_TEST_TIMEOUT_MS;
+      const ms = parseDurationMs(data.command_test_timeout);
+      if (ms === null) {
+        console.warn(
+          `\u26A0 Config warning: command_test_timeout must be a duration string like "10s" or "1m", got "${data.command_test_timeout}", using default (${DEFAULT_COMMAND_TEST_TIMEOUT_MS / 1e3}s)`
+        );
+        return DEFAULT_COMMAND_TEST_TIMEOUT_MS;
+      }
+      if (ms <= 0) {
+        console.warn(
+          `\u26A0 Config warning: command_test_timeout must be a positive duration, got "${data.command_test_timeout}", using default (${DEFAULT_COMMAND_TEST_TIMEOUT_MS / 1e3}s)`
+        );
+        return DEFAULT_COMMAND_TEST_TIMEOUT_MS;
+      }
+      return ms;
+    })(),
     agentCommand: typeof data.agent_command === "string" ? data.agent_command : null,
     agents: parseAgents(data),
     usageLimits: {
@@ -1750,6 +1791,7 @@ var ToolTimeoutError = class extends Error {
 };
 var SIGKILL_GRACE_MS = 5e3;
 var MIN_PARTIAL_RESULT_LENGTH = 50;
+var STDOUT_LIVENESS_TIMEOUT_MS = 3e5;
 var MAX_STDERR_LENGTH = 1e3;
 function validateCommandBinary(commandTemplate) {
   const { command } = parseCommandTemplate(commandTemplate);
@@ -1846,7 +1888,7 @@ function parseTokenUsage(stdout, stderr) {
   const estimated = estimateTokens(stdout);
   return { tokens: estimated, parsed: false, input: 0, output: estimated };
 }
-function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
+function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd, livenessTimeoutMs) {
   const promptViaArg = commandTemplate.includes("${PROMPT}");
   const allVars = { ...vars, PROMPT: prompt2 };
   if (cwd && !allVars["CODEBASE_DIR"]) {
@@ -1866,7 +1908,12 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
     let stderr = "";
     let settled = false;
     let sigkillTimer;
+    let killedByLiveness = false;
+    const effectiveLivenessMs = livenessTimeoutMs === void 0 ? STDOUT_LIVENESS_TIMEOUT_MS : livenessTimeoutMs;
+    let killScheduled = false;
     function scheduleKillEscalation() {
+      if (killScheduled) return;
+      killScheduled = true;
       child.kill("SIGTERM");
       if (sigkillTimer) clearTimeout(sigkillTimer);
       sigkillTimer = setTimeout(() => {
@@ -1876,8 +1923,26 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
       }, SIGKILL_GRACE_MS);
     }
     const timer = setTimeout(scheduleKillEscalation, timeoutMs);
+    let livenessTimer;
+    if (effectiveLivenessMs > 0) {
+      livenessTimer = setTimeout(() => {
+        if (!settled) {
+          killedByLiveness = true;
+          scheduleKillEscalation();
+        }
+      }, effectiveLivenessMs);
+    }
     child.stdout?.on("data", (chunk) => {
       stdout += chunk.toString();
+      if (livenessTimer) {
+        clearTimeout(livenessTimer);
+        livenessTimer = setTimeout(() => {
+          if (!settled) {
+            killedByLiveness = true;
+            scheduleKillEscalation();
+          }
+        }, effectiveLivenessMs);
+      }
     });
     child.stderr?.on("data", (chunk) => {
       stderr += chunk.toString();
@@ -1893,6 +1958,7 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
     }
     function cleanup() {
       clearTimeout(timer);
+      if (livenessTimer) clearTimeout(livenessTimer);
       if (sigkillTimer) clearTimeout(sigkillTimer);
       if (onAbort && signal) {
         signal.removeEventListener("abort", onAbort);
@@ -1917,11 +1983,19 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
         return;
       }
       if (sig === "SIGTERM" || sig === "SIGKILL") {
-        reject(
-          new ToolTimeoutError(
-            `Tool "${command}" timed out after ${Math.round(timeoutMs / 1e3)}s`
-          )
-        );
+        if (killedByLiveness) {
+          reject(
+            new ToolTimeoutError(
+              `Tool "${command}" killed: no stdout for ${Math.round(effectiveLivenessMs / 1e3)}s (process may be stuck)`
+            )
+          );
+        } else {
+          reject(
+            new ToolTimeoutError(
+              `Tool "${command}" timed out after ${Math.round(timeoutMs / 1e3)}s`
+            )
+          );
+        }
         return;
       }
       if (code !== 0) {
@@ -1968,11 +2042,11 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd) {
   });
 }
 var TEST_COMMAND_PROMPT = "Respond with: OK";
-var TEST_COMMAND_TIMEOUT_MS = 1e4;
-async function testCommand(commandTemplate) {
+var DEFAULT_TEST_COMMAND_TIMEOUT_MS = 1e4;
+async function testCommand(commandTemplate, timeoutMs = DEFAULT_TEST_COMMAND_TIMEOUT_MS) {
   const start = Date.now();
   try {
-    await executeTool(commandTemplate, TEST_COMMAND_PROMPT, TEST_COMMAND_TIMEOUT_MS);
+    await executeTool(commandTemplate, TEST_COMMAND_PROMPT, timeoutMs);
     return { ok: true, elapsedMs: Date.now() - start };
   } catch (err) {
     const elapsed = Date.now() - start;
@@ -1980,7 +2054,7 @@ async function testCommand(commandTemplate) {
       return {
         ok: false,
         elapsedMs: elapsed,
-        error: `command timed out after ${TEST_COMMAND_TIMEOUT_MS / 1e3}s`
+        error: `command timed out after ${timeoutMs / 1e3}s`
       };
     }
     const msg = err instanceof Error ? err.message : String(err);
@@ -2500,7 +2574,8 @@ ${userMessage}`;
       effectiveTimeout,
       abortController.signal,
       void 0,
-      deps.codebaseDir ?? void 0
+      deps.codebaseDir ?? void 0,
+      deps.livenessTimeoutMs
     );
     const { verdict, review } = extractVerdict(result.stdout);
     const inputTokens = result.tokensParsed ? 0 : estimateTokens(fullPrompt);
@@ -2615,7 +2690,8 @@ ${userMessage}`;
       effectiveTimeout,
       abortController.signal,
       void 0,
-      deps.codebaseDir ?? void 0
+      deps.codebaseDir ?? void 0,
+      deps.livenessTimeoutMs
     );
     const { verdict, review } = extractVerdict(result.stdout);
     const flaggedReviews = extractFlaggedReviews(result.stdout);
@@ -5663,7 +5739,7 @@ function sleep2(ms, signal) {
 async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumptionDeps, options) {
   const client = new ApiClient(platformUrl, {
     authToken: options?.authToken,
-    cliVersion: "0.23.7",
+    cliVersion: "0.23.9",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5697,7 +5773,7 @@ async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumpti
   }
   if (reviewDeps.commandTemplate && !options?.routerRelay) {
     log("Testing command...");
-    const result = await testCommand(reviewDeps.commandTemplate);
+    const result = await testCommand(reviewDeps.commandTemplate, options?.commandTestTimeoutMs);
     if (result.ok) {
       log(`${icons.success} Command test ok (${(result.elapsedMs / 1e3).toFixed(1)}s)`);
     } else {
@@ -5959,7 +6035,7 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
   const { versionOverride, verbose, instancesOverride, agentOwner, userOrgs } = options;
   const client = new ApiClient(config.platformUrl, {
     authToken: oauthToken,
-    cliVersion: "0.23.7",
+    cliVersion: "0.23.9",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6005,7 +6081,8 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
       commandTemplate,
       maxDiffSizeKb: config.maxDiffSizeKb,
       maxRepoSizeMb: config.maxRepoSizeMb,
-      codebaseDir
+      codebaseDir,
+      livenessTimeoutMs: agentConfig.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
     };
     const session = createSessionTracker();
     const usageTracker = new UsageTracker();
@@ -6054,7 +6131,10 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
   await Promise.all(
     agentStates.filter((state) => state.reviewDeps.commandTemplate && !state.routerRelay).map(async (state) => {
       state.logger.log("Testing command...");
-      const result = await testCommand(state.reviewDeps.commandTemplate);
+      const result = await testCommand(
+        state.reviewDeps.commandTemplate,
+        config.commandTestTimeoutMs
+      );
       if (result.ok) {
         state.logger.log(
           `${icons.success} Command test ok (${(result.elapsedMs / 1e3).toFixed(1)}s)`
@@ -6161,7 +6241,8 @@ async function startAgentRouter() {
     commandTemplate: commandTemplate ?? "",
     maxDiffSizeKb: config.maxDiffSizeKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
-    codebaseDir
+    codebaseDir,
+    livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
   };
   const session = createSessionTracker();
   const usageTracker = new UsageTracker();
@@ -6197,7 +6278,8 @@ async function startAgentRouter() {
       userOrgs,
       usageLimits: config.usageLimits,
       versionOverride,
-      codebaseTtl: config.codebaseTtl
+      codebaseTtl: config.codebaseTtl,
+      commandTestTimeoutMs: config.commandTestTimeoutMs
     }
   );
   router.stop();
@@ -6228,7 +6310,8 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
     commandTemplate,
     maxDiffSizeKb: config.maxDiffSizeKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
-    codebaseDir
+    codebaseDir,
+    livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
   };
   const model = agentConfig?.model ?? "unknown";
   const tool = agentConfig?.tool ?? "unknown";
@@ -6274,7 +6357,8 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
         codebaseTtl: config.codebaseTtl,
         verbose,
         agentOwner,
-        userOrgs
+        userOrgs,
+        commandTestTimeoutMs: config.commandTestTimeoutMs
       }
     ).finally(() => {
       routerRelay?.stop();
@@ -6302,7 +6386,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.23.7", "eb8ff83"));
+    console.log(formatVersionBanner("0.23.9", "84eda5a"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7125,7 +7209,7 @@ var statusCommand = new Command4("status").description("Show agent config, conne
 });
 
 // src/index.ts
-var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.23.7"} (${"eb8ff83"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.23.9"} (${"84eda5a"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencara",
-  "version": "0.23.7",
+  "version": "0.23.9",
   "description": "Distributed AI code review agent — poll, review, and submit PR reviews using your own AI tools",
   "type": "module",
   "license": "MIT",