@hauptsache.net/clickup-mcp 1.1.0 → 1.2.0

package/dist/tools/task-tools.js

@@ -1,112 +1,19 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.registerTaskToolsRead = registerTaskToolsRead;
-exports.registerTaskToolsWrite = registerTaskToolsWrite;
 exports.generateTaskMetadata = generateTaskMetadata;
 const zod_1 = require("zod");
 const clickup_text_1 = require("../clickup-text");
 const config_1 = require("../shared/config");
 const utils_1 = require("../shared/utils");
-// Shared schemas for task parameters
-const taskNameSchema = zod_1.z.string().min(1).describe("The name/title of the task");
-const taskDescriptionSchema = zod_1.z.string().optional().describe("Optional description for the task");
-const taskPrioritySchema = zod_1.z.enum(["urgent", "high", "normal", "low"]).optional().describe("Optional priority level");
-const taskDueDateSchema = zod_1.z.string().optional().describe("Optional due date as ISO date string (e.g., '2024-10-06T23:59:59+02:00')");
-const taskStartDateSchema = zod_1.z.string().optional().describe("Optional start date as ISO date string (e.g., '2024-10-06T09:00:00+02:00')");
-const taskTimeEstimateSchema = zod_1.z.number().optional().describe("Optional time estimate in hours (will be converted to milliseconds)");
-const taskTagsSchema = zod_1.z.array(zod_1.z.string()).optional().describe("Optional array of tag names");
-// Shared utility functions
-function createAssigneeDescription(userData) {
-    const user = userData.user;
-    return `Optional array of user IDs to assign to the task (defaults to current user: ${user.username} - ${user.email} (${user.id}))`;
-}
-function convertPriorityToNumber(priority) {
-    switch (priority) {
-        case "urgent": return 1;
-        case "high": return 2;
-        case "normal": return 3;
-        case "low": return 4;
-        default: return 3;
-    }
-}
-function convertPriorityToString(priority) {
-    const priorityMap = { 1: 'urgent', 2: 'high', 3: 'normal', 4: 'low' };
-    return priorityMap[priority] || 'unknown';
-}
-function formatTimeEstimate(hours) {
-    const displayHours = Math.floor(hours);
-    const displayMinutes = Math.round((hours - displayHours) * 60);
-    return displayHours > 0 ? `${displayHours}h ${displayMinutes}m` : `${displayMinutes}m`;
-}
-function buildTaskRequestBody(params, currentUserId) {
-    const requestBody = {};
-    if (params.name !== undefined) {
-        requestBody.name = params.name;
-    }
-    if (params.priority !== undefined) {
-        requestBody.priority = convertPriorityToNumber(params.priority);
-    }
-    if (params.due_date !== undefined) {
-        requestBody.due_date = new Date(params.due_date).getTime();
-    }
-    if (params.start_date !== undefined) {
-        requestBody.start_date = new Date(params.start_date).getTime();
-    }
-    if (params.time_estimate !== undefined) {
-        requestBody.time_estimate = Math.round(params.time_estimate * 60 * 60 * 1000);
-    }
-    if (params.tags !== undefined && params.tags.length > 0) {
-        requestBody.tags = params.tags;
-    }
-    if (params.assignees !== undefined) {
-        requestBody.assignees = params.assignees;
-    }
-    else if (currentUserId) {
-        requestBody.assignees = [currentUserId];
-    }
-    if (params.parent !== undefined) {
-        requestBody.parent = params.parent;
-    }
-    return requestBody;
-}
-function formatTaskResponse(task, operation, params, userData) {
-    const responseLines = [
-        `Task ${operation} successfully!`,
-        `task_id: ${task.id}`,
-        `name: ${task.name}`,
-        ...(operation === 'created' ? [`url: ${task.url}`] : []),
-        `status: ${task.status?.status || 'Unknown'}`,
-        `assignees: ${task.assignees?.map((a) => `${a.username} (${a.id})`).join(', ') || 'None'}`,
-        ...(operation === 'created' && params.list_id ? [`list_id: ${params.list_id}`] : []),
-        ...(operation === 'updated' ? [
-            `updated_by: ${userData.user.username}`,
-            `updated_at: ${timestampToIso(Date.now())}`
-        ] : [])
-    ];
-    if (params.priority !== undefined || task.priority) {
-        const priority = task.priority ? convertPriorityToString(task.priority.priority) :
-            params.priority ? params.priority : 'unknown';
-        responseLines.push(`priority: ${priority}`);
-    }
-    if (params.due_date !== undefined) {
-        responseLines.push(`due_date: ${params.due_date}`);
-    }
-    if (params.start_date !== undefined) {
-        responseLines.push(`start_date: ${params.start_date}`);
-    }
-    if (params.time_estimate !== undefined) {
-        responseLines.push(`time_estimate: ${formatTimeEstimate(params.time_estimate)}`);
-    }
-    if (params.tags !== undefined && params.tags.length > 0) {
-        responseLines.push(`tags: ${params.tags.join(', ')}`);
-    }
-    if (params.parent !== undefined) {
-        responseLines.push(`parent_task: ${params.parent}`);
-    }
-    return responseLines;
-}
+const image_processing_1 = require("../shared/image-processing");
+// Read-specific utility functions
 function registerTaskToolsRead(server, userData) {
-    server.tool("getTaskById", "Get a Clickup task with images and comments by ID", {
+    server.tool("getTaskById", [
+        "Get a ClickUp task with images and comments by ID.",
+        "Always use this URL when referencing tasks in conversations or sharing with others.",
+        "The response provides complete context including task details, comments, and status history."
+    ].join("\n"), {
         id: zod_1.z
             .string()
             .min(6)
@@ -137,242 +44,55 @@ function registerTaskToolsRead(server, userData) {
         }
         // 5. Combine task details with processed event blocks
         const allContentBlocks = [...taskDetailContentBlocks, ...processedEventBlocks];
-        // 6. Limit images
-        const limitedContent = (0, utils_1.limitImages)(allContentBlocks, config_1.CONFIG.maxImages);
+        // 6. Download images with smart size limiting
+        const limitedContent = await (0, image_processing_1.downloadImages)(allContentBlocks);
         return {
             content: limitedContent,
         };
     });
 }
-function registerTaskToolsWrite(server, userData) {
-    server.tool("addComment", [
-        "Adds a comment to a specific task.",
-        "BEST PRACTICES: Include progress information and current status updates.",
-        "If external links are provided, verify they are publicly accessible and incorporate relevant information from those links into the comment.",
-        "Check the task's current status - if it's in 'backlog' or similar inactive states, suggest moving it to an active status like 'in progress' when work is being done."
-    ].join("\n"), {
-        task_id: zod_1.z.string().min(6).max(9).describe("The 6-9 character task ID to comment on"),
-        comment: zod_1.z.string().min(1).describe("The comment text to add to the task"),
-    }, async ({ task_id, comment }) => {
-        try {
-            const requestBody = {
-                comment_text: comment,
-                notify_all: true
-            };
-            const response = await fetch(`https://api.clickup.com/api/v2/task/${task_id}/comment`, {
-                method: 'POST',
-                headers: {
-                    Authorization: config_1.CONFIG.apiKey,
-                    'Content-Type': 'application/json'
-                },
-                body: JSON.stringify(requestBody)
-            });
-            if (!response.ok) {
-                const errorData = await response.json().catch(() => ({}));
-                throw new Error(`Error adding comment: ${response.status} ${response.statusText} - ${JSON.stringify(errorData)}`);
-            }
-            const commentData = await response.json();
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: [
-                            `Comment added successfully!`,
-                            `comment_id: ${commentData.id || 'N/A'}`,
-                            `task_id: ${task_id}`,
-                            `comment: ${comment}`,
-                            `date: ${timestampToIso(commentData.date || Date.now())}`,
-                            `user: ${commentData.user?.username || 'Current user'}`,
-                        ].join('\n')
-                    }
-                ],
-            };
-        }
-        catch (error) {
-            console.error('Error adding comment:', error);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `Error adding comment: ${error instanceof Error ? error.message : 'Unknown error'}`,
-                    },
-                ],
-            };
-        }
-    });
-    server.tool("updateTask", [
-        "Updates various aspects of an existing task.",
-        "Use getListInfo first to see valid status options.",
-        "SAFETY FEATURE: Description updates are APPEND-ONLY to prevent data loss - existing content is preserved.",
-        "IMPORTANT: When updating tasks (especially when booking time or adding progress), ensure the status makes sense for the work being done - tasks in 'backlog' or 'closed' states usually shouldn't have active work.",
-        "Suggest appropriate status transitions."
-    ].join("\n"), {
-        task_id: zod_1.z.string().min(6).max(9).describe("The 6-9 character task ID to update"),
-        name: taskNameSchema.optional(),
-        append_description: zod_1.z.string().optional().describe("Optional markdown content to APPEND to existing task description (preserves existing content for safety)"),
-        status: zod_1.z.string().optional().describe("Optional new status name - use getListInfo to see valid options"),
-        priority: taskPrioritySchema,
-        due_date: taskDueDateSchema,
-        start_date: taskStartDateSchema,
-        time_estimate: taskTimeEstimateSchema,
-        tags: taskTagsSchema.describe("Optional array of tag names (will replace existing tags)"),
-        assignees: zod_1.z.array(zod_1.z.string()).optional().describe(createAssigneeDescription(userData))
-    }, async ({ task_id, name, append_description, status, priority, due_date, start_date, time_estimate, tags, assignees }) => {
-        try {
-            const userData = await (0, utils_1.getCurrentUser)();
-            // Get task details including current markdown description
-            const taskResponse = await fetch(`https://api.clickup.com/api/v2/task/${task_id}?include_markdown_description=true`, {
-                headers: { Authorization: config_1.CONFIG.apiKey },
-            });
-            if (!taskResponse.ok) {
-                throw new Error(`Error fetching task: ${taskResponse.status} ${taskResponse.statusText}`);
-            }
-            const taskData = await taskResponse.json();
-            // Handle append-only description update with markdown support
-            let finalDescription;
-            if (append_description) {
-                const currentDescription = taskData.markdown_description || "";
-                const timestamp = new Date().toISOString().split('T')[0]; // YYYY-MM-DD format
-                const separator = currentDescription.trim() ? "\n\n---\n" : "";
-                finalDescription = currentDescription + separator + `**Edit (${timestamp}):** ${append_description}`;
-            }
-            // Build update body using shared utility (without description since we handle it separately)
-            const updateBody = buildTaskRequestBody({
-                name, priority, due_date, start_date, time_estimate, tags, assignees
-            });
-            // Add markdown description if we have content to append
-            if (finalDescription !== undefined) {
-                updateBody.markdown_description = finalDescription;
-            }
-            // Add status field (not handled by buildTaskRequestBody since it's update-specific)
-            if (status !== undefined) {
-                updateBody.status = status;
-            }
-            // Handle assignees for updates (different from creates)
-            if (assignees !== undefined) {
-                updateBody.assignees = { add: assignees, rem: [] }; // Add new assignees, remove none
-            }
-            // Check if there's anything to update
-            if (Object.keys(updateBody).length === 0) {
-                return {
-                    content: [
-                        {
-                            type: "text",
-                            text: "No updates provided. Please specify at least one field to update.",
-                        },
-                    ],
-                };
-            }
-            // Update the task
-            const updateResponse = await fetch(`https://api.clickup.com/api/v2/task/${task_id}`, {
-                method: 'PUT',
-                headers: {
-                    Authorization: config_1.CONFIG.apiKey,
-                    'Content-Type': 'application/json'
-                },
-                body: JSON.stringify(updateBody)
-            });
-            if (!updateResponse.ok) {
-                const errorData = await updateResponse.json().catch(() => ({}));
-                throw new Error(`Error updating task: ${updateResponse.status} ${updateResponse.statusText} - ${JSON.stringify(errorData)}`);
-            }
-            const updatedTask = await updateResponse.json();
-            const responseLines = formatTaskResponse(updatedTask, 'updated', {
-                name, append_description, status, priority, due_date, start_date, time_estimate, tags, assignees
-            }, userData);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: responseLines.join('\n')
-                    }
-                ],
-            };
-        }
-        catch (error) {
-            console.error('Error updating task:', error);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `Error updating task: ${error instanceof Error ? error.message : 'Unknown error'}`,
-                    },
-                ],
-            };
-        }
-    });
-    server.tool("createTask", [
-        "Creates a new task in a specific list and assigns it to specified users (defaults to current user).",
-        "IMPORTANT: Before creating, always search for similar existing tasks first using searchTasks to avoid duplicates - ask the user if they want to use an existing task instead.",
-        "If related tasks exist, reference them with links in the description (format: https://app.clickup.com/t/TASK_ID).",
-        "Use getListInfo first to understand the list context and available statuses.",
-        "Task descriptions support full markdown formatting including **bold**, *italic*, lists, links, and code blocks."
-    ].join("\n"), {
-        list_id: zod_1.z.string().min(1).describe("The ID of the list where the task will be created"),
-        name: taskNameSchema,
-        description: zod_1.z.string().optional().describe("Optional markdown description for the task - supports full markdown formatting"),
-        priority: taskPrioritySchema,
-        due_date: taskDueDateSchema,
-        start_date: taskStartDateSchema,
-        time_estimate: taskTimeEstimateSchema,
-        tags: taskTagsSchema,
-        parent: zod_1.z.string().optional().describe("Optional parent task ID to create this as a subtask"),
-        assignees: zod_1.z.array(zod_1.z.string()).optional().describe(createAssigneeDescription(userData))
-    }, async ({ list_id, name, description, priority, due_date, start_date, time_estimate, tags, parent, assignees }) => {
-        try {
-            const userData = await (0, utils_1.getCurrentUser)();
-            const currentUserId = userData.user.id;
-            const requestBody = buildTaskRequestBody({
-                name, priority, due_date, start_date, time_estimate, tags, assignees, parent
-            }, currentUserId);
-            // Add markdown description if provided
-            if (description) {
-                requestBody.markdown_description = description;
-            }
-            const response = await fetch(`https://api.clickup.com/api/v2/list/${list_id}/task`, {
-                method: 'POST',
-                headers: {
-                    Authorization: config_1.CONFIG.apiKey,
-                    'Content-Type': 'application/json'
-                },
-                body: JSON.stringify(requestBody)
-            });
-            if (!response.ok) {
-                const errorData = await response.json().catch(() => ({}));
-                throw new Error(`Error creating task: ${response.status} ${response.statusText} - ${JSON.stringify(errorData)}`);
-            }
-            const createdTask = await response.json();
-            const responseLines = formatTaskResponse(createdTask, 'created', {
-                list_id, name, description, priority, due_date, start_date, time_estimate, tags, parent, assignees
-            }, userData);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: responseLines.join('\n')
-                    }
-                ],
-            };
+/**
+ * Fetch time entries for a specific task (all time, not date-limited for detail view)
+ */
+async function fetchTaskTimeEntries(taskId) {
+    try {
+        // Get all team members for assignee filter
+        const teamMembers = await (0, utils_1.getAllTeamMembers)();
+        const params = new URLSearchParams({
+            task_id: taskId,
+            include_location_names: 'true',
+            start_date: '0', // overwrite the default 30 days
+        });
+        if (teamMembers.length > 0) {
+            params.append('assignee', teamMembers.join(','));
         }
-        catch (error) {
-            console.error('Error creating task:', error);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `Error creating task: ${error instanceof Error ? error.message : 'Unknown error'}`,
-                    },
-                ],
-            };
+        const response = await fetch(`https://api.clickup.com/api/v2/team/${config_1.CONFIG.teamId}/time_entries?${params}`, {
+            headers: { Authorization: config_1.CONFIG.apiKey },
+        });
+        if (!response.ok) {
+            console.error(`Error fetching time entries for task ${taskId}: ${response.status} ${response.statusText}`);
+            return [];
         }
