wire-mesh-core 4.1.4 → 4.2.0

package/dist/domain/mesh-session.mjs

@@ -6,6 +6,7 @@ import { a as OWN_VERSION, i as CORE_VERSION_GOSSIP_KEY, n as TOPOLOGY_PEERS_GOS
 import { signPeerAdvert, verifyPeerAdvert } from "./peer-advert.mjs";
 import { createAsyncQueue } from "./async-queue.mjs";
 import { ChannelExhaustedError } from "./channel-exhausted-error.mjs";
+import { deliverContained } from "./fault-report.mjs";
 import { localHandshake, negotiate } from "./handshake.mjs";
 import { isDataFrame } from "./hub-mailbox.mjs";
 import { createPingRoundTrips } from "./ping-round-trips.mjs";
@@ -279,17 +280,9 @@ async function establishChannelFrom(identity, { pending, remote, peer, firstSend
 function createRelayChannels(identity, sendHello, options = {}) {
 	if (options.firstSendCounter !== void 0) assertFirstSendCounter(options.firstSendCounter);
 	const channels = /* @__PURE__ */ new Map();
-	/** Tells the owner of a failure that is local to this side. The callback is the owner's code: its own failure, thrown or as a rejected promise it returns, must not end the session or go unobserved, so it is reported once and swallowed. */
+	/** Tells the owner of a failure that is local to this side. The callback is the owner's code: its own failure, thrown or as a rejected promise it returns, must not end the session or go unobserved, so it goes through the shared containment in fault-report.ts. */
 	function reportFault(error) {
-		const report = (faultError) => {
-			console.error("the local fault callback failed", faultError);
-		};
-		try {
-			const result = options.onFault?.(error);
-			if (typeof result === "object" && result !== null && "then" in result && typeof result.then === "function") Promise.resolve(result).catch(report);
-		} catch (faultError) {
-			report(faultError);
-		}
+		deliverContained(async () => options.onFault?.(error), 0);
 	}
 	/** Abandons a handshake that failed: the entry goes if it is still the one held for the peer, and whoever awaits it is told. */
 	function abandon(entry, error) {
@@ -502,7 +495,7 @@ function createSessionCore(identity, clock, reconnect, dial, onPeerAdvert, addre
 	}
 	/** The "not connected" guard every send* method on the public session needs, deduplicated into one place rather than repeated inline at each call site: throws unless there is a live connection AND the session's own state machine agrees it is "connected" (transmit's own connection===null check alone doesn't cover the latter). Returns the connection itself, narrowed non-null, for a caller that needs to send directly on it (sendPing/sendPingMeasureRtt); a caller that only needs the throwing side effect (sendManageRequest and friends, which go through transmit instead) calls this and discards the result. */
 	function requireConnectedLink() {
-		if (connection === null || state.status !== "connected") throw new Error("not connected");
+		if (feedCancelled || connection === null || state.status !== "connected") throw new Error("not connected");
 		return connection;
 	}
 	/** Sends a frame. Through a relay it is sealed for toDevice on their secure channel first (spec/secure-channel.cddl) and only then wrapped as relay-data, so the hub carries ciphertext. This is the single choke point every outbound manage-request and manage-response passes through, so a consumer of sendManageRequest or respond never needs its own relay logic. */
@@ -833,7 +826,10 @@ function createSessionCore(identity, clock, reconnect, dial, onPeerAdvert, addre
 			},
 			async sendManageRequest(command, scope, targetDevice, token, timeoutMs) {
 				requireConnectedLink();
-				if (targetDevice !== void 0) await ensureRelayPairing(targetDevice);
+				if (targetDevice !== void 0) {
+					await ensureRelayPairing(targetDevice);
+					requireConnectedLink();
+				}
 				const frame = buildManageRequest(command, scope, token);
 				const requestId = frame["request-id"];
 				const outcome = new Promise((resolve, reject) => {
@@ -843,6 +839,7 @@ function createSessionCore(identity, clock, reconnect, dial, onPeerAdvert, addre
 						expectedFrom: targetDevice
 					});
 				});
+				outcome.catch(() => void 0);
 				recordFrame({
 					direction: "sent",
 					frame
@@ -852,7 +849,6 @@ function createSessionCore(identity, clock, reconnect, dial, onPeerAdvert, addre
 						await transmit(frame, targetDevice !== void 0, targetDevice);
 					} catch (error) {
 						pendingManageRequests.delete(requestId);
-						outcome.catch(() => void 0);
 						throw error;
 					}
 					emit();

package/dist/domain/ping-round-trips.cjs

@@ -9,8 +9,8 @@ var PingTimeoutError = class extends Error {
 };
 function createPingRoundTrips() {
 	const pending = [];
-	function cancel(sentAt) {
-		const index = pending.findIndex((entry) => entry.sentAt === sentAt);
+	function cancel(registration) {
+		const index = pending.indexOf(registration);
 		if (index === -1) return false;
 		pending.splice(index, 1);
 		return true;
@@ -24,20 +24,33 @@ function createPingRoundTrips() {
 			for (const entry of pending.splice(0)) entry.reject(new Error(reason));
 		},
 		async sendAndAwait(sentAt, sendPing, timeoutMs) {
+			let registration;
 			const rtt = new Promise((resolve, reject) => {
-				pending.push({
+				registration = {
 					sentAt,
 					resolve,
 					reject
-				});
+				};
+				pending.push(registration);
 			});
-			await sendPing();
+			if (registration === void 0) throw new Error("the round trip was not registered");
+			const registered = registration;
+			rtt.catch(() => void 0);
+			try {
+				await sendPing();
+			} catch (error) {
+				cancel(registered);
+				throw error;
+			}
 			if (timeoutMs === void 0) return rtt;
+			let timer;
 			return Promise.race([rtt, new Promise((_resolve, reject) => {
-				setTimeout(() => {
-					if (cancel(sentAt)) reject(new PingTimeoutError());
+				timer = setTimeout(() => {
+					if (cancel(registered)) reject(new PingTimeoutError());
 				}, timeoutMs);
-			})]);
+			})]).finally(() => {
+				clearTimeout(timer);
+			});
 		}
 	};
 }

package/dist/domain/ping-round-trips.mjs

@@ -8,8 +8,8 @@ var PingTimeoutError = class extends Error {
 };
 function createPingRoundTrips() {
 	const pending = [];
-	function cancel(sentAt) {
-		const index = pending.findIndex((entry) => entry.sentAt === sentAt);
+	function cancel(registration) {
+		const index = pending.indexOf(registration);
 		if (index === -1) return false;
 		pending.splice(index, 1);
 		return true;
@@ -23,20 +23,33 @@ function createPingRoundTrips() {
 			for (const entry of pending.splice(0)) entry.reject(new Error(reason));
 		},
 		async sendAndAwait(sentAt, sendPing, timeoutMs) {
+			let registration;
 			const rtt = new Promise((resolve, reject) => {
-				pending.push({
+				registration = {
 					sentAt,
 					resolve,
 					reject
-				});
+				};
+				pending.push(registration);
 			});
-			await sendPing();
+			if (registration === void 0) throw new Error("the round trip was not registered");
+			const registered = registration;
+			rtt.catch(() => void 0);
+			try {
+				await sendPing();
+			} catch (error) {
+				cancel(registered);
+				throw error;
+			}
 			if (timeoutMs === void 0) return rtt;
+			let timer;
 			return Promise.race([rtt, new Promise((_resolve, reject) => {
-				setTimeout(() => {
-					if (cancel(sentAt)) reject(new PingTimeoutError());
+				timer = setTimeout(() => {
+					if (cancel(registered)) reject(new PingTimeoutError());
 				}, timeoutMs);
-			})]);
+			})]).finally(() => {
+				clearTimeout(timer);
+			});
 		}
 	};
 }

package/dist/timers-DTJNuBa8.cjs

@@ -0,0 +1,12 @@
+//#region src/domain/timers.ts
+/** Stops a pending timer from keeping the process alive, where the runtime has that notion. Node's `setTimeout` returns an object with `unref()`; a browser's or a Worker's returns a number, which has nothing to unref and must not be called as if it did. */
+function unrefIfSupported(handle) {
+	if (typeof handle === "object" && handle !== null && "unref" in handle && typeof handle.unref === "function") Reflect.apply(handle.unref, handle, []);
+}
+//#endregion
+Object.defineProperty(exports, "unrefIfSupported", {
+	enumerable: true,
+	get: function() {
+		return unrefIfSupported;
+	}
+});

package/dist/timers-Q1_kDTyY.mjs

@@ -0,0 +1,7 @@
+//#region src/domain/timers.ts
+/** Stops a pending timer from keeping the process alive, where the runtime has that notion. Node's `setTimeout` returns an object with `unref()`; a browser's or a Worker's returns a number, which has nothing to unref and must not be called as if it did. */
+function unrefIfSupported(handle) {
+	if (typeof handle === "object" && handle !== null && "unref" in handle && typeof handle.unref === "function") Reflect.apply(handle.unref, handle, []);
+}
+//#endregion
+export { unrefIfSupported as t };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wire-mesh-core",
-  "version": "4.1.4",
+  "version": "4.2.0",
   "dependencies": {
     "cbor2": "2.3.0",
     "cddl.js": "2.0.0",
@@ -90,6 +90,10 @@
       "import": "./dist/domain/channel-exhausted-error.mjs",
       "require": "./dist/domain/channel-exhausted-error.cjs"
     },
+    "./domain/consume-incoming": {
+      "import": "./dist/domain/consume-incoming.mjs",
+      "require": "./dist/domain/consume-incoming.cjs"
+    },
     "./domain/coordinator-election": {
       "import": "./dist/domain/coordinator-election.mjs",
       "require": "./dist/domain/coordinator-election.cjs"
@@ -114,6 +118,10 @@
       "import": "./dist/domain/erasure-coding.mjs",
       "require": "./dist/domain/erasure-coding.cjs"
     },
+    "./domain/fault-report": {
+      "import": "./dist/domain/fault-report.mjs",
+      "require": "./dist/domain/fault-report.cjs"
+    },
     "./domain/gossip-expansion": {
       "import": "./dist/domain/gossip-expansion.mjs",
       "require": "./dist/domain/gossip-expansion.cjs"