@botbuddy/cli 1.63.40 → 1.63.42

package/daemon-release/darwin-arm64.json

@@ -1,7 +1,7 @@
 {
-  "release_version": "1.63.40",
-  "source_commit": "6a30ed1a16dc6a97351c3cea5cc2d03918dbe0fc",
-  "source_sha256": "de12a3e513169b3a152a617b48c7e9c4c25c30e47b549854994744c369268ff1",
+  "release_version": "1.63.42",
+  "source_commit": "68cd7a93519197cc7b96a190fa0c4b4799a9c45d",
+  "source_sha256": "3a4e94f577771c564aa24d54529368c6b6d325be1e64ef0a30da1531f02be6ed",
   "target": "darwin-arm64",
-  "sha256": "9732532ac31d3e184fedafe19089b5efa938b2f3c2c4952cbc95866d86f4b8ae"
+  "sha256": "3c5f90061df675f1e06408ce5e1f3c9b6350e5a237216b0dbc11d71d656a1b35"
 }

package/daemon-release/linux-x64.json

@@ -1,7 +1,7 @@
 {
-  "release_version": "1.63.40",
-  "source_commit": "6a30ed1a16dc6a97351c3cea5cc2d03918dbe0fc",
-  "source_sha256": "de12a3e513169b3a152a617b48c7e9c4c25c30e47b549854994744c369268ff1",
+  "release_version": "1.63.42",
+  "source_commit": "68cd7a93519197cc7b96a190fa0c4b4799a9c45d",
+  "source_sha256": "3a4e94f577771c564aa24d54529368c6b6d325be1e64ef0a30da1531f02be6ed",
   "target": "linux-x64",
-  "sha256": "8abdd484047020cb2d96a7adc2a2bacb3b860bafaaf0f56e517eb844bb8550a1"
+  "sha256": "a5a474195eca0a8b57ddf8c5d50cfddb88f083a57bc99e80dc4f1a7d10e99535"
 }

package/daemon-release/manifest.json

