Merge pull request #210 from manoj-k04/PMAA-20-Update-MCP-SDK

gaurav-singh-9227 · web-flow · commit 5977b3a51f4a · 2025-12-19T12:40:11.000+05:30
Chore : update dependencies and improve error handling
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -35,31 +35,31 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.18.1",
+    "@modelcontextprotocol/sdk": "^1.25.1",
     "@types/form-data": "^2.5.2",
-    "axios": "^1.8.4",
-    "browserstack-local": "^1.5.6",
-    "csv-parse": "^5.6.0",
-    "dotenv": "^16.5.0",
-    "form-data": "^4.0.2",
-    "pino": "^9.6.0",
-    "pino-pretty": "^13.0.0",
-    "sharp": "^0.34.1",
-    "uuid": "^11.1.0",
-    "webdriverio": "^9.13.0",
-    "zod": "^3.24.3"
+    "axios": "^1.13.2",
+    "browserstack-local": "^1.5.8",
+    "csv-parse": "^6.1.0",
+    "dotenv": "^17.2.3",
+    "form-data": "^4.0.5",
+    "pino": "^10.1.0",
+    "pino-pretty": "^13.1.3",
+    "sharp": "^0.34.5",
+    "uuid": "^13.0.0",
+    "webdriverio": "^9.21.0",
+    "zod": "^4.2.1"
   },
   "devDependencies": {
-    "@eslint/js": "^9.25.0",
-    "@types/csv-parse": "^1.1.12",
-    "@types/node": "^22.14.1",
-    "@types/uuid": "^10.0.0",
-    "eslint": "^9.25.0",
-    "prettier": "^3.5.3",
-    "tsx": "^4.19.3",
-    "typescript": "^5.8.3",
-    "typescript-eslint": "^8.30.1",
-    "vite": "^6.3.5",
-    "vitest": "^3.1.3"
+    "@eslint/js": "^9.39.2",
+    "@types/csv-parse": "^1.2.5",
+    "@types/node": "^25.0.3",
+    "@types/uuid": "^11.0.0",
+    "eslint": "^9.39.2",
+    "prettier": "^3.7.4",
+    "tsx": "^4.21.0",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.50.0",
+    "vite": "^7.3.0",
+    "vitest": "^4.0.16"
   }
 }
diff --git a/src/tools/accessibility.ts b/src/tools/accessibility.ts
@@ -50,16 +50,15 @@ function setupAuth(config: BrowserStackConfig): AuthCredentials {
   return { username, password };
 }
 
