opencara 0.23.10 → 0.23.12

package/dist/index.js

@@ -34,15 +34,12 @@ function isRepoAllowed(repoConfig, targetOwner, targetRepo, agentOwner, userOrgs
     case "public":
       return true;
     case "private": {
+      if (repoConfig.list && repoConfig.list.length > 0 && repoConfig.list.includes(fullRepo)) {
+        return true;
+      }
       const normalizedTarget = targetOwner.toLowerCase();
       const normalizedOwner = agentOwner?.toLowerCase();
-      const hasAccess = normalizedOwner === normalizedTarget || userOrgs != null && userOrgs.has(normalizedTarget);
-      if (!hasAccess)
-        return false;
-      if (repoConfig.list && repoConfig.list.length > 0) {
-        return repoConfig.list.includes(fullRepo);
-      }
-      return true;
+      return normalizedOwner === normalizedTarget || userOrgs != null && userOrgs.has(normalizedTarget);
     }
     case "whitelist":
       return (repoConfig.list ?? []).includes(fullRepo);
@@ -53,84 +50,6 @@ function isRepoAllowed(repoConfig, targetOwner, targetRepo, agentOwner, userOrgs
   }
 }
 
-// ../shared/dist/api.js
-var DEFAULT_REGISTRY = {
-  tools: [
-    {
-      name: "claude",
-      displayName: "Claude",
-      binary: "claude",
-      commandTemplate: "claude --model ${MODEL} --allowedTools '*' --print",
-      tokenParser: "claude"
-    },
-    {
-      name: "codex",
-      displayName: "Codex",
-      binary: "codex",
-      commandTemplate: "codex --model ${MODEL} exec",
-      tokenParser: "codex"
-    },
-    {
-      name: "gemini",
-      displayName: "Gemini",
-      binary: "gemini",
-      commandTemplate: "gemini -m ${MODEL}",
-      tokenParser: "gemini"
-    },
-    {
-      name: "qwen",
-      displayName: "Qwen",
-      binary: "qwen",
-      commandTemplate: "qwen --model ${MODEL} -y",
-      tokenParser: "qwen"
-    }
-  ],
-  models: [
-    {
-      name: "claude-opus-4-6",
-      displayName: "Claude Opus 4.6",
-      tools: ["claude"]
-    },
-    {
-      name: "claude-opus-4-6[1m]",
-      displayName: "Claude Opus 4.6 (1M context)",
-      tools: ["claude"]
-    },
-    {
-      name: "claude-sonnet-4-6",
-      displayName: "Claude Sonnet 4.6",
-      tools: ["claude"]
-    },
-    {
-      name: "claude-sonnet-4-6[1m]",
-      displayName: "Claude Sonnet 4.6 (1M context)",
-      tools: ["claude"]
-    },
-    {
-      name: "gpt-5-codex",
-      displayName: "GPT-5 Codex",
-      tools: ["codex"]
-    },
-    {
-      name: "gemini-2.5-pro",
-      displayName: "Gemini 2.5 Pro",
-      tools: ["gemini"]
-    },
-    {
-      name: "qwen3.5-plus",
-      displayName: "Qwen 3.5 Plus",
-      tools: ["qwen"]
-    },
-    { name: "glm-5", displayName: "GLM-5", tools: ["qwen"] },
-    { name: "kimi-k2.5", displayName: "Kimi K2.5", tools: ["qwen"] },
-    {
-      name: "minimax-m2.5",
-      displayName: "Minimax M2.5",
-      tools: ["qwen"]
-    }
-  ]
-};
-
 // ../shared/dist/review-config.js
 import { parse as parseToml } from "smol-toml";
 function isObject(value) {
@@ -552,6 +471,26 @@ import * as fs from "fs";
 import * as path from "path";
 import * as os from "os";
 import { parse as parseToml2, stringify as stringifyToml } from "smol-toml";
+
+// src/tool-defs.ts
+var _cache = null;
+function loadToolDefs() {
+  if (!_cache) {
+    _cache = JSON.parse('[{"name":"claude","binary":"claude","models":["claude-sonnet-4-6","claude-opus-4-6"],"command":"claude --model ${MODEL} --allowedTools \'*\' --print","scannable":true,"installLink":"https://docs.anthropic.com/en/docs/claude-code"},{"name":"codex","binary":"codex","models":["gpt-5-codex"],"command":"codex --model ${MODEL} exec","scannable":true,"installLink":"https://github.com/openai/codex"},{"name":"gemini","binary":"gemini","models":["gemini-2.5-pro"],"command":"gemini -m ${MODEL}","scannable":true,"installLink":"https://github.com/google-gemini/gemini-cli"},{"name":"qwen","binary":"qwen","models":["qwen3.5-plus","glm-5","kimi-k2.5","minimax-m2.5"],"command":"qwen --model ${MODEL} -y","scannable":false}]');
+  }
+  return _cache;
+}
+function getToolDef(name) {
+  return loadToolDefs().find((t) => t.name === name);
+}
+function getScannableTools() {
+  return loadToolDefs().filter((t) => t.scannable);
+}
+function getKnownToolNames() {
+  return new Set(loadToolDefs().map((t) => t.name));
+}
+
+// src/config.ts
 var DEFAULT_PLATFORM_URL = "https://api.opencara.com";
 var CONFIG_DIR = path.join(os.homedir(), ".opencara");
 var CONFIG_FILE = process.env.OPENCARA_CONFIG && process.env.OPENCARA_CONFIG.trim() ? path.resolve(process.env.OPENCARA_CONFIG) : path.join(CONFIG_DIR, "config.toml");
@@ -589,7 +528,7 @@ var ConfigValidationError = class extends Error {
     this.name = "ConfigValidationError";
   }
 };
