@orkestrel/browser 0.0.21 → 0.0.22

package/dist/src/server/index.cjs

@@ -868,7 +868,7 @@ var FileBrowserWriter = class {
 };
 //#endregion
 //#region package.json
-var version = "0.0.21";
+var version = "0.0.22";
 //#endregion
 //#region src/server/BrowserMCPServer.ts
 /**
@@ -876,9 +876,9 @@ var version = "0.0.21";
 * the Model Context Protocol on stdio, and launches Chromium on the first tool call.
 *
 * @remarks
-* The server's own manager holds one dispatcher per name of the vocabulary — `look`, `read`,
+* The server's own manager holds one dispatcher per name of the vocabulary — `look`, `read`, `plain`,
 * `click`, `type`, `press`, `navigate`, `wait`, `dialog`, `tabs`, `switch`, `record`, `save`,
-* `journeys`, `edit`, and `replay` — each carrying the description, parameters, and annotations
+* `journeys`, `edit`, `replay`, and `forget` — each carrying the description, parameters, and annotations
 * `BROWSER_TOOL_COPY` gives, so `tools/list` answers before Chromium starts and with Chromium
 * absent. After the launch, a tool the toolset's manager adds under another name, such as a page
 * tool it adopts, is mirrored as a dispatcher with that tool's definition, and the mirror is
@@ -1228,7 +1228,13 @@ var FileBrowserStore = class {
 			await this.check(path, options);
 			await this.check(temporary, options);
 			options?.signal?.throwIfAborted();
-			await (0, node_fs_promises.rename)(temporary, path);
+			try {
+				await (0, node_fs_promises.rename)(temporary, path);
+			} catch (error) {
+				const target = await (0, node_fs_promises.lstat)(path).catch(() => void 0);
+				if (error instanceof Error && "code" in error && (error.code === "EISDIR" || error.code === "EPERM") && target?.isDirectory()) throw new _src_core.BrowserError(`Cannot replace directory: ${path}`, "BROWSER_JOURNEY_FILE", { path });
+				throw error;
+			}
 			owned = false;
 		} catch (error) {
 			options?.signal?.throwIfAborted();
@@ -1429,7 +1435,14 @@ var FileBrowserStore = class {
 		try {
 			await (0, node_fs_promises.rmdir)(path);
 		} catch (error) {
-			if (error instanceof Error && "code" in error && (error.code === "ENOENT" || error.code === "ENOTEMPTY")) return;
+			if (error instanceof Error && "code" in error) {
+				if (error.code === "ENOTEMPTY") return;
+				if (error.code === "ENOENT" || error.code === "EPERM") try {
+					if ((await (0, node_fs_promises.lstat)(path)).isDirectory()) return;
+				} catch (cause) {
+					if (cause instanceof Error && "code" in cause && cause.code === "ENOENT") return;
+				}
+			}
 			throw new _src_core.BrowserError(`Cannot remove lock directory: ${path}`, "BROWSER_JOURNEY_ACCESS", { path });
 		}
 	}
@@ -2354,6 +2367,7 @@ var Browser = class {
 			const process = this.#process;
 			if (process !== void 0) {
 				if (!await this.#waitForTerminationWithin(process, 3e3)) await this.#terminate(process);
+				await this.#closeProcessPipe(process);
 			}
 			this.#process = void 0;
 			this.#servingPid = void 0;
@@ -2525,12 +2539,23 @@ var Browser = class {
 		return results[0] && (results[1] === true || final && results[1] === void 0);
 	}
 	async #terminate(process) {
-		if (process === void 0 || (process.exitCode !== null || process.signalCode !== null) && this.#inspectRemainder(process) === false) return;
-		if (this.#signalProcess(process, "SIGTERM") === false) return;
-		if (await this.#waitForTerminationWithin(process, 3e3)) return;
-		if (this.#signalProcess(process, "SIGKILL") === false) return;
-		if (await this.#waitForTerminationWithin(process, 3e3, true)) return;
-		throw new _src_core.BrowserConnectionError("Browser process did not exit after SIGKILL", { pid: this.#servingPid ?? process.pid });
+		try {
+			if (process === void 0 || (process.exitCode !== null || process.signalCode !== null) && this.#inspectRemainder(process) === false) return;
+			if (this.#signalProcess(process, "SIGTERM") === false) return;
+			if (await this.#waitForTerminationWithin(process, 3e3)) return;
+			if (this.#signalProcess(process, "SIGKILL") === false) return;
+			if (await this.#waitForTerminationWithin(process, 3e3, true)) return;
+			throw new _src_core.BrowserConnectionError("Browser process did not exit after SIGKILL", { pid: this.#servingPid ?? process.pid });
+		} finally {
+			await this.#closeProcessPipe(process);
+		}
+	}
+	async #closeProcessPipe(process) {
+		const stream = process?.stderr;
+		if (stream === void 0 || stream === null || stream.closed) return;
+		const closed = (0, node_events.once)(stream, "close");
+		stream.destroy();
+		await closed;
 	}
 	#finish() {
 		this.#unbindTransport();