nano-spawn-compat 2.0.1 → 2.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "nano-spawn-compat",
-	"version": "2.0.1",
+	"version": "2.0.2",
 	"description": "Tiny process execution for humans — a better child_process",
 	"license": "MIT",
 	"repository": "leonsilicon/nano-spawn-compat",

package/readme.md

@@ -1,4 +1,4 @@
-> This repo is a fork of [nano-spawn](https://github.com/sindresorhus/nano-spawn) by [@sindresorhus](https://github.com/sindresorhus) without depending on `node:readline` and `node:util` in order to support the [QuickJS-based LLRT runtime](https://github.com/awslabs/llrt).
+> This repo is a fork of [nano-spawn](https://github.com/sindresorhus/nano-spawn) by [@sindresorhus](https://github.com/sindresorhus) without depending on `node:readline`, `node:events`, and `node:util` in order to support the [QuickJS-based LLRT runtime](https://github.com/awslabs/llrt).
 >
 > Why? Because this package is otherwise perfect for the LLRT runtime since it only relies on `child_process.spawn` which is currently the only [implemented export in `llrt_child_process`](https://github.com/awslabs/llrt/blob/36bd4a837d3fd85b5e3246906f23b1873c067503/modules/llrt_child_process/src/lib.rs#L580).
 

package/source/once.js

@@ -0,0 +1,68 @@
+export function once(emitter, event) {
+	return new Promise((resolve, reject) => {
+		function onEvent(...arguments_) {
+			cleanup();
+			resolve(arguments_);
+		}
+
+		function onError(error) {
+			cleanup();
+			reject(error);
+		}
+
+		function cleanup() {
+			if (emitter.off) {
+				emitter.off(event, onEvent);
+				emitter.off('error', onError);
+			}
+
+			if (emitter.removeListener) {
+				emitter.removeListener(event, onEvent);
+				emitter.removeListener('error', onError);
+			}
+		}
+
+		emitter.on(event, onEvent);
+
+		if (event !== 'error') {
+			emitter.on('error', onError);
+		}
+	});
+}
+
+export async function * on(emitter, event) {
+	const queue = [];
+	let resolve_;
+
+	function handler(...arguments_) {
+		if (resolve_) {
+			resolve_(arguments_);
+			resolve_ = null;
+		} else {
+			queue.push(arguments_);
+		}
+	}
+
+	emitter.on(event, handler);
+
+	try {
+		while (true) {
+			if (queue.length > 0) {
+				yield queue.shift();
+			} else {
+				// eslint-disable-next-line no-await-in-loop
+				yield await new Promise(resolve => {
+					resolve_ = resolve;
+				});
+			}
+		}
+	} finally {
+		if (emitter.off) {
+			emitter.off(event, handler);
+		}
+
+		if (emitter.removeListener) {
+			emitter.removeListener(event, handler);
+		}
+	}
+}

package/source/result.js

@@ -1,5 +1,5 @@
-import {once, on} from 'node:events';
 import process from 'node:process';
+import {once, on} from './once.js';
 
 export const getResult = async (nodeChildProcess, {input}, context) => {
 	const instance = await nodeChildProcess;

package/source/spawn.js

@@ -1,6 +1,5 @@
 import {spawn} from 'node:child_process';
-import {once} from 'node:events';
-import process from 'node:process';
+import {once} from './once.js';
 import {applyForceShell} from './windows.js';
 import {getResultError} from './result.js';
 
@@ -10,10 +9,10 @@ export const spawnSubprocess = async (file, commandArguments, options, context)
 		// Not applied with file paths to `.../node` since those indicate a clear intent to use a specific Node version.
 		// This also provides a way to opting out, e.g. using `process.execPath` instead of `node` to discard current CLI flags.
 		// Does not work with shebangs, but those don't work cross-platform anyway.
-		if (['node', 'node.exe'].includes(file.toLowerCase())) {
-			file = process.execPath;
-			commandArguments = [...process.execArgv.filter(flag => !flag.startsWith('--inspect')), ...commandArguments];
-		}
+		// if (['node', 'node.exe'].includes(file.toLowerCase())) {
+		// 	file = process.execPath;
+		// 	commandArguments = [...process.execArgv.filter(flag => !flag.startsWith('--inspect')), ...commandArguments];
+		// }
 
 		[file, commandArguments, options] = await applyForceShell(file, commandArguments, options);
 		[file, commandArguments, options] = concatenateShell(file, commandArguments, options);