xshell 1.0.208 → 1.0.210

package/net.browser.d.ts

@@ -207,13 +207,13 @@ export declare class Remote {
         verbose?: boolean;
         websocket?: WebSocket;
         keeper?: RemoteKeeperOptions;
-        on_error?(error: WebSocketConnectionError, websocket?: WebSocket): void;
+        on_error?(error: WebSocketConnectionError | Error, websocket?: WebSocket): void;
     });
-    /** 统一处理首次连接错误和连接后的错误 */
+    /** 统一处理首次连接和连接后的 websocket 错误 */
     _on_error: (error: WebSocketConnectionError, websocket?: WebSocket) => void;
     _on_message: (data: ArrayBuffer, websocket: WebSocket) => void;
-    /** 使用者自定义的在连接后出错时的处理 */
-    on_error?(error: WebSocketConnectionError, websocket?: WebSocket): void;
+    /** 使用者自定义的在 websocket 连接出错时，或者 handlers 出错时的处理 */
+    on_error(error: WebSocketConnectionError | Error, websocket?: WebSocket): void;
     /** 幂等，保证 websocket 已连接，否则抛出异常
         一般情况不需要手动调用，在其它方法中会自动调用这个方法，除非需要手动建立 websocket 连接并确保成功
         连接断开后，通过传入 url 参数构造的 remote 实例会自动创建新的 websocket 连接，而通过传入 websocket 参数构造的实例只会检查连接状态，在断开时抛出异常
@@ -230,7 +230,8 @@ export declare class Remote {
     /** 处理接收到的 websocket message 并解析, 根据 message.id 或 message.func 分发到对应的 handler 进行处理
         如果 message.done == true 则对端指示当前 remote 可以清理 handler
         如果 handler 返回了值 (一定是数组)，则包装为 message 发送
-        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214  */
+        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214
+        这个方法一般不会抛出错误，也不需要 await，一般在 websocket on_message 时使用 */
     handle(data: Uint8Array, websocket: WebSocket): Promise<void>;
     /** 调用对端 remote 中的 func, 只适用于最简单的一元 rpc (请求, 响应)
         作为 websocket 连接发起方，不需要传入 websocket

package/net.browser.js

@@ -369,10 +369,8 @@ export class Remote {
             };
         }
     }
-    /** 统一处理首次连接错误和连接后的错误 */
+    /** 统一处理首次连接和连接后的 websocket 错误 */
     _on_error = (error, websocket) => {
-        if (websocket) // 连接后出现的错误
-            this.on_error?.(error, websocket);
         if (this.keeper && !this.reconnecting && !this.disconnected) // 在一段时间后调度错误重连
             (async () => {
                 this.reconnecting = true;
@@ -396,18 +394,17 @@ export class Remote {
                         // 重连由 this.connect 里面调用 this._on_error 处理
                     }
             })();
+        this.on_error(error, websocket);
     };
     _on_message = (data, websocket) => {
         this.handle(new Uint8Array(data), websocket);
     };
