single-file-cli 2.0.25 → 2.0.27

package/deno.json

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

package/lib/browser.js

@@ -34,7 +34,7 @@ async function launchBrowser(options = {}, indexPath = 0) {
 	const executablePath = options.executablePath || BROWSER_PATHS[build.os][indexPath];
 	const args = Array.from(BROWSER_ARGS);
 	profilePath = await makeTempDir();
-	if (options.headless) {
+	if (options.headless && !options.debug) {
 		args.push("--headless");
 	} else {
 		args.push("--start-maximized");

package/lib/cdp-client.js

@@ -142,12 +142,11 @@ async function hasPageTargets() {
 }
 
 async function loadPage({ Page, Runtime }, options) {
-	let loadTimeoutAbortController, loadTimeoutAbortSignal;
 	await Runtime.enable();
 	await Page.enable();
+	const loadTimeoutAbortController = new AbortController();
+	const loadTimeoutAbortSignal = loadTimeoutAbortController.signal;
 	try {
-		loadTimeoutAbortController = new AbortController();
-		loadTimeoutAbortSignal = loadTimeoutAbortController.signal;
 		const [contextId] = await Promise.race([
 			Promise.all([getTopFrameContextId({ Page, Runtime }, options), Page.navigate({ url: options.url })]),
 			waitForLoadTimeout(loadTimeoutAbortSignal, options.browserLoadMaxTime)
@@ -166,52 +165,50 @@ function getTopFrameContextId({ Page, Runtime }, options) {
 	return new Promise((resolve, reject) => {
 		const FRAME_NAVIGATED_EVENT = "frameNavigated";
 		const CONTEXT_CREATED_EVENT = "executionContextCreated";
-		const CONTEXT_DESTROYED_EVENT = "executionContextDestroyed";
 		const contextIds = [];
-		let frameId, navigationAbortController;
+		let frameId;
 		Page.addEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
 		Runtime.addEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
-		Runtime.addEventListener(CONTEXT_DESTROYED_EVENT, onExecutionContextDestroyed);
 
 		async function onFrameNavigated({ params }) {
-			const { frame } = params;
-			if (frame.unreachableUrl) {
-				reject(new Error("Unreachable URL: " + frame.unreachableUrl));
-			} else if (!frame.parentId) {
-				frameId = frame.id;
-				if (navigationAbortController) {
-					navigationAbortController.abort();
-					contextIds.length = 0;
+			try {
+				const { frame } = params;
+				if (!frame.parentId) {
+					if (frame.unreachableUrl) {
+						reject(new Error("Unreachable URL: " + frame.unreachableUrl));
+					} else {
+						frameId = frame.id;
+						await onTopFrameNavigated();
+					}
 				}
-				navigationAbortController = new AbortController();
-				await onTopFrameNavigated(navigationAbortController.signal);
+			} finally {
+				Page.removeEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
+				Runtime.removeEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
 			}
 		}
 
-		async function onTopFrameNavigated(navigationSignal) {
-			await waitForPageReady({ Page }, frameId, options, navigationSignal);
-			if (!navigationSignal.aborted) {
-				Page.removeEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
-				Runtime.removeEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
-				Runtime.removeEventListener(CONTEXT_DESTROYED_EVENT, onExecutionContextDestroyed);
-				let contextId;
-				if (contextIds.length) {
-					let contextIdIndex = 0;
-					do {
-						if (await testExecutionContext(contextIds[contextIdIndex])) {
-							contextId = contextIds[contextIdIndex];
-						}
-						contextIdIndex++;
-					} while (contextId === undefined && contextIdIndex < contextIds.length && !navigationSignal.aborted);
-				}
-				if (!navigationSignal.aborted) {
-					if (contextId === undefined) {
-						reject(new Error("Execution context not found"));
-					} else {
-						resolve(contextId);
+		async function onTopFrameNavigated() {
+			await waitForPageReady({ Page }, frameId, options);
+			const contextId = await getContextId();
+			if (contextId === undefined) {
+				reject(new Error("Execution context not found"));
+			} else {
+				resolve(contextId);
+			}
+		}
+
+		async function getContextId() {
+			let contextId;
+			if (contextIds.length) {
+				let contextIdIndex = 0;
+				do {
+					if (await testExecutionContext(contextIds[contextIdIndex])) {
+						contextId = contextIds[contextIdIndex];
 					}
-				}
+					contextIdIndex++;
+				} while (contextId === undefined && contextIdIndex < contextIds.length);
 			}
+			return contextId;
 		}
 
 		function onExecutionContextCreated({ params }) {
@@ -221,14 +218,6 @@ function getTopFrameContextId({ Page, Runtime }, options) {
 				contextIds.push(context.id);
 			}
 		}
-
-		function onExecutionContextDestroyed({ params }) {
-			const { executionContextId } = params;
-			const index = contextIds.indexOf(executionContextId);
-			if (index > -1) {
-				contextIds.splice(index, 1);
-			}
-		}
 	});
 
 	async function testExecutionContext(contextId) {
@@ -250,6 +239,7 @@ function waitForLoadTimeout(abortSignal, maxDelay) {
 		const ABORT_EVENT = "abort";
 		abortSignal.addEventListener(ABORT_EVENT, onAbort);
 		const timeoutId = setTimeout(() => {
+			abortSignal.removeEventListener(ABORT_EVENT, onAbort);
 			const error = new Error("Load timeout");
 			error.code = LOAD_TIMEOUT_ERROR;
 			reject(error);
@@ -263,7 +253,7 @@ function waitForLoadTimeout(abortSignal, maxDelay) {
 	});
 }
 
-async function waitForPageReady({ Page }, topFrameId, options, navigationSignal) {
+async function waitForPageReady({ Page }, topFrameId, options) {
 	await Page.setLifecycleEventsEnabled({ enabled: true });
 	try {
 		await new Promise(resolve => {
@@ -272,16 +262,14 @@ async function waitForPageReady({ Page }, topFrameId, options, navigationSignal)
 
 			function onLifecycleEvent({ params }) {
 				const { name, frameId } = params;
-				if (frameId === topFrameId && (name === options.browserWaitUntil || name === DEFAULT_NETWORK_STATE || navigationSignal.aborted)) {
+				if (frameId === topFrameId && (name === options.browserWaitUntil || name === DEFAULT_NETWORK_STATE)) {
 					Page.removeEventListener(LIFE_CYCLE_EVENT, onLifecycleEvent);
 					resolve();
 				}
 			}
 		});
 	} finally {
-		if (!navigationSignal.aborted) {
-			await Page.setLifecycleEventsEnabled({ enabled: false });
-		}
+		await Page.setLifecycleEventsEnabled({ enabled: false });
 	}
 }
 
@@ -306,7 +294,7 @@ async function waitForDebuggerReady({ Debugger }) {
 function getBrowserOptions(options) {
 	const browserOptions = {};
 	browserOptions.args = options.browserArgs;
-	browserOptions.headless = options.browserHeadless && !options.browserDebug;
+	browserOptions.headless = options.browserHeadless;
 	browserOptions.executablePath = options.browserExecutablePath;
 	browserOptions.debug = options.browserDebug;
 	browserOptions.disableWebSecurity = options.browserDisableWebSecurity === undefined || options.browserDisableWebSecurity;

package/lib/version.js

@@ -1 +1 @@
-export const version = "2.0.25";
+export const version = "2.0.27";

package/package.json

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

package/single-file-node.bat

@@ -1,2 +1,2 @@
 @echo off
-node "%~dp0\single-file" %*
+node "%~dp0\single-file-node.js" %*