opencara 0.20.0 → 0.20.1

package/dist/index.js

@@ -4325,6 +4325,34 @@ var DEFAULT_RECHECK_INTERVAL = 50;
 var DEFAULT_POLL_INTERVAL_MS = 1e4;
 var MAX_CONSECUTIVE_AUTH_ERRORS = 3;
 var MAX_POLL_BACKOFF_MS = 3e5;
+var SHUTDOWN_GRACE_MS = 5e3;
+function registerShutdownHandlers(controller, log, graceMs = SHUTDOWN_GRACE_MS) {
+  let shutdownInitiated = false;
+  let forceTimer;
+  const onSignal = (signal) => {
+    if (shutdownInitiated) {
+      log(`${icons.stop} Received ${signal} again \u2014 forcing exit`);
+      process.exit(1);
+    }
+    shutdownInitiated = true;
+    log(`${icons.stop} Received ${signal} \u2014 shutting down gracefully...`);
+    controller.abort();
+    forceTimer = setTimeout(() => {
+      log(`${icons.stop} Shutdown timed out after ${graceMs / 1e3}s \u2014 forcing exit`);
+      process.exit(1);
+    }, graceMs);
+    forceTimer.unref();
+  };
+  const onSigint = () => onSignal("SIGINT");
+  const onSigterm = () => onSignal("SIGTERM");
+  process.on("SIGINT", onSigint);
+  process.on("SIGTERM", onSigterm);
+  return () => {
+    process.removeListener("SIGINT", onSigint);
+    process.removeListener("SIGTERM", onSigterm);
+    if (forceTimer) clearTimeout(forceTimer);
+  };
+}
 var NON_RETRYABLE_STATUSES = /* @__PURE__ */ new Set([401, 403, 404]);
 function toApiDiffUrl(webUrl) {
   const match = webUrl.match(/^https?:\/\/github\.com\/([^/]+)\/([^/]+)\/pull\/(\d+)(?:\.diff)?$/);
@@ -5318,7 +5346,7 @@ function sleep2(ms, signal) {
 async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumptionDeps, options) {
   const client = new ApiClient(platformUrl, {
     authToken: options?.authToken,
-    cliVersion: "0.20.0",
+    cliVersion: "0.20.1",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5370,44 +5398,43 @@ async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumpti
   }
   const cleanupTracker = ttlMs > 0 ? new CodebaseCleanupTracker(ttlMs) : void 0;
   const abortController = new AbortController();
-  process.on("SIGINT", () => {
-    abortController.abort();
-  });
-  process.on("SIGTERM", () => {
-    abortController.abort();
-  });
-  await pollLoop(client, agentId, reviewDeps, deps, agentInfo, logger, agentSession, {
-    pollIntervalMs: options?.pollIntervalMs ?? DEFAULT_POLL_INTERVAL_MS,
-    maxConsecutiveErrors: options?.maxConsecutiveErrors ?? DEFAULT_MAX_CONSECUTIVE_ERRORS,
-    routerRelay: options?.routerRelay,
-    reviewOnly: options?.reviewOnly,
-    repoConfig: options?.repoConfig,
-    roles: options?.roles,
-    synthesizeRepos: options?.synthesizeRepos,
-    signal: abortController.signal,
-    cleanupTracker,
-    verbose: options?.verbose,
-    agentOwner: options?.agentOwner,
-    userOrgs: options?.userOrgs
-  });
-  if (cleanupTracker && cleanupTracker.size > 0) {
-    const finalSwept = await cleanupTracker.sweep(cleanupWorktree);
-    if (finalSwept > 0) {
+  const removeShutdownHandlers = registerShutdownHandlers(abortController, log);
+  try {
+    await pollLoop(client, agentId, reviewDeps, deps, agentInfo, logger, agentSession, {
+      pollIntervalMs: options?.pollIntervalMs ?? DEFAULT_POLL_INTERVAL_MS,
+      maxConsecutiveErrors: options?.maxConsecutiveErrors ?? DEFAULT_MAX_CONSECUTIVE_ERRORS,
+      routerRelay: options?.routerRelay,
+      reviewOnly: options?.reviewOnly,
+      repoConfig: options?.repoConfig,
+      roles: options?.roles,
+      synthesizeRepos: options?.synthesizeRepos,
+      signal: abortController.signal,
+      cleanupTracker,
+      verbose: options?.verbose,
+      agentOwner: options?.agentOwner,
+      userOrgs: options?.userOrgs
+    });
+    if (cleanupTracker && cleanupTracker.size > 0) {
+      const finalSwept = await cleanupTracker.sweep(cleanupWorktree);
+      if (finalSwept > 0) {
+        log(
+          `${icons.info} Cleaned up ${finalSwept} codebase director${finalSwept === 1 ? "y" : "ies"} on shutdown`
+        );
+      }
+    }
+    if (deps.usageTracker) {
       log(
-        `${icons.info} Cleaned up ${finalSwept} codebase director${finalSwept === 1 ? "y" : "ies"} on shutdown`
+        deps.usageTracker.formatSummary(
+          deps.usageLimits ?? usageLimits,
+          deps.agentLimits,
+          deps.agentId
+        )
       );
     }
+    log(formatExitSummary(agentSession));
+  } finally {
+    removeShutdownHandlers();
   }
-  if (deps.usageTracker) {
-    log(
-      deps.usageTracker.formatSummary(
-        deps.usageLimits ?? usageLimits,
-        deps.agentLimits,
-        deps.agentId
-      )
-    );
-  }
-  log(formatExitSummary(agentSession));
 }
 async function batchPollLoop(client, agentStates, options) {
   const {
@@ -5605,7 +5632,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.20.0",
+    cliVersion: "0.20.1",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -5726,45 +5753,48 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
     }
   }
   const abortController = new AbortController();
-  process.on("SIGINT", () => abortController.abort());
-  process.on("SIGTERM", () => abortController.abort());
+  const removeShutdownHandlers = registerShutdownHandlers(abortController, log);
   log(`${agentStates.length} agent instance(s) running in batch mode. Press Ctrl+C to stop.
 `);
-  await batchPollLoop(client, agentStates, {
-    pollIntervalMs,
-    maxConsecutiveErrors: config.maxConsecutiveErrors,
-    signal: abortController.signal,
-    accessibleRepos,
-    githubToken: oauthToken
-  });
-  await Promise.allSettled(
-    agentStates.map(async (state) => {
-      state.routerRelay?.stop();
-      if (state.cleanupTracker && state.cleanupTracker.size > 0) {
-        const swept = await state.cleanupTracker.sweep(cleanupWorktree);
-        if (swept > 0) {
+  try {
+    await batchPollLoop(client, agentStates, {
+      pollIntervalMs,
+      maxConsecutiveErrors: config.maxConsecutiveErrors,
+      signal: abortController.signal,
+      accessibleRepos,
+      githubToken: oauthToken
+    });
+    await Promise.allSettled(
+      agentStates.map(async (state) => {
+        state.routerRelay?.stop();
+        if (state.cleanupTracker && state.cleanupTracker.size > 0) {
+          const swept = await state.cleanupTracker.sweep(cleanupWorktree);
+          if (swept > 0) {
+            state.logger.log(
+              `${icons.info} Cleaned up ${swept} codebase director${swept === 1 ? "y" : "ies"} on shutdown`
+            );
+          }
+        }
+        if (state.consumptionDeps.usageTracker) {
+          const limits = state.consumptionDeps.usageLimits ?? {
+            maxTasksPerDay: null,
+            maxTokensPerDay: null,
+            maxTokensPerReview: null
+          };
           state.logger.log(
-            `${icons.info} Cleaned up ${swept} codebase director${swept === 1 ? "y" : "ies"} on shutdown`
+            state.consumptionDeps.usageTracker.formatSummary(
+              limits,
+              state.consumptionDeps.agentLimits,
+              state.consumptionDeps.agentId
+            )
           );
         }
-      }
-      if (state.consumptionDeps.usageTracker) {
-        const limits = state.consumptionDeps.usageLimits ?? {
-          maxTasksPerDay: null,
-          maxTokensPerDay: null,
-          maxTokensPerReview: null
-        };
-        state.logger.log(
-          state.consumptionDeps.usageTracker.formatSummary(
-            limits,
-            state.consumptionDeps.agentLimits,
-            state.consumptionDeps.agentId
-          )
-        );
-      }
-      state.logger.log(formatExitSummary(state.agentSession));
-    })
-  );
+        state.logger.log(formatExitSummary(state.agentSession));
+      })
+    );
+  } finally {
+    removeShutdownHandlers();
+  }
 }
 async function startAgentRouter() {
   const config = loadConfig();
@@ -5945,7 +5975,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.20.0", "787d0af"));
+    console.log(formatVersionBanner("0.20.1", "c37d84d"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -6768,7 +6798,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.20.0"} (${"787d0af"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.20.1"} (${"c37d84d"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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