opencara 0.25.0 → 0.25.2

package/dist/index.js

@@ -499,6 +499,7 @@ function ensureConfigDir() {
   fs.mkdirSync(dir, { recursive: true });
 }
 var DEFAULT_MAX_DIFF_SIZE_KB = 100;
+var DEFAULT_MAX_SUMMARY_INPUT_KB = 500;
 var DEFAULT_MAX_CONSECUTIVE_ERRORS = 10;
 var DEFAULT_MAX_REPO_SIZE_MB = 100;
 var DEFAULT_COMMAND_TEST_TIMEOUT_MS = 1e4;
@@ -705,6 +706,12 @@ function validateConfigData(data, envPlatformUrl) {
     );
     overrides.maxDiffSizeKb = DEFAULT_MAX_DIFF_SIZE_KB;
   }
+  if (typeof data.max_summary_input_kb === "number" && data.max_summary_input_kb <= 0) {
+    console.warn(
+      `\u26A0 Config warning: max_summary_input_kb must be > 0, got ${data.max_summary_input_kb}, using default (${DEFAULT_MAX_SUMMARY_INPUT_KB})`
+    );
+    overrides.maxSummaryInputKb = DEFAULT_MAX_SUMMARY_INPUT_KB;
+  }
   if (typeof data.max_consecutive_errors === "number" && data.max_consecutive_errors <= 0) {
     console.warn(
       `\u26A0 Config warning: max_consecutive_errors must be > 0, got ${data.max_consecutive_errors}, using default (${DEFAULT_MAX_CONSECUTIVE_ERRORS})`
@@ -745,6 +752,7 @@ function loadConfig() {
     platformUrl: envPlatformUrl || DEFAULT_PLATFORM_URL,
     authFile: null,
     maxDiffSizeKb: DEFAULT_MAX_DIFF_SIZE_KB,
+    maxSummaryInputKb: DEFAULT_MAX_SUMMARY_INPUT_KB,
     maxConsecutiveErrors: DEFAULT_MAX_CONSECUTIVE_ERRORS,
     maxRepoSizeMb: DEFAULT_MAX_REPO_SIZE_MB,
     codebaseDir: null,
@@ -804,6 +812,7 @@ function loadConfig() {
     platformUrl: envPlatformUrl || (typeof data.platform_url === "string" ? data.platform_url : DEFAULT_PLATFORM_URL),
     authFile: typeof data.auth_file === "string" && data.auth_file.trim() ? resolveFilePath(data.auth_file) : null,
     maxDiffSizeKb: overrides.maxDiffSizeKb ?? (typeof data.max_diff_size_kb === "number" ? data.max_diff_size_kb : DEFAULT_MAX_DIFF_SIZE_KB),
+    maxSummaryInputKb: overrides.maxSummaryInputKb ?? (typeof data.max_summary_input_kb === "number" ? data.max_summary_input_kb : DEFAULT_MAX_SUMMARY_INPUT_KB),
     maxConsecutiveErrors: overrides.maxConsecutiveErrors ?? (typeof data.max_consecutive_errors === "number" ? data.max_consecutive_errors : DEFAULT_MAX_CONSECUTIVE_ERRORS),
     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,
@@ -905,7 +914,27 @@ var ToolTimeoutError = class extends Error {
 var SIGKILL_GRACE_MS = 5e3;
 var MIN_PARTIAL_RESULT_LENGTH = 50;
 var STDOUT_LIVENESS_TIMEOUT_MS = 3e5;
+var DEFAULT_HEARTBEAT_INTERVAL_MS = 6e4;
 var MAX_STDERR_LENGTH = 1e3;
+function startHeartbeatTimer(heartbeat, isSettled = () => false) {
+  if (!heartbeat) return () => {
+  };
+  const intervalMs = heartbeat.intervalMs ?? DEFAULT_HEARTBEAT_INTERVAL_MS;
+  if (intervalMs <= 0) return () => {
+  };
+  const timer = setInterval(() => {
+    if (isSettled()) return;
+    try {
+      const r = heartbeat.callback();
+      if (r && typeof r.catch === "function") {
+        r.catch(() => {
+        });
+      }
+    } catch {
+    }
+  }, intervalMs);
+  return () => clearInterval(timer);
+}
 function validateCommandBinary(commandTemplate) {
   const { command } = parseCommandTemplate(commandTemplate);
   if (path3.isAbsolute(command)) {
@@ -1001,7 +1030,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, livenessTimeoutMs) {
+function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd, livenessTimeoutMs, heartbeat) {
   const promptViaArg = commandTemplate.includes("${PROMPT}");
   const allVars = { ...vars, PROMPT: prompt2 };
   if (cwd && !allVars["CODEBASE_DIR"]) {
@@ -1045,6 +1074,7 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd, liv
         }
       }, effectiveLivenessMs);
     }
+    const stopHeartbeat = startHeartbeatTimer(heartbeat, () => settled);
     child.stdout?.on("data", (chunk) => {
       stdout += chunk.toString();
       if (livenessTimer) {
@@ -1073,6 +1103,7 @@ function executeTool(commandTemplate, prompt2, timeoutMs, signal, vars, cwd, liv
       clearTimeout(timer);
       if (livenessTimer) clearTimeout(livenessTimer);
       if (sigkillTimer) clearTimeout(sigkillTimer);
+      stopHeartbeat();
       if (onAbort && signal) {
         signal.removeEventListener("abort", onAbort);
       }
@@ -1688,7 +1719,8 @@ ${userMessage}`;
       abortController.signal,
       void 0,
       deps.codebaseDir ?? void 0,
-      deps.livenessTimeoutMs
+      deps.livenessTimeoutMs,
+      deps.heartbeat
     );
     const { verdict, review } = extractVerdict(result.stdout);
     const inputTokens = result.tokensParsed ? 0 : estimateTokens(fullPrompt);
@@ -2613,7 +2645,7 @@ function sleep(ms, signal) {
 
 // src/summary.ts
 var TIMEOUT_SAFETY_MARGIN_MS2 = 3e4;
-var MAX_INPUT_SIZE_BYTES = 200 * 1024;
+var MAX_INPUT_SIZE_BYTES = 500 * 1024;
 var InputTooLargeError = class extends Error {
   constructor(message) {
     super(message);
@@ -2663,9 +2695,10 @@ function calculateInputSize(prompt2, reviews, diffContent, contextBlock) {
 }
 async function executeSummary(req, deps, runTool = executeTool) {
   const inputSize = calculateInputSize(req.prompt, req.reviews, req.diffContent, req.contextBlock);
-  if (inputSize > MAX_INPUT_SIZE_BYTES) {
+  const maxInputBytes = deps.maxSummaryInputKb !== void 0 ? deps.maxSummaryInputKb * 1024 : MAX_INPUT_SIZE_BYTES;
+  if (inputSize > maxInputBytes) {
     throw new InputTooLargeError(
-      `Summary input too large (${Math.round(inputSize / 1024)}KB > ${Math.round(MAX_INPUT_SIZE_BYTES / 1024)}KB limit)`
+      `Summary input too large (${Math.round(inputSize / 1024)}KB > ${Math.round(maxInputBytes / 1024)}KB limit)`
     );
   }
   const timeoutMs = req.timeout * 1e3;
@@ -2695,7 +2728,8 @@ ${userMessage}`;
       abortController.signal,
       void 0,
       deps.codebaseDir ?? void 0,
-      deps.livenessTimeoutMs
+      deps.livenessTimeoutMs,
+      deps.heartbeat
     );
     const { verdict, review } = extractVerdict(result.stdout);
     const flaggedReviews = extractFlaggedReviews(result.stdout);
@@ -4036,7 +4070,7 @@ function createPR(worktreePath, issueNumber, issueTitle, summary, branchName) {
 function isAgenticCommand(commandTemplate) {
   return commandTemplate.includes("${PROMPT}") && !commandTemplate.includes("--print");
 }
-function executeAgentic(commandTemplate, prompt2, timeoutMs, cwd, signal) {
+function executeAgentic(commandTemplate, prompt2, timeoutMs, cwd, signal, heartbeat) {
   const allVars = { PROMPT: prompt2, CODEBASE_DIR: cwd };
   const { command, args } = parseCommandTemplate(commandTemplate, allVars);
   return new Promise((resolve2, reject) => {
@@ -4057,6 +4091,7 @@ function executeAgentic(commandTemplate, prompt2, timeoutMs, cwd, signal) {
         }, 5e3);
       }
     }, timeoutMs);
+    const stopHeartbeat = startHeartbeatTimer(heartbeat, () => settled);
     let onAbort;
     if (signal) {
       onAbort = () => {
@@ -4064,16 +4099,19 @@ function executeAgentic(commandTemplate, prompt2, timeoutMs, cwd, signal) {
       };
       signal.addEventListener("abort", onAbort, { once: true });
     }
-    child.on("error", (err) => {
+    function agenticCleanup() {
       clearTimeout(timer);
+      stopHeartbeat();
       if (onAbort && signal) signal.removeEventListener("abort", onAbort);
+    }
+    child.on("error", (err) => {
+      agenticCleanup();
       if (settled) return;
       settled = true;
       reject(err);
     });
     child.on("close", (code, sig) => {
-      clearTimeout(timer);
-      if (onAbort && signal) signal.removeEventListener("abort", onAbort);
+      agenticCleanup();
       if (settled) return;
       settled = true;
       if (sig === "SIGTERM" || sig === "SIGKILL") {
@@ -4097,7 +4135,8 @@ async function executeImplement(task, worktreePath, deps, timeoutSeconds, signal
       prompt2,
       effectiveTimeout,
       worktreePath,
-      signal
+      signal,
+      deps.heartbeat
     );
     return {
       output: {
@@ -4116,7 +4155,9 @@ async function executeImplement(task, worktreePath, deps, timeoutSeconds, signal
     effectiveTimeout,
     signal,
     void 0,
-    worktreePath
+    worktreePath,
+    void 0,
+    deps.heartbeat
   );
   const output = parseImplementOutput(result.stdout);
   const inputTokens = result.tokensParsed ? 0 : estimateTokens(prompt2);
@@ -4338,7 +4379,9 @@ async function executeFix(task, diffContent, deps, timeoutSeconds, worktreePath,
     effectiveTimeout,
     signal,
     void 0,
-    worktreePath
+    worktreePath,
+    void 0,
+    deps.heartbeat
   );
   const inputTokens = result.tokensParsed ? 0 : estimateTokens(prompt2);
   const detail = result.tokenDetail;
@@ -5057,6 +5100,49 @@ async function pollLoop(client, agentId, reviewDeps, consumptionDeps, agentInfo,
     await sleep2(pollIntervalMs, signal);
   }
 }
+function isLongRunningRole(role) {
+  return isFixRole(role) || isImplementRole(role) || role === "review" || role === "summary";
+}
+async function runWithHeartbeat(heartbeat, work) {
+  let done = false;
+  const stop = startHeartbeatTimer(heartbeat, () => done);
+  try {
+    return await work();
+  } finally {
+    done = true;
+    stop();
+  }
+}
+function createHeartbeatControl(client, taskId, agentId, role, logger, intervalMs = DEFAULT_HEARTBEAT_INTERVAL_MS) {
+  let sawNotFound = false;
+  let failureStreakLogged = false;
+  return {
+    intervalMs,
+    callback: async () => {
+      try {
+        await client.post(`/api/tasks/${taskId}/heartbeat`, {
+          agent_id: agentId,
+          role
+        });
+        failureStreakLogged = false;
+      } catch (err) {
+        if (err instanceof HttpError && err.status === 404) {
+          if (!sawNotFound) {
+            sawNotFound = true;
+            logger.log(`  (heartbeat endpoint not available \u2014 old server, continuing)`);
+          }
+          return;
+        }
+        if (!failureStreakLogged) {
+          failureStreakLogged = true;
+          logger.logWarn(
+            `  ${icons.warn} Heartbeat failed for task ${taskId}: ${err.message} (further failures suppressed until next success)`
+          );
+        }
+      }
+    }
+  };
+}
 async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, agentInfo, logger, agentSession, routerRelay, signal, cleanupTracker, verbose) {
   const { task_id, owner, repo, pr_number, diff_url, timeout_seconds, prompt: prompt2, role, base_ref } = task;
   const { log, logError, logWarn } = logger;
@@ -5201,12 +5287,14 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
       log("  (suspicious prompt report not sent \u2014 endpoint not available)");
     }
   }
+  const heartbeat = isLongRunningRole(role) ? createHeartbeatControl(client, task_id, agentId, role, logger) : void 0;
   try {
     if (isImplementRole(role)) {
       const codebaseDir = reviewDeps.codebaseDir || path9.join(CONFIG_DIR, "repos");
       const implementDeps = {
         commandTemplate: reviewDeps.commandTemplate,
-        codebaseDir
+        codebaseDir,
+        heartbeat
       };
       const implementResult = await executeImplementTask(
         client,
@@ -5240,7 +5328,8 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         throw new Error("Fix task requires a codebase worktree but checkout failed");
       }
       const fixDeps = {
-        commandTemplate: reviewDeps.commandTemplate
+        commandTemplate: reviewDeps.commandTemplate,
+        heartbeat
       };
       const fixResult = await executeFixTask(
         client,
@@ -5354,6 +5443,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         { execGh: defaultExecGh }
       );
     } else if (role === "summary" && "reviews" in claimResponse && claimResponse.reviews) {
+      const summaryDeps = { ...taskReviewDeps, heartbeat };
       await executeSummaryTask(
         client,
         agentId,
@@ -5365,7 +5455,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         prompt2,
         timeout_seconds,
         claimResponse.reviews,
-        taskReviewDeps,
+        summaryDeps,
         consumptionDeps,
         logger,
         agentInfo,
@@ -5375,6 +5465,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         verbose
       );
     } else {
+      const reviewDepsWithHeartbeat = { ...taskReviewDeps, heartbeat };
       await executeReviewTask(
         client,
         agentId,
@@ -5385,7 +5476,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         diffContent,
         prompt2,
         timeout_seconds,
-        taskReviewDeps,
+        reviewDepsWithHeartbeat,
         consumptionDeps,
         logger,
         agentInfo,
@@ -5478,11 +5569,9 @@ async function executeReviewTask(client, agentId, taskId, owner, repo, prNumber,
       diffContent,
       contextBlock
     });
-    const response = await routerRelay.sendPrompt(
-      "review_request",
-      taskId,
-      fullPrompt,
-      timeoutSeconds
+    const response = await runWithHeartbeat(
+      reviewDeps.heartbeat,
+      () => routerRelay.sendPrompt("review_request", taskId, fullPrompt, timeoutSeconds)
     );
     const parsed = routerRelay.parseReviewResponse(response);
     reviewText = parsed.review;
@@ -5577,11 +5666,9 @@ async function executeSummaryTask(client, agentId, taskId, owner, repo, prNumber
         diffContent,
         contextBlock
       });
-      const response = await routerRelay.sendPrompt(
-        "review_request",
-        taskId,
-        fullPrompt,
-        timeoutSeconds
+      const response = await runWithHeartbeat(
+        reviewDeps.heartbeat,
+        () => routerRelay.sendPrompt("review_request", taskId, fullPrompt, timeoutSeconds)
       );
       const parsed = routerRelay.parseReviewResponse(response);
       reviewText = parsed.review;
@@ -5680,11 +5767,9 @@ async function executeSummaryTask(client, agentId, taskId, owner, repo, prNumber
       diffContent,
       contextBlock
     });
-    const response = await routerRelay.sendPrompt(
-      "summary_request",
-      taskId,
-      fullPrompt,
-      timeoutSeconds
+    const response = await runWithHeartbeat(
+      reviewDeps.heartbeat,
+      () => routerRelay.sendPrompt("summary_request", taskId, fullPrompt, timeoutSeconds)
     );
     const parsed = extractVerdict(response);
     summaryText = parsed.review;
@@ -5793,7 +5878,7 @@ function sleep2(ms, signal) {
 async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumptionDeps, options) {
   const client = new ApiClient(platformUrl, {
     authToken: options?.authToken,
-    cliVersion: "0.25.0",
+    cliVersion: "0.25.2",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -6158,7 +6243,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.25.0",
+    cliVersion: "0.25.2",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6364,6 +6449,7 @@ async function startAgentRouter() {
   const reviewDeps = {
     commandTemplate: commandTemplate ?? "",
     maxDiffSizeKb: config.maxDiffSizeKb,
+    maxSummaryInputKb: config.maxSummaryInputKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
     codebaseDir,
     livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
@@ -6430,6 +6516,7 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
   const reviewDeps = {
     commandTemplate,
     maxDiffSizeKb: config.maxDiffSizeKb,
+    maxSummaryInputKb: config.maxSummaryInputKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
     codebaseDir,
     livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
@@ -6507,7 +6594,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.25.0", "a984ae2"));
+    console.log(formatVersionBanner("0.25.2", "c374877"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7329,7 +7416,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.25.0"} (${"a984ae2"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.25.2"} (${"c374877"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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