@orkestrel/browser 0.0.21 → 0.0.22

package/dist/src/server/index.d.cts

@@ -448,9 +448,9 @@ export declare type BrowserLaunchFunction = (options: BrowserOptions) => Browser
  * 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

package/dist/src/server/index.d.ts

@@ -448,9 +448,9 @@ export declare type BrowserLaunchFunction = (options: BrowserOptions) => Browser
  * 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

package/dist/src/server/index.js

@@ -867,7 +867,7 @@ var FileBrowserWriter = class {
 };
 //#endregion
 //#region package.json
-var version = "0.0.21";
+var version = "0.0.22";
 //#endregion
 //#region src/server/BrowserMCPServer.ts
 /**
@@ -875,9 +875,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
@@ -1227,7 +1227,13 @@ var FileBrowserStore = class {
 			await this.check(path, options);
 			await this.check(temporary, options);
 			options?.signal?.throwIfAborted();
-			await rename(temporary, path);
+			try {
+				await rename(temporary, path);
+			} catch (error) {
+				const target = await lstat(path).catch(() => void 0);
+				if (error instanceof Error && "code" in error && (error.code === "EISDIR" || error.code === "EPERM") && target?.isDirectory()) throw new BrowserError(`Cannot replace directory: ${path}`, "BROWSER_JOURNEY_FILE", { path });
+				throw error;
+			}
 			owned = false;
 		} catch (error) {
 			options?.signal?.throwIfAborted();
@@ -1428,7 +1434,14 @@ var FileBrowserStore = class {
 		try {
 			await 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 lstat(path)).isDirectory()) return;
+				} catch (cause) {
+					if (cause instanceof Error && "code" in cause && cause.code === "ENOENT") return;
+				}
+			}
 			throw new BrowserError(`Cannot remove lock directory: ${path}`, "BROWSER_JOURNEY_ACCESS", { path });
 		}
 	}
@@ -2353,6 +2366,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;
@@ -2524,12 +2538,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 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 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 = once(stream, "close");
+		stream.destroy();
+		await closed;
 	}
 	#finish() {
 		this.#unbindTransport();