feat(deps): improve dependency management

Author: RajaRakoto

src/core/taskmaster/TaskMaster.ts

@@ -177,6 +177,58 @@ export class TaskMaster {
 		);
 	}
 
+	// TODO: done
+	/**
+	 * @description Fixes the IDs of all tasks and subtasks in tasks.json to be sequential
+	 * by reorganizing them incrementally starting from 1.
+	 * This method ensures better organization and readability of the task list.
+	 */
+	public async fixIdsAsync(): Promise<void> {
+		const oraOptions = {
+			text: "Fixing task IDs...",
+			successText: chalk.bgGreen("Task IDs fixed successfully!"),
+			failText: chalk.bgRed("Failed to fix task IDs"),
+		};
+
+		await oraPromise(async () => {
+			// Read the current tasks
+			const tasks = await this.getTasksContentAsync();
+
+			// Sort tasks by their current ID to maintain logical order
+			tasks.master.tasks.sort((a, b) => a.id - b.id);
+
+			// Fix main task IDs
+			const idMap = new Map<number, number>();
+			for (const [index, task] of tasks.master.tasks.entries()) {
+				const oldId = task.id;
+				const newId = index + 1;
+				if (oldId !== newId) {
+					idMap.set(oldId, newId);
+					task.id = newId;
+				}
+			}
+
+			// Fix subtask IDs and update dependencies
+			for (const task of tasks.master.tasks) {
+				if (task.subtasks && task.subtasks.length > 0) {
+					for (const [index, subtask] of task.subtasks.entries()) {
+						subtask.id = index + 1;
+					}
+				}
+			}
+
+			// Write the updated tasks back to the file
+			await writeFile(this._tasksFilePath, JSON.stringify(tasks, null, 2));
+
+			// Update the internal tasks file path if needed
+			this._tasksFilePath = this._tasksFilePath;
+
+			// Ensure dependencies are valid after ID reorganization
+			await this.fixDependenciesAsync();
+			await this.validateDependenciesAsync();
+		}, oraOptions);
+	}
+
 	// TODO: done
 	/**
 	 * @description Fixes the format of the tasks.json file if necessary
@@ -570,21 +622,30 @@ export class TaskMaster {
 			if (showParent) {
 				hasTasks = true;
 				const title = truncate(task.title, MAX_TITLE_TRUNC_LENGTH);
+				const dependencies =
+					task.dependencies.length > 0
+						? `deps: ${chalk.cyan(task.dependencies.join(","))}`
+						: `deps: ${chalk.gray("none")}`;
 				output +=
 					`${chalk.bgGreen.bold(`#${task.id}`)} ${chalk.magenta(title)} ` +
 					`[status: ${formatStatus(task.status)}] - ` +
-					`[priority: ${formatPriority(task.priority)}]\n`;
+					`[priority: ${formatPriority(task.priority)}] - ` +
+					`[${dependencies}]\n`;
 
 				// Only show matching subtasks
 				if (withSubtasks && matchingSubtasks.length > 0) {
 					for (const { subtask } of matchingSubtasks) {
 						const subTitle = truncate(subtask.title, MAX_TITLE_TRUNC_LENGTH);
 						const hierarchicalId = `${task.id}.${subtask.id}`;
+						const subDependencies =
+							subtask.dependencies.length > 0
+								? `deps: ${chalk.cyan(subtask.dependencies.join(","))}`
+								: `deps: ${chalk.gray("none")}`;
 						output +=
 							`  ${chalk.dim("↳")} ${chalk.bold(`#${hierarchicalId}`)} ` +
 							`${chalk.magenta(subTitle)} [status: ${formatStatus(
 								subtask.status,
-							)}]\n`;
+							)}] - [${subDependencies}]\n`;
 					}
 				}
 			}
@@ -920,7 +981,7 @@ export class TaskMaster {
 		subtaskId: number,
 		parentId: number,
 	): Promise<void> {
-		await this.clearAllDependenciesAsync(subtaskId.toString());
+		await this.deleteAllDepsFromTaskAsync(subtaskId.toString());
 		await this.executeCommandAsync(
 			`Converting task ${subtaskId} to subtask of ${parentId}...`,
 			`Task ${subtaskId} converted to subtask successfully!`,
@@ -934,6 +995,7 @@ export class TaskMaster {
 	// Deleting Methods
 	// ==============================================
 
+	// TODO: done
 	/**
 	 * @description Delete a task by ID (including subtasks)
 	 * @param id The ID of the task to remove
@@ -958,6 +1020,7 @@ export class TaskMaster {
 		}
 	}
 
+	// TODO: done
 	/**
 	 * @description Delete a specific subtask
 	 * @param hierarchicalId The hierarchical ID of the subtask
@@ -987,6 +1050,7 @@ export class TaskMaster {
 		}
 	}
 
+	// TODO: done
 	/**
 	 * @description Deletes all subtasks from a specific task
 	 * @param id The ID of the task to clear subtasks from
@@ -1016,6 +1080,39 @@ export class TaskMaster {
 		}
 	}
 
+	// TODO: done
+	/**
+	 * @description Clears all dependencies for the specified task or subtask.
+	 * @param taskId The task ID or hierarchical ID of the subtask
+	 */
+	public async deleteAllDepsFromTaskAsync(taskId: string): Promise<void> {
+		const tasks = await this.getTasksContentAsync();
+		const dependencyIds = await this.getAllDependenciesAsync(tasks, taskId);
+
+		if (dependencyIds.length === 0) {
+			console.log(chalk.yellow(`No dependencies found for task ${taskId}.`));
+			return;
+		}
+
+		// For subtasks, dependency IDs are already in the correct format
+		// For main tasks, dependency IDs are numbers that need to be converted to string
+		const isSubtask = taskId.includes(".");
+
+		for (const dependencyId of dependencyIds) {
+			const dependsOnId = isSubtask
+				? dependencyId.toString() // For subtasks, dependencyId is already the full ID
+				: dependencyId.toString(); // For main tasks, convert number to string
+
+			await this.executeCommandAsync(
+				`Removing dependency ${dependsOnId} from task ${taskId}...`,
+				`Dependency ${dependsOnId} removed successfully from task ${taskId}!`,
+				`Failed to remove dependency ${dependsOnId} from task ${taskId}`,
+				this._mainCommand,
+				["remove-dependency", `--id=${taskId}`, `--depends-on=${dependsOnId}`],
+			);
+		}
+	}
+
 	// ==============================================
 	// Dependencies Methods
 	// ==============================================
