@statewalker/webrun-http-browser 0.12.1 → 0.12.3

package/dist/relay-worker.js

@@ -1137,7 +1137,10 @@ async function httpFetch(call, env, body, options = {}) {
 		}
 		const onAbort = () => void release();
 		signal?.addEventListener("abort", onAbort, { once: true });
-		drain(output).finally(() => signal?.removeEventListener("abort", onAbort));
+		options.keepAlive;
+		drain(output).finally(() => {
+			signal?.removeEventListener("abort", onAbort);
+		});
 	};
 	let decoded;
 	try {
@@ -1387,7 +1390,8 @@ async function fetchOverDuplex(call, request, options = {}) {
 	}
 	const { envelope, body: respBody, release, finish } = await httpFetch(call, env, body, {
 		...options,
-		signal: request.signal ?? void 0
+		signal: request.signal ?? void 0,
+		keepAlive: request
 	});
 	const responseInit = {
 		status: envelope.status,
@@ -1399,7 +1403,7 @@ async function fetchOverDuplex(call, request, options = {}) {
 		else await release?.();
 		return new Response(null, responseInit);
 	}
-	return new Response(responseBody(respBody, request.signal, release), responseInit);
+	return new Response(responseBody(respBody, request, release), responseInit);
 }
 /**
 * How long a cancelled body waits for the read in flight before it abandons the call anyway.
@@ -1422,7 +1426,9 @@ const CANCEL_GRACE_MS = 1e3;
 * A cancel abandons the call too — unless the read in flight comes back with the
 * body's end, which finishes it instead (see `cancel`).
 */
-function responseBody(body, signal, release) {
+function responseBody(body, request, release) {
+	const keep = { request };
+	const signal = request.signal ?? void 0;
 	const it = body[Symbol.asyncIterator]();
 	let controller;
 	let ended = false;
@@ -1430,6 +1436,7 @@ function responseBody(body, signal, release) {
 	const end = () => {
 		ended = true;
 		signal?.removeEventListener("abort", onAbort);
+		keep.request = void 0;
 	};
 	const onAbort = () => {
 		if (ended) return;
@@ -2393,12 +2400,6 @@ async function sendChunks(port, channelName, output, { maxMessageSize, log }, si
 	const producer = output;
 	const returnProducer = () => void Promise.resolve(producer.return?.()).catch(() => {});
 	if (framed !== output) signal.addEventListener("abort", returnProducer, { once: true });
-	let onAborted = () => {};
-	const aborted = new Promise((resolve) => {
-		onAborted = () => resolve();
-	});
-	if (signal.aborted) onAborted();
-	else signal.addEventListener("abort", onAborted, { once: true });
 	try {
 		await sendIterator(async ({ done, value, error }) => {
 			if (signal.aborted) return;
@@ -2420,7 +2421,7 @@ async function sendChunks(port, channelName, output, { maxMessageSize, log }, si
 				});
 				touch();
 				if (done && !error) onSent();
-				if (portBufferedAmount(port) > 0) await Promise.race([whenPortReady(port), aborted]);
+				if (portBufferedAmount(port) > 0) await whenReadyOrAborted(port, signal);
 				return;
 			}
 			await callPort(port, chunk, {
@@ -2436,10 +2437,25 @@ async function sendChunks(port, channelName, output, { maxMessageSize, log }, si
 		throw err;
 	} finally {
 		signal.removeEventListener("abort", returnProducer);
-		signal.removeEventListener("abort", onAborted);
 	}
 }
 /**
+* A pipelined sender waits only for room in the window, or for the abort. The
+* abort listener lives for this one wait: a promise held for the whole stream
+* would keep one reaction per wait until the stream ends.
+*/
+function whenReadyOrAborted(port, signal) {
+	if (signal.aborted) return Promise.resolve();
+	return new Promise((resolve) => {
+		const settle = () => {
+			signal.removeEventListener("abort", settle);
+			resolve();
+		};
+		signal.addEventListener("abort", settle, { once: true });
+		whenPortReady(port).then(settle, settle);
+	});
+}
+/**
 * One virtual port.
 *
 * `deliver` and `markClosed` are deliberately not on `port`: the consumer holds
@@ -3365,7 +3381,12 @@ function trackedLink(send, release) {
 				finish();
 				return response;
 			}
-			return new Response(guardBody(response.body, lost, () => lostReason, finish, request.signal), {
+			failers.delete(fail);
+			const body = guardBody(response.body, finish, request);
+			fail = body.fail;
+			if (closed) fail(lostReason);
+			else if (!finished) failers.add(fail);
+			return new Response(body.stream, {
 				status: response.status,
 				statusText: response.statusText,
 				headers: response.headers
@@ -3384,56 +3405,62 @@ function trackedLink(send, release) {
 	return asyncDisposable(link, () => link.close());
 }
 /**
-* A response body that reports its end (done, cancelled, failed or aborted) and
-* fails with the link's reason when the link is lost mid-stream. An abort of the
-* request's signal ends it at once, read or not: the exchange is over (the layer
-* below has given its port back), so the request leaves `inflight` then.
+* A response body that reports its end (done, cancelled, failed or aborted).
+* `fail(error)` ends it with `error` at once — the link calls it when it is lost
+* mid-stream. An abort of the request's signal does the same with the signal's
+* reason, read or not: the exchange is over (the layer below has given its port
+* back), so the request leaves `inflight` then. A read waits on the reader
+* alone, so nothing outlives the chunk it brings.
 */
-function guardBody(body, lost, reason, finish, signal) {
+function guardBody(body, finish, request) {
+	const keep = { request };
+	const signal = request.signal ?? void 0;
 	const reader = body.getReader();
 	let controller;
 	let ended = false;
 	const end = () => {
 		ended = true;
 		signal?.removeEventListener("abort", onAbort);
+		keep.request = void 0;
 		finish();
 	};
-	const onAbort = () => {
+	const fail = (error) => {
 		if (ended) return;
 		end();
 		reader.cancel().catch(() => {});
 		try {
-			controller.error(abortError(signal));
+			controller.error(error);
 		} catch {}
 	};
-	return new ReadableStream({
-		start(c) {
-			controller = c;
-			if (signal?.aborted) onAbort();
-			else signal?.addEventListener("abort", onAbort, { once: true });
-		},
-		async pull(controller) {
-			try {
-				const { done, value } = await Promise.race([reader.read(), lost]);
-				if (ended) return;
-				if (done) {
-					end();
-					controller.close();
-					return;
+	const onAbort = () => fail(abortError(signal));
+	return {
+		stream: new ReadableStream({
+			start(c) {
+				controller = c;
+				if (signal?.aborted) onAbort();
+				else signal?.addEventListener("abort", onAbort, { once: true });
+			},
+			async pull(controller) {
+				try {
+					const { done, value } = await reader.read();
+					if (ended) return;
+					if (done) {
+						end();
+						controller.close();
+						return;
+					}
+					controller.enqueue(value);
+				} catch (error) {
+					fail(error);
 				}
-				controller.enqueue(value);
-			} catch (error) {
-				if (ended) return;
+			},
+			cancel(why) {
 				end();
-				reader.cancel().catch(() => {});
-				controller.error(reason() ?? error);
+				setTimeout(() => void reader.cancel(why).catch(() => {}), 0);
 			}
-		},
-		async cancel(why) {
-			end();
-			await reader.cancel(why).catch(() => {});
-		}
-	});
+		}),
+		fail
+	};
 }
 /** The signal's reason as an `Error` (`AbortError` when it gave none). */
 function abortError(signal) {

package/dist/sw/worker-service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"worker-service.d.ts","sourceRoot":"","sources":["../../src/sw/worker-service.ts"],"names":[],"mappings":"AACA,OAAO,EAEL,KAAK,kBAAkB,EACvB,KAAK,qBAAqB,EAC1B,KAAK,mBAAmB,EACzB,MAAM,iBAAiB,CAAC;AACzB,OAAO,EACL,KAAK,YAAY,EAEjB,KAAK,sBAAsB,EAC5B,MAAM,uBAAuB,CAAC;AAG/B,MAAM,WAAW,wBACf,SAAQ,IAAI,CAAC,sBAAsB,EAAE,eAAe,GAAG,WAAW,CAAC;IACnE,mFAAmF;IACnF,KAAK,CAAC,EAAE,IAAI,CAAC,WAAW,EAAE,kBAAkB,GAAG,qBAAqB,CAAC,CAAC;IACtE,8DAA8D;IAC9D,SAAS,CAAC,EAAE,qBAAqB,CAAC;CACnC;AAED,yDAAyD;AACzD,MAAM,WAAW,iBAAiB;IAChC;;;;;;OAMG;IACH,KAAK,CAAC,KAAK,EAAE,WAAW,GAAG,GAAG,EAAE,IAAI,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,QAAQ,CAAC,CAAC;IACvE;;;OAGG;IACH,gBAAgB,CACd,GAAG,EAAE,MAAM,GAAG,GAAG,EACjB,SAAS,CAAC,EAAE,MAAM,EAAE,EACpB,OAAO,CAAC,EAAE,WAAW,GACpB,OAAO,CAAC,mBAAmB,CAAC,CAAC;IAChC,0DAA0D;IAC1D,QAAQ,CAAC,SAAS,EAAE,kBAAkB,CAAC;IACvC,0CAA0C;IAC1C,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC;IACzB,8CAA8C;IAC9C,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACxB;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,iBAAiB,CAC/B,OAAO,EAAE,YAAY,EACrB,OAAO,GAAE,wBAA6B,GACrC,iBAAiB,CAoFnB"}
+{"version":3,"file":"worker-service.d.ts","sourceRoot":"","sources":["../../src/sw/worker-service.ts"],"names":[],"mappings":"AACA,OAAO,EAEL,KAAK,kBAAkB,EACvB,KAAK,qBAAqB,EAC1B,KAAK,mBAAmB,EACzB,MAAM,iBAAiB,CAAC;AACzB,OAAO,EACL,KAAK,YAAY,EAEjB,KAAK,sBAAsB,EAC5B,MAAM,uBAAuB,CAAC;AAG/B,MAAM,WAAW,wBACf,SAAQ,IAAI,CAAC,sBAAsB,EAAE,eAAe,GAAG,WAAW,CAAC;IACnE,mFAAmF;IACnF,KAAK,CAAC,EAAE,IAAI,CAAC,WAAW,EAAE,kBAAkB,GAAG,qBAAqB,CAAC,CAAC;IACtE,8DAA8D;IAC9D,SAAS,CAAC,EAAE,qBAAqB,CAAC;CACnC;AAED,yDAAyD;AACzD,MAAM,WAAW,iBAAiB;IAChC;;;;;;OAMG;IACH,KAAK,CAAC,KAAK,EAAE,WAAW,GAAG,GAAG,EAAE,IAAI,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,QAAQ,CAAC,CAAC;IACvE;;;OAGG;IACH,gBAAgB,CACd,GAAG,EAAE,MAAM,GAAG,GAAG,EACjB,SAAS,CAAC,EAAE,MAAM,EAAE,EACpB,OAAO,CAAC,EAAE,WAAW,GACpB,OAAO,CAAC,mBAAmB,CAAC,CAAC;IAChC,0DAA0D;IAC1D,QAAQ,CAAC,SAAS,EAAE,kBAAkB,CAAC;IACvC,0CAA0C;IAC1C,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC;IACzB,8CAA8C;IAC9C,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACxB;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,iBAAiB,CAC/B,OAAO,EAAE,YAAY,EACrB,OAAO,GAAE,wBAA6B,GACrC,iBAAiB,CA+FnB"}

package/dist/sw-worker.js

@@ -1312,7 +1312,10 @@
 			}
 			const onAbort = () => void release();
 			signal?.addEventListener("abort", onAbort, { once: true });
-			drain(output).finally(() => signal?.removeEventListener("abort", onAbort));
+			options.keepAlive;
+			drain(output).finally(() => {
+				signal?.removeEventListener("abort", onAbort);
+			});
 		};
 		let decoded;
 		try {
@@ -1675,7 +1678,8 @@
 		}
 		const { envelope, body: respBody, release, finish } = await httpFetch(call, env, body, {
 			...options,
-			signal: request.signal ?? void 0
+			signal: request.signal ?? void 0,
+			keepAlive: request
 		});
 		const responseInit = {
 			status: envelope.status,
@@ -1687,7 +1691,7 @@
 			else await release?.();
 			return new Response(null, responseInit);
 		}
-		return new Response(responseBody(respBody, request.signal, release), responseInit);
+		return new Response(responseBody(respBody, request, release), responseInit);
 	}
 	/**
 	* Wrap a `(Request) => Promise<Response>` handler as a `Duplex` so it can be
@@ -1840,7 +1844,9 @@
 	* A cancel abandons the call too — unless the read in flight comes back with the
 	* body's end, which finishes it instead (see `cancel`).
 	*/
-	function responseBody(body, signal, release) {
+	function responseBody(body, request, release) {
+		const keep = { request };
+		const signal = request.signal ?? void 0;
 		const it = body[Symbol.asyncIterator]();
 		let controller;
 		let ended = false;
@@ -1848,6 +1854,7 @@
 		const end = () => {
 			ended = true;
 			signal?.removeEventListener("abort", onAbort);
+			keep.request = void 0;
 		};
 		const onAbort = () => {
 			if (ended) return;
@@ -2879,12 +2886,6 @@
 		const producer = output;
 		const returnProducer = () => void Promise.resolve(producer.return?.()).catch(() => {});
 		if (framed !== output) signal.addEventListener("abort", returnProducer, { once: true });
-		let onAborted = () => {};
-		const aborted = new Promise((resolve) => {
-			onAborted = () => resolve();
-		});
-		if (signal.aborted) onAborted();
-		else signal.addEventListener("abort", onAborted, { once: true });
 		try {
 			await sendIterator(async ({ done, value, error }) => {
 				if (signal.aborted) return;
@@ -2906,7 +2907,7 @@
 					});
 					touch();
 					if (done && !error) onSent();
-					if (portBufferedAmount(port) > 0) await Promise.race([whenPortReady(port), aborted]);
+					if (portBufferedAmount(port) > 0) await whenReadyOrAborted(port, signal);
 					return;
 				}
 				await callPort(port, chunk, {
@@ -2922,10 +2923,25 @@
 			throw err;
 		} finally {
 			signal.removeEventListener("abort", returnProducer);
-			signal.removeEventListener("abort", onAborted);
 		}
 	}
 	/**
+	* A pipelined sender waits only for room in the window, or for the abort. The
+	* abort listener lives for this one wait: a promise held for the whole stream
+	* would keep one reaction per wait until the stream ends.
+	*/
+	function whenReadyOrAborted(port, signal) {
+		if (signal.aborted) return Promise.resolve();
+		return new Promise((resolve) => {
+			const settle = () => {
+				signal.removeEventListener("abort", settle);
+				resolve();
+			};
+			signal.addEventListener("abort", settle, { once: true });
+			whenPortReady(port).then(settle, settle);
+		});
+	}
+	/**
 	* One virtual port.
 	*
 	* `deliver` and `markClosed` are deliberately not on `port`: the consumer holds
@@ -3923,7 +3939,12 @@
 					finish();
 					return response;
 				}
-				return new Response(guardBody(response.body, lost, () => lostReason, finish, request.signal), {
+				failers.delete(fail);
+				const body = guardBody(response.body, finish, request);
+				fail = body.fail;
+				if (closed) fail(lostReason);
+				else if (!finished) failers.add(fail);
+				return new Response(body.stream, {
 					status: response.status,
 					statusText: response.statusText,
 					headers: response.headers
@@ -3942,56 +3963,62 @@
 		return asyncDisposable(link, () => link.close());
 	}
 	/**
-	* A response body that reports its end (done, cancelled, failed or aborted) and
-	* fails with the link's reason when the link is lost mid-stream. An abort of the
-	* request's signal ends it at once, read or not: the exchange is over (the layer
-	* below has given its port back), so the request leaves `inflight` then.
+	* A response body that reports its end (done, cancelled, failed or aborted).
+	* `fail(error)` ends it with `error` at once — the link calls it when it is lost
+	* mid-stream. An abort of the request's signal does the same with the signal's
+	* reason, read or not: the exchange is over (the layer below has given its port
+	* back), so the request leaves `inflight` then. A read waits on the reader
+	* alone, so nothing outlives the chunk it brings.
 	*/
-	function guardBody(body, lost, reason, finish, signal) {
+	function guardBody(body, finish, request) {
+		const keep = { request };
+		const signal = request.signal ?? void 0;
 		const reader = body.getReader();
 		let controller;
 		let ended = false;
 		const end = () => {
 			ended = true;
 			signal?.removeEventListener("abort", onAbort);
+			keep.request = void 0;
 			finish();
 		};
-		const onAbort = () => {
+		const fail = (error) => {
 			if (ended) return;
 			end();
 			reader.cancel().catch(() => {});
 			try {
-				controller.error(abortError(signal));
+				controller.error(error);
 			} catch {}
 		};
-		return new ReadableStream({
-			start(c) {
-				controller = c;
-				if (signal?.aborted) onAbort();
-				else signal?.addEventListener("abort", onAbort, { once: true });
-			},
-			async pull(controller) {
-				try {
-					const { done, value } = await Promise.race([reader.read(), lost]);
-					if (ended) return;
-					if (done) {
-						end();
-						controller.close();
-						return;
+		const onAbort = () => fail(abortError(signal));
+		return {
+			stream: new ReadableStream({
+				start(c) {
+					controller = c;
+					if (signal?.aborted) onAbort();
+					else signal?.addEventListener("abort", onAbort, { once: true });
+				},
+				async pull(controller) {
+					try {
+						const { done, value } = await reader.read();
+						if (ended) return;
+						if (done) {
+							end();
+							controller.close();
+							return;
+						}
+						controller.enqueue(value);
+					} catch (error) {
+						fail(error);
 					}
-					controller.enqueue(value);
-				} catch (error) {
-					if (ended) return;
+				},
+				cancel(why) {
 					end();
-					reader.cancel().catch(() => {});
-					controller.error(reason() ?? error);
+					setTimeout(() => void reader.cancel(why).catch(() => {}), 0);
 				}
-			},
-			async cancel(why) {
-				end();
-				await reader.cancel(why).catch(() => {});
-			}
-		});
+			}),
+			fail
+		};
 	}
 	/** The signal's reason as an `Error` (`AbortError` when it gave none). */
 	function abortError(signal) {