@tablation/crew 0.54.2 → 0.54.4

package/dist/cli.js

@@ -3404,6 +3404,69 @@ function validateContract(c) {
 }
 var EVENT_LOG_MISSING = "the Issues table has no event log \u2014 crew writes its audit trail there (CREW-1574). In the app, open the Issues table's Data & lifecycle tab and turn on Event log, then rerun.";
 
+// src/callout.ts
+var calloutHeader = (kind, label) => `> [!${kind}]
+> **Crew** \u2014 ${label}`;
+var calloutLabelLine = (label) => `> **Crew** \u2014 ${label}`;
+
+// src/gate-red.ts
+var GATE_RED_MARKER = calloutHeader("CAUTION", "Test Gate Red");
+var shaLine = (g) => `Head \`${g.sha.slice(0, 8)}\`.`;
+var GATE_RED_AFTER = 3;
+var lastLines = (output, n) => output.replace(/\u001b\[[0-9;]*m/g, "").trimEnd().split("\n").slice(-n).join("\n");
+function gateRedTickets(merged, merges) {
+  const seen = /* @__PURE__ */ new Set();
+  const out = [];
+  for (const c of [...merged, ...merges.filter((m) => m.skipReason === "already-merged")]) {
+    if (seen.has(c.ticket.id)) continue;
+    seen.add(c.ticket.id);
+    out.push(c.ticket);
+  }
+  return out;
+}
+function gateRedComment(g) {
+  return `${GATE_RED_MARKER}
+> ${shaLine(g)}
+
+**This ticket is merged on the base, but the release test gate is red \u2014 nothing has shipped.**
+
+The \`test\` hook (\`${g.hook}\`) has failed ${g.count} cycles running on the same head (\`${g.sha.slice(0, 8)}\`). The ticket's commit is on the base and will ship once the gate passes; it is not stranded, and re-setting it to \`verified\` will not help. Last lines of the hook's output:
+
+\`\`\`
+${g.tail}
+\`\`\`
+`;
+}
+var alreadyReported = (ticket, g, events) => events.some((e) => e.record_id === ticket.id && e.event_type === "gate_red" && (e.message ?? "").includes(shaLine(g)));
+
+// src/test-evidence.ts
+var MAX_FAIL_LINES = 20;
+var ANSI = /\u001b\[[0-9;]*m/g;
+var FAIL_LINE = /^\s*(FAIL\b|●\s|not ok\b)/;
+var ERROR_LINE = /\b(?:[A-Z]\w*)?Error\b[:\s]/;
+function failureSummary(output, tailLines = 15) {
+  const lines = output.replace(ANSI, "").trimEnd().split("\n");
+  const names = [];
+  for (const l of lines) {
+    if (FAIL_LINE.test(l) && !names.includes(l.trim())) names.push(l.trim());
+    if (names.length >= MAX_FAIL_LINES) break;
+  }
+  const at = lines.findIndex((l) => ERROR_LINE.test(l));
+  if (names.length === 0 && at < 0) return lastLines(output, tailLines);
+  const parts = [...names];
+  if (at >= 0) parts.push("", ...lines.slice(at, at + 4).map((l) => l.trimEnd()));
+  return parts.join("\n").trim();
+}
+var headLine = (head) => `Candidate head \`${head.slice(0, 8)}\`.`;
+function describeComposition(c) {
+  const rows = c.branches.length ? c.branches.map((b) => `- ${b.key} \`${b.branch}\` @ \`${b.tip.slice(0, 8)}\``).join("\n") : "- (none merged this cycle)";
+  return `${headLine(c.head)} Base before merging \`${c.base.slice(0, 8)}\`. Merged branches:
+
+${rows}
+
+The gate runs on the base plus every verified branch above, not on the base alone: reproduce on that composition.`;
+}
+
 // src/shell.ts
 import { spawn } from "child_process";
 import { platform } from "process";
@@ -3515,7 +3578,13 @@ function describeRelease(o, route) {
     return {
       level: "fail",
       headline: `${route}: release blocked \u2014 tests failed`,
-      detail: "the target stays on the previous release"
+      detail: withHookTail(
+        o.composition ? `the target stays on the previous release
+
+${describeComposition(o.composition)}` : "the target stays on the previous release",
+        o.hookFailure
+      ),
+      ...o.composition ? { head: o.composition.head } : {}
     };
   }
   if (o.stopped === "setup failed") {
@@ -6497,11 +6566,6 @@ function materializeBranchForTicket(dir, cfg, t, role, onProblem) {
 import { writeFileSync as writeFileSync7 } from "fs";
 import { join as join11 } from "path";
 
-// src/callout.ts
-var calloutHeader = (kind, label) => `> [!${kind}]
-> **Crew** \u2014 ${label}`;
-var calloutLabelLine = (label) => `> **Crew** \u2014 ${label}`;
-
 // src/stalled.ts
 var STALLED_AFTER_CYCLES = 30;
 var CYCLE_MS = 120 * 1e3;
@@ -7229,36 +7293,6 @@ async function applyStrandedVerified(writer, s, contract, operatorId, memberId,
   }
 }
 
-// src/gate-red.ts
-var GATE_RED_MARKER = calloutHeader("CAUTION", "Test Gate Red");
-var shaLine = (g) => `Head \`${g.sha.slice(0, 8)}\`.`;
-var GATE_RED_AFTER = 3;
-var lastLines = (output, n) => output.replace(/\u001b\[[0-9;]*m/g, "").trimEnd().split("\n").slice(-n).join("\n");
-function gateRedTickets(merged, merges) {
-  const seen = /* @__PURE__ */ new Set();
-  const out = [];
-  for (const c of [...merged, ...merges.filter((m) => m.skipReason === "already-merged")]) {
-    if (seen.has(c.ticket.id)) continue;
-    seen.add(c.ticket.id);
-    out.push(c.ticket);
-  }
-  return out;
-}
-function gateRedComment(g) {
-  return `${GATE_RED_MARKER}
-> ${shaLine(g)}
-
-**This ticket is merged on the base, but the release test gate is red \u2014 nothing has shipped.**
-
-The \`test\` hook (\`${g.hook}\`) has failed ${g.count} cycles running on the same head (\`${g.sha.slice(0, 8)}\`). The ticket's commit is on the base and will ship once the gate passes; it is not stranded, and re-setting it to \`verified\` will not help. Last lines of the hook's output:
-
-\`\`\`
-${g.tail}
-\`\`\`
-`;
-}
-var alreadyReported = (ticket, g, events) => events.some((e) => e.record_id === ticket.id && e.event_type === "gate_red" && (e.message ?? "").includes(shaLine(g)));
-
 // src/agent.ts
 import { spawn as spawn2, spawnSync as spawnSync5 } from "child_process";
 import { readFileSync as readFileSync12, existsSync as existsSync14, statSync as statSync5, mkdirSync as mkdirSync6, writeFileSync as writeFileSync8, createWriteStream, openSync as openSync3, closeSync as closeSync3, readSync } from "fs";
@@ -10507,7 +10541,7 @@ var isolateCap = (o) => o.repo.release.isolateFailures?.maxCandidates ?? DEFAULT
 async function isolateAndRetry(o, name, output, merged, preMerge, decision, fresh, releaseBranch) {
   const applied = merged.filter((c) => c.sha && c.branch);
   const cap = isolateCap(o);
-  if (o.dryRun || !preMerge || applied.length < 2 || cap === 0) return null;
+  if (o.dryRun || !preMerge || applied.length < 1 || cap === 0) return null;
   const head = headSha(o.cwd);
   const withFiles = applied.map((c) => ({
     c,
@@ -10520,6 +10554,17 @@ async function isolateAndRetry(o, name, output, merged, preMerge, decision, fres
   if (byFile) {
     o.emit.warn(`${name} failed; its output names files only ${byFile.c.branch} changed \u2014 treating it as the culprit`, { ticket: byFile.c.ticket.issue_id });
     culprits.push({ c: byFile.c, how: "files", tail: lastLines(output, HOOK_FAILURE_TAIL_LINES) });
+  } else if (applied.length === 1) {
+    const only = applied[0];
+    gitOk(o.cwd, ["reset", "--hard", preMerge]);
+    const baseRun = await hook(o, name);
+    if (baseRun && baseRun.code !== 0) {
+      o.emit.warn(`${name} fails on the base without ${only.branch} \u2014 not blaming the branch`, { ticket: only.ticket.issue_id });
+      gitOk(o.cwd, ["reset", "--hard", head]);
+      return null;
+    }
+    o.emit.warn(`${name} failed with only ${only.branch} merged and passes without it \u2014 treating it as the culprit`, { ticket: only.ticket.issue_id });
+    culprits.push({ c: only, how: "solo", tail: lastLines(output, HOOK_FAILURE_TAIL_LINES) });
   } else if (applied.length > cap) {
     o.emit.warn(`${name} failed with ${applied.length} branches merged, over release.isolateFailures.maxCandidates (${cap}) \u2014 not isolating`);
     return null;
@@ -10682,7 +10727,21 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch, isolated = fal
         const sha = headSha(o.cwd);
         const count = o.state?.noteTestGateFailed?.(sha) ?? 1;
         const gateRed = count >= GATE_RED_AFTER && !o.state?.testGateReported?.(sha) ? { sha, count, hook: hookLabel(o.repo, "test"), tail: lastLines(r.output, 15) } : void 0;
-        return { merged, conflicts, unbuildable, deployed: false, stopped: "tests failed", decision, gateRed };
+        const composition = {
+          head: sha,
+          base: preMerge,
+          branches: merged.filter((c) => c.branch).map((c) => ({
+            key: c.ticket.issue_tag ?? c.ticket.issue_id,
+            branch: c.branch,
+            tip: resolve2(o.cwd, c.branch) ?? ""
+          }))
+        };
+        const hookFailure2 = {
+          hook: "test",
+          label: hookLabel(o.repo, "test"),
+          tail: failureSummary(r.output, HOOK_FAILURE_TAIL_LINES)
+        };
+        return { merged, conflicts, unbuildable, deployed: false, stopped: "tests failed", decision, gateRed, hookFailure: hookFailure2, composition };
       }
       o.state?.clearTestGateFailed?.();
     }
@@ -10708,7 +10767,7 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch, isolated = fal
   if (o.repo.hooks.build) {
     if (o.dryRun) {
       o.emit.emit(`would run: ${hookLabel(o.repo, "build")}`);
-      if (applied > 1) {
+      if (applied >= 1) {
         o.emit.emit(
           `if it fails, would find the one of ${applied} merged branches that broke it (by the files it names, else by building each alone, up to ${isolateCap(o)}), hand that back to the dev lane and ship the rest`
         );
@@ -10854,9 +10913,15 @@ async function applyFailureAlert(writer, n, route, openTickets, memberId, log, d
       return { kind: "commented", issueId: label };
     }
     try {
+      let body = n.detail ?? n.headline;
+      if (n.head && writer.events) {
+        const line = headLine(n.head);
+        const shown = (await writer.events("failure_alert")).some((e) => e.record_id === existing.id && (e.message ?? "").includes(line));
+        if (shown) body = `${n.headline} \u2014 ${line}`;
+      }
       await writer.postEvent(existing.id, `${FAILURE_ALERT_MARKER}
 
-Still failing: ${n.detail ?? n.headline}`, memberId, "failure_alert");
+Still failing: ${body}`, memberId, "failure_alert");
       log.warn(`release failure recurring \u2014 commented on ${label}`, { step: "release" });
       return { kind: "commented", issueId: label };
     } catch (e) {
@@ -10878,6 +10943,12 @@ Still failing: ${n.detail ?? n.headline}`, memberId, "failure_alert");
       ...placement.project_id ? { project_id: placement.project_id } : {}
     }, memberId);
     const label = created.issue_id ? displayKey(created) : created.id;
+    if (n.head) {
+      await writer.postEvent(created.id, `${FAILURE_ALERT_MARKER}
+
+Filed with the evidence above. ${headLine(n.head)}`, memberId, "failure_alert").catch(() => {
+      });
+    }
     log.warn(`release failure \u2014 filed ${label}`, { step: "release" });
     return { kind: "filed", issueId: label };
   } catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.54.2",
+  "version": "0.54.4",
   "description": "A standing team of headless agents that picks work off a Tablation board, does it on a machine you control, and reports back.",
   "type": "module",
   "license": "MIT",