single-file-cli 2.0.29 → 2.0.30

package/deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@single-file/single-file-cli",
-  "version": "2.0.29",
+  "version": "2.0.30",
   "description": "SingleFile CLI",
   "nodeModulesDir": true,
   "exports": {

package/lib/browser.js

@@ -34,6 +34,8 @@ async function launchBrowser(options = {}, indexPath = 0) {
 	const executablePath = options.executablePath || BROWSER_PATHS[build.os][indexPath];
 	const args = Array.from(BROWSER_ARGS);
 	profilePath = await makeTempDir();
+	const debugPort = await getDebugPort();
+	args.push("--remote-debugging-port=" + debugPort);
 	if (options.headless && !options.debug) {
 		args.push("--headless");
 	} else {
@@ -64,7 +66,7 @@ async function launchBrowser(options = {}, indexPath = 0) {
 	} catch (error) {
 		if (error instanceof errors.NotFound) {
 			if (indexPath + 1 < BROWSER_PATHS[build.os].length) {
-				await launchBrowser(options, indexPath + 1);
+				return launchBrowser(options, indexPath + 1);
 			} else {
 				throw error;
 			}
@@ -73,7 +75,20 @@ async function launchBrowser(options = {}, indexPath = 0) {
 		}
 	}
 	child.ref();
-	return child;
+	return debugPort;
+}
+
+async function getDebugPort(port = 9222) {
+	try {
+		await fetch("http://localhost:" + port + "/json/version");
+	} catch (error) {
+		return port;
+	}
+	if (port < 9286) {
+		return getDebugPort(port + 1);
+	} else {
+		throw new Error("No available debugging port");
+	}
 }
 
 async function closeBrowser() {

package/lib/cdp-client.js

@@ -25,22 +25,21 @@
 
 import { launchBrowser, closeBrowser } from "./browser.js";
 import { getScriptSource, getHookScriptSource } from "./single-file-script.js";
-import { CDP } from "simple-cdp";
+import { CDP, options } from "simple-cdp";
 
 const LOAD_TIMEOUT_ERROR = "ERR_LOAD_TIMEOUT";
-const DEFAULT_NETWORK_STATE = "networkIdle";
 const NETWORK_STATES = ["InteractiveTime", "networkIdle", "networkAlmostIdle", "load", "DOMContentLoaded"];
 const MINIMIZED_WINDOW_STATE = "minimized";
 const SINGLE_FILE_WORLD_NAME = "singlefile";
 const EMPTY_PAGE_URL = "about:blank";
 
-export { initialize, getPageData, hasPageTargets, closeBrowser };
+export { initialize, getPageData, closeBrowser };
 
-async function initialize(options) {
+async function initialize(singleFileOptions) {
 	if (options.browserRemoteDebuggingURL) {
-		CDP.remoteDebuggingUrl = options.browserRemoteDebuggingURL;
+		options.apiUrl = singleFileOptions.browserRemoteDebuggingURL;
 	} else {
-		await launchBrowser(getBrowserOptions(options));
+		options.apiUrl = "http://localhost:" + (await launchBrowser(getBrowserOptions(singleFileOptions)));
 	}
 }
 
@@ -136,11 +135,6 @@ async function getPageData(options) {
 	}
 }
 
-async function hasPageTargets() {
-	const targets = await CDP.getTargets();
-	return targets.filter(target => target.type === "page").length > 0;
-}
-
 async function loadPage({ Page, Runtime }, options) {
 	await Runtime.enable();
 	await Page.enable();
@@ -197,7 +191,7 @@ async function getTopFrameContextId({ Page, Runtime }, options) {
 
 				function onLifecycleEvent({ params }) {
 					const { frameId, name } = params;
-					if (frameId === topFrameId && (name === options.browserWaitUntil || name === DEFAULT_NETWORK_STATE)) {
+					if (frameId === topFrameId && name === options.browserWaitUntil) {
 						removeListeners();
 						resolve();
 					}

package/lib/constants.js

@@ -52,7 +52,6 @@ const BROWSER_ARGS = [
 	"--enable-use-zoom-for-dsf=false",
 	"--no-sandbox",
 	"--no-startup-window",
-	"--remote-debugging-port=9222",
 	"--bwsi"
 ];
 const BROWSER_PATHS = {

package/lib/version.js

@@ -1 +1 @@
-export const version = "2.0.29";
+export const version = "2.0.30";

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "single-file-cli",
-	"version": "2.0.29",
+	"version": "2.0.30",
 	"description": "SingleFile CLI",
 	"author": "Gildas Lormeau",
 	"engines": {

package/single-file-cli-api.js

@@ -151,7 +151,7 @@ async function finish(options) {
 			}
 		}
 	}
-	if (!options.browserDebug && !options.browserRemoteDebuggingURL && !(await backend.hasPageTargets())) {
+	if (!options.browserDebug && !options.browserRemoteDebuggingURL) {
 		return backend.closeBrowser();
 	}
 }