@tablation/crew 0.19.3 → 0.19.5

package/dist/cli.js

@@ -2706,7 +2706,9 @@ function pushTag(cwd, remote, tag) {
   git(cwd, ["push", remote, tag]);
 }
 function fetchRemote(cwd, remote = "origin") {
-  return gitOk(cwd, ["fetch", "--prune", "--quiet", remote]) !== null;
+  if (gitOk(cwd, ["fetch", "--prune", "--quiet", remote]) === null) return false;
+  gitOk(cwd, ["fetch", "--quiet", remote, "refs/tags/*:refs/tags/*"]);
+  return true;
 }
 function remoteConfigured(cwd, remote = "origin") {
   return gitOk(cwd, ["remote", "get-url", remote]) !== null;
@@ -6432,6 +6434,15 @@ function describeUnplaceable(unplaceable, verifiedStatus) {
   }
   return notes;
 }
+var unplaceableReported = /* @__PURE__ */ new Set();
+function reportUnplaceable(routeName, unplaceable, verified, emit) {
+  if (unplaceableReported.has(routeName)) return;
+  unplaceableReported.add(routeName);
+  for (const note of describeUnplaceable(unplaceable, verified)) {
+    if (note.level === "warn") emit.warn(note.message, { step: "release" });
+    else emit.emit(note.message, { step: "release" });
+  }
+}
 function checkGuards(cwd, base = "main") {
   const branch = currentBranch(cwd);
   if (branch !== base) {
@@ -6561,6 +6572,7 @@ ${section}`;
   }
   return [...lines.slice(0, anchor), section, ...lines.slice(anchor)].join("\n");
 }
+var RELEASE_COMMIT_SUBJECT = /^Release \S+$/;
 function decideRelease(cwd, tickets, contract, opts = {}) {
   const pattern = opts.tagPattern ?? "v*";
   const base = opts.base ?? "main";
@@ -6570,6 +6582,7 @@ function decideRelease(cwd, tickets, contract, opts = {}) {
   const head = resolve2(cwd, "HEAD") ?? "";
   const merges = block ? [] : planMerge(cwd, tickets, contract, lastReleased, opts.branchFor ?? ((t) => branchForIssue(cwd, t.issue_id)), base);
   const unreleasedCommits = lastReleased ? countCommits(cwd, `${lastReleased}..HEAD`) : 0;
+  const unreleasedWork = lastReleased ? (gitOk(cwd, ["log", "--format=%s", `${lastReleased}..HEAD`]) ?? "").split("\n").filter((l) => l && !RELEASE_COMMIT_SUBJECT.test(l)).length : 0;
   return {
     block,
     merges,
@@ -6577,10 +6590,14 @@ function decideRelease(cwd, tickets, contract, opts = {}) {
     lastTag,
     lastReleased,
     unreleasedCommits,
-    upToDate: lastReleased === head,
+    upToDate: lastReleased === head || lastReleased !== null && unreleasedWork === 0,
     unseeded: lastTag === null
   };
 }
+function releaseIdle(i) {
+  if (i.force || i.external || i.unseeded || !i.upToDate || i.unstamped > 0) return false;
+  return !i.tickets.some((t) => t.status === i.verifiedStatus);
+}
 var renderTag = (template, version) => template.replace(/\{version\}/g, version);
 
 // src/stamp.ts
@@ -7489,7 +7506,8 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch) {
       o.emit.emit(`tagged ${tag}`);
     }
   }
-  if (tag && (o.repo.release.mode === "ci_manual" || o.repo.release.mode === "ci_auto")) {
+  const tagIsCiTrigger = o.repo.release.mode === "ci_manual" || o.repo.release.mode === "ci_auto";
+  if (tag && (tagIsCiTrigger || remoteConfigured(o.cwd, o.repo.branch.remote))) {
     if (o.dryRun) {
       o.emit.emit(`would push tag ${tag} to ${o.repo.branch.remote}`);
     } else {
@@ -7497,8 +7515,12 @@ async function mergeAndRelease(o, decision, fresh, releaseBranch) {
         pushTag(o.cwd, o.repo.branch.remote, tag);
         o.emit.emit(`pushed tag ${tag} to ${o.repo.branch.remote}`);
       } catch (e) {
-        o.emit.error(`failed to push tag ${tag} to ${o.repo.branch.remote} \u2014 ${e.message}`);
-        return { merged, conflicts, unbuildable, version, tag, deployed, integrated, decision, stopped: "tag push failed" };
+        const msg = `failed to push tag ${tag} to ${o.repo.branch.remote} \u2014 ${e.message}`;
+        if (tagIsCiTrigger) {
+          o.emit.error(msg);
+          return { merged, conflicts, unbuildable, version, tag, deployed, integrated, decision, stopped: "tag push failed" };
+        }
+        o.emit.warn(`${msg}; other ships will not see this release marker`);
       }
     }
   }
@@ -10517,15 +10539,8 @@ if (command === "passenger-sync-daemon") {
     const out = refreshBaseBranch(r.dir, remote, base, true);
     const unsafe = baseBranchUnsafe(out);
     return { action: out.action, unsafe, detail: unsafe ? describeUnsafeBase(out, remote, base) : out.detail };
-  }, reportUnplaceable = function(c, unplaceable, verified, emit2) {
-    if (unplaceableReported.has(c.route)) return;
-    unplaceableReported.add(c.route);
-    for (const note of describeUnplaceable(unplaceable, verified)) {
-      if (note.level === "warn") emit2.warn(note.message, { step: "release" });
-      else emit2.emit(note.message, { step: "release" });
-    }
   };
-  requireArmed2 = requireArmed, parseTicketNumber2 = parseTicketNumber, passengerApiKeys2 = passengerApiKeys, endpointApiFor2 = endpointApiFor, endpointReconcileDeps2 = endpointReconcileDeps, releaseRefused2 = releaseRefused, baseHealth2 = baseHealth, reportUnplaceable2 = reportUnplaceable;
+  requireArmed2 = requireArmed, parseTicketNumber2 = parseTicketNumber, passengerApiKeys2 = passengerApiKeys, endpointApiFor2 = endpointApiFor, endpointReconcileDeps2 = endpointReconcileDeps, releaseRefused2 = releaseRefused, baseHealth2 = baseHealth;
   let cfg;
   let firstRun;
   try {
@@ -10659,6 +10674,7 @@ if (command === "passenger-sync-daemon") {
   }
   async function releaseFleet(opts = {}) {
     if (releaseRefused()) return;
+    precheckTickets.clear();
     const summaries = [];
     if (!dryRun) state.releaseHeartbeat().touch();
     for (const c of cfg.routes) {
@@ -10670,6 +10686,7 @@ if (command === "passenger-sync-daemon") {
   }
   async function releaseTargets(opts = {}) {
     if (releaseRefused()) return;
+    precheckTickets.clear();
     const only = value("repo");
     const targets = reposOf(route);
     const chosen = only ? targets.filter((t) => t.name === only) : targets;
@@ -10906,7 +10923,7 @@ ${changed2.map((c) => `  - ${c}
     }
     return `${who2}; ${lock2}`;
   }
