chore: improve changeset

.changeset/chilly-chicken-leave.md

@@ -1,5 +0,0 @@
----
-'task-master-ai': patch
----
-
-Fix remove-task command to handle multiple comma-separated task IDs

.changeset/true-adults-build.md

@@ -0,0 +1,5 @@
+---
+'task-master-ai': patch
+---
+
+Fixed a bug that prevented the task-master from running in a Linux container

index.js

@@ -46,22 +46,18 @@ export const initProject = async (options = {}) => {
 };
 
 // Export a function to run init as a CLI command
-export const runInitCLI = async () => {
-	// Using spawn to ensure proper handling of stdio and process exit
-	const child = spawn('node', [resolve(__dirname, './scripts/init.js')], {
-		stdio: 'inherit',
-		cwd: process.cwd()
-	});
-
-	return new Promise((resolve, reject) => {
-		child.on('close', (code) => {
-			if (code === 0) {
-				resolve();
-			} else {
-				reject(new Error(`Init script exited with code ${code}`));
-			}
-		});
-	});
+export const runInitCLI = async (options = {}) => {
+	try {
+		const init = await import('./scripts/init.js');
+		const result = await init.initializeProject(options);
+		return result;
+	} catch (error) {
+		console.error('Initialization failed:', error.message);
+		if (process.env.DEBUG === 'true') {
+			console.error('Debug stack trace:', error.stack);
+		}
+		throw error; // Re-throw to be handled by the command handler
+	}
 };
 
 // Export version information
@@ -79,11 +75,21 @@ if (import.meta.url === `file://${process.argv[1]}`) {
 	program
 		.command('init')
 		.description('Initialize a new project')
-		.action(() => {
-			runInitCLI().catch((err) => {
+		.option('-y, --yes', 'Skip prompts and use default values')
+		.option('-n, --name <n>', 'Project name')
+		.option('-d, --description <description>', 'Project description')
+		.option('-v, --version <version>', 'Project version', '0.1.0')
+		.option('-a, --author <author>', 'Author name')
+		.option('--skip-install', 'Skip installing dependencies')
+		.option('--dry-run', 'Show what would be done without making changes')
+		.option('--aliases', 'Add shell aliases (tm, taskmaster)')
+		.action(async (cmdOptions) => {
+			try {
+				await runInitCLI(cmdOptions);
+			} catch (err) {
 				console.error('Init failed:', err.message);
 				process.exit(1);
-			});
+			}
 		});
 
 	program

mcp-server/src/core/direct-functions/remove-task.js

@@ -3,23 +3,18 @@
  * Direct function implementation for removing a task
  */
 
-import {
-	removeTask,
-	taskExists
-} from '../../../../scripts/modules/task-manager.js';
+import { removeTask } from '../../../../scripts/modules/task-manager.js';
 import {
 	enableSilentMode,
-	disableSilentMode,
-	readJSON
+	disableSilentMode
 } from '../../../../scripts/modules/utils.js';
 
 /**
  * Direct function wrapper for removeTask with error handling.
- * Supports removing multiple tasks at once with comma-separated IDs.
  *
  * @param {Object} args - Command arguments
  * @param {string} args.tasksJsonPath - Explicit path to the tasks.json file.
- * @param {string} args.id - The ID(s) of the task(s) or subtask(s) to remove (comma-separated for multiple).
+ * @param {string} args.id - The ID of the task or subtask to remove.
  * @param {Object} log - Logger object
  * @returns {Promise<Object>} - Remove task result { success: boolean, data?: any, error?: { code: string, message: string }, fromCache: false }
  */
@@ -41,7 +36,8 @@ export async function removeTaskDirect(args, log) {
 		}
 
 		// Validate task ID parameter
-		if (!id) {
+		const taskId = id;
+		if (!taskId) {
 			log.error('Task ID is required');
 			return {
 				success: false,
@@ -53,103 +49,46 @@ export async function removeTaskDirect(args, log) {
 			};
 		}
 
-		// Split task IDs if comma-separated
-		const taskIdArray = id.split(',').map((taskId) => taskId.trim());
-
-		log.info(
-			`Removing ${taskIdArray.length} task(s) with ID(s): ${taskIdArray.join(', ')} from ${tasksJsonPath}`
-		);
-
-		// Validate all task IDs exist before proceeding
-		const data = readJSON(tasksJsonPath);
-		if (!data || !data.tasks) {
-			return {
-				success: false,
-				error: {
-					code: 'INVALID_TASKS_FILE',
-					message: `No valid tasks found in ${tasksJsonPath}`
-				},
-				fromCache: false
-			};
-		}
-
-		const invalidTasks = taskIdArray.filter(
-			(taskId) => !taskExists(data.tasks, taskId)
-		);
-
-		if (invalidTasks.length > 0) {
-			return {
-				success: false,
-				error: {
-					code: 'INVALID_TASK_ID',
-					message: `The following tasks were not found: ${invalidTasks.join(', ')}`
-				},
-				fromCache: false
-			};
-		}
-
-		// Remove tasks one by one
-		const results = [];
-
-		// Enable silent mode to prevent console logs from interfering with JSON response
-		enableSilentMode();
+		// Skip confirmation in the direct function since it's handled by the client
+		log.info(`Removing task with ID: ${taskId} from ${tasksJsonPath}`);
 
 		try {
-			for (const taskId of taskIdArray) {
-				try {
-					const result = await removeTask(tasksJsonPath, taskId);
-					results.push({
-						taskId,
-						success: true,
-						message: result.message,
-						removedTask: result.removedTask
-					});
-					log.info(`Successfully removed task: ${taskId}`);
-				} catch (error) {
-					results.push({
-						taskId,
-						success: false,
-						error: error.message
-					});
-					log.error(`Error removing task ${taskId}: ${error.message}`);
-				}
-			}
-		} finally {
+			// Enable silent mode to prevent console logs from interfering with JSON response
+			enableSilentMode();
+
+			// Call the core removeTask function using the provided path
+			const result = await removeTask(tasksJsonPath, taskId);
+
 			// Restore normal logging
 			disableSilentMode();
-		}
 
-		// Check if all tasks were successfully removed
-		const successfulRemovals = results.filter((r) => r.success);
-		const failedRemovals = results.filter((r) => !r.success);
+			log.info(`Successfully removed task: ${taskId}`);
 
-		if (successfulRemovals.length === 0) {
-			// All removals failed
+			// Return the result
+			return {
+				success: true,
+				data: {
+					message: result.message,
+					taskId: taskId,
+					tasksPath: tasksJsonPath,
+					removedTask: result.removedTask
+				},
+				fromCache: false
+			};
+		} catch (error) {
+			// Make sure to restore normal logging even if there's an error
+			disableSilentMode();
+
+			log.error(`Error removing task: ${error.message}`);
 			return {
 				success: false,
 				error: {
-					code: 'REMOVE_TASK_ERROR',
-					message: 'Failed to remove any tasks',
-					details: failedRemovals
-						.map((r) => `${r.taskId}: ${r.error}`)
-						.join('; ')
+					code: error.code || 'REMOVE_TASK_ERROR',
+					message: error.message || 'Failed to remove task'
 				},
 				fromCache: false
 			};
 		}
-
-		// At least some tasks were removed successfully
-		return {
-			success: true,
-			data: {
-				totalTasks: taskIdArray.length,
-				successful: successfulRemovals.length,
-				failed: failedRemovals.length,
-				results: results,
-				tasksPath: tasksJsonPath
-			},
-			fromCache: false
-		};
 	} catch (error) {
 		// Ensure silent mode is disabled even if an outer error occurs
 		disableSilentMode();

mcp-server/src/tools/remove-task.js

@@ -23,9 +23,7 @@ export function registerRemoveTaskTool(server) {
 		parameters: z.object({
 			id: z
 				.string()
-				.describe(
-					"ID(s) of the task(s) or subtask(s) to remove (e.g., '5' or '5.2' or '5,6,7')"
-				),
+				.describe("ID of the task or subtask to remove (e.g., '5' or '5.2')"),
 			file: z.string().optional().describe('Absolute path to the tasks file'),
 			projectRoot: z
 				.string()
@@ -37,7 +35,7 @@ export function registerRemoveTaskTool(server) {
 		}),
 		execute: async (args, { log, session }) => {
 			try {
-				log.info(`Removing task(s) with ID(s): ${args.id}`);
+				log.info(`Removing task with ID: ${args.id}`);
 
 				// Get project root from args or session
 				const rootFolder =

scripts/modules/commands.js

@@ -1374,18 +1374,18 @@ function registerCommands(programInstance) {
 	// remove-task command
 	programInstance
 		.command('remove-task')
-		.description('Remove one or more tasks or subtasks permanently')
+		.description('Remove a task or subtask permanently')
 		.option(
 			'-i, --id <id>',
-			'ID(s) of the task(s) or subtask(s) to remove (e.g., "5" or "5.2" or "5,6,7")'
+			'ID of the task or subtask to remove (e.g., "5" or "5.2")'
 		)
 		.option('-f, --file <file>', 'Path to the tasks file', 'tasks/tasks.json')
 		.option('-y, --yes', 'Skip confirmation prompt', false)
 		.action(async (options) => {
 			const tasksPath = options.file;
-			const taskIds = options.id;
+			const taskId = options.id;
 
-			if (!taskIds) {
+			if (!taskId) {
 				console.error(chalk.red('Error: Task ID is required'));
 				console.error(
 					chalk.yellow('Usage: task-master remove-task --id=<taskId>')
@@ -1394,7 +1394,7 @@ function registerCommands(programInstance) {
 			}
 
 			try {
-				// Check if the tasks file exists and is valid
+				// Check if the task exists
 				const data = readJSON(tasksPath);
 				if (!data || !data.tasks) {
 					console.error(
@@ -1403,89 +1403,75 @@ function registerCommands(programInstance) {
 					process.exit(1);
 				}
 
-				// Split task IDs if comma-separated
-				const taskIdArray = taskIds.split(',').map((id) => id.trim());
-
-				// Validate all task IDs exist before proceeding
-				const invalidTasks = taskIdArray.filter(
-					(id) => !taskExists(data.tasks, id)
-				);
-				if (invalidTasks.length > 0) {
-					console.error(
-						chalk.red(
-							`Error: The following tasks were not found: ${invalidTasks.join(', ')}`
-						)
-					);
+				if (!taskExists(data.tasks, taskId)) {
+					console.error(chalk.red(`Error: Task with ID ${taskId} not found`));
 					process.exit(1);
 				}
 
+				// Load task for display
+				const task = findTaskById(data.tasks, taskId);
+
 				// Skip confirmation if --yes flag is provided
 				if (!options.yes) {
-					// Display tasks to be removed
+					// Display task information
 					console.log();
 					console.log(
 						chalk.red.bold(
-							'⚠️ WARNING: This will permanently delete the following tasks:'
+							'⚠️ WARNING: This will permanently delete the following task:'
 						)
 					);
 					console.log();
 
-					for (const taskId of taskIdArray) {
-						const task = findTaskById(data.tasks, taskId);
+					if (typeof taskId === 'string' && taskId.includes('.')) {
+						// It's a subtask
+						const [parentId, subtaskId] = taskId.split('.');
+						console.log(chalk.white.bold(`Subtask ${taskId}: ${task.title}`));
+						console.log(
+							chalk.gray(
+								`Parent Task: ${task.parentTask.id} - ${task.parentTask.title}`
+							)
+						);
+					} else {
+						// It's a main task
+						console.log(chalk.white.bold(`Task ${taskId}: ${task.title}`));
 
-						if (typeof taskId === 'string' && taskId.includes('.')) {
-							// It's a subtask
-							const [parentId, subtaskId] = taskId.split('.');
-							console.log(chalk.white.bold(`Subtask ${taskId}: ${task.title}`));
+						// Show if it has subtasks
+						if (task.subtasks && task.subtasks.length > 0) {
 							console.log(
-								chalk.gray(
-									`Parent Task: ${task.parentTask.id} - ${task.parentTask.title}`
+								chalk.yellow(
+									`⚠️ This task has ${task.subtasks.length} subtasks that will also be deleted!`
 								)
 							);
-						} else {
-							// It's a main task
-							console.log(chalk.white.bold(`Task ${taskId}: ${task.title}`));
-
-							// Show if it has subtasks
-							if (task.subtasks && task.subtasks.length > 0) {
-								console.log(
-									chalk.yellow(
-										`⚠️ This task has ${task.subtasks.length} subtasks that will also be deleted!`
-									)
-								);
-							}
-
-							// Show if other tasks depend on it
-							const dependentTasks = data.tasks.filter(
-								(t) =>
-									t.dependencies &&
-									t.dependencies.includes(parseInt(taskId, 10))
-							);
-
-							if (dependentTasks.length > 0) {
-								console.log(
-									chalk.yellow(
-										`⚠️ Warning: ${dependentTasks.length} other tasks depend on this task!`
-									)
-								);
-								console.log(
-									chalk.yellow('These dependencies will be removed:')
-								);
-								dependentTasks.forEach((t) => {
-									console.log(chalk.yellow(`  - Task ${t.id}: ${t.title}`));
-								});
-							}
 						}
-						console.log();
+
+						// Show if other tasks depend on it
+						const dependentTasks = data.tasks.filter(
+							(t) =>
+								t.dependencies && t.dependencies.includes(parseInt(taskId, 10))
+						);
+
+						if (dependentTasks.length > 0) {
+							console.log(
+								chalk.yellow(
+									`⚠️ Warning: ${dependentTasks.length} other tasks depend on this task!`
+								)
+							);
+							console.log(chalk.yellow('These dependencies will be removed:'));
+							dependentTasks.forEach((t) => {
+								console.log(chalk.yellow(`  - Task ${t.id}: ${t.title}`));
+							});
+						}
 					}
 
+					console.log();
+
 					// Prompt for confirmation
 					const { confirm } = await inquirer.prompt([
 						{
 							type: 'confirm',
 							name: 'confirm',
 							message: chalk.red.bold(
-								`Are you sure you want to permanently delete ${taskIdArray.length > 1 ? 'these tasks' : 'this task'}?`
+								'Are you sure you want to permanently delete this task?'
 							),
 							default: false
 						}
@@ -1497,72 +1483,31 @@ function registerCommands(programInstance) {
 					}
 				}
 
-				const indicator = startLoadingIndicator('Removing tasks...');
+				const indicator = startLoadingIndicator('Removing task...');
 
-				// Remove each task
-				const results = [];
-				for (const taskId of taskIdArray) {
-					try {
-						const result = await removeTask(tasksPath, taskId);
-						results.push({ taskId, success: true, ...result });
-					} catch (error) {
-						results.push({ taskId, success: false, error: error.message });
-					}
-				}
+				// Remove the task
+				const result = await removeTask(tasksPath, taskId);
 
 				stopLoadingIndicator(indicator);
 
-				// Display results
-				const successfulRemovals = results.filter((r) => r.success);
-				const failedRemovals = results.filter((r) => !r.success);
-
-				if (successfulRemovals.length > 0) {
+				// Display success message with appropriate color based on task or subtask
+				if (typeof taskId === 'string' && taskId.includes('.')) {
+					// It was a subtask
 					console.log(
 						boxen(
-							chalk.green(
-								`Successfully removed ${successfulRemovals.length} task${successfulRemovals.length > 1 ? 's' : ''}`
-							) +
-								'\n\n' +
-								successfulRemovals
-									.map((r) =>
-										chalk.white(
-											`✓ ${r.taskId.includes('.') ? 'Subtask' : 'Task'} ${r.taskId}`
-										)
-									)
-									.join('\n'),
-							{
-								padding: 1,
-								borderColor: 'green',
-								borderStyle: 'round',
-								margin: { top: 1 }
-							}
+							chalk.green(`Subtask ${taskId} has been successfully removed`),
+							{ padding: 1, borderColor: 'green', borderStyle: 'round' }
 						)
 					);
-				}
-
-				if (failedRemovals.length > 0) {
+				} else {
+					// It was a main task
 					console.log(
-						boxen(
-							chalk.red(
-								`Failed to remove ${failedRemovals.length} task${failedRemovals.length > 1 ? 's' : ''}`
-							) +
-								'\n\n' +
-								failedRemovals
-									.map((r) => chalk.white(`✗ ${r.taskId}: ${r.error}`))
-									.join('\n'),
-							{
-								padding: 1,
-								borderColor: 'red',
-								borderStyle: 'round',
-								margin: { top: 1 }
-							}
-						)
+						boxen(chalk.green(`Task ${taskId} has been successfully removed`), {
+							padding: 1,
+							borderColor: 'green',
+							borderStyle: 'round'
+						})
 					);
-
-					// Exit with error if any removals failed
-					if (successfulRemovals.length === 0) {
-						process.exit(1);
-					}
 				}
 			} catch (error) {
 				console.error(