@spooky-sync/core 0.0.1-canary.287 → 0.0.1-canary.288

package/dist/index.js

@@ -1752,8 +1752,8 @@ function selfAllowlistedVariant(flag, userId) {
 
 //#endregion
 //#region src/modules/devtools/index.ts
-const CORE_VERSION = "0.0.1-canary.287";
-const WASM_VERSION = "0.0.1-canary.287";
+const CORE_VERSION = "0.0.1-canary.288";
+const WASM_VERSION = "0.0.1-canary.288";
 const SURREAL_VERSION = "3.0.3";
 var DevToolsService = class DevToolsService {
 	eventsHistory = [];
@@ -12507,6 +12507,8 @@ var TabsCoordinator = class {
 			this.enqueue(async () => {
 				if (this.role === "leader") await this.teardownLeader();
 				else if (this.role === "follower") this.teardownFollower("bucket switch");
+				this.setRole("solo");
+				this.leaderTabId = null;
 				this.bucketId = bucketId;
 				const timeout = setTimeout(() => {
 					this.startResolve = null;
@@ -14011,7 +14013,7 @@ function createTabsCoordinator(config, s, host) {
 	return new TabsCoordinator({
 		tabId: s.tabId,
 		fingerprint: computeTabsFingerprint({
-			coreVersion: "0.0.1-canary.287",
+			coreVersion: "0.0.1-canary.288",
 			schemaHash: hash53(config.schemaSurql),
 			endpoint: config.database.endpoint ?? "",
 			namespace: config.database.namespace,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spooky-sync/core",
-  "version": "0.0.1-canary.287",
+  "version": "0.0.1-canary.288",
   "type": "module",
   "sideEffects": false,
   "main": "./dist/index.js",
@@ -75,8 +75,8 @@
     }
   },
   "dependencies": {
-    "@spooky-sync/query-builder": "0.0.1-canary.287",
-    "@spooky-sync/ssp-wasm": "0.0.1-canary.287",
+    "@spooky-sync/query-builder": "0.0.1-canary.288",
+    "@spooky-sync/ssp-wasm": "0.0.1-canary.288",
     "@sqlite.org/sqlite-wasm": "3.53.0-build1",
     "@surrealdb/wasm": "^3.0.3",
     "blurhash": "^2.0.5",

package/src/services/tabs/coordinator.test.ts

@@ -362,4 +362,25 @@ describe('TabsCoordinator integration', () => {
     await b.coordinator.stop();
     await a.coordinator.stop();
   });
+
+  it('signs in from an anonymous leader into a user bucket another tab already leads', async () => {
+    const signedIn = makeCoordinator('signed-in', {
+      // The real hook awaits a worker RPC. FakePort does not buffer messages
+      // before onmessage is set, so let the joining tab bind its sync port.
+      async exposeClientPort() { await flush(20); },
+    });
+    const signingIn = makeCoordinator('signing-in');
+    try {
+      await signedIn.coordinator.start('user1');
+      await signingIn.coordinator.start('anon');
+
+      await expect(signingIn.coordinator.moveToBucket('user1')).resolves.toBe('follower');
+      expect(signingIn.log.adoptAttached.map((s) => s.bucketId)).toEqual(['user1']);
+      expect(signingIn.log.released).toBe(1);
+      expect(signingIn.coordinator.leaderTabId).toBe('signed-in');
+    } finally {
+      await signingIn.coordinator.stop();
+      await signedIn.coordinator.stop();
+    }
+  }, 20_000);
 });

package/src/services/tabs/coordinator.ts

@@ -369,6 +369,11 @@ export class TabsCoordinator {
         // us from the old namespace and re-elects there.
         if (this.role === 'leader') await this.teardownLeader();
         else if (this.role === 'follower') this.teardownFollower('bucket switch');
+        // Ownership ended with the old bucket. Keeping `leader` here makes
+        // attachToLeader reject the new bucket's follower ports when another
+        // tab already owns it, leaving sign-in waiting until a reload.
+        this.setRole('solo');
+        this.leaderTabId = null;
         this.bucketId = bucketId;
         const timeout = setTimeout(() => {
           this.startResolve = null;