-  const unplaceableReported = /* @__PURE__ */ new Set();
+  const precheckTickets = /* @__PURE__ */ new Map();
   async function releasePhase(c, target, opts = {}) {
     const scope = releaseLockScope(c.route, target.name);
     const remit = emit.forRoute(c.route);
@@ -10941,6 +10958,38 @@ ${changed2.map((c) => `  - ${c}
       }
       if (!dryRun && !c.enabled) return { scope, tests: "skipped", outcome: "skipped", detail: "route not enabled" };
       const tracker = new Tracker(c, cfg.ship);
+      if (!dryRun) {
+        const force2 = command === "deploy" || flag("force");
+        let seen = precheckTickets.get(c.route);
+        if (!seen) precheckTickets.set(c.route, seen = tracker.openTickets());
+        const { byRepo: preByRepo, unplaceable: preUnplaceable } = ticketsByRepo(c, await seen);
+        const pre = preByRepo.get(target.name) ?? [];
+        reportUnplaceable(c.route, preUnplaceable, tracker.contract.statuses.verified, remit);
+        const d = decideRelease(target.dir, pre, tracker.contract, {
+          tagPattern: repo.release.tagPattern ?? "v*",
+          base: repo.branch.base
+        });
+        const unstamped = planReleasedBranches(
+          target.dir,
+          pre,
+          tracker.contract,
+          (t) => locateBranchForTicket(target.dir, repo, t)?.branch ?? null,
+          repo.branch.base,
+          repo.release.tagPattern ?? "v*"
+        ).length;
+        if (releaseIdle({
+          tickets: pre,
+          verifiedStatus: tracker.contract.statuses.verified,
+          force: force2,
+          external: repo.release.mode === "external",
+          upToDate: d.upToDate,
+          unseeded: d.unseeded,
+          unstamped
+        })) {
+          remit.emit(`${target.name}: nothing verified \u2014 not locking`, { step: "release" });
+          return { scope, tests: "skipped", outcome: "skipped", detail: "nothing verified \u2014 not locking" };
+        }
+      }
       if (!dryRun) {
         const holderLabel = `${cfg.ship.name}:${process.pid}`;
         const got = await tracker.acquireBoardLock(scope, holderLabel, RELEASE_LOCK_TTL_MS);
@@ -10963,7 +11012,7 @@ ${changed2.map((c) => `  - ${c}
       const all = await tracker.openTickets();
       const { byRepo, unplaceable } = ticketsByRepo(c, all);
       const tickets = byRepo.get(target.name) ?? [];
-      reportUnplaceable(c, unplaceable, tracker.contract.statuses.verified, remit);
+      reportUnplaceable(c.route, unplaceable, tracker.contract.statuses.verified, remit);
       const outcome = await runRelease({
         cwd: target.dir,
         repo,
@@ -13249,4 +13298,3 @@ var endpointApiFor2;
 var endpointReconcileDeps2;
 var releaseRefused2;
 var baseHealth2;
-var reportUnplaceable2;

package/package.json

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