Refactor php-wasm-cli to maintain consistency across the different clis

Author: mho22

.gitignore

@@ -64,7 +64,7 @@ php.js.bak
 # @php-wasm/cli saves this file to disk so PHP has access to it.
 # Since it is dynamically generated from the Node.js TLS module,
 # we do not want to commit it to the repository.
-packages/php-wasm/cli/src/ca-bundle.crt
+packages/php-wasm/cli/src/lib/ca-bundle.crt
 
 # PHPUnit
 .phpunit.result.cache

packages/php-wasm/cli/package.json

@@ -21,7 +21,7 @@
 	},
 	"license": "GPL-2.0-or-later",
 	"type": "module",
-	"main": "main.js",
+	"main": "cli.js",
 	"bin": {
 		"php-wasm-cli": "php-wasm.js"
 	},

packages/php-wasm/cli/project.json

@@ -29,7 +29,7 @@
 			"executor": "@nx/vite:build",
 			"outputs": ["{options.outputPath}"],
 			"options": {
-				"main": "dist/packages/php-wasm/cli/main.js",
+				"main": "dist/packages/php-wasm/cli/cli.js",
 				"outputPath": "dist/packages/php-wasm/cli"
 			},
 			"defaultConfiguration": "production",
@@ -45,7 +45,7 @@
 		"dev": {
 			"executor": "nx:run-commands",
 			"options": {
-				"command": "node --no-warnings --experimental-wasm-stack-switching --experimental-wasm-jspi --loader=./packages/meta/src/node-es-module-loader/loader.mts ./packages/php-wasm/cli/src/main.ts",
+				"command": "node --no-warnings --experimental-wasm-stack-switching --experimental-wasm-jspi --loader=./packages/meta/src/node-es-module-loader/loader.mts ./packages/php-wasm/cli/src/cli.ts",
 				"tty": true
 			}
 		},
@@ -57,7 +57,7 @@
 						"command": "node -e \"if (parseInt(process.versions.node) < 22) { console.error('Node.js version 22 or greater is required'); process.exit(1); }\"",
 						"forwardAllArgs": false
 					},
-					"node --stack-trace-limit=100 --inspect --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts --watch ./packages/php-wasm/cli/src/main.ts"
+					"node --stack-trace-limit=100 --inspect --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts --watch ./packages/php-wasm/cli/src/cli.ts"
 				],
 				"tty": true,
 				"parallel": false
@@ -71,7 +71,7 @@
 						"command": "node -e \"if (parseInt(process.versions.node) < 22) { console.error('Node.js version 22 or greater is required'); process.exit(1); }\"",
 						"forwardAllArgs": false
 					},
-					"node --inspect --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts ./packages/php-wasm/cli/src/main.ts"
+					"node --inspect --experimental-strip-types --experimental-transform-types --import ./packages/meta/src/node-es-module-loader/register.mts ./packages/php-wasm/cli/src/cli.ts"
 				],
 				"tty": true,
 				"parallel": false
@@ -80,7 +80,7 @@
 		"start": {
 			"executor": "@wp-playground/nx-extensions:built-script",
 			"options": {
-				"scriptPath": "dist/packages/php-wasm/cli/main.js"
+				"scriptPath": "dist/packages/php-wasm/cli/php-wasm.js"
 			},
 			"dependsOn": ["build"]
 		},

packages/php-wasm/cli/public/php-wasm.js

@@ -1,2 +1,2 @@
 #!/usr/bin/env node
-import './main.js';
+import './run-cli.js';

packages/php-wasm/cli/src/cli.ts

@@ -0,0 +1,3 @@
+import { parseOptionsAndRunCLI } from './lib/run-cli';
+
+parseOptionsAndRunCLI();

packages/php-wasm/cli/src/php.ini renamed to packages/php-wasm/cli/src/lib/php.ini

@@ -1,56 +1,117 @@
-/**
- * A CLI script that runs PHP CLI via the WebAssembly build.
- */
+import yargs from 'yargs';
+import { hideBin } from 'yargs/helpers';
 import os from 'os';
 import { writeFileSync, existsSync, mkdtempSync, chmodSync } from 'fs';
 import { rootCertificates } from 'tls';
-
 import {
 	LatestSupportedPHPVersion,
 	SupportedPHPVersionsList,
 } from '@php-wasm/universal';
 import type { SupportedPHPVersion } from '@php-wasm/universal';
-
 import { FileLockManagerForNode } from '@php-wasm/node';
 import { PHP } from '@php-wasm/universal';
 import { loadNodeRuntime, useHostFilesystem } from '@php-wasm/node';
 import { startBridge } from '@php-wasm/xdebug-bridge';
 import path from 'path';
 
