v2.7.2 - better reconnection handlings

Author: Tomato6966

docs/src/content/docs/extra/version-log.mdx

@@ -26,7 +26,7 @@ Key Changes:
 - there is now a new getter: `node.reconnectionAttemptCount` (which allows you to get the count of the current reconnection attempts, considering the new option `node.options.retryTimespan`)
 - Before the `node.reconnectAttempts` was a number-counter. Now it's just an array of Date-Times, to be able to allow Retrys per a timespan (e.g. you can have 20 retries every hour.) - setup able through `node.options.retryTimespan`, on default it's -1. so it behaves as before.
 - FIX: Now instead of counting from "1 retry up" it counts from 0 up, means if you set `node.retryAmount` to "1" it actually tries 1 time to retry, and not 0. (issue before)
-
+- New node-event "reconnectinprogress" which is triggered before "reconnecting" is triggered, but "reconnecting" might not be triggered, when the max retryAmount is reached, therefore "reconnectinprogress" exists.
 
 ## **Version 2.7.1**
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "lavalink-client",
-  "version": "2.7.1",
+  "version": "2.7.2",
   "description": "Easy, flexible and feature-rich lavalink@v4 Client. Both for Beginners and Proficients.",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",

src/structures/Node.ts

@@ -431,6 +431,7 @@ export class LavalinkNode {
 
         this.resetAckTimeouts(false, true);
 
+        if(this.pingTimeout) clearTimeout(this.pingTimeout);
         this.pingTimeout = setTimeout(() => {
             this.pingTimeout = null;
             if (!this.socket) {
@@ -1029,7 +1030,7 @@ export class LavalinkNode {
      *
      * @example
      * ```ts
-     * await player.node.reconnect();
+     * await player.node.reconnect(true); //true forcefully trys the reconnect
      * ```
      */
     private reconnect(force = false): void {
@@ -1046,6 +1047,8 @@ export class LavalinkNode {
             this.executeReconnect();
             return;
         }
+
+        if(this.reconnectTimeout) clearTimeout(this.reconnectTimeout);
         this.reconnectTimeout = setTimeout(() => {
             this.reconnectTimeout = null;
             this.executeReconnect();
@@ -1180,10 +1183,7 @@ export class LavalinkNode {
 
         if (code !== 1000 || reason !== "Node-Destroy") {
             if (this.NodeManager.nodes.has(this.id)) { // try to reconnect only when the node is still in the nodeManager.nodes list
-                // Only reconnect if not already in progress
-                if (this.reconnectionState === ReconnectionState.IDLE) {
-                    this.reconnect();
-                }
+                this.reconnect();
             }
         }
 
@@ -1688,11 +1688,7 @@ export class LavalinkNode {
 
                 this.NodeManager.LavalinkManager.emit("playerQueueEmptyStart", player, this.NodeManager.LavalinkManager.options.playerOptions.onEmptyQueue?.destroyAfterMs);
 
-                if (player.get("internal_queueempty")) {
-                    clearTimeout(player.get("internal_queueempty"));
-                    player.set("internal_queueempty", undefined);
-                }
-
+                if (player.get("internal_queueempty")) clearTimeout(player.get("internal_queueempty"));
                 player.set("internal_queueempty", setTimeout(() => {
                     player.set("internal_queueempty", undefined);
                     if (player.queue.current) {