@nolag/queue 0.1.3 → 1.1.0

package/dist/index.cjs

@@ -1,7 +1,5 @@
 'use strict';
 
-var jsSdk = require('@nolag/js-sdk');
-
 /**
  * Tiny typed event emitter — framework-agnostic base for NoLag SDKs.
  *
@@ -358,6 +356,29 @@ function createLogger(prefix, enabled) {
         console.log(`[${prefix}]`, ...args);
     };
 }
+// ============ Wrapper registry ============
+// One wrapper instance per (client, appName): two wrappers sharing an app on
+// one connection would collide on topics, presence and the online lobby.
+// Warn (not throw): HMR and tests legitimately construct before disposing.
+const wrapperRegistry = new WeakMap();
+/** Register a wrapper against a client + appName; warns on collision. */
+function registerWrapper(client, appName, wrapperName) {
+    let apps = wrapperRegistry.get(client);
+    if (!apps) {
+        apps = new Map();
+        wrapperRegistry.set(client, apps);
+    }
+    const existing = apps.get(appName);
+    if (existing) {
+        console.warn(`[${wrapperName}] Another wrapper (${existing}) is already attached to this client for app "${appName}". ` +
+            `Use one wrapper per (client, app) — detach the other instance first.`);
+    }
+    apps.set(appName, wrapperName);
+}
+/** Release a wrapper's (client, appName) registration on detach. */
+function releaseWrapper(client, appName) {
+    wrapperRegistry.get(client)?.delete(appName);
+}
 
 /** Default app name for NoLag queue SDK */
 const DEFAULT_APP_NAME = 'queue';
@@ -371,6 +392,8 @@ const TOPIC_JOBS = 'jobs';
 const TOPIC_PROGRESS = '_progress';
 /** Lobby ID for global online presence */
 const LOBBY_ID = 'online';
