opencara 0.20.0 → 0.21.0

package/dist/index.js

@@ -240,6 +240,9 @@ var DEFAULT_TRIAGE_TRIGGER = {
   events: ["opened"],
   comment: "/opencara triage"
 };
+var DEFAULT_ISSUE_REVIEW_TRIGGER = {
+  comment: "/opencara review-issue"
+};
 var DEFAULT_TRIGGER = DEFAULT_REVIEW_TRIGGER;
 var DEFAULT_FEATURE_CONFIG = {
   prompt: "Review this pull request for bugs, security issues, and code quality.",
@@ -308,6 +311,24 @@ function parseAgentSlots(value) {
   }
   return slots.length > 0 ? slots : void 0;
 }
+function parseNamedAgents(value) {
+  if (!Array.isArray(value))
+    return void 0;
+  const agents = [];
+  for (const item of value) {
+    if (!isObject(item))
+      continue;
+    if (typeof item.id !== "string" || typeof item.prompt !== "string")
+      continue;
+    const agent = { id: item.id, prompt: item.prompt };
+    if (typeof item.model === "string")
+      agent.model = item.model;
+    if (typeof item.tool === "string")
+      agent.tool = item.tool;
+    agents.push(agent);
+  }
+  return agents.length > 0 ? agents : void 0;
+}
 function parseFeatureFields(raw, defaults) {
   const agentSlots = parseAgentSlots(raw.agents);
   return {
@@ -408,12 +429,16 @@ var DEFAULT_IMPLEMENT_FEATURE = {
   modelDiversityGraceMs: DEFAULT_MODEL_DIVERSITY_GRACE_MS
 };
 function parseImplementSection(raw) {
-  const base = parseFeatureFields(raw, DEFAULT_IMPLEMENT_FEATURE);
+  const { agents: _slots, ...base } = parseFeatureFields(raw, DEFAULT_IMPLEMENT_FEATURE);
   const triggerRaw = isObject(raw.trigger) ? raw.trigger : void 0;
+  const namedAgents = parseNamedAgents(raw.agents);
+  const agentField = typeof raw.agent_field === "string" ? raw.agent_field : void 0;
   return {
     ...base,
     enabled: typeof raw.enabled === "boolean" ? raw.enabled : true,
-    trigger: parseTriggerSection(triggerRaw, DEFAULT_IMPLEMENT_TRIGGER)
+    trigger: parseTriggerSection(triggerRaw, DEFAULT_IMPLEMENT_TRIGGER),
+    ...namedAgents ? { agents: namedAgents } : {},
+    ...agentField ? { agent_field: agentField } : {}
   };
 }
 var DEFAULT_FIX_FEATURE = {
@@ -425,12 +450,33 @@ var DEFAULT_FIX_FEATURE = {
   modelDiversityGraceMs: DEFAULT_MODEL_DIVERSITY_GRACE_MS
 };
 function parseFixSection(raw) {
-  const base = parseFeatureFields(raw, DEFAULT_FIX_FEATURE);
+  const { agents: _slots, ...base } = parseFeatureFields(raw, DEFAULT_FIX_FEATURE);
+  const triggerRaw = isObject(raw.trigger) ? raw.trigger : void 0;
+  const namedAgents = parseNamedAgents(raw.agents);
+  const agentField = typeof raw.agent_field === "string" ? raw.agent_field : void 0;
+  return {
+    ...base,
+    enabled: typeof raw.enabled === "boolean" ? raw.enabled : true,
+    trigger: parseTriggerSection(triggerRaw, DEFAULT_FIX_TRIGGER),
+    ...namedAgents ? { agents: namedAgents } : {},
+    ...agentField ? { agent_field: agentField } : {}
+  };
+}
+var DEFAULT_ISSUE_REVIEW_FEATURE = {
+  prompt: "Review this issue for clarity, completeness, and actionability.",
+  agentCount: 2,
+  timeout: "5m",
+  preferredModels: [],
+  preferredTools: [],
+  modelDiversityGraceMs: DEFAULT_MODEL_DIVERSITY_GRACE_MS
+};
+function parseIssueReviewSection(raw) {
+  const base = parseFeatureFields(raw, DEFAULT_ISSUE_REVIEW_FEATURE);
   const triggerRaw = isObject(raw.trigger) ? raw.trigger : void 0;
   return {
     ...base,
     enabled: typeof raw.enabled === "boolean" ? raw.enabled : true,
-    trigger: parseTriggerSection(triggerRaw, DEFAULT_FIX_TRIGGER)
+    trigger: parseTriggerSection(triggerRaw, DEFAULT_ISSUE_REVIEW_TRIGGER)
   };
 }
 function parseOpenCaraConfig(toml) {
@@ -473,6 +519,9 @@ function parseOpenCaraConfig(toml) {
   if (isObject(raw.fix)) {
     config.fix = parseFixSection(raw.fix);
   }
+  if (isObject(raw.issue_review)) {
+    config.issue_review = parseIssueReviewSection(raw.issue_review);
+  }
   return config;
 }
 function parseLegacyReviewConfig(raw) {
@@ -2125,15 +2174,10 @@ ${reviewSections}`);
 }
 var TRIAGE_SYSTEM_PROMPT = `You are a triage agent for a software project. Your job is to analyze a GitHub issue and produce a structured triage report.
 
-The project is a monorepo with the following packages:
-- server \u2014 Hono server on Cloudflare Workers (webhook receiver, REST task API, GitHub integration)
-- cli \u2014 Agent CLI npm package (HTTP polling, local review execution, router mode)
-- shared \u2014 Shared TypeScript types (REST API contracts, review config parser)
-
 ## Instructions
 
 1. **Categorize** the issue into one of: bug, feature, improvement, question, docs, chore
-2. **Identify the module** most relevant to this issue: server, cli, shared (or omit if unclear)
+2. **Identify the module** most relevant to this issue (use the most appropriate component, package, or area name from the repository \u2014 or omit if unclear)
 3. **Assess priority**: critical (service down / data loss), high (blocks users), medium (important but not urgent), low (nice to have)
 4. **Estimate size**: XS (< 1hr), S (1-4hr), M (4hr-2d), L (2-5d), XL (> 5d)
 5. **Suggest labels** relevant to the issue (e.g., "bug", "enhancement", "docs", module names, etc.)
@@ -2148,7 +2192,7 @@ Respond with ONLY a JSON object (no markdown fences, no preamble, no explanation
 \`\`\`
 {
   "category": "bug" | "feature" | "improvement" | "question" | "docs" | "chore",
-  "module": "server" | "cli" | "shared",
+  "module": "<string \u2014 component, package, or area name from the repository>",
   "priority": "critical" | "high" | "medium" | "low",
   "size": "XS" | "S" | "M" | "L" | "XL",
   "labels": ["label1", "label2"],
@@ -4325,6 +4369,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 +5390,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.21.0",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5370,44 +5442,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 +5676,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.21.0",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -5726,45 +5797,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 +6019,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.20.0", "787d0af"));
+    console.log(formatVersionBanner("0.21.0", "a24bc6d"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -6768,7 +6842,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.21.0"} (${"a24bc6d"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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