@freqhole/midden 0.2.1 → 0.2.3

package/midden.d.ts

@@ -18,6 +18,15 @@ export type ReadableStreamType = "bytes";
  * the send and recv halves use RefCell<Option<...>> so that async read
  * and write operations can proceed concurrently (safe because WASM is
  * single-threaded).
+ *
+ * also holds the parent `Connection` handle, purely to keep the QUIC
+ * connection alive for as long as this stream is alive: iroh/quinn tears
+ * a connection down once its last `Connection` handle is dropped, and
+ * neither `accept()` nor `open_bi()` keep any other handle around after
+ * handing a `BiStream` to JS. without this field, the connection was
+ * dropped (and the peer's in-flight read failed with "connection lost")
+ * the moment `accept()`/`open_bi()` returned - often before a response
+ * written moments later on the same stream even finished flushing.
  */
 export class BiStream {
     private constructor();
@@ -233,6 +242,18 @@ export class MiddenNode {
      *
      * the caller should check `stream.alpn()` to route the connection
      * to the appropriate handler.
+     *
+     * a single incoming attempt failing during the TLS handshake (e.g. the
+     * peer aborts mid-handshake - normal during connection-path racing, or
+     * a peer that redials before noticing an earlier attempt is still
+     * live) does not end this call: it's logged and the loop moves on to
+     * the next queued incoming connection. propagating that failure to the
+     * caller instead would surface as a JS-level error on every accept()
+     * call, forcing the caller through a full error-handling/backoff cycle
+     * (see `IrohNetworkAdapter`'s accept loop) before the next, perfectly
+     * good, already-queued connection is even looked at - under a burst of
+     * aborted handshakes this can visibly stall new connections from ever
+     * completing.
      */
     accept(): Promise<any>;
     /**

package/midden_bg.js

@@ -8,6 +8,15 @@
  * the send and recv halves use RefCell<Option<...>> so that async read
  * and write operations can proceed concurrently (safe because WASM is
  * single-threaded).
+ *
+ * also holds the parent `Connection` handle, purely to keep the QUIC
+ * connection alive for as long as this stream is alive: iroh/quinn tears
+ * a connection down once its last `Connection` handle is dropped, and
+ * neither `accept()` nor `open_bi()` keep any other handle around after
+ * handing a `BiStream` to JS. without this field, the connection was
+ * dropped (and the peer's in-flight read failed with "connection lost")
+ * the moment `accept()`/`open_bi()` returned - often before a response
+ * written moments later on the same stream even finished flushing.
  */
 export class BiStream {
     static __wrap(ptr) {
@@ -514,6 +523,18 @@ export class MiddenNode {
      *
      * the caller should check `stream.alpn()` to route the connection
      * to the appropriate handler.
+     *
+     * a single incoming attempt failing during the TLS handshake (e.g. the
+     * peer aborts mid-handshake - normal during connection-path racing, or
+     * a peer that redials before noticing an earlier attempt is still
+     * live) does not end this call: it's logged and the loop moves on to
+     * the next queued incoming connection. propagating that failure to the
+     * caller instead would surface as a JS-level error on every accept()
+     * call, forcing the caller through a full error-handling/backoff cycle
+     * (see `IrohNetworkAdapter`'s accept loop) before the next, perfectly
+     * good, already-queued connection is even looked at - under a burst of
+     * aborted handshakes this can visibly stall new connections from ever
+     * completing.
      * @returns {Promise<any>}
      */
     accept() {
@@ -2085,37 +2106,37 @@ export function __wbg_write_e557b5312ec23477() { return handleError(function (ar
     return ret;
 }, arguments); }
 export function __wbindgen_cast_0000000000000001(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [Externref], shim_idx: 3530, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [Externref], shim_idx: 3531, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke___wasm_bindgen_eec8db4f48ed3396___JsValue______true_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000002(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [Externref], shim_idx: 5576, ret: Result(Unit), inner_ret: Some(Result(Unit)) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [Externref], shim_idx: 5577, ret: Result(Unit), inner_ret: Some(Result(Unit)) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke___wasm_bindgen_eec8db4f48ed3396___JsValue__core_9b3796e30d99ddb7___result__Result_____wasm_bindgen_eec8db4f48ed3396___JsError___true_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000003(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [NamedExternref("CloseEvent")], shim_idx: 2808, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [NamedExternref("CloseEvent")], shim_idx: 2809, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke___web_sys_a9844cda14584e26___features__gen_CloseEvent__CloseEvent______true_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000004(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [NamedExternref("MessageEvent")], shim_idx: 4161, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [NamedExternref("MessageEvent")], shim_idx: 4162, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke___web_sys_a9844cda14584e26___features__gen_MessageEvent__MessageEvent______true_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000005(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 3449, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 3450, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke_______true_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000006(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 3703, ret: Unit, inner_ret: Some(Unit) }, mutable: false }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 3704, ret: Unit, inner_ret: Some(Unit) }, mutable: false }) -> Externref`.
     const ret = makeClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke_______true__1_);
     return ret;
 }
 export function __wbindgen_cast_0000000000000007(arg0, arg1) {
-    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 5540, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
+    // Cast intrinsic for `Closure(Closure { owned: true, function: Function { arguments: [], shim_idx: 5541, ret: Unit, inner_ret: Some(Unit) }, mutable: true }) -> Externref`.
     const ret = makeMutClosure(arg0, arg1, wasm_bindgen_eec8db4f48ed3396___convert__closures_____invoke_______true__2_);
     return ret;
 }

package/package.json

@@ -2,7 +2,7 @@
   "name": "@freqhole/midden",
   "type": "module",
   "description": "WASM bindings for freqhole P2P client (iroh-based)",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/freqhole/tomb"