@tablation/crew 0.3.0 → 0.4.0

package/dist/cli.js

@@ -1328,6 +1328,8 @@ var State = class _State {
     const suffix = _State.safe(route);
     const blocked = `.release-blocked-${suffix}`;
     const failed = `.deploy-failed-sha-${suffix}`;
+    const gateFailed = `.test-gate-failed-${suffix}`;
+    const gateReported = `.test-gate-reported-${suffix}`;
     const read = (f) => {
       try {
         return readFileSync3(this.path(f), "utf8").trim() || null;
@@ -1361,6 +1363,27 @@ var State = class _State {
       },
       clearDeployFailed: () => {
         rmSync(this.path(failed), { force: true });
+      },
+      /**
+       * Consecutive test-gate failures on the same head (CREW-1368). A failure
+       * on a new head restarts the count. Held on disk because one process is
+       * one cycle, and "failed three cycles running" only exists across them.
+       */
+      noteTestGateFailed: (sha) => {
+        const [prev, n] = (read(gateFailed) ?? "").split(" ");
+        const next = prev === sha ? (Number.parseInt(n ?? "", 10) || 0) + 1 : 1;
+        writeFileSync(this.path(gateFailed), `${sha} ${next}
+`);
+        return next;
+      },
+      clearTestGateFailed: () => {
+        rmSync(this.path(gateFailed), { force: true });
+      },
+      /** Whether the board was already told about this head's red gate. */
+      testGateReported: (sha) => read(gateReported) === sha,
+      noteTestGateReported: (sha) => {
+        writeFileSync(this.path(gateReported), `${sha}
+`);
       }
     };
   }
@@ -2869,6 +2892,10 @@ var countCommits = (cwd, range) => Number.parseInt(gitOk(cwd, ["rev-list", "--co
 function latestReleaseTag(cwd, pattern = "v*", ref = "HEAD") {
   return gitOk(cwd, ["describe", "--tags", "--abbrev=0", "--match", pattern, ref]);
 }
+function firstReleaseTagContaining(cwd, sha, pattern = "v*") {
+  const out = gitOk(cwd, ["tag", "--contains", sha, "--list", pattern, "--sort=creatordate"]);
+  return out?.split("\n").map((l) => l.trim()).find(Boolean) ?? null;
+}
 var tagCommit = (cwd, tag) => resolve2(cwd, tag);
 function createReleaseTag(cwd, tag, sha, message) {
   git(cwd, ["tag", "-a", tag, sha, "-m", message]);
@@ -3579,7 +3606,7 @@ function flagComment(s) {
   return `${STRANDED_MARKER}
 **Verified, but nothing to merge \u2014 flagging before this repeats forever.**
 
-\`${key}\` is \`verified\` with no branch on this ship, and its key names no commit on \`${s.base}\` (searched \`${s.remote}/${s.base}\`'s history for a bounded \`${key}\` in a commit subject or body). One of three things is true: it landed under a commit that never named the key, the branch was deleted before it merged, or it was verified in error \u2014 the crew cannot tell these apart and will not guess.
+\`${key}\` is \`verified\` with no branch on this ship, and its key names no commit anywhere in this ship's \`${s.base}\` history (searched all of it, local commits included, for a bounded \`${key}\` in a commit subject or body). One of three things is true: it landed under a commit that never named the key, the branch was deleted before it merged, or it was verified in error \u2014 the crew cannot tell these apart and will not guess.
 
 If it is still unplaceable next cycle, this ticket is escalated to a person. If the work already landed, closing this ticket by hand now avoids that.`;
 }
@@ -3588,7 +3615,7 @@ function escalateComment2(s) {
   return `${STRANDED_MARKER}
 **Still nothing to merge \u2014 stopping rather than repeating.**
 
-\`${key}\` was flagged as stranded at \`verified\` on an earlier cycle and still has no branch on this ship or on \`${s.remote}\` and no commit on \`${s.base}\` naming its key (searched \`${s.remote}/${s.base}\`). This needs a person: either the work landed under a commit that doesn't name the key and this ticket can simply close, or the branch is gone and the work needs redoing.
+\`${key}\` was flagged as stranded at \`verified\` on an earlier cycle and still has no branch on this ship or on \`${s.remote}\` and no commit in this ship's \`${s.base}\` history naming its key (searched all of it, local commits included). This needs a person: either the work landed under a commit that doesn't name the key and this ticket can simply close, or the branch is gone and the work needs redoing.
 
 The crew will not close this on its own.`;
 }
@@ -3632,6 +3659,33 @@ async function applyStrandedVerified(writer, s, contract, operatorId, memberId,
   }
 }
 
+// src/gate-red.ts
+var GATE_RED_MARKER = "<!-- crew:test-gate-red";
+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} ${g.sha.slice(0, 8)} -->
+**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, comments) => comments.some((c) => c.ticket_id === ticket.id && (c.body ?? "").includes(`${GATE_RED_MARKER} ${g.sha.slice(0, 8)}`));
+
 // src/agent.ts
 import { spawn } from "child_process";
 import { readFileSync as readFileSync5, existsSync as existsSync6, statSync as statSync2, mkdirSync as mkdirSync4, writeFileSync as writeFileSync3, createWriteStream } from "fs";
@@ -4888,7 +4942,32 @@ function planMerge(cwd, tickets, contract, lastReleased, branchFor = (t) => bran
   return verified.map((ticket) => {
     const branch = branchFor(ticket);
     if (!branch) {
-      const onMain = lastReleased ? findKeyInRange(cwd, referenceKeys(ticket), `${lastReleased}..HEAD`) !== null : false;
+      const keys = referenceKeys(ticket);
+      const unreleased = lastReleased ? findKeyInRange(cwd, keys, `${lastReleased}..HEAD`) : null;
+      if (unreleased) {
+        return {
+          ticket,
+          branch: null,
+          entries: [],
+          usedFallback: false,
+          bump: "patch",
+          majorRequested: false,
+          skipReason: "already-merged"
+        };
+      }
+      const anywhere = findKeyInRange(cwd, keys, "HEAD");
+      if (anywhere) {
+        return {
+          ticket,
+          branch: null,
+          entries: [],
+          usedFallback: false,
+          bump: "patch",
+          majorRequested: false,
+          skipReason: lastReleased ? "already-released" : "already-merged",
+          mergedSha: anywhere
+        };
+      }
       return {
         ticket,
         branch: null,
@@ -4896,7 +4975,7 @@ function planMerge(cwd, tickets, contract, lastReleased, branchFor = (t) => bran
         usedFallback: false,
         bump: "patch",
         majorRequested: false,
-        skipReason: onMain ? "already-merged" : "never-built"
+        skipReason: "never-built"
       };
     }
     const squash = findSquashOfBranch(
@@ -5465,7 +5544,12 @@ async function runRelease(o) {
   let applied = 0;
   for (const c of decision.merges) {
     if (!c.branch || c.mergedSha) {
-      if (c.skipReason === "already-merged") {
+      if (c.skipReason === "already-released") {
+        o.emit.emit(
+          `verified and already released (on the base as ${c.mergedSha?.slice(0, 7)}), waiting to be stamped`,
+          { ticket: c.ticket.issue_id }
+        );
+      } else if (c.skipReason === "already-merged") {
         o.emit.emit(
           `verified and already merged${c.mergedSha ? ` (on the base as ${c.mergedSha.slice(0, 7)})` : ""}, waiting on a successful release`,
           { ticket: c.ticket.issue_id }
@@ -5570,8 +5654,12 @@ async function runRelease(o) {
       }
       if (r && r.code !== 0) {
         o.emit.error(`test gate FAILED \u2014 not deploying; the target stays on the previous release`);
-        return { merged, conflicts, unbuildable, deployed: false, stopped: "tests failed", decision };
+        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 };
       }
+      o.state?.clearTestGateFailed?.();
     }
   }
   const size = merged.some((m) => m.bump === "minor") ? "minor" : "patch";
@@ -8983,6 +9071,36 @@ ${changed2.map((c) => `  - ${c}
       } else if (outcome.stopped) {
         remit.emit(`nothing stamped \u2014 ${outcome.stopped}`);
       }
+      for (const m of outcome.decision.merges) {
+        if (m.skipReason !== "already-released" || !m.mergedSha) continue;
+        const tag = firstReleaseTagContaining(target.dir, m.mergedSha, repo.release.tagPattern ?? "v*");
+        if (!tag) continue;
+        remit.enter("reconcile");
+        await applyStamp(
+          tracker,
+          [{ ticket: m.ticket, reason: `carried by ${tag}`, sha: m.mergedSha }],
+          tag.replace(/^v/, ""),
+          tracker.contract,
+          remit,
+          dryRun,
+          target.dir
+        );
+      }
+      if (outcome.gateRed && !dryRun) {
+        try {
+          const g = outcome.gateRed;
+          const comments = await tracker.comments();
+          const seat = c.resolved?.seats.dev ?? c.resolved?.seats.qa ?? "";
+          for (const t of gateRedTickets(outcome.merged, outcome.decision.merges)) {
+            if (alreadyReported(t, g, comments)) continue;
+            await tracker.postEvent(t.id, gateRedComment(g), seat);
+            remit.warn(`test gate red for ${g.count} cycles \u2014 noted on the ticket`, { ticket: t.issue_id, step: "merge" });
+          }
+          state.release(scope).noteTestGateReported(g.sha);
+        } catch (e) {
+          remit.warn(`could not note the red test gate on the board: ${e.message}`, { step: "merge" });
+        }
+      }
       if (outcome.externalClosures?.length) {
         await applyExternalClosures(tracker, outcome.externalClosures, tracker.contract, remit, dryRun, target.dir);
       }

package/package.json

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