-    });
+        const data = await response.json();
+        return data.data || [];
+    }
+    catch (error) {
+        console.error('Error fetching task time entries:', error);
+        return [];
+    }
 }
 async function loadTaskContent(taskId) {
     const response = await fetch(`https://api.clickup.com/api/v2/task/${taskId}?include_markdown_description=true&include_subtasks=true`, { headers: { Authorization: config_1.CONFIG.apiKey } });
     const task = await response.json();
-    const content = await (0, clickup_text_1.processClickUpMarkdown)(task.markdown_description || "", task.attachments);
-    // Create the task metadata block using the helper function
-    const taskMetadata = await generateTaskMetadata(task);
+    const [taskMetadata, content] = await Promise.all([
+        // Create the task metadata block using the helper functions
+        (async () => {
+            const timeEntries = await fetchTaskTimeEntries(task.id);
+            return await generateTaskMetadata(task, timeEntries, true);
+        })(),
+        // process markdown and download images
+        (0, clickup_text_1.processClickUpMarkdown)(task.markdown_description || "", task.attachments),
+    ]);
     return [taskMetadata, ...content];
 }
 async function loadTaskComments(id) {
@@ -437,8 +157,11 @@ async function loadTimeInStatusHistory(taskId) {
                 events.push(event);
         }
         // Deduplicate events based on date and status name to avoid adding current_status if it's identical to the last history entry
-        const uniqueEvents = Array.from(new Map(events.map(event => [`${event.date}-${event.contentBlocks[0]?.text}`, event] // Keying by date and text content of first block
-        )).values());
+        const uniqueEvents = Array.from(new Map(events.map(event => {
+            const firstBlock = event.contentBlocks[0];
+            const textKey = firstBlock && 'text' in firstBlock ? firstBlock.text : 'unknown';
+            return [`${event.date}-${textKey}`, event];
+        })).values());
         return uniqueEvents;
     }
     catch (error) {
@@ -446,52 +169,6 @@ async function loadTimeInStatusHistory(taskId) {
         return [];
     }
 }
