@ascegu/teamily 1.0.5 → 1.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ascegu/teamily",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "OpenClaw Teamily channel plugin - Team instant messaging server integration",
   "type": "module",
   "main": "index.ts",

package/src/monitor.ts

@@ -80,8 +80,17 @@ export class TeamilyMonitor {
     }
 
     if (this.ws) {
-      this.ws.close(1000, "Monitoring stopped");
+      const ws = this.ws;
+      ws.onopen = null;
+      ws.onmessage = null;
+      ws.onerror = null;
+      ws.onclose = null;
       this.ws = null;
+      try {
+        ws.close(1000, "Monitoring stopped");
+      } catch {
+        // Ignore – socket may already be closed.
+      }
     }
 
     this.setState("disconnected");
@@ -201,15 +210,28 @@ export class TeamilyMonitor {
 
   /**
    * Handle WebSocket error.
+   * Detaches event handlers before closing to prevent recursive calls
+   * (ws.close() on an errored socket can re-fire onerror → stack overflow).
    */
   private handleError(error: unknown): void {
     const errorMessage = error instanceof Error ? error.message : String(error);
     this.setState("error", errorMessage);
 
-    // Close the connection so it can be re-established
-    if (this.ws) {
-      this.ws.close();
+    // Detach handlers and grab ref before nulling, so close() cannot recurse.
+    const ws = this.ws;
+    if (ws) {
+      ws.onopen = null;
+      ws.onmessage = null;
+      ws.onerror = null;
+      ws.onclose = null;
       this.ws = null;
+      try {
+        ws.close();
+      } catch {
+        // Ignore – socket may already be closed/invalid.
+      }
+      // onclose was detached, so manually trigger reconnect logic.
+      this.handleClose();
     }
   }