magmastream 2.10.4 → 2.10.5

package/dist/structures/Manager.js

@@ -438,7 +438,9 @@ class Manager extends events_1.EventEmitter {
         const player = this.getPlayer(update.guild_id);
         if (!player)
             return;
-        this.emit(Enums_1.ManagerEventTypes.Debug, `[MANAGER] Updating voice state: ${Utils_1.JSONUtils.safe(update, 2)}`);
+        if (this.listenerCount(Enums_1.ManagerEventTypes.Debug)) {
+            this.emit(Enums_1.ManagerEventTypes.Debug, `[MANAGER] Updating voice state: ${Utils_1.JSONUtils.safe(update, 2)}`);
+        }
         if ("token" in update) {
             return await this.handleVoiceServerUpdate(player, update);
         }

package/dist/structures/Node.d.ts

@@ -23,7 +23,6 @@ export declare class Node {
     private reconnectTimeout?;
     private reconnectAttempts;
     private redisPrefix?;
-    /** Session ID sent in the reconnect header for resumption — cleared once the ready op is received. */
     private pendingResumeSessionId;
     /**
      * Creates an instance of Node.

package/dist/structures/Node.js

@@ -31,7 +31,6 @@ class Node {
     reconnectTimeout;
     reconnectAttempts = 0;
     redisPrefix;
-    /** Session ID sent in the reconnect header for resumption — cleared once the ready op is received. */
     pendingResumeSessionId = null;
     /**
      * Creates an instance of Node.
@@ -256,12 +255,7 @@ class Node {
             "User-Id": this.manager.options.clientId,
             "Client-Name": this.manager.options.clientName,
         };
-        // Capture resume session ID for the WS header, then clear this.sessionId.
-        // REST calls guard on this.sessionId being non-null — keeping the stale value
-        // would let updatePlayer/destroyPlayer fire with an invalid session during the
-        // reconnect window (between connect() and the 'ready' op).
-        // pendingResumeSessionId is kept so the 'ready' handler can still compute
-        // hadPreviousSession correctly. this.sessionId is re-set by 'ready'.
+        // Keep the resume header while blocking REST calls until the next ready event.
         this.pendingResumeSessionId = this.sessionId;
         this.sessionId = null;
         if (typeof this.pendingResumeSessionId === "string" && this.pendingResumeSessionId.length > 0) {
@@ -310,11 +304,9 @@ class Node {
         if (players.size) {
             await Promise.all(Array.from(players.values(), (player) => player.autoMoveNode()));
         }
-        // Always clear reconnect state regardless of connection status
         clearTimeout(this.reconnectTimeout);
         this.reconnectTimeout = undefined;
         this.reconnectAttempts = 0;
-        // Only close the socket if it is actually open
         if (this.connected) {
             this.socket.close(1000, "destroy");
             this.socket.removeAllListeners();
@@ -419,9 +411,7 @@ class Node {
         };
         this.manager.emit(Enums_1.ManagerEventTypes.NodeDisconnect, this, { code, reason });
         this.manager.emit(Enums_1.ManagerEventTypes.Debug, `[NODE] Disconnected node: ${Utils_1.JSONUtils.safe(debugInfo, 2)}`);
-        // Null the session ID immediately so REST calls during the reconnect delay
-        // window don't fire with a stale session. connect() reloads it from storage
-        // into pendingResumeSessionId for the WS Session-Id header.
+        // Prevent REST calls from using a stale session during reconnect.
         this.sessionId = null;
         if (this.manager.useableNode) {
             const players = this.manager.players.filter((player) => player.node.options.identifier == this.options.identifier);
@@ -500,9 +490,6 @@ class Node {
             case "ready":
                 this.manager.emit(Enums_1.ManagerEventTypes.Debug, `[NODE] Node message: ${Utils_1.JSONUtils.safe(payload, 2)}`);
                 this.rest.setSessionId(payload.sessionId);
-                // pendingResumeSessionId holds what we sent in Session-Id header (if anything).
-                // Use it — not this.sessionId which was nulled in connect() — to detect whether
-                // we attempted resumption with a different session than what Lavalink gave back.
                 const hadPreviousSession = this.pendingResumeSessionId && this.pendingResumeSessionId !== payload.sessionId;
                 this.pendingResumeSessionId = null;
                 this.sessionId = payload.sessionId;
@@ -544,7 +531,6 @@ class Node {
         if (!player || player.node.options.identifier !== this.options.identifier)
             return;
         player.position = payload.state.position ?? 0;
-        // Transport status can lag lifecycle events; expose it without overwriting player.state.
         this.manager.emit(Enums_1.ManagerEventTypes.PlayerUpdate, player, payload);
     }
     /**
@@ -1239,7 +1225,7 @@ class Node {
      * @param {WebSocketClosedEvent} payload - The event payload containing additional data about the WebSocket close event.
      */
     socketClosed(player, payload) {
-        player.state = Enums_1.StateTypes.Disconnected;
+        // A move can close the old socket after the new connection is ready.
         this.manager.emit(Enums_1.ManagerEventTypes.SocketClosed, player, payload);
         this.manager.emit(Enums_1.ManagerEventTypes.Debug, `[NODE] Websocket closed for player: ${player.guildId} with payload: ${Utils_1.JSONUtils.safe(payload, 2)}`);
     }

package/dist/structures/Player.js

@@ -1136,9 +1136,6 @@ class Player {
                     context: { guildId: this.guildId },
                 });
             }
-            // Only destroy the player on the old node if it is still reachable.
-            // If the server is down the REST call would fail anyway; skipping it
-            // also prevents a spurious error when switching nodes during an outage.
             if (this.node.connected) {
                 await this.node.rest.destroyPlayer(this.guildId).catch(() => { });
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "magmastream",
-    "version": "2.10.4",
+    "version": "2.10.5",
     "description": "A user-friendly Lavalink client designed for NodeJS.",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",