feat(core): add task validation and improve initialization

Author: RajaRakoto

src/core/TaskMaster.ts

@@ -169,7 +169,7 @@ export class TaskMaster {
 	 * by encapsulating the 'tasks' and 'metadata' keys under a 'master' key
 	 * @param mode Operation mode: 'init' for initialization or 'repair' for fixing existing file
 	 */
-	public async _fixTasksFileFormatAsync(
+	private async _fixTasksFileFormatAsync(
 		mode: "init" | "repair" = "repair",
 	): Promise<void> {
 		const isInitMode = mode === "init";
@@ -318,6 +318,65 @@ export class TaskMaster {
 		);
 	}
 
+	// TODO: done
+	/**
+	 * @description Validates that tasks are ready (file exists and has at least one task)
+	 * @returns True if tasks are ready, false otherwise
+	 */
+	public async validateTasksReadyAsync(): Promise<boolean> {
+		// Check if tasks.json file exists
+		if (!fs.existsSync(this._tasksFilePath)) {
+			console.log(
+				chalk.yellow(
+					"TMAI is not initialized. Please run the initialization process and generate at least one task.",
+				),
+			);
+			await this.countdownAsync(10);
+			return false;
+		}
+
+		try {
+			// Read the tasks file
+			const tasks = await this.getTasksContentAsync();
+
+			// Check if tasks object and master property exist
+			if (!tasks || !tasks.master) {
+				console.log(
+					chalk.yellow(
+						"TMAI is not initialized. Please run the initialization process and generate at least one task.",
+					),
+				);
+				await this.countdownAsync(10);
+				return false;
+			}
+
+			// Check if there's at least one task in master.tasks
+			if (
+				!tasks.master.tasks ||
+				!Array.isArray(tasks.master.tasks) ||
+				tasks.master.tasks.length === 0
+			) {
+				console.log(
+					chalk.yellow(
+						"TMAI is not initialized. Please generate at least one task.",
+					),
+				);
+				await this.countdownAsync(10);
+				return false;
+			}
+
+			return true;
+		} catch {
+			console.log(
+				chalk.yellow(
+					"TMAI is not initialized. Please run the initialization process and generate at least one task.",
+				),
+			);
+			await this.countdownAsync(10);
+			return false;
+		}
+	}
+
 	// TODO: done
 	/**
 	 * @description Fixes the IDs of all tasks and subtasks in tasks.json to be sequential
@@ -662,6 +721,7 @@ export class TaskMaster {
 
 		await runCommandAsync(this._mainCommand, ["init"], false, false);
 		console.log(chalk.bgGreen("Task-master project initialized successfully!"));
+		await this._fixTasksFileFormatAsync("init");
 	}
 
 	// TODO: done
@@ -670,7 +730,7 @@ export class TaskMaster {
 	 */
 	public async interactiveConfigModelAsync(): Promise<void> {
 		await this._executeCommandAsync(
-			"Configuring AI models...",
+			"Configuring AI models...\n",
 			"AI models configured successfully!",
 			"AI model configuration failed",
 			this._mainCommand,
@@ -693,7 +753,7 @@ export class TaskMaster {
 		provider?: string,
 	): Promise<void> {
 		const oraOptions = {
-			text: "Configuring AI models...",
+			text: "Configuring AI models...\n",
 			successText: chalk.bgGreen("AI models configured successfully!"),
 			failText: chalk.bgRed("AI model configuration failed"),
 		};

src/core/exec.ts

@@ -1,6 +1,7 @@
 /* libs */
 import inquirer from "inquirer";
 import path from "node:path";
+import fs from "node:fs";
 
 /* index */
 import { taskmasterCLI } from "@/index";
@@ -141,9 +142,30 @@ export async function tmaiGenAsync() {
 	if (choice.tmaiGenDecMenu === "tmai-parse") {
 		const tasksJsonPath = path.join(".taskmaster", "tasks", "tasks.json");
 		if (await existsAsync(tasksJsonPath)) {
-			const overwrite = await askOverwriteConfirmationAsync();
-			if (!overwrite) {
-				return restartAsync();
+			const fileContent = fs.readFileSync(tasksJsonPath, "utf8");
+			let isEmpty = fileContent.trim() === "[]";
+
+			// Also check for the specific empty structure
+			if (!isEmpty) {
+				try {
+					const tasksData = JSON.parse(fileContent);
+					isEmpty =
+						tasksData.master &&
+						Array.isArray(tasksData.master.tasks) &&
+						tasksData.master.tasks.length === 0 &&
+						typeof tasksData.master.metadata === "object";
+				} catch {
+					// If parsing fails, we'll assume it's not empty
+					isEmpty = false;
+				}
+			}
+
+			// Only ask for overwrite confirmation if the file is not empty
+			if (!isEmpty) {
+				const overwrite = await askOverwriteConfirmationAsync();
+				if (!overwrite) {
+					return restartAsync();
+				}
 			}
 		}
 
@@ -186,6 +208,10 @@ export async function tmaiGenAsync() {
  * - Delete tasks
  */
 export async function tmaiManageAsync() {
+	if (!(await tmai.validateTasksReadyAsync())) {
+		return taskmasterCLI();
+	}
+
 	let tasks = await tmai.getTasksContentAsync();
 	const { mainIDs, subtasksIDs } = await tmai.getAllTaskIdsAsync(tasks);
 	const { tmaiManageMenu } = await inquirer.prompt(tmaiManageMenu_prompt);
@@ -441,6 +467,10 @@ export async function tmaiManageAsync() {
  * dependencies. Based on the user's choice, it executes the corresponding operation.
  */
 export async function tmaiDependenciesAsync() {
+	if (!(await tmai.validateTasksReadyAsync())) {
+		return taskmasterCLI();
+	}
+
 	let tasks = await tmai.getTasksContentAsync();
 	const { tmaiDepsMenu } = await inquirer.prompt(tmaiDepsMenu_prompt);
 
@@ -492,6 +522,10 @@ export async function tmaiDependenciesAsync() {
  * choice, it executes the corresponding operation.
  */
 export async function tmaiAnalysisReportDocsAsync() {
+	if (!(await tmai.validateTasksReadyAsync())) {
+		return taskmasterCLI();
+	}
+
 	const { tmaiAnalysisReportDocsMenu } = await inquirer.prompt(
 		tmaiAnalysisReportDocs_prompt,
 	);