@tablation/crew 0.47.3 → 0.48.0

package/dist/cli.js

@@ -9081,6 +9081,30 @@ function releaseIdle(i) {
 }
 var renderTag = (template, version) => template.replace(/\{version\}/g, version);
 
+// src/verified-tip.ts
+var PUSHED_RE = /^pushed ([0-9a-f]{40}) to \S+$/;
+function parsePushedSha(message) {
+  return PUSHED_RE.exec((message ?? "").trim())?.[1] ?? null;
+}
+function verifiedShaFor(ticket, verifiedShaColumn, pushedEvents) {
+  const col = ticket[verifiedShaColumn];
+  if (typeof col === "string" && /^[0-9a-f]{7,40}$/i.test(col.trim())) return col.trim();
+  const when = (e) => Date.parse(e.occurred_at ?? e.created_at ?? "") || 0;
+  const newest = pushedEvents.filter((e) => e.record_id === ticket.id && parsePushedSha(e.message)).sort((a, b) => when(b) - when(a))[0];
+  return newest ? parsePushedSha(newest.message) : null;
+}
+function movedPastVerified(cwd, branch, verified) {
+  const tip = gitOk(cwd, ["rev-parse", "-q", "--verify", `refs/heads/${branch}^{commit}`]);
+  const ver = gitOk(cwd, ["rev-parse", "-q", "--verify", `${verified}^{commit}`]);
+  if (!tip || !ver || tip === ver) return null;
+  if (isAncestor(cwd, tip, ver)) return null;
+  if (isAncestor(cwd, ver, tip)) {
+    const own = gitOk(cwd, ["rev-list", "--first-parent", "--no-merges", "--count", `${ver}..${tip}`]);
+    if (own === "0") return null;
+  }
+  return { tip, verified: ver };
+}
+
 // src/stamp.ts
 function planStamp(cwd, tickets, contract, fromSha, toSha, merged = /* @__PURE__ */ new Map()) {
   const range = fromSha ? `${fromSha.slice(0, 8)}..${toSha.slice(0, 8)}` : null;
@@ -9280,7 +9304,7 @@ ${h.prFailure.tail}
   }
   for (const q of outcome.requeues) {
     const t = q.ticket;
-    const body = `The branch is no longer what was verified: \`verified_sha\` is ${q.verified.slice(0, 8)}, the branch is now at ${q.head.slice(0, 8)}. Returned to ${contract.statuses.handoff} for QA to check what changed.`;
+    const body = `The branch is no longer what was verified: QA verified ${q.verified.slice(0, 8)}, the branch is now at ${q.head.slice(0, 8)}. Returned to ${contract.statuses.handoff} for QA to check what changed.`;
     if (dryRun) {
       emit.emit(`would return to ${contract.statuses.handoff} \u2014 branch moved past ${q.verified.slice(0, 8)}`, { ticket: t.issue_id });
       continue;
@@ -9691,6 +9715,11 @@ function reportHeld(o, held) {
   o.state?.noteHeldReported?.(signature);
 }
 async function runRelease(o) {
+  const movedPast2 = [];
+  const out = await runReleaseCycle(o, movedPast2);
+  return movedPast2.length ? { ...out, movedPast: movedPast2 } : out;
+}
+async function runReleaseCycle(o, movedPast2) {
   const scoped = (msg) => o.scope ? `${o.scope}: ${msg}` : msg;
   const tagPattern = o.repo.release.tagPattern ?? "v*";
   o.emit.enter("release");
@@ -9707,6 +9736,23 @@ async function runRelease(o) {
     }
     if (skipped.size) releasable = o.tickets.filter((t) => !skipped.has(t.id));
   }
+  if (o.verifiedShas?.size) {
+    const drop = /* @__PURE__ */ new Set();
+    for (const t of releasable) {
+      const verified = o.verifiedShas.get(t.id);
+      if (t.status !== o.contract.statuses.verified || !verified) continue;
+      const branch = locateBranchForTicket(o.cwd, o.repo, t)?.branch;
+      const moved = branch ? movedPastVerified(o.cwd, branch, verified) : null;
+      if (!moved) continue;
+      drop.add(t.id);
+      movedPast2.push({ ticket: t, verified: moved.verified, head: moved.tip });
+      o.emit.error(
+        `not releasing ${t.issue_id}: ${branch} is at ${moved.tip.slice(0, 8)}, past the ${moved.verified.slice(0, 8)} QA verified`,
+        { ticket: t.issue_id }
+      );
+    }
+    if (drop.size) releasable = releasable.filter((t) => !drop.has(t.id));
+  }
   const decision = decideRelease(o.cwd, releasable, o.contract, {
     tagPattern,
     base: o.repo.branch.base,
@@ -13462,6 +13508,21 @@ ${changed2.map((c) => `  - ${c}
       );
       reportUnplaceable(c.route, unplaceable, tracker.contract.statuses.verified, remit);
       const blockerInfo = blockerInfoMap(all, await tracker.ticketsByIds(missingBlockerIds(all)));
+      const verifiedStatus = tracker.contract.statuses.verified;
+      const verifiedTickets = tickets.filter((t) => t.status === verifiedStatus);
+      const verifiedShas = /* @__PURE__ */ new Map();
+      if (verifiedTickets.length && repo.release.mode !== "external") {
+        let pushedEvents = [];
+        try {
+          pushedEvents = await tracker.events("pushed");
+        } catch (e) {
+          remit.warn(`could not read hand-off events; not checking branches against what QA verified: ${e.message}`);
+        }
+        for (const t of verifiedTickets) {
+          const sha = verifiedShaFor(t, tracker.contract.columns.verifiedSha, pushedEvents);
+          if (sha) verifiedShas.set(t.id, sha);
+        }
+      }
       const outcome = await runRelease({
         cwd: target.dir,
         repo,
@@ -13470,6 +13531,7 @@ ${changed2.map((c) => `  - ${c}
         blockerInfo,
         emit: remit,
         scope,
+        verifiedShas,
         state: state.release(scope),
         reviewColumns: repo.release.mode === "external" ? await tracker.reviewColumns() : void 0,
         dryRun,
@@ -13565,6 +13627,18 @@ ${changed2.map((c) => `  - ${c}
           dryRun
         );
       }
+      if (outcome.movedPast?.length) {
+        await applyReview(
+          tracker,
+          tracker.contract,
+          { handoffs: [], requeues: outcome.movedPast },
+          repo.branch.remote,
+          remit,
+          releaseSeat(c),
+          { verifiedSha: false, prRef: false },
+          dryRun
+        );
+      }
       const news = describeRelease(outcome, scope);
       if (news) {
         const notified = await notify(c, cfg.ship, news, remit, dryRun);

package/package.json

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