fix: Improve MCP server robustness and debugging

- Refactor  for more reliable project root detection, particularly when running within integrated environments like Cursor IDE. Includes deriving root from script path and avoiding fallback to '/'.
- Enhance error handling in :
    - Add detailed debug information (paths searched, CWD, etc.) to the error message when  is not found in the provided project root.
    - Improve clarity of error messages and potential solutions.
- Add verbose logging in  to trace session object content and the finally resolved project root path, aiding in debugging path-related issues.
- Add default values for  and  to the example  environment configuration.

mcp-server/src/tools/set-task-status.js

@@ -6,7 +6,8 @@
 import { z } from "zod";
 import {
   handleApiResult,
-  createErrorResponse
+  createErrorResponse,
+  getProjectRootFromSession
 } from "./utils.js";
 import { setTaskStatusDirect } from "../core/task-master-core.js";
 
@@ -17,14 +18,14 @@ import { setTaskStatusDirect } from "../core/task-master-core.js";
 export function registerSetTaskStatusTool(server) {
   server.addTool({
     name: "set_task_status",
-    description: "Set the status of a task",
+    description: "Set the status of one or more tasks or subtasks.",
     parameters: z.object({
       id: z
         .string()
-        .describe("Task ID (can be comma-separated for multiple tasks)"),
+        .describe("Task ID or subtask ID (e.g., '15', '15.2'). Can be comma-separated for multiple updates."),
       status: z
         .string()
-        .describe("New status (todo, in-progress, review, done)"),
+        .describe("New status to set (e.g., 'pending', 'done', 'in-progress', 'review', 'deferred', 'cancelled'."),
       file: z.string().optional().describe("Path to the tasks file"),
       projectRoot: z
         .string()
@@ -33,17 +34,31 @@ export function registerSetTaskStatusTool(server) {
           "Root directory of the project (default: automatically detected)"
         ),
     }),
-    execute: async (args, { log }) => {
+    execute: async (args, { log, session, reportProgress }) => {
       try {
         log.info(`Setting status of task(s) ${args.id} to: ${args.status}`);
+        await reportProgress({ progress: 0 });
         
-        // Call the direct function wrapper
-        const result = await setTaskStatusDirect(args, log);
+        let rootFolder = getProjectRootFromSession(session, log);
         
-        // Log result
-        log.info(`${result.success ? `Successfully updated task ${args.id} status to "${args.status}"` : 'Failed to update task status'}`);
+        if (!rootFolder && args.projectRoot) {
+          rootFolder = args.projectRoot;
+          log.info(`Using project root from args as fallback: ${rootFolder}`);
+        }
+        
+        const result = await setTaskStatusDirect({
+          projectRoot: rootFolder,
+          ...args
+        }, log, { reportProgress, mcpLog: log, session});
+        
+        await reportProgress({ progress: 100 });
+        
+        if (result.success) {
+          log.info(`Successfully updated status for task(s) ${args.id} to "${args.status}": ${result.data.message}`);
+        } else {
+          log.error(`Failed to update task status: ${result.error?.message || 'Unknown error'}`);
+        }
         
-        // Use handleApiResult to format the response
         return handleApiResult(result, log, 'Error setting task status');
       } catch (error) {
         log.error(`Error in setTaskStatus tool: ${error.message}`);