opencara 0.25.1 → 0.25.2

package/dist/index.js

@@ -914,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)) {
@@ -1010,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"]) {
@@ -1054,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) {
@@ -1082,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);
       }
@@ -1697,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);
@@ -2705,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);
@@ -4046,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) => {
@@ -4067,6 +4091,7 @@ function executeAgentic(commandTemplate, prompt2, timeoutMs, cwd, signal) {
         }, 5e3);
       }
     }, timeoutMs);
+    const stopHeartbeat = startHeartbeatTimer(heartbeat, () => settled);
     let onAbort;
     if (signal) {
       onAbort = () => {
@@ -4074,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") {
@@ -4107,7 +4135,8 @@ async function executeImplement(task, worktreePath, deps, timeoutSeconds, signal
       prompt2,
       effectiveTimeout,
       worktreePath,
-      signal
+      signal,
+      deps.heartbeat
     );
     return {
       output: {
@@ -4126,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);
@@ -4348,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;
@@ -5067,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;
@@ -5211,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,
@@ -5250,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,
@@ -5364,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,
@@ -5375,7 +5455,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         prompt2,
         timeout_seconds,
         claimResponse.reviews,
-        taskReviewDeps,
+        summaryDeps,
         consumptionDeps,
         logger,
         agentInfo,
@@ -5385,6 +5465,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         verbose
       );
     } else {
+      const reviewDepsWithHeartbeat = { ...taskReviewDeps, heartbeat };
       await executeReviewTask(
         client,
         agentId,
@@ -5395,7 +5476,7 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         diffContent,
         prompt2,
         timeout_seconds,
-        taskReviewDeps,
+        reviewDepsWithHeartbeat,
         consumptionDeps,
         logger,
         agentInfo,
@@ -5488,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;
@@ -5587,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;
@@ -5690,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;
@@ -5803,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.1",
+    cliVersion: "0.25.2",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -6168,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.1",
+    cliVersion: "0.25.2",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6519,7 +6594,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.25.1", "085f9a8"));
+    console.log(formatVersionBanner("0.25.2", "c374877"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7341,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.1"} (${"085f9a8"})`);
+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.1",
+  "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",