-/**
- * Helper function to fetch time entries for a task
- */
-async function getTaskTimeEntries(taskId) {
-    try {
-        const response = await fetch(`https://api.clickup.com/api/v2/team/${config_1.CONFIG.teamId}/time_entries?task_id=${taskId}`, {
-            headers: { Authorization: config_1.CONFIG.apiKey },
-        });
-        if (!response.ok) {
-            return null;
-        }
-        const data = await response.json();
-        if (!data.data || !Array.isArray(data.data) || data.data.length === 0) {
-            return null;
-        }
-        // Group time entries by user
-        const timeByUser = new Map();
-        data.data.forEach((entry) => {
-            const username = entry.user?.username || 'Unknown User';
-            const currentTime = timeByUser.get(username) || 0;
-            const entryDurationMs = parseInt(entry.duration) || 0;
-            timeByUser.set(username, currentTime + entryDurationMs);
-        });
-        // Format results
-        const userTimeEntries = [];
-        let totalTimeMs = 0;
-        for (const [username, totalMs] of timeByUser.entries()) {
-            const hours = totalMs / (1000 * 60 * 60);
-            const displayHours = Math.floor(hours);
-            const displayMinutes = Math.round((hours - displayHours) * 60);
-            const timeDisplay = displayHours > 0 ?
-                `${displayHours}h ${displayMinutes}m` :
-                `${displayMinutes}m`;
-            userTimeEntries.push(`${username}: ${timeDisplay}`);
-            totalTimeMs += totalMs;
-        }
-        if (userTimeEntries.length === 0) {
-            return null;
-        }
-        return userTimeEntries.join(', ');
-    }
-    catch (error) {
-        console.error(`Error fetching time entries for task ${taskId}:`, error);
-        return null;
-    }
-}
 /**
  * Formats timestamp to ISO string with local timezone (not UTC)
  */
