aqualink 2.20.1 → 3.0.0

package/build/structures/Node.js

@@ -102,10 +102,10 @@ class Node {
     this.host = connOptions.host || 'localhost'
     this.name = connOptions.name || this.host
     this.port = connOptions.port || 2333
-    this.auth = connOptions.auth || 'youshallnotpass'
+    this.auth = connOptions.auth || connOptions.password || 'youshallnotpass'
     this.sessionId = connOptions.sessionId || null
     this.regions = connOptions.regions || []
-    this.ssl = !!connOptions.ssl
+    this.ssl = !!connOptions.ssl || !!connOptions.secure || false
     this.wsUrl = _functions.buildWsUrl(this.host, this.port, this.ssl)
 
     this.rest = new Rest(aqua, this)
@@ -187,10 +187,12 @@ class Node {
     this._isConnecting = false
     this.reconnectAttempted = 0
     this._emitDebug('WebSocket connection established')
-    this.aqua?._trace?.('node.ws.open', {
-      node: this.name,
-      reconnectAttempted: this.reconnectAttempted
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.ws.open', {
+        node: this.name,
+        reconnectAttempted: this.reconnectAttempted
+      })
+    }
 
     if (!this.aqua?.bypassChecks?.nodeFetchInfo && !this.info) {
       const timeoutId = setTimeout(() => {
@@ -272,12 +274,14 @@ class Node {
       code,
       reason: _functions.reasonToString(reason)
     })
-    this.aqua?._trace?.('node.ws.close', {
-      node: this.name,
-      code,
-      reason: _functions.reasonToString(reason),
-      hasSessionId: !!this.sessionId
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.ws.close', {
+        node: this.name,
+        code,
+        reason: _functions.reasonToString(reason),
+        hasSessionId: !!this.sessionId
+      })
+    }
 
     if (this.isDestroyed) return
 
@@ -311,11 +315,13 @@ class Node {
     this._clearReconnectTimeout()
 
     const attempt = ++this.reconnectAttempted
-    this.aqua?._trace?.('node.ws.reconnect.scheduled', {
-      node: this.name,
-      attempt,
-      infinite: !!this.infiniteReconnects
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.ws.reconnect.scheduled', {
+        node: this.name,
+        attempt,
+        infinite: !!this.infiniteReconnects
+      })
+    }
 
     if (this.infiniteReconnects) {
       this.aqua.emit(AqualinkEvents.NodeReconnect, this, {
@@ -384,10 +390,12 @@ class Node {
     this._isConnecting = true
     this.state = NODE_STATE.CONNECTING
     this._cleanup()
-    this.aqua?._trace?.('node.ws.connect', {
-      node: this.name,
-      url: this.wsUrl
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.ws.connect', {
+        node: this.name,
+        url: this.wsUrl
+      })
+    }
 
     try {
       const h = this._boundHandlers
@@ -453,7 +461,7 @@ class Node {
 
       const ws = new WebSocketImpl(this.wsUrl, {
         headers: this._headers,
-        perMessageDeflate: true,
+        perMessageDeflate: false,
         handshakeTimeout: this.timeout,
         maxPayload: this.maxPayload,
         skipUTF8Validation: this.skipUTF8Validation
@@ -588,12 +596,14 @@ class Node {
       oldSessionId && oldSessionId !== sessionId && !payload.resumed
 
     this.sessionId = sessionId
-    this.aqua?._trace?.('node.ready.packet', {
-      node: this.name,
-      resumed: !!payload.resumed,
-      oldSessionId,
-      newSessionId: sessionId
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.ready.packet', {
+        node: this.name,
+        resumed: !!payload.resumed,
+        oldSessionId,
+        newSessionId: sessionId
+      })
+    }
     this.rest.setSessionId(sessionId)
     this._headers['Session-Id'] = sessionId
 
@@ -601,19 +611,38 @@ class Node {
       this._emitDebug(
         `Session changed from ${oldSessionId} to ${sessionId}, invalidating stale players`
       )
+      try {
+        await this.aqua._storeBrokenPlayers?.(this)
+      } catch (e) {
+        this._emitDebug(
+          `Failed to snapshot stale players before invalidation: ${e?.message || e}`
+        )
+      }
       const playersToDestroy = []
       for (const [guildId, player] of this.aqua.players) {
         if (player?.nodes === this || player?.nodes?.name === this.name) {
-          playersToDestroy.push(guildId)
+          playersToDestroy.push({ guildId, player })
         }
       }
-      for (const guildId of playersToDestroy) {
+      for (const { guildId, player } of playersToDestroy) {
         try {
-          this._emitDebug(`Destroying stale player for guild ${guildId}`)
-          await this.aqua.destroyPlayer(guildId)
+          this._emitDebug(
+            `Invalidating stale player for guild ${guildId} without voice disconnect`
+          )
+          player?.destroy?.({
+            preserveClient: true,
+            skipRemote: true,
+            preserveMessage: true,
+            preserveTracks: true,
+            preserveReconnecting: true
+          })
+          if (this.aqua.players.get(String(guildId)) === player) {
+            this.aqua.players.delete(String(guildId))
+          }
+          this.players?.delete?.(player)
         } catch (e) {
           this._emitDebug(
-            `Failed to destroy stale player ${guildId}: ${e?.message || e}`
+            `Failed to invalidate stale player ${guildId}: ${e?.message || e}`
           )
         }
       }
@@ -635,11 +664,13 @@ class Node {
 
   async _resumePlayers() {
     if (!this.sessionId) return
-    this.aqua?._trace?.('node.resume.begin', {
-      node: this.name,
-      sessionId: this.sessionId,
-      players: this.aqua?.players?.size || 0
-    })
+    if (this.aqua?.debugTrace) {
+      this.aqua._trace('node.resume.begin', {
+        node: this.name,
+        sessionId: this.sessionId,
+        players: this.aqua?.players?.size || 0
+      })
+    }
 
     try {
       await this.rest.makeRequest('PATCH', `/v4/sessions/${this.sessionId}`, {
@@ -653,18 +684,24 @@ class Node {
             (player?.nodes === this || player?.nodes?.name === this.name) &&
             player.voiceChannel
           ) {
-            try {
-              this._emitDebug(`Rejoining voice for guild ${guildId} on resume`)
-              this.aqua?._trace?.('node.resume.rejoin', {
-                node: this.name,
-                guildId,
-                voiceChannel: player.voiceChannel
-              })
-              player.connect({
-                voiceChannel: player.voiceChannel,
-                deaf: player.deaf,
-                mute: player.mute
-              })
+              try {
+                const recoveryToken = player._claimVoiceRecovery?.(
+                  'node_resume_rejoin'
+                )
+                this._emitDebug(`Rejoining voice for guild ${guildId} on resume`)
+                if (this.aqua?.debugTrace) {
+                  this.aqua._trace('node.resume.rejoin', {
+                    node: this.name,
+                    guildId,
+                    voiceChannel: player.voiceChannel
+                  })
+                }
+                if (player._isVoiceRecoveryActive?.(recoveryToken))
+                  player.connect({
+                    voiceChannel: player.voiceChannel,
+                    deaf: player.deaf,
+                    mute: player.mute
+                  })
             } catch (e) {
               this._emitDebug(
                 `Failed to rejoin voice for ${guildId}: ${e?.message || e}`
@@ -678,10 +715,12 @@ class Node {
         await this.aqua.loadPlayers()
       }
     } catch (err) {
-      this.aqua?._trace?.('node.resume.error', {
-        node: this.name,
-        error: _functions.errMsg(err)
-      })
+      if (this.aqua?.debugTrace) {
+        this.aqua._trace('node.resume.error', {
+          node: this.name,
+          error: _functions.errMsg(err)
+        })
+      }
       this._emitError(`Failed to resume session: ${_functions.errMsg(err)}`)
       throw err
     }