@feigi/fleet-ctl 3.22.7 → 3.22.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feigi/fleet-ctl",
-  "version": "3.22.7",
+  "version": "3.22.9",
   "description": "Agent fleet: run-team controller, merge bot, PR reviewer, and the ticket pipeline they share",
   "license": "Apache-2.0",
   "repository": {

package/scripts/reaping-prose.test.mjs

@@ -113,6 +113,34 @@ test("run-team/SKILL.md: the release-claims section carries the same limitation"
   );
 });
 
+test("run-team/SKILL.md: the release deletes the branch with update-ref's compare-and-swap, not `-D`", () => {
+  // release-ticket.sh deletes with `git update-ref -d refs/heads/<branch> <tip>`
+  // on a tip read once, which refuses if the ref moved off that tip. SKILL.md
+  // kept saying `-D` after the script stopped using it, so a controller read
+  // the release as a force-delete of whatever the ref held at that moment.
+  const section = between(
+    RUN_TEAM,
+    "### Release the claims that never became PRs",
+    "**That refusal is the",
+    "run-team/SKILL.md",
+  );
+  assert.match(
+    section,
+    phrase("deletes the branch with `git update-ref -d` on the tip it read once"),
+    "SKILL.md's release section no longer names `git update-ref -d` on a tip read once as the branch delete",
+  );
+  assert.match(
+    section,
+    phrase("compare-and-swap that refuses if the branch has moved off that tip"),
+    "SKILL.md's release section no longer says the update-ref delete is a compare-and-swap that refuses if the branch has moved off the tip",
+  );
+  assert.doesNotMatch(
+    section,
+    /(?:deletes?|deleted)\s+(?:the\s+branch\s+)?(?:is\s+)?with\s+`(?:git\s+branch\s+)?-D`/,
+    "SKILL.md's release section says the branch is deleted with `-D` — release-ticket.sh deletes it with `git update-ref -d`",
+  );
+});
+
 test("run-team/SKILL.md: the by-hand fallback names worktree-audit.sh's no-argument contract", () => {
   // #525. The by-hand fallback for a claim `release-ticket.sh` refused sends
   // the reader to `worktree-audit.sh`, which takes NO argument and audits

package/skills/run-team/SKILL.md

@@ -3149,9 +3149,12 @@ immediately before the branch delete, because a commit can land across the
 pushed-branch checks are **not** re-run there — a commit landing in that window
 is ahead of `origin/main` by construction, so the recount covers it. All clear →
 drops the label, removes the worktree without `--force`, deletes the branch with
-`-D`, authorized by those two commit checks plus the recount and by nothing else
-(`-d` measures against local HEAD, and a claim has no upstream, so it refuses a
-pristine claim whenever local `main` is behind `origin/main` — #760). Any one of
+`git update-ref -d` on the tip it read once before the recount — a
+compare-and-swap that refuses if the branch has moved off that tip — authorized
+by those two commit checks, the recount and a fresh check that no worktree holds
+the branch, and by nothing else (not `git branch -d`: it measures against local
+HEAD, and a claim has no upstream, so it refuses a pristine claim whenever local
+`main` is behind `origin/main`). Any one of
 them failing → it touches nothing and names the blocker. **That refusal is the
 finding, never an obstacle**: a claim carrying commits or a pushed branch is not
 auto-released, ever — run `worktree-audit.sh` (it audits every worktree; find