single-file-cli 2.0.27 → 2.0.28

package/deno.json

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

package/lib/cdp-client.js

@@ -161,64 +161,78 @@ async function loadPage({ Page, Runtime }, options) {
 	}
 }
 
-function getTopFrameContextId({ Page, Runtime }, options) {
-	return new Promise((resolve, reject) => {
-		const FRAME_NAVIGATED_EVENT = "frameNavigated";
-		const CONTEXT_CREATED_EVENT = "executionContextCreated";
-		const contextIds = [];
-		let frameId;
+async function getTopFrameContextId({ Page, Runtime }, options) {
+	const FRAME_NAVIGATED_EVENT = "frameNavigated";
+	const CONTEXT_CREATED_EVENT = "executionContextCreated";
+	const contextIds = [];
+	let topFrameId;
+	try {
 		Page.addEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
 		Runtime.addEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
-
-		async function onFrameNavigated({ params }) {
-			try {
-				const { frame } = params;
-				if (!frame.parentId) {
-					if (frame.unreachableUrl) {
-						reject(new Error("Unreachable URL: " + frame.unreachableUrl));
-					} else {
-						frameId = frame.id;
-						await onTopFrameNavigated();
-					}
-				}
-			} finally {
-				Page.removeEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
-				Runtime.removeEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
-			}
+		await waitForPageReady({ Page }, options);
+		const contextId = await getContextId();
+		if (contextId === undefined) {
+			throw new Error("Execution context not found");
+		} else {
+			return contextId;
 		}
+	} finally {
+		Page.removeEventListener(FRAME_NAVIGATED_EVENT, onFrameNavigated);
+		Runtime.removeEventListener(CONTEXT_CREATED_EVENT, onExecutionContextCreated);
+	}
 
-		async function onTopFrameNavigated() {
-			await waitForPageReady({ Page }, frameId, options);
-			const contextId = await getContextId();
-			if (contextId === undefined) {
-				reject(new Error("Execution context not found"));
+	function onFrameNavigated({ params }) {
+		const { frame } = params;
+		if (!frame.parentId) {
+			if (frame.unreachableUrl) {
+				throw new Error("Unreachable URL: " + frame.unreachableUrl);
 			} else {
-				resolve(contextId);
+				topFrameId = frame.id;
 			}
 		}
+	}
 
-		async function getContextId() {
-			let contextId;
-			if (contextIds.length) {
-				let contextIdIndex = 0;
-				do {
-					if (await testExecutionContext(contextIds[contextIdIndex])) {
-						contextId = contextIds[contextIdIndex];
+	function onExecutionContextCreated({ params }) {
+		const { context } = params;
+		const { name, auxData = {} } = context;
+		if (name === SINGLE_FILE_WORLD_NAME && topFrameId !== undefined && auxData.frameId === topFrameId) {
+			contextIds.unshift(context.id);
+		}
+	}
+
+	async function waitForPageReady({ Page }, options) {
+		await Page.setLifecycleEventsEnabled({ enabled: true });
+		try {
+			await new Promise(resolve => {
+				const LIFE_CYCLE_EVENT = "lifecycleEvent";
+				Page.addEventListener(LIFE_CYCLE_EVENT, onLifecycleEvent);
+
+				function onLifecycleEvent({ params }) {
+					const { frameId, name } = params;
+					if (frameId === topFrameId && (name === options.browserWaitUntil || name === DEFAULT_NETWORK_STATE)) {
+						Page.removeEventListener(LIFE_CYCLE_EVENT, onLifecycleEvent);
+						resolve();
 					}
-					contextIdIndex++;
-				} while (contextId === undefined && contextIdIndex < contextIds.length);
-			}
-			return contextId;
+				}
+			});
+		} finally {
+			await Page.setLifecycleEventsEnabled({ enabled: false });
 		}
+	}
 
-		function onExecutionContextCreated({ params }) {
-			const { context } = params;
-			const { name, origin, auxData } = context;
-			if (frameId !== undefined && origin === "://" && name === SINGLE_FILE_WORLD_NAME && auxData && auxData.frameId === frameId) {
-				contextIds.push(context.id);
-			}
+	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;
+	}
 
 	async function testExecutionContext(contextId) {
 		try {
@@ -253,26 +267,6 @@ function waitForLoadTimeout(abortSignal, maxDelay) {
 	});
 }
 
-async function waitForPageReady({ Page }, topFrameId, options) {
-	await Page.setLifecycleEventsEnabled({ enabled: true });
-	try {
-		await new Promise(resolve => {
-			const LIFE_CYCLE_EVENT = "lifecycleEvent";
-			Page.addEventListener(LIFE_CYCLE_EVENT, onLifecycleEvent);
-
-			function onLifecycleEvent({ params }) {
-				const { name, frameId } = params;
-				if (frameId === topFrameId && (name === options.browserWaitUntil || name === DEFAULT_NETWORK_STATE)) {
-					Page.removeEventListener(LIFE_CYCLE_EVENT, onLifecycleEvent);
-					resolve();
-				}
-			}
-		});
-	} finally {
-		await Page.setLifecycleEventsEnabled({ enabled: false });
-	}
-}
-
 async function waitForDebuggerReady({ Debugger }) {
 	await Debugger.enable();
 	try {

package/lib/deno-polyfill.js

@@ -80,7 +80,7 @@ const Command = DENO_RUNTIME_DETECTED ? Deno.Command : class Command {
 		return {
 			status: new Promise((resolve, reject) => {
 				child.on("exit", code => {
-					if (code === 0 || code === 143) {
+					if (code === 0 || code === 143 || code === 130) {
 						resolve();
 					} else {
 						reject(new Error(`Process exited with code ${code}`));
@@ -108,6 +108,7 @@ const DenoAPI = {
 	remove,
 	stdout,
 	exit,
+	addSignalListener,
 	build,
 	errors,
 	Command
@@ -207,6 +208,14 @@ function exit(code) {
 	}
 }
 
+function addSignalListener(signal, listener) {
+	if (DENO_RUNTIME_DETECTED) {
+		Deno.addSignalListener(signal, listener);
+	} else {
+		process.once(signal, listener);
+	}
+}
+
 async function dirname(filePath) {
 	return path.dirname(filePath);
 }

package/lib/version.js

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

package/package.json

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

package/single-file-launcher.js

@@ -22,10 +22,21 @@
  */
 
 import { initialize } from "./single-file-cli-api.js";
+import { closeBrowser } from "./lib/browser.js";
 import { Deno } from "./lib/deno-polyfill.js";
 import options from "./options.js";
 
-const { readTextFile, exit } = Deno;
+const { readTextFile, exit, addSignalListener } = Deno;
+
+addSignalListener("SIGTERM", async () => {
+	await closeBrowser();
+	exit();
+});
+
+addSignalListener("SIGINT", async () => {
+	await closeBrowser();
+	exit();
+});
 
 export { run };
 
@@ -104,4 +115,4 @@ function parseCookies(textValue) {
 			}
 		})
 		.filter(cookieData => cookieData);
-}
+}