@dxos/messaging 0.3.1-main.5e68e9d → 0.3.1-main.b9bb100

package/dist/lib/node/index.cjs

@@ -54,6 +54,8 @@ var MESSAGE_TIMEOUT = 1e4;
 var __dxlog_file = "/home/runner/work/dxos/dxos/packages/core/mesh/messaging/src/messenger.ts";
 var ReliablePayload = import_protocols.schema.getCodecForType("dxos.mesh.messaging.ReliablePayload");
 var Acknowledgement = import_protocols.schema.getCodecForType("dxos.mesh.messaging.Acknowledgement");
+var ERROR_LIMIT = 5;
+var NETWORK_REBOOT_DELAY = 3e3;
 var RECEIVED_MESSAGES_GC_INTERVAL = 12e4;
 var Messenger = class {
   constructor({ signalManager, retryDelay = 300 }) {
@@ -68,6 +70,8 @@ var Messenger = class {
     */
     this._toClear = new import_util.ComplexSet(import_keys.PublicKey.hash);
     this._closed = true;
+    this._errorCount = 0;
+    this._rebooting = false;
     this._signalManager = signalManager;
     this._retryDelay = retryDelay;
     this.open();
@@ -81,14 +85,14 @@ var Messenger = class {
       id: traceId
     }), {
       F: __dxlog_file,
-      L: 69,
+      L: 74,
       S: this,
       C: (f, a) => f(...a)
     });
     this._ctx = new import_context.Context({
       onError: (err) => import_log.log.catch(err, void 0, {
         F: __dxlog_file,
-        L: 71,
+        L: 76,
         S: this,
         C: (f, a) => f(...a)
       })
@@ -98,7 +102,7 @@ var Messenger = class {
         from: message.author
       }, {
         F: __dxlog_file,
-        L: 75,
+        L: 80,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -112,7 +116,7 @@ var Messenger = class {
       id: traceId
     }), {
       F: __dxlog_file,
-      L: 90,
+      L: 95,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -127,7 +131,7 @@ var Messenger = class {
   async sendMessage({ author, recipient, payload }) {
     (0, import_invariant.invariant)(!this._closed, "Closed", {
       F: __dxlog_file,
-      L: 102,
+      L: 107,
       S: this,
       A: [
         "!this._closed",
@@ -141,7 +145,7 @@ var Messenger = class {
     };
     (0, import_invariant.invariant)(!this._onAckCallbacks.has(reliablePayload.messageId), void 0, {
       F: __dxlog_file,
-      L: 109,
+      L: 114,
       S: this,
       A: [
         "!this._onAckCallbacks.has(reliablePayload.messageId!)",
@@ -154,7 +158,7 @@ var Messenger = class {
       recipient
     }, {
       F: __dxlog_file,
-      L: 110,
+      L: 115,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -169,7 +173,7 @@ var Messenger = class {
         messageId: reliablePayload.messageId
       }, {
         F: __dxlog_file,
-        L: 124,
+        L: 129,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -181,21 +185,22 @@ var Messenger = class {
         err
       }, {
         F: __dxlog_file,
-        L: 126,
+        L: 131,
         S: this,
         C: (f, a) => f(...a)
       }));
     }, this._retryDelay);
-    (0, import_async.scheduleTask)(messageContext, () => {
+    (0, import_async.scheduleTask)(messageContext, async () => {
       (0, import_log.log)("message not delivered", {
         messageId: reliablePayload.messageId
       }, {
         F: __dxlog_file,
-        L: 135,
+        L: 140,
         S: this,
         C: (f, a) => f(...a)
       });
       this._onAckCallbacks.delete(reliablePayload.messageId);
+      await this.errorLimitCheck();
       timeoutHit(new import_protocols.TimeoutError("signaling message not delivered", new import_async.TimeoutError(MESSAGE_TIMEOUT, "Message not delivered")));
       void messageContext.dispose();
     }, MESSAGE_TIMEOUT);
@@ -211,6 +216,54 @@ var Messenger = class {
     });
     return promise;
   }
+  async errorLimitCheck() {
+    (0, import_log.log)(`checking error limit ${this._errorCount}`, void 0, {
+      F: __dxlog_file,
+      L: 165,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    if (this._errorCount++ > ERROR_LIMIT) {
+      await this.rebootNetwork();
+    }
+  }
+  async rebootNetwork() {
+    if (this._rebooting) {
+      return;
+    }
+    this._rebooting = true;
+    (0, import_log.log)("rebooting Messenger/SignalManager", void 0, {
+      F: __dxlog_file,
+      L: 177,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    await this.close();
+    await this._signalManager.close();
+    (0, import_log.log)("pausing", void 0, {
+      F: __dxlog_file,
+      L: 180,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    await new Promise((resolve) => setTimeout(resolve, NETWORK_REBOOT_DELAY));
+    (0, import_log.log)("done pausing", void 0, {
+      F: __dxlog_file,
+      L: 182,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    await this.open();
+    await this._signalManager.open();
+    (0, import_log.log)("done rebooting", void 0, {
+      F: __dxlog_file,
+      L: 185,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    this._errorCount = 0;
+    this._rebooting = false;
+  }
   /**
   * Subscribes onMessage function to messages that contains payload with payloadType.
   * @param payloadType if not specified, onMessage will be subscribed to all types of messages.
@@ -218,7 +271,7 @@ var Messenger = class {
   async listen({ peerId, payloadType, onMessage }) {
     (0, import_invariant.invariant)(!this._closed, "Closed", {
       F: __dxlog_file,
-      L: 171,
+      L: 204,
       S: this,
       A: [
         "!this._closed",
@@ -282,7 +335,7 @@ var Messenger = class {
   async _handleReliablePayload({ author, recipient, payload }) {
     (0, import_invariant.invariant)(payload.type_url === "dxos.mesh.messaging.ReliablePayload", void 0, {
       F: __dxlog_file,
-      L: 232,
+      L: 265,
       S: this,
       A: [
         "payload.type_url === 'dxos.mesh.messaging.ReliablePayload'",
@@ -296,7 +349,7 @@ var Messenger = class {
       messageId: reliablePayload.messageId
     }, {
       F: __dxlog_file,
-      L: 235,
+      L: 268,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -318,7 +371,7 @@ var Messenger = class {
   async _handleAcknowledgement({ payload }) {
     (0, import_invariant.invariant)(payload.type_url === "dxos.mesh.messaging.Acknowledgement", void 0, {
       F: __dxlog_file,
-      L: 257,
+      L: 290,
       S: this,
       A: [
         "payload.type_url === 'dxos.mesh.messaging.Acknowledgement'",
@@ -334,7 +387,7 @@ var Messenger = class {
       to: author
     }, {
       F: __dxlog_file,
-      L: 270,
+      L: 303,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -385,7 +438,7 @@ var Messenger = class {
         elapsed
       }, {
         F: __dxlog_file,
-        L: 318,
+        L: 351,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -1386,14 +1439,11 @@ function _ts_decorate(decorators, target, key, desc) {
   return c > 3 && r && Object.defineProperty(target, key, r), r;
 }
 var __dxlog_file5 = "/home/runner/work/dxos/dxos/packages/core/mesh/messaging/src/signal-manager/websocket-signal-manager.ts";
-var MAX_SERVER_FAILURES = 5;
-var WSS_SIGNAL_SERVER_REBOOT_DELAY = 3e3;
 var WebsocketSignalManager = class {
   constructor(_hosts) {
     this._hosts = _hosts;
     this._servers = /* @__PURE__ */ new Map();
     this._opened = false;
-    this.failureCount = /* @__PURE__ */ new Map();
     this.statusChanged = new import_async5.Event();
     this.commandTrace = new import_async5.Event();
     this.swarmEvent = new import_async5.Event();
@@ -1403,7 +1453,7 @@ var WebsocketSignalManager = class {
       hosts: this._hosts
     }, {
       F: __dxlog_file5,
-      L: 46,
+      L: 43,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -1414,7 +1464,6 @@ var WebsocketSignalManager = class {
       const server = new SignalClient(host.server, async (message) => this.onMessage.emit(message), async (data) => this.swarmEvent.emit(data));
       server.statusChanged.on(() => this.statusChanged.emit(this.getStatus()));
       this._servers.set(host.server, server);
-      this.failureCount.set(host.server, 0);
       server.commandTrace.on((trace5) => this.commandTrace.emit(trace5));
     }
   }
@@ -1426,7 +1475,7 @@ var WebsocketSignalManager = class {
       hosts: this._hosts
     }, {
       F: __dxlog_file5,
-      L: 69,
+      L: 65,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -1434,7 +1483,7 @@ var WebsocketSignalManager = class {
       id: this._instanceId
     }), {
       F: __dxlog_file5,
-      L: 70,
+      L: 66,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -1447,7 +1496,7 @@ var WebsocketSignalManager = class {
       id: this._instanceId
     }), {
       F: __dxlog_file5,
-      L: 77,
+      L: 73,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -1460,38 +1509,6 @@ var WebsocketSignalManager = class {
     await this._ctx.dispose();
     await Promise.all(Array.from(this._servers.values()).map((server) => server.close()));
   }
-  async restartServer(serverName) {
-    (0, import_log5.log)("Restarting server", {
-      serverName
-    }, {
-      F: __dxlog_file5,
-      L: 93,
-      S: this,
-      C: (f, a) => f(...a)
-    });
-    (0, import_invariant5.invariant)(this._opened, "server already closed", {
-      F: __dxlog_file5,
-      L: 94,
-      S: this,
-      A: [
-        "this._opened",
-        "'server already closed'"
-      ]
-    });
-    const server = this._servers.get(serverName);
-    (0, import_invariant5.invariant)(server, "server not found", {
-      F: __dxlog_file5,
-      L: 97,
-      S: this,
-      A: [
-        "server",
-        "'server not found'"
-      ]
-    });
-    await server.close();
-    await (0, import_async5.sleep)(WSS_SIGNAL_SERVER_REBOOT_DELAY);
-    await server.open();
-  }
   getStatus() {
     return Array.from(this._servers.values()).map((server) => server.getStatus());
   }
@@ -1501,13 +1518,13 @@ var WebsocketSignalManager = class {
       peerId
     }, {
       F: __dxlog_file5,
-      L: 110,
+      L: 93,
       S: this,
       C: (f, a) => f(...a)
     });
     (0, import_invariant5.invariant)(this._opened, "Closed", {
       F: __dxlog_file5,
-      L: 111,
+      L: 94,
       S: this,
       A: [
         "this._opened",
@@ -1525,13 +1542,13 @@ var WebsocketSignalManager = class {
       peerId
     }, {
       F: __dxlog_file5,
-      L: 117,
+      L: 100,
       S: this,
       C: (f, a) => f(...a)
     });
     (0, import_invariant5.invariant)(this._opened, "Closed", {
       F: __dxlog_file5,
-      L: 118,
+      L: 101,
       S: this,
       A: [
         "this._opened",
@@ -1544,75 +1561,44 @@ var WebsocketSignalManager = class {
     }));
   }
   async sendMessage({ author, recipient, payload }) {
-    (0, import_log5.log)(`Signal ${recipient.truncate()}`, void 0, {
+    (0, import_log5.log)(`Signal ${recipient}`, void 0, {
       F: __dxlog_file5,
-      L: 132,
+      L: 115,
       S: this,
       C: (f, a) => f(...a)
     });
     (0, import_invariant5.invariant)(this._opened, "Closed", {
       F: __dxlog_file5,
-      L: 133,
+      L: 116,
       S: this,
       A: [
         "this._opened",
         "'Closed'"
       ]
     });
-    void this._forEachServer(async (server, serverName) => {
+    void this._forEachServer(async (server) => {
       void server.sendMessage({
         author,
         recipient,
         payload
-      }).catch((err) => {
-        if (err instanceof import_protocols5.RateLimitExceededError) {
-          (0, import_log5.log)("WSS rate limit exceeded", {
-            err
-          }, {
-            F: __dxlog_file5,
-            L: 138,
-            S: this,
-            C: (f, a) => f(...a)
-          });
-        } else {
-          (0, import_log5.log)(`error sending to ${serverName}`, {
-            err
-          }, {
-            F: __dxlog_file5,
-            L: 140,
-            S: this,
-            C: (f, a) => f(...a)
-          });
-          void this.checkServerFailure(serverName);
-        }
-      });
-    });
-  }
-  async checkServerFailure(serverName) {
-    const failureCount = this.failureCount.get(serverName) ?? 0;
-    if (failureCount > MAX_SERVER_FAILURES) {
-      import_log5.log.warn(`Too many failures sending to ${serverName} (${failureCount} > ${MAX_SERVER_FAILURES}), restarting`, void 0, {
+      }).catch((err) => (0, import_log5.log)(err, void 0, {
         F: __dxlog_file5,
-        L: 151,
+        L: 119,
         S: this,
         C: (f, a) => f(...a)
-      });
-      await this.restartServer(serverName);
-      this.failureCount.set(serverName, 0);
-      return;
-    }
-    this.failureCount.set(serverName, (this.failureCount.get(serverName) ?? 0) + 1);
+      }));
+    });
   }
   async subscribeMessages(peerId) {
     (0, import_log5.log)(`Subscribed for message stream peerId=${peerId}`, void 0, {
       F: __dxlog_file5,
-      L: 160,
+      L: 124,
       S: this,
       C: (f, a) => f(...a)
     });
     (0, import_invariant5.invariant)(this._opened, "Closed", {
       F: __dxlog_file5,
-      L: 161,
+      L: 125,
       S: this,
       A: [
         "this._opened",
@@ -1624,13 +1610,13 @@ var WebsocketSignalManager = class {
   async unsubscribeMessages(peerId) {
     (0, import_log5.log)(`Subscribed for message stream peerId=${peerId}`, void 0, {
       F: __dxlog_file5,
-      L: 167,
+      L: 131,
       S: this,
       C: (f, a) => f(...a)
     });
     (0, import_invariant5.invariant)(this._opened, "Closed", {
       F: __dxlog_file5,
-      L: 168,
+      L: 132,
       S: this,
       A: [
         "this._opened",
@@ -1643,31 +1629,25 @@ var WebsocketSignalManager = class {
     this._ctx = new import_context4.Context({
       onError: (err) => import_log5.log.catch(err, void 0, {
         F: __dxlog_file5,
-        L: 175,
+        L: 139,
         S: this,
         C: (f, a) => f(...a)
       })
     });
   }
   async _forEachServer(fn) {
-    return Promise.all(Array.from(this._servers.entries()).map(([serverName, server]) => fn(server, serverName)));
+    return Promise.all(Array.from(this._servers.values()).map(fn));
   }
 };
 _ts_decorate([
   import_async5.synchronized
 ], WebsocketSignalManager.prototype, "open", null);
-_ts_decorate([
-  import_async5.synchronized
-], WebsocketSignalManager.prototype, "close", null);
 _ts_decorate([
   import_async5.synchronized
 ], WebsocketSignalManager.prototype, "join", null);
 _ts_decorate([
   import_async5.synchronized
 ], WebsocketSignalManager.prototype, "leave", null);
-_ts_decorate([
-  import_async5.synchronized
-], WebsocketSignalManager.prototype, "checkServerFailure", null);
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   MemorySignalManager,