@larksuiteoapi/node-sdk 1.62.2 → 1.63.0

package/es/index.js

@@ -85418,10 +85418,16 @@ class WSClient {
                     timeout: 15000,
                 });
                 if (code !== ErrorCode.ok) {
-                    this.logger.error('[ws]', `code: ${code}, ${code === ErrorCode.system_busy ? msg : 'system busy'}`);
-                    if (code === ErrorCode.system_busy || code === ErrorCode.internal_error) {
-                        return false;
+                    const reason = code === ErrorCode.system_busy ? 'system busy' : msg;
+                    this.logger.error('[ws]', `code: ${code}, ${reason}`);
+                    if (code === ErrorCode.internal_error) {
+                        return { ok: false, retryable: true };
                     }
+                    return {
+                        ok: false,
+                        retryable: false,
+                        error: `pullConnectConfig failed: code=${code}, msg=${reason}`,
+                    };
                 }
                 const { device_id, service_id } = qs.parse(URL);
                 this.wsConfig.updateWs({
@@ -85434,11 +85440,11 @@ class WSClient {
                     reconnectNonce: ClientConfig.ReconnectNonce * 1000
                 });
                 this.logger.debug('[ws]', `get connect config success, ws url: ${URL}`);
-                return true;
+                return { ok: true };
             }
             catch (e) {
                 this.logger.error('[ws]', (e === null || e === void 0 ? void 0 : e.message) || 'system busy');
-                return false;
+                return { ok: false, retryable: true };
             }
         });
     }
