@orkestrel/browser 0.0.24 → 0.0.25

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

@@ -107,7 +107,7 @@ export declare const BROWSER_FRAME_WORLD_NAME = "__orkestrelBrowserFrame";
  */
 export declare const BROWSER_HAR_CREATOR: Readonly<{
     name: "@orkestrel/browser";
-    version: "0.0.24";
+    version: "0.0.25";
 }>;
 
 /** Names accessibility roles that receive actionable outline references. */
@@ -711,6 +711,7 @@ export declare class BrowserContext implements BrowserContextInterface {
     constructor(client: CDPClientInterface, id?: string, viewport?: BrowserViewport, writer?: BrowserWriterInterface, emulation?: BrowserEmulationOptions, downloads?: BrowserDownloadOptions, options?: BrowserContextOptions);
     get emitter(): EmitterInterface<BrowserContextEventMap>;
     get id(): string | undefined;
+    get disposal(): BrowserContextDisposal | undefined;
     get cookies(): BrowserCookieManagerInterface;
     get permissions(): BrowserPermissionManagerInterface;
     get storage(): BrowserStorageManagerInterface;
@@ -723,6 +724,14 @@ export declare class BrowserContext implements BrowserContextInterface {
     close(): Promise<void>;
 }
 
+/** Records Chromium's disposal acknowledgement or the failure that left disposal unconfirmed. */
+export declare type BrowserContextDisposal = {
+    readonly confirmed: true;
+} | {
+    readonly confirmed: false;
+    readonly error: unknown;
+};
+
 /** Maps the browser-context lifecycle events. */
 export declare type BrowserContextEventMap = {
     readonly page: readonly [page: BrowserPageInterface];
@@ -746,6 +755,8 @@ export declare type BrowserContextEventMap = {
 export declare interface BrowserContextInterface {
     readonly emitter: EmitterInterface<BrowserContextEventMap>;
     readonly id: string | undefined;
+    /** Holds the remote disposal receipt after `close()` attempts disposal; wrapper destruction has none. */
+    readonly disposal: BrowserContextDisposal | undefined;
     readonly cookies: BrowserCookieManagerInterface;
     readonly permissions: BrowserPermissionManagerInterface;
     readonly storage: BrowserStorageManagerInterface;
@@ -4761,9 +4772,7 @@ export declare interface BrowserRunStoreInterface {
     }
 
     /**
-     * Reports that a CDP request could not be sent or completed because the client was not in a
-     * connectable state — not connected, closed while connecting, or the connection dropped
-     * mid-request — under the code `BROWSER_CDP_CONNECTION_ERROR`.
+     * Reports that a CDP request could not be sent or completed because the client was not connected, closed while connecting, lost its connection, or lost the request's target session, under the code `BROWSER_CDP_CONNECTION_ERROR`.
      */
     export declare class CDPConnectionError extends BrowserError {
         constructor(message: string, context?: Readonly<Record<string, unknown>>);

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

@@ -107,7 +107,7 @@ export declare const BROWSER_FRAME_WORLD_NAME = "__orkestrelBrowserFrame";
  */
 export declare const BROWSER_HAR_CREATOR: Readonly<{
     name: "@orkestrel/browser";
-    version: "0.0.24";
+    version: "0.0.25";
 }>;
 
 /** Names accessibility roles that receive actionable outline references. */
@@ -711,6 +711,7 @@ export declare class BrowserContext implements BrowserContextInterface {
     constructor(client: CDPClientInterface, id?: string, viewport?: BrowserViewport, writer?: BrowserWriterInterface, emulation?: BrowserEmulationOptions, downloads?: BrowserDownloadOptions, options?: BrowserContextOptions);
     get emitter(): EmitterInterface<BrowserContextEventMap>;
     get id(): string | undefined;
+    get disposal(): BrowserContextDisposal | undefined;
     get cookies(): BrowserCookieManagerInterface;
     get permissions(): BrowserPermissionManagerInterface;
     get storage(): BrowserStorageManagerInterface;
@@ -723,6 +724,14 @@ export declare class BrowserContext implements BrowserContextInterface {
     close(): Promise<void>;
 }
 
+/** Records Chromium's disposal acknowledgement or the failure that left disposal unconfirmed. */
+export declare type BrowserContextDisposal = {
+    readonly confirmed: true;
+} | {
+    readonly confirmed: false;
+    readonly error: unknown;
+};
+
 /** Maps the browser-context lifecycle events. */
 export declare type BrowserContextEventMap = {
     readonly page: readonly [page: BrowserPageInterface];
@@ -746,6 +755,8 @@ export declare type BrowserContextEventMap = {
 export declare interface BrowserContextInterface {
     readonly emitter: EmitterInterface<BrowserContextEventMap>;
     readonly id: string | undefined;
+    /** Holds the remote disposal receipt after `close()` attempts disposal; wrapper destruction has none. */
+    readonly disposal: BrowserContextDisposal | undefined;
     readonly cookies: BrowserCookieManagerInterface;
     readonly permissions: BrowserPermissionManagerInterface;
     readonly storage: BrowserStorageManagerInterface;
@@ -4761,9 +4772,7 @@ export declare interface BrowserRunStoreInterface {
     }
 
     /**
-     * Reports that a CDP request could not be sent or completed because the client was not in a
-     * connectable state — not connected, closed while connecting, or the connection dropped
-     * mid-request — under the code `BROWSER_CDP_CONNECTION_ERROR`.
+     * Reports that a CDP request could not be sent or completed because the client was not connected, closed while connecting, lost its connection, or lost the request's target session, under the code `BROWSER_CDP_CONNECTION_ERROR`.
      */
     export declare class CDPConnectionError extends BrowserError {
         constructor(message: string, context?: Readonly<Record<string, unknown>>);

package/dist/src/core/index.js

@@ -162,7 +162,7 @@ var BROWSER_MOUSE_BUTTON_MASKS = Object.freeze({
 */
 var BROWSER_HAR_CREATOR = Object.freeze({
 	name: "@orkestrel/browser",
-	version: "0.0.24"
+	version: "0.0.25"
 });
 /** Names the attribute that tags temporary screenshot styles and masks. */
 var BROWSER_SCREENSHOT_ATTRIBUTE = "data-orkestrel-screenshot";
@@ -1012,9 +1012,7 @@ var CDPError = class extends BrowserError {
 	}
 };
 /**
-* Reports that a CDP request could not be sent or completed because the client was not in a
-* connectable state — not connected, closed while connecting, or the connection dropped
-* mid-request — under the code `BROWSER_CDP_CONNECTION_ERROR`.
+* Reports that a CDP request could not be sent or completed because the client was not connected, closed while connecting, lost its connection, or lost the request's target session, under the code `BROWSER_CDP_CONNECTION_ERROR`.
 */
 var CDPConnectionError = class extends BrowserError {
 	constructor(message, context) {
@@ -5224,6 +5222,7 @@ var CDPClient = class {
 			const listener = signal === void 0 ? void 0 : this.#abort(id, signal);
 			this.#pending.set(id, {
 				method,
+				session: options?.session,
 				resolve,
 				reject,
 				timer,
@@ -5368,6 +5367,16 @@ var CDPClient = class {
 			const method = parsed["method"];
 			const rawParams = parsed["params"];
 			const params = isRecord(rawParams) ? rawParams : Object.freeze({});
+			if (method === "Target.detachedFromTarget" && isString(params["sessionId"])) {
+				const detached = params["sessionId"];
+				for (const [id, entry] of this.#pending) {
+					if (entry.session !== detached) continue;
+					this.#settle(id)?.reject(new CDPConnectionError("CDP session detached", {
+						method: entry.method,
+						session: detached
+					}));
+				}
+			}
 			const global = this.#subscriptions.get(void 0)?.get(method);
 			if (global !== void 0) this.#dispatch(global, method, params);
 			const session = parsed["sessionId"];
@@ -7207,10 +7216,12 @@ var BrowserRegistry = class {
 		for (const frame of new Set([...this.tools(), ...this.#pending].map((entry) => entry.frame))) this.#invalidate(frame);
 	}
 	#navigated(params) {
+		if (this.#page.closed || this.#destroying !== void 0) return;
 		const frame = params["frame"];
 		if (isRecord(frame) && isString(frame["id"])) this.#invalidate(frame["id"]);
 	}
 	#detached(params) {
+		if (this.#page.closed || this.#destroying !== void 0) return;
 		if (isString(params["frameId"])) this.#invalidate(params["frameId"]);
 	}
 	#invalidate(frame) {
@@ -7224,6 +7235,7 @@ var BrowserRegistry = class {
 		if (removed) this.#emitter.emit("change");
 	}
 	#add(session, params) {
+		if (this.#page.closed || this.#destroying !== void 0) return;
 		if (!isArray(params["tools"])) return;
 		for (const value of params["tools"]) {
 			const tool = parseBrowserTool(value);
@@ -7234,6 +7246,7 @@ var BrowserRegistry = class {
 		this.#emitter.emit("change");
 	}
 	#remove(params) {
+		if (this.#page.closed || this.#destroying !== void 0) return;
 		if (!isArray(params["tools"])) return;
 		for (const value of params["tools"]) {
 			const key = parseBrowserRemoval(value);
@@ -10169,10 +10182,10 @@ var BrowserPage = class BrowserPage extends BrowserFrame {
 		} catch {}
 	}
 	async #close() {
-		await this.#release();
 		try {
 			await this.#client.send("Target.closeTarget", { targetId: this.#targetId });
 		} catch {}
+		await this.#release();
 	}
 	#release() {
 		const active = this.#releasing;
@@ -10473,12 +10486,14 @@ var BrowserPage = class BrowserPage extends BrowserFrame {
 	}
 	async #attachWorker(session, id, url, category) {
 		try {
-			await this.#client.send("Runtime.enable", void 0, { session });
+			const enabling = this.#client.send("Runtime.enable", void 0, { session });
+			if (category === "service_worker") this.#resumeTarget(session);
+			await enabling;
 			if (this.#closed) {
 				await this.#detachChild(session, this.#sessionId);
 				return;
 			}
-			this.#resumeTarget(session);
+			if (category !== "service_worker") this.#resumeTarget(session);
 			const worker = new BrowserWorker(this.#client, session, id, url, category);
 			this.#workers.set(id, worker);
 			this.#emitter.emit("worker", worker);
@@ -10815,6 +10830,7 @@ var BrowserPage = class BrowserPage extends BrowserFrame {
 		this.#emitter.emit("navigate", params["url"], true);
 	}
 	#handleFrameDetached(session, params) {
+		if (this.#closed) return;
 		const frame = params["frameId"];
 		if (!isString(frame) || !this.#accepts(session, frame)) return;
 		const swapped = params["reason"] === "swap";
@@ -10915,7 +10931,10 @@ var BrowserPage = class BrowserPage extends BrowserFrame {
 		const session = params["sessionId"];
 		if (!isRecord(target) || !isString(target["targetId"]) || !isString(session)) return;
 		const category = target["type"];
-		if (owner !== this.#sessionId && category !== "iframe") return;
+		if (owner !== this.#sessionId && category !== "iframe") {
+			this.#detachChild(session, owner);
+			return;
+		}
 		if (category === "worker" || category === "service_worker" || category === "shared_worker") {
 			this.#attachWorker(session, target["targetId"], isString(target["url"]) ? target["url"] : "", category);
 			return;
@@ -11223,6 +11242,7 @@ var BrowserContext = class {
 	#publishing = /* @__PURE__ */ new Map();
 	#observed = /* @__PURE__ */ new WeakSet();
 	#shutdown;
+	#disposal;
 	#reference = 0;
 	constructor(client, id, viewport, writer, emulation, downloads, options) {
 		this.#client = client;
@@ -11247,6 +11267,9 @@ var BrowserContext = class {
 	get id() {
 		return this.#id;
 	}
+	get disposal() {
+		return this.#disposal;
+	}
 	get cookies() {
 		return this.#cookies;
 	}
@@ -11289,14 +11312,14 @@ var BrowserContext = class {
 	destroy() {
 		const active = this.#shutdown;
 		if (active !== void 0) return active;
-		const shutdown = this.#destroyResources();
+		const shutdown = Promise.resolve().then(() => this.#destroyResources());
 		this.#shutdown = shutdown;
 		return shutdown;
 	}
 	close() {
 		const active = this.#shutdown;
 		if (active !== void 0) return active;
-		const shutdown = this.#closeResources();
+		const shutdown = Promise.resolve().then(() => this.#closeResources());
 		this.#shutdown = shutdown;
 		return shutdown;
 	}
@@ -11418,9 +11441,15 @@ var BrowserContext = class {
 			let failure = await settleBrowserTeardown(...[...this.#pages.values()].map((page) => () => page.close()));
 			this.#pages.clear();
 			const id = this.#id;
-			if (id !== void 0) {
-				const settled = await settleBrowserTeardown(() => this.#client.send("Target.disposeBrowserContext", { browserContextId: id }));
-				failure ??= settled;
+			if (id !== void 0) try {
+				await this.#client.send("Target.disposeBrowserContext", { browserContextId: id });
+				this.#disposal = { confirmed: true };
+			} catch (error) {
+				this.#disposal = {
+					confirmed: false,
+					error
+				};
+				failure ??= error;
 			}
 			if (failure !== void 0) throw failure;
 		} finally {