- Fixed critical bug. Server is operational

- Bump package version to 0.4.59

Author: taazkareem

README.md

@@ -61,7 +61,7 @@ npx -y @taazkareem/clickup-mcp-server \
 | [update_task](docs/api-reference.md#task-management) | Modify task | `taskId`/`taskName` |
 | [get_tasks](docs/api-reference.md#task-retrieval) | Get tasks from list | `listId`/`listName` |
 | [get_task](docs/api-reference.md#task-retrieval) | Get task details | `taskId`/`taskName` |
-| [delete_task](docs/api-reference.md#task-management) | Remove task | `taskId` |
+| [delete_task](docs/api-reference.md#task-management) | Remove task | `taskId`/`taskName` |
 | [move_task](docs/api-reference.md#task-management) | Move task | `taskId`/`taskName`, `listId`/`listName` |
 | [duplicate_task](docs/api-reference.md#task-management) | Copy task | `taskId`/`taskName`, `listId`/`listName` |
 | [create_list](docs/api-reference.md#list-management) | Create list in space | `name`, `spaceId`/`spaceName` |

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taazkareem/clickup-mcp-server",
-  "version": "0.4.55",
+  "version": "0.4.59",
   "description": "ClickUp MCP Server - Integrate ClickUp tasks with AI through Model Context Protocol",
   "type": "module",
   "main": "build/index.js",

src/index.ts

@@ -131,7 +131,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               description: "Due date of the task (Unix timestamp in milliseconds). Convert dates to this format before submitting."
             }
           },
-          required: []
+          required: ["name"]
         }
       },
       {
@@ -190,7 +190,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               }
             }
           },
-          required: []
+          required: ["tasks"]
         }
       },
       {
@@ -232,7 +232,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               description: "Status of the list"
             }
           },
-          required: []
+          required: ["name"]
         }
       },
       {
@@ -258,7 +258,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               description: "Whether to override space statuses with folder-specific statuses"
             }
           },
-          required: []
+          required: ["name"]
         }
       },
       {
@@ -267,6 +267,10 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
         inputSchema: {
           type: "object",
           properties: {
+            name: {
+              type: "string",
+              description: "Name of the list"
+            },
             folderId: {
               type: "string",
               description: "ID of the folder to create the list in (optional if using folderName instead). If you have this ID from a previous response, use it directly rather than looking up by name."
@@ -283,10 +287,6 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               type: "string",
               description: "Name of the space containing the folder - will automatically find the space by name (optional if using spaceId instead). Only use this if you don't already have the space ID from previous responses."
             },
-            name: {
-              type: "string",
-              description: "Name of the list"
-            },
             content: {
               type: "string",
               description: "Description or content of the list"
@@ -296,7 +296,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               description: "Status of the list (uses folder default if not specified)"
             }
           },
-          required: []
+          required: ["name"]
         }
       },
       {
@@ -1123,33 +1123,17 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       }
 
       case "delete_task": {
-        const args = request.params.arguments as { 
-          taskId: string; // Make taskId required
-          taskName?: string;
-          listName?: string;
-        };
-
-        // Validate the required taskId parameter
-        if (!args.taskId) {
-          throw new Error("taskId is required for deletion operations");
-        }
-
-        // Store the task name before deletion for the response message
-        let taskName = args.taskName;
-        if (!taskName) {
-          try {
-            const task = await clickup.getTask(args.taskId);
-            taskName = task.name;
-          } catch (error) {
-            // If we can't get the task details, just use the ID in the response
-          }
+        const args = request.params.arguments as { taskId?: string; taskName?: string; listName?: string };
+        
+        if (!args.taskId && !args.taskName) {
+          throw new Error("Either taskId or taskName is required");
         }
 
-        await clickup.deleteTask(args.taskId);
+        await clickup.deleteTask(args.taskId, args.taskName, args.listName);
         return {
           content: [{
             type: "text",
-            text: `Successfully deleted task ${taskName || args.taskId}`
+            text: `Successfully deleted task${args.taskName ? ` "${args.taskName}"` : ` with ID ${args.taskId}`}`
           }]
         };
       }

src/services/clickup.ts

@@ -281,11 +281,31 @@ export class ClickUpService {
 
   /**
    * Deletes a task from the workspace.
+   * Can delete by taskId directly or find by taskName first.
    * Handles rate limiting automatically.
+   * @param taskId - ID of the task to delete (optional if taskName provided)
+   * @param taskName - Name of the task to delete (optional if taskId provided)
+   * @param listName - Optional list name to narrow down task search
+   * @throws Error if neither taskId nor taskName is provided, or if task not found
    */
-  async deleteTask(taskId: string): Promise<void> {
+  async deleteTask(taskId?: string, taskName?: string, listName?: string): Promise<void> {
     return this.makeRequest(async () => {
-      await this.client.delete(`/task/${taskId}`);
+      let finalTaskId = taskId;
+
+      // If no taskId but taskName provided, find the task first
+      if (!taskId && taskName) {
+        const taskInfo = await this.findTaskByName(taskName, undefined, listName);
+        if (!taskInfo) {
+          throw new Error(`Task "${taskName}" not found${listName ? ` in list "${listName}"` : ''}`);
+        }
+        finalTaskId = taskInfo.id;
+      }
+
+      if (!finalTaskId) {
+        throw new Error('Either taskId or taskName must be provided');
+      }
+
+      await this.client.delete(`/task/${finalTaskId}`);
     });
   }