-function createErrorResponse(message: string, isError = true): CallToolResult {
+function createErrorResponse(message: string): CallToolResult {
   return {
     content: [
       {
         type: "text",
         text: message,
-        isError,
       },
     ],
-    isError,
+    isError: true,
   };
 }
 
diff --git a/src/tools/appautomate-utils/appium-sdk/formatter.ts b/src/tools/appautomate-utils/appium-sdk/formatter.ts
@@ -17,7 +17,6 @@ ${formattedInstructions}`;
       {
         type: "text",
         text: fullInstructions,
-        isError: false,
       },
     ],
   };
diff --git a/src/tools/appautomate-utils/appium-sdk/handler.ts b/src/tools/appautomate-utils/appium-sdk/handler.ts
@@ -124,7 +124,6 @@ export async function setupAppAutomateHandler(
       {
         type: "text",
         text: formatAppInstructionsWithNumbers(combinedInstructions),
-        isError: false,
       },
     ],
     isError: false,
diff --git a/src/tools/appautomate.ts b/src/tools/appautomate.ts
@@ -154,7 +154,6 @@ async function takeAppScreenshot(args: {
           type: "image",
           data: compressed,
           mimeType: "image/png",
-          name: `screenshot-${selectedDevice.device}-${Date.now()}`,
         },
       ],
     };
@@ -422,7 +421,6 @@ export default function addAppAutomationTools(
             {
               type: "text",
               text: `Failed to bootstrap project with BrowserStack App Automate SDK. Error: ${error_message}. Please open an issue on GitHub if the problem persists`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/applive.ts b/src/tools/applive.ts
@@ -128,7 +128,6 @@ export default function addAppLiveTools(
             {
               type: "text",
               text: `Failed to start app live session: ${error instanceof Error ? error.message : String(error)}`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/automate.ts b/src/tools/automate.ts
@@ -36,10 +36,9 @@ export async function fetchAutomationScreenshotsTool(
 
     const results = screenshots.map((screenshot, index) => ({
       type: "image" as const,
-      text: `Screenshot ${index + 1}`,
       data: screenshot.base64,
       mimeType: "image/png",
-      metadata: { url: screenshot.url },
+      _meta: { url: screenshot.url, index: index + 1 },
     }));
 
     return {
diff --git a/src/tools/get-failure-logs.ts b/src/tools/get-failure-logs.ts
@@ -89,9 +89,9 @@ export async function getFailureLogs(
         {
           type: "text",
           text: `No valid log types found for ${args.sessionType} session.\nErrors encountered:\n${errors.join("\n")}`,
-          isError: true,
         },
       ],
+      isError: true,
     };
   }
   let response;
@@ -156,7 +156,6 @@ export async function getFailureLogs(
     results.push({
       type: "text",
       text: `Errors encountered:\n${errors.join("\n")}`,
-      isError: true,
     });
   }
 
@@ -226,7 +225,6 @@ export default function registerGetFailureLogs(
               text: `Failed to fetch failure logs: ${
                 error instanceof Error ? error.message : "Unknown error"
               }`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/live.ts b/src/tools/live.ts
@@ -150,7 +150,6 @@ export default function addBrowserLiveTools(
             {
               type: "text" as const,
               text: `Failed to start a browser live session. Error: ${error}`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/observability.ts b/src/tools/observability.ts
@@ -97,7 +97,6 @@ export default function addObservabilityTools(
             {
               type: "text",
               text: `Failed to get failures in the last run. Error: ${error}. Please open an issue on GitHub if this is an issue with BrowserStack`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/sdk-utils/common/schema.ts b/src/tools/sdk-utils/common/schema.ts
@@ -49,31 +49,30 @@ export const SetUpPercyParamsShape = {
 };
 
 // Device schema for BrowserStack Automate (supports desktop and mobile)
-const DeviceSchema = z
-  .object({
-    platform: z
-      .enum(["windows", "macos", "android", "ios"])
-      .describe("Platform name, e.g. 'windows', 'macos', 'android', 'ios'"),
-    deviceName: z
-      .string()
-      .optional()
-      .describe(
-        "Device name for mobile platforms, e.g. 'iPhone 15', 'Samsung Galaxy S24'",
-      ),
-    osVersion: z
-      .string()
-      .describe("OS version, e.g. '11', 'Sequoia', '14', '17', 'latest'"),
-    browser: z
-      .string()
-      .optional()
-      .describe("Browser name, e.g. 'chrome', 'safari', 'edge', 'firefox'"),
-    browserVersion: z
-      .string()
-      .optional()
-      .describe(
-        "Browser version for desktop platforms only (windows, macos), e.g. '132', 'latest', 'oldest'. Not used for mobile devices (android, ios).",
-      ),
-  });
+const DeviceSchema = z.object({
+  platform: z
+    .enum(["windows", "macos", "android", "ios"])
+    .describe("Platform name, e.g. 'windows', 'macos', 'android', 'ios'"),
+  deviceName: z
+    .string()
+    .optional()
+    .describe(
+      "Device name for mobile platforms, e.g. 'iPhone 15', 'Samsung Galaxy S24'",
+    ),
+  osVersion: z
+    .string()
+    .describe("OS version, e.g. '11', 'Sequoia', '14', '17', 'latest'"),
+  browser: z
+    .string()
+    .optional()
+    .describe("Browser name, e.g. 'chrome', 'safari', 'edge', 'firefox'"),
+  browserVersion: z
+    .string()
+    .optional()
+    .describe(
+      "Browser version for desktop platforms only (windows, macos), e.g. '132', 'latest', 'oldest'. Not used for mobile devices (android, ios).",
+    ),
+});
 
 export const RunTestsOnBrowserStackParamsShape = {
   projectName: z
diff --git a/src/tools/sdk-utils/handler.ts b/src/tools/sdk-utils/handler.ts
@@ -206,7 +206,7 @@ export async function simulatePercyChangeHandler(
 
     if (Array.isArray(percyInstruction.content)) {
       percyInstruction.content = percyInstruction.content.map((item) => {
-        if (typeof item.text === "string") {
+        if (item.type === "text" && typeof item.text === "string") {
           const updatedText = item.text
             .replace(PERCY_REPLACE_REGEX, PERCY_SIMULATE_INSTRUCTION)
             .replace(PERCY_VERIFICATION_REGEX, "");
diff --git a/src/tools/testmanagement-utils/create-lca-steps.ts b/src/tools/testmanagement-utils/create-lca-steps.ts
@@ -187,7 +187,6 @@ export async function createLCASteps(
             {
               type: "text",
               text: `Error: ${error.message}. Please verify that the project identifier "${args.project_identifier}" and test case identifier "${args.test_case_identifier}" are correct.`,
-              isError: true,
             },
           ],
           isError: true,
diff --git a/src/tools/testmanagement-utils/create-testcase.ts b/src/tools/testmanagement-utils/create-testcase.ts
@@ -180,7 +180,6 @@ export async function createTestCase(
             text: `Failed to create test case: ${JSON.stringify(
               response.data,
             )}`,
-            isError: true,
           },
         ],
         isError: true,
diff --git a/src/tools/testmanagement-utils/list-testcases.ts b/src/tools/testmanagement-utils/list-testcases.ts
@@ -72,7 +72,6 @@ export async function listTestCases(
           {
             type: "text",
             text: `Failed to list test cases: ${JSON.stringify(resp_data)}`,
-            isError: true,
           },
         ],
         isError: true,
diff --git a/src/tools/testmanagement-utils/list-testruns.ts b/src/tools/testmanagement-utils/list-testruns.ts
@@ -59,7 +59,6 @@ export async function listTestRuns(
           {
             type: "text",
             text: `Failed to list test runs: ${JSON.stringify(data)}`,
-            isError: true,
           },
         ],
         isError: true,
diff --git a/src/tools/testmanagement-utils/testcase-from-file.ts b/src/tools/testmanagement-utils/testcase-from-file.ts
@@ -45,7 +45,6 @@ export async function createTestCasesFromFile(
         {
           type: "text",
           text: `Document with ID ${args.documentId} not found.`,
-          isError: true,
         },
       ],
       isError: true,
diff --git a/src/tools/testmanagement-utils/update-testcase.ts b/src/tools/testmanagement-utils/update-testcase.ts
@@ -105,7 +105,6 @@ export async function updateTestCase(
             text: `Failed to update test case: ${JSON.stringify(
               response.data,
             )}`,
-            isError: true,
           },
         ],
         isError: true,
@@ -154,9 +153,8 @@ The test case has been updated successfully and is now available in your Browser
           {
             type: "text",
             text: `Test case not found. Please verify the project_identifier ("${params.project_identifier}") and test_case_identifier ("${params.test_case_identifier}") are correct. Make sure to use actual values, not placeholders like "your_project_id".
-            
+
 Error details: ${JSON.stringify(err.response?.data || err.message)}`,
-            isError: true,
           },
         ],
         isError: true,
@@ -169,7 +167,6 @@ Error details: ${JSON.stringify(err.response?.data || err.message)}`,
           {
             type: "text",
             text: "Access denied. You don't have permission to update this test case.",
-            isError: true,
           },
         ],
         isError: true,
