mergejudge-runner 0.4.4 → 0.4.5

package/README.md

@@ -124,6 +124,7 @@ Logs go to stderr.
 
 ## Changelog
 
+- **0.4.5** (2026-10-17): policy files share the total budget fairly (MergeJudge #100): a large first file (e.g. AGENTS.md) no longer silently drops later ones (e.g. APPROVAL_POLICY.md); truncation is reported. Local-code mode keeps one extra char per file so the cut is visible. No config or protocol changes; works with older Workers.
 - **0.4.4** (2026-10-17): learning v2 (MergeJudge #94/#96/#97): the runner prompt now includes the repo's team preferences (known false alarms, recent writer lessons, outcome hints) sent by the Worker, re-sanitised and capped; context only, never lowers a lane. Additive job field; works with older Workers. No config changes.
 - **0.4.3** (2026-10-17): bundles the MergeJudge #92 rules fix: a safety guard (e.g. `persist-credentials: false`) removed together with its whole YAML step or job no longer forces block. No config or protocol changes.
 - **0.4.2** (2026-10-17): a heartbeat answered with `done` after the result was accepted is treated as benign, not as a lost lease (MergeJudge #86). In deep-local mode the runner secret-scans the full local diff, including files GitHub strips from the API diff (large or generated files) (MergeJudge #87). No config changes.

package/dist/cli.js

@@ -30535,28 +30535,48 @@ async function retrieveDeepContext(snap, files, opts = {}, log = () => {
 // ../../src/policy.ts
 var POLICY_MAX_EACH = 8 * 1024;
 var POLICY_MAX_TOTAL = 8 * 1024;
+function fairShares(sizes, total) {
+  const out = sizes.map(() => 0);
+  let left = Math.max(0, total);
+  let pending = sizes.map((_, i) => i).sort((x, y) => sizes[x] - sizes[y]);
+  while (pending.length) {
+    const share = Math.floor(left / pending.length);
+    const i = pending[0];
+    if (sizes[i] <= share) {
+      out[i] = sizes[i];
+      left -= sizes[i];
+      pending = pending.slice(1);
+      continue;
+    }
+    for (const j of pending) out[j] = share;
+    break;
+  }
+  return out;
+}
 async function loadPolicies(paths, fetchFile, caps = { each: POLICY_MAX_EACH, total: POLICY_MAX_TOTAL }) {
-  const parts = [];
-  const used = [];
-  let total = 0;
+  const found = [];
+  const skipped = [];
   for (const path2 of paths) {
-    if (total >= caps.total) break;
     let t;
     try {
       t = await fetchFile(path2);
     } catch {
       t = null;
     }
-    if (!t || !t.trim()) continue;
-    const budget = Math.min(caps.each, caps.total - total);
+    if (!t || !t.trim()) skipped.push(path2);
+    else found.push({ path: path2, t });
+  }
+  const shares = fairShares(found.map((f) => Math.min(f.t.length, caps.each)), caps.total);
+  const parts = [];
+  const used = [];
+  found.forEach(({ path: path2, t }, i) => {
+    const budget = shares[i];
     const truncated = t.length > budget;
-    const body = truncated ? t.slice(0, budget) + "\n[... truncated by MergeJudge ...]" : t;
-    total += Math.min(t.length, budget);
     parts.push(`--- ${path2} ---
-${body}`);
+${truncated ? t.slice(0, budget) + "\n[... truncated by MergeJudge ...]" : t}`);
     used.push({ path: path2, truncated });
-  }
-  return { text: parts.join("\n\n"), used };
+  });
+  return { text: parts.join("\n\n"), used, skipped };
 }
 function filterPolicyPaths(paths, exclude) {
   if (!exclude.length) return paths;
@@ -31445,7 +31465,7 @@ async function prepareLocalJob(o, j, log, signal) {
         const policySource = cfgSha === j.baseSha ? base : new GitSource(dir, cfgSha, o.env, o.limits, signal);
         for (const p of filterPolicyPaths(j.cfg.policy_files, j.cfg.policy_exclude)) {
           const t = await policySource.readFile(p).catch(() => null);
-          if (t) policyFiles[p] = t.slice(0, j.cfg.policy.max_each_bytes);
+          if (t) policyFiles[p] = t.slice(0, j.cfg.policy.max_each_bytes + 1);
         }
         if (policySource !== base) policySource.close();
         notes.push("Deep mode on a runner with a local clone: repo context was retrieved from an index on this machine and reviewed in one pass (no tool calls).");
@@ -32423,7 +32443,7 @@ async function produceReviewResult(o) {
   const labels = (pr.labels ?? []).map((l) => l.name);
   if (o.providerNote && cfg.provider.type !== "none") notes.push(`LLM disabled (${!provider && o.keyNote || o.providerNote}); rules-only.`);
   let llm = null;
-  let policies = { text: "", used: [] };
+  let policies = { text: "", used: [], skipped: [] };
   let tokens2 = 0;
   let usedMode = cfg.review.mode;
   let breakdown = null;
@@ -32432,15 +32452,16 @@ async function produceReviewResult(o) {
     llm = cached2.llm;
     tokens2 = cached2.tokens;
     breakdown = cached2.breakdown;
-    policies = { text: "", used: cached2.policiesUsed };
+    policies = { text: "", used: cached2.policiesUsed, skipped: [] };
     log.info(`Re-using LLM result for head ${pr.head.sha.slice(0, 7)} (no LLM call).`);
   } else if (provider) {
     const paths = filterPolicyPaths(cfg.policy_files, cfg.policy_exclude);
     policies = paths.length ? await loadPolicies(paths, (p) => o.readRepoFile(p, cfgRef), {
       each: cfg.policy.max_each_bytes,
       total: cfg.policy.max_total_bytes
-    }) : { text: "", used: [] };
-    if (policies.used.length) log.info(`Policy files: ${policies.used.map((u) => u.path).join(", ")}`);
+    }) : { text: "", used: [], skipped: [] };
+    if (policies.used.length) log.info(`Policy files: ${policies.used.map((u) => u.path).join(", ")}${policies.used.some((u) => u.truncated) ? " (truncated)" : ""}`);
+    if (policies.skipped.length) log.info(`Policy files not found or empty: ${policies.skipped.join(", ")}`);
     else if (!cfg.policy_files.length) log.info("Policy files disabled (policy_files: []).");
     const criteria = cfg.rubrics.enabled ? selectCriteria(cfg.rubrics.packs, cfg.rubrics.custom, cfg.rubrics.notes) : [];
     const { kept, skipped } = excludeFromLlm(files, cfg.diff_exclude);
@@ -32790,7 +32811,7 @@ function installAutostart(d, bundle) {
 }
 
 // src/version.ts
-var VERSION = "0.4.4";
+var VERSION = "0.4.5";
 var MODES = ["diff", "context", "deep"];
 var FEATURES = ["local-code"];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mergejudge-runner",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "license": "MIT",
   "type": "module",
   "description": "Run MergeJudge reviews on your own machine with your own Claude Code login (runner protocol /runner/v1)",