opencara 0.23.8 → 0.23.10

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 = {
@@ -797,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: {
@@ -855,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: {
@@ -2015,11 +2042,11 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd, liv
   });
 }
 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;
@@ -2027,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);
@@ -4659,6 +4686,17 @@ function agentConfigToDescriptor(config, agentId, index, agentOwner, userOrgs) {
 var DEFAULT_RECHECK_INTERVAL = 50;
 
 // src/commands/agent.ts
+function resolveCommandTemplate(agentConfig, globalCommand) {
+  if (agentConfig?.command) return agentConfig.command;
+  if (globalCommand) return globalCommand;
+  if (agentConfig?.tool) {
+    const registryTool = DEFAULT_REGISTRY.tools.find((t) => t.name === agentConfig.tool);
+    if (registryTool) {
+      return registryTool.commandTemplate.replaceAll("${MODEL}", agentConfig.model ?? "");
+    }
+  }
+  return void 0;
+}
 var DEFAULT_POLL_INTERVAL_MS = 1e4;
 var MAX_CONSECUTIVE_AUTH_ERRORS = 3;
 var MAX_POLL_BACKOFF_MS = 3e5;
@@ -5712,7 +5750,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.8",
+    cliVersion: "0.23.10",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5746,7 +5784,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 {
@@ -6008,7 +6046,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.8",
+    cliVersion: "0.23.10",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6036,7 +6074,7 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
   let skipped = 0;
   for (let i = 0; i < agents.length; i++) {
     const agentConfig = agents[i];
-    const commandTemplate = agentConfig.command ?? config.agentCommand ?? void 0;
+    const commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
     const label = agentConfig.name ?? `agent[${i}]`;
     if (!commandTemplate) {
       logError(`[${label}] No command configured. Skipping.`);
@@ -6104,7 +6142,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)`
@@ -6180,10 +6221,8 @@ async function startAgentRouter() {
   let agentConfig;
   if (config.agents && config.agents.length > 0) {
     agentConfig = config.agents.find((a) => a.router) ?? config.agents[0];
-    commandTemplate = agentConfig.command ?? config.agentCommand ?? void 0;
-  } else {
-    commandTemplate = config.agentCommand ?? void 0;
   }
+  commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
   const router = new RouterRelay();
   router.start();
   const logger = createLogger(agentConfig?.name ?? "agent[0]");
@@ -6248,7 +6287,8 @@ async function startAgentRouter() {
       userOrgs,
       usageLimits: config.usageLimits,
       versionOverride,
-      codebaseTtl: config.codebaseTtl
+      codebaseTtl: config.codebaseTtl,
+      commandTestTimeoutMs: config.commandTestTimeoutMs
     }
   );
   router.stop();
@@ -6258,10 +6298,8 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
   let agentConfig;
   if (config.agents && config.agents.length > agentIndex) {
     agentConfig = config.agents[agentIndex];
-    commandTemplate = agentConfig.command ?? config.agentCommand ?? void 0;
-  } else {
-    commandTemplate = config.agentCommand ?? void 0;
   }
+  commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
   const label = agentConfig?.name ?? `agent[${agentIndex}]`;
   if (!commandTemplate) {
     console.error(`[${label}] No command configured. Skipping.`);
@@ -6326,7 +6364,8 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
         codebaseTtl: config.codebaseTtl,
         verbose,
         agentOwner,
-        userOrgs
+        userOrgs,
+        commandTestTimeoutMs: config.commandTestTimeoutMs
       }
     ).finally(() => {
       routerRelay?.stop();
@@ -6354,7 +6393,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.23.8", "4d448ed"));
+    console.log(formatVersionBanner("0.23.10", "3fd8827"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7177,7 +7216,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.8"} (${"4d448ed"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.23.10"} (${"3fd8827"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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