@dxos/teleport 0.3.4 → 0.3.5-main.1ae5af4

package/dist/lib/browser/{chunk-J3RZ6SXA.mjs → chunk-PWWDYMDW.mjs}

@@ -4,7 +4,7 @@ import "@dxos/node-std/globals";
 import { pipeline } from "@dxos/node-std/stream";
 import { invariant as invariant4 } from "@dxos/invariant";
 import { PublicKey as PublicKey2 } from "@dxos/keys";
-import { log as log4 } from "@dxos/log";
+import { log as log5 } from "@dxos/log";
 
 // packages/core/mesh/teleport/src/teleport.ts
 import { asyncTimeout, scheduleTaskInterval as scheduleTaskInterval2, runInContextAsync, synchronized, scheduleTask } from "@dxos/async";
@@ -12,8 +12,8 @@ import { Context as Context2 } from "@dxos/context";
 import { failUndefined as failUndefined2 } from "@dxos/debug";
 import { invariant as invariant3 } from "@dxos/invariant";
 import { PublicKey } from "@dxos/keys";
-import { log as log3 } from "@dxos/log";
-import { schema as schema2, RpcClosedError, TimeoutError } from "@dxos/protocols";
+import { log as log4 } from "@dxos/log";
+import { schema as schema2, RpcClosedError, TimeoutError as TimeoutError2 } from "@dxos/protocols";
 import { createProtoRpcPeer } from "@dxos/rpc";
 import { Callback } from "@dxos/util";
 
@@ -21,7 +21,8 @@ import { Callback } from "@dxos/util";
 import { Duplex } from "@dxos/node-std/stream";
 import { Event } from "@dxos/async";
 import { invariant } from "@dxos/invariant";
