@browserstack/mcp-server 1.4.1-beta.1 → 1.4.1

package/dist/tools/testmanagement-utils/TCG-utils/api.js

@@ -1,5 +1,5 @@
 import { apiClient } from "../../../lib/apiClient.js";
-import { TCG_TRIGGER_URL, TCG_POLL_URL, FETCH_DETAILS_URL, FORM_FIELDS_URL, BULK_CREATE_URL, TC_DETAILS_MAX_BATCH, } from "./config.js";
+import { TCG_TRIGGER_URL, TCG_POLL_URL, FETCH_DETAILS_URL, FORM_FIELDS_URL, BULK_CREATE_URL, TC_DETAILS_MAX_BATCH, TCG_POLL_INTERVAL_MS, TCG_POLL_MAX_WAIT_MS, } from "./config.js";
 import { createTestCasePayload } from "./helpers.js";
 import { getBrowserStackAuth } from "../../../lib/get-auth.js";
 import { getTMBaseURL } from "../../../lib/tm-base-url.js";
@@ -118,9 +118,15 @@ export async function pollTestCaseDetails(traceRequestId, config) {
     let done = false;
     const tmBaseUrl = await getTMBaseURL(config);
     const TCG_POLL_URL_VALUE = TCG_POLL_URL(tmBaseUrl);
+    const deadline = Date.now() + TCG_POLL_MAX_WAIT_MS;
     while (!done) {
+        // Bail out if the backend never sends a "termination" message, so a stuck
+        // job cannot keep this loop (and its callers) alive forever.
+        if (Date.now() > deadline) {
+            throw new Error(`TCG test-case detail polling timed out after ${TCG_POLL_MAX_WAIT_MS}ms (trace ${traceRequestId})`);
+        }
         // add a bit of jitter to avoid synchronized polling storms
-        await new Promise((r) => setTimeout(r, 10000 + Math.random() * 5000));
+        await new Promise((r) => setTimeout(r, TCG_POLL_INTERVAL_MS + Math.random() * 5000));
         const poll = await apiClient.post({
             url: `${TCG_POLL_URL_VALUE}?x-bstack-traceRequestId=${encodeURIComponent(traceRequestId)}`,
             headers: {
@@ -159,7 +165,20 @@ export async function pollScenariosTestDetails(args, traceId, context, documentI
     const TCG_POLL_URL_VALUE = TCG_POLL_URL(tmBaseUrl);
     // Promisify interval-style polling using a wrapper
     await new Promise((resolve, reject) => {
-        const intervalId = setInterval(async () => {
+        const timers = {};
+        const stop = () => {
+            if (timers.interval)
+                clearInterval(timers.interval);
+            if (timers.timeout)
+                clearTimeout(timers.timeout);
+        };
+        // Hard wall-clock deadline: if the backend never sends a "termination"
+        // message, reject instead of letting the interval fire forever.
+        timers.timeout = setTimeout(() => {
+            stop();
+            reject(new Error(`TCG scenario polling timed out after ${TCG_POLL_MAX_WAIT_MS}ms (trace ${traceId})`));
+        }, TCG_POLL_MAX_WAIT_MS);
+        timers.interval = setInterval(async () => {
             try {
                 const poll = await apiClient.post({
                     url: `${TCG_POLL_URL_VALUE}?x-bstack-traceRequestId=${encodeURIComponent(traceId)}`,
@@ -169,7 +188,7 @@ export async function pollScenariosTestDetails(args, traceId, context, documentI
                     body: {},
                 });
                 if (poll.status !== 200) {
-                    clearInterval(intervalId);
+                    stop();
                     reject(new Error(`Polling error: ${poll.statusText || poll.status}`));
                     return;
                 }
@@ -221,16 +240,16 @@ export async function pollScenariosTestDetails(args, traceId, context, documentI
                         }
                     }
                     if (msg.type === "termination") {
-                        clearInterval(intervalId);
+                        stop();
                         resolve();
                     }
                 }
             }
             catch (err) {
-                clearInterval(intervalId);
+                stop();
                 reject(err);
             }
-        }, 10000); // 10 second interval
+        }, TCG_POLL_INTERVAL_MS);
     });
     // once all detail fetches are triggered, wait for them to complete
     const detailsList = await Promise.all(detailPromises);

package/dist/tools/testmanagement-utils/TCG-utils/config.d.ts

@@ -1,4 +1,6 @@
 export declare const TC_DETAILS_MAX_BATCH = 10;
+export declare const TCG_POLL_INTERVAL_MS = 10000;
+export declare const TCG_POLL_MAX_WAIT_MS: number;
 export declare const TCG_TRIGGER_URL: (baseUrl: string) => string;
 export declare const TCG_POLL_URL: (baseUrl: string) => string;
 export declare const FETCH_DETAILS_URL: (baseUrl: string) => string;

package/dist/tools/testmanagement-utils/TCG-utils/config.js

@@ -1,4 +1,9 @@
 export const TC_DETAILS_MAX_BATCH = 10;
+// Wall-clock bounds for the TCG generation polling loops. Without a hard
+// deadline, a job that never emits a "termination" message keeps the loop
+// (and every caller awaiting it) alive forever.
+export const TCG_POLL_INTERVAL_MS = 10_000;
+export const TCG_POLL_MAX_WAIT_MS = 10 * 60 * 1000; // 10 minutes
 export const TCG_TRIGGER_URL = (baseUrl) => `${baseUrl}/api/v1/integration/tcg/test-generation/suggest-test-cases`;
 export const TCG_POLL_URL = (baseUrl) => `${baseUrl}/api/v1/integration/tcg/test-generation/test-cases-polling`;
 export const FETCH_DETAILS_URL = (baseUrl) => `${baseUrl}/api/v1/integration/tcg/test-generation/fetch-test-case-details`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@browserstack/mcp-server",
-  "version": "1.4.1-beta.1",
+  "version": "1.4.1",
   "description": "BrowserStack's Official MCP Server",
   "mcpName": "io.github.browserstack/mcp-server",
   "main": "dist/index.js",