@@ -510,10 +187,43 @@ function timestampToIso(timestamp) {
     const timezoneOffset = sign + String(offsetHours).padStart(2, '0') + ':' + String(offsetMinutes).padStart(2, '0');
     return `${year}-${month}-${day}T${hours}:${minutes}${timezoneOffset}`;
 }
+/**
+ * Helper function to filter and format time entries for a specific task
+ */
+function filterTaskTimeEntries(taskId, timeEntries) {
+    if (!timeEntries || timeEntries.length === 0) {
+        return null;
+    }
+    // Filter entries for this specific task
+    const taskEntries = timeEntries.filter((entry) => entry.task?.id === taskId);
+    if (taskEntries.length === 0) {
+        return null;
+    }
+    // Group time entries by user (same logic as original getTaskTimeEntries)
+    const timeByUser = new Map();
+    taskEntries.forEach((entry) => {
+        const username = entry.user?.username || 'Unknown User';
+        const currentTime = timeByUser.get(username) || 0;
+        const entryDurationMs = parseInt(entry.duration) || 0;
+        timeByUser.set(username, currentTime + entryDurationMs);
+    });
+    // Format results (same logic as original)
+    const userTimeEntries = [];
+    for (const [username, totalMs] of timeByUser.entries()) {
+        const hours = totalMs / (1000 * 60 * 60);
+        const displayHours = Math.floor(hours);
+        const displayMinutes = Math.round((hours - displayHours) * 60);
+        const timeDisplay = displayHours > 0 ?
+            `${displayHours}h ${displayMinutes}m` :
+            `${displayMinutes}m`;
+        userTimeEntries.push(`${username}: ${timeDisplay}`);
+    }
+    return userTimeEntries.length > 0 ? userTimeEntries.join(', ') : null;
+}
 /**
  * Helper function to generate consistent task metadata
  */
