@vincemakes/kiso-runtime 0.51.0 → 0.51.1

package/dist/lock-adapter.js

@@ -183,6 +183,16 @@ function unavailable(err) {
     return new LockUnavailableError(`${e.code ?? "?"}: ${e.message}`);
 }
 const sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+/**
+ * The tokens of the lock handles THIS process holds, or is in the middle of
+ * publishing. A lock naming our pid is a same-process writer's residue only
+ * when its token is in here; any other token was left by a PREVIOUS process
+ * that had the same pid — in a container node is pid 1, so a SIGKILLed
+ * container's lock names the next container's pid too. Kept on globalThis so
+ * a second copy of this module in the same process shares one answer.
+ */
+const HELD_KEY = Symbol.for("kiso.lock-adapter.held-tokens");
+const held = (globalThis[HELD_KEY] ??= new Set());
 class NativeLockHandle {
     pid;
     token;
@@ -205,6 +215,7 @@ class NativeLockHandle {
         if (this.#released)
             return;
         this.#released = true;
+        held.delete(this.token);
         try {
             renameSync(this.#lockPath, this.#tombstone);
         }
@@ -258,8 +269,20 @@ class NativeLockHandle {
 class NativeLock {
     name = "link-lock";
     async acquire(lockPath, sessionId, cancelled) {
-        const pid = process.pid;
+        // Our token is registered before any file can name it; a handle's
+        // release drops it, and a failed acquire drops it here.
         const token = randomUUID();
+        held.add(token);
+        try {
+            return await this.#take(lockPath, sessionId, cancelled, token);
+        }
+        catch (err) {
+            held.delete(token);
+            throw err;
+        }
+    }
+    async #take(lockPath, sessionId, cancelled, token) {
+        const pid = process.pid;
         // The staging name is PER-ATTEMPT: a later rename-away in the same
         // acquire must never atomically overwrite an earlier attempt's
         // abandoned staging (that would clobber the displacement residue,
@@ -276,7 +299,11 @@ class NativeLock {
             cancelled();
             const staging = `${lockPath}.staging-${pid}-${token}-${attempt}`;
             const seen = readLockIdentity(lockPath);
-            if (seen === null || seen.pid === undefined || !isAlive(seen.pid)) {
+            // A lock naming OUR pid with a token this process never issued is a
+            // previous process's (a killed container's node was pid 1 too): a
+            // dead holder, however alive the pid looks.
+            const stalePid = seen !== null && seen.pid === pid && seen.token !== undefined && !held.has(seen.token);
+            if (seen === null || seen.pid === undefined || stalePid || !isAlive(seen.pid)) {
                 // Fresh path, dead holder, or stale residue — take it over
                 // by identity confirmation: rename-away → verify → link.
                 if (readyDir !== undefined)
@@ -366,8 +393,8 @@ class NativeLock {
                 return new NativeLockHandle(lockPath, tombstone, { pid, token });
             }
             if (seen.pid === pid && seen.token !== undefined) {
-                // Same-process writer's residue (round 5): another store in
-                // THIS process holds the lock — it will release; retry until
+                // Same-process writer (round 5): another store in THIS process
+                // holds the lock (its token is in `held`) — it will release; retry until
                 // it does (never a spurious self-conflict). A legacy
                 // bare-pid lock naming our own process is refused like any
                 // live foreign owner.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vincemakes/kiso-runtime",
-	"version": "0.51.0",
+	"version": "0.51.1",
 	"description": "kiso runtime \u2014 durable multi-turn agent sessions: AgentDefinition, AgentRuntime, AgentSession, Run, append-only JSONL store.",
 	"type": "module",
 	"license": "MIT",
@@ -25,12 +25,12 @@
 		"test": "vitest run"
 	},
 	"dependencies": {
-		"@vincemakes/kiso-core": "0.51.0"
+		"@vincemakes/kiso-core": "0.51.1"
 	},
 	"peerDependencies": {
-		"@vincemakes/kiso-provider-anthropic": "0.51.0",
-		"@vincemakes/kiso-provider-openai": "0.51.0",
-		"@vincemakes/kiso-provider-openai-responses": "0.51.0"
+		"@vincemakes/kiso-provider-anthropic": "0.51.1",
+		"@vincemakes/kiso-provider-openai": "0.51.1",
+		"@vincemakes/kiso-provider-openai-responses": "0.51.1"
 	},
 	"peerDependenciesMeta": {
 		"@vincemakes/kiso-provider-anthropic": {
@@ -44,7 +44,7 @@
 		}
 	},
 	"devDependencies": {
-		"@vincemakes/kiso-evals": "0.51.0",
+		"@vincemakes/kiso-evals": "0.51.1",
 		"@types/node": "^26.1.2",
 		"typescript": "^5.7.2",
 		"vitest": "^3.0.0"