feat: implement set-status MCP command and update changeset

mcp-server/src/core/direct-functions/set-task-status.js

@@ -0,0 +1,83 @@
+/**
+ * set-task-status.js
+ * Direct function implementation for setting task status
+ */
+
+import { setTaskStatus } from '../../../../scripts/modules/task-manager.js';
+import { findTasksJsonPath } from '../utils/path-utils.js';
+
+/**
+ * Direct function wrapper for setTaskStatus with error handling.
+ * 
+ * @param {Object} args - Command arguments containing id, status and file path options.
+ * @param {Object} log - Logger object.
+ * @returns {Promise<Object>} - Result object with success status and data/error information.
+ */
+export async function setTaskStatusDirect(args, log) {
+  try {
+    log.info(`Setting task status with args: ${JSON.stringify(args)}`);
+    
+    // Check required parameters
+    if (!args.id) {
+      const errorMessage = 'No task ID specified. Please provide a task ID to update.';
+      log.error(errorMessage);
+      return { 
+        success: false, 
+        error: { code: 'MISSING_TASK_ID', message: errorMessage },
+        fromCache: false 
+      };
+    }
+    
+    if (!args.status) {
+      const errorMessage = 'No status specified. Please provide a new status value.';
+      log.error(errorMessage);
+      return { 
+        success: false, 
+        error: { code: 'MISSING_STATUS', message: errorMessage },
+        fromCache: false 
+      };
+    }
+    
+    // Get tasks file path
+    let tasksPath;
+    try {
+      tasksPath = findTasksJsonPath(args, log);
+    } catch (error) {
+      log.error(`Error finding tasks file: ${error.message}`);
+      return { 
+        success: false, 
+        error: { code: 'TASKS_FILE_ERROR', message: error.message },
+        fromCache: false 
+      };
+    }
+    
+    // Execute core setTaskStatus function
+    // We need to handle the arguments correctly - this function expects tasksPath, taskIdInput, newStatus
+    const taskId = args.id;
+    const newStatus = args.status;
+    
+    log.info(`Setting task ${taskId} status to "${newStatus}"`);
+    
+    // Execute the setTaskStatus function with source=mcp to avoid console output
+    await setTaskStatus(tasksPath, taskId, newStatus);
+    
+    // Return success data
+    return {
+      success: true,
+      data: {
+        message: `Successfully updated task ${taskId} status to "${newStatus}"`,
+        taskId,
+        status: newStatus,
+        tasksPath
+      },
+      fromCache: false // This operation always modifies state and should never be cached
+    };
+  } catch (error) {
+    log.error(`Error setting task status: ${error.message}`);
+    return { 
+      success: false, 
+      error: { code: 'SET_STATUS_ERROR', message: error.message || 'Unknown error setting task status' },
+      fromCache: false 
+    };
+  }
+} 

mcp-server/src/core/task-master-core.js

@@ -12,6 +12,7 @@ import { updateTasksDirect } from './direct-functions/update-tasks.js';
 import { updateTaskByIdDirect } from './direct-functions/update-task-by-id.js';
 import { updateSubtaskByIdDirect } from './direct-functions/update-subtask-by-id.js';
 import { generateTaskFilesDirect } from './direct-functions/generate-task-files.js';
+import { setTaskStatusDirect } from './direct-functions/set-task-status.js';
 
 // Re-export utility functions
 export { findTasksJsonPath } from './utils/path-utils.js';
@@ -25,6 +26,7 @@ export {
   updateTaskByIdDirect,
   updateSubtaskByIdDirect,
   generateTaskFilesDirect,
+  setTaskStatusDirect,
 };
 
 /**
@@ -39,5 +41,6 @@ export const directFunctions = {
   updateTask: updateTaskByIdDirect,
   updateSubtask: updateSubtaskByIdDirect,
   generate: generateTaskFilesDirect,
+  setStatus: setTaskStatusDirect,
   // Add more functions as we implement them
 }; 

mcp-server/src/tools/setTaskStatus.js

@@ -5,10 +5,10 @@
 
 import { z } from "zod";
 import {
-  executeTaskMasterCommand,
-  createContentResponse,
-  createErrorResponse,
+  handleApiResult,
+  createErrorResponse
 } from "./utils.js";
+import { setTaskStatusDirect } from "../core/task-master-core.js";
 
 /**
  * Register the setTaskStatus tool with the MCP server
@@ -28,6 +28,7 @@ export function registerSetTaskStatusTool(server) {
       file: z.string().optional().describe("Path to the tasks file"),
       projectRoot: z
         .string()
+        .optional()
         .describe(
           "Root directory of the project (default: current working directory)"
         ),
@@ -35,29 +36,18 @@ export function registerSetTaskStatusTool(server) {
     execute: async (args, { log }) => {
       try {
         log.info(`Setting status of task(s) ${args.id} to: ${args.status}`);
-
-        const cmdArgs = [`--id=${args.id}`, `--status=${args.status}`];
-        if (args.file) cmdArgs.push(`--file=${args.file}`);
-
-        const projectRoot = args.projectRoot;
-
-        const result = executeTaskMasterCommand(
-          "set-status",
-          log,
-          cmdArgs,
-          projectRoot
-        );
-
-        if (!result.success) {
-          throw new Error(result.error);
-        }
-
-        return createContentResponse(result.stdout);
+        
+        // Call the direct function wrapper
+        const result = await setTaskStatusDirect(args, log);
+        
+        // Log result
+        log.info(`${result.success ? `Successfully updated task ${args.id} status to "${args.status}"` : 'Failed to update task status'}`);
+        
+        // Use handleApiResult to format the response
+        return handleApiResult(result, log, 'Error setting task status');
       } catch (error) {
-        log.error(`Error setting task status: ${error.message}`);
-        return createErrorResponse(
-          `Error setting task status: ${error.message}`
-        );
+        log.error(`Error in setTaskStatus tool: ${error.message}`);
+        return createErrorResponse(`Error setting task status: ${error.message}`);
       }
     },
   });