-    /** 使用者自定义的在连接后出错时的处理 */
+    /** 使用者自定义的在 websocket 连接出错时，或者 handlers 出错时的处理 */
     on_error(error, websocket) {
-        if (this.keeper) {
-            if (this.print)
-                console.log(error.message);
-        }
-        else
-            throw error;
+        // 使用者未定义 Remote 如何处理 error 时，一般来说直接忽略即可，因为 handlers 中报错了也会返回给对端
+        if (this.print)
+            console.log(error);
+        // 这里继续往上层抛没有太大意义，上面一般都是 websocket on_message 这些
     }
     /** 幂等，保证 websocket 已连接，否则抛出异常
         一般情况不需要手动调用，在其它方法中会自动调用这个方法，除非需要手动建立 websocket 连接并确保成功
@@ -499,7 +496,8 @@ export class Remote {
     /** 处理接收到的 websocket message 并解析, 根据 message.id 或 message.func 分发到对应的 handler 进行处理
         如果 message.done == true 则对端指示当前 remote 可以清理 handler
         如果 handler 返回了值 (一定是数组)，则包装为 message 发送
-        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214  */
+        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214
+        这个方法一般不会抛出错误，也不需要 await，一般在 websocket on_message 时使用 */
     async handle(data, websocket) {
         const message = Remote.parse(data);
         const { id, func, done } = message;
@@ -528,7 +526,8 @@ export class Remote {
                 !message.error // 防止无限循环往对方发送 error, 只有在对方无错误时才可以发送
             )
                 await this.send({ id, error, /* 不能设置 done 清理对面 handler, 理由同上 */ }, websocket);
-            throw error;
+            // 这里继续往上层抛没有太大意义，上面一般都是 websocket on_message 这些，交给自定义或默认的 on_error 处理
+            this.on_error(error);
         }
     }
     /** 调用对端 remote 中的 func, 只适用于最简单的一元 rpc (请求, 响应)

package/net.d.ts

@@ -274,13 +274,13 @@ export declare class Remote {
         verbose?: boolean;
         websocket?: WebSocket;
         keeper?: RemoteKeeperOptions;
-        on_error?(error: WebSocketConnectionError, websocket?: WebSocket): void;
+        on_error?(error: WebSocketConnectionError | Error, websocket?: WebSocket): void;
     });
-    /** 统一处理首次连接错误和连接后的错误 */
+    /** 统一处理首次连接和连接后的 websocket 错误 */
     _on_error: (error: WebSocketConnectionError, websocket?: WebSocket) => void;
     _on_message: (data: ArrayBuffer, websocket: WebSocket) => void;
-    /** 使用者自定义的在连接后出错时的处理 */
-    on_error?(error: WebSocketConnectionError, websocket?: WebSocket): void;
+    /** 使用者自定义的在 websocket 连接出错时，或者 handlers 出错时的处理 */
+    on_error(error: WebSocketConnectionError | Error, websocket?: WebSocket): void;
     /** 幂等，保证 websocket 已连接，否则抛出异常
         一般情况不需要手动调用，在其它方法中会自动调用这个方法，除非需要手动建立 websocket 连接并确保成功
         连接断开后，通过传入 url 参数构造的 remote 实例会自动创建新的 websocket 连接，而通过传入 websocket 参数构造的实例只会检查连接状态，在断开时抛出异常
@@ -297,7 +297,8 @@ export declare class Remote {
     /** 处理接收到的 websocket message 并解析, 根据 message.id 或 message.func 分发到对应的 handler 进行处理
         如果 message.done == true 则对端指示当前 remote 可以清理 handler
         如果 handler 返回了值 (一定是数组)，则包装为 message 发送
-        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214  */
+        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214
+        这个方法一般不会抛出错误，也不需要 await，一般在 websocket on_message 时使用 */
     handle(data: Uint8Array, websocket: WebSocket): Promise<void>;
     /** 调用对端 remote 中的 func, 只适用于最简单的一元 rpc (请求, 响应)
         作为 websocket 连接发起方，不需要传入 websocket

package/net.js

@@ -601,10 +601,8 @@ export class Remote {
             };
         }
     }
-    /** 统一处理首次连接错误和连接后的错误 */
+    /** 统一处理首次连接和连接后的 websocket 错误 */
     _on_error = (error, websocket) => {
-        if (websocket) // 连接后出现的错误
-            this.on_error?.(error, websocket);
         if (this.keeper && !this.reconnecting && !this.disconnected) // 在一段时间后调度错误重连
             (async () => {
                 this.reconnecting = true;
@@ -628,18 +626,17 @@ export class Remote {
                         // 重连由 this.connect 里面调用 this._on_error 处理
                     }
             })();
+        this.on_error(error, websocket);
     };
     _on_message = (data, websocket) => {
         this.handle(new Uint8Array(data), websocket);
     };
