@botbuddy/cli 1.59.8 → 1.59.9

package/daemon-release/darwin-arm64.json

@@ -1,7 +1,7 @@
 {
-  "release_version": "1.59.8",
-  "source_commit": "d72599a3ac43155a4d66e30b2d73e5f2a46cf956",
-  "source_sha256": "5ccca5bc66c82f02e856a25c0cedf9056f69b3e72b95a668dc62a3dff4a0f72f",
+  "release_version": "1.59.9",
+  "source_commit": "d9c73efb211e953ac2d57257a0d7d3cd21660d67",
+  "source_sha256": "57b561506694e9a2d326bba24731ab1b0354ef38c509b452686b8a450b59bd4b",
   "target": "darwin-arm64",
-  "sha256": "24810c0fd70f40e77d860b0c247f1a7112a6a2a09b8754c83c7ca50b023e38c1"
+  "sha256": "ad8d7b6e3255d85f1c1237ae1b6bbd0a6b359b7909fba60b4819566bd7362c31"
 }

package/daemon-release/linux-x64.json

@@ -1,7 +1,7 @@
 {
-  "release_version": "1.59.8",
-  "source_commit": "d72599a3ac43155a4d66e30b2d73e5f2a46cf956",
-  "source_sha256": "5ccca5bc66c82f02e856a25c0cedf9056f69b3e72b95a668dc62a3dff4a0f72f",
+  "release_version": "1.59.9",
+  "source_commit": "d9c73efb211e953ac2d57257a0d7d3cd21660d67",
+  "source_sha256": "57b561506694e9a2d326bba24731ab1b0354ef38c509b452686b8a450b59bd4b",
   "target": "linux-x64",
-  "sha256": "61b9f79104394a9fe11e43b25cf3aa3c84dc1538f44c4439b7ed0555b8e8b858"
+  "sha256": "bf2f3a46ef151ace3e5b21633c03dc1824bee2e42fc6edd8f6898ca35d48f10c"
 }

package/daemon-release/manifest.json