@@ -1,21 +1,21 @@
 {
   "schema_version": 1,
-  "release_version": "1.63.40",
-  "source_commit": "6a30ed1a16dc6a97351c3cea5cc2d03918dbe0fc",
-  "source_sha256": "de12a3e513169b3a152a617b48c7e9c4c25c30e47b549854994744c369268ff1",
+  "release_version": "1.63.42",
+  "source_commit": "68cd7a93519197cc7b96a190fa0c4b4799a9c45d",
+  "source_sha256": "3a4e94f577771c564aa24d54529368c6b6d325be1e64ef0a30da1531f02be6ed",
   "targets": {
     "darwin-arm64": {
       "goos": "darwin",
       "goarch": "arm64",
       "binary": "daemon/bin/darwin-arm64/botbuddy-daemon",
-      "sha256": "9732532ac31d3e184fedafe19089b5efa938b2f3c2c4952cbc95866d86f4b8ae",
+      "sha256": "3c5f90061df675f1e06408ce5e1f3c9b6350e5a237216b0dbc11d71d656a1b35",
       "mode": "0755"
     },
     "linux-x64": {
       "goos": "linux",
       "goarch": "amd64",
       "binary": "daemon/bin/linux-x64/botbuddy-daemon",
-      "sha256": "8abdd484047020cb2d96a7adc2a2bacb3b860bafaaf0f56e517eb844bb8550a1",
+      "sha256": "a5a474195eca0a8b57ddf8c5d50cfddb88f083a57bc99e80dc4f1a7d10e99535",
       "mode": "0755"
     }
   }

package/package.json

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

package/src/pw/staging-auth.mjs

@@ -295,6 +295,8 @@ export async function runStagingAuth({ plan, host, env, deps, socketRun, resolve
   }
   const found = { binding: ctxPool.poolBinding };
   Object.assign(vars, { allowed: Object.keys(found.binding.personas ?? {}), tenant: ctxPool.tenant ?? undefined, poolId: ctxPool.poolId ?? undefined });
+  // BOT-2836: a retry addresses the same pool when one was named (several installed).
+  if (plan.pool) vars.retry = `${authLine} --pool ${ctxPool.poolId}`;
   const { findPersona } = await loadBinding();
   const persona = findPersona(found.binding, plan.persona);
   if (!persona) return refuse(io, "persona_not_allowlisted", 2, {}, vars);
@@ -317,7 +319,10 @@ export async function runStagingAuth({ plan, host, env, deps, socketRun, resolve
       // The refusal's code is the pool's health reason when it is unhealthy (e.g. a
       // count-0 persona is persona_not_configured), so code, message and help agree.
       const reasonCode = typeof error.reason === "string" && error.reason !== "healthy" && isKnownCode(error.reason) ? error.reason : "pool_empty";
-      return refuse(io, reasonCode, 3, Object.fromEntries(["reason", "detail", ...GUIDED_FIELDS].filter((key) => error[key] !== undefined).map((key) => [key, error[key]])), vars);
+      // BOT-2836: a healthy empty pool is the agent's own wait-and-retry, so it is
+      // rendered here with this lane; an unhealthy pool keeps the pool's health guidance.
+      const forwarded = reasonCode === "pool_empty" ? ["reason", "detail"] : ["reason", "detail", ...GUIDED_FIELDS];
+      return refuse(io, reasonCode, 3, Object.fromEntries(forwarded.filter((key) => error[key] !== undefined).map((key) => [key, error[key]])), vars);
     }
     const code = typeof error?.code === "string" && isKnownCode(error.code) ? error.code : "pool_take_failed";
     return refuse(io, code, 3, { detail: String(error?.message ?? error).slice(0, 300) }, vars);

package/src/stack.mjs

@@ -1356,7 +1356,7 @@ export function leaseRefusalExit(code) {
   // server refusing the credential — never BACKEND, which a gate reads as "down".
   if (CALL_AUTH_ERROR_CODES.has(code)) return EXIT.AUTH;
   // BOT-1926: a reused lease requested in the other execution mode is fixable by the caller.
-  return code === STACK_PATH_ROOT_REFUSED_CODE || code === "STACK_EXECUTION_MODE_MISMATCH" ? EXIT.INVALID : EXIT.BACKEND;
+  return code === STACK_PATH_ROOT_REFUSED_CODE || code === "STACK_EXECUTION_MODE_MISMATCH" || code === "AGENT_DEEPLINK_REQUIRED" ? EXIT.INVALID : EXIT.BACKEND;
 }
 
 /**
@@ -1386,6 +1386,8 @@ export const REMOTE_REFUSAL_GUIDANCE = Object.freeze({
     "re-run (the bundle is built automatically unless it failed — see the warning above) or use --local-only.",
 });
 
+const AGENT_DEEPLINK_REQUIRED_GUIDANCE = "the control plane refused this stack because its owning agent has no canonical profile or native session link. Reconnect with `bb register`, then retry. No lease was created.";
+
 /** The refusal text for a receipt: the server's message, else the CLI's guidance, else the code. */
 export function leaseRefusalError(data) {
   if (data?.lease_id && ["STACK_CONTEXT_BIND_FAILED", "STACK_CONTEXT_ALREADY_BOUND"].includes(data?.code)) {
@@ -1398,7 +1400,7 @@ export function leaseRefusalError(data) {
     return `the server's identified admission failed in the database${data.sqlstate ? ` (SQLSTATE ${data.sqlstate})` : ""} and rolled back, ` +
       "so no lease was created and this stack's port claim is released. This is a BotBuddy server fault: report it with this run's admission request id.";
   }
-  return data?.message || REMOTE_REFUSAL_GUIDANCE[data?.code] || data?.code || "lease request refused";
+  return data?.message || (data?.code === "AGENT_DEEPLINK_REQUIRED" ? AGENT_DEEPLINK_REQUIRED_GUIDANCE : null) || REMOTE_REFUSAL_GUIDANCE[data?.code] || data?.code || "lease request refused";
 }
 
 /** One stderr line naming a remote-lease refusal (null for every other code). */

package/src/staging-pool/cli.mjs

@@ -236,6 +236,16 @@ export function resolveInstalledPool({ home = homedir(), poolId = null, tenantHi
   throw Object.assign(new ConfigError(`Several staging session pools are installed: ${installed.map((pool) => pool.poolId).join(", ")}.`, "pool_ambiguous"), { poolIds: installed.map((pool) => pool.poolId) });
 }
 
+// BOT-2836: after a take, ask the installed launchd job to run its tick now, so the
+// pool refills without waiting for the interval or a human `tick`. The job signs in
+// (with its own backoff); the agent never does. No -k: a running tick is never killed.
+// Best effort: an unloaded job surfaces through health, and never fails the take.
+async function requestRefill({ ctx, exec, uid }) {
+  const label = ctx.poolBinding?.launchdLabel;
+  if (!label) return;
+  await createLaunchctl({ exec, uid }).kickstart(label).catch(() => {});
+}
+
 /**
  * Agent path, used by `bb pw … auth`: claim one verified session for a persona
  * (alias or email) of the installed pool. Returns { taken, target, binding,
@@ -263,9 +273,11 @@ export async function takeForLane({
   const pool = buildPool({ ctx, target, now, logger, intervalSeconds, fetchImpl });
   try {
     const taken = await pool.take({ persona: entry.email, owner });
+    await requestRefill({ ctx, exec, uid });
     return { taken, target, binding: ctx.poolBinding, persona: entry, discard: () => pool.discardTaken(taken.slotId) };
   } catch (error) {
     if (!(error instanceof PoolEmptyError)) throw error;
+    await requestRefill({ ctx, exec, uid });
     const health = await assessHealth({ ctx, pool, now, exec, uid, requestedPersona: entry.email });
     let effective = health;
     if (health.ok) {

package/src/staging-pool/errors.mjs

@@ -131,10 +131,12 @@ export const ERROR_CODES = Object.freeze({
     `${AUTH}   # once; if it repeats, report this code and detail`, "retry_once"),
 
   // ── the pool's health (human repairs; agents report once and stop)
-  pool_empty: entry("human", "repair",
+  // BOT-2836: a healthy pool emptied between ticks is a wait, not a repair. The take
+  // already asked the human-installed job to refill now, so the agent waits once and retries.
+  pool_empty: entry("agent", "take",
     "The staging session pool is healthy but has no ready session for {persona}.",
-    "Agents took every ready session since the last tick, or the ready ones were dead or AAL1 (factor rotated) and were discarded.",
-    "{tickCommand}"),
+    "Agents took every ready session since the last tick, or the ready ones were dead or AAL1 (factor rotated) and were discarded; the pool job was asked to refill now.",
+    "bb wait 'timer:duration=60' && {retry}   # once; if it repeats, report this code", "retry_once"),
   job_not_loaded: entry("human", "install",
     "The staging session pool is not installed (or not loaded) on this Mac.",
     "The one-time install has not been run here, or the Mac was migrated.",

package/src/staging-pool/pool.mjs

@@ -30,8 +30,8 @@ import {
 export class PoolEmptyError extends Error {
   constructor(persona, intervalHint) {
     super(
-      `No ready ${persona} session in the pool. The human-owned pool job refills on its next tick${intervalHint}; ` +
-        "report this blocker and its remedy once, then stop; ask the human to check the job. " +
+      `No ready ${persona} session in the pool. The human-owned pool job refills on its next tick${intervalHint} ` +
+        "and was asked to refill now; wait once, retry once, and report it if it repeats. " +
         "Agents never sign in to staging themselves.",
     );
     this.name = "PoolEmptyError";