-var __dxlog_file = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/muxing/framer.ts";
+import { log } from "@dxos/log";
+var __dxlog_file = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/muxing/framer.ts";
 var FRAME_LENGTH_SIZE = 2;
 var Framer = class {
   constructor() {
@@ -43,7 +44,7 @@ var Framer = class {
       write: (chunk, encoding, callback) => {
         invariant(!this._subscribeCb, "Internal Framer bug. Concurrent writes detected.", {
           F: __dxlog_file,
-          L: 39,
+          L: 40,
           S: this,
           A: [
             "!this._subscribeCb",
@@ -87,7 +88,7 @@ var Framer = class {
       subscribe: (callback) => {
         invariant(!this._messageCb, "Rpc port already has a message listener.", {
           F: __dxlog_file,
-          L: 78,
+          L: 79,
           S: this,
           A: [
             "!this._messageCb",
@@ -141,6 +142,22 @@ var Framer = class {
     }
   }
   destroy() {
+    if (this._stream.readableLength > 0) {
+      log.warn("framer destroyed while there are still read bytes in the buffer.", void 0, {
+        F: __dxlog_file,
+        L: 140,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+    }
+    if (this._stream.writableLength > 0) {
+      log.warn("framer destroyed while there are still write bytes in the buffer.", void 0, {
+        F: __dxlog_file,
+        L: 143,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+    }
     this._stream.destroy();
   }
 };
@@ -172,14 +189,14 @@ import { scheduleTaskInterval, Event as Event3, Trigger } from "@dxos/async";
 import { Context } from "@dxos/context";
 import { failUndefined } from "@dxos/debug";
 import { invariant as invariant2 } from "@dxos/invariant";
-import { log as log2 } from "@dxos/log";
-import { schema } from "@dxos/protocols";
+import { log as log3 } from "@dxos/log";
+import { schema, TimeoutError } from "@dxos/protocols";
 
 // packages/core/mesh/teleport/src/muxing/balancer.ts
 import * as varint from "varint";
 import { Event as Event2 } from "@dxos/async";
-import { log } from "@dxos/log";
-var __dxlog_file2 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/muxing/balancer.ts";
+import { log as log2 } from "@dxos/log";
+var __dxlog_file2 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/muxing/balancer.ts";
 var MAX_CHUNK_SIZE = 8192;
 var Balancer = class {
   constructor(_sysChannelId) {
@@ -231,7 +248,7 @@ var Balancer = class {
       });
     });
     if (noCalls) {
-      this._sendChunk().catch((err) => log.catch(err, void 0, {
+      this._sendChunk().catch((err) => log2.catch(err, void 0, {
         F: __dxlog_file2,
         L: 100,
         S: this,
@@ -240,6 +257,14 @@ var Balancer = class {
     }
   }
   destroy() {
+    if (this._sendBuffers.size !== 0) {
+      log2.warn("destroying balancer with pending calls", void 0, {
+        F: __dxlog_file2,
+        L: 106,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+    }
     this._sendBuffers.clear();
     this._framer.destroy();
   }
@@ -336,11 +361,12 @@ var decodeChunk = (data, withLength) => {
 };
 
 // packages/core/mesh/teleport/src/muxing/muxer.ts
-var __dxlog_file3 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/muxing/muxer.ts";
+var __dxlog_file3 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/muxing/muxer.ts";
 var Command = schema.getCodecForType("dxos.mesh.muxer.Command");
 var STATS_INTERVAL = 1e3;
 var MAX_SAFE_FRAME_SIZE = 1e6;
 var SYSTEM_CHANNEL_ID = 0;
+var GRACEFUL_CLOSE_TIMEOUT = 3e3;
 var Muxer = class {
   constructor() {
     this._balancer = new Balancer(SYSTEM_CHANNEL_ID);
@@ -350,9 +376,10 @@ var Muxer = class {
     this._nextId = 1;
     this._destroyed = false;
     this._destroying = false;
+    this._closing = false;
     this._lastStats = void 0;
     this._lastChannelStats = /* @__PURE__ */ new Map();
-    this.close = new Event3();
+    this.afterClosed = new Event3();
     this.statsUpdated = new Event3();
     this.stream = this._balancer.stream;
     this._balancer.incomingData.on(async (msg) => {
@@ -373,7 +400,7 @@ var Muxer = class {
     });
     invariant2(!channel.push, `Channel already open: ${tag}`, {
       F: __dxlog_file3,
-      L: 136,
+      L: 138,
       S: this,
       A: [
         "!channel.push",
@@ -421,7 +448,7 @@ var Muxer = class {
     });
     invariant2(!channel.push, `Channel already open: ${tag}`, {
       F: __dxlog_file3,
-      L: 188,
+      L: 190,
       S: this,
       A: [
         "!channel.push",
@@ -445,7 +472,7 @@ var Muxer = class {
       subscribe: (cb) => {
         invariant2(!callback, "Only one subscriber is allowed", {
           F: __dxlog_file3,
-          L: 210,
+          L: 212,
           S: this,
           A: [
             "!callback",
@@ -473,61 +500,153 @@ var Muxer = class {
     }
     return port;
   }
-  /**
-  * Force-close with optional error.
-  */
-  async destroy(err) {
+  // initiate graceful close
+  async close(err) {
     if (this._destroying) {
+      log3("already destroying, ignoring graceful close request", void 0, {
+        F: __dxlog_file3,
+        L: 245,
+        S: this,
+        C: (f, a) => f(...a)
+      });
       return;
     }
-    this._destroying = true;
-    this._sendCommand({
-      destroy: {
+    if (this._closing) {
+      log3("already closing, ignoring graceful close request", void 0, {
+        F: __dxlog_file3,
+        L: 249,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+      return;
+    }
+    this._closing = true;
+    await this._sendCommand({
+      close: {
         error: err?.message
       }
-    }, SYSTEM_CHANNEL_ID).then(() => {
-      this._dispose();
-    }).catch((err2) => {
-      this._dispose(err2);
+    }, SYSTEM_CHANNEL_ID).catch(async (err2) => {
+      log3("error sending close command", {
+        err: err2
+      }, {
+        F: __dxlog_file3,
+        L: 263,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+      await this.dispose(err2);
     });
+    await Promise.race([
+      new Promise((_resolve, reject) => {
+        setTimeout(() => {
+          reject(new TimeoutError("gracefully closing muxer"));
+        }, GRACEFUL_CLOSE_TIMEOUT);
+      }),
+      (async () => {
+        await this.dispose(err);
+      })()
+    ]);
+  }
+  // force close without confirmation
+  async destroy(err) {
+    if (this._destroying) {
+      log3("already destroying, ignoring destroy request", void 0, {
+        F: __dxlog_file3,
+        L: 284,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+      return;
+    }
+    this._destroying = true;
     void this._ctx.dispose();
+    if (this._closing) {
+      log3("destroy cancelling graceful close", void 0, {
+        F: __dxlog_file3,
+        L: 290,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+      this._closing = false;
+    } else {
+      await this._sendCommand({
+        close: {
+          error: err?.message
+        }
+      }, SYSTEM_CHANNEL_ID).catch(async (err2) => {
+        log3("error sending courtesy close command", {
+          err: err2
+        }, {
+          F: __dxlog_file3,
+          L: 303,
+          S: this,
+          C: (f, a) => f(...a)
+        });
+      });
+    }
+    this.dispose(err).catch((err2) => {
+      log3("error disposing after destroy", {
+        err: err2
+      }, {
+        F: __dxlog_file3,
+        L: 308,
+        S: this,
+        C: (f, a) => f(...a)
+      });
+    });
   }
-  _dispose(err) {
+  // complete the termination, graceful or otherwise
+  async dispose(err) {
     if (this._destroyed) {
+      log3("already destroyed, ignoring dispose request", void 0, {
+        F: __dxlog_file3,
+        L: 316,
+        S: this,
+        C: (f, a) => f(...a)
+      });
       return;
     }
-    this._destroyed = true;
-    this._balancer.destroy();
+    void this._ctx.dispose();
+    await this._balancer.destroy();
     for (const channel of this._channelsByTag.values()) {
       channel.destroy?.(err);
     }
-    this.close.emit(err);
+    this._destroyed = true;
+    this.afterClosed.emit(err);
     this._channelsByLocalId.clear();
     this._channelsByTag.clear();
   }
   async _handleCommand(cmd) {
-    log2("Received command", {
-      cmd
-    }, {
-      F: __dxlog_file3,
-      L: 286,
-      S: this,
-      C: (f, a) => f(...a)
-    });
-    if (this._destroyed || this._destroying) {
-      if (cmd.destroy) {
-        return;
-      }
-      log2.warn("Received command after destroy", {
+    if (this._destroyed) {
+      log3.warn("Received command after destroy", {
         cmd
       }, {
         F: __dxlog_file3,
-        L: 293,
+        L: 338,
         S: this,
         C: (f, a) => f(...a)
       });
       return;
     }
+    if (cmd.close) {
+      if (!this._closing) {
+        log3("received peer close, initiating my own graceful close", void 0, {
+          F: __dxlog_file3,
+          L: 344,
+          S: this,
+          C: (f, a) => f(...a)
+        });
+        await this.close();
+      } else {
+        log3("received close from peer, already closing", void 0, {
+          F: __dxlog_file3,
+          L: 347,
+          S: this,
+          C: (f, a) => f(...a)
+        });
+      }
+      return;
+    }
     if (cmd.openChannel) {
       const channel = this._getOrCreateStream({
         tag: cmd.openChannel.tag,
@@ -546,19 +665,17 @@ var Muxer = class {
     } else if (cmd.data) {
       const stream = this._channelsByLocalId.get(cmd.data.channelId) ?? failUndefined();
       if (!stream.push) {
-        log2.warn("Received data for channel before it was opened", {
+        log3.warn("Received data for channel before it was opened", {
           tag: stream.tag
         }, {
           F: __dxlog_file3,
-          L: 320,
+          L: 376,
           S: this,
           C: (f, a) => f(...a)
         });
         return;
       }
       stream.push(cmd.data.data);
-    } else if (cmd.destroy) {
-      this._dispose();
     }
   }
   async _sendCommand(cmd, channelId = -1) {
@@ -594,12 +711,12 @@ var Muxer = class {
   }
   async _sendData(channel, data) {
     if (data.length > MAX_SAFE_FRAME_SIZE) {
-      log2.warn("frame size exceeds maximum safe value", {
+      log3.warn("frame size exceeds maximum safe value", {
         size: data.length,
         threshold: MAX_SAFE_FRAME_SIZE
       }, {
         F: __dxlog_file3,
-        L: 365,
+        L: 419,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -676,13 +793,13 @@ function _ts_decorate(decorators, target, key, desc) {
         r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
   return c > 3 && r && Object.defineProperty(target, key, r), r;
 }
-var __dxlog_file4 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/teleport.ts";
+var __dxlog_file4 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/teleport.ts";
 var Teleport = class {
   constructor({ initiator, localPeerId, remotePeerId }) {
     this._ctx = new Context2({
       onError: (err) => {
         void this.destroy(err).catch(() => {
-          log3.error("Error during destroy", err, {
+          log4.error("Error during destroy", err, {
             F: __dxlog_file4,
             L: 44,
             S: this,
@@ -699,13 +816,13 @@ var Teleport = class {
         if (this._destroying || this._aborting) {
           return;
         }
-        log3("destroy teleport due to onTimeout in ControlExtension", void 0, {
+        log4("destroy teleport due to onTimeout in ControlExtension", void 0, {
           F: __dxlog_file4,
           L: 59,
           S: this,
           C: (f, a) => f(...a)
         });
-        this.destroy(new TimeoutError("control extension")).catch((err) => log3.catch(err, void 0, {
+        this.destroy(new TimeoutError2("control extension")).catch((err) => log4.catch(err, void 0, {
           F: __dxlog_file4,
           L: 60,
           S: this,
@@ -749,7 +866,7 @@ var Teleport = class {
     this.localPeerId = localPeerId;
     this.remotePeerId = remotePeerId;
     this._control.onExtensionRegistered.set(async (name) => {
-      log3("remote extension", {
+      log4("remote extension", {
         name
       }, {
         F: __dxlog_file4,
@@ -778,7 +895,7 @@ var Teleport = class {
     {
       this._muxer.stream.on("close", async () => {
         if (this._destroying || this._aborting) {
-          log3("destroy teleport due to muxer stream close, skipping due to already destroying/aborting", void 0, {
+          log4("destroy teleport due to muxer stream close, skipping due to already destroying/aborting", void 0, {
             F: __dxlog_file4,
             L: 97,
             S: this,
@@ -793,7 +910,7 @@ var Teleport = class {
       });
     }
     this._muxer.statsUpdated.on((stats) => {
-      log3.trace("dxos.mesh.teleport.stats", {
+      log4.trace("dxos.mesh.teleport.stats", {
         localPeerId,
         remotePeerId,
         bytesSent: stats.bytesSent,
@@ -839,7 +956,7 @@ var Teleport = class {
       try {
         await extension.onAbort(err);
       } catch (err2) {
-        log3.catch(err2, void 0, {
+        log4.catch(err2, void 0, {
           F: __dxlog_file4,
           L: 163,
           S: this,
@@ -862,7 +979,7 @@ var Teleport = class {
       try {
         await extension.onClose(err);
       } catch (err2) {
-        log3.catch(err2, void 0, {
+        log4.catch(err2, void 0, {
           F: __dxlog_file4,
           L: 187,
           S: this,
@@ -870,13 +987,13 @@ var Teleport = class {
         });
       }
     }
-    await this._muxer.destroy(err);
+    await this._muxer.close();
   }
   addExtension(name, extension) {
     if (!this._open) {
       throw new Error("Not open");
     }
-    log3("addExtension", {
+    log4("addExtension", {
       name
     }, {
       F: __dxlog_file4,
@@ -923,7 +1040,7 @@ var Teleport = class {
     this._extensions.set(extensionName, extension);
   }
   async _openExtension(extensionName) {
-    log3("open extension", {
+    log4("open extension", {
       extensionName
     }, {
       F: __dxlog_file4,
@@ -967,7 +1084,7 @@ var Teleport = class {
       }
     };
     await extension.onOpen(context);
-    log3("extension opened", {
+    log4("extension opened", {
       extensionName
     }, {
       F: __dxlog_file4,
@@ -1040,7 +1157,7 @@ var ControlExtension = class {
 };
 
 // packages/core/mesh/teleport/src/testing/test-builder.ts
-var __dxlog_file5 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/testing/test-builder.ts";
+var __dxlog_file5 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/testing/test-builder.ts";
 var TestBuilder = class {
   constructor() {
     this._peers = /* @__PURE__ */ new Set();
@@ -1208,7 +1325,7 @@ var TestPeer = class {
 var pipeStreams = (stream1, stream2) => {
   pipeline(stream1, stream2, (err) => {
     if (err && err.code !== "ERR_STREAM_PREMATURE_CLOSE") {
-      log4.catch(err, void 0, {
+      log5.catch(err, void 0, {
         F: __dxlog_file5,
         L: 106,
         S: void 0,
@@ -1218,7 +1335,7 @@ var pipeStreams = (stream1, stream2) => {
   });
   pipeline(stream2, stream1, (err) => {
     if (err && err.code !== "ERR_STREAM_PREMATURE_CLOSE") {
-      log4.catch(err, void 0, {
+      log5.catch(err, void 0, {
         F: __dxlog_file5,
         L: 111,
         S: void 0,
@@ -1243,10 +1360,10 @@ var TestConnection = class {
 // packages/core/mesh/teleport/src/testing/test-extension.ts
 import { asyncTimeout as asyncTimeout2, Trigger as Trigger2 } from "@dxos/async";
 import { invariant as invariant5 } from "@dxos/invariant";
-import { log as log5 } from "@dxos/log";
+import { log as log6 } from "@dxos/log";
 import { schema as schema3 } from "@dxos/protocols";
 import { createProtoRpcPeer as createProtoRpcPeer2 } from "@dxos/rpc";
-var __dxlog_file6 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/testing/test-extension.ts";
+var __dxlog_file6 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/testing/test-extension.ts";
 var TestExtension = class {
   constructor(callbacks = {}) {
     this.callbacks = callbacks;
@@ -1258,7 +1375,7 @@ var TestExtension = class {
     return this.extensionContext?.remotePeerId;
   }
   async onOpen(context) {
-    log5("onOpen", {
+    log6("onOpen", {
       localPeerId: context.localPeerId,
       remotePeerId: context.remotePeerId
     }, {
@@ -1296,7 +1413,7 @@ var TestExtension = class {
     this.open.wake();
   }
   async onClose(err) {
-    log5("onClose", {
+    log6("onClose", {
       err
     }, {
       F: __dxlog_file6,
@@ -1309,7 +1426,7 @@ var TestExtension = class {
     await this._rpc?.close();
   }
   async onAbort(err) {
-    log5("onAbort", {
+    log6("onAbort", {
       err
     }, {
       F: __dxlog_file6,
@@ -1350,10 +1467,10 @@ var TestExtension = class {
 import { randomBytes } from "@dxos/node-std/crypto";
 import { Trigger as Trigger3 } from "@dxos/async";
 import { invariant as invariant6 } from "@dxos/invariant";
-import { log as log6 } from "@dxos/log";
+import { log as log7 } from "@dxos/log";
 import { schema as schema4 } from "@dxos/protocols";
 import { createProtoRpcPeer as createProtoRpcPeer3 } from "@dxos/rpc";
-var __dxlog_file7 = "/mnt/ramdisk/work/packages/core/mesh/teleport/src/testing/test-extension-with-streams.ts";
+var __dxlog_file7 = "/home/runner/work/dxos/dxos/packages/core/mesh/teleport/src/testing/test-extension-with-streams.ts";
 var TestExtensionWithStreams = class {
   constructor(callbacks = {}) {
     this.callbacks = callbacks;
@@ -1415,7 +1532,7 @@ var TestExtensionWithStreams = class {
     });
     streamEntry.reportingTimer = setInterval(() => {
       const { bytesSent, bytesReceived, sendErrors, receiveErrors } = streamEntry;
-      log6.trace("dxos.test.stream-stats", {
+      log7.trace("dxos.test.stream-stats", {
         streamTag,
         bytesSent,
         bytesReceived,
@@ -1456,7 +1573,7 @@ var TestExtensionWithStreams = class {
     };
   }
   async onOpen(context) {
-    log6("onOpen", {
+    log7("onOpen", {
       localPeerId: context.localPeerId,
       remotePeerId: context.remotePeerId
     }, {
@@ -1506,7 +1623,7 @@ var TestExtensionWithStreams = class {
     this.open.wake();
   }
   async onClose(err) {
-    log6("onClose", {
+    log7("onClose", {
       err
     }, {
       F: __dxlog_file7,
@@ -1517,7 +1634,7 @@ var TestExtensionWithStreams = class {
     await this.callbacks.onClose?.();
     this.closed.wake();
     for (const [streamTag, stream] of Object.entries(this._streams)) {
-      log6("closing stream", {
+      log7("closing stream", {
         streamTag
       }, {
         F: __dxlog_file7,
@@ -1531,7 +1648,7 @@ var TestExtensionWithStreams = class {
     await this._rpc?.close();
   }
   async onAbort(err) {
-    log6("onAbort", {
+    log7("onAbort", {
       err
     }, {
       F: __dxlog_file7,
@@ -1618,4 +1735,4 @@ export {
   TestExtension,
   TestExtensionWithStreams
 };
-//# sourceMappingURL=chunk-J3RZ6SXA.mjs.map
+//# sourceMappingURL=chunk-PWWDYMDW.mjs.map