@tablation/crew 0.54.1 → 0.54.3

package/dist/cli.js

@@ -6229,7 +6229,16 @@ function operatorTodo(tickets, contract, operatorId) {
 }
 
 // src/digest.ts
-var DIRECTION_EVENT_TYPES = ["conflict_handed_back", "requeued", "bounced", "conflict_escalated", "interrupted"];
+var DIRECTION_EVENT_TYPES = [
+  "conflict_handed_back",
+  "requeued",
+  "bounced",
+  "conflict_escalated",
+  "interrupted",
+  // The release gate's isolation path hands a ticket back as `build_failed`: it builds alone but not merged with the base.
+  "build_failed"
+];
+var STATUS_EVENT_TYPES = ["gate_red"];
 function displayKey2(t) {
   return t.issue_tag ?? t.issue_id ?? "?";
 }
@@ -6278,10 +6287,18 @@ function pendingDirection(i, ticketId) {
 }
 function directionCell(i, ticketId) {
   const d = pendingDirection(i, ticketId);
-  if (!d) return "\u2014";
+  if (!d) return statusCell(i, ticketId);
   const msg = (d.message ?? "").replace(/\s+/g, " ").trim().slice(0, 160);
   return `${d.event_type} ${eventAt(d).slice(0, 16)}Z${msg ? ` \u2014 ${msg}` : ""}`.replace(/\|/g, "/");
 }
+function statusCell(i, ticketId) {
+  const mine = (i.events ?? []).filter((e) => e.record_id === ticketId);
+  const pushed = mine.filter((e) => e.event_type === "pushed").map(eventAt).sort().pop() ?? "";
+  const s = mine.filter((e) => STATUS_EVENT_TYPES.includes(e.event_type) && eventAt(e) > pushed).sort((a, b) => eventAt(a) < eventAt(b) ? 1 : -1)[0];
+  if (!s) return "\u2014";
+  const msg = (s.message ?? "").replace(/\s+/g, " ").trim().slice(0, 160);
+  return `${s.event_type} ${eventAt(s).slice(0, 16)}Z (status only, nothing to build)${msg ? ` \u2014 ${msg}` : ""}`.replace(/\|/g, "/");
+}
 function newFromOthers(i, ticketId) {
   const comments = i.comments.filter(
     (c) => c.ticket_id === ticketId && c.team_member_id !== i.me && c.created_at > i.watermark
@@ -6364,7 +6381,7 @@ ${sorted(held).map((t) => `| ${displayKey2(t)} | ${stopOf(t)} |`).join("\n")}
     "## Current queue \u2014 built for you by the poll\n",
     "\nAlready filtered to your lane, and already ordered by the Step 2 rule.\nTicket bodies are deliberately omitted: fetch the full record of only the\nticket you actually pick up. **Do not re-fetch the whole tracker.** This\ndigest comes from the same API call the poll just made, moments ago.\n",
     '\n"new since last poll" counts comments from someone other than you since\nthe poll watermark \u2014 the same signal that woke this run.\n',
-    '\n"runner direction" is the newest instruction the crew itself posted to the\nevent log (not Comments) that no push has answered yet: `conflict_handed_back`\n(merge the base into the branch and resolve), `requeued`, `bounced`,\n`conflict_escalated`, `interrupted` (an earlier run on this seat was cut short; its\nuncommitted worktree is yours, resume it). It is direction, same as a comment from a hold: act on it.\n',
+    '\n"runner direction" is the newest instruction the crew itself posted to the\nevent log (not Comments) that no push has answered yet: `conflict_handed_back`\n(merge the base into the branch and resolve), `requeued`, `bounced`,\n`conflict_escalated`, `interrupted` (an earlier run on this seat was cut short; its\nuncommitted worktree is yours, resume it), `build_failed` (the release gate found the branch builds alone but\nnot merged with the current base: merge the base in, fix, push, hand back to QA; the output is quoted). It is\ndirection, same as a comment from a hold: act on it. `gate_red` is status only: the ticket is merged and the\ngate is red, so there is nothing to build on the branch.\n',
     "\n**`repo` is the checkout a ticket's work happens in** \u2014 an area spans\nseveral repositories, so cut the worktree beside THAT directory, not beside\nwhichever one you started in. A ticket marked **NO CHECKOUT** is not yours:\nthis ship has no clone of its repository, and another ship may serve it.\n\n**Use the `branch` and `worktree` columns verbatim.** They are that\nrepository's own naming convention, rendered for THIS ticket \u2014 cut the\nworktree at `<repo>/../<worktree>`, on branch `<branch>`. Do not derive\neither yourself, and prefer these over the Environment section's own worked\nexample if the two ever disagree \u2014 that example is illustrative only,\nrendered before any ticket is picked.\n",
     ...i.workModes && [...i.workModes.modes.values()].some((m) => m !== "automatic") ? ["\n**Manual / hybrid projects.** Some projects on this board are not fully automatic. In a `manual` project no lane touches any ticket, and a ticket with no project is treated the same way, so none appear below. In a `hybrid` project a ticket appears below only when it is assigned to an agent. An `accepted` ticket that is absent for either reason is not yours to claim \u2014 do not go looking for it.\n"] : [],
     "\n### Step 1 \u2014 open tickets that may be yours to act on\n",
@@ -6603,7 +6620,7 @@ async function decideCycle(o) {
     optional("project work modes", tracker.projectWorkModes(), /* @__PURE__ */ new Map()),
     tracker.events("stalled")
   ]);
-  const directionEvents = (await Promise.all([...DIRECTION_EVENT_TYPES, "pushed"].map((t) => tracker.events(t)))).flat();
+  const directionEvents = (await Promise.all([...DIRECTION_EVENT_TYPES, ...STATUS_EVENT_TYPES, "pushed"].map((t) => tracker.events(t)))).flat();
   emit.emit(`${tickets.length} open ticket(s), ${comments.length} comment(s)`, {
     data: { tickets: tickets.length, comments: comments.length }
   });
@@ -10490,7 +10507,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,
@@ -10503,6 +10520,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;
@@ -10691,7 +10719,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`
         );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.54.1",
+  "version": "0.54.3",
   "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",