zephyr-scale-mcp-server 0.4.5 → 0.4.7

package/build/tool-handlers.js

@@ -12,8 +12,18 @@ export class ZephyrToolHandlers {
         const { test_case_key } = args;
         try {
             const response = await this.axiosInstance.get(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}`);
+            const testCase = response.data;
+            // Fetch the test script content and embed it inline.
+            // The base GET /testcases/{key} only returns a link to the script, not the content.
+            try {
+                const scriptResponse = await this.axiosInstance.get(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}/testscript`);
+                testCase.testScript = scriptResponse.data;
+            }
+            catch {
+                // Script fetch failed (e.g. no script yet) — leave testScript as the link object
+            }
             return {
-                content: [{ type: 'text', text: JSON.stringify(response.data, null, 2) }],
+                content: [{ type: 'text', text: JSON.stringify(testCase, null, 2) }],
             };
         }
         catch (error) {
@@ -240,6 +250,18 @@ export class ZephyrToolHandlers {
                 if (!tc) {
                     throw scriptUpdateError; // no test case data to build PUT payload, surface original error
                 }
+                // Fetch the existing test script to get its ID — required for the PUT to update in place
+                let existingScriptId;
+                try {
+                    const scriptRes = await this.axiosInstance.get(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}/testscript`);
+                    existingScriptId = scriptRes.data?.id;
+                }
+                catch {
+                    // no existing script — proceed without id
+                }
+                const testScriptPayload = { type: 'bdd', text: finalText };
+                if (existingScriptId)
+                    testScriptPayload.id = existingScriptId;
                 const putPayload = {
                     id: tc.id,
                     key: test_case_key,
@@ -247,7 +269,7 @@ export class ZephyrToolHandlers {
                     status: tc.status,
                     priority: tc.priority,
                     project: tc.project,
-                    testScript: { type: 'bdd', text: finalText },
+                    testScript: testScriptPayload,
                 };
                 for (const field of ['objective', 'precondition', 'estimatedTime', 'component', 'owner', 'folder']) {
                     if (tc[field] !== undefined && tc[field] !== null)
@@ -258,12 +280,15 @@ export class ZephyrToolHandlers {
                 if (tc.customFields && Object.keys(tc.customFields).length > 0)
                     putPayload.customFields = tc.customFields;
                 await this.axiosInstance.put(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}`, putPayload);
-                return {
-                    content: [{
-                            type: 'text',
-                            text: `✅ Updated ${test_case_key} with BDD content successfully (Cloud v2, via PUT fallback for migrated project)`,
-                        }],
-                };
+                // Note: the Zephyr Scale Cloud API's UpdateTestCaseInput does not include a testScript field,
+                // so the testScript embedded in the PUT payload is silently ignored.
+                // The only endpoint that updates script content (POST /testscript) requires the project to be active.
+                // For test cases migrated from Data Center to Cloud, the original project key may be deactivated,
+                // blocking script updates at the API level. Contact your Zephyr Scale admin to re-enable the project.
+                throw new McpError(ErrorCode.InternalError, `Cannot update test script for ${test_case_key}: the project "${test_case_key.replace(/-T\d+$/, '')}" is deactivated in Zephyr Scale Cloud. ` +
+                    `This test case was likely migrated from Data Center and retains its original project key prefix. ` +
+                    `POST /testcases/{key}/testscript requires an active project. ` +
+                    `Please ask your Zephyr Scale admin to re-enable the project, or recreate this test case under an active Cloud project.`);
             }
             // Primary path succeeded — optionally rename
             if (typeof name === 'string' && name.trim().length > 0) {

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "zephyr-scale-mcp-server",
-  "version": "0.4.5",
+  "version": "0.4.7",
   "description": "Model Context Protocol (MCP) server for Zephyr Scale test case management with comprehensive STEP_BY_STEP, PLAIN_TEXT, and BDD support",
   "type": "module",
   "main": "./build/index.js",
   "bin": {
-    "zephyr-scale-mcp": "./build/index.js"
+    "zephyr-scale-mcp": "build/index.js"
   },
   "files": [
     "build/",

package/src/tool-handlers.ts

@@ -24,8 +24,21 @@ export class ZephyrToolHandlers {
     const { test_case_key } = args;
     try {
       const response = await this.axiosInstance.get(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}`);
+      const testCase = response.data;
+
+      // Fetch the test script content and embed it inline.
+      // The base GET /testcases/{key} only returns a link to the script, not the content.
+      try {
+        const scriptResponse = await this.axiosInstance.get(
+          `${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}/testscript`
+        );
+        testCase.testScript = scriptResponse.data;
+      } catch {
+        // Script fetch failed (e.g. no script yet) — leave testScript as the link object
+      }
+
       return {
-        content: [{ type: 'text', text: JSON.stringify(response.data, null, 2) }],
+        content: [{ type: 'text', text: JSON.stringify(testCase, null, 2) }],
       };
     } catch (error) {
       throw new McpError(ErrorCode.InternalError, `Failed to get test case: ${this.formatError(error)}`);
@@ -260,6 +273,21 @@ export class ZephyrToolHandlers {
         if (!tc) {
           throw scriptUpdateError; // no test case data to build PUT payload, surface original error
         }
+
+        // Fetch the existing test script to get its ID — required for the PUT to update in place
+        let existingScriptId: number | undefined;
+        try {
+          const scriptRes = await this.axiosInstance.get(
+            `${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}/testscript`
+          );
+          existingScriptId = scriptRes.data?.id;
+        } catch {
+          // no existing script — proceed without id
+        }
+
+        const testScriptPayload: any = { type: 'bdd', text: finalText };
+        if (existingScriptId) testScriptPayload.id = existingScriptId;
+
         const putPayload: any = {
           id: tc.id,
           key: test_case_key,
@@ -267,7 +295,7 @@ export class ZephyrToolHandlers {
           status: tc.status,
           priority: tc.priority,
           project: tc.project,
-          testScript: { type: 'bdd', text: finalText },
+          testScript: testScriptPayload,
         };
         for (const field of ['objective', 'precondition', 'estimatedTime', 'component', 'owner', 'folder']) {
           if (tc[field] !== undefined && tc[field] !== null) putPayload[field] = tc[field];
@@ -277,12 +305,18 @@ export class ZephyrToolHandlers {
 
         await this.axiosInstance.put(`${this.jiraConfig.apiEndpoints.testcase}/${test_case_key}`, putPayload);
 
-        return {
-          content: [{
-            type: 'text',
-            text: `✅ Updated ${test_case_key} with BDD content successfully (Cloud v2, via PUT fallback for migrated project)`,
-          }],
-        };
+        // Note: the Zephyr Scale Cloud API's UpdateTestCaseInput does not include a testScript field,
+        // so the testScript embedded in the PUT payload is silently ignored.
+        // The only endpoint that updates script content (POST /testscript) requires the project to be active.
+        // For test cases migrated from Data Center to Cloud, the original project key may be deactivated,
+        // blocking script updates at the API level. Contact your Zephyr Scale admin to re-enable the project.
+        throw new McpError(
+          ErrorCode.InternalError,
+          `Cannot update test script for ${test_case_key}: the project "${test_case_key.replace(/-T\d+$/, '')}" is deactivated in Zephyr Scale Cloud. ` +
+          `This test case was likely migrated from Data Center and retains its original project key prefix. ` +
+          `POST /testcases/{key}/testscript requires an active project. ` +
+          `Please ask your Zephyr Scale admin to re-enable the project, or recreate this test case under an active Cloud project.`
+        );
       }
 
       // Primary path succeeded — optionally rename