Lint

Author: adamziel

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

@@ -20,7 +20,7 @@ interface CLIArgs {
 }
 
 function parseCliArgs(): CLIArgs {
-	return yargs(hideBin(process.argv))
+	const { _, $0, ...parsed } = yargs<CLIArgs>(hideBin(process.argv))
 		.usage(
 			`
 XDebug Bridge Server CLI
@@ -62,7 +62,10 @@ Examples:
 		`
 		)
 		.wrap(null)
-		.parseSync() as CLIArgs;
+		.strict()
+		.parseSync();
+
+	return parsed;
 }
 
 export async function main(): Promise<void> {

packages/playground/cli/src/run-cli.ts

@@ -338,31 +338,41 @@ export async function parseOptionsAndRunCLI(argsToParse: string[]) {
 					args['wordpressInstallMode'] = 'do-not-attempt-installing';
 				}
 
-				if (args.wp !== undefined && !isValidWordPressSlug(args.wp)) {
+				if (
+					args['wp'] !== undefined &&
+					typeof args['wp'] === 'string' &&
+					!isValidWordPressSlug(args['wp'])
+				) {
 					try {
 						// Check if is valid URL
-						new URL(args.wp);
+						new URL(args['wp']);
 					} catch {
 						throw new Error(
 							'Unrecognized WordPress version. Please use "latest", a URL, or a numeric version such as "6.2", "6.0.1", "6.2-beta1", or "6.2-RC1"'
 						);
 					}
 				}
 
-				if (args['site-url'] !== undefined && args['site-url'] !== '') {
+				const siteUrlArg = args['site-url'];
+				if (
+					typeof siteUrlArg === 'string' &&
+					siteUrlArg.trim() !== ''
+				) {
 					try {
-						new URL(args['site-url']);
+						new URL(siteUrlArg);
 					} catch {
 						throw new Error(
-							`Invalid site-url "${args['site-url']}". Please provide a valid URL (e.g., http://localhost:8080 or https://example.com)`
+							`Invalid site-url "${siteUrlArg}". Please provide a valid URL (e.g., http://localhost:8080 or https://example.com)`
 						);
 					}
 				}
 
 				if (args['auto-mount']) {
 					let autoMountIsDir = false;
 					try {
-						const autoMountStats = fs.statSync(args['auto-mount']);
+						const autoMountStats = fs.statSync(
+							args['auto-mount'] as string
+						);
 						autoMountIsDir = autoMountStats.isDirectory();
 					} catch {
 						autoMountIsDir = false;
@@ -382,7 +392,11 @@ export async function parseOptionsAndRunCLI(argsToParse: string[]) {
 							'The --experimental-multi-worker flag is only supported when running the server command.'
 						);
 					}
-					if (args['experimental-multi-worker'] <= 1) {
+					if (
+						args['experimental-multi-worker'] !== undefined &&
+						typeof args['experimental-multi-worker'] === 'number' &&
+						args['experimental-multi-worker'] <= 1
+					) {
 						throw new Error(
 							'The --experimental-multi-worker flag must be a positive integer greater than 1.'
 						);
@@ -454,10 +468,13 @@ export async function parseOptionsAndRunCLI(argsToParse: string[]) {
 		const cliArgs = {
 			...args,
 			command,
-			mount: [...(args.mount || []), ...(args['mount-dir'] || [])],
+			mount: [
+				...((args['mount'] as Mount[]) || []),
+				...((args['mount-dir'] as Mount[]) || []),
+			],
 			'mount-before-install': [
-				...(args['mount-before-install'] || []),
-				...(args['mount-dir-before-install'] || []),
+				...((args['mount-before-install'] as Mount[]) || []),
+				...((args['mount-dir-before-install'] as Mount[]) || []),
 			],
 		} as RunCLIArgs;