@taazkareem/clickup-mcp-server 0.4.72 → 0.4.74

package/build/services/clickup/task.js

@@ -14,7 +14,6 @@ export class TaskService extends BaseClickUpService {
     constructor(apiKey, teamId, baseUrl, workspaceService) {
         super(apiKey, teamId, baseUrl);
         this.workspaceService = null;
-        // Cache workspace service if provided
         if (workspaceService) {
             this.workspaceService = workspaceService;
             this.logOperation('constructor', { usingSharedWorkspaceService: true });
@@ -35,6 +34,122 @@ export class TaskService extends BaseClickUpService {
         }
         return new ClickUpServiceError(message || `Task service error: ${error.message}`, ErrorCode.UNKNOWN, error);
     }
+    /**
+     * Build URL parameters from task filters
+     * @param filters Task filters to convert to URL parameters
+     * @returns URLSearchParams object
+     */
+    buildTaskFilterParams(filters) {
+        const params = new URLSearchParams();
+        // Add all filters to the query parameters
+        if (filters.include_closed)
+            params.append('include_closed', String(filters.include_closed));
+        if (filters.subtasks)
+            params.append('subtasks', String(filters.subtasks));
+        if (filters.page)
+            params.append('page', String(filters.page));
+        if (filters.order_by)
+            params.append('order_by', filters.order_by);
+        if (filters.reverse)
+            params.append('reverse', String(filters.reverse));
+        // Array parameters
+        if (filters.statuses && filters.statuses.length > 0) {
+            filters.statuses.forEach(status => params.append('statuses[]', status));
+        }
+        if (filters.assignees && filters.assignees.length > 0) {
+            filters.assignees.forEach(assignee => params.append('assignees[]', assignee));
+        }
+        // Date filters
+        if (filters.due_date_gt)
+            params.append('due_date_gt', String(filters.due_date_gt));
+        if (filters.due_date_lt)
+            params.append('due_date_lt', String(filters.due_date_lt));
+        if (filters.date_created_gt)
+            params.append('date_created_gt', String(filters.date_created_gt));
+        if (filters.date_created_lt)
+            params.append('date_created_lt', String(filters.date_created_lt));
+        if (filters.date_updated_gt)
+            params.append('date_updated_gt', String(filters.date_updated_gt));
+        if (filters.date_updated_lt)
+            params.append('date_updated_lt', String(filters.date_updated_lt));
+        // Handle custom fields if present
+        if (filters.custom_fields) {
+            Object.entries(filters.custom_fields).forEach(([key, value]) => {
+                params.append(`custom_fields[${key}]`, String(value));
+            });
+        }
+        return params;
+    }
+    /**
+     * Extract priority value from a task
+     * @param task The task to extract priority from
+     * @returns TaskPriority or null
+     */
+    extractPriorityValue(task) {
+        if (!task.priority || !task.priority.id) {
+            return null;
+        }
+        const priorityValue = parseInt(task.priority.id);
+        // Ensure it's in the valid range 1-4
+        if (isNaN(priorityValue) || priorityValue < 1 || priorityValue > 4) {
+            return null;
+        }
+        return priorityValue;
+    }
+    /**
+     * Extract task data for creation/duplication
+     * @param task The source task
+     * @param nameOverride Optional override for the task name
+     * @returns CreateTaskData object
+     */
+    extractTaskData(task, nameOverride) {
+        return {
+            name: nameOverride || task.name,
+            description: task.description || '',
+            status: task.status?.status,
+            priority: this.extractPriorityValue(task),
+            due_date: task.due_date ? Number(task.due_date) : undefined,
+            assignees: task.assignees?.map(a => a.id) || []
+        };
+    }
+    /**
+     * Find a matching task by name using different matching strategies
+     * @param tasks List of tasks to search
+     * @param taskName Name to search for
+     * @returns Matching task or null
+     */
+    findMatchingTask(tasks, taskName) {
+        // Normalize the search term
+        const normalizedSearchTerm = taskName.toLowerCase().trim();
+        // First try exact match
+        let matchingTask = tasks.find(task => task.name.toLowerCase().trim() === normalizedSearchTerm);
+        // If no exact match, try substring match
+        if (!matchingTask) {
+            matchingTask = tasks.find(task => task.name.toLowerCase().trim().includes(normalizedSearchTerm) ||
+                normalizedSearchTerm.includes(task.name.toLowerCase().trim()));
+        }
+        // If still no match and there are emoji characters, try matching without emoji
+        if (!matchingTask && /[\p{Emoji}]/u.test(normalizedSearchTerm)) {
+            matchingTask = this.findMatchingTaskWithoutEmoji(tasks, normalizedSearchTerm);
+        }
+        return matchingTask || null;
+    }
+    /**
+     * Find matching task with emoji characters removed
+     * @param tasks List of tasks to search
+     * @param searchTerm Search term (with emoji)
+     * @returns Matching task or null
+     */
+    findMatchingTaskWithoutEmoji(tasks, searchTerm) {
+        // Remove emoji and try again (simple approximation)
+        const withoutEmoji = searchTerm.replace(/[\p{Emoji}]/gu, '').trim();
+        return tasks.find(task => {
+            const taskNameWithoutEmoji = task.name.toLowerCase().replace(/[\p{Emoji}]/gu, '').trim();
+            return taskNameWithoutEmoji === withoutEmoji ||
+                taskNameWithoutEmoji.includes(withoutEmoji) ||
+                withoutEmoji.includes(taskNameWithoutEmoji);
+        }) || null;
+    }
     /**
      * Create a new task in the specified list
      * @param listId The ID of the list to create the task in
@@ -63,42 +178,7 @@ export class TaskService extends BaseClickUpService {
         this.logOperation('getTasks', { listId, filters });
         try {
             return await this.makeRequest(async () => {
-                const params = new URLSearchParams();
-                // Add all filters to the query parameters
-                if (filters.include_closed)
-                    params.append('include_closed', String(filters.include_closed));
-                if (filters.subtasks)
-                    params.append('subtasks', String(filters.subtasks));
-                if (filters.page)
-                    params.append('page', String(filters.page));
-                if (filters.order_by)
-                    params.append('order_by', filters.order_by);
-                if (filters.reverse)
-                    params.append('reverse', String(filters.reverse));
-                if (filters.statuses && filters.statuses.length > 0) {
-                    filters.statuses.forEach(status => params.append('statuses[]', status));
-                }
-                if (filters.assignees && filters.assignees.length > 0) {
-                    filters.assignees.forEach(assignee => params.append('assignees[]', assignee));
-                }
-                if (filters.due_date_gt)
-                    params.append('due_date_gt', String(filters.due_date_gt));
-                if (filters.due_date_lt)
-                    params.append('due_date_lt', String(filters.due_date_lt));
-                if (filters.date_created_gt)
-                    params.append('date_created_gt', String(filters.date_created_gt));
-                if (filters.date_created_lt)
-                    params.append('date_created_lt', String(filters.date_created_lt));
-                if (filters.date_updated_gt)
-                    params.append('date_updated_gt', String(filters.date_updated_gt));
-                if (filters.date_updated_lt)
-                    params.append('date_updated_lt', String(filters.date_updated_lt));
-                // Handle custom fields if present
-                if (filters.custom_fields) {
-                    Object.entries(filters.custom_fields).forEach(([key, value]) => {
-                        params.append(`custom_fields[${key}]`, String(value));
-                    });
-                }
+                const params = this.buildTaskFilterParams(filters);
                 const response = await this.client.get(`/list/${listId}/task?${params.toString()}`);
                 return response.data.tasks;
             });
@@ -171,27 +251,7 @@ export class TaskService extends BaseClickUpService {
         this.logOperation('findTaskByName', { listId, taskName });
         try {
             const tasks = await this.getTasks(listId);
-            // Normalize the search term
-            const normalizedSearchTerm = taskName.toLowerCase().trim();
-            // First try exact match
-            let matchingTask = tasks.find(task => task.name.toLowerCase().trim() === normalizedSearchTerm);
-            // If no exact match, try substring match
-            if (!matchingTask) {
-                matchingTask = tasks.find(task => task.name.toLowerCase().trim().includes(normalizedSearchTerm) ||
-                    normalizedSearchTerm.includes(task.name.toLowerCase().trim()));
-            }
-            // If still no match and there are emoji characters, try matching without emoji
-            if (!matchingTask && /[\p{Emoji}]/u.test(normalizedSearchTerm)) {
-                // Remove emoji and try again (simple approximation)
-                const withoutEmoji = normalizedSearchTerm.replace(/[\p{Emoji}]/gu, '').trim();
-                matchingTask = tasks.find(task => {
-                    const taskNameWithoutEmoji = task.name.toLowerCase().replace(/[\p{Emoji}]/gu, '').trim();
-                    return taskNameWithoutEmoji === withoutEmoji ||
-                        taskNameWithoutEmoji.includes(withoutEmoji) ||
-                        withoutEmoji.includes(taskNameWithoutEmoji);
-                });
-            }
-            return matchingTask || null;
+            return this.findMatchingTask(tasks, taskName);
         }
         catch (error) {
             throw this.handleError(error, `Failed to find task by name: ${error instanceof Error ? error.message : String(error)}`);
@@ -217,28 +277,9 @@ export class TaskService extends BaseClickUpService {
             let newStatus = availableStatuses.includes(currentStatus || '')
                 ? currentStatus // Keep the same status if available in destination list
                 : destinationList.statuses?.[0]?.status; // Otherwise use the default (first) status
-            // Priority mapping: convert string priority to numeric value if needed
-            let priorityValue = null;
-            if (originalTask.priority) {
-                // If priority.id exists and is numeric, use that
-                if (originalTask.priority.id) {
-                    priorityValue = parseInt(originalTask.priority.id);
-                    // Ensure it's in the valid range 1-4
-                    if (isNaN(priorityValue) || priorityValue < 1 || priorityValue > 4) {
-                        priorityValue = null;
-                    }
-                }
-            }
             // Prepare the task data for the new list
-            const taskData = {
-                name: originalTask.name,
-                description: originalTask.description,
-                status: newStatus,
-                priority: priorityValue,
-                due_date: originalTask.due_date ? Number(originalTask.due_date) : undefined,
-                assignees: originalTask.assignees?.map(a => a.id) || [],
-                // Add any other relevant fields from the original task
-            };
+            const taskData = this.extractTaskData(originalTask);
+            taskData.status = newStatus;
             // Create new task and delete old one in a single makeRequest call
             return await this.makeRequest(async () => {
                 // First create the new task
@@ -269,15 +310,8 @@ export class TaskService extends BaseClickUpService {
         try {
             // Get the original task to duplicate
             const originalTask = await this.getTask(taskId);
-            // Create a copy of the task data
-            const newTaskData = {
-                name: `${originalTask.name} (copy)`,
-                description: originalTask.description || '',
-                status: originalTask.status?.status,
-                priority: originalTask.priority?.id ? parseInt(originalTask.priority.id) : null,
-                due_date: originalTask.due_date ? new Date(originalTask.due_date).getTime() : undefined,
-                assignees: originalTask.assignees?.map(a => a.id) || []
-            };
+            // Create a copy of the task data with "(copy)" appended to the name
+            const newTaskData = this.extractTaskData(originalTask, `${originalTask.name} (copy)`);
             // Create the new task in the specified list or original list
             const targetListId = listId || originalTask.list.id;
             return await this.createTask(targetListId, newTaskData);

package/build/services/clickup/types.js

@@ -1,4 +1,19 @@
 /**
  * Common type definitions for ClickUp API entities
  */
-export {};
+// Helper function to validate and convert priority values
+export function toTaskPriority(value) {
+    if (value === null)
+        return null;
+    if (value === undefined)
+        return undefined;
+    if (value === "null")
+        return null;
+    // Convert string to number if needed
+    const numValue = typeof value === 'string' ? parseInt(value, 10) : value;
+    // Validate it's a valid priority number
+    if (typeof numValue === 'number' && !isNaN(numValue) && [1, 2, 3, 4].includes(numValue)) {
+        return numValue;
+    }
+    return undefined;
+}

package/build/services/shared.js

@@ -8,26 +8,26 @@ import { createClickUpServices } from './clickup/index.js';
 import config from '../config.js';
 import { Logger } from '../logger.js';
 const logger = new Logger('SharedServices');
-// Singleton instance
-let servicesInstance = null;
+// Singleton instances
+let clickUpServicesInstance = null;
 /**
  * Get or create the ClickUp services instance
  */
 function getClickUpServices() {
-    if (!servicesInstance) {
+    if (!clickUpServicesInstance) {
         logger.info('Creating shared ClickUp services singleton');
         // Create the services instance
-        servicesInstance = createClickUpServices({
+        clickUpServicesInstance = createClickUpServices({
             apiKey: config.clickupApiKey,
             teamId: config.clickupTeamId
         });
         // Log what services were initialized with more clarity
         logger.info('Services initialization complete', {
-            services: Object.keys(servicesInstance).join(', '),
+            services: Object.keys(clickUpServicesInstance).join(', '),
             teamId: config.clickupTeamId
         });
     }
-    return servicesInstance;
+    return clickUpServicesInstance;
 }
 // Create a single instance of ClickUp services to be shared
 export const clickUpServices = getClickUpServices();

package/build/tools/folder.js

@@ -5,6 +5,7 @@
  * updating, and deleting folders in the ClickUp workspace hierarchy.
  */
 import { clickUpServices } from '../services/shared.js';
+import { sponsorService } from '../utils/sponsor-service.js';
 // Use shared services instance
 const { folder: folderService, workspace: workspaceService } = clickUpServices;
 /**
@@ -12,7 +13,19 @@ const { folder: folderService, workspace: workspaceService } = clickUpServices;
  */
 export const createFolderTool = {
     name: "create_folder",
-    description: "Create a new folder in a ClickUp space for organizing related lists. You MUST provide:\n1. A folder name\n2. Either spaceId (preferred) or spaceName\n\nAfter creating a folder, you can add lists to it using create_list_in_folder.",
+    description: `Purpose: Create a new folder in a ClickUp space for organizing related lists.
+
+Valid Usage:
+1. Provide spaceId (preferred) + folder name
+2. Provide spaceName + folder name
+
+Requirements:
+- name: REQUIRED
+- EITHER spaceId OR spaceName: REQUIRED
+
+Notes:
+- After creating a folder, you can add lists to it using create_list_in_folder
+- Use override_statuses to set folder-specific statuses`,
     inputSchema: {
         type: "object",
         properties: {
@@ -41,7 +54,18 @@ export const createFolderTool = {
  */
 export const getFolderTool = {
     name: "get_folder",
-    description: "Retrieve details about a specific folder including name, status, and metadata. Valid parameter combinations:\n1. Use folderId alone (preferred)\n2. Use folderName + (spaceId or spaceName)\n\nHelps you understand folder structure before creating or updating lists.",
+    description: `Purpose: Retrieve details about a specific folder including name, status, and metadata.
+
+Valid Usage:
+1. Use folderId alone (preferred)
+2. Use folderName + (spaceId or spaceName)
+
+Requirements:
+- EITHER folderId OR (folderName + space information) is REQUIRED
+- When using folderName, you MUST provide EITHER spaceId OR spaceName
+
+Notes:
+- Helps you understand folder structure before creating or updating lists`,
     inputSchema: {
         type: "object",
         properties: {
@@ -70,7 +94,19 @@ export const getFolderTool = {
  */
 export const updateFolderTool = {
     name: "update_folder",
-    description: "Modify an existing folder's properties. Valid parameter combinations:\n1. Use folderId alone (preferred)\n2. Use folderName + (spaceId or spaceName)\n\nAt least one update field (name or override_statuses) must be provided.",
+    description: `Purpose: Modify an existing folder's properties.
+
+Valid Usage:
+1. Use folderId alone (preferred)
+2. Use folderName + (spaceId or spaceName)
+
+Requirements:
+- At least one update field (name or override_statuses) must be provided
+- EITHER folderId OR (folderName + space information) is REQUIRED
+- When using folderName, you MUST provide EITHER spaceId OR spaceName
+
+Notes:
+- Changes apply immediately to all lists within the folder`,
     inputSchema: {
         type: "object",
         properties: {
@@ -107,7 +143,20 @@ export const updateFolderTool = {
  */
 export const deleteFolderTool = {
     name: "delete_folder",
-    description: "⚠️ PERMANENTLY DELETE a folder and all its contents. This action cannot be undone. Valid parameter combinations:\n1. Use folderId alone (preferred and safest)\n2. Use folderName + (spaceId or spaceName)\n\nWARNING: This will also delete all lists and tasks within the folder.",
+    description: `Purpose: PERMANENTLY DELETE a folder and all its contents.
+
+Valid Usage:
+1. Use folderId alone (preferred and safest)
+2. Use folderName + (spaceId or spaceName)
+
+Requirements:
+- EITHER folderId OR (folderName + space information) is REQUIRED
+- When using folderName, you MUST provide EITHER spaceId OR spaceName
+
+⚠️ CRITICAL WARNING:
+- This action CANNOT be undone
+- All lists and tasks within the folder will also be permanently deleted
+- Using folderName is risky as names may not be unique across different spaces`,
     inputSchema: {
         type: "object",
         properties: {
@@ -163,23 +212,18 @@ export async function handleCreateFolder(parameters) {
     try {
         // Create the folder
         const newFolder = await folderService.createFolder(targetSpaceId, folderData);
-        return {
-            content: [{
-                    type: "text",
-                    text: JSON.stringify({
-                        id: newFolder.id,
-                        name: newFolder.name,
-                        space: {
-                            id: newFolder.space.id,
-                            name: newFolder.space.name
-                        },
-                        message: `Folder "${newFolder.name}" created successfully`
-                    }, null, 2)
-                }]
-        };
+        return sponsorService.createResponse({
+            id: newFolder.id,
+            name: newFolder.name,
+            space: {
+                id: newFolder.space.id,
+                name: newFolder.space.name
+            },
+            message: `Folder "${newFolder.name}" created successfully`
+        }, true);
     }
     catch (error) {
-        throw new Error(`Failed to create folder: ${error.message}`);
+        return sponsorService.createErrorResponse(`Failed to create folder: ${error.message}`);
     }
 }
 /**
@@ -215,26 +259,17 @@ export async function handleGetFolder(parameters) {
     try {
         // Get the folder
         const folder = await folderService.getFolder(targetFolderId);
-        return {
-            content: [{
-                    type: "text",
-                    text: JSON.stringify({
-                        id: folder.id,
-                        name: folder.name,
-                        lists: folder.lists.map((list) => ({
-                            id: list.id,
-                            name: list.name
-                        })),
-                        space: {
-                            id: folder.space.id,
-                            name: folder.space.name
-                        }
-                    }, null, 2)
-                }]
-        };
+        return sponsorService.createResponse({
+            id: folder.id,
+            name: folder.name,
+            space: {
+                id: folder.space.id,
+                name: folder.space.name
+            }
+        }, true);
     }
     catch (error) {
-        throw new Error(`Failed to retrieve folder: ${error.message}`);
+        return sponsorService.createErrorResponse(`Failed to retrieve folder: ${error.message}`);
     }
 }
 /**
@@ -280,23 +315,18 @@ export async function handleUpdateFolder(parameters) {
     try {
         // Update the folder
         const updatedFolder = await folderService.updateFolder(targetFolderId, updateData);
-        return {
-            content: [{
-                    type: "text",
-                    text: JSON.stringify({
-                        id: updatedFolder.id,
-                        name: updatedFolder.name,
-                        space: {
-                            id: updatedFolder.space.id,
-                            name: updatedFolder.space.name
-                        },
-                        message: `Folder "${updatedFolder.name}" updated successfully`
-                    }, null, 2)
-                }]
-        };
+        return sponsorService.createResponse({
+            id: updatedFolder.id,
+            name: updatedFolder.name,
+            space: {
+                id: updatedFolder.space.id,
+                name: updatedFolder.space.name
+            },
+            message: `Folder "${updatedFolder.name}" updated successfully`
+        }, true);
     }
     catch (error) {
-        throw new Error(`Failed to update folder: ${error.message}`);
+        return sponsorService.createErrorResponse(`Failed to update folder: ${error.message}`);
     }
 }
 /**
@@ -335,16 +365,12 @@ export async function handleDeleteFolder(parameters) {
         const folderName = folder.name;
         // Delete the folder
         await folderService.deleteFolder(targetFolderId);
-        return {
-            content: [{
-                    type: "text",
-                    text: JSON.stringify({
-                        message: `Folder "${folderName}" deleted successfully`
-                    }, null, 2)
-                }]
-        };
+        return sponsorService.createResponse({
+            success: true,
+            message: `Folder "${folderName || targetFolderId}" deleted successfully`
+        }, true);
     }
     catch (error) {
-        throw new Error(`Failed to delete folder: ${error.message}`);
+        return sponsorService.createErrorResponse(`Failed to delete folder: ${error.message}`);
     }
 }

package/build/tools/index.js

@@ -6,6 +6,6 @@
  * for better maintainability.
  */
 export * from './workspace.js';
-export * from './task.js';
+export * from './task/index.js';
 export * from './list.js';
 export * from './folder.js';