fix: Improve CLI flag validation for single-word flags

Fix issue with kebab-case validator incorrectly flagging single-word flags like --prompt. Refactor detectCamelCaseFlags to properly handle all single-word flags. Update tests to verify correct behavior with single-word and camelCase flags. Add support for alternative flag formats in init command (e.g., -my_name). This fixes a bug where users couldn't use the --prompt flag directly and had to use -p instead.
This commit is contained in:
Eyal Toledano
2025-03-26 15:54:51 -04:00
parent c75e518380
commit b3fc14a4c2
9 changed files with 188 additions and 27 deletions

View File

@@ -562,6 +562,29 @@ function registerCommands(programInstance) {
}
});
// init command (documentation only, implementation is in init.js)
programInstance
.command('init')
.description('Initialize a new project with Task Master structure')
.option('-n, --name <name>', 'Project name')
.option('-my_name <name>', 'Project name (alias for --name)')
.option('--my_name <name>', 'Project name (alias for --name)')
.option('-d, --description <description>', 'Project description')
.option('-my_description <description>', 'Project description (alias for --description)')
.option('-v, --version <version>', 'Project version')
.option('-my_version <version>', 'Project version (alias for --version)')
.option('-a, --author <author>', 'Author name')
.option('-y, --yes', 'Skip prompts and use default values')
.option('--skip-install', 'Skip installing dependencies')
.action(() => {
console.log(chalk.yellow('The init command must be run as a standalone command: task-master init'));
console.log(chalk.cyan('Example usage:'));
console.log(chalk.white(' task-master init -n "My Project" -d "Project description"'));
console.log(chalk.white(' task-master init -my_name "My Project" -my_description "Project description"'));
console.log(chalk.white(' task-master init -y'));
process.exit(0);
});
// Add more commands as needed...
return programInstance;

View File

@@ -301,8 +301,8 @@ function detectCamelCaseFlags(args) {
if (arg.startsWith('--')) {
const flagName = arg.split('=')[0].slice(2); // Remove -- and anything after =
// Skip if it's a single word (no hyphens) or already in kebab-case
if (!flagName.includes('-')) {
// Skip single-word flags - they can't be camelCase
if (!flagName.includes('-') && !/[A-Z]/.test(flagName)) {
continue;
}