-    /** 使用者自定义的在连接后出错时的处理 */
+    /** 使用者自定义的在 websocket 连接出错时，或者 handlers 出错时的处理 */
     on_error(error, websocket) {
-        if (this.keeper) {
-            if (this.print)
-                console.log(error.message);
-        }
-        else
-            throw error;
+        // 使用者未定义 Remote 如何处理 error 时，一般来说直接忽略即可，因为 handlers 中报错了也会返回给对端
+        if (this.print)
+            console.log(error);
+        // 这里继续往上层抛没有太大意义，上面一般都是 websocket on_message 这些
     }
     /** 幂等，保证 websocket 已连接，否则抛出异常
         一般情况不需要手动调用，在其它方法中会自动调用这个方法，除非需要手动建立 websocket 连接并确保成功
@@ -731,7 +728,8 @@ export class Remote {
     /** 处理接收到的 websocket message 并解析, 根据 message.id 或 message.func 分发到对应的 handler 进行处理
         如果 message.done == true 则对端指示当前 remote 可以清理 handler
         如果 handler 返回了值 (一定是数组)，则包装为 message 发送
-        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214  */
+        使用 Uint8Array 作为参数更灵活 https://stackoverflow.com/a/74505197/7609214
+        这个方法一般不会抛出错误，也不需要 await，一般在 websocket on_message 时使用 */
     async handle(data, websocket) {
         const message = Remote.parse(data);
         const { id, func, done } = message;
@@ -760,7 +758,8 @@ export class Remote {
                 !message.error // 防止无限循环往对方发送 error, 只有在对方无错误时才可以发送
             )
                 await this.send({ id, error, /* 不能设置 done 清理对面 handler, 理由同上 */ }, websocket);
-            throw error;
+            // 这里继续往上层抛没有太大意义，上面一般都是 websocket on_message 这些，交给自定义或默认的 on_error 处理
+            this.on_error(error);
         }
     }
     /** 调用对端 remote 中的 func, 只适用于最简单的一元 rpc (请求, 响应)

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "xshell",
-    "version": "1.0.208",
+    "version": "1.0.210",
     "type": "module",
     "main": "./index.js",
     "bin": {
@@ -50,8 +50,8 @@
     },
     "dependencies": {
         "@babel/core": "^7.26.0",
-        "@babel/parser": "^7.26.2",
-        "@babel/traverse": "^7.25.9",
+        "@babel/parser": "^7.26.3",
+        "@babel/traverse": "^7.26.3",
         "@koa/cors": "^5.0.0",
         "@stylistic/eslint-plugin": "^2.11.0",
         "@svgr/webpack": "^8.1.0",
@@ -95,7 +95,7 @@
         "react-object-model": "^1.2.18",
         "resolve-path": "^1.4.0",
         "sass": "^1.82.0",
-        "sass-loader": "^16.0.3",
+        "sass-loader": "^16.0.4",
         "source-map-loader": "^5.0.0",
         "strip-ansi": "^7.1.0",
         "style-loader": "^4.0.0",
@@ -113,7 +113,7 @@
         "ws": "^8.18.0"
     },
     "devDependencies": {
-        "@babel/types": "^7.26.0",
+        "@babel/types": "^7.26.3",
         "@types/ali-oss": "^6.16.11",
         "@types/archiver": "^6.0.3",
         "@types/babel__traverse": "^7.20.6",
@@ -127,7 +127,7 @@
         "@types/lodash": "^4.17.13",
         "@types/mime-types": "^2.1.4",
         "@types/node": "^22.10.1",
-        "@types/react": "^18.3.12",
+        "@types/react": "^18.3.13",
         "@types/through2": "^2.0.41",
         "@types/tough-cookie": "^4.0.5",
         "@types/ua-parser-js": "^0.7.39",

package/server.js

@@ -148,16 +148,8 @@ export class Server {
                 maxPayload: 2 ** 30 // 1 GB
             });
             this.websocket_server.on('connection', (ws, request) => {
-                ws.addEventListener('message', async ({ data }) => {
-                    // 处理 funcs 中已经注册的 handlers
-                    try {
-                        await this.remote.handle(new Uint8Array(data), ws);
-                    }
-                    catch (error) {
-                        this.on_error(error);
-                        // 这里再往上扔会成为 uncaughtException 使进程退出，
-                        // 通过上面的处理应该足够了
-                    }
+                ws.addEventListener('message', ({ data }) => {
+                    this.remote.handle(new Uint8Array(data), ws);
                 });
             });
             const on_upgrade = this.on_upgrade.bind(this);
@@ -261,7 +253,8 @@ export class Server {
             console.log(`${error.code}:`, ctx?.request?.url);
         else {
             console.error(error);
-            console.log('ctx:', ctx);
+            if (ctx)
+                console.log('ctx:', ctx);
         }
     }
     on_upgrade(request, socket, head) {