@@ -182,7 +179,6 @@ Error details: ${JSON.stringify(err.response?.data || err.message)}`,
         {
           type: "text",
           text: `Failed to update test case: ${errorMessage}. Please verify your credentials and try again.`,
-          isError: true,
         },
       ],
       isError: true,
diff --git a/src/tools/testmanagement-utils/update-testrun.ts b/src/tools/testmanagement-utils/update-testrun.ts
@@ -66,7 +66,6 @@ export async function updateTestRun(
           {
             type: "text",
             text: `Failed to update test run: ${JSON.stringify(data)}`,
-            isError: true,
           },
         ],
         isError: true,
diff --git a/src/tools/testmanagement-utils/upload-file.ts b/src/tools/testmanagement-utils/upload-file.ts
@@ -42,7 +42,6 @@ export async function uploadFile(
           {
             type: "text",
             text: `File ${file_path} does not exist.`,
-            isError: true,
           },
         ],
         isError: true,
@@ -122,7 +121,6 @@ export async function uploadFile(
           text: `Failed to upload file: ${
             error instanceof Error ? error.message : "Unknown error"
           }. Please check your credentials and try again.`,
-          isError: true,
         },
       ],
       isError: true,
diff --git a/src/tools/testmanagement.ts b/src/tools/testmanagement.ts
diff --git a/tests/tools/getFailureLogs.test.ts b/tests/tools/getFailureLogs.test.ts
diff --git a/tests/tools/testmanagement.test.ts b/tests/tools/testmanagement.test.ts

Original file line number	Diff line number	Diff line change
`@@ -50,16 +50,15 @@ function setupAuth(config: BrowserStackConfig): AuthCredentials {`
`50`	`50`	`return { username, password };`
`51`	`51`	`}`
`52`	`52`
`53`		`-function createErrorResponse(message: string, isError = true): CallToolResult {`
	`53`	`+function createErrorResponse(message: string): CallToolResult {`
`54`	`54`	`return {`
`55`	`55`	`content: [`
`56`	`56`	`{`
`57`	`57`	`type: "text",`
`58`	`58`	`text: message,`
`59`		`- isError,`
`60`	`59`	`},`
`61`	`60`	`],`
`62`		`- isError,`
	`61`	`+ isError: true,`
`63`	`62`	`};`
`64`	`63`	`}`
`65`	`64`
Original file line number	Diff line number	Diff line change
@@ -17,7 +17,6 @@ ${formattedInstructions}`;
`17`	`17`	`{`
`18`	`18`	`type: "text",`
`19`	`19`	`text: fullInstructions,`
`20`		`- isError: false,`
`21`	`20`	`},`
`22`	`21`	`],`
`23`	`22`	`};`
Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,6 @@ export async function setupAppAutomateHandler(`
`124`	`124`	`{`
`125`	`125`	`type: "text",`
`126`	`126`	`text: formatAppInstructionsWithNumbers(combinedInstructions),`
`127`		`- isError: false,`
`128`	`127`	`},`
`129`	`128`	`],`
`130`	`129`	`isError: false,`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,6 @@ export default function addAppLiveTools(`
`128`	`128`	`{`
`129`	`129`	`type: "text",`
`130`	`130`	text: `Failed to start app live session: ${error instanceof Error ? error.message : String(error)}`,
`131`		`- isError: true,`
`132`	`131`	`},`
`133`	`132`	`],`
`134`	`133`	`isError: true,`
Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,6 @@ export default function addBrowserLiveTools(`
`150`	`150`	`{`
`151`	`151`	`type: "text" as const,`
`152`	`152`	text: `Failed to start a browser live session. Error: ${error}`,
`153`		`- isError: true,`
`154`	`153`	`},`
`155`	`154`	`],`
`156`	`155`	`isError: true,`