-let args = process.argv.slice(2);
-if (!args.length) {
-	args = ['--help'];
+interface CLIDefaults {
+	directive: string[];
+	config: string;
 }
 
-const baseUrl = (import.meta || {}).url;
+interface CLIArgs {
+	_: string[];
+	config: string;
+	directive: string[];
+	xdebug?: boolean;
+	experimentalDevtools?: boolean;
+}
 
-// Write the ca-bundle.crt file to disk so that PHP can find it.
-const caBundlePath = new URL('ca-bundle.crt', baseUrl).pathname;
-if (!existsSync(caBundlePath)) {
-	writeFileSync(caBundlePath, rootCertificates.join('\n'));
+function parseCliArgs(defaults: CLIDefaults) {
+	return yargs(hideBin(process.argv))
+		.usage(
+			`
+PHP.wasm CLI
+
+Usage: php-wasm-cli <command> [options]
+		`
+		)
+		.option('config', {
+			alias: 'c',
+			type: 'string',
+			describe: 'Path to configuration file',
+			default: defaults.config,
+		})
+		.option('directive', {
+			alias: 'd',
+			type: 'array',
+			describe: 'Set configuration directives (key=value)',
+			default: defaults.directive,
+		})
+		.option('xdebug', {
+			type: 'boolean',
+			describe: 'Enable Xdebug.',
+			default: false,
+		})
+		.option('experimental-devtools', {
+			type: 'boolean',
+			describe: 'Enable experimental browser development tools.',
+			default: false,
+		})
+		.strictCommands()
+		.help()
+		.epilog(
+			`
+Examples:
+  php-wasm-cli                                             # Start with default settings
+  php-wasm-cli --config /path/to/php.ini           # Run PHP CLI with custom config file
+  php-wasm-cli --xdebug --experimental-devtools             # Enable Xdebug and Devtools
+		`
+		)
+		.parseSync() as CLIArgs;
 }
-args.unshift('-d', `openssl.cafile=${caBundlePath}`);
 
-async function run() {
-	const defaultPhpIniPath = new URL('php.ini', baseUrl).pathname;
+function mapArgsArray(args: CLIArgs): string[] {
+	const argv: string[] = [];
+
+	if (args.config) {
+		argv.push('-c', args.config);
+	}
+
+	if (Array.isArray(args.directive)) {
+		args.directive.forEach((d) => {
+			argv.push('-d', d);
+		});
+	}
+
+	if (Array.isArray(args._)) {
+		argv.push(...args._);
+	}
+
+	return argv;
+}
+
+export async function parseOptionsAndRunCLI(): Promise<void> {
 	const phpVersion = (process.env['PHP'] ||
 		LatestSupportedPHPVersion) as SupportedPHPVersion;
 	if (!SupportedPHPVersionsList.includes(phpVersion)) {
 		throw new Error(`Unsupported PHP version ${phpVersion}`);
 	}
 
-	const hasXdebugOption = args.some((arg) => arg.startsWith('--xdebug'));
-	if (hasXdebugOption) {
-		args = args.filter((arg) => arg !== '--xdebug');
-	}
+	const baseUrl = (import.meta || {}).url;
 
-	const hasDevtoolsOption = args.some((arg) =>
-		arg.startsWith('--experimental-devtools')
-	);
-	if (hasDevtoolsOption) {
-		args = args.filter((arg) => arg !== '--experimental-devtools');
+	const defaultPhpIniPath = new URL('php.ini', baseUrl).pathname;
+	// Write the ca-bundle.crt file to disk so that PHP can find it.
+	const caBundlePath = new URL('ca-bundle.crt', baseUrl).pathname;
+	if (!existsSync(caBundlePath)) {
+		writeFileSync(caBundlePath, rootCertificates.join('\n'));
 	}
 
+	const args = parseCliArgs({
+		directive: [`openssl.cafile=${caBundlePath}`],
+		config: defaultPhpIniPath,
+	});
+
 	// npm scripts set the TMPDIR env variable
 	// PHP accepts a TMPDIR env variable and expects it to
 	// be a writable directory within the PHP filesystem.
@@ -99,24 +160,20 @@ ${process.argv[0]} ${process.execArgv.join(' ')} ${process.argv[1]}
 					PATH: `${tempDir}:${envVariables['PATH']}`,
 				},
 			},
-			withXdebug: hasXdebugOption,
+			withXdebug: args.xdebug,
 		})
 	);
 
 	useHostFilesystem(php);
 
-	if (hasDevtoolsOption && hasXdebugOption) {
+	if (args.experimentalDevtools && args.xdebug) {
 		const bridge = await startBridge({ breakOnFirstLine: true });
 
 		bridge.start();
 	}
 
-	const hasMinusCOption = args.some((arg) => arg.startsWith('-c'));
-	if (!hasMinusCOption) {
-		args.unshift('-c', defaultPhpIniPath);
-	}
+	const response = await php.cli(['php', ...mapArgsArray(args)]);
 
-	const response = await php.cli(['php', ...args]);
 	response.stderr.pipeTo(
 		new WritableStream({
 			write(chunk) {
@@ -147,5 +204,3 @@ ${process.argv[0]} ${process.execArgv.join(' ')} ${process.argv[1]}
 			}, 100);
 		});
 }
-
-run();

packages/php-wasm/cli/src/main.ts renamed to packages/php-wasm/cli/src/lib/run-cli.ts

@@ -35,7 +35,7 @@ export default defineConfig({
 				'ws',
 				'os',
 			],
-			input: 'packages/php-wasm/cli/src/main.ts',
+			input: 'packages/php-wasm/cli/src/cli.ts',
 			output: {
 				format: 'esm',
 				entryFileNames: '[name].js',

packages/php-wasm/cli/vite.config.ts

@@ -1,3 +1,3 @@
-import { main } from './lib/run-cli';
+import { parseOptionsAndRunCLI } from './lib/run-cli';
 
-main();
+parseOptionsAndRunCLI();

packages/php-wasm/xdebug-bridge/src/cli.ts

@@ -31,13 +31,13 @@ Usage: xdebug-bridge [options]
 		.option('port', {
 			alias: 'p',
 			type: 'number',
-			description: 'Xdebug port to listen on',
+			describe: 'Xdebug port to listen on',
 			default: 9003,
 		})
 		.option('host', {
 			alias: 'h',
 			type: 'string',
-			description: 'Xdebug host to bind to',
+			describe: 'Xdebug host to bind to',
 			default: 'localhost',
 		})
 		.option('php-root', {
@@ -65,7 +65,7 @@ Examples:
 		.parseSync() as CLIArgs;
 }
 
-export async function main(): Promise<void> {
+export async function parseOptionsAndRunCLI(): Promise<void> {
 	const args = parseCliArgs();
 
 	if (args.help) {