@nolag/signal 0.1.2 → 1.0.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.
  *
@@ -144,6 +142,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 signal SDK */
 const DEFAULT_APP_NAME = 'signal';
@@ -151,6 +172,8 @@ const DEFAULT_APP_NAME = 'signal';
 const TOPIC_SIGNALING = 'signaling';
 /** 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;
 
 /**
  * SignalRoom — a single signaling room for WebRTC peer discovery and exchange.
@@ -159,13 +182,17 @@ const LOBBY_ID = 'online';
  */
 class SignalRoom extends EventEmitter {
     /** @internal */
-    constructor(name, roomContext, localPeer, options, log) {
+    constructor(name, roomContext, localPeer, options, log, isConnected) {
         super();
+        // Stored topic handler ref — cleanup removes exactly this, never all
+        // handlers for a topic (the client may be shared with other consumers).
+        this._onSignalingRef = null;
         this.name = name;
         this._roomContext = roomContext;
         this._localPeer = localPeer;
         this._options = options;
         this._log = log;
+        this._isConnected = isConnected;
         this._peerManager = new PeerManager(localPeer.actorTokenId);
     }
     // ============ Public Properties ============
@@ -231,9 +258,11 @@ class SignalRoom extends EventEmitter {
     _subscribe() {
         this._log('Room subscribe:', this.name);
         this._roomContext.subscribe(TOPIC_SIGNALING);
-        this._roomContext.on(TOPIC_SIGNALING, (data) => {
+        // Listen for signals (ref stored for handler-specific removal)
+        this._onSignalingRef = (data) => {
             this._handleIncomingSignal(data);
-        });
+        };
+        this._roomContext.on(TOPIC_SIGNALING, this._onSignalingRef);
     }
     /** @internal Set presence and fetch room members */
     _activate() {
@@ -280,8 +309,16 @@ class SignalRoom extends EventEmitter {
     /** @internal Unsubscribe and clean up */
     _cleanup() {
         this._log('Room cleanup:', this.name);
-        this._roomContext.unsubscribe(TOPIC_SIGNALING);
-        this._roomContext.off(TOPIC_SIGNALING);
+        // 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_SIGNALING);
+        }
+        // Handler-specific removal only: the client may be shared, and a bare
+        // off(topic) would strip other consumers' handlers too.
+        if (this._onSignalingRef)
+            this._roomContext.off(TOPIC_SIGNALING, this._onSignalingRef);
+        this._onSignalingRef = null;
         this._peerManager.clear();
         this.removeAllListeners();
     }
@@ -298,6 +335,9 @@ class SignalRoom extends EventEmitter {
         const presenceData = {
             peerId: this._localPeer.peerId,
             metadata: this._localPeer.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);
     }
@@ -309,50 +349,116 @@ class SignalRoom extends EventEmitter {
  * Provides peer discovery, offer/answer/ICE exchange, 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 { NoLagSignal } from '@nolag/signal';
  *
- * const signal = new NoLagSignal(token, { debug: true });
+ * const client = NoLag(async () => (await (await fetch('/api/nolag-token')).json()).token);
+ * const signal = new NoLagSignal({ client, appName: 'my-signal' });
  *
- * signal.on('connected', () => console.log('Connected!'));
  * signal.on('peerOnline', (peer) => console.log(peer.peerId, 'is online'));
  *
- * await signal.connect();
+ * await client.connect();   // the app owns the connection
+ * await signal.ready();     // wrapper setup done (identity, lobby)
  *
  * const room = signal.joinRoom('call-room');
  * room.on('signal', (msg) => {
  *   if (msg.type === 'offer') handleOffer(msg);
  * });
  * room.sendOffer(remotePeerId, offer);
+ *
+ * signal.detach();          // wrapper releases its handlers and topics
+ * client.disconnect();      // the app closes the socket
  * ```
  */
 class NoLagSignal extends EventEmitter {
-    constructor(token, options = {}) {
+    constructor(options) {
         super();
-        this._client = null;
         this._localPeer = null;
         this._rooms = new Map();
         this._lobby = null;
         this._onlinePeers = new Map();
         this._actorToPeerId = 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('NoLagSignal requires an injected NoLag client: new NoLagSignal({ client, ... })');
+        }
+        this._client = options.client;
         this._peerId = generateId();
         this._options = {
             metadata: options.metadata,
             appName: options.appName ?? DEFAULT_APP_NAME,
-            url: options.url,
             debug: options.debug ?? false,
-            reconnect: options.reconnect ?? true,
         };
         this._log = createLogger('NoLagSignal', 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, 'NoLagSignal');
+        // 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 peer's info (available after connect) */
+    /** The local peer's info (available after ready) */
     get localPeer() {
         return this._localPeer;
     }
@@ -362,92 +468,151 @@ class NoLagSignal extends EventEmitter {
     }
     // ============ Lifecycle ============
     /**
-     * Connect to NoLag and set up global presence.
+     * Resolves once the wrapper's first setup completed (identity and lobby
+     * 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._rooms.size > 0) {
-                this._log('Reconnected — restoring rooms...');
-                this._restoreRooms();
-                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 peer
-        this._localPeer = {
-            peerId: this._peerId,
-            actorTokenId: this._client.actorId,
-            connectionState: 'new',
-            metadata: this._options.metadata,
-            joinedAt: Date.now(),
-            isLocal: true,
-        };
-        this._log('Local peer:', this._localPeer.peerId, '→', this._localPeer.actorTokenId);
-        // Set up lobby for global presence
-        await this._setupLobby();
-        // Emit connected now that _localPeer and lobby are ready
-        this.emit('connected');
-        // Deferred lobby refetch to catch peers who joined during the setup window
-        setTimeout(() => {
-            if (this._lobby && this._client?.connected) {
-                this._lobby.fetchPresence().then((state) => {
-                    this._hydrateOnlinePeers(state);
-                }).catch(() => { });
-            }
-        }, 2000);
+    ready() {
+        return this._readyPromise;
     }
     /**
-     * Disconnect from NoLag and clean up all 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 signaling
+     * again, construct a new instance.
      */
-    disconnect() {
-        this._log('Disconnecting...');
-        // Clean up 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);
+        // Rooms: handler-specific off + connected-gated server unsubscribe
         for (const name of [...this._rooms.keys()]) {
-            this.leaveRoom(name);
+            this._rooms.get(name)._cleanup();
+            this._rooms.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._onlinePeers.clear();
         this._actorToPeerId.clear();
         this._localPeer = null;
+        releaseWrapper(this._client, this._options.appName);
+        if (!this._isReady) {
+            this._readyReject(new Error('NoLagSignal 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._localPeer) {
+            this._localPeer = {
+                peerId: this._peerId,
+                actorTokenId: this._client.actorId,
+                connectionState: 'new',
+                metadata: this._options.metadata,
+                joinedAt: Date.now(),
+                isLocal: true,
+            };
+            this._log('Local peer:', this._localPeer.peerId, '→', this._localPeer.actorTokenId);
+        }
+        else {
+            this._localPeer.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._diffHydrateOnlinePeers(state);
+            this._log('Lobby subscribed, online peers:', this._onlinePeers.size);
+        }
+        catch (err) {
+            if (stale())
+                return;
+            this._log('Lobby subscription failed:', err);
+        }
+        if (this._isReady) {
+            // Server auto-restored topic subscriptions; only room-scoped presence
+            // needs re-applying (the core does not restore it).
+            for (const room of this._rooms.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 peers 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._diffHydrateOnlinePeers(state);
+            })
+                .catch(() => {
+                /* best-effort */
+            });
+        }, LOBBY_REFRESH_DELAY_MS);
     }
     // ============ Room Management ============
     /**
@@ -455,9 +620,7 @@ class NoLagSignal extends EventEmitter {
      * Returns an existing room if already joined.
      */
     joinRoom(name) {
-        if (!this._client || !this._localPeer) {
-            throw new Error('Not connected — call connect() first');
-        }
+        this._assertUsable();
         let room = this._rooms.get(name);
         if (!room) {
             room = this._subscribeRoom(name);
@@ -489,24 +652,41 @@ class NoLagSignal extends EventEmitter {
     getOnlinePeers() {
         return Array.from(this._onlinePeers.values());
     }
+    // ============ Private: Guards ============
+    _assertUsable() {
+        if (this._detached) {
+            throw new Error('NoLagSignal has been detached — construct a new instance');
+        }
+        if (!this._isReady || !this._localPeer) {
+            throw new Error('NoLagSignal not ready — await ready() or the "connected" event');
+        }
+    }
     // ============ Private: Room Setup ============
     _subscribeRoom(name) {
-        if (!this._client || !this._localPeer) {
-            throw new Error('Not connected — call connect() first');
-        }
         this._log('Subscribing room:', name);
         const roomContext = this._client.setApp(this._options.appName).setRoom(name);
-        const room = new SignalRoom(name, roomContext, this._localPeer, this._options, createLogger(`SignalRoom:${name}`, this._options.debug));
+        const room = new SignalRoom(name, roomContext, this._localPeer, this._options, createLogger(`SignalRoom:${name}`, this._options.debug), () => this._client.connected);
         this._rooms.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._localPeer?.actorTokenId)
             return;
         const presenceData = data.presence;
-        if (!presenceData?.peerId)
+        if (!presenceData?.peerId || this._foreignScope(presenceData))
             return;
         const peer = this._presenceToPeer(data.actorTokenId, presenceData);
         this._actorToPeerId.set(data.actorTokenId, peer.peerId);
@@ -531,7 +711,7 @@ class NoLagSignal extends EventEmitter {
         if (data.actorTokenId === this._localPeer?.actorTokenId)
             return;
         const presenceData = data.presence;
-        if (!presenceData?.peerId)
+        if (!presenceData?.peerId || this._foreignScope(presenceData))
             return;
         if (this._onlinePeers.has(presenceData.peerId)) {
             const peer = this._presenceToPeer(data.actorTokenId, presenceData);
@@ -543,37 +723,12 @@ class NoLagSignal 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._hydrateOnlinePeers(initialState);
-            this._log('Lobby subscribed, online peers:', this._onlinePeers.size);
-        }
-        catch (err) {
-            this._log('Lobby subscription failed:', err);
-        }
-    }
     _handleLobbyJoin(event) {
         const { actorId, data } = event;
         if (actorId === this._localPeer?.actorTokenId)
             return;
         const presenceData = data;
-        if (!presenceData.peerId)
+        if (!presenceData.peerId || this._foreignScope(presenceData))
             return;
         const peer = this._presenceToPeer(actorId, presenceData);
         this._actorToPeerId.set(actorId, peer.peerId);
@@ -587,6 +742,8 @@ class NoLagSignal extends EventEmitter {
         if (actorId === this._localPeer?.actorTokenId)
             return;
         const presenceData = data;
+        if (this._foreignScope(presenceData))
+            return;
         const peerId = presenceData?.peerId
             || this._actorToPeerId.get(actorId)
             || this._findPeerIdByActorId(actorId);
@@ -604,29 +761,57 @@ class NoLagSignal extends EventEmitter {
         if (actorId === this._localPeer?.actorTokenId)
             return;
         const presenceData = data;
-        if (!presenceData.peerId)
+        if (!presenceData.peerId || this._foreignScope(presenceData))
             return;
         const peer = this._presenceToPeer(actorId, presenceData);
         this._onlinePeers.set(peer.peerId, peer);
     }
-    _hydrateOnlinePeers(state) {
+    /**
+     * Reconcile the online-peer map against a fresh lobby snapshot, emitting
+     * only the deltas (peerOffline for vanished, peerOnline for new). One path
+     * for initial hydration, reconnect restore, and the deferred refetch.
+     */
+    _diffHydrateOnlinePeers(state) {
+        // Build the fresh peer 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._localPeer?.actorTokenId)
                     continue;
                 const raw = roomPresence[actorId];
+                // Server returns full actor records with presence nested under .presence
                 const presenceData = (raw?.presence ?? raw);
-                if (presenceData?.peerId) {
-                    const peer = this._presenceToPeer(actorId, presenceData);
-                    this._actorToPeerId.set(actorId, peer.peerId);
-                    if (!this._onlinePeers.has(peer.peerId)) {
-                        this._onlinePeers.set(peer.peerId, peer);
-                        this.emit('peerOnline', peer);
+                if (presenceData?.peerId && !this._foreignScope(presenceData)) {
+                    if (!fresh.has(presenceData.peerId)) {
+                        fresh.set(presenceData.peerId, this._presenceToPeer(actorId, presenceData));
                     }
+                    freshActors.set(actorId, presenceData.peerId);
                 }
             }
         }
+        // Vanished peers
+        for (const [peerId, peer] of [...this._onlinePeers]) {
+            if (!fresh.has(peerId)) {
+                this._onlinePeers.delete(peerId);
+                for (const [actorId, mappedPeerId] of [...this._actorToPeerId]) {
+                    if (mappedPeerId === peerId)
+                        this._actorToPeerId.delete(actorId);
+                }
+                this.emit('peerOffline', peer);
+            }
+        }
+        // New peers
+        for (const [peerId, peer] of fresh) {
+            if (!this._onlinePeers.has(peerId)) {
+                this._onlinePeers.set(peerId, peer);
+                this.emit('peerOnline', peer);
+            }
+        }
+        for (const [actorId, peerId] of freshActors) {
+            this._actorToPeerId.set(actorId, peerId);
+        }
     }
     // ============ Private: Helpers ============
     _presenceToPeer(actorTokenId, data) {
@@ -646,21 +831,6 @@ class NoLagSignal extends EventEmitter {
         }
         return undefined;
     }
-    _restoreRooms() {
-        // On reconnect, js-sdk auto-restores subscriptions.
-        // Re-set presence on all active rooms.
-        for (const room of this._rooms.values()) {
-            room._updateLocalPresence();
-        }
-        // Re-fetch lobby presence
-        this._lobby?.fetchPresence().then((state) => {
-            this._onlinePeers.clear();
-            this._actorToPeerId.clear();
-            this._hydrateOnlinePeers(state);
-        }).catch((err) => {
-            this._log('Failed to re-fetch lobby presence:', err);
-        });
-    }
 }
 
 exports.EventEmitter = EventEmitter;