+/** Delay before the post-setup lobby presence refetch (catches simultaneous joiners) */
+const LOBBY_REFRESH_DELAY_MS = 2000;
 
 /**
  * QueueRoom — a single named queue with job lifecycle, progress tracking, and worker presence.
@@ -379,13 +402,18 @@ const LOBBY_ID = 'online';
  */
 class QueueRoom extends EventEmitter {
     /** @internal */
-    constructor(name, roomContext, localWorkerId, options, log) {
+    constructor(name, roomContext, localWorkerId, options, log, isConnected) {
         super();
+        // Stored topic handler refs — cleanup removes exactly these, never all
+        // handlers for a topic (the client may be shared with other consumers).
+        this._onJobsRef = null;
+        this._onProgressRef = null;
         this.name = name;
         this._roomContext = roomContext;
         this._localWorkerId = localWorkerId;
         this._options = options;
         this._log = log;
+        this._isConnected = isConnected;
         this._jobStore = new JobStore(options.maxJobCache);
         this._workerManager = new WorkerManager();
         this._presenceManager = new PresenceManager(''); // local actor set after connect
@@ -558,12 +586,15 @@ class QueueRoom extends EventEmitter {
             this._roomContext.subscribe(TOPIC_JOBS);
         }
         this._roomContext.subscribe(TOPIC_PROGRESS);
-        this._roomContext.on(TOPIC_JOBS, (data) => {
+        // Listen for job lifecycle messages (refs stored for handler-specific removal)
+        this._onJobsRef = (data) => {
             this._handleJobMessage(data);
-        });
-        this._roomContext.on(TOPIC_PROGRESS, (data) => {
+        };
+        this._roomContext.on(TOPIC_JOBS, this._onJobsRef);
+        this._onProgressRef = (data) => {
             this._handleProgressMessage(data);
-        });
+        };
+        this._roomContext.on(TOPIC_PROGRESS, this._onProgressRef);
     }
     /** @internal Set presence and fetch room members */
     _activate() {
@@ -616,10 +647,20 @@ class QueueRoom extends EventEmitter {
     /** @internal Unsubscribe and clean up */
     _cleanup() {
         this._log('Room cleanup:', this.name);
-        this._roomContext.unsubscribe(TOPIC_JOBS);
-        this._roomContext.unsubscribe(TOPIC_PROGRESS);
-        this._roomContext.off(TOPIC_JOBS);
-        this._roomContext.off(TOPIC_PROGRESS);
+        // Server unsubscribes need a live socket; skip when disconnected
+        // (best-effort — the core would no-op with an error callback anyway).
+        if (this._isConnected()) {
+            this._roomContext.unsubscribe(TOPIC_JOBS);
+            this._roomContext.unsubscribe(TOPIC_PROGRESS);
+        }
+        // Handler-specific removal only: the client may be shared, and a bare
+        // off(topic) would strip other consumers' handlers too.
+        if (this._onJobsRef)
+            this._roomContext.off(TOPIC_JOBS, this._onJobsRef);
+        if (this._onProgressRef)
+            this._roomContext.off(TOPIC_PROGRESS, this._onProgressRef);
+        this._onJobsRef = null;
+        this._onProgressRef = null;
         this._jobStore.clear();
         this._workerManager.clear();
         this._presenceManager.clear();
@@ -692,6 +733,9 @@ class QueueRoom extends EventEmitter {
             activeJobs: 0,
             concurrency: this._options.concurrency,
             metadata: this._options.metadata,
+            // Scope tag: on a shared client, other apps' wrappers filter our
+            // presence out by this (and we filter theirs).
+            __scope: this._options.appName,
         };
         this._roomContext.setPresence(presenceData);
     }
@@ -703,35 +747,73 @@ class QueueRoom extends EventEmitter {
  * Provides job lifecycle management, progress tracking, worker management,
  * and global presence tracking — all framework-agnostic via events.
  *
+ * The wrapper NEVER manages the connection. The app owns one core NoLag
+ * client (shared by any number of wrappers on distinct apps) and the
+ * wrapper attaches to it at construction and releases it via `detach()`.
+ *
  * @example
  * ```typescript
+ * import { NoLag } from '@nolag/js-sdk';
  * import { NoLagQueue } from '@nolag/queue';
  *
- * const queue = new NoLagQueue(token, { role: 'worker', concurrency: 2 });
+ * const client = NoLag(async () => (await (await fetch('/api/nolag-token')).json()).token);
+ * const queue = new NoLagQueue({ client, role: 'worker', concurrency: 2 });
  *
  * queue.on('connected', () => console.log('Connected!'));
  *
- * await queue.connect();
+ * await client.connect();   // the app owns the connection
+ * await queue.ready();      // wrapper setup done (identity, lobby, queues)
  *
  * const room = queue.joinQueue('image-processing');
  * room.on('jobAdded', (job) => {
  *   room.claimJob(job.id);
- *   // process...
  *   room.reportProgress(job.id, 50);
  *   room.completeJob(job.id, { output: 'result' });
  * });
+ *
+ * queue.detach();           // wrapper releases its handlers and topics
+ * client.disconnect();      // the app closes the socket
  * ```
  */
 class NoLagQueue extends EventEmitter {
-    constructor(token, options = {}) {
+    constructor(options) {
         super();
-        this._client = null;
         this._localWorker = null;
         this._queues = new Map();
         this._lobby = null;
         this._onlineWorkers = new Map();
         this._actorToWorkerId = new Map();
-        this._token = token;
+        // Lifecycle: one setup run per connection epoch; detach is terminal.
+        this._epoch = 0;
+        this._detached = false;
+        this._isReady = false;
+        this._lobbyRefreshTimer = null;
+        // Stored client handler refs. INVARIANT: every client.on() below has a
+        // matching client.off() in detach() — never bare off(event), never inline
+        // closures on the client.
+        this._onConnectRef = () => this._onConnect();
+        this._onDisconnectRef = (reason) => {
+            this._log('Disconnected:', reason);
+            this.emit('disconnected', reason);
+        };
+        this._onReconnectRef = () => {
+            this._log('Reconnecting...');
+            this.emit('reconnecting');
+        };
+        this._onErrorRef = (error) => {
+            this._log('Error:', error);
+            this.emit('error', error);
+        };
+        this._onPresenceJoinRef = (data) => this._handleRoomPresenceJoin(data);
+        this._onPresenceLeaveRef = (data) => this._handleRoomPresenceLeave(data);
+        this._onPresenceUpdateRef = (data) => this._handleRoomPresenceUpdate(data);
+        this._onLobbyJoinRef = (data) => this._handleLobbyJoin(data);
+        this._onLobbyLeaveRef = (data) => this._handleLobbyLeave(data);
+        this._onLobbyUpdateRef = (data) => this._handleLobbyUpdate(data);
+        if (!options?.client) {
+            throw new TypeError('NoLagQueue requires an injected NoLag client: new NoLagQueue({ client, role, ... })');
+        }
+        this._client = options.client;
         this._workerId = options.workerId ?? generateId();
         this._options = {
             workerId: this._workerId,
@@ -739,21 +821,49 @@ class NoLagQueue extends EventEmitter {
             concurrency: options.concurrency ?? 1,
             metadata: options.metadata,
             appName: options.appName ?? DEFAULT_APP_NAME,
-            url: options.url,
             maxJobCache: options.maxJobCache ?? DEFAULT_MAX_JOB_CACHE,
             debug: options.debug ?? false,
-            reconnect: options.reconnect ?? true,
             queues: options.queues ?? [],
             loadBalanceGroup: options.loadBalanceGroup,
         };
         this._log = createLogger('NoLagQueue', this._options.debug);
+        this._readyPromise = new Promise((resolve, reject) => {
+            this._readyResolve = resolve;
+            this._readyReject = reject;
+        });
+        // ready() rejection is only meaningful to callers that await it
+        this._readyPromise.catch(() => { });
+        registerWrapper(this._client, this._options.appName, 'NoLagQueue');
+        // Construction = attach: wire everything now, with stored refs.
+        this._client.on('connect', this._onConnectRef);
+        this._client.on('disconnect', this._onDisconnectRef);
+        this._client.on('reconnect', this._onReconnectRef);
+        this._client.on('error', this._onErrorRef);
+        this._client.on('presence:join', this._onPresenceJoinRef);
+        this._client.on('presence:leave', this._onPresenceLeaveRef);
+        this._client.on('presence:update', this._onPresenceUpdateRef);
+        this._client.on('lobbyPresence:join', this._onLobbyJoinRef);
+        this._client.on('lobbyPresence:leave', this._onLobbyLeaveRef);
+        this._client.on('lobbyPresence:update', this._onLobbyUpdateRef);
+        // Attach-to-connected: if the client is already authenticated, run setup.
+        // The microtask lets the caller wire wrapper event handlers synchronously
+        // first; a racing real 'connect' event wins via the epoch guard.
+        queueMicrotask(() => {
+            if (this._epoch === 0 && !this._detached && this._client.connected) {
+                this._onConnect();
+            }
+        });
     }
     // ============ Public Properties ============
-    /** Whether the underlying connection is established */
+    /** Whether the underlying connection is established (connected ≠ ready) */
     get connected() {
-        return this._client?.connected ?? false;
+        return !this._detached && this._client.connected;
+    }
+    /** The injected core client (owned by the app, not the wrapper) */
+    get client() {
+        return this._client;
     }
-    /** The local worker's info (available after connect) */
+    /** The local worker's info (available after ready) */
     get localWorker() {
         return this._localWorker;
     }
@@ -763,94 +873,159 @@ class NoLagQueue extends EventEmitter {
     }
     // ============ Lifecycle ============
     /**
-     * Connect to NoLag and set up global presence.
+     * Resolves once the wrapper's first setup completed (identity, lobby and
+     * configured queues ready — equivalently, once 'connected' has fired).
+     * Rejects only if detach() is called before that. Client auth failures
+     * surface via the app's own `await client.connect()`, not here.
      */
-    async connect() {
-        this._log('Connecting...');
-        const clientOptions = {
-            debug: this._options.debug,
-            reconnect: this._options.reconnect,
-        };
-        if (this._options.url) {
-            clientOptions.url = this._options.url;
-        }
-        this._client = jsSdk.NoLag(this._token, clientOptions);
-        // Wire client lifecycle events
-        this._client.on('connect', () => {
-            this._log('Connected');
-            if (this._queues.size > 0) {
-                this._log('Reconnected — restoring queues...');
-                this._restoreQueues();
-                this.emit('reconnected');
-            }
-        });
-        this._client.on('disconnect', (reason) => {
-            this._log('Disconnected:', reason);
-            this.emit('disconnected', reason);
-        });
-        this._client.on('reconnect', () => {
-            this._log('Reconnecting...');
-        });
-        this._client.on('error', (error) => {
-            this._log('Error:', error);
-            this.emit('error', error);
-        });
-        // Connect
-        await this._client.connect();
-        // Wire room-level presence events
-        this._client.on('presence:join', (data) => {
-            this._handleRoomPresenceJoin(data);
-        });
-        this._client.on('presence:leave', (data) => {
-            this._handleRoomPresenceLeave(data);
-        });
-        this._client.on('presence:update', (data) => {
-            this._handleRoomPresenceUpdate(data);
-        });
-        // Create local worker record
-        this._localWorker = {
-            workerId: this._workerId,
-            actorTokenId: this._client.actorId,
-            role: this._options.role,
-            activeJobs: 0,
-            concurrency: this._options.concurrency,
-            metadata: this._options.metadata,
-            joinedAt: Date.now(),
-            isLocal: true,
-        };
-        this._log('Local worker:', this._localWorker.workerId, '→', this._localWorker.actorTokenId);
-        // Set up lobby for global presence
-        await this._setupLobby();
-        // Emit connected now that _localWorker and lobby are ready
-        this.emit('connected');
-        // Deferred lobby refetch to catch workers who joined during the setup window
-        setTimeout(() => {
-            if (this._lobby && this._client?.connected) {
-                this._lobby.fetchPresence().then((state) => {
-                    this._hydrateOnlineWorkers(state);
-                }).catch(() => { });
-            }
-        }, 2000);
+    ready() {
+        return this._readyPromise;
     }
     /**
-     * Disconnect from NoLag and clean up all queue rooms.
+     * Detach from the client: remove every handler this wrapper added,
+     * unsubscribe its topics and lobby (when connected), clear state.
+     * Terminal and idempotent; never touches the socket. To use the queue
+     * again, construct a new instance.
      */
-    disconnect() {
-        this._log('Disconnecting...');
-        // Clean up queue rooms
+    detach() {
+        if (this._detached)
+            return;
+        this._log('Detaching...');
+        this._detached = true;
+        this._epoch++; // aborts any in-flight setup at its next checkpoint
+        if (this._lobbyRefreshTimer) {
+            clearTimeout(this._lobbyRefreshTimer);
+            this._lobbyRefreshTimer = null;
+        }
+        // Remove all client handlers by stored ref
+        this._client.off('connect', this._onConnectRef);
+        this._client.off('disconnect', this._onDisconnectRef);
+        this._client.off('reconnect', this._onReconnectRef);
+        this._client.off('error', this._onErrorRef);
+        this._client.off('presence:join', this._onPresenceJoinRef);
+        this._client.off('presence:leave', this._onPresenceLeaveRef);
+        this._client.off('presence:update', this._onPresenceUpdateRef);
+        this._client.off('lobbyPresence:join', this._onLobbyJoinRef);
+        this._client.off('lobbyPresence:leave', this._onLobbyLeaveRef);
+        this._client.off('lobbyPresence:update', this._onLobbyUpdateRef);
+        // Queue rooms: handler-specific off + connected-gated server unsubscribe
         for (const name of [...this._queues.keys()]) {
-            this.leaveQueue(name);
+            this._queues.get(name)._cleanup();
+            this._queues.delete(name);
+        }
+        // Lobby: server unsubscribe is best-effort and needs a live socket
+        if (this._lobby && this._client.connected) {
+            try {
+                this._lobby.unsubscribe();
+            }
+            catch {
+                /* best-effort */
+            }
         }
-        // Unsubscribe from lobby
-        this._lobby?.unsubscribe();
         this._lobby = null;
-        // Disconnect client
-        this._client?.disconnect();
-        this._client = null;
-        // Clear state
         this._onlineWorkers.clear();
         this._actorToWorkerId.clear();
         this._localWorker = null;
+        releaseWrapper(this._client, this._options.appName);
+        if (!this._isReady) {
+            this._readyReject(new Error('NoLagQueue detached before ready'));
+        }
+    }
+    // ============ Private: Epoch Setup ============
+    _onConnect() {
+        this._epoch++;
+        void this._runSetup(this._epoch);
+    }
+    /**
+     * One setup pass per connection epoch. Serves both initial setup (epoch 1)
+     * and reconnect restore (epoch > 1). Aborts silently whenever a newer
+     * epoch started or the wrapper detached — checked after every await.
+     */
+    async _runSetup(epoch) {
+        const stale = () => epoch !== this._epoch || this._detached;
+        this._log(this._isReady ? 'Restoring after reconnect...' : 'Setting up...');
+        // Identity (client.actorId is guaranteed post-auth)
+        if (!this._localWorker) {
+            this._localWorker = {
+                workerId: this._workerId,
+                actorTokenId: this._client.actorId,
+                role: this._options.role,
+                activeJobs: 0,
+                concurrency: this._options.concurrency,
+                metadata: this._options.metadata,
+                joinedAt: Date.now(),
+                isLocal: true,
+            };
+            this._log('Local worker:', this._localWorker.workerId, '→', this._localWorker.actorTokenId);
+        }
+        else {
+            this._localWorker.actorTokenId = this._client.actorId;
+        }
+        // Lobby: subscribe every epoch (idempotent server-side) and diff-hydrate
+        // from the returned snapshot — one path for setup and restore.
+        if (!this._lobby) {
+            this._lobby = this._client.setApp(this._options.appName).setLobby(LOBBY_ID);
+        }
+        try {
+            const state = await this._lobby.subscribe();
+            if (stale())
+                return;
+            this._diffHydrateOnlineWorkers(state);
+            this._log('Lobby subscribed, online workers:', this._onlineWorkers.size);
+        }
+        catch (err) {
+            if (stale())
+                return;
+            this._log('Lobby subscription failed:', err);
+        }
+        if (!this._isReady) {
+            // First successful setup: pre-subscribe configured queues.
+            for (const queueName of this._options.queues) {
+                this._subscribeQueue(queueName);
+            }
+        }
+        else {
+            // Server auto-restored topic subscriptions; only room-scoped presence
+            // needs re-applying (the core does not restore it).
+            for (const room of this._queues.values()) {
+                room._updateLocalPresence();
+            }
+        }
+        if (stale())
+            return;
+        // Ready keys on the first setup that COMPLETES, not on epoch 1: an
+        // epoch aborted by a racing reconnect must not strand ready().
+        if (!this._isReady) {
+            this._isReady = true;
+            this._readyResolve();
+            this.emit('connected');
+        }
+        else {
+            this.emit('reconnected');
+        }
+        // Deferred lobby refetch: catches workers who joined during the setup
+        // window (e.g. simultaneous multi-tab connects).
+        this._scheduleLobbyRefresh(epoch);
+    }
+    _scheduleLobbyRefresh(epoch) {
+        if (this._lobbyRefreshTimer)
+            clearTimeout(this._lobbyRefreshTimer);
+        this._lobbyRefreshTimer = setTimeout(() => {
+            this._lobbyRefreshTimer = null;
+            if (epoch !== this._epoch || this._detached || !this._client.connected || !this._lobby) {
+                return;
+            }
+            this._lobby
+                .fetchPresence()
+                .then((state) => {
+                if (epoch !== this._epoch || this._detached)
+                    return;
+                this._diffHydrateOnlineWorkers(state);
+            })
+                .catch(() => {
+                /* best-effort */
+            });
+        }, LOBBY_REFRESH_DELAY_MS);
     }
     // ============ Queue Management ============
     /**
@@ -858,9 +1033,7 @@ class NoLagQueue extends EventEmitter {
      * Returns an existing room if already joined.
      */
     joinQueue(name) {
-        if (!this._client || !this._localWorker) {
-            throw new Error('Not connected — call connect() first');
-        }
+        this._assertUsable();
         let room = this._queues.get(name);
         if (!room) {
             room = this._subscribeQueue(name);
@@ -892,25 +1065,42 @@ class NoLagQueue extends EventEmitter {
     getOnlineWorkers() {
         return Array.from(this._onlineWorkers.values());
     }
+    // ============ Private: Guards ============
+    _assertUsable() {
+        if (this._detached) {
+            throw new Error('NoLagQueue has been detached — construct a new instance');
+        }
+        if (!this._isReady || !this._localWorker) {
+            throw new Error('NoLagQueue not ready — await ready() or the "connected" event');
+        }
+    }
     // ============ Private: Queue Setup ============
     _subscribeQueue(name) {
-        if (!this._client || !this._localWorker) {
-            throw new Error('Not connected — call connect() first');
-        }
         this._log('Subscribing queue:', name);
         const roomContext = this._client.setApp(this._options.appName).setRoom(name);
-        const room = new QueueRoom(name, roomContext, this._workerId, this._options, createLogger(`QueueRoom:${name}`, this._options.debug));
+        const room = new QueueRoom(name, roomContext, this._workerId, this._options, createLogger(`QueueRoom:${name}`, this._options.debug), () => this._client.connected);
         room._setLocalActorId(this._localWorker.actorTokenId);
         this._queues.set(name, room);
         room._subscribe();
         return room;
     }
+    // ============ Private: Scope Filtering ============
+    /**
+     * On a shared client, presence events from other apps' wrappers arrive on
+     * the same connection-level events. Wrappers stamp their presence with a
+     * `__scope` (their appName); a mismatched tag means another app's data.
+     * Untagged presence is accepted (older peers in this same app).
+     */
+    _foreignScope(data) {
+        const scope = data?.__scope;
+        return typeof scope === 'string' && scope !== this._options.appName;
+    }
     // ============ Private: Room Presence ============
     _handleRoomPresenceJoin(data) {
         if (data.actorTokenId === this._localWorker?.actorTokenId)
             return;
         const presenceData = data.presence;
-        if (!presenceData?.workerId)
+        if (!presenceData?.workerId || this._foreignScope(presenceData))
             return;
         const worker = this._presenceToWorker(data.actorTokenId, presenceData);
         this._actorToWorkerId.set(data.actorTokenId, worker.workerId);
@@ -935,7 +1125,7 @@ class NoLagQueue extends EventEmitter {
         if (data.actorTokenId === this._localWorker?.actorTokenId)
             return;
         const presenceData = data.presence;
-        if (!presenceData?.workerId)
+        if (!presenceData?.workerId || this._foreignScope(presenceData))
             return;
         if (this._onlineWorkers.has(presenceData.workerId)) {
             const worker = this._presenceToWorker(data.actorTokenId, presenceData);
@@ -947,37 +1137,12 @@ class NoLagQueue extends EventEmitter {
         }
     }
     // ============ Private: Lobby ============
-    async _setupLobby() {
-        if (!this._client)
-            return;
-        this._lobby = this._client.setApp(this._options.appName).setLobby(LOBBY_ID);
-        const lobbyHandler = (type) => (data) => {
-            const event = data;
-            if (type === 'join')
-                this._handleLobbyJoin(event);
-            else if (type === 'leave')
-                this._handleLobbyLeave(event);
-            else
-                this._handleLobbyUpdate(event);
-        };
-        this._client.on('lobbyPresence:join', lobbyHandler('join'));
-        this._client.on('lobbyPresence:leave', lobbyHandler('leave'));
-        this._client.on('lobbyPresence:update', lobbyHandler('update'));
-        try {
-            const initialState = await this._lobby.subscribe();
-            this._hydrateOnlineWorkers(initialState);
-            this._log('Lobby subscribed, online workers:', this._onlineWorkers.size);
-        }
-        catch (err) {
-            this._log('Lobby subscription failed:', err);
-        }
-    }
     _handleLobbyJoin(event) {
         const { actorId, data } = event;
         if (actorId === this._localWorker?.actorTokenId)
             return;
         const presenceData = data;
-        if (!presenceData.workerId)
+        if (!presenceData?.workerId || this._foreignScope(presenceData))
             return;
         const worker = this._presenceToWorker(actorId, presenceData);
         this._actorToWorkerId.set(actorId, worker.workerId);
@@ -991,6 +1156,8 @@ class NoLagQueue extends EventEmitter {
         if (actorId === this._localWorker?.actorTokenId)
             return;
         const presenceData = data;
+        if (this._foreignScope(presenceData))
+            return;
         const workerId = presenceData?.workerId
             || this._actorToWorkerId.get(actorId)
             || this._findWorkerIdByActorId(actorId);
@@ -1008,29 +1175,57 @@ class NoLagQueue extends EventEmitter {
         if (actorId === this._localWorker?.actorTokenId)
             return;
         const presenceData = data;
-        if (!presenceData.workerId)
+        if (!presenceData?.workerId || this._foreignScope(presenceData))
             return;
         const worker = this._presenceToWorker(actorId, presenceData);
         this._onlineWorkers.set(worker.workerId, worker);
     }
-    _hydrateOnlineWorkers(state) {
+    /**
+     * Reconcile the online-worker map against a fresh lobby snapshot, emitting
+     * only the deltas (workerOffline for vanished, workerOnline for new). One
+     * path for initial hydration, reconnect restore, and the deferred refetch.
+     */
+    _diffHydrateOnlineWorkers(state) {
+        // Build the fresh worker set from the snapshot
+        const fresh = new Map();
+        const freshActors = new Map();
         for (const roomId of Object.keys(state)) {
             const roomPresence = state[roomId];
             for (const actorId of Object.keys(roomPresence)) {
                 if (actorId === this._localWorker?.actorTokenId)
                     continue;
                 const raw = roomPresence[actorId];
+                // Server returns full actor records with presence nested under .presence
                 const presenceData = (raw?.presence ?? raw);
-                if (presenceData?.workerId) {
-                    const worker = this._presenceToWorker(actorId, presenceData);
-                    this._actorToWorkerId.set(actorId, worker.workerId);
-                    if (!this._onlineWorkers.has(worker.workerId)) {
-                        this._onlineWorkers.set(worker.workerId, worker);
-                        this.emit('workerOnline', worker);
+                if (presenceData?.workerId && !this._foreignScope(presenceData)) {
+                    if (!fresh.has(presenceData.workerId)) {
+                        fresh.set(presenceData.workerId, this._presenceToWorker(actorId, presenceData));
                     }
+                    freshActors.set(actorId, presenceData.workerId);
                 }
             }
         }
+        // Vanished workers
+        for (const [workerId, worker] of [...this._onlineWorkers]) {
+            if (!fresh.has(workerId)) {
+                this._onlineWorkers.delete(workerId);
+                for (const [actorId, mappedWorkerId] of [...this._actorToWorkerId]) {
+                    if (mappedWorkerId === workerId)
+                        this._actorToWorkerId.delete(actorId);
+                }
+                this.emit('workerOffline', worker);
+            }
+        }
+        // New workers
+        for (const [workerId, worker] of fresh) {
+            if (!this._onlineWorkers.has(workerId)) {
+                this._onlineWorkers.set(workerId, worker);
+                this.emit('workerOnline', worker);
+            }
+        }
+        for (const [actorId, workerId] of freshActors) {
+            this._actorToWorkerId.set(actorId, workerId);
+        }
     }
     // ============ Private: Helpers ============
     _presenceToWorker(actorTokenId, data) {
@@ -1052,21 +1247,6 @@ class NoLagQueue extends EventEmitter {
         }
         return undefined;
     }
-    _restoreQueues() {
-        // On reconnect, js-sdk auto-restores subscriptions.
-        // Re-set presence on all active queue rooms.
-        for (const room of this._queues.values()) {
-            room._updateLocalPresence();
-        }
-        // Re-fetch lobby presence
-        this._lobby?.fetchPresence().then((state) => {
-            this._onlineWorkers.clear();
-            this._actorToWorkerId.clear();
-            this._hydrateOnlineWorkers(state);
-        }).catch((err) => {
-            this._log('Failed to re-fetch lobby presence:', err);
-        });
-    }
 }
 
 exports.EventEmitter = EventEmitter;