ework-qq-bridge 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ework-qq-bridge",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "QQ group <-> ework issue bridge (OneBot 11 reverse WebSocket)",
   "type": "module",
   "main": "src/index.ts",

package/src/onebot.ts

@@ -123,7 +123,11 @@ export function createOneBotServer(opts: OneBotServerOptions) {
           void opts.onEvent(ev);
         }
       },
-      close() {
+      close(client: ServerWebSocket<unknown>) {
+        // A reconnect race can fire the OLD socket's close after the NEW
+        // socket's open; only tear down when the active client is the one
+        // that closed, else the live connection is orphaned.
+        if (ws !== client) return;
         rejectAll("connection closed");
       },
     },

package/tests/bridge.test.ts

@@ -144,3 +144,20 @@ describe("chat mode", () => {
     expect(replies[0]).toContain("没看懂指令");
   });
 });
+
+test("onebot close ignores non-active client (reconnect race)", () => {
+  const { createOneBotServer } = require("../src/onebot");
+  let ready = 0;
+  const srv = createOneBotServer({ path: "/ws", accessToken: "t", onEvent: () => {}, onReady: () => { ready++; } });
+  const fake = (id: string) => ({ id, send: () => {}, close: () => {} });
+  const a = fake("a"), b = fake("b");
+  srv.handlers.open(a);
+  expect(ready).toBe(1);
+  srv.handlers.open(b);
+  expect(ready).toBe(1);
+  srv.handlers.close(a);
+  expect(srv.connected).toBe(true);
+  srv.handlers.close(b);
+  expect(srv.connected).toBe(false);
+});
+