@@ -85477,18 +85483,17 @@ class WSClient {
             this.isConnecting = true;
             // Invalidate any in-flight reconnect loops from previous sessions
             const currentGeneration = ++this.reconnectGeneration;
-            const tryConnect = () => {
+            const tryConnect = () => __awaiter(this, void 0, void 0, function* () {
                 this.reconnectInfo.lastConnectTime = Date.now();
-                return this.pullConnectConfig()
-                    .then(isSuccess => isSuccess ? this.connect() : Promise.resolve(false))
-                    .then(isSuccess => {
-                    if (isSuccess) {
-                        this.communicate();
-                        return Promise.resolve(true);
-                    }
-                    return Promise.resolve(false);
-                });
-            };
+                const pullResult = yield this.pullConnectConfig();
+                if (!pullResult.ok)
+                    return pullResult;
+                const connected = yield this.connect();
+                if (!connected)
+                    return { ok: false, retryable: true };
+                this.communicate();
+                return { ok: true };
+            });
             if (this.pingInterval) {
                 clearTimeout(this.pingInterval);
             }
@@ -85500,17 +85505,25 @@ class WSClient {
                 if (wsInstance) {
                     wsInstance === null || wsInstance === void 0 ? void 0 : wsInstance.terminate();
                 }
-                let isSuccess = false;
+                let result = { ok: false, retryable: true };
                 try {
-                    isSuccess = yield tryConnect();
+                    result = yield tryConnect();
                 }
                 finally {
                     this.isConnecting = false;
                 }
-                if (isSuccess) {
+                if (result.ok) {
                     this.hasEverConnected = true;
                     this.safeInvoke('onReady', this.onReady);
                 }
+                else if (!result.retryable) {
+                    // Non-recoverable error from pullConnectConfig — bail out without retry.
+                    // Reset hasEverConnected so a subsequent start() is treated as a fresh
+                    // session (onReady fires, not onReconnected).
+                    this.hasEverConnected = false;
+                    this.safeInvoke('onError', this.onError, new Error(result.error));
+                    return;
+                }
                 else {
                     this.logger.error('[ws]', 'connect failed');
                     yield this.reConnect();
@@ -85542,13 +85555,13 @@ class WSClient {
                             return;
                         }
                         count++;
-                        const isSuccess = yield tryConnect();
+                        const result = yield tryConnect();
                         // Re-check after async operation in case a new session started
                         if (currentGeneration !== this.reconnectGeneration) {
                             return;
                         }
                         // if reconnectCount < 0, the reconnect time is infinite
-                        if (isSuccess) {
+                        if (result.ok) {
                             this.logger.debug('[ws]', 'reconnect success');
                             if (this.hasEverConnected) {
                                 this.safeInvoke('onReconnected', this.onReconnected);
@@ -85560,6 +85573,15 @@ class WSClient {
                             this.isConnecting = false;
                             return;
                         }
+                        if (!result.retryable) {
+                            // Non-recoverable error — abort the loop, do not schedule another attempt.
+                            // Reset hasEverConnected so a subsequent start() is treated as a fresh
+                            // session (onReady fires, not onReconnected).
+                            this.isConnecting = false;
+                            this.hasEverConnected = false;
+                            this.safeInvoke('onError', this.onError, new Error(result.error));
+                            return;
+                        }
                         this.logger.info('ws', `unable to connect to the server after trying ${count} times")`);
                         if (reconnectCount >= 0 && count >= reconnectCount) {
                             this.isConnecting = false;

package/lib/index.js

@@ -85436,10 +85436,16 @@ class WSClient {
                     timeout: 15000,
                 });
                 if (code !== ErrorCode.ok) {
-                    this.logger.error('[ws]', `code: ${code}, ${code === ErrorCode.system_busy ? msg : 'system busy'}`);
-                    if (code === ErrorCode.system_busy || code === ErrorCode.internal_error) {
-                        return false;
+                    const reason = code === ErrorCode.system_busy ? 'system busy' : msg;
+                    this.logger.error('[ws]', `code: ${code}, ${reason}`);
+                    if (code === ErrorCode.internal_error) {
+                        return { ok: false, retryable: true };
                     }
+                    return {
+                        ok: false,
+                        retryable: false,
+                        error: `pullConnectConfig failed: code=${code}, msg=${reason}`,
+                    };
                 }
                 const { device_id, service_id } = qs__default["default"].parse(URL);
                 this.wsConfig.updateWs({
@@ -85452,11 +85458,11 @@ class WSClient {
                     reconnectNonce: ClientConfig.ReconnectNonce * 1000
                 });
                 this.logger.debug('[ws]', `get connect config success, ws url: ${URL}`);
-                return true;
+                return { ok: true };
             }
             catch (e) {
                 this.logger.error('[ws]', (e === null || e === void 0 ? void 0 : e.message) || 'system busy');
-                return false;
+                return { ok: false, retryable: true };
             }
         });
     }
@@ -85495,18 +85501,17 @@ class WSClient {
             this.isConnecting = true;
             // Invalidate any in-flight reconnect loops from previous sessions
             const currentGeneration = ++this.reconnectGeneration;
-            const tryConnect = () => {
+            const tryConnect = () => __awaiter(this, void 0, void 0, function* () {
                 this.reconnectInfo.lastConnectTime = Date.now();
-                return this.pullConnectConfig()
-                    .then(isSuccess => isSuccess ? this.connect() : Promise.resolve(false))
-                    .then(isSuccess => {
-                    if (isSuccess) {
-                        this.communicate();
-                        return Promise.resolve(true);
-                    }
-                    return Promise.resolve(false);
-                });
-            };
+                const pullResult = yield this.pullConnectConfig();
+                if (!pullResult.ok)
+                    return pullResult;
+                const connected = yield this.connect();
+                if (!connected)
+                    return { ok: false, retryable: true };
+                this.communicate();
+                return { ok: true };
+            });
             if (this.pingInterval) {
                 clearTimeout(this.pingInterval);
             }
@@ -85518,17 +85523,25 @@ class WSClient {
                 if (wsInstance) {
                     wsInstance === null || wsInstance === void 0 ? void 0 : wsInstance.terminate();
                 }
-                let isSuccess = false;
+                let result = { ok: false, retryable: true };
                 try {
-                    isSuccess = yield tryConnect();
+                    result = yield tryConnect();
                 }
                 finally {
                     this.isConnecting = false;
                 }
-                if (isSuccess) {
+                if (result.ok) {
                     this.hasEverConnected = true;
                     this.safeInvoke('onReady', this.onReady);
                 }
+                else if (!result.retryable) {
+                    // Non-recoverable error from pullConnectConfig — bail out without retry.
+                    // Reset hasEverConnected so a subsequent start() is treated as a fresh
+                    // session (onReady fires, not onReconnected).
+                    this.hasEverConnected = false;
+                    this.safeInvoke('onError', this.onError, new Error(result.error));
+                    return;
+                }
                 else {
                     this.logger.error('[ws]', 'connect failed');
                     yield this.reConnect();
@@ -85560,13 +85573,13 @@ class WSClient {
                             return;
                         }
                         count++;
-                        const isSuccess = yield tryConnect();
+                        const result = yield tryConnect();
                         // Re-check after async operation in case a new session started
                         if (currentGeneration !== this.reconnectGeneration) {
                             return;
                         }
                         // if reconnectCount < 0, the reconnect time is infinite
-                        if (isSuccess) {
+                        if (result.ok) {
                             this.logger.debug('[ws]', 'reconnect success');
                             if (this.hasEverConnected) {
                                 this.safeInvoke('onReconnected', this.onReconnected);
@@ -85578,6 +85591,15 @@ class WSClient {
                             this.isConnecting = false;
                             return;
                         }
+                        if (!result.retryable) {
+                            // Non-recoverable error — abort the loop, do not schedule another attempt.
+                            // Reset hasEverConnected so a subsequent start() is treated as a fresh
+                            // session (onReady fires, not onReconnected).
+                            this.isConnecting = false;
+                            this.hasEverConnected = false;
+                            this.safeInvoke('onError', this.onError, new Error(result.error));
+                            return;
+                        }
                         this.logger.info('ws', `unable to connect to the server after trying ${count} times")`);
                         if (reconnectCount >= 0 && count >= reconnectCount) {
                             this.isConnecting = false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@larksuiteoapi/node-sdk",
-  "version": "1.62.2",
+  "version": "1.63.0",
   "description": "larksuite open sdk for nodejs",
   "keywords": [
     "feishu",

package/types/index.d.ts

@@ -293069,6 +293069,7 @@ interface IConstructorParams {
     /** Fires after the reconnect loop successfully re-establishes the connection. */
     onReconnected?: () => void;
 }
+
 declare class WSClient {
     private wsConfig;
     private logger;