@orkestrel/mcp 0.0.20 → 0.0.22

package/dist/src/server/index.cjs

@@ -85,6 +85,18 @@ var DEFAULT_MCP_SESSION_CAPACITY = 1024;
 * this bounds the replay log paired with it.
 */
 var DEFAULT_MCP_SESSION_TTL = 3e5;
+/**
+* The default bound in milliseconds on one unconfirmed write to a stdio client transport's
+* child `stdin` — the `delivery` a `createStdioClientTransport` caller who supplies none gets.
+*
+* @remarks
+* Ten seconds. The load-bearing property is the ordering, not the magnitude: this bound stays
+* BELOW {@link import('@orkestrel/mcp').DEFAULT_MCP_REQUEST_TIMEOUT}, so a write the child never
+* reads fails as an undeliverable message while the request that carried it is still open,
+* rather than being masked by that request's own deadline expiring first. Override per
+* transport with `delivery`; an explicit `0` there removes the bound.
+*/
+var DEFAULT_MCP_DELIVERY = 1e4;
 //#endregion
 //#region src/server/helpers.ts
 /**
@@ -1363,7 +1375,7 @@ var WebSocketClientTransport = class {
 //#region src/server/transports/StdioClientTransport.ts
 /**
 * The stdio CLIENT transport for the Model Context Protocol — a
-* {@link MCPClientTransportInterface} that drives a CHILD PROCESS MCP server over
+* {@link StdioClientTransportInterface} that drives a CHILD PROCESS MCP server over
 * newline-delimited JSON-RPC on `stdin`/`stdout`, the stdio sibling of {@link
 * import('./HTTPClientTransport.js').HTTPClientTransport} and {@link
 * import('./WebSocketClientTransport.js').WebSocketClientTransport}.
@@ -1381,19 +1393,36 @@ var WebSocketClientTransport = class {
 * - **Outbound (`send`).** `send(message)` writes one newline-terminated `JSON.stringify`d line
 *   through the supervisor's `send` and AWAITS its answer, so this promise settles only after the
 *   host reports the line handled rather than the moment the write is queued. The supervisor never
-*   rejects — it answers `false` for a channel that was closed, destroyed, or ended, and for a write
-*   that failed — so a `false` answer REJECTS here with the same not-connected error a transport
-*   that was never started raises. A dead peer surfaces at the caller instead of vanishing.
-* - **`close()`** releases this transport's line pump without waiting for the child's stdout
-*   iterator, then runs the supervisor's bounded `SIGTERM` → grace → `SIGKILL` group-kill and
-*   teardown before firing `close` once (idempotent). A descendant can retain an inherited stdout
-*   pipe after the child exits; the pump's release barrier keeps that substrate limit from keeping
-*   this transport's `close()` pending. On a POSIX host the child leads its own process group, so
-*   the group-kill reaches its grandchildren rather than orphaning them.
+*   rejects — it answers `false` for a channel that was closed, destroyed, or ended, for a write
+*   that failed, or for one that remained unconfirmed through `delivery`. A call made without a
+*   live child rejects as not connected; a `false` answer from a live child rejects as unable to
+*   deliver. The supervisor does not disclose which cause produced that answer.
+* - **`close()`** runs the supervisor's bounded termination and teardown, then fires `close` once
+*   (idempotent). That teardown reaches the child's TERMINAL MOMENT, where the supervisor freezes
+*   `evidence`, ends `lines`, and settles `exit` together, so this transport needs no release of
+*   its own to get its line pump back: the stream ends under the pump rather than throwing at it.
+*   A line the supervisor had already framed behind the one being delivered is dropped rather than
+*   emitted onto a transport whose teardown has begun. A `close()` issued while that teardown runs
+*   joins it rather than opening a second one, so it resolves only after `close` has fired, and a
+*   `start()` issued while it runs waits behind the same barrier, so lifetimes never overlap. A
+*   descendant can retain an inherited stdout pipe after the child exits; the supervisor's `drain`
+*   bound cuts that wait off, so this transport's `close()` settles within that bound rather than
+*   on the descendant. The termination itself belongs to the host: a POSIX host signals the
+*   child's own process group `SIGTERM`, waits the grace window, then `SIGKILL`s through the same
+*   route, so the kill reaches grandchildren rather than orphaning them, while Windows ends the
+*   tree with `taskkill /F /T`, which nothing in the child can intercept.
+* - **Evidence.** `evidence` reports that retained stderr tail off the HELD child — its live tail
+*   while the child runs, and the value the supervisor froze at that child's terminal moment
+*   afterwards. The reference is held past that moment and replaced only by the next `start()`,
+*   which is what keeps a post-`close()` read stable without a private copy: the frozen value
+*   never moves again, so a detached descendant writing to the inherited stderr after the cutoff
+*   cannot grow it. See {@link StdioClientTransportInterface.evidence} for the readings and the
+*   byte bound.
 * - **Observable.** Owns the `emitter` ({@link MCPClientTransportEventMap}); the
 *   emitter isolates a listener throw; `error` is a DOMAIN event (a transport-level
-*   fault, including the child spawn cause the supervisor surfaces), distinct from the emitter's
-*   own listener-error channel.
+*   fault, including the child spawn cause the supervisor surfaces and the notice that this
+*   lifetime's `evidence` was cut off at the `drain` bound), distinct from the emitter's own
+*   listener-error channel.
 *
 * @example
 * ```ts
@@ -1407,15 +1436,16 @@ var StdioClientTransport = class {
 	#command;
 	#args;
 	#env;
+	#delivery;
 	#process = void 0;
-	#release = Promise.withResolvers();
-	#pumping = Promise.resolve();
+	#closing = void 0;
 	#closed = false;
 	constructor(options) {
 		this.#emitter = new _orkestrel_emitter.Emitter();
 		this.#command = options.command;
 		this.#args = options.args ?? [];
 		this.#env = options.env;
+		this.#delivery = options.delivery ?? 1e4;
 	}
 	get emitter() {
 		return this.#emitter;
@@ -1424,10 +1454,21 @@ var StdioClientTransport = class {
 	get duplex() {
 		return true;
 	}
+	get evidence() {
+		return this.#process?.evidence;
+	}
 	async start() {
-		if (this.#process !== void 0) return;
+		let closing = this.#closing;
+		while (closing !== void 0) {
+			await closing;
+			if (this.#closing === closing) {
+				this.#closing = void 0;
+				break;
+			}
+			closing = this.#closing;
+		}
+		if (this.#process !== void 0 && !this.#closed) return;
 		this.#closed = false;
-		this.#release = Promise.withResolvers();
 		const child = new _orkestrel_process_server.Process({
 			command: {
 				file: this.#command,
@@ -1436,43 +1477,65 @@ var StdioClientTransport = class {
 			},
 			workspace: process.cwd(),
 			grace: _orkestrel_process.PROCESS_GRACE,
+			delivery: this.#delivery,
 			writable: true
 		});
 		this.#process = child;
 		child.emitter.on("error", (cause) => this.#emitter.emit("error", cause));
-		child.exit.then(() => this.#onExit(child));
-		this.#pumping = this.#pump(child, this.#release.promise);
+		child.exit.then((exit) => this.#onExit(child, exit));
+		this.#pump(child);
 	}
+	/**
+	* Sends one newline-delimited JSON-RPC message to the live child.
+	*
+	* @param message - The message to write to the child's `stdin`
+	* @returns Resolves when the supervisor confirms the write
+	* @throws Thrown with `stdio transport is not connected` when no live child is available before
+	*   the write
+	* @throws Thrown with `stdio transport could not deliver the message` when a live child's write
+	*   resolves `false`
+	*/
 	async send(message) {
-		const child = this.#process;
-		if (!(child === void 0 ? false : await child.send(JSON.stringify(message)))) throw new Error("stdio transport is not connected");
+		const child = this.#closed ? void 0 : this.#process;
+		if (child === void 0) throw new Error("stdio transport is not connected");
+		if (!await child.send(JSON.stringify(message))) throw new Error("stdio transport could not deliver the message");
 	}
 	async close() {
+		if (this.#closed && this.#closing === void 0) return;
+		this.#closing ??= this.#teardown();
+		await this.#closing;
+	}
+	async #teardown() {
 		if (this.#closed) return;
 		this.#closed = true;
 		const child = this.#process;
-		const pumping = this.#pumping;
-		this.#release.resolve();
-		this.#process = void 0;
-		if (child !== void 0) await child.destroy();
-		await pumping;
+		if (child !== void 0) {
+			await child.destroy();
+			this.#report(await child.exit);
+		}
 		this.#emitter.emit("close");
 	}
-	async #pump(child, release) {
-		const iterator = child.lines[Symbol.asyncIterator]();
-		while (true) {
-			const next = await Promise.race([iterator.next(), release]);
-			if (next === void 0 || next.done) return;
-			if (this.#process !== child) return;
-			dispatchLines(this.#emitter, [next.value]);
+	async #pump(child) {
+		for await (const line of child.lines) {
+			if (this.#closed || this.#process !== child) return;
+			dispatchLines(this.#emitter, [line]);
 		}
 	}
-	#onExit(child) {
-		if (this.#closed || this.#process !== child) return;
+	#onExit(child, exit) {
+		if (this.#process !== child) return;
+		if (this.#closed) return;
 		this.#closed = true;
-		this.#process = void 0;
+		const barrier = Promise.withResolvers();
+		this.#closing ??= barrier.promise;
+		this.#report(exit);
+		barrier.resolve();
+		if (this.#closing === barrier.promise) this.#closing = void 0;
 		this.#emitter.emit("close");
 	}
+	#report(exit) {
+		if (exit.drained) return;
+		this.#emitter.emit("error", /* @__PURE__ */ new Error("stdio transport evidence may be incomplete: the child streams stayed open past the supervisor drain bound"));
+	}
 };
 //#endregion
 //#region src/server/transports/StdioServerTransport.ts
@@ -1817,22 +1880,28 @@ function createWebSocketClientTransport(options) {
 }
 /**
 * Creates the stdio CLIENT transport for an {@link import('@orkestrel/mcp').MCPClientInterface}
-* — a {@link MCPClientTransportInterface} that spawns and drives a CHILD PROCESS MCP server
+* — a {@link StdioClientTransportInterface} that spawns and drives a CHILD PROCESS MCP server
 * over newline-delimited JSON-RPC on `stdin`/`stdout`, the stdio sibling of {@link
 * createHTTPClientTransport} and {@link createWebSocketClientTransport}.
 *
 * @remarks
 * Hand it to `createMCPClient({ transport })`: `start()` (run by `client.connect()`)
 * spawns `options.command` with `options.args` and `options.env`, piping its
-* `stdin`/`stdout` for the JSON-RPC channel (its `stderr` inherits the parent's for
-* diagnostics). Each JSON-RPC message the client `send`s is written as one
+* `stdin`/`stdout` for the JSON-RPC channel. The child's `stderr` is piped too, and
+* retained as a bounded tail this transport reports as `evidence` — the parent never
+* inherits it. Each JSON-RPC message the client `send`s is written as one
 * newline-terminated line to the child's `stdin`; each decoded reply line from the
 * child's `stdout` is surfaced on the transport's `message` event for the client's
-* id correlation.
+* id correlation. That write is bounded: a child that stays alive without ever reading
+* its `stdin` fills the pipe, and `options.delivery` is how long the unconfirmed write
+* waits before the `send` rejects. An omitted `delivery` selects {@link
+* import('./constants.js').DEFAULT_MCP_DELIVERY}; an explicit `0` removes the bound.
 *
 * @param options - `command` (the executable to spawn; REQUIRED), optional `args`,
-*   and optional `env`; see {@link StdioClientTransportOptions}
-* @returns A working {@link MCPClientTransportInterface} over a child process's stdio
+*   optional `env`, and an optional `delivery` bound in milliseconds on an unconfirmed
+*   `stdin` write; see {@link StdioClientTransportOptions}
+* @returns A working {@link StdioClientTransportInterface} over a child process's stdio,
+*   whose `evidence` carries the supervised child's bounded stderr tail
 *
 * @example
 * ```ts
@@ -2069,6 +2138,7 @@ function createMCPSession(options) {
 	};
 }
 //#endregion
+exports.DEFAULT_MCP_DELIVERY = DEFAULT_MCP_DELIVERY;
 exports.DEFAULT_MCP_KEEPALIVE_INTERVAL = DEFAULT_MCP_KEEPALIVE_INTERVAL;
 exports.DEFAULT_MCP_PATH = DEFAULT_MCP_PATH;
 exports.DEFAULT_MCP_SESSION_CAPACITY = DEFAULT_MCP_SESSION_CAPACITY;