@taazkareem/clickup-mcp-server 0.4.56 → 0.4.57

package/build/index.js

@@ -109,7 +109,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Due date of the task (Unix timestamp in milliseconds). Convert dates to this format before submitting."
                         }
                     },
-                    required: ["name"]
+                    required: []
                 }
             },
             {
@@ -164,11 +164,11 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                                         description: "Array of user IDs to assign to the task"
                                     }
                                 },
-                                required: ["name"]
+                                required: []
                             }
                         }
                     },
-                    required: ["listId", "tasks"]
+                    required: []
                 }
             },
             {
@@ -210,7 +210,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Status of the list"
                         }
                     },
-                    required: ["name"]
+                    required: []
                 }
             },
             {
@@ -236,7 +236,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Whether to override space statuses with folder-specific statuses"
                         }
                     },
-                    required: ["name"]
+                    required: []
                 }
             },
             {
@@ -274,7 +274,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Status of the list (uses folder default if not specified)"
                         }
                     },
-                    required: ["name"]
+                    required: []
                 }
             },
             {
@@ -304,7 +304,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Name of the destination list - will automatically find the list by name (optional if using listId instead). Only use this if you don't already have the list ID from previous responses."
                         }
                     },
-                    required: ["taskName", "listName"]
+                    required: []
                 }
             },
             {
@@ -334,7 +334,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Name of the list to create the duplicate in - will automatically find the list by name (optional if using listId instead). Only use this if you don't already have the list ID from previous responses."
                         }
                     },
-                    required: ["taskName", "listName"]
+                    required: []
                 }
             },
             {
@@ -363,20 +363,22 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             type: "string",
                             description: "New plain text description for the task"
                         },
+                        markdown_description: {
+                            type: "string",
+                            description: "New markdown formatted description for the task. If provided, this takes precedence over description"
+                        },
                         status: {
                             type: "string",
                             description: "New status for the task (must be a valid status in the task's list)"
                         },
                         priority: {
-                            type: "number",
-                            description: "New priority for the task (1-4), where 1 is urgent/highest priority and 4 is lowest priority"
-                        },
-                        dueDate: {
-                            type: "string",
-                            description: "New due date for the task (Unix timestamp in milliseconds). Convert dates to this format before submitting."
+                            type: ["number", "null"],
+                            enum: [1, 2, 3, 4, null],
+                            description: "New priority for the task (1-4 or null), where 1 is urgent/highest priority and 4 is lowest priority. Set to null to clear priority.",
+                            optional: true
                         }
                     },
-                    required: ["taskName"]
+                    required: []
                 }
             },
             {
@@ -456,7 +458,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Object with custom field IDs as keys and desired values for filtering"
                         }
                     },
-                    required: ["listName"]
+                    required: []
                 }
             },
             {
@@ -478,7 +480,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Optional: Name of the list to narrow down task search when multiple tasks have the same name"
                         }
                     },
-                    required: ["taskName"]
+                    required: []
                 }
             },
             {
@@ -500,7 +502,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             description: "Optional: Name of the list to narrow down task search when multiple tasks have the same name"
                         }
                     },
-                    required: ["taskId"]
+                    required: []
                 }
             },
             {
@@ -730,12 +732,17 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             }
             case "create_bulk_tasks": {
                 const args = request.params.arguments;
-                if (!args.listId && !args.listName) {
-                    throw new Error("Either listId or listName is required");
-                }
-                if (!args.tasks || args.tasks.length === 0) {
+                // First validate tasks array
+                if (!args.tasks || !Array.isArray(args.tasks) || args.tasks.length === 0) {
                     throw new Error("tasks array is required and must not be empty");
                 }
+                // Validate each task has required fields
+                args.tasks.forEach((task, index) => {
+                    if (!task.name) {
+                        throw new Error(`Task at index ${index} is missing required field 'name'`);
+                    }
+                });
+                // Get listId from name if needed
                 let listId = args.listId;
                 if (!listId && args.listName) {
                     const result = await clickup.findListIDByName(args.listName);
@@ -744,12 +751,23 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     }
                     listId = result.id;
                 }
+                // Now validate we have a listId
+                if (!listId) {
+                    throw new Error("Either listId or listName must be provided");
+                }
                 const { listId: _, listName: __, tasks } = args;
                 const createdTasks = await clickup.createBulkTasks(listId, { tasks });
                 return {
                     content: [{
                             type: "text",
-                            text: `Created ${createdTasks.length} tasks`
+                            text: JSON.stringify({
+                                message: `Created ${createdTasks.length} tasks`,
+                                tasks: createdTasks.map(task => ({
+                                    id: task.id,
+                                    name: task.name,
+                                    url: task.url
+                                }))
+                            }, null, 2)
                         }]
                 };
             }
