@syncular/client 0.15.43 → 0.15.44

package/dist/worker-host.d.ts

@@ -157,6 +157,14 @@ export declare class SyncClientHandle {
         leadershipListeners?: Set<(state: LeadershipState) => void>;
         leadership?: LeadershipState;
     });
+    /**
+     * @internal — whether `close()` has run.
+     *
+     * A follower keeps a blocking `acquire` outstanding for the whole time it is a
+     * follower, so its promotion can fire long after the application has discarded
+     * the handle. The promotion path consults this before opening anything.
+     */
+    get __isClosed(): boolean;
     /** @internal — swap this handle from follower to leader (promotion). */
     __becomeLeader(core: LeaderCore): void;
     /** @internal — apply a follower reachability snapshot in place. */

package/dist/worker-host.js

@@ -113,6 +113,16 @@ export class SyncClientHandle {
             onInvalidate: (listener) => this.onInvalidate(listener),
         });
     }
+    /**
+     * @internal — whether `close()` has run.
+     *
+     * A follower keeps a blocking `acquire` outstanding for the whole time it is a
+     * follower, so its promotion can fire long after the application has discarded
+     * the handle. The promotion path consults this before opening anything.
+     */
+    get __isClosed() {
+        return this.#closed;
+    }
     /** @internal — swap this handle from follower to leader (promotion). */
     __becomeLeader(core) {
         this.#follower?.close();
@@ -652,6 +662,14 @@ async function bootFollower(config, lockName, lock, parts) {
             await lease.release();
             return;
         }
+        if (handle.__isClosed) {
+            // The application discarded this handle while it was queued for
+            // leadership. Promoting now would open a database nobody is holding and
+            // then keep the lock forever, so no other tab could ever take over.
+            // Release instead, and let the next waiter have it.
+            await lease.release();
+            return;
+        }
         // The follower saw the departing leader's epoch; the new leader must
         // strictly exceed it so stale replies/events are discarded everywhere.
         const nextEpoch = follower.maxEpochSeen + 1;
@@ -681,6 +699,13 @@ async function bootFollower(config, lockName, lock, parts) {
                     }
                     : {}),
             });
+            if (handle.__isClosed) {
+                // Closed while the worker was starting. `close()` already ran and found
+                // no core to shut down, so this one would leak: tear it down here rather
+                // than hand it to a discarded handle. `close(true)` releases the lease.
+                await core.close(true);
+                return;
+            }
             handle.__becomeLeader(core);
         }
         catch {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syncular/client",
-  "version": "0.15.43",
+  "version": "0.15.44",
   "description": "Syncular TypeScript client core — offline-first sync over SQLite (WASM/OPFS, Bun, Node)",
   "license": "Apache-2.0",
   "author": "Benjamin Kniffler",
@@ -89,7 +89,7 @@
   },
   "dependencies": {
     "@sqlite.org/sqlite-wasm": "^3.53.0-build1",
-    "@syncular/core": "0.15.43"
+    "@syncular/core": "0.15.44"
   },
   "peerDependencies": {
     "better-sqlite3": ">=11"
@@ -100,7 +100,7 @@
     }
   },
   "devDependencies": {
-    "@syncular/server": "0.15.43",
+    "@syncular/server": "0.15.44",
     "@types/better-sqlite3": "^7.6.13",
     "better-sqlite3": "^12.11.1"
   }

package/src/worker-host.ts

@@ -334,6 +334,17 @@ export class SyncClientHandle {
     });
   }
 
+  /**
+   * @internal — whether `close()` has run.
+   *
+   * A follower keeps a blocking `acquire` outstanding for the whole time it is a
+   * follower, so its promotion can fire long after the application has discarded
+   * the handle. The promotion path consults this before opening anything.
+   */
+  get __isClosed(): boolean {
+    return this.#closed;
+  }
+
   /** @internal — swap this handle from follower to leader (promotion). */
   __becomeLeader(core: LeaderCore): void {
     this.#follower?.close();
@@ -1020,6 +1031,14 @@ async function bootFollower(
       await lease.release();
       return;
     }
+    if (handle.__isClosed) {
+      // The application discarded this handle while it was queued for
+      // leadership. Promoting now would open a database nobody is holding and
+      // then keep the lock forever, so no other tab could ever take over.
+      // Release instead, and let the next waiter have it.
+      await lease.release();
+      return;
+    }
     // The follower saw the departing leader's epoch; the new leader must
     // strictly exceed it so stale replies/events are discarded everywhere.
     const nextEpoch = follower.maxEpochSeen + 1;
@@ -1058,6 +1077,13 @@ async function bootFollower(
             }
           : {}),
       });
+      if (handle.__isClosed) {
+        // Closed while the worker was starting. `close()` already ran and found
+        // no core to shut down, so this one would leak: tear it down here rather
+        // than hand it to a discarded handle. `close(true)` releases the lease.
+        await core.close(true);
+        return;
+      }
       handle.__becomeLeader(core);
     } catch {
       // Promotion failed to spawn a worker — release so the next tab tries.