@brutalsystems/muster 3.3.2 → 3.3.3

package/dist/help.js

@@ -138,7 +138,7 @@ export const COMMANDS = [
             ["--format FORMAT", "json (default) or human"],
         ],
         notes: [
-            "Prints the final record, with state, exit_code and signal (null for a session, whose terminal keeps no exit status). Exits 0 when it has ended, 2 for an id that names nothing, and 124 on timeout, printing the record as it stands.",
+            "Prints the final record, with state, exit_code and signal (null for a session, whose terminal keeps no exit status). Exits 0 when it has ended, 2 for an id that names nothing, 124 on timeout, printing the record as it stands, and 75 when the registry is locked (as every command does): retry.",
         ],
         examples: [
             "muster wait 6fae755b-8f5d-4425-8ed7-69e84bfac8c5",

package/dist/muster.js

@@ -6,6 +6,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
 import { z } from "zod";
 import { Muster, UnknownRun, durableId, failedLaunchId } from "./run.js";
+import { RegistryLocked } from "./registry.js";
 import { waitSeconds } from "./duration.js";
 import { delay } from "./identity/processes.js";
 import { formatHuman, humanColorEnabled, statusText } from "./format.js";
@@ -486,5 +487,5 @@ main().catch((e) => {
         ? `\n[muster] see \`muster ${command} --help\` for its options`
         : "";
     process.stderr.write(`[muster] ${e.message}${hint}\n`);
-    process.exitCode = 1;
+    process.exitCode = e instanceof RegistryLocked ? RegistryLocked.exitCode : 1;
 });

package/dist/registry.js

@@ -38,12 +38,25 @@ export class IdempotencyKeyReuse extends Error {
     }
 }
 /**
- * A lock directory with no owner file is presumed live for this long. Long
- * enough that no transaction comes near it, and the holder writes its owner
- * file immediately after taking the lock, so in practice only a lock left by a
- * version that wrote none (an empty directory) ever reaches it.
+ * A lock directory with no owner file is presumed live for this long. A
+ * transaction holds the lock for milliseconds and the holder writes its owner
+ * file immediately after taking it, so a lock still unowned this long was left
+ * by a holder killed in between, or by a version that wrote none (#64). Three
+ * default lock waits: one waiter may still time out on it, the next breaks it.
  */
-const UNOWNED_LOCK_MS = 60_000;
+const UNOWNED_LOCK_MS = 15_000;
+/**
+ * The registry is busy, not broken and not empty: the CLI exits 75
+ * (EX_TEMPFAIL, "try again later") for it, so a script polling `list` or
+ * `wait` can tell it from a run that is gone (#64) and retry.
+ */
+export class RegistryLocked extends Error {
+    static exitCode = 75;
+    constructor(message) {
+        super(message);
+        this.name = "RegistryLocked";
+    }
+}
 let self;
 /** Statuses whose process is over: such an entry no longer holds its id. */
 const FINISHED = ["exited", "failed", "stopped"];
@@ -84,9 +97,14 @@ export class Registry {
                     continue;
                 if (Date.now() > deadline) {
                     const owner = await lockOwner(lock);
-                    throw new Error(owner
-                        ? `Registry locked, held by pid ${owner.pid}; verify no Muster operation is running before removing registry.lock`
-                        : "Registry locked; verify no Muster operation is running before removing registry.lock");
+                    const age = await ageMs(lock);
+                    const held = [
+                        owner ? `held by pid ${owner.pid}` : "with no owner recorded",
+                        age === undefined ? undefined : `for ${Math.round(age / 1000)}s`,
+                    ]
+                        .filter(Boolean)
+                        .join(" ");
+                    throw new RegistryLocked(`Registry locked, ${held}; retry shortly. If it persists, verify no Muster operation is running before removing ${lock}`);
                 }
                 await delay(20);
             }

package/dist/run.js

@@ -11,7 +11,7 @@ import { assertEnforcementMeetsMinimum, assertLevelWithinCeiling, assertMayAct,
 import { IdentityError, readIdentity, } from "./identity-store.js";
 import { resolveCredential } from "./identity-token.js";
 import { copyIdentity, identityEnv } from "./identity-copy.js";
-import { currentEntry, Registry } from "./registry.js";
+import { currentEntry, Registry, RegistryLocked, } from "./registry.js";
 import { LaunchLog } from "./log.js";
 import { hosts, selectHost } from "./hosts/index.js";
 import { AGENTS } from "./agents.js";
@@ -1470,6 +1470,10 @@ export class Muster {
             return await this.find(id);
         }
         catch (e) {
+            // Busy is not gone: `wait` exits 2 for an UnknownRun, and a poller that
+            // read a lock as "no such run" is how #64 ended a wait loop early.
+            if (e instanceof RegistryLocked)
+                throw e;
             const message = e.message;
             // The resolver's "unknown: <every live name>" reads as a list of
             // matches; the ambiguous case does want its candidates, so it keeps them.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brutalsystems/muster",
-  "version": "3.3.2",
+  "version": "3.3.3",
   "description": "Launch instructed, reachable local agent sessions",
   "type": "module",
   "engines": {