-var KNOWN_TOOL_NAMES = new Set(DEFAULT_REGISTRY.tools.map((t) => t.name));
+var KNOWN_TOOL_NAMES = getKnownToolNames();
 var TOOL_ALIASES = {
   "claude-code": "claude"
 };
@@ -4416,17 +4355,6 @@ function countReviewComments(commentsText) {
 import { execFileSync as execFileSync8 } from "child_process";
 import * as fs9 from "fs";
 import * as readline2 from "readline";
-var SCANNABLE_TOOLS = ["claude", "codex", "gemini"];
-var DEFAULT_MODELS = {
-  claude: "claude-sonnet-4-6",
-  codex: "gpt-5-codex",
-  gemini: "gemini-2.5-pro"
-};
-var INSTALL_LINKS = {
-  claude: "https://docs.anthropic.com/en/docs/claude-code",
-  codex: "https://github.com/openai/codex",
-  gemini: "https://github.com/google-gemini/gemini-cli"
-};
 function checkPrerequisites() {
   const gitInstalled = validateCommandBinary("git");
   const ghInstalled = validateCommandBinary("gh");
@@ -4450,21 +4378,13 @@ function checkPrerequisites() {
 }
 function discoverTools() {
   const results = [];
-  for (const toolName of SCANNABLE_TOOLS) {
-    if (validateCommandBinary(toolName)) {
-      const defaultModel = resolveDefaultModel(toolName);
-      results.push({ toolName, defaultModel });
+  for (const tool of getScannableTools()) {
+    if (validateCommandBinary(tool.binary)) {
+      results.push({ toolName: tool.name, defaultModel: tool.models[0] });
     }
   }
   return results;
 }
-function resolveDefaultModel(toolName) {
-  if (DEFAULT_MODELS[toolName]) {
-    return DEFAULT_MODELS[toolName];
-  }
-  const registryModel = DEFAULT_REGISTRY.models.find((m) => m.tools.includes(toolName));
-  return registryModel?.name ?? toolName;
-}
 function generateConfig(tools) {
   const lines = [
     "# Auto-generated by opencara \u2014 edit to customize",
@@ -4541,22 +4461,29 @@ No config found at ${CONFIG_FILE}
     process.stdout.write("Continuing without gh \u2014 some features may be limited.\n");
   }
   process.stdout.write("\nScanning for AI tools...\n");
+  const scannableTools = getScannableTools();
   const found = discoverTools();
-  for (const tool of SCANNABLE_TOOLS) {
-    const disc = found.find((t) => t.toolName === tool);
+  for (const tool of scannableTools) {
+    const disc = found.find((t) => t.toolName === tool.name);
     if (disc) {
-      process.stdout.write(`  \u2713 ${tool} (${disc.defaultModel})
+      process.stdout.write(`  \u2713 ${tool.name} (${disc.defaultModel})
 `);
     } else {
-      process.stdout.write(`  \u2717 ${tool} (not found)
+      process.stdout.write(`  \u2717 ${tool.name} (not found)
 `);
     }
   }
   if (found.length === 0) {
-    process.stdout.write("\nNo AI tools found. Install one of: claude, codex, gemini\n");
-    for (const tool of SCANNABLE_TOOLS) {
-      process.stdout.write(`  ${tool}:  ${INSTALL_LINKS[tool]}
+    process.stdout.write(
+      `
+No AI tools found. Install one of: ${scannableTools.map((t) => t.name).join(", ")}
+`
+    );
+    for (const tool of scannableTools) {
+      if (tool.installLink) {
+        process.stdout.write(`  ${tool.name}:  ${tool.installLink}
 `);
+      }
     }
     return false;
   }
@@ -4649,21 +4576,39 @@ function buildBatchPollRequest(agents) {
   });
   return { agents: batchAgents };
 }
-function filterTasksForAgent(tasks, agent, maxDiffSizeKb, diffFailCounts, maxDiffFetchAttempts = 3, accessibleRepos) {
+function filterTasksForAgent(tasks, agent, maxDiffSizeKb, diffFailCounts, maxDiffFetchAttempts = 3, accessibleRepos, log) {
   return tasks.filter((t) => {
-    if (accessibleRepos && !accessibleRepos.has(`${t.owner}/${t.repo}`)) {
+    const repo = `${t.owner}/${t.repo}`;
+    if (accessibleRepos && !accessibleRepos.has(repo)) {
+      log?.(
+        `Skipping task ${t.task_id.slice(0, 8)}\u2026 (${repo} PR#${t.pr_number}) \u2014 repo not in accessible set`
+      );
       return false;
     }
     if (agent.repoConfig && !isRepoAllowed(agent.repoConfig, t.owner, t.repo, agent.agentOwner, agent.userOrgs)) {
+      log?.(
+        `Skipping task ${t.task_id.slice(0, 8)}\u2026 (${repo} PR#${t.pr_number}) \u2014 repo not allowed by config (mode=${agent.repoConfig.mode})`
+      );
       return false;
     }
     if (agent.synthesizeRepos && !isRepoAllowed(agent.synthesizeRepos, t.owner, t.repo, agent.agentOwner, agent.userOrgs)) {
+      log?.(
+        `Skipping task ${t.task_id.slice(0, 8)}\u2026 (${repo} PR#${t.pr_number}) \u2014 repo not allowed by synthesize_repos config`
+      );
       return false;
     }
-    if (maxDiffSizeKb && t.diff_size != null && t.diff_size * ESTIMATED_BYTES_PER_DIFF_LINE / 1024 > maxDiffSizeKb) {
+    const isExplicitlyListed = agent.repoConfig?.list?.includes(repo) ?? false;
+    if (!isExplicitlyListed && maxDiffSizeKb && t.diff_size != null && t.diff_size * ESTIMATED_BYTES_PER_DIFF_LINE / 1024 > maxDiffSizeKb) {
+      const estimatedKb = Math.round(t.diff_size * ESTIMATED_BYTES_PER_DIFF_LINE / 1024);
+      log?.(
+        `Skipping task ${t.task_id.slice(0, 8)}\u2026 (${repo} PR#${t.pr_number}) \u2014 estimated diff ${estimatedKb}KB exceeds max_diff_size_kb (${maxDiffSizeKb}KB)`
+      );
       return false;
     }
     if (diffFailCounts && (diffFailCounts.get(t.task_id) ?? 0) >= maxDiffFetchAttempts) {
+      log?.(
+        `Skipping task ${t.task_id.slice(0, 8)}\u2026 (${repo} PR#${t.pr_number}) \u2014 diff fetch failed ${maxDiffFetchAttempts} times`
+      );
       return false;
     }
     return true;
@@ -4690,9 +4635,9 @@ function resolveCommandTemplate(agentConfig, globalCommand) {
   if (agentConfig?.command) return agentConfig.command;
   if (globalCommand) return globalCommand;
   if (agentConfig?.tool) {
-    const registryTool = DEFAULT_REGISTRY.tools.find((t) => t.name === agentConfig.tool);
-    if (registryTool) {
-      return registryTool.commandTemplate.replaceAll("${MODEL}", agentConfig.model ?? "");
+    const toolDef = getToolDef(agentConfig.tool);
+    if (toolDef) {
+      return toolDef.command.replaceAll("${MODEL}", agentConfig.model ?? "");
     }
   }
   return void 0;
@@ -5750,7 +5695,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.10",
+    cliVersion: "0.23.12",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -5935,7 +5880,8 @@ async function batchPollLoop(client, agentStates, options) {
           state.reviewDeps.maxDiffSizeKb,
           state.diffFailCounts,
           MAX_DIFF_FETCH_ATTEMPTS,
-          accessibleRepos
+          accessibleRepos,
+          (msg) => state.logger.logWarn(`${icons.warn} ${msg}`)
         );
         const task = eligible[0];
         if (!task) continue;
@@ -6046,7 +5992,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.23.10",
+    cliVersion: "0.23.12",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6217,12 +6163,11 @@ async function startBatchAgents(config, agents, pollIntervalMs, oauthToken, opti
 async function startAgentRouter() {
   const config = loadConfig();
   const agentId = crypto2.randomUUID();
-  let commandTemplate;
   let agentConfig;
   if (config.agents && config.agents.length > 0) {
     agentConfig = config.agents.find((a) => a.router) ?? config.agents[0];
   }
-  commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
+  const commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
   const router = new RouterRelay();
   router.start();
   const logger = createLogger(agentConfig?.name ?? "agent[0]");
@@ -6294,12 +6239,11 @@ async function startAgentRouter() {
   router.stop();
 }
 function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versionOverride, verbose, instancesOverride, agentOwner, userOrgs) {
-  let commandTemplate;
   let agentConfig;
   if (config.agents && config.agents.length > agentIndex) {
     agentConfig = config.agents[agentIndex];
   }
-  commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
+  const commandTemplate = resolveCommandTemplate(agentConfig, config.agentCommand);
   const label = agentConfig?.name ?? `agent[${agentIndex}]`;
   if (!commandTemplate) {
     console.error(`[${label}] No command configured. Skipping.`);
@@ -6375,7 +6319,7 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
   return promises;
 }
 var agentCommand = new Command("agent").description("Manage review agents");
-agentCommand.command("start").description("Start agents in polling mode").option("--poll-interval <seconds>", "Poll interval in seconds", "10").option("--agent <index>", "Agent index from config.toml (0-based)", "0").option("--all", "Start all configured agents concurrently").option(
+agentCommand.command("start").description("Start agents in polling mode").option("--poll-interval <seconds>", "Poll interval in seconds", "10").option("--agent <index>", "Start a single agent by index from config.toml (0-based)").option("--all", "Start all configured agents concurrently (default when --agent is not set)").option(
   "--version-override <value>",
   "Cloudflare Workers version override (e.g. opencara-server=abc123)"
 ).option("-v, --verbose", "Log tool stdout/stderr after each review/summary for debugging").option("--instances <count>", "Number of concurrent instances per agent (overrides config)").action(
@@ -6393,7 +6337,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.23.10", "3fd8827"));
+    console.log(formatVersionBanner("0.23.12", "d2879c9"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -6462,21 +6406,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
     } else if (needsOrgs && userOrgs.size > 0) {
       console.log(`Org memberships: ${[...userOrgs].join(", ")}`);
     }
-    if (opts.all) {
-      if (!config.agents || config.agents.length === 0) {
-        console.error("No agents configured in ~/.opencara/config.toml");
-        process.exit(1);
-        return;
-      }
-      console.log(`Starting ${config.agents.length} agent config(s) in batch mode...`);
-      await startBatchAgents(config, config.agents, pollIntervalMs, oauthToken, {
-        versionOverride,
-        verbose: opts.verbose,
-        instancesOverride,
-        agentOwner,
-        userOrgs
-      });
-    } else {
+    if (opts.agent != null) {
       const maxIndex = (config.agents?.length ?? 0) - 1;
       const agentIndex = Number(opts.agent);
       if (!Number.isInteger(agentIndex) || agentIndex < 0 || agentIndex > maxIndex) {
@@ -6509,6 +6439,20 @@ agentCommand.command("start").description("Start agents in polling mode").option
         }
         process.exit(1);
       }
+    } else {
+      if (!config.agents || config.agents.length === 0) {
+        console.error("No agents configured in ~/.opencara/config.toml");
+        process.exit(1);
+        return;
+      }
+      console.log(`Starting ${config.agents.length} agent config(s) in batch mode...`);
+      await startBatchAgents(config, config.agents, pollIntervalMs, oauthToken, {
+        versionOverride,
+        verbose: opts.verbose,
+        instancesOverride,
+        agentOwner,
+        userOrgs
+      });
     }
   }
 );
@@ -6799,11 +6743,10 @@ function resolveAgentCommand(toolName) {
       if (cmd) return cmd;
     }
   }
-  const registryTool = DEFAULT_REGISTRY.tools.find((t) => t.name === toolName);
-  if (registryTool) {
-    const defaultModel = DEFAULT_REGISTRY.models.find((m) => m.tools.includes(toolName));
-    const modelName = defaultModel?.name ?? "";
-    return registryTool.commandTemplate.replaceAll("${MODEL}", modelName);
+  const toolDef = getToolDef(toolName);
+  if (toolDef) {
+    const modelName = toolDef.models[0] ?? "";
+    return toolDef.command.replaceAll("${MODEL}", modelName);
   }
   return null;
 }
@@ -7050,7 +6993,7 @@ async function runDedupInit(options, deps = {}) {
     const cmd = resolveCmd(options.agent);
     if (!cmd) {
       logError(
-        `${icons.error} Unknown agent tool "${options.agent}". Available: ${DEFAULT_REGISTRY.tools.map((t) => t.name).join(", ")}`
+        `${icons.error} Unknown agent tool "${options.agent}". Available: ${loadToolDefs().map((t) => t.name).join(", ")}`
       );
       process.exitCode = 1;
       return;
@@ -7105,13 +7048,13 @@ function agentRoleLabel(agent) {
   return "reviewer+synthesizer";
 }
 function resolveToolBinary(toolName) {
-  const entry = DEFAULT_REGISTRY.tools.find((t) => t.name === toolName);
-  return entry?.binary ?? toolName;
+  const def = getToolDef(toolName);
+  return def?.binary ?? toolName;
 }
 function resolveCommand(agent) {
   if (agent.command) return agent.command;
-  const entry = DEFAULT_REGISTRY.tools.find((t) => t.name === agent.tool);
-  return entry?.commandTemplate ?? null;
+  const def = getToolDef(agent.tool);
+  return def?.command ?? null;
 }
 async function checkConnectivity(platformUrl, fetchFn = fetch) {
   const start = Date.now();
@@ -7216,7 +7159,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.10"} (${"3fd8827"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.23.12"} (${"d2879c9"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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