@browserstack/mcp-server 1.2.27-beta.1 → 1.2.27-beta.3

package/dist/tools/rca-agent-utils/get-failed-test-id.d.ts

@@ -1,3 +1,4 @@
 import { TestStatus, FailedTestInfo, TestDetails } from "./types.js";
 export declare function getTestIds(buildId: string, authString: string, status?: TestStatus, includeFailureDetail?: boolean): Promise<FailedTestInfo[]>;
-export declare function extractFailedTestIds(hierarchy: TestDetails[], status?: TestStatus, includeFailureDetail?: boolean): FailedTestInfo[];
+export declare function extractTestIds(hierarchy: TestDetails[], status?: TestStatus, includeFailureDetail?: boolean): FailedTestInfo[];
+export declare const extractFailedTestIds: typeof extractTestIds;

package/dist/tools/rca-agent-utils/get-failed-test-id.js

@@ -1,12 +1,15 @@
 import logger from "../../logger.js";
 import { getAutomationBaseUrl } from "./constants.js";
+import { TestStatus, } from "./types.js";
 // Cap on the failure summary line — keep the response payload lean (we never
 // return full stack traces into the MCP client's context window).
 const ERROR_SUMMARY_MAX = 200;
 export async function getTestIds(buildId, authString, status, includeFailureDetail = false) {
+    // No `status` → no `test_statuses` filter → the endpoint returns ALL tests
+    // (the default). A `status` narrows both the query and the extraction.
     const baseUrl = `${getAutomationBaseUrl()}/ext/v1/builds/${buildId}/testRuns`;
     let url = status ? `${baseUrl}?test_statuses=${status}` : baseUrl;
-    let allFailedTests = [];
+    let allTests = [];
     let requestNumber = 0;
     // Construct Basic auth header
     const encodedCredentials = Buffer.from(authString).toString("base64");
@@ -24,10 +27,10 @@ export async function getTestIds(buildId, authString, status, includeFailureDeta
                 throw new Error(`Failed to fetch test runs: ${response.status} ${response.statusText}`);
             }
             const data = (await response.json());
-            // Extract failed IDs from current page
+            // Extract test IDs from the current page (all tests unless narrowed).
             if (data.hierarchy && data.hierarchy.length > 0) {
-                const currentFailedTests = extractFailedTestIds(data.hierarchy, status, includeFailureDetail);
-                allFailedTests = allFailedTests.concat(currentFailedTests);
+                const currentTests = extractTestIds(data.hierarchy, status, includeFailureDetail);
+                allTests = allTests.concat(currentTests);
             }
             // Check for pagination termination conditions
             if (!data.pagination?.has_next ||
@@ -42,44 +45,50 @@ export async function getTestIds(buildId, authString, status, includeFailureDeta
                 params.test_statuses = status;
             url = `${baseUrl}?${new URLSearchParams(params).toString()}`;
         }
-        // Return unique failed test IDs
-        return allFailedTests;
+        return allTests;
     }
     catch (error) {
-        logger.error("Error fetching failed tests:", error);
+        logger.error("Error fetching test runs:", error);
         throw error;
     }
 }
-export function extractFailedTestIds(hierarchy, status, includeFailureDetail = false) {
-    let failedTests = [];
+export function extractTestIds(hierarchy, status, includeFailureDetail = false) {
+    let tests = [];
     for (const node of hierarchy) {
-        // Match on status alone — the observability_url `details=<id>` check below
-        // already filters to real test nodes (suite/hook nodes carry no status and
-        // no such URL). Do NOT also require run_count: JUnit-uploaded builds report
-        // run_count=0 even for genuinely failed tests, which would drop them all.
-        if (node.details?.status === status) {
-            if (node.details?.observability_url) {
-                const idMatch = node.details.observability_url.match(/details=(\d+)/);
-                if (idMatch) {
-                    const entry = {
-                        test_id: idMatch[1],
-                        test_name: node.display_name || `Test ${idMatch[1]}`,
-                    };
-                    if (includeFailureDetail) {
-                        const signature = buildFailureSignature(node.details);
-                        if (signature)
-                            entry.failure = signature;
-                    }
-                    failedTests.push(entry);
+        // Include every REAL test node. The observability_url `details=<id>` check
+        // already filters out suite/hook nodes (they carry no such URL). We do NOT
+        // require run_count: JUnit-uploaded builds report run_count=0 even for
+        // genuine tests, which would drop them all.
+        // Status filtering is optional: when `status` is omitted we return ALL
+        // tests (the default); when provided we narrow to that status.
+        const nodeStatus = node.details?.status;
+        const statusMatches = status === undefined || nodeStatus === status;
+        if (statusMatches && node.details?.observability_url) {
+            const idMatch = node.details.observability_url.match(/details=(\d+)/);
+            if (idMatch) {
+                const entry = {
+                    test_id: idMatch[1],
+                    test_name: node.display_name || `Test ${idMatch[1]}`,
+                    status: nodeStatus,
+                };
+                // Failure signatures only exist for failed tests; include when asked.
+                if (includeFailureDetail && nodeStatus === TestStatus.FAILED) {
+                    const signature = buildFailureSignature(node.details);
+                    if (signature)
+                        entry.failure = signature;
                 }
+                tests.push(entry);
             }
         }
         if (node.children && node.children.length > 0) {
-            failedTests = failedTests.concat(extractFailedTestIds(node.children, status, includeFailureDetail));
+            tests = tests.concat(extractTestIds(node.children, status, includeFailureDetail));
         }
     }
-    return failedTests;
+    return tests;
 }
+// Back-compat alias — prefer extractTestIds. Kept so existing imports/tests
+// referencing the old name continue to resolve.
+export const extractFailedTestIds = extractTestIds;
 // Build a trimmed failure signature from a test node's `details`. Returns
 // undefined when no signal is available so the field is simply omitted.
 function buildFailureSignature(details) {

package/dist/tools/rca-agent-utils/types.d.ts

@@ -28,6 +28,7 @@ export interface TestFailureSignature {
 export interface FailedTestInfo {
     test_id: number;
     test_name: string;
+    status?: TestStatus;
     failure?: TestFailureSignature;
 }
 export declare enum RCAState {

package/dist/tools/rca-agent.js

@@ -157,7 +157,7 @@ export default function addRCATools(server, config) {
             return handleMCPError("listBuildId", server, config, error);
         }
     });
-    tools.listTestIds = server.tool("listTestIds", "List test IDs from a BrowserStack Automate build, optionally filtered by status", LIST_TEST_IDS_PARAMS, async (args) => {
+    tools.listTestIds = server.tool("listTestIds", "List all tests of a BrowserStack build (each with its status); optional status filter.", LIST_TEST_IDS_PARAMS, async (args) => {
         try {
             trackMCP("listTestIds", server.server.getClientVersion(), undefined, config);
             return await listTestIdsTool(args, config);

package/package.json

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