pi-agent-browser-native 0.9.3 → 0.9.4

package/scripts/platform-smoke.mjs

@@ -17,7 +17,9 @@ const require = createRequire(import.meta.url);
 let config;
 try {
 	config = require(resolve(repoRoot, "platform-smoke.config.mjs"));
-	if (config.default) config = config.default;
+	if (config.default) {
+		config = config.default;
+	}
 } catch {
 	config = null;
 }
@@ -75,26 +77,23 @@ export function parseArgs(argv = process.argv.slice(2)) {
 	const parsed = { command: null, target: null, suite: null };
 	for (let index = 0; index < argv.length; index += 1) {
 		const arg = argv[index];
-		if (arg === "--help" || arg === "-h") {
+		if (["--help", "-h"].includes(arg)) {
 			parsed.command = "help";
 			return parsed;
 		}
-		if (arg === "doctor" || arg === "run") {
-			if (parsed.command) throw new Error(`multiple commands provided: ${parsed.command}, ${arg}`);
+		if (["doctor", "run"].includes(arg)) {
+			if (parsed.command) {
+				throw new Error(`multiple commands provided: ${parsed.command}, ${arg}`);
+			}
 			parsed.command = arg;
 			continue;
 		}
-		if (arg === "--target") {
-			const value = argv[index + 1];
-			if (!value || value.startsWith("-")) throw new Error("--target requires a value");
-			parsed.target = value;
-			index += 1;
-			continue;
-		}
-		if (arg === "--suite") {
+		if (arg === "--target" || arg === "--suite") {
 			const value = argv[index + 1];
-			if (!value || value.startsWith("-")) throw new Error("--suite requires a value");
-			parsed.suite = value;
+			if (!value || value.startsWith("-")) {
+				throw new Error(`${arg} requires a value`);
+			}
+			parsed[arg.slice(2)] = value;
 			index += 1;
 			continue;
 		}
@@ -105,7 +104,39 @@ export function parseArgs(argv = process.argv.slice(2)) {
 
 function validateNames(kind, names, allowed) {
 	const invalid = names.filter((name) => !allowed.includes(name));
-	if (invalid.length > 0) throw new Error(`unknown ${kind}: ${invalid.join(", ")}`);
+	if (invalid.length > 0) {
+		throw new Error(`unknown ${kind}: ${invalid.join(", ")}`);
+	}
+}
+
+async function runTargets(args) {
+	const { runTargetSuites } = await import("./platform-smoke/targets.mjs");
+	const targets = args.target
+		? args.target
+				.split(",")
+				.map((name) => name.trim())
+				.filter(Boolean)
+		: config.requiredTargets;
+	const suites = args.suite ? [args.suite] : config.requiredSuites;
+	validateNames("target", targets, config.requiredTargets);
+	validateNames("suite", suites, config.requiredSuites);
+	const results = await Promise.all(
+		targets.map(async (targetName) => {
+			console.log(`\n=== Target: ${targetName} ===`);
+			return { targetName, result: await runTargetSuites(config, targetName, suites) };
+		}),
+	);
+	const failed = results.filter(({ result }) => !result.ok);
+	if (failed.length > 0) {
+		console.error(
+			`\nPlatform smoke failed for ${failed.map(({ targetName }) => targetName).join(", ")}. See ${config.artifactRoot}.`,
+		);
+		return 1;
+	}
+	console.log(
+		`\nPlatform smoke passed for ${results.map(({ targetName }) => targetName).join(", ")}.`,
+	);
+	return 0;
 }
 
 export async function main(argv = process.argv.slice(2)) {
@@ -114,7 +145,9 @@ export async function main(argv = process.argv.slice(2)) {
 		printHelp();
 		return args.command === "help" ? 0 : 2;
 	}
-	if (!config) throw new Error("platform-smoke.config.mjs not found or invalid");
+	if (!config) {
+		throw new Error("platform-smoke.config.mjs not found or invalid");
+	}
 
 	if (args.command === "doctor") {
 		const { runDoctor } = await import("./platform-smoke/doctor.mjs");
@@ -123,37 +156,19 @@ export async function main(argv = process.argv.slice(2)) {
 	}
 
 	if (args.command === "run") {
-		const { runTargetSuites } = await import("./platform-smoke/targets.mjs");
-		const targets = args.target ? args.target.split(",").map((name) => name.trim()).filter(Boolean) : config.requiredTargets;
-		const suites = args.suite ? [args.suite] : config.requiredSuites;
-		validateNames("target", targets, config.requiredTargets);
-		validateNames("suite", suites, config.requiredSuites);
-		const runs = targets.map(async (targetName) => {
-			console.log(`\n=== Target: ${targetName} ===`);
-			const result = await runTargetSuites(config, targetName, suites);
-			return { targetName, result };
-		});
-		const results = await Promise.all(runs);
-		const failed = results.filter(({ result }) => !result.ok);
-		if (failed.length > 0) {
-			console.error(`\nPlatform smoke failed for ${failed.map(({ targetName }) => targetName).join(", ")}. See ${config.artifactRoot}.`);
-			return 1;
-		}
-		console.log(`\nPlatform smoke passed for ${results.map(({ targetName }) => targetName).join(", ")}.`);
-		return 0;
+		return runTargets(args);
 	}
 
 	throw new Error(`unknown command: ${args.command}`);
 }
 
 if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {
-	main().then(
-		(exitCode) => {
+	main()
+		.then((exitCode) => {
 			process.exitCode = exitCode;
-		},
-		(error) => {
+		})
+		.catch((error) => {
 			console.error(error instanceof Error ? error.message : String(error));
 			process.exitCode = 1;
-		},
-	);
+		});
 }

package/scripts/prepare.mjs

@@ -15,6 +15,7 @@ import { promisify } from "node:util";
 const execFile = promisify(execFileCallback);
 const REQUIRED_SOURCE_BUILD_MODULES = [
 	"typescript",
+	"esbuild",
 	"typebox",
 	"@earendil-works/pi-coding-agent",
 	"@earendil-works/pi-tui",
@@ -34,11 +35,15 @@ async function runNpmInstallDevDependencies() {
 	const npmExecPath = process.env.npm_execpath;
 	const options = process.platform === "win32" ? { shell: true } : {};
 	if (npmExecPath) {
-		await execFile(process.execPath, [npmExecPath, "install", "--include=dev", "--ignore-scripts"], {
-			...options,
-			cwd: process.cwd(),
-			maxBuffer: 20 * 1024 * 1024,
-		});
+		await execFile(
+			process.execPath,
+			[npmExecPath, "install", "--include=dev", "--ignore-scripts"],
+			{
+				...options,
+				cwd: process.cwd(),
+				maxBuffer: 20 * 1024 * 1024,
+			},
+		);
 		return;
 	}
 	await execFile("npm", ["install", "--include=dev", "--ignore-scripts"], {
@@ -59,8 +64,12 @@ async function main() {
 }
 
 main().catch((error) => {
-	if (error?.stdout) process.stdout.write(error.stdout);
-	if (error?.stderr) process.stderr.write(error.stderr);
+	if (error?.stdout) {
+		process.stdout.write(error.stdout);
+	}
+	if (error?.stderr) {
+		process.stderr.write(error.stderr);
+	}
 	console.error(error instanceof Error ? error.message : String(error));
 	process.exitCode = 1;
 });