@tablation/crew 0.49.0 → 0.49.2

package/dist/cli.js

@@ -8691,8 +8691,13 @@ ${v.reason[0].toUpperCase()}${v.reason.slice(1)}. Set back to \`in_progress\`, s
 async function enforceHandoff(i) {
   try {
     const t = await i.writer.ticket(i.ticketId);
-    if (t.status !== i.contract.statuses.handoff) return { kind: "not-handed-off" };
+    const st = i.contract.statuses;
     const log = { ticket: t.issue_id, step: "handoff" };
+    const claimed = [st.verifying, st.verified, st.deployed].some((s) => s && t.status === s);
+    if (t.status !== st.handoff && !claimed) {
+      i.log.emit(`${t.issue_id} is at ${t.status}, not handed off \u2014 nothing to check`, log);
+      return { kind: "not-handed-off" };
+    }
     const v = inspectHandoff(i.dir, i.cfg, t, i.role);
     if (v.kind === "unverified") {
       i.log.warn(`${t.issue_id} is at ${t.status} but the push was not checked: ${v.why}`, log);
@@ -8703,6 +8708,10 @@ async function enforceHandoff(i) {
       i.log.emit(`${t.issue_id} pushed at ${v.sha.slice(0, 7)} \u2014 handed on to QA`, log);
       return { kind: "pushed", sha: v.sha };
     }
+    if (claimed) {
+      i.log.warn(`${t.issue_id} is at ${t.status} but its branch is not on the remote at HEAD: ${v.reason}`, log);
+      return { kind: "failed", why: v.reason };
+    }
     const heldBy = i.hold ? t[i.hold.column] : void 0;
     await i.writer.updateTicket(t.id, {
       status: i.contract.statuses.building,
@@ -9244,10 +9253,10 @@ function movedPast(r, t, branch, pushedAs) {
   const head = resolve2(r.cwd, branch);
   const remoteHead = resolve2(r.cwd, `${r.repo.branch.remote}/${pushedAs}`);
   if (remoteHead && !isAncestor(r.cwd, remoteHead, verified) && remoteHead !== head) {
-    return { ticket: t, verified, head: remoteHead };
+    return { ticket: t, verified, head: remoteHead, branch: pushedAs };
   }
   if (head && head !== verified && !isAncestor(r.cwd, head, verified)) {
-    return { ticket: t, verified, head };
+    return { ticket: t, verified, head, branch: pushedAs };
   }
   return null;
 }
@@ -9336,6 +9345,12 @@ ${h.prFailure.tail}
     try {
       await tracker.updateTicket(t.id, { [contract.columns.status]: contract.statuses.handoff }, t.updated_at);
       await tracker.postEvent(t.id, body, seat, "requeued");
+      await tracker.postEvent(
+        t.id,
+        pushedEventBody(q.head, remote, q.branch),
+        seat,
+        "pushed"
+      );
       emit.emit(`returned to ${contract.statuses.handoff} \u2014 branch moved past ${q.verified.slice(0, 8)}`, { ticket: t.issue_id });
     } catch (e) {
       emit.warn(`could not return to ${contract.statuses.handoff}: ${e.message}`, { ticket: t.issue_id });
@@ -9769,7 +9784,7 @@ async function runReleaseCycle(o, movedPast2) {
       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 });
+      movedPast2.push({ ticket: t, verified: moved.verified, head: moved.tip, branch: pushedBranchForTicket(o.repo, t) });
       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 }

package/package.json

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