@@ -902,9 +920,11 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             }
             case "update_task": {
                 const args = request.params.arguments;
+                // Require either taskId or taskName
                 if (!args.taskId && !args.taskName) {
                     throw new Error("Either taskId or taskName is required");
                 }
+                // Get taskId from taskName if needed
                 let taskId = args.taskId;
                 if (!taskId && args.taskName) {
                     const result = await clickup.findTaskByName(args.taskName, undefined, args.listName);
@@ -913,7 +933,16 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     }
                     taskId = result.id;
                 }
+                // Remove helper fields before updating
                 const { taskId: _, taskName: __, listName: ___, ...updateData } = args;
+                // Ensure priority is properly handled
+                if (updateData.priority !== undefined && updateData.priority !== null) {
+                    const priority = Number(updateData.priority);
+                    if (isNaN(priority) || ![1, 2, 3, 4].includes(priority)) {
+                        throw new Error("Priority must be a number between 1 and 4, or null to clear priority");
+                    }
+                    updateData.priority = priority;
+                }
                 const task = await clickup.updateTask(taskId, updateData);
                 return {
                     content: [{

package/build/services/clickup.js

@@ -59,16 +59,7 @@ export class ClickUpService {
      */
     async makeRequest(requestFn) {
         await this.checkRateLimit();
-        try {
-            return await requestFn();
-        }
-        catch (error) {
-            if (error.response?.status === 429) {
-                // Let the interceptor handle it
-                throw error;
-            }
-            throw error;
-        }
+        return await requestFn();
     }
     /**
      * Initializes the ClickUpService singleton instance.
@@ -183,10 +174,15 @@ export class ClickUpService {
     async createTask(listId, data) {
         return this.makeRequest(async () => {
             const taskData = { ...data };
-            if (taskData.description && /[#*`\-\[\]>]/.test(taskData.description)) {
-                taskData.markdown_description = taskData.description;
+            // If markdown_description is provided, it takes precedence
+            if (taskData.markdown_description) {
+                // Ensure we don't send both to avoid confusion
                 delete taskData.description;
             }
+            else if (taskData.description) {
+                // Only use description as-is, don't auto-convert to markdown
+                taskData.description = taskData.description.trim();
+            }
             const response = await this.client.post(`/list/${listId}/task`, taskData);
             return response.data;
         });
@@ -200,10 +196,15 @@ export class ClickUpService {
         for (const taskData of data.tasks) {
             await this.makeRequest(async () => {
                 const processedTask = { ...taskData };
-                if (processedTask.description && /[#*`\-\[\]>]/.test(processedTask.description)) {
-                    processedTask.markdown_description = processedTask.description;
+                // If markdown_description is provided, it takes precedence
+                if (processedTask.markdown_description) {
+                    // Ensure we don't send both to avoid confusion
                     delete processedTask.description;
                 }
+                else if (processedTask.description) {
+                    // Only use description as-is, don't auto-convert to markdown
+                    processedTask.description = processedTask.description.trim();
+                }
                 const response = await this.client.post(`/list/${listId}/task`, processedTask);
                 createdTasks.push(response.data);
             });
@@ -216,7 +217,21 @@ export class ClickUpService {
      */
     async updateTask(taskId, data) {
         return this.makeRequest(async () => {
-            const response = await this.client.put(`/task/${taskId}`, data);
+            const updateData = { ...data };
+            // If markdown_description is provided, it takes precedence
+            if (updateData.markdown_description) {
+                // Ensure we don't send both to avoid confusion
+                delete updateData.description;
+            }
+            else if (updateData.description) {
+                // Only use description as-is, don't auto-convert to markdown
+                updateData.description = updateData.description.trim();
+            }
+            // Handle null priority explicitly
+            if (updateData.priority === null) {
+                updateData.priority = null;
+            }
+            const response = await this.client.put(`/task/${taskId}`, updateData);
             return response.data;
         });
     }
@@ -730,11 +745,4 @@ export class ClickUpService {
             : `${task.space.name} > ${task.list.name} > ${task.name}`;
         return { id: task.id, path };
     }
-    async getTaskStatuses(listId) {
-        const response = await this.getTasks(listId);
-        const statuses = [...new Set(response.tasks
-                .filter((task) => task.status !== undefined)
-                .map((task) => task.status.status))];
-        return statuses;
-    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taazkareem/clickup-mcp-server",
-  "version": "0.4.56",
+  "version": "0.4.57",
   "description": "ClickUp MCP Server - Integrate ClickUp tasks with AI through Model Context Protocol",
   "type": "module",
   "main": "build/index.js",
@@ -44,10 +44,8 @@
   "homepage": "https://github.com/taazkareem/clickup-mcp-server#readme",
   "dependencies": {
     "@modelcontextprotocol/sdk": "0.6.0",
-    "@types/express": "^5.0.0",
     "axios": "^1.6.7",
-    "dotenv": "^16.4.1",
-    "express": "^4.21.2"
+    "dotenv": "^16.4.1"
   },
   "devDependencies": {
     "@types/node": "^20.11.16",