@mmerterden/multi-agent-pipeline 17.6.0 → 19.0.0

package/pipeline/scripts/write-state.mjs

@@ -16,12 +16,22 @@
  *
  *   # Replace: full overwrite from stdin JSON
  *   cat new-state.json | node write-state.mjs --replace /path/to/agent-state.json
+ *   cat patch.json | node write-state.mjs --if-rev=7 /path/to/agent-state.json
+ *
+ * `--if-rev=<n>` is compare-and-swap: write only if the record on disk is
+ * still at revision n. The lock closes the window between two WRITERS; it does
+ * nothing about the one between a read and a write, which is where an editor
+ * lives - `/multi-agent:steer` reads the state, a human thinks, and the run may
+ * move three phases before the edit lands. Without this the edit merges over
+ * the top and reinstates decisions the run already replaced.
  *
  * Exit codes:
  *   0  -  write succeeded
  *   1  -  invalid JSON on stdin
  *   2  -  lock timeout (another writer held the lock past the acquire window)
+ *   4  -  the lock was taken from us mid-write; nothing was written, retry
  *   3  -  I/O error (disk full, permission denied, parent dir missing)
+ *   5  -  --if-rev did not match; the record moved, nothing was written
  *
  * Tunables (env):
  *   WRITE_STATE_LOCK_TIMEOUT_MS  acquire window before exit 2 (default 15000)
@@ -52,9 +62,11 @@ import {
 } from "fs";
 import { dirname, basename } from "path";
 import { Buffer } from "node:buffer";
+import { runMain } from "../lib/fatal.mjs";
+import { invokedDirectly } from "../lib/invoked-directly.mjs";
 
 /**
- * @typedef {{ replace: boolean, path: string }} ParsedArgs
+ * @typedef {{ replace: boolean, path: string, ifRev: number|null }} ParsedArgs
  */
 
 /** @returns {ParsedArgs} */
