Bug fixes

src/cli.js

@@ -211,6 +211,7 @@ export class CLI {
 		applyCommandConfig(this, cmdName, cmd, conf);
 
 		if (conf.platforms) {
+			this.debugLogger.trace(`Detected conf.platforms applying config for "${cmd.name()}", overriding createHelp()`);
 			this.command.createHelp = () => {
 				return Object.assign(new TiHelp(this, conf.platforms), this.command.configureHelp());
 			};
@@ -548,6 +549,10 @@ export class CLI {
 		await program.parseAsync();
 		this.debugLogger.trace('Finished parsing arguments first pass');
 
+		if (this.command === program && program.args.length) {
+			throw new TiError(`Unknown command "${program.args[0]}"`, { showHelp: true });
+		}
+
 		// reset hooks
 		const resetHooksAndOptionHandlers = ctx => {
 			ctx._lifeCycleHooks = {};
@@ -612,6 +617,8 @@ export class CLI {
 			const platformConf = this.command.conf.platforms[this.argv.platform];
 
 			this.argv.$platform = this.argv.platform;
+
+			// set platform context
 			this.command.platform = {
 				conf: platformConf
 			};
@@ -688,23 +695,6 @@ export class CLI {
 
 	// TODO: Prompt for missing required options
 
-	// // any keys in the conf object that aren't explicitly 'flags',
-	// // 'options', 'args', or 'subcommands' is probably a option branch
-	// // that changes the available flags/options
-	// const skipRegExp = /^(flags|options|args|subcommands)$/;
-	// const optionBranches = Object.keys(conf)
-	// 	.filter(name => conf.options && conf.options[name] && !skipRegExp.test(name))
-	// 	.sort((a, b) => {
-	// 		// if we have multiple option groups, then try to process them in order
-	// 		if (!conf.options[a] || !conf.options[a].order) {
-	// 			return 1;
-	// 		}
-	// 		if (!conf.options[b] || !conf.options[b].order) {
-	// 			return -1;
-	// 		}
-	// 		return conf.options[b].order - conf.options[a].order;
-	// 	});
-
 	// for (const name of optionBranches) {
 	// 	const option = conf.options[name];
 	// 	const optionBranch = conf[name];
@@ -785,7 +775,7 @@ export class CLI {
 			cmdName,
 			config: this.config,
 			cwd,
-			logger: this.debugLogger,
+			logger: this.logger,
 			promptingEnabled: this.promptingEnabled,
 			selectedSdk: this.argv.sdk
 		});
@@ -878,6 +868,13 @@ export class CLI {
 		this.debugLogger.trace(`Importing: ${commandFile}`);
 		cmd.module = (await import(commandFile)) || {};
 
+		// check if this command is compatible with this version of the CLI
+		if (cmd.module.cliVersion && !version.satisfies(this.version, cmd.module.cliVersion)) {
+			throw new TiError(`Command "${cmdName}" incompatible with this version of the CLI`, {
+				after: `Requires version ${cmd.module.cliVersion}, currently ${this.version}`
+			});
+		}
+
 		if (typeof cmd.module.extendedDesc === 'string') {
 			desc = cmd.module.extendedDesc;
 		} else if (desc) {
@@ -900,6 +897,7 @@ export class CLI {
 			}
 
 			if (conf.platforms) {
+				this.debugLogger.trace(`Detected conf.platforms loading "${cmdName}", overriding createHelp()`);
 				this.command.createHelp = () => {
 					return Object.assign(new TiHelp(this, conf.platforms), this.command.configureHelp());
 				};

src/commands/info.js

@@ -80,7 +80,7 @@ export async function run(logger, config, cli) {
 		({
 			data,
 			platformInfo
-		} = await detect(logger, config, cli, types));
+		} = await detect(cli.debugLogger, config, cli, types));
 	} finally {
 		busy?.stop();
 	}

src/commands/module.js

@@ -50,7 +50,11 @@ export function config(logger, config, cli) {
  * @param {Function} finished - Callback when the command finishes
  */
 export async function run(logger, config, cli) {
-	const action = cli.command.name();
+	let action = cli.command.name();
+	if (action === 'list' && cli.command.args.length) {
+		action = cli.command.args[0];
+		cli.command = cli.command.parent;
+	}
 	for (const [name, subcommand] of Object.entries(ModuleSubcommands)) {
 		if (action === name || action === subcommand.alias) {
 			await ModuleSubcommands[name].fn(logger, config, cli);
@@ -145,7 +149,7 @@ ModuleSubcommands.list = {
 			}
 		}
 
-		const results = await detect(searchPaths, config, logger);
+		const results = await detect(searchPaths, config, cli.debugLogger);
 
 		if (isJson) {
 			logger.log(JSON.stringify(results, null, '\t'));

src/commands/sdk.js

@@ -53,7 +53,11 @@ export function config(logger, config, cli) {
  * @param {CLI} cli - The CLI instance
  */
 export async function run(logger, config, cli) {
-	const action = cli.command.name();
+	let action = cli.command.name();
+	if (action === 'list' && cli.command.args.length) {
+		action = cli.command.args[0];
+		cli.command = cli.command.parent;
+	}
 	for (const [name, subcommand] of Object.entries(SdkSubcommands)) {
 		if (action === name || action === subcommand.alias) {
 			await SdkSubcommands[name].fn(logger, config, cli);

src/util/prompt.js

@@ -6,6 +6,11 @@
 export async function prompt(opts) {
 	const { default: prompts } = await import('prompts');
 	const { prompt } = prompts;
+
+	if (Array.isArray(opts)) {
+		return await prompt(opts);
+	}
+
 	const { value } = await prompt({
 		...opts,
 		name: 'value'

src/util/setup-screens.js

@@ -103,10 +103,9 @@ export class SetupScreens {
 			}     Exit`
 		);
 
-		const { value } = await prompt({
+		const value = await prompt({
 			type: 'text',
-			message: 'Where do you want to go?',
-			name: 'value'
+			message: 'Where do you want to go?'
 		});
 
 		const next = lookup[value];
@@ -124,7 +123,7 @@ export class SetupScreens {
 		busy.start();
 
 		try {
-			({ data } = await detect(this.logger, this.config, this.cli, { all: true }));
+			({ data } = await detect(this.cli.debugLogger, this.config, this.cli, { all: true }));
 		} finally {
 			busy.stop();
 		}
@@ -209,7 +208,7 @@ export class SetupScreens {
 		}
 
 		try {
-			({ data } = await detect(this.logger, this.config, this.cli, { all: true }));
+			({ data } = await detect(this.cli.debugLogger, this.config, this.cli, { all: true }));
 
 			data.titaniumCLI.latest = await request('https://registry.npmjs.org/-/package/titanium/dist-tags')
 				.then(res => res.body.json())
@@ -287,7 +286,7 @@ export class SetupScreens {
 			busy.start();
 
 			try {
-				({ data } = await detect(this.logger, this.config, this.cli, { all: true }));
+				({ data } = await detect(this.cli.debugLogger, this.config, this.cli, { all: true }));
 			} finally {
 				busy.stop();
 			}
@@ -714,7 +713,7 @@ export class SetupScreens {
 		busy.start();
 
 		try {
-			({ data } = await detect(this.logger, this.config, this.cli, { all: true }));
+			({ data } = await detect(this.cli.debugLogger, this.config, this.cli, { all: true }));
 		} finally {
 			busy.stop();
 		}
@@ -797,7 +796,7 @@ export class SetupScreens {
 		busy.start();
 
 		try {
-			({ data } = await detect(this.logger, this.config, this.cli, { all: true }));
+			({ data } = await detect(this.cli.debugLogger, this.config, this.cli, { all: true }));
 		} finally {
 			busy.stop();
 		}

src/util/tisdk.js

@@ -193,13 +193,12 @@ export async function initSDK({ cmdName, config, cwd, logger, promptingEnabled,
 				}
 			}
 
-			({ sdkVersion } = await prompt({
+			sdkVersion = await prompt({
 				type: 'select',
 				message: 'Which Titanium SDK would you like to use?',
-				name: 'sdkVersion',
 				initial: sdk ? choices.find(s => s.name === sdk.name)?.name : undefined,
 				choices
-			}));
+			});
 
 			if (sdkVersion === undefined) {
 				// sigint