opencara 0.23.6 → 0.23.7

package/dist/index.js

@@ -5663,7 +5663,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.6",
+    cliVersion: "0.23.7",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5770,6 +5770,8 @@ async function batchPollLoop(client, agentStates, options) {
   let consecutiveAuthErrors = 0;
   let consecutiveErrors = 0;
   let pollCycleCount = 0;
+  const busyAgents = /* @__PURE__ */ new Set();
+  const inflightPromises = /* @__PURE__ */ new Set();
   while (!signal?.aborted) {
     if (accessibleRepos && githubToken && recheckInterval > 0 && pollCycleCount > 0 && pollCycleCount % recheckInterval === 0) {
       const allRepos = extractRepoUrls(
@@ -5826,13 +5828,17 @@ async function batchPollLoop(client, agentStates, options) {
       break;
     }
     try {
-      const descriptors = agentStates.map((s) => s.descriptor);
+      const availableStates = agentStates.filter((s) => !busyAgents.has(s));
+      if (availableStates.length === 0) {
+        await sleep2(pollIntervalMs, signal);
+        continue;
+      }
+      const descriptors = availableStates.map((s) => s.descriptor);
       const request = buildBatchPollRequest(descriptors);
       const response = await client.post("/api/tasks/poll/batch", request);
       consecutiveAuthErrors = 0;
       consecutiveErrors = 0;
-      const handlePromises = [];
-      for (const state of agentStates) {
+      for (const state of availableStates) {
         const agentName = state.descriptor.name;
         const pollResponse = response.assignments[agentName];
         if (!pollResponse || pollResponse.tasks.length === 0) continue;
@@ -5846,8 +5852,9 @@ async function batchPollLoop(client, agentStates, options) {
         );
         const task = eligible[0];
         if (!task) continue;
-        handlePromises.push(
-          (async () => {
+        busyAgents.add(state);
+        const p = (async () => {
+          try {
             const result = await handleTask(
               client,
               state.descriptor.agentId,
@@ -5876,28 +5883,27 @@ async function batchPollLoop(client, agentStates, options) {
                 );
               }
             }
-          })()
-        );
-      }
-      if (handlePromises.length > 0) {
-        const results = await Promise.allSettled(handlePromises);
-        for (const r of results) {
-          if (r.status === "rejected") {
-            logError(`${icons.error} Task handler failed: ${r.reason}`);
+          } catch (err) {
+            logError(`${icons.error} Task handler failed: ${err.message}`);
             consecutiveErrors++;
+          } finally {
+            busyAgents.delete(state);
+            if (state.cleanupTracker) {
+              try {
+                const swept = await state.cleanupTracker.sweep(cleanupWorktree);
+                if (swept > 0) {
+                  state.logger.log(
+                    `${icons.info} Cleaned up ${swept} stale codebase director${swept === 1 ? "y" : "ies"}`
+                  );
+                }
+              } catch {
+              }
+            }
           }
-        }
+        })();
+        inflightPromises.add(p);
+        void p.finally(() => inflightPromises.delete(p));
       }
-      await Promise.allSettled(
-        agentStates.filter((state) => state.cleanupTracker).map(async (state) => {
-          const swept = await state.cleanupTracker.sweep(cleanupWorktree);
-          if (swept > 0) {
-            state.logger.log(
-              `${icons.info} Cleaned up ${swept} stale codebase director${swept === 1 ? "y" : "ies"}`
-            );
-          }
-        })
-      );
     } catch (err) {
       if (signal?.aborted) break;
       if (err instanceof UpgradeRequiredError) {
@@ -5944,12 +5950,16 @@ async function batchPollLoop(client, agentStates, options) {
     }
     await sleep2(pollIntervalMs, signal);
   }
+  if (inflightPromises.size > 0) {
+    log(`${icons.info} Waiting for ${inflightPromises.size} in-flight task(s) to complete...`);
+    await Promise.allSettled([...inflightPromises]);
+  }
 }
 async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, options) {
   const { versionOverride, verbose, instancesOverride, agentOwner, userOrgs } = options;
   const client = new ApiClient(config.platformUrl, {
     authToken: oauthToken,
-    cliVersion: "0.23.6",
+    cliVersion: "0.23.7",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6292,7 +6302,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.23.6", "9696efa"));
+    console.log(formatVersionBanner("0.23.7", "eb8ff83"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7115,7 +7125,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.6"} (${"9696efa"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.23.7"} (${"eb8ff83"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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