@@ -1069,34 +1166,6 @@ export class TaskMaster {
 		);
 	}
 
-	// TODO: done
-	/**
-	 * @description Clears all dependencies for the specified task or subtask.
-	 * @param taskId The task ID (either a simple number as string or hierarchical like "1.2")
-	 */
-	public async clearAllDependenciesAsync(taskId: string): Promise<void> {
-		const tasks = await this.getTasksContentAsync();
-		const dependencyIds = await this.getAllDependenciesAsync(tasks, taskId);
-
-		if (dependencyIds.length === 0) {
-			console.log(chalk.yellow(`No dependencies found for task ${taskId}.`));
-			return;
-		}
-
-		for (const dependencyId of dependencyIds) {
-			const dependsOnId = taskId.includes(".")
-				? `${taskId.split(".")[0]}.${dependencyId}`
-				: dependencyId;
-			await this.executeCommandAsync(
-				`Removing dependency ${dependsOnId} from task ${taskId}...`,
-				`Dependency ${dependsOnId} removed successfully from task ${taskId}!`,
-				`Failed to remove dependency ${dependsOnId} from task ${taskId}`,
-				this._mainCommand,
-				["remove-dependency", `--id=${taskId}`, `--depends-on=${dependsOnId}`],
-			);
-		}
-	}
-
 	// ==============================================
 	// Backup, Restore and Clear Methods
 	// ==============================================
@@ -1232,4 +1301,40 @@ export class TaskMaster {
 		if (allFilesCleared)
 			console.log(chalk.green("All task-related files are cleared!"));
 	}
