feat(tasks): enhance task and subtask deletion

Author: RajaRakoto

src/core/taskmaster/TaskMaster.ts

@@ -428,7 +428,7 @@ export class TaskMaster {
 	 * @param numTasksToGenerate Number of tasks to generate (default: 10)
 	 * @param allowAdvancedResearch Allow advanced research for task generation
 	 * @param appendToExistingTasks Whether to append to existing tasks
-	 * @param tag tag for the generated tasks
+	 * @param tag Context tag
 	 */
 	public async parseAsync(
 		inputFilePath: string,
@@ -480,7 +480,7 @@ export class TaskMaster {
 	// TODO: done
 	/**
 	 * @description Decomposes all tasks using AI
-	 * @param tag tag for the tasks to decompose
+	 * @param tag Context tag
 	 */
 	public async decomposeAsync(tag: string): Promise<void> {
 		await this.executeCommandAsync(
@@ -690,7 +690,7 @@ export class TaskMaster {
 	 * @description Adds a new task using AI
 	 * @param prompt Description of the task to create
 	 * @param allowAdvancedResearch Use research capabilities
-	 * @param tag Tag context for the task
+	 * @param tag Context tag
 	 */
 	public async addTaskByAIAsync(
 		prompt: string,
@@ -888,26 +888,25 @@ export class TaskMaster {
 	 * @description Updates the status of one or more tasks
 	 * @param ids Array of task IDs (can be main tasks or hierarchical subtask IDs)
 	 * @param status The new status to set for the tasks
-	 * @param tag Optional tag to filter tasks
+	 * @param tag Context tag
 	 */
 	public async updateTaskStatusAsync(
 		ids: string[],
 		status: string,
-		tag?: string,
+		tag: string,
 	): Promise<void> {
 		const formatedIds = ids.length > 1 ? ids.join(",") : ids[0];
-		const args = ["set-status", `--id=${formatedIds}`, `--status=${status}`];
-
-		if (tag) {
-			args.push(`--tag=${tag}`);
-		}
-
 		await this.executeCommandAsync(
 			`Updating status of task(s) ${formatedIds} to ${status}...`,
 			`Status of task(s) ${formatedIds} updated successfully!`,
 			`Failed to update status of task(s) ${formatedIds}`,
 			this._mainCommand,
-			args,
+			[
+				"set-status",
+				`--id=${formatedIds}`,
+				`--status=${status}`,
+				`--tag=${tag}`,
+			],
 		);
 	}
 
@@ -932,7 +931,93 @@ export class TaskMaster {
 	}
 
 	// ==============================================
-	// Dependencies
+	// Deleting Methods
+	// ==============================================
+
+	/**
+	 * @description Delete a task by ID (including subtasks)
+	 * @param id The ID of the task to remove
+	 * @param tag Context tag
+	 */
+	public async deleteTaskAsync(id: number, tag: string): Promise<void> {
+		const { confirm } = await inquirer.prompt({
+			type: "confirm",
+			name: "confirm",
+			message: chalk.red(`Are you sure you want to delete task ${id}?`),
+			default: false,
+		});
+
+		if (confirm) {
+			await this.executeCommandAsync(
+				`Deleting task ${id}...`,
+				`Task ${id} deleted successfully!`,
+				`Failed to delete task ${id}`,
+				this._mainCommand,
+				["remove-task", `--id=${id}`, `--tag=${tag}`, "-y"],
+			);
+		}
+	}
+
+	/**
+	 * @description Delete a specific subtask
+	 * @param hierarchicalId The hierarchical ID of the subtask
+	 * @param tag Context tag
+	 */
+	public async deleteSubtaskAsync(
+		hierarchicalId: string,
+		tag: string,
+	): Promise<void> {
+		const { confirm } = await inquirer.prompt({
+			type: "confirm",
+			name: "confirm",
+			message: chalk.red(
+				`Are you sure you want to delete subtask ${hierarchicalId}?`,
+			),
+			default: false,
+		});
+
+		if (confirm) {
+			await this.executeCommandAsync(
+				`Deleting subtask ${hierarchicalId}...`,
+				`Subtask ${hierarchicalId} deleted successfully!`,
+				`Failed to delete subtask ${hierarchicalId}`,
+				this._mainCommand,
+				["remove-subtask", `--id=${hierarchicalId}`, `--tag=${tag}`],
+			);
+		}
+	}
+
+	/**
+	 * @description Deletes all subtasks from a specific task
+	 * @param id The ID of the task to clear subtasks from
+	 * @param tag Context tag
+	 */
+	public async deleteAllSubtasksFromTaskAsync(
+		id: number,
+		tag: string,
+	): Promise<void> {
+		const { confirm } = await inquirer.prompt({
+			type: "confirm",
+			name: "confirm",
+			message: chalk.red(
+				`Are you sure you want to delete all subtasks from task ${id}?`,
+			),
+			default: false,
+		});
+
+		if (confirm) {
+			await this.executeCommandAsync(
+				`Deleting all subtasks from task ${id}...`,
+				`All subtasks deleted from task ${id} successfully!`,
+				`Failed to delete subtasks from task ${id}`,
+				this._mainCommand,
+				["clear-subtasks", `--id=${id}`, `--tag=${tag}`],
+			);
+		}
+	}
+
+	// ==============================================
+	// Dependencies Methods
 	// ==============================================
 
 	// TODO: in-progress

src/core/taskmaster/exec.ts

@@ -284,8 +284,34 @@ export async function tmaiManageAsync() {
 			const { tmaiDeleteTasksMenu } = await inquirer.prompt(
 				tmaiDeleteTasksMenu_prompt,
 			);
-			if (tmaiDeleteTasksMenu === "tmai-deletetasks") {
-				console.log("Executing task deletion...");
+			const { mainIDs, subtasksIDs } = await tmai.getAllTaskIdsAsync(tasks);
+			const tag = await askTaskTagAsync();
+
+			switch (tmaiDeleteTasksMenu) {
+				case "tmai-deletetask": {
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					const taskId = await askTaskIdAsync(mainIDs);
+					await tmai.deleteTaskAsync(taskId, tag);
+					tasks = await tmai.getTasksContentAsync();
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					break;
+				}
+				case "tmai-deletesubtask": {
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					const subtaskId = await askHierarchicalTaskIdAsync(subtasksIDs);
+					await tmai.deleteSubtaskAsync(subtaskId, tag);
+					tasks = await tmai.getTasksContentAsync();
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					break;
+				}
+				case "tmai-deleteallsubtasksfromtask": {
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					const taskId = await askTaskIdAsync(mainIDs);
+					await tmai.deleteAllSubtasksFromTaskAsync(taskId, tag);
+					tasks = await tmai.getTasksContentAsync();
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					break;
+				}
 			}
 			break;
 		}

src/prompt.ts

@@ -275,28 +275,20 @@ export const tmaiDeleteTasksMenu_prompt = [
 		name: "tmaiDeleteTasksMenu",
 		message: chalk.bgBlue("Choose an operation"),
 		loop: true,
-		pageSize: 15,
+		pageSize: 7,
 		choices: [
 			new inquirer.Separator("=== Deleting tasks ==="),
 			{
-				name: `${emoji.get("wastebasket")} Delete task`,
+				name: `${emoji.get("wastebasket")} Delete a task (including subtasks)`,
 				value: "tmai-deletetask",
 			},
 			{
-				name: `${emoji.get("scissors")} Delete subtask`,
+				name: `${emoji.get("wastebasket")} Delete a subtask`,
 				value: "tmai-deletesubtask",
 			},
 			{
-				name: `${emoji.get("recycle")} Convert subtask to task`,
-				value: "tmai-deletesubtaskconvert",
-			},
-			{
-				name: `${emoji.get("broom")} Clear subtasks from task`,
-				value: "tmai-clearsubtasks",
-			},
-			{
-				name: `${emoji.get("fire")} Clear all subtasks`,
-				value: "tmai-clearsubtasksall",
+				name: `${emoji.get("wastebasket")} Clear all subtasks from a task`,
+				value: "tmai-deleteallsubtasksfromtask",
 			},
 		],
 	},
@@ -392,13 +384,13 @@ export const tmaiBackupRestoreClearClear_prompt = [
 			},
 			{
 				name: chalk.red(
-					`${emoji.get("boom")} Clear all subtasks only (excluding main tasks)`,
+					`${emoji.get("broom")} Clear all subtasks only (excluding main tasks)`,
 				),
 				value: "tmai-clearallsubtasks",
 			},
 			{
 				name: chalk.red(
-					`${emoji.get("boom")} Clear all current tasks (including subtasks) and all related tmai files`,
+					`${emoji.get("broom")} Clear all current tasks (including subtasks) and all related tmai files`,
 				),
 				value: "tmai-clearall",
 			},