@mark1russell7/service 0.5.4 → 0.5.6

package/build/index.js

@@ -3,9 +3,9 @@ import { detectionTime, heartbeat } from "./heartbeat.js";
 import { a as recoverSpan, i as heartbeatDetection, n as asWindow, r as failSpan, t as asDebounce } from "./timing-DGRi3XCI.js";
 import { n as operationProblems, t as mayReissue } from "./operation-DL5RxFds.js";
 import { a as limiterProblems, i as limitRule, n as freeAt, o as longestRetryAfter, r as idleSpan, s as retryAfterSeconds, t as decide } from "./policy-CgS4y94U.js";
+import { a as lateHolds, l as takeOutside, n as countOf, r as millisecondsOf, s as ownStream, t as boundedMillisecondsOf } from "./knobs-D2ClI68I.js";
 import { backoffBand, backoffDelay, backoffProblems } from "./backoff.js";
 import { n as memoryStore, t as keyedLimiter } from "./limiter-wv0uXDk9.js";
-import { n as countOf, r as millisecondsOf, t as boundedMillisecondsOf } from "./knobs-BxG-c7Zb.js";
 import { convertExact, convertible, quantity, rationals, valueIn } from "@mark1russell7/quantity";
 import { replay } from "@mark1russell7/algebra/replay";
 //#region src/description.ts
@@ -1150,8 +1150,11 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 		if (found.as === void 0) triggered("ipc-message");
 		else triggered(found.as, `${found.as} (a message of the IPC channel)`);
 	};
-	/** This function does the work of one phase, until its cap. It gives true when the work ended in time. */
-	const runPhase = async (phase, hold, work) => {
+	/**
+	* This function does the work of one phase, until its cap. It tells if the work ended in time. When
+	* work is left at the cap, `excuse` gives the cause of a cap that is not a failure, or null.
+	*/
+	const runPhase = async (phase, hold, work, excuse = () => null) => {
 		const cap = valueIn(phase.cap, "ms");
 		const deadline = clock.now() + cap;
 		const capped = new AbortController();
@@ -1175,9 +1178,13 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 			cancelCap = clock.after(cap, () => resolve("cap"));
 		});
 		if (await Promise.race([all.then(() => "work"), end]) === "work" && hold) await end;
+		const excused = pending > 0 ? excuse(pending) : null;
 		cancelCap();
 		capped.abort(/* @__PURE__ */ new Error(`the phase ${phase.name} reached its cap`));
-		return pending === 0;
+		return {
+			inTime: pending === 0,
+			excused
+		};
 	};
 	async function shutdown(cause = "the service asked for it", request = {}) {
 		if (request.code !== void 0 && !isExitCode(request.code)) throw new Error(`${description.name}: the exit code ${request.code} is not a whole number from 0 to 255`);
@@ -1209,10 +1216,11 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 		};
 		const phases = description.shutdown.phases;
 		let on = "exit.request";
+		let detail;
 		for (const phase of phases) {
 			if (exitState === "exited") break;
 			const rank = stageRank(phase.name);
-			moveDrain(phase.name, on);
+			moveDrain(phase.name, on, detail);
 			if (rank > ABORT_AFTER) abortService();
 			const work = [];
 			const owned = components.phaseWork.get(phase.name) ?? [];
@@ -1224,6 +1232,7 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 				for (const [, impl] of startedTransports) work.push(() => impl.stopIntake());
 			}
 			let cancelAbort = () => {};
+			let excuse;
 			if (phase.name === "drain") {
 				const cap = valueIn(phase.cap, "ms");
 				const cooperative = new AbortController();
@@ -1234,7 +1243,20 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 					record("drain", drainState, drainState, "drain.cooperative-abort");
 				});
 				work.push(({ deadline }) => requests.drain(deadline));
-				for (const [, impl] of startedTransports) work.push(({ deadline }) => impl.drain(deadline, cooperative.signal));
+				let transportsLeft = 0;
+				for (const [, impl] of startedTransports) work.push(async ({ deadline }) => {
+					transportsLeft += 1;
+					try {
+						await impl.drain(deadline, cooperative.signal);
+					} finally {
+						transportsLeft -= 1;
+					}
+				});
+				excuse = (left) => {
+					const holds = lateHolds(life);
+					if (left > transportsLeft || transportsLeft > holds.length || holds.some((each) => each.running > 0)) return null;
+					return `answers not flushed: ${holds.reduce((sum, each) => sum + each.unflushed, 0)}`;
+				};
 			}
 			if (phase.name === "close transports") for (const [, impl] of startedTransports) work.push(() => impl.close());
 			if (phase.name === "stop the poller") work.push(() => {
@@ -1259,12 +1281,15 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 				await port.flush?.();
 			});
 			else for (const each of owned) work.push(each);
-			const inTime = await runPhase(phase, phase.name === "fail readiness", work);
+			const { inTime, excused } = await runPhase(phase, phase.name === "fail readiness", work, excuse);
 			cancelAbort();
-			if (!inTime) fail(`the phase ${phase.name} did not end in its cap of ${valueIn(phase.cap, "s")} s`);
+			if (!inTime && excused === null) fail(`the phase ${phase.name} did not end in its cap of ${valueIn(phase.cap, "s")} s`);
 			on = inTime ? "phase.done" : "phase.cap";
+			const outsideCount = phase.name === "drain" ? takeOutside(life) : 0;
+			const parts = [inTime ? null : excused, outsideCount > 0 ? `streams outside the lifecycle: ${outsideCount}` : null].filter((part) => part !== null);
+			detail = parts.length > 0 ? parts.join(", ") : void 0;
 		}
-		if (exitState !== "exited") moveDrain("done", on);
+		if (exitState !== "exited") moveDrain("done", on, detail);
 		abortService();
 		runtime.stop();
 		components.stopPollers();
@@ -1289,7 +1314,7 @@ const lifecycle = (description, runtime, components, options = {}, bind = () =>
 		done,
 		signal: service.signal,
 		shutdown,
-		request: (controller, route, work, request) => requests.request(controller, route, work, request),
+		request: (controller, route, work, request) => ownStream(requests.request(controller, route, work, request), life),
 		inFlight: (controller) => requests.inFlight(controller),
 		keepAlive: () => state === "running" && runtime.verdict().ready
 	};