@dyno181cm.nexsoft/zentao_mcp 1.2.1 → 1.2.3

package/README.md

@@ -20,7 +20,7 @@ A Model Context Protocol (MCP) server for integrating with the Zentao API. This
 1. Clone the repository:
 
    ```bash
-   git clone <repository-url>
+   git clone https://github.com/dyno-nexsoft/zentao_mcp.git
    cd zentao_mcp
    ```
 

package/build/index.js

@@ -8,7 +8,7 @@ dotenv.config();
 // Create the MCP server instance
 const server = new McpServer({
     name: "Zentao MCP Server",
-    version: "1.2.1"
+    version: "1.2.2"
 });
 // Register all Zentao-specific tools onto the server
 registerTools(server);

package/build/tools.js

@@ -1,8 +1,10 @@
 import { z } from "zod";
 import * as os from "os";
 import * as path from "path";
+import fs from "fs";
 import { ZentaoClient } from "./zentaoClient.js";
 const client = new ZentaoClient();
+const activeDownloads = new Map();
 /**
  * Registers Zentao-specific tools to the Model Context Protocol (MCP) server.
  *
@@ -129,10 +131,31 @@ export function registerTools(server) {
         const ext = extension ? (extension.startsWith('.') ? extension : `.${extension}`) : '';
         const targetPath = path.join(os.tmpdir(), `zentao_file_${fileId}${ext}`);
         try {
-            const savedPath = await client.downloadFile(fileId, targetPath);
-            return {
-                content: [{ type: "text", text: `File downloaded successfully to: ${savedPath}` }],
-            };
+            // If it already exists on disk and is not currently being downloaded
+            if (fs.existsSync(targetPath) && !activeDownloads.has(targetPath)) {
+                return {
+                    content: [{ type: "text", text: `File already exists locally at: ${targetPath}` }],
+                };
+            }
+            // If it is currently being downloaded, await the active promise
+            if (activeDownloads.has(targetPath)) {
+                await activeDownloads.get(targetPath);
+                return {
+                    content: [{ type: "text", text: `File downloaded successfully to: ${targetPath}` }],
+                };
+            }
+            // Start new download and track it
+            const downloadPromise = client.downloadFile(fileId, targetPath);
+            activeDownloads.set(targetPath, downloadPromise);
+            try {
+                const savedPath = await downloadPromise;
+                return {
+                    content: [{ type: "text", text: `File downloaded successfully to: ${savedPath}` }],
+                };
+            }
+            finally {
+                activeDownloads.delete(targetPath);
+            }
         }
         catch (error) {
             return {

package/build/zentaoClient.js

@@ -13,12 +13,38 @@ export class ZentaoClient {
     account = process.env.ZENTAO_ACCOUNT || '';
     password = process.env.ZENTAO_PASSWORD || '';
     baseUrl = process.env.ZENTAO_BASE_URL || '';
+    getCache = new Map();
+    cacheTtlMs = 2 * 60 * 1000; // 2 minutes TTL
+    /**
+     * Clears the in-memory cache for GET requests. Helpful for testing.
+     */
+    clearCache() {
+        this.getCache.clear();
+    }
+    /**
+     * Helper method to perform GET requests with caching.
+     *
+     * @param url The API endpoint path.
+     * @returns The response data.
+     */
+    async get(url) {
+        const cached = this.getCache.get(url);
+        const now = Date.now();
+        if (cached && (now - cached.timestamp < this.cacheTtlMs)) {
+            return cached.data;
+        }
+        const res = await this.client.get(url);
+        this.getCache.set(url, { data: res.data, timestamp: now });
+        return res.data;
+    }
     /**
      * Initializes the Axios client with base configurations and registers interceptors
      * to automatically inject token headers and handle token expiration (401 errors).
+     *
+     * @param customClient Optional custom AxiosInstance for dependency injection/testing.
      */
-    constructor() {
-        this.client = axios.create({
+    constructor(customClient) {
+        this.client = customClient || axios.create({
             baseURL: this.baseUrl,
             headers: {
                 'Content-Type': 'application/json',
@@ -58,45 +84,39 @@ export class ZentaoClient {
      * @throws {Error} If account/password env vars are missing or if the API request fails.
      */
     async login() {
-        try {
-            if (!this.account || !this.password) {
-                throw new Error('ZENTAO_ACCOUNT or ZENTAO_PASSWORD is not set in environment variables');
-            }
-            const response = await this.client.post('/tokens', {
-                account: this.account,
-                password: this.password,
-            });
-            if (response.data && response.data.token) {
-                this.token = response.data.token;
-            }
-            else {
-                throw new Error('Login failed: Token not found in response');
-            }
+        if (!this.account || !this.password) {
+            throw new Error('ZENTAO_ACCOUNT or ZENTAO_PASSWORD is not set in environment variables');
         }
-        catch (error) {
-            console.error('Failed to login to Zentao:', error.message);
-            throw error;
+        const response = await this.client.post('/tokens', {
+            account: this.account,
+            password: this.password,
+        });
+        if (response.data && response.data.token) {
+            this.token = response.data.token;
+        }
+        else {
+            throw new Error('Login failed: Token not found in response');
         }
     }
     /**
      * Retrieves details of a specific task.
+     * Checks the cache first, otherwise fetches from API.
      *
      * @param taskId The ID of the task.
      * @returns Resolves with the task details.
      */
     async getTaskDetails(taskId) {
-        const res = await this.client.get(`/tasks/${taskId}`);
-        return res.data;
+        return this.get(`/tasks/${taskId}`);
     }
     /**
      * Retrieves details of a specific bug.
+     * Checks the cache first, otherwise fetches from API.
      *
      * @param bugId The ID of the bug.
      * @returns Resolves with the bug details.
      */
     async getBugDetails(bugId) {
-        const res = await this.client.get(`/bugs/${bugId}`);
-        return res.data;
+        return this.get(`/bugs/${bugId}`);
     }
     /**
      * Downloads a file from Zentao API and pipes it to a local file path.

package/package.json

@@ -1,7 +1,15 @@
 {
   "name": "@dyno181cm.nexsoft/zentao_mcp",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Zentao MCP Server",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/dyno-nexsoft/zentao_mcp.git"
+  },
+  "bugs": {
+    "url": "https://github.com/dyno-nexsoft/zentao_mcp/issues"
+  },
+  "homepage": "https://github.com/dyno-nexsoft/zentao_mcp#readme",
   "main": "build/index.js",
   "bin": {
     "zentao_mcp": "build/index.js"
@@ -14,6 +22,7 @@
     "build": "tsc",
     "start": "node build/index.js",
     "dev": "tsc --watch",
+    "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js",
     "test:api": "tsx tests/testApi.ts"
   },
   "dependencies": {