@@ -62,18 +74,26 @@ function parseArgs() {
   const args = process.argv.slice(2);
   let replace = false;
   let path = null;
+  let ifRev = null;
   for (const a of args) {
     if (a === "--replace") replace = true;
-    else if (a.startsWith("--")) {
+    else if (a.startsWith("--if-rev=")) {
+      const v = Number(a.slice("--if-rev=".length));
+      if (!Number.isInteger(v) || v < 0) {
+        console.error(`write-state: --if-rev takes a non-negative integer, got ${a.slice(9)}`);
+        process.exit(3);
+      }
+      ifRev = v;
+    } else if (a.startsWith("--")) {
       console.error(`write-state: unknown flag ${a}`);
       process.exit(3);
     } else path = a;
   }
   if (!path) {
-    console.error("usage: write-state.mjs [--replace] <path>");
+    console.error("usage: write-state.mjs [--replace] [--if-rev=<n>] <path>");
     process.exit(3);
   }
-  return { replace, path };
+  return { replace, path, ifRev };
 }
 
 /**
@@ -103,8 +123,13 @@ async function acquireLock(
     try {
       writeFileSync(stagePath, String(process.pid));
       linkSync(stagePath, lockPath); // fails EEXIST if another writer holds it
+      // The inode of the lock WE created. Everything after this point checks
+      // against it: a lock at the same path with a different inode is somebody
+      // else's, and both releasing it and writing under it are the same bug the
+      // reclaim path below already refuses to commit.
+      const ino = statSync(lockPath).ino;
       unlinkSync(stagePath);
-      return;
+      return ino;
     } catch (err) {
       try {
         unlinkSync(stagePath);
@@ -208,16 +233,46 @@ function lockIdentityIfStale(lockPath, staleMs) {
 // Path whose lock this process currently holds (null when none). Lets the
 // top-level error handler release our lock on unexpected failures.
 let heldLockFor = null;
+// The inode of that lock. The path alone does not identify it: a lock deleted
+// and re-created at the same path is a different file and a different owner.
+let heldLockIno = null;
+
+/**
+ * True when the lock at `path` is still the one this process created.
+ *
+ * @param {string} path
+ * @param {number | null} ino
+ * @returns {boolean}
+ */
+function stillOurs(path, ino) {
+  if (ino === null) return true; // nothing was ever locked; nothing to lose
+  try {
+    return statSync(`${path}.lock`).ino === ino;
+  } catch {
+    return false; // our lock is gone, which means somebody removed it
+  }
+}
 
 /** @param {string} path */
-function releaseLock(path) {
+function releaseLock(path, ino = heldLockIno) {
   const lockPath = `${path}.lock`;
   try {
+    // By identity, never by path - the same rule acquireLock states and this
+    // function used to break. If our lock was reclaimed as stale while we were
+    // starved (the abandon ceiling, or the unreadable-PID window), the lock now
+    // at this path belongs to a LIVE writer, and unlinking it puts two writers
+    // inside at once and loses one update. That is the third point in this
+    // cycle where "delete by path" produces the same data loss; the other two
+    // are documented in acquireLock.
+    if (ino !== null && statSync(lockPath).ino !== ino) return;
     unlinkSync(lockPath);
   } catch {
     /* already gone  -  fine */
   }
-  if (heldLockFor === path) heldLockFor = null;
+  if (heldLockFor === path) {
+    heldLockFor = null;
+    heldLockIno = null;
+  }
 }
 
 /**
@@ -225,6 +280,22 @@ function releaseLock(path) {
  * @param {object} patch
  * @returns {object}
  */
+/**
+ * The version a NEWLY created state document is stamped with, read from the
+ * schema so the writer and the schema cannot disagree. The literal is a
+ * fallback for an unreadable schema, which validate-schemas.mjs would already
+ * have failed on.
+ */
+const STATE_SCHEMA_VERSION = (() => {
+  try {
+    const schemaPath = new URL("../schemas/agent-state.schema.json", import.meta.url);
+    const sv = JSON.parse(readFileSync(schemaPath, "utf-8"))?.properties?.schemaVersion;
+    return sv?.const ?? sv?.enum?.at(-1) ?? "2.1.0";
+  } catch {
+    return "2.1.0";
+  }
+})();
+
 function deepMerge(base, patch) {
   if (Array.isArray(patch)) return patch; // arrays replace
   if (typeof patch !== "object" || patch === null) return patch;
@@ -236,7 +307,7 @@ function deepMerge(base, patch) {
 }
 
 async function main() {
-  const { replace, path } = parseArgs();
+  const { replace, path, ifRev } = parseArgs();
 
   // Read stdin
   const chunks = [];
@@ -257,7 +328,7 @@ async function main() {
   }
 
   try {
-    await acquireLock(path);
+    heldLockIno = await acquireLock(path);
     heldLockFor = path;
   } catch (err) {
     if (err.code === "LOCK_TIMEOUT") {
@@ -272,6 +343,8 @@ async function main() {
   // here must release OUR lock explicitly before exiting - otherwise a corrupt
   // state file (or any I/O error) leaks agent-state.json.lock and blocks the
   // next writer until the stale-lock window elapses.
+  const fileExistedBefore = existsSync(path);
+
   try {
     let next;
     if (replace) {
@@ -304,16 +377,67 @@ async function main() {
     // carries a `rev` from before the record moved, and following it down
     // would tell the next reader that nothing changed.
     let priorRev = Number.isInteger(next.rev) ? next.rev : 0;
+    let diskRev = null;
     if (existsSync(path)) {
       try {
         const parsed = JSON.parse(readFileSync(path, "utf-8"));
-        if (Number.isInteger(parsed?.rev)) priorRev = Math.max(priorRev, parsed.rev);
+        if (Number.isInteger(parsed?.rev)) {
+          diskRev = parsed.rev;
+          priorRev = Math.max(priorRev, parsed.rev);
+        }
       } catch {
         /* unreadable on disk - the merge path above already threw for that */
       }
     }
+
+    // Compare-and-swap. `rev` already told a reader whether the record moved;
+    // --if-rev is what lets it ACT on that instead of merging over the top.
+    //
+    // The lock closes the window between two writers. It does not close the
+    // one between a READ and a write, and `/multi-agent:steer` puts a second
+    // writer in exactly that window on purpose: it reads the state, a human
+    // thinks, and by the time the edit lands the run may have moved three
+    // phases. A merge then reinstates decisions the run already replaced.
+    //
+    // Refusing costs a re-read. Merging costs a phase that silently reverts.
+    if (ifRev !== null) {
+      const seen = diskRev ?? 0;
+      if (seen !== ifRev) {
+        console.error(
+          `write-state: rev is ${seen}, expected ${ifRev}  -  the record moved since you read it; re-read and retry`,
+        );
+        releaseLock(path);
+        process.exit(5);
+      }
+    }
+
     next.rev = priorRev + 1;
 
+    // schemaVersion is stamped only when this call CREATES the document. The
+    // migration runner keys on it, and a document that never carries one can
+    // never be migrated - measured on a real install, 43 of 43 state files had
+    // no schemaVersion, so pipeline/schemas/migrations/ was unreachable code.
+    // A merge into an existing unstamped file is deliberately left alone:
+    // stamping there would claim a conformance nothing verified.
+    if (!fileExistedBefore && typeof next.schemaVersion !== "string") {
+      next.schemaVersion = STATE_SCHEMA_VERSION;
+    }
+
+    // Last check before the write lands: is the lock still ours? Everything
+    // above read the file under a lock we believed we held, and if that belief
+    // stopped being true the rename would overwrite a concurrent writer's
+    // update with a merge that never saw it. Reporting success while losing an
+    // update is the one outcome this script exists to prevent, so it refuses
+    // instead - a caller that sees exit 4 can read and retry, which a caller
+    // that saw exit 0 never will.
+    if (!stillOurs(path, heldLockIno)) {
+      console.error(
+        `write-state: the lock was taken from us mid-write (${path}.lock). ` +
+          `Nothing was written. Re-read the state and retry.`,
+      );
+      process.exit(4);
+    }
+
     const tmp = `${dirname(path)}/.${basename(path)}.${process.pid}.tmp`;
     writeFileSync(tmp, JSON.stringify(next, null, 2) + "\n");
     renameSync(tmp, path); // atomic on POSIX
@@ -327,8 +451,23 @@ async function main() {
   process.exit(0);
 }
 
-main().catch((err) => {
-  console.error(`write-state: unhandled  -  ${err.message}`);
-  if (heldLockFor) releaseLock(heldLockFor);
-  process.exit(3);
-});
+// Only run when invoked as a program. The lock helpers are the part with the
+// subtle behaviour and they are unreachable from a test otherwise: the window
+// they guard is microseconds wide, so a test that drives this file as a CLI
+// measures the scheduler rather than the code.
+const isDirectRun = invokedDirectly(import.meta.url);
+if (isDirectRun) {
+  // Exit 3 is this file's documented "died without writing" code, and the
+  // cleanup is the reason the handler exists at all: a writer that dies holding
+  // the advisory lock makes the NEXT writer wait out the acquire window and
+  // then decide, on nothing but age, whether to take a lock that may still be
+  // live. Releasing on the abnormal path keeps that decision from arising.
+  runMain("write-state", main, {
+    code: 3,
+    cleanup: () => {
+      if (heldLockFor) releaseLock(heldLockFor);
+    },
+  });
+}
+
+export { acquireLock, releaseLock, stillOurs, lockIdentityIfStale };

package/pipeline/skills/.skill-manifest.json

@@ -1,15 +1,15 @@
 {
   "schemaVersion": "1.0.0",
-  "generatedAt": "2026-09-15T20:06:53Z",
-  "skillCount": 212,
+  "generatedAt": "2026-09-18T07:17:50Z",
+  "skillCount": 216,
   "entries": [
     {
       "path": "shared/core/apple-archive-compliance/SKILL.md",
-      "sha256": "0d2f81825bf1610c765a4a7a223247703812e86a642e78f4f367cfc35cea76af"
+      "sha256": "e5c38c240a674903acf3f32e5b492a4a6c5dc438a323bb3ca9adc361b41a6884"
     },
     {
       "path": "shared/core/google-play-compliance/SKILL.md",
-      "sha256": "356ff2b30f414e582bb215c4728696af8e249dfe924e16a550294e48fd9bdcf9"
+      "sha256": "49412b53af57ba31fe7d19983144043a2333249d075ff90a9fde9a9d3482ce83"
     },
     {
       "path": "shared/core/multi-agent-analysis-jira/SKILL.md",
@@ -37,7 +37,7 @@
     },
     {
       "path": "shared/core/multi-agent-autopilot/SKILL.md",
-      "sha256": "c02525ad003c34d53834ecd04eb150764b36779f4e08941c9016926418e74333"
+      "sha256": "7446424d3dd480751358a0e3de917f3d22bb4f1ce6dec1b99217877c6b217e76"
     },
     {
       "path": "shared/core/multi-agent-build-optimize/SKILL.md",
@@ -45,7 +45,7 @@
     },
     {
       "path": "shared/core/multi-agent-channels/SKILL.md",
-      "sha256": "fa128726aa0009699e7ac2de566812e7192681db98c90dfc5da501ad50a05583"
+      "sha256": "414eead38b461c8aa95f6af3175366be905338233e6872313f5d4f59a02351af"
     },
     {
       "path": "shared/core/multi-agent-complaint-analysis/SKILL.md",
@@ -61,7 +61,7 @@
     },
     {
       "path": "shared/core/multi-agent-diff-explain/SKILL.md",
-      "sha256": "7cb37a224e61d66e350202ecfc5fbc6c33da3fe0718504a093602f5cde5b648f"
+      "sha256": "ec3827e0457c693812d11b3a96f88f0e2d186af369c353e1b5c40795d96933dd"
     },
     {
       "path": "shared/core/multi-agent-doctor/SKILL.md",
@@ -81,11 +81,11 @@
     },
     {
       "path": "shared/core/multi-agent-graph/SKILL.md",
-      "sha256": "918feff1a0a691535901c20a0625340a0f88cd5294cff1a1da20f799f6746f2a"
+      "sha256": "d6781a34d98a542aac3df4ce9c7c52d83902851de2d239bed022ac382716a26e"
     },
     {
       "path": "shared/core/multi-agent-help/SKILL.md",
-      "sha256": "87d45a04c782b03569b0ab9ee54da8414cbadd5a2a82a7701f38dc8a3ee94627"
+      "sha256": "6ebe7d4de1a1ff815f572abb5d5db5d5af9ea507b3047ec60597a54455176da5"
     },
     {
       "path": "shared/core/multi-agent-ios-coding-standard/SKILL.md",
@@ -105,15 +105,15 @@
     },
     {
       "path": "shared/core/multi-agent-language/SKILL.md",
-      "sha256": "c44d951996a208d962d969b0ac309ce88a38c048a2d8b332546c532045c189fc"
+      "sha256": "63e746bc63ec9963c54e852e6ca5cdb30afd9309a02ee3fdb35965eb931d0300"
     },
     {
       "path": "shared/core/multi-agent-local-autopilot/SKILL.md",
-      "sha256": "b0fac3509c2fa513b3fea9bfde4f2554642cb60373a5b570ef29ba8bf760e2a1"
+      "sha256": "5e367258e57fa710d16a1ad6363d5974b89aa99fb0ac791d1ce0070c81191f6b"
     },
     {
       "path": "shared/core/multi-agent-local/SKILL.md",
-      "sha256": "90dd95a0dc569f1451bf095a77cffccae607e4dedd271a3362cce32b53b5d819"
+      "sha256": "0a5f20782de81ac3376088579ff6fbc3a6f3781cb285860aae796408aa57e6aa"
     },
     {
       "path": "shared/core/multi-agent-log/SKILL.md",
@@ -121,7 +121,11 @@
     },
     {
       "path": "shared/core/multi-agent-manual-test/SKILL.md",
-      "sha256": "e7f7f8004d286c5b5b596f8659bcf57b2350eb9c67a55470bb2e9b714513a3a4"
+      "sha256": "53e1d58848673690aa0b146908eae98a2de9861dae8b613f63f63672adf6d69b"
+    },
+    {
+      "path": "shared/core/multi-agent-model/SKILL.md",
+      "sha256": "eb66f48b4772f41b395b0aff0c3d0303e29da0a470abae8b61a602b93ba7335c"
     },
     {
       "path": "shared/core/multi-agent-prune-logs/SKILL.md",
@@ -137,15 +141,15 @@
     },
     {
       "path": "shared/core/multi-agent-refactor/SKILL.md",
-      "sha256": "56b49a8216086961bc407a60b559942252a1aae3005b73811b1d40668758cc84"
+      "sha256": "56eb2be076a6840ddfbf57b8e82c8b4bcf76209553d41a06cc71e56dc894364e"
     },
     {
       "path": "shared/core/multi-agent-resume-local/SKILL.md",
-      "sha256": "a4bedf949ab648d977dd1409339d08af204341f555d69f71f58c1a54dffaff6d"
+      "sha256": "9f5089c748a8f0ae5983a3f392879a86017a33c883a87607169e32705c68c5e9"
     },
     {
       "path": "shared/core/multi-agent-resume/SKILL.md",
-      "sha256": "5e80eb098996fdea36f013acc8ddec44d462cace031c78341f11f0b59592a6ba"
+      "sha256": "6701424edde0a19b55a7de62bb5ba5f9dbaef7d9fb9ea2bce632374863145774"
     },
     {
       "path": "shared/core/multi-agent-review-analysis/SKILL.md",
@@ -163,6 +167,18 @@
       "path": "shared/core/multi-agent-review/SKILL.md",
       "sha256": "f3903b07fa7176ff9430f46cc1e981a0f0d06ab028df5ef1d45ad9271bada1e5"
     },
+    {
+      "path": "shared/core/multi-agent-route-off/SKILL.md",
+      "sha256": "c569e9ba1fbbebc879b69229cf2154edc89998fcdcfe74fb4220f5af70daafa0"
+    },
+    {
+      "path": "shared/core/multi-agent-route-on/SKILL.md",
+      "sha256": "40fab187a6054b90c9b8f1e8ad198a08ab94fdfd9cd89eef9b439e59b570d821"
+    },
+    {
+      "path": "shared/core/multi-agent-route-status/SKILL.md",
+      "sha256": "f104d3de6974837b8ebbcc6242269c06ea78a0d97a3e216c1b62cf6d91a72fd8"
+    },
     {
       "path": "shared/core/multi-agent-routines/SKILL.md",
       "sha256": "a8cc83140b5d84365348a2b0858f06a9e92808f608544eea61ab5760c82f9f97"
@@ -181,7 +197,7 @@
     },
     {
       "path": "shared/core/multi-agent-setup/SKILL.md",
-      "sha256": "d475f14de5db9ffe0c8ae5b3b83374d5f82a41c29da5271c7399d151e6bd4c1d"
+      "sha256": "d2c167082f40af3c2a7cbb0be55a8cdf8076a4f06a3df54c89b1318759421a29"
     },
     {
       "path": "shared/core/multi-agent-stack/SKILL.md",
@@ -189,11 +205,11 @@
     },
     {
       "path": "shared/core/multi-agent-status/SKILL.md",
-      "sha256": "436d1716ce5fc4977dadfce8a85b09877f93c626953739a7ad70eefbbcd25869"
+      "sha256": "ae9a7507e0a0291d8e786820b1a0394363f416fd7715f2deb162c9f542ae5351"
     },
     {
       "path": "shared/core/multi-agent-steer/SKILL.md",
-      "sha256": "a5988d6945dabea45d9827c76e270bb3d12fb2b7689e8893ea09f650ae97baa9"
+      "sha256": "d2a3602c1acbd1a01b30fcba65fbbe669ac825ec4ac9b0d6679bba07e0d1207c"
     },
     {
       "path": "shared/core/multi-agent-store-ready/SKILL.md",
@@ -201,7 +217,7 @@
     },
     {
       "path": "shared/core/multi-agent-sync/SKILL.md",
-      "sha256": "51e9a8a9053ce9d0450bcaf66555075f83c340203bb753946938f4b29cf93b79"
+      "sha256": "3d3331d34f9610ea02b96ee27c8029734f3a3447d2857fc0dca92d5b43d0f8e2"
     },
     {
       "path": "shared/core/multi-agent-test-accessibility/SKILL.md",
@@ -237,7 +253,7 @@
     },
     {
       "path": "shared/core/multi-agent/SKILL.md",
-      "sha256": "faac7ceb535d18778efe0ea91c19534c8e076bc709e7b1dbc567f5e71692bbbd"
+      "sha256": "849a2c7db9fb779d9919c7ed6cf3e0c3c6fd105ff961c45af31055ad21fd181c"
     },
     {
       "path": "shared/external/accessibility-compliance-accessibility-audit/SKILL.md",

package/pipeline/skills/.skills-index.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": "1.0.0",
-  "skillCount": 207,
+  "skillCount": 216,
   "entries": [
     {
       "name": "accessibility-compliance-accessibility-audit",
@@ -959,6 +959,17 @@
       "triggerPaths": [],
       "relativePath": "shared/core/multi-agent-analysis/SKILL.md"
     },
+    {
+      "name": "multi-agent-analysis-jira",
+      "description": "Turn a rendered analysis document into a Jira story tree: derive stories from the document's own rule ids, check coverage both ways, preview every byte, then create only what does not already exist. Use when an analysis is final and the work needs tickets.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-analysis-jira",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-analysis-jira/SKILL.md"
+    },
     {
       "name": "multi-agent-analysis-resolve",
       "description": "Resolve the Section 20 Risks and Open Questions of an analysis v3 document one row at a time: up to 3 source-labeled answer candidates per row (evidence / repo / AI reasoned), chosen answer merged into the target section, doc updated in place with a changelog bump. Same locked decisions as /multi-ag",
@@ -981,6 +992,39 @@
       "triggerPaths": [],
       "relativePath": "shared/core/multi-agent-autopilot/SKILL.md"
     },
+    {
+      "name": "multi-agent-autopilot-off",
+      "description": "Turn off continuous mode on this machine: the schedule is removed, work already running finishes, the repo selection is kept. Use when the machine should stop picking work up.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-autopilot-off",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-autopilot-off/SKILL.md"
+    },
+    {
+      "name": "multi-agent-autopilot-on",
+      "description": "Turn on continuous mode on THIS machine: pick the repos it watches; labelled GitHub issues and assigned+labelled Jira items then run in a worktree and stop at an open PR. Use when the machine should pick work up unattended.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-autopilot-on",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-autopilot-on/SKILL.md"
+    },
+    {
+      "name": "multi-agent-autopilot-status",
+      "description": "Show what continuous mode is doing here: what runs at which phase, what is queued, what waits for an answer, what PRs it opened today. Use when asked whether autopilot is on.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-autopilot-status",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-autopilot-status/SKILL.md"
+    },
     {
       "name": "multi-agent-build-optimize",
       "description": "Wrapper that dispatches to xcode-build-orchestrator on iOS repos. Benchmarks the current Xcode build, runs compilation / project / SPM analyzers, produces a recommend-first optimization plan in .build-benchmark/, and re-benchmarks after the developer approves changes. Fails fast on non-iOS stacks. U",
@@ -994,7 +1038,7 @@
     },
     {
       "name": "multi-agent-channels",
-      "description": "Multi-channel reporter  -  Jira/Confluence/Wiki/PR description. Multi-select channels + content, humanizer pass, reviewer-preserving Bitbucket PR PUT. Phase 7 delegates to this skill; users can invoke post-hoc by Jira ID / PR URL / branch.",
+      "description": "Multi-channel reporter  -  Jira/Confluence/Wiki/PR description. Multi-select channels + content, humanizer pass, reviewer-preserving Bitbucket PR PUT. Phase 5 delegates to this skill; users can invoke post-hoc by Jira ID / PR URL / branch.",
       "platform": null,
       "group": "core",
       "plugin": null,
@@ -1038,7 +1082,7 @@
     },
     {
       "name": "multi-agent-diff-explain",
-      "description": "Map Phase 4 triage findings to branch diff lines. Read-only post-hoc command, used after review to answer 'which finding lines up with which code change'. Use when a review finding has to be traced to the exact lines that caused it.",
+      "description": "Map Phase 3 triage findings to branch diff lines. Read-only post-hoc command, used after review to answer 'which finding lines up with which code change'. Use when a review finding has to be traced to the exact lines that caused it.",
       "platform": null,
       "group": "core",
       "plugin": null,
@@ -1047,6 +1091,17 @@
       "triggerPaths": [],
       "relativePath": "shared/core/multi-agent-diff-explain/SKILL.md"
     },
+    {
+      "name": "multi-agent-doctor",
+      "description": "Health check for the installed pipeline: layout, preferences, credentials, hooks and host capabilities, each with one actionable step. Exit code is the verdict. Use when a run failed for an environmental reason, before a sync, or after an update.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-doctor",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-doctor/SKILL.md"
+    },
     {
       "name": "multi-agent-feedback",
       "description": "Send one message to the maintainer: a bug, an idea or a question. Only the text you type is sent - no logs, no repo names, no paths. Shows the payload and asks before sending. Use when a run went wrong and the maintainer should know.",
@@ -1170,7 +1225,7 @@
     },
     {
       "name": "multi-agent-local-autopilot",
-      "description": "Full pipeline + local + autopilot  -  no worktree, no confirmations, all 8 phases run end-to-end on the current branch. Use when the full pipeline should run on the current branch with no worktree and no prompts.",
+      "description": "Full pipeline + local + autopilot  -  no worktree, no confirmations, all 6 phases run end-to-end on the current branch. Use when the full pipeline should run on the current branch with no worktree and no prompts.",
       "platform": null,
       "group": "core",
       "plugin": null,
@@ -1201,6 +1256,17 @@
       "triggerPaths": [],
       "relativePath": "shared/core/multi-agent-manual-test/SKILL.md"
     },
+    {
+      "name": "multi-agent-model",
+      "description": "Turn the top model rung on or off and keep the cost ledger's pricing in step with it. Use when asked to enable or disable Fable, or which model the pipeline dispatches.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-model",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-model/SKILL.md"
+    },
     {
       "name": "multi-agent-prune-logs",
       "description": "Delete per-task project logs under ~/.claude/logs/multi-agent (filter by age/project/task). Audit trail + metrics are preserved. Dry-run first; confirms before deleting. Use when task logs have built up and need trimming by age, project or task.",
@@ -1258,7 +1324,7 @@
     },
     {
       "name": "multi-agent-resume-local",
-      "description": "Continue already-done LOCAL work through the pipeline tail: Review → Build+Test → Commit/PR → Report (technical analysis + Jira test-scenario comment). No dev phase. Use when local work is already done and only review, build, commit and reporting remain.",
+      "description": "Continue already-done LOCAL work through the pipeline tail: Review (with its build gate) → Commit/PR → Report (technical analysis + Jira test-scenario comment). No dev phase. Use when local work is already done and only review, build, commit and reporting remain.",
       "platform": null,
       "group": "core",
       "plugin": null,
@@ -1311,6 +1377,39 @@
       "triggerPaths": [],
       "relativePath": "shared/core/multi-agent-review-jira/SKILL.md"
     },
+    {
+      "name": "multi-agent-route-off",
+      "description": "Disable model routing and keep its rules, so turning it back on does not re-ask for the same configuration. Use when asked to turn model routing off.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-route-off",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-route-off/SKILL.md"
+    },
+    {
+      "name": "multi-agent-route-on",
+      "description": "Enable policy-driven model routing: pick a strategy, a scope, and the rules that say which rung a call lands on. Use when asked to turn model routing on.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-route-on",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-route-on/SKILL.md"
+    },
+    {
+      "name": "multi-agent-route-status",
+      "description": "Report model routing: whether it is armed, which rule applies where, which rung the last dispatches took, and what this run has cost. Use when asked which model is being used or why.",
+      "platform": null,
+      "group": "core",
+      "plugin": null,
+      "invokeAs": "multi-agent-route-status",
+      "triggerKeywords": [],
+      "triggerPaths": [],
+      "relativePath": "shared/core/multi-agent-route-status/SKILL.md"
+    },
     {
       "name": "multi-agent-routines",
       "description": "List your saved /multi-agent routines (from /multi-agent:save) with what each one does, rendered in outputLanguage. Use when asked which saved routines exist or what one of them does.",