@dyno181cm.nexsoft/zentao_mcp 1.4.0 → 1.4.2

package/build/tools/commentTool.js

@@ -1,6 +1,6 @@
 import { z } from "zod";
 import { renderHistoryAndComments } from "../formatters/actionFormatter.js";
-import { buildMcpResponse } from "../utils/mcpResponse.js";
+import { buildMcpResponse, mcpText } from "../utils/mcpResponse.js";
 /**
  * Register the `zentao_get_comments` MCP tool on the given server.
  *
@@ -31,4 +31,15 @@ export function registerCommentTool(server, client) {
         ].join("\n\n");
         return buildMcpResponse(output, downloadedImages);
     });
+    server.registerTool("zentao_add_comment", {
+        description: "Add a comment/remark to a task or a bug",
+        inputSchema: {
+            type: z.enum(["task", "bug"]).describe("Type of the object ('task' or 'bug')"),
+            id: z.union([z.string(), z.number()]).describe("Task or Bug ID"),
+            comment: z.string().describe("Comment content"),
+        },
+    }, async ({ type, id, comment }) => {
+        const result = await client.addComment(type, id, comment);
+        return mcpText(result.message);
+    });
 }

package/build/zentaoClient.js

@@ -113,25 +113,177 @@ export class ZentaoClient {
             throw new Error('Login failed: Token not found in response');
         }
     }
+    /**
+     * Helper to generate standard action descriptions when they are not present in classic API.
+     */
+    generateActionDesc(act, userMap) {
+        const actorName = userMap[act.actor] || act.actor || 'User';
+        let extraName = act.extra || '';
+        if (userMap[extraName]) {
+            extraName = userMap[extraName];
+        }
+        else {
+            const resMap = {
+                'fixed': '已解决',
+                'design': '设计如此',
+                'duplicate': '重复Bug',
+                'external': '外部原因',
+                'notrepro': '无法重现',
+                'postponed': '延期处理',
+                'willnotfix': '不予解决',
+                'tostory': '转为需求',
+            };
+            if (resMap[extraName]) {
+                extraName = resMap[extraName];
+            }
+        }
+        switch (act.action) {
+            case 'opened':
+                return `由 <strong>${actorName}</strong> 创建。`;
+            case 'assigned':
+                return `由 <strong>${actorName}</strong> 指派给 <strong>${extraName}</strong>。`;
+            case 'started':
+                if (act.extra === 'autobychild') {
+                    return `由 <strong>${actorName}</strong> 启动子任务，该任务自动启动。`;
+                }
+                return `由 <strong>${actorName}</strong> 启动。`;
+            case 'finished':
+                return `由 <strong>${actorName}</strong> 完成。`;
+            case 'resolved':
+                return `由 <strong>${actorName}</strong> 解决，方案为 <strong>${extraName}</strong>。`;
+            case 'closed':
+                return `由 <strong>${actorName}</strong> 关闭。`;
+            case 'edited':
+                return `由 <strong>${actorName}</strong> 编辑。`;
+            case 'bugconfirmed':
+            case 'confirmed':
+                return `由 <strong>${actorName}</strong> 确认Bug。`;
+            case 'activated':
+                return `由 <strong>${actorName}</strong> 激活。`;
+            case 'commented':
+                return `由 <strong>${actorName}</strong> 备注。`;
+            case 'createchildren':
+                return `由 <strong>${actorName}</strong> 创建子任务 ${act.extra || ''}。`;
+            default:
+                if (act.action && act.action.startsWith('subtask')) {
+                    return `由 <strong>${actorName}</strong> ${act.action}。`;
+                }
+                return `由 <strong>${actorName}</strong> 执行了 <strong>${act.action || '未知'}</strong> 操作。`;
+        }
+    }
+    /**
+     * Fetches classic JSON API fallback data when REST API returns empty/errors.
+     */
+    async getFallbackClassicData(type, id) {
+        const webBaseUrl = this.baseUrl.includes('/api.php/v1')
+            ? this.baseUrl.split('/api.php/v1')[0]
+            : this.baseUrl.split('/api/v1')[0];
+        const url = `${webBaseUrl}/${type}-view-${id}.json`;
+        const res = await this.client.get(url, {
+            baseURL: '', // override baseURL
+        });
+        const responseData = typeof res.data === 'string' ? JSON.parse(res.data) : res.data;
+        if (responseData && responseData.status === 'success') {
+            const detailData = typeof responseData.data === 'string' ? JSON.parse(responseData.data) : responseData.data;
+            if (detailData) {
+                const entity = detailData[type];
+                if (entity) {
+                    const userMap = detailData.users || {};
+                    // Normalize user fields
+                    const userFields = ['openedBy', 'assignedTo', 'resolvedBy', 'closedBy', 'finishedBy', 'canceledBy', 'lastEditedBy'];
+                    for (const field of userFields) {
+                        const val = entity[field];
+                        if (val && typeof val === 'string') {
+                            entity[field] = {
+                                account: val,
+                                realname: userMap[val] || val
+                            };
+                        }
+                    }
+                    if (entity.assignedTo?.account === 'closed') {
+                        entity.assignedTo = null;
+                    }
+                    // Normalize actions list to sorted array
+                    let actionsArray = [];
+                    if (detailData.actions) {
+                        if (Array.isArray(detailData.actions)) {
+                            actionsArray = detailData.actions;
+                        }
+                        else if (typeof detailData.actions === 'object') {
+                            actionsArray = Object.values(detailData.actions);
+                            actionsArray.sort((a, b) => (Number(a.id) || 0) - (Number(b.id) || 0));
+                        }
+                    }
+                    // Generate descriptions if missing
+                    for (const act of actionsArray) {
+                        if (!act.desc) {
+                            act.desc = `${act.date || ''}, ` + this.generateActionDesc(act, userMap);
+                        }
+                    }
+                    entity.actions = actionsArray;
+                    return entity;
+                }
+            }
+        }
+        throw new Error(`Failed to fetch ${type} details from classic fallback API for ID ${id}`);
+    }
     /**
      * Retrieves details of a specific task.
      * Checks the cache first, otherwise fetches from API.
+     * Falls back to classic JSON API if REST API returns empty/invalid response.
      *
      * @param taskId The ID of the task.
      * @returns Resolves with the task details.
      */
     async getTaskDetails(taskId) {
-        return this.get(`/tasks/${taskId}`);
+        try {
+            const task = await this.get(`/tasks/${taskId}`);
+            if (!task || task === "") {
+                const fallbackTask = await this.getFallbackClassicData('task', taskId);
+                this.getCache.set(`/tasks/${taskId}`, { data: fallbackTask, timestamp: Date.now() });
+                return fallbackTask;
+            }
+            return task;
+        }
+        catch (error) {
+            try {
+                const fallbackTask = await this.getFallbackClassicData('task', taskId);
+                this.getCache.set(`/tasks/${taskId}`, { data: fallbackTask, timestamp: Date.now() });
+                return fallbackTask;
+            }
+            catch (fallbackError) {
+                throw error; // throw original error if fallback also fails
+            }
+        }
     }
     /**
      * Retrieves details of a specific bug.
      * Checks the cache first, otherwise fetches from API.
+     * Falls back to classic JSON API if REST API returns empty/invalid response.
      *
      * @param bugId The ID of the bug.
      * @returns Resolves with the bug details.
      */
     async getBugDetails(bugId) {
-        return this.get(`/bugs/${bugId}`);
+        try {
+            const bug = await this.get(`/bugs/${bugId}`);
+            if (!bug || bug === "") {
+                const fallbackBug = await this.getFallbackClassicData('bug', bugId);
+                this.getCache.set(`/bugs/${bugId}`, { data: fallbackBug, timestamp: Date.now() });
+                return fallbackBug;
+            }
+            return bug;
+        }
+        catch (error) {
+            try {
+                const fallbackBug = await this.getFallbackClassicData('bug', bugId);
+                this.getCache.set(`/bugs/${bugId}`, { data: fallbackBug, timestamp: Date.now() });
+                return fallbackBug;
+            }
+            catch (fallbackError) {
+                throw error; // throw original error if fallback also fails
+            }
+        }
     }
     /**
      * Downloads a file from Zentao API and pipes it to a local file path.
@@ -224,4 +376,38 @@ export class ZentaoClient {
     async updateBugStatus(bugId, action, payload) {
         return this.post(`/bugs/${bugId}/${action}`, payload);
     }
+    /**
+     * Adds a comment to a task or a bug using the classic action comment endpoint.
+     *
+     * @param type The object type ('task' or 'bug').
+     * @param id The ID of the object.
+     * @param comment The text of the comment to add.
+     */
+    async addComment(type, id, comment) {
+        if (!this.token) {
+            await this.login();
+        }
+        const webBaseUrl = this.baseUrl.includes('/api.php/v1')
+            ? this.baseUrl.split('/api.php/v1')[0]
+            : this.baseUrl.split('/api/v1')[0];
+        const url = `${webBaseUrl}/action-comment-${type}-${id}.json?zentaosid=${this.token}`;
+        // Clear the cache so subsequent fetches get the new comment
+        this.clearCache();
+        const params = new URLSearchParams();
+        params.append('comment', comment);
+        const res = await this.client.post(url, params, {
+            baseURL: '', // override baseURL
+            headers: {
+                'Content-Type': 'application/x-www-form-urlencoded'
+            }
+        });
+        if (res.status === 200 && typeof res.data === 'string' && res.data.includes('reload')) {
+            return { result: 'success', message: 'Comment added successfully' };
+        }
+        const responseData = typeof res.data === 'string' && res.data.startsWith('{') ? JSON.parse(res.data) : res.data;
+        if (responseData && (responseData.status === 'success' || responseData.result === 'success')) {
+            return { result: 'success', message: 'Comment added successfully' };
+        }
+        throw new Error(`Failed to add comment to ${type} #${id}. Response: ${typeof res.data === 'string' ? res.data.substring(0, 200) : JSON.stringify(res.data)}`);
+    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dyno181cm.nexsoft/zentao_mcp",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "Zentao MCP Server",
   "repository": {
     "type": "git",