+
+	// TODO: done
+	/**
+	 * @description Clears all dependencies from all tasks and subtasks
+	 */
+	public async clearAllDepsAsync(): Promise<void> {
+		const { confirm } = await inquirer.prompt({
+			type: "confirm",
+			name: "confirm",
+			message: chalk.red("Clear all dependencies from all tasks and subtasks?"),
+			default: false,
+		});
+
+		if (confirm) {
+			const oraOptions = {
+				text: "Clearing all dependencies...",
+				successText: chalk.bgGreen("All dependencies cleared successfully!"),
+				failText: chalk.bgRed("Failed to clear all dependencies"),
+			};
+
+			await oraPromise(async () => {
+				const tasks = await this.getTasksContentAsync();
+
+				for (const task of tasks.master.tasks) {
+					task.dependencies = [];
+					if (task.subtasks && task.subtasks.length > 0) {
+						for (const subtask of task.subtasks) {
+							subtask.dependencies = [];
+						}
+					}
+				}
+
+				await writeFile(this._tasksFilePath, JSON.stringify(tasks, null, 2));
+			}, oraOptions);
+		}
+	}
 }

src/core/taskmaster/exec.ts

@@ -312,6 +312,14 @@ export async function tmaiManageAsync() {
 					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
 					break;
 				}
+				case "tmai-deletealldepsfromtask": {
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					const taskId = await askHybridTaskIdAsync(mainIDs, subtasksIDs);
+					await tmai.deleteAllDepsFromTaskAsync(taskId);
+					tasks = await tmai.getTasksContentAsync();
+					await tmai.listAsync(tasks, TASKS_STATUSES.join(","), true, true);
+					break;
+				}
 			}
 			break;
 		}
@@ -348,12 +356,6 @@ export async function tmaiDependenciesAsync() {
 			await tmai.fixDependenciesAsync();
 			break;
 		}
-		case "tmai-clearalldeps": {
-			await tmai.listAsync(tasks, TASKS_STATUSES.join(","), false, true);
-			const taskId = await askHybridTaskIdAsync(mainIDs, subtasksIDs);
-			await tmai.clearAllDependenciesAsync(taskId);
-			break;
-		}
 		default:
 			console.log("Invalid option selected.");
 	}
@@ -391,6 +393,10 @@ export async function tmaiBackupRestoreClearAsync() {
 			}
 			break;
 		}
+		case "tmai-clearalldeps": {
+			await tmai.clearAllDepsAsync();
+			break;
+		}
 		case "tmai-clearallsubtasks": {
 			await tmai.clearAllSubtasksAsync();
 			break;

src/prompt.ts

@@ -287,9 +287,13 @@ export const tmaiDeleteTasksMenu_prompt = [
 				value: "tmai-deletesubtask",
 			},
 			{
-				name: `${emoji.get("wastebasket")} Clear all subtasks from a task`,
+				name: `${emoji.get("wastebasket")} Delete all subtasks from a task`,
 				value: "tmai-deleteallsubtasksfromtask",
 			},
+			{
+				name: `${emoji.get("wastebasket")} Delete all dependencies from a task`,
+				value: "tmai-deletealldepsfromtask",
+			},
 		],
 	},
 ];
@@ -320,10 +324,6 @@ export const tmaiDepsMenu_prompt = [
 				name: `${emoji.get("wrench")} Fix dependencies`,
 				value: "tmai-fixdeps",
 			},
-			{
-				name: `${emoji.get("broken_heart")} Clear all dependencies from a task`,
-				value: "tmai-clearalldeps",
-			},
 		],
 	},
 ];
@@ -382,6 +382,12 @@ export const tmaiBackupRestoreClearClear_prompt = [
 				),
 				value: "tmai-restore",
 			},
+			{
+				name: chalk.red(
+					`${emoji.get("broom")} Clear all dependencies only (for all tasks)`,
+				),
+				value: "tmai-clearalldeps",
+			},
 			{
 				name: chalk.red(
 					`${emoji.get("broom")} Clear all subtasks only (excluding main tasks)`,