-async function generateTaskMetadata(task) {
+async function generateTaskMetadata(task, timeEntries, isDetailView = false) {
     let spaceName = task.space?.name || 'Unknown Space';
     let spaceIdForDisplay = task.space?.id || 'N/A';
     if (spaceName === 'Unknown Space' && task.space?.id) {
@@ -524,6 +234,7 @@ async function generateTaskMetadata(task) {
     }
     const metadataLines = [
         `task_id: ${task.id}`,
+        `task_url: ${task.url}`,
         `name: ${task.name}`,
         `status: ${task.status.status}`,
         `date_created: ${timestampToIso(task.date_created)}`,
@@ -552,10 +263,13 @@ async function generateTaskMetadata(task) {
         const minutes = Math.floor((task.time_estimate % 3600000) / 60000);
         metadataLines.push(`time_estimate: ${hours}h ${minutes}m`);
     }
-    // Add time booked (tracked time entries)
-    const timeBooked = await getTaskTimeEntries(task.id);
-    if (timeBooked) {
-        metadataLines.push(`time_booked: ${timeBooked}`);
+    // Add time booked (tracked time entries) - only if timeEntries provided
+    if (timeEntries) {
+        const timeBooked = filterTaskTimeEntries(task.id, timeEntries);
+        if (timeBooked) {
+            const disclaimer = isDetailView ? "" : " (last 30 days)";
+            metadataLines.push(`time_booked${disclaimer}: ${timeBooked}`);
+        }
     }
     // Add tags if they exist
     if (task.tags && task.tags.length > 0) {
@@ -573,8 +287,6 @@ async function generateTaskMetadata(task) {
     if (task.subtasks && task.subtasks.length > 0) {
         metadataLines.push(`child_task_ids: ${task.subtasks.map((st) => st.id).join(', ')}`);
     }
-    // Add task URL
-    metadataLines.push(`url: ${task.url}`);
     // Add archived status if true
     if (task.archived) {
         metadataLines.push(`archived: true`);

package/dist/tools/task-write-tools.d.ts

@@ -0,0 +1,3 @@
+import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+export declare function registerTaskToolsWrite(server: McpServer, userData: any): void;
+//# sourceMappingURL=task-write-tools.d.ts.map

package/dist/tools/task-write-tools.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"task-write-tools.d.ts","sourceRoot":"","sources":["../../src/tools/task-write-tools.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAapE,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,GAAG,QA4StE"}