opencara 0.24.3 → 0.25.1

package/dist/index.js

@@ -499,6 +499,7 @@ function ensureConfigDir() {
   fs.mkdirSync(dir, { recursive: true });
 }
 var DEFAULT_MAX_DIFF_SIZE_KB = 100;
+var DEFAULT_MAX_SUMMARY_INPUT_KB = 500;
 var DEFAULT_MAX_CONSECUTIVE_ERRORS = 10;
 var DEFAULT_MAX_REPO_SIZE_MB = 100;
 var DEFAULT_COMMAND_TEST_TIMEOUT_MS = 1e4;
@@ -705,6 +706,12 @@ function validateConfigData(data, envPlatformUrl) {
     );
     overrides.maxDiffSizeKb = DEFAULT_MAX_DIFF_SIZE_KB;
   }
+  if (typeof data.max_summary_input_kb === "number" && data.max_summary_input_kb <= 0) {
+    console.warn(
+      `\u26A0 Config warning: max_summary_input_kb must be > 0, got ${data.max_summary_input_kb}, using default (${DEFAULT_MAX_SUMMARY_INPUT_KB})`
+    );
+    overrides.maxSummaryInputKb = DEFAULT_MAX_SUMMARY_INPUT_KB;
+  }
   if (typeof data.max_consecutive_errors === "number" && data.max_consecutive_errors <= 0) {
     console.warn(
       `\u26A0 Config warning: max_consecutive_errors must be > 0, got ${data.max_consecutive_errors}, using default (${DEFAULT_MAX_CONSECUTIVE_ERRORS})`
@@ -745,6 +752,7 @@ function loadConfig() {
     platformUrl: envPlatformUrl || DEFAULT_PLATFORM_URL,
     authFile: null,
     maxDiffSizeKb: DEFAULT_MAX_DIFF_SIZE_KB,
+    maxSummaryInputKb: DEFAULT_MAX_SUMMARY_INPUT_KB,
     maxConsecutiveErrors: DEFAULT_MAX_CONSECUTIVE_ERRORS,
     maxRepoSizeMb: DEFAULT_MAX_REPO_SIZE_MB,
     codebaseDir: null,
@@ -804,6 +812,7 @@ function loadConfig() {
     platformUrl: envPlatformUrl || (typeof data.platform_url === "string" ? data.platform_url : DEFAULT_PLATFORM_URL),
     authFile: typeof data.auth_file === "string" && data.auth_file.trim() ? resolveFilePath(data.auth_file) : null,
     maxDiffSizeKb: overrides.maxDiffSizeKb ?? (typeof data.max_diff_size_kb === "number" ? data.max_diff_size_kb : DEFAULT_MAX_DIFF_SIZE_KB),
+    maxSummaryInputKb: overrides.maxSummaryInputKb ?? (typeof data.max_summary_input_kb === "number" ? data.max_summary_input_kb : DEFAULT_MAX_SUMMARY_INPUT_KB),
     maxConsecutiveErrors: overrides.maxConsecutiveErrors ?? (typeof data.max_consecutive_errors === "number" ? data.max_consecutive_errors : DEFAULT_MAX_CONSECUTIVE_ERRORS),
     maxRepoSizeMb: overrides.maxRepoSizeMb ?? (typeof data.max_repo_size_mb === "number" ? data.max_repo_size_mb : DEFAULT_MAX_REPO_SIZE_MB),
     codebaseDir: typeof data.codebase_dir === "string" ? data.codebase_dir : null,
@@ -1880,18 +1889,71 @@ function gitExec(command, args, cwd, opts) {
     throw new Error(sanitizeTokens(message));
   }
 }
-function diffFromWorktree(bareRepoPath, worktreePath, baseRef, ghAvailable, maxDiffBytes = 128 * 1024 * 1024) {
-  if (!/^[A-Za-z0-9_./-]+$/.test(baseRef) || baseRef.startsWith("-")) {
-    throw new Error(`Invalid base ref: ${baseRef}`);
-  }
+var DEFAULT_BRANCH_FALLBACKS = ["main", "master"];
+function isValidBranchName(name) {
+  if (!name) return false;
+  if (name.startsWith("-")) return false;
+  if (name.includes("..")) return false;
+  return /^[A-Za-z0-9_./-]+$/.test(name);
+}
+function fetchBranch(bareRepoPath, branch, ghAvailable) {
   const credArgs = ghAvailable ? ["-c", `credential.helper=${GH_CREDENTIAL_HELPER}`] : [];
   gitExec(
     "git",
-    [...credArgs, "fetch", "--force", "origin", `${baseRef}:refs/remotes/origin/${baseRef}`],
+    [...credArgs, "fetch", "--force", "origin", `${branch}:refs/remotes/origin/${branch}`],
     bareRepoPath
   );
+}
+function isRemoteRefMissingError(err) {
+  const msg = err instanceof Error ? err.message : String(err);
+  return /couldn't find remote ref/i.test(msg) || /couldnt find remote ref/i.test(msg) || /no such ref/i.test(msg) || /remote ref.*not found/i.test(msg) || /unknown revision or path not in the working tree/i.test(msg);
+}
+function deriveDefaultBranch(bareRepoPath, ghAvailable) {
+  try {
+    const out = gitExec("git", ["symbolic-ref", "refs/remotes/origin/HEAD"], bareRepoPath).trim();
+    const prefix = "refs/remotes/origin/";
+    if (out.startsWith(prefix)) {
+      const branch = out.slice(prefix.length);
+      if (isValidBranchName(branch)) {
+        try {
+          fetchBranch(bareRepoPath, branch, ghAvailable);
+          return branch;
+        } catch {
+        }
+      }
+    }
+  } catch {
+  }
+  for (const candidate of DEFAULT_BRANCH_FALLBACKS) {
+    try {
+      fetchBranch(bareRepoPath, candidate, ghAvailable);
+      return candidate;
+    } catch {
+    }
+  }
+  throw new Error("Cannot derive default branch: origin/HEAD, main, and master all failed");
+}
+function diffFromWorktree(bareRepoPath, worktreePath, baseRef, ghAvailable, maxDiffBytes = 128 * 1024 * 1024) {
+  let resolvedBaseRef;
+  if (baseRef) {
+    if (!isValidBranchName(baseRef)) {
+      throw new Error(`Invalid base ref: ${baseRef}`);
+    }
+    try {
+      fetchBranch(bareRepoPath, baseRef, ghAvailable);
+      resolvedBaseRef = baseRef;
+    } catch (err) {
+      if (!isRemoteRefMissingError(err)) {
+        throw err;
+      }
+      resolvedBaseRef = void 0;
+    }
+  }
+  if (!resolvedBaseRef) {
+    resolvedBaseRef = deriveDefaultBranch(bareRepoPath, ghAvailable);
+  }
   try {
-    return gitExec("git", ["diff", `origin/${baseRef}...HEAD`], worktreePath, {
+    return gitExec("git", ["diff", `origin/${resolvedBaseRef}...HEAD`], worktreePath, {
       maxBuffer: maxDiffBytes
     });
   } catch (err) {
@@ -2560,7 +2622,7 @@ function sleep(ms, signal) {
 
 // src/summary.ts
 var TIMEOUT_SAFETY_MARGIN_MS2 = 3e4;
-var MAX_INPUT_SIZE_BYTES = 200 * 1024;
+var MAX_INPUT_SIZE_BYTES = 500 * 1024;
 var InputTooLargeError = class extends Error {
   constructor(message) {
     super(message);
@@ -2610,9 +2672,10 @@ function calculateInputSize(prompt2, reviews, diffContent, contextBlock) {
 }
 async function executeSummary(req, deps, runTool = executeTool) {
   const inputSize = calculateInputSize(req.prompt, req.reviews, req.diffContent, req.contextBlock);
-  if (inputSize > MAX_INPUT_SIZE_BYTES) {
+  const maxInputBytes = deps.maxSummaryInputKb !== void 0 ? deps.maxSummaryInputKb * 1024 : MAX_INPUT_SIZE_BYTES;
+  if (inputSize > maxInputBytes) {
     throw new InputTooLargeError(
-      `Summary input too large (${Math.round(inputSize / 1024)}KB > ${Math.round(MAX_INPUT_SIZE_BYTES / 1024)}KB limit)`
+      `Summary input too large (${Math.round(inputSize / 1024)}KB > ${Math.round(maxInputBytes / 1024)}KB limit)`
     );
   }
   const timeoutMs = req.timeout * 1e3;
@@ -4687,6 +4750,12 @@ function registerShutdownHandlers(controller, log, graceMs = SHUTDOWN_GRACE_MS)
   };
 }
 var NON_RETRYABLE_STATUSES = /* @__PURE__ */ new Set([401, 403, 404]);
+function build404Hint(isGhAuthenticated) {
+  if (isGhAuthenticated) {
+    return ". Diff fetch returned 404. Possible causes: (a) PR not found, (b) the installation can't access this repo, or (c) a transient GitHub outage. See any `[fetchDiffViaGh]` warning above for the underlying `gh api` error.";
+  }
+  return ". If this is a private repo, ensure gh CLI is installed and authenticated: gh auth login";
+}
 function toApiDiffUrl(webUrl) {
   const match = webUrl.match(/^https?:\/\/github\.com\/([^/]+)\/([^/]+)\/pull\/(\d+)(?:\.diff)?$/);
   if (!match) return null;
@@ -4748,7 +4817,7 @@ function computeRoles(agent) {
   return ["review", "summary", "implement", "fix"];
 }
 var DIFF_FETCH_TIMEOUT_MS = 6e4;
-async function fetchDiffHttp(url, headers, signal, maxDiffSizeKb) {
+async function fetchDiffHttp(url, headers, signal, maxDiffSizeKb, isGhAuthenticatedFn = isGhAvailable) {
   const maxBytes = maxDiffSizeKb ? maxDiffSizeKb * 1024 : Infinity;
   const controller = new AbortController();
   const timer = setTimeout(() => controller.abort(), DIFF_FETCH_TIMEOUT_MS);
@@ -4769,7 +4838,7 @@ async function fetchDiffHttp(url, headers, signal, maxDiffSizeKb) {
   clearTimeout(timer);
   signal?.removeEventListener("abort", onParentAbort);
   if (!response.ok) {
-    const hint = response.status === 404 ? ". If this is a private repo, ensure gh CLI is installed and authenticated: gh auth login" : "";
+    const hint = response.status === 404 ? build404Hint(isGhAuthenticatedFn()) : "";
     const msg = `Failed to fetch diff: ${response.status} ${response.statusText}${hint}`;
     if (NON_RETRYABLE_STATUSES.has(response.status)) {
       throw new NonRetryableError(msg);
@@ -5054,7 +5123,12 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
       );
       taskReviewDeps = { ...reviewDeps, codebaseDir: null };
     }
-    if (taskCheckoutPath && taskBareRepoPath && base_ref) {
+    if (taskCheckoutPath && taskBareRepoPath) {
+      if (!base_ref) {
+        logWarn(
+          `  Warning: task ${task_id} has no base_ref \u2014 deriving default branch from worktree`
+        );
+      }
       try {
         const ghAvailable = isGhAvailable();
         const maxDiffBytes = reviewDeps.maxDiffSizeKb ? reviewDeps.maxDiffSizeKb * 1024 : void 0;
@@ -5077,23 +5151,16 @@ async function handleTask(client, agentId, task, reviewDeps, consumptionDeps, ag
         diffContent = gitDiff;
         log(`  Diff generated via git (${Math.round(diffContent.length / 1024)}KB)`);
       } catch (err) {
+        const message = err.message;
         if (err instanceof DiffTooLargeError) {
-          logError(`  ${err.message}`);
-          await safeReject(
-            client,
-            task_id,
-            agentId,
-            `Cannot access diff: ${err.message}`,
-            logger
-          );
-          return { diffFetchFailed: true };
+          logError(`  ${message}`);
+        } else {
+          logError(`  git diff failed for task ${task_id}: ${message}`);
         }
-        logWarn(
-          `  Warning: git diff failed (${err.message}) \u2014 falling back to API fetch`
-        );
+        await safeReject(client, task_id, agentId, `Cannot access diff: ${message}`, logger);
+        return { diffFetchFailed: true };
       }
-    }
-    if (!diffContent) {
+    } else {
       try {
         const result = await fetchDiff(diff_url, owner, repo, pr_number, {
           githubToken: client.currentToken,
@@ -5736,7 +5803,7 @@ function sleep2(ms, signal) {
 async function startAgent(agentId, platformUrl, agentInfo, reviewDeps, consumptionDeps, options) {
   const client = new ApiClient(platformUrl, {
     authToken: options?.authToken,
-    cliVersion: "0.24.3",
+    cliVersion: "0.25.1",
     versionOverride: options?.versionOverride,
     onTokenRefresh: options?.onTokenRefresh
   });
@@ -6101,7 +6168,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.24.3",
+    cliVersion: "0.25.1",
     versionOverride,
     onTokenRefresh: () => getValidToken(config.platformUrl, { configPath: config.authFile })
   });
@@ -6307,6 +6374,7 @@ async function startAgentRouter() {
   const reviewDeps = {
     commandTemplate: commandTemplate ?? "",
     maxDiffSizeKb: config.maxDiffSizeKb,
+    maxSummaryInputKb: config.maxSummaryInputKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
     codebaseDir,
     livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
@@ -6373,6 +6441,7 @@ function startAgentByIndex(config, agentIndex, pollIntervalMs, oauthToken, versi
   const reviewDeps = {
     commandTemplate,
     maxDiffSizeKb: config.maxDiffSizeKb,
+    maxSummaryInputKb: config.maxSummaryInputKb,
     maxRepoSizeMb: config.maxRepoSizeMb,
     codebaseDir,
     livenessTimeoutMs: agentConfig?.livenessTimeout != null ? agentConfig.livenessTimeout * 1e3 : void 0
@@ -6450,7 +6519,7 @@ agentCommand.command("start").description("Start agents in polling mode").option
       }
       config = loadConfig();
     }
-    console.log(formatVersionBanner("0.24.3", "c08a9b3"));
+    console.log(formatVersionBanner("0.25.1", "085f9a8"));
     if (config.agents && config.agents.length > 0) {
       const toolEntries = config.agents.map((a) => ({
         tool: a.tool,
@@ -7272,7 +7341,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.24.3"} (${"c08a9b3"})`);
+var program = new Command5().name("opencara").description("OpenCara \u2014 distributed AI code review agent").version(`${"0.25.1"} (${"085f9a8"})`);
 program.addCommand(agentCommand);
 program.addCommand(authCommand());
 program.addCommand(dedupCommand());

package/package.json

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