@@ -1,21 +1,21 @@
 {
   "schema_version": 1,
-  "release_version": "1.59.8",
-  "source_commit": "d72599a3ac43155a4d66e30b2d73e5f2a46cf956",
-  "source_sha256": "5ccca5bc66c82f02e856a25c0cedf9056f69b3e72b95a668dc62a3dff4a0f72f",
+  "release_version": "1.59.9",
+  "source_commit": "d9c73efb211e953ac2d57257a0d7d3cd21660d67",
+  "source_sha256": "57b561506694e9a2d326bba24731ab1b0354ef38c509b452686b8a450b59bd4b",
   "targets": {
     "darwin-arm64": {
       "goos": "darwin",
       "goarch": "arm64",
       "binary": "daemon/bin/darwin-arm64/botbuddy-daemon",
-      "sha256": "24810c0fd70f40e77d860b0c247f1a7112a6a2a09b8754c83c7ca50b023e38c1",
+      "sha256": "ad8d7b6e3255d85f1c1237ae1b6bbd0a6b359b7909fba60b4819566bd7362c31",
       "mode": "0755"
     },
     "linux-x64": {
       "goos": "linux",
       "goarch": "amd64",
       "binary": "daemon/bin/linux-x64/botbuddy-daemon",
-      "sha256": "61b9f79104394a9fe11e43b25cf3aa3c84dc1538f44c4439b7ed0555b8e8b858",
+      "sha256": "bf2f3a46ef151ace3e5b21633c03dc1824bee2e42fc6edd8f6898ca35d48f10c",
       "mode": "0755"
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@botbuddy/cli",
-  "version": "1.59.8",
+  "version": "1.59.9",
   "description": "BotBuddy \u2014 Swarm coordination CLI for multi-agent workflows",
   "type": "module",
   "bin": {

package/src/wait-core.mjs

@@ -287,9 +287,10 @@ const VALIDATORS = {
   // BOT-1507 — scope=next adds a fourth, LIVE-ONLY selector: it names no run or PR
   // (they must already exist at arm time) and wakes on the first terminal CI run for
   // the repo delivered AFTER the wait was armed — the post-merge / cron train case.
-  // Optional branch=/workflow= filter that first run; branch is valid only with
-  // scope=next. A workflow also pins a level-triggered sha= recovery to one
-  // canonical workflow identity.
+  // Optional branch=/workflow=/workflow_id=/workflow_path= filters narrow that
+  // first run; branch and stable workflow identities are valid only with
+  // scope=next. The legacy display-name filter also pins a level-triggered
+  // sha= recovery to one historical workflow.
   // BOT-1685 — scope=next also accepts an optional sha= FILTER (distinct from the
   // level-triggered sha= selector): a wait armed BEFORE re-dispatching a run for a
   // known commit then wakes on the NEXT terminal run for THAT commit, instead of
@@ -309,9 +310,13 @@ const VALIDATORS = {
     }
     const hasBranchFilter = p.branch !== undefined;
     const hasWorkflowFilter = p.workflow !== undefined;
+    const hasWorkflowIdFilter = p.workflow_id !== undefined;
+    const hasWorkflowPathFilter = p.workflow_path !== undefined;
     if (p.scope !== undefined) {
       if (p.scope === "latest") {
-        if (hasBranchFilter || hasWorkflowFilter) throw new Error("ci branch=/workflow= are only valid with scope=next");
+        if (hasBranchFilter || hasWorkflowFilter || hasWorkflowIdFilter || hasWorkflowPathFilter) {
+          throw new Error("ci branch=/workflow=/workflow_id=/workflow_path= are only valid with scope=next");
+        }
         if (p.pr === undefined || !/^\d+$/.test(String(p.pr).trim())) {
           throw new Error("ci scope=latest needs pr=<number>");
         }
@@ -329,12 +334,23 @@ const VALIDATORS = {
           if (v === "") throw new Error(`ci ${key}= must be non-empty`);
           return v;
         };
-        return { repo, scope: "next", branch: filter("branch"), workflow: filter("workflow"), sha: filter("sha"), pr: null };
+        const workflowId = filter("workflow_id");
+        const workflowPath = filter("workflow_path");
+        return {
+          repo,
+          scope: "next",
+          branch: filter("branch"),
+          workflow: filter("workflow"),
+          ...(workflowId == null ? {} : { workflowId }),
+          ...(workflowPath == null ? {} : { workflowPath }),
+          sha: filter("sha"),
+          pr: null,
+        };
       }
       throw new Error("ci scope must be 'latest' or 'next' (or use run_id=/sha=)");
     }
-    if (hasBranchFilter || (hasWorkflowFilter && p.sha === undefined)) {
-      throw new Error("ci branch= is only valid with scope=next; workflow= also accepts an exact sha selector");
+    if (hasBranchFilter || hasWorkflowIdFilter || hasWorkflowPathFilter || (hasWorkflowFilter && p.sha === undefined)) {
+      throw new Error("ci branch=/workflow_id=/workflow_path= are only valid with scope=next; legacy workflow= also accepts an exact sha selector");
     }
     if (p.run_id !== undefined) {
       if (!String(p.run_id).trim()) throw new Error("ci run_id must be non-empty");
@@ -343,7 +359,13 @@ const VALIDATORS = {
     if (!String(p.sha).trim()) throw new Error("ci sha must be non-empty");
     const workflow = p.workflow === undefined ? null : String(p.workflow).trim();
     if (p.workflow !== undefined && workflow === "") throw new Error("ci workflow= must be non-empty");
-    return { repo, scope: "sha", sha: String(p.sha).trim(), workflow, pr: p.pr != null ? String(p.pr).trim() : null };
+    return {
+      repo,
+      scope: "sha",
+      sha: String(p.sha).trim(),
+      workflow,
+      pr: p.pr != null ? String(p.pr).trim() : null,
+    };
   },
   "staging-green"(p) {
     const repo = p.repo != null ? String(p.repo).trim() : null;
@@ -1013,7 +1035,7 @@ export function formatUnknownWorkflowGuidance(detail, candidates) {
   const named = Array.isArray(candidates) ? candidates.filter((c) => typeof c === "string" && c.trim() !== "") : [];
   if (named.length === 0) return lines;
   lines.push("closest workflow names this repo has actually reported:");
-  for (const name of named) lines.push(`  workflow=${name}`);
+  for (const name of named) lines.push(`  ${name.includes("=") ? name : `workflow=${name}`}`);
   return lines;
 }
 
@@ -1152,16 +1174,20 @@ function conditionMatchesSignal(condition, signal, waitSessionId) {
       if (params.scope === "run_id") return String(signal.payload?.run_id) === params.runId;
       if (params.scope === "sha") {
         if (signal.payload?.head_sha !== params.sha) return false;
-        if (params.workflow == null) return true;
-        const workflow = signal.payload?.workflow_name;
-        return typeof workflow === "string" && workflow.trim().toLowerCase() === params.workflow.trim().toLowerCase();
+        if (params.workflow != null) {
+          const workflow = signal.payload?.workflow_name;
+          if (typeof workflow !== "string" || workflow.trim().toLowerCase() !== params.workflow.trim().toLowerCase()) return false;
+        }
+        if (params.workflowId != null && String(signal.payload?.workflow_id ?? "").trim() !== params.workflowId) return false;
+        if (params.workflowPath != null && String(signal.payload?.workflow_path ?? "").trim() !== params.workflowPath) return false;
+        return true;
       }
       if (params.scope === "next") {
         // BOT-1507 — live-only: the first TERMINAL run for the repo. Never keys on
         // subject_key (it is run:<id> for post-merge runs and owner/repo#n for PR
         // runs — both are legitimate wakes); a null/empty conclusion is an
         // in-progress signal and never terminal. Optional branch/workflow filters:
-        // branch is exact, workflow is the GitHub display name (case/space-insensitive).
+        // branch/path/id are exact; the display name is case/space-insensitive.
         const conclusion = signal.payload?.conclusion;
         if (typeof conclusion !== "string" || conclusion.trim() === "") return false;
         // BOT-1685: an optional sha filter pins scope=next to a commit, so a
@@ -1174,6 +1200,8 @@ function conditionMatchesSignal(condition, signal, waitSessionId) {
             return false;
           }
         }
+        if (params.workflowId != null && String(signal.payload?.workflow_id ?? "").trim() !== params.workflowId) return false;
+        if (params.workflowPath != null && String(signal.payload?.workflow_path ?? "").trim() !== params.workflowPath) return false;
         return true;
       }
       // scope=latest: the PR's run (by subject owner/repo#pr or payload pr_number).