@browserstack/mcp-server 1.3.1-beta.1 → 1.4.0-beta.1

package/dist/tools/testmanagement.js

@@ -19,6 +19,7 @@ import { getTestPlan, GetTestPlanSchema, } from "./testmanagement-utils/get-test
 import { listSubTestPlans, ListSubTestPlansSchema, } from "./testmanagement-utils/list-sub-testplans.js";
 import { getSubTestPlan, GetSubTestPlanSchema, } from "./testmanagement-utils/get-sub-testplan.js";
 import { elicitCredentialsIfSupported } from "../lib/elicit-credentials.js";
+import { NEEDS_PROJECT_ID, NEEDS_TEST_PLAN_ID } from "./tool-handoff.js";
 //TODO: Moving the traceMCP and catch block to the parent(server) function
 /**
  * Wrapper to call createProjectOrFolder util.
@@ -432,35 +433,40 @@ export async function getSubTestPlanTool(args, config, server) {
  */
 export default function addTestManagementTools(server, config) {
     const tools = {};
-    tools.createProjectOrFolder = server.tool("createProjectOrFolder", "Create a project and/or folder in BrowserStack Test Management.", CreateProjFoldSchema.shape, {
+    tools.createProjectOrFolder = server.tool("createProjectOrFolder", "Create a project and/or folder in BrowserStack Test Management." +
+        NEEDS_PROJECT_ID, CreateProjFoldSchema.shape, {
         title: "Create Project or Folder",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: false,
     }, (args) => createProjectOrFolderTool(args, config, server));
-    tools.createTestCase = server.tool("createTestCase", "Use this tool to create a test case in BrowserStack Test Management.", CreateTestCaseSchema.shape, {
+    tools.createTestCase = server.tool("createTestCase", "Use this tool to create a test case in BrowserStack Test Management." +
+        NEEDS_PROJECT_ID, CreateTestCaseSchema.shape, {
         title: "Create Test Case",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: false,
     }, (args) => createTestCaseTool(args, config, server));
-    tools.updateTestCase = server.tool("updateTestCase", "Update an existing test case in BrowserStack Test Management. Any subset of the following fields may be changed: name, description, preconditions, test_case_steps, owner, priority, case_type, automation_status, status, tags, issues, custom_fields. Only the supplied fields are modified.", UpdateTestCaseSchema.shape, {
+    tools.updateTestCase = server.tool("updateTestCase", "Update an existing test case in BrowserStack Test Management. Any subset of the following fields may be changed: name, description, preconditions, test_case_steps, owner, priority, case_type, automation_status, status, tags, issues, custom_fields. Only the supplied fields are modified." +
+        NEEDS_PROJECT_ID, UpdateTestCaseSchema.shape, {
         title: "Update Test Case",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: true,
         idempotentHint: true,
     }, (args) => updateTestCaseTool(args, config, server));
-    tools.listTestCases = server.tool("listTestCases", "List test cases in a project, optionally scoped to a specific folder. Omit folder_id to list all test cases in the project; provide folder_id (discoverable via listFolders) to list only that folder's cases. Supports filters: case_type, priority, pagination.", ListTestCasesSchema.shape, {
+    tools.listTestCases = server.tool("listTestCases", "List test cases in a project, optionally scoped to a specific folder. Omit folder_id to list all test cases in the project; provide folder_id (discoverable via listFolders) to list only that folder's cases. Supports filters: case_type, priority, pagination." +
+        NEEDS_PROJECT_ID, ListTestCasesSchema.shape, {
         title: "List Test Cases",
         readOnlyHint: true,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => listTestCasesTool(args, config, server));
-    tools.listFolders = server.tool("listFolders", "List folders in a BrowserStack Test Management project, returning each folder's id and name (plus case counts and sub-folder counts). Pass parent_id to list sub-folders under a specific folder instead of top-level folders.", ListFoldersSchema.shape, {
+    tools.listFolders = server.tool("listFolders", "List folders in a BrowserStack Test Management project, returning each folder's id and name (plus case counts and sub-folder counts). Pass parent_id to list sub-folders under a specific folder instead of top-level folders." +
+        NEEDS_PROJECT_ID, ListFoldersSchema.shape, {
         title: "List Folders",
         readOnlyHint: true,
         openWorldHint: false,
@@ -474,35 +480,39 @@ export default function addTestManagementTools(server, config) {
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => listTemplatesTool(args, config, server));
-    tools.createTestRun = server.tool("createTestRun", "Create a test run in BrowserStack Test Management.", CreateTestRunSchema.shape, {
+    tools.createTestRun = server.tool("createTestRun", "Create a test run in BrowserStack Test Management." + NEEDS_PROJECT_ID, CreateTestRunSchema.shape, {
         title: "Create Test Run",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: false,
     }, (args) => createTestRunTool(args, config, server));
-    tools.listTestRuns = server.tool("listTestRuns", "List test runs in a project with optional filters (date ranges, assignee, state, etc.)", ListTestRunsSchema.shape, {
+    tools.listTestRuns = server.tool("listTestRuns", "List test runs in a project with optional filters (date ranges, assignee, state, etc.)" +
+        NEEDS_PROJECT_ID, ListTestRunsSchema.shape, {
         title: "List Test Runs",
         readOnlyHint: true,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => listTestRunsTool(args, config, server));
-    tools.updateTestRun = server.tool("updateTestRun", "Update a test run's metadata and/or add test cases to it.", UpdateTestRunSchema.shape, {
+    tools.updateTestRun = server.tool("updateTestRun", "Update a test run's metadata and/or add test cases to it." +
+        NEEDS_PROJECT_ID, UpdateTestRunSchema.shape, {
         title: "Update Test Run",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: true,
         idempotentHint: true,
     }, (args) => updateTestRunTool(args, config, server));
-    tools.addTestResult = server.tool("addTestResult", "Add a test result to a specific test run via BrowserStack Test Management API.", AddTestResultSchema.shape, {
+    tools.addTestResult = server.tool("addTestResult", "Add a test result to a specific test run via BrowserStack Test Management API." +
+        NEEDS_PROJECT_ID, AddTestResultSchema.shape, {
         title: "Add Test Result",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: false,
     }, (args) => addTestResultTool(args, config, server));
-    tools.uploadProductRequirementFile = server.tool("uploadProductRequirementFile", "Upload files (e.g., PDRs, PDFs) to BrowserStack Test Management and retrieve a file mapping ID. This is utilized for generating test cases from files and is part of the Test Case Generator AI Agent in BrowserStack.", UploadFileSchema.shape, {
+    tools.uploadProductRequirementFile = server.tool("uploadProductRequirementFile", "Upload files (e.g., PDRs, PDFs) to BrowserStack Test Management and retrieve a file mapping ID. This is utilized for generating test cases from files and is part of the Test Case Generator AI Agent in BrowserStack." +
+        NEEDS_PROJECT_ID, UploadFileSchema.shape, {
         title: "Upload Product Requirement File",
         readOnlyHint: false,
         openWorldHint: false,
@@ -516,35 +526,43 @@ export default function addTestManagementTools(server, config) {
         destructiveHint: false,
         idempotentHint: false,
     }, (args, context) => createTestCasesFromFileTool(args, context, config, server));
-    tools.createLCASteps = server.tool("createLCASteps", "Generate Low Code Automation (LCA) steps for a test case in BrowserStack Test Management using the Low Code Automation Agent.", CreateLCAStepsSchema.shape, {
+    tools.createLCASteps = server.tool("createLCASteps", "Generate Low Code Automation (LCA) steps for a test case in BrowserStack Test Management using the Low Code Automation Agent." +
+        NEEDS_PROJECT_ID, CreateLCAStepsSchema.shape, {
         title: "Create LCA Steps",
         readOnlyHint: false,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: false,
     }, (args, context) => createLCAStepsTool(args, context, config, server));
-    tools.listTestPlans = server.tool("listTestPlans", "List test plans in a BrowserStack Test Management project. Returns each plan's identifier (TP-*), name, status, description, dates, and active/closed test-run counts. Supports pagination.", ListTestPlansSchema.shape, {
+    tools.listTestPlans = server.tool("listTestPlans", "List test plans in a BrowserStack Test Management project. Returns each plan's identifier (TP-*), name, status, description, dates, and active/closed test-run counts. Supports pagination." +
+        NEEDS_PROJECT_ID, ListTestPlansSchema.shape, {
         title: "List Test Plans",
         readOnlyHint: true,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => listTestPlansTool(args, config, server));
-    tools.getTestPlan = server.tool("getTestPlan", "Fetch a test plan by identifier (TP-*) from BrowserStack Test Management. Returns plan metadata, the full list of linked test runs, total test-case count across runs, and a status summary — suitable for generating test documentation or QA status reports.", GetTestPlanSchema.shape, {
+    tools.getTestPlan = server.tool("getTestPlan", "Fetch a test plan by identifier (TP-*) from BrowserStack Test Management. Returns plan metadata, the full list of linked test runs, total test-case count across runs, and a status summary — suitable for generating test documentation or QA status reports." +
+        NEEDS_PROJECT_ID +
+        NEEDS_TEST_PLAN_ID, GetTestPlanSchema.shape, {
         title: "Get Test Plan",
         readOnlyHint: true,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => getTestPlanTool(args, config, server));
-    tools.listSubTestPlans = server.tool("listSubTestPlans", "List sub-test-plans under a parent test plan (TP-*) in a Test Management project. Supports pagination.", ListSubTestPlansSchema.shape, {
+    tools.listSubTestPlans = server.tool("listSubTestPlans", "List sub-test-plans under a parent test plan (TP-*) in a Test Management project. Supports pagination." +
+        NEEDS_PROJECT_ID +
+        NEEDS_TEST_PLAN_ID, ListSubTestPlansSchema.shape, {
         title: "List Sub Test Plans",
         readOnlyHint: true,
         openWorldHint: false,
         destructiveHint: false,
         idempotentHint: true,
     }, (args) => listSubTestPlansTool(args, config, server));
-    tools.getSubTestPlan = server.tool("getSubTestPlan", "Fetch a sub-test-plan (STP-*) under a parent plan (TP-*). Returns metadata and linked test runs.", GetSubTestPlanSchema.shape, {
+    tools.getSubTestPlan = server.tool("getSubTestPlan", "Fetch a sub-test-plan (STP-*) under a parent plan (TP-*). Returns metadata and linked test runs." +
+        NEEDS_PROJECT_ID +
+        NEEDS_TEST_PLAN_ID, GetSubTestPlanSchema.shape, {
         title: "Get Sub Test Plan",
         readOnlyHint: true,
         openWorldHint: false,

package/dist/tools/tool-handoff.d.ts

@@ -0,0 +1,37 @@
+/**
+ * Precondition sentences appended to a tool's description.
+ *
+ * WHY THESE EXIST. Nothing routes an MCP call: the client's model picks a tool from the
+ * descriptions alone. A tool that needs an identifier the caller does not have is a dead
+ * end — the model either asks the user for something they also do not know, or gives up.
+ * These sentences turn that dead end into a HANDOFF by naming, in the description itself,
+ * where the missing identifier comes from.
+ *
+ * Point at a sibling tool whenever one can produce the id — it is faster and more
+ * predictable than an agent. Point at `askBrowserstackAI` only when NO tool here can.
+ *
+ * The one that matters most: 15 of the 17 Test Management tools require a project
+ * identifier and NONE of them accepts its absence, yet no tool in this server lists
+ * projects. "List my projects" is the first step of nearly every Test Management journey
+ * and it was unserved, which is exactly why that request did not reach any tool unless a
+ * user named one explicitly.
+ *
+ * Keep these as shared constants, not per-tool prose: the wording is a routing signal, and
+ * twenty hand-written variants drift into twenty different signals.
+ */
+/** No tool lists projects, so this genuinely has to go to the agent. */
+export declare const NEEDS_PROJECT_ID: string;
+/** A sibling tool can produce the id — prefer it over the agent. */
+export declare function needsIdFrom(idLabel: string, sourceTool: string): string;
+/** A test plan id (TP-*) comes from listTestPlans. */
+export declare const NEEDS_TEST_PLAN_ID: string;
+/** A build id comes from either build-lookup tool. */
+export declare const NEEDS_BUILD_ID: string;
+/** Session ids are not listable by any tool here. */
+export declare const NEEDS_SESSION_ID: string;
+/** A completed scan's ids come from startAccessibilityScan, or from the agent. */
+export declare const NEEDS_A11Y_SCAN_ID: string;
+/** Auth-config ids are not listable by any tool here. */
+export declare const NEEDS_A11Y_CONFIG_ID: string;
+/** Test ids come from listTestIds, which itself needs a build id. */
+export declare const NEEDS_TEST_IDS: string;

package/dist/tools/tool-handoff.js

@@ -0,0 +1,47 @@
+/**
+ * Precondition sentences appended to a tool's description.
+ *
+ * WHY THESE EXIST. Nothing routes an MCP call: the client's model picks a tool from the
+ * descriptions alone. A tool that needs an identifier the caller does not have is a dead
+ * end — the model either asks the user for something they also do not know, or gives up.
+ * These sentences turn that dead end into a HANDOFF by naming, in the description itself,
+ * where the missing identifier comes from.
+ *
+ * Point at a sibling tool whenever one can produce the id — it is faster and more
+ * predictable than an agent. Point at `askBrowserstackAI` only when NO tool here can.
+ *
+ * The one that matters most: 15 of the 17 Test Management tools require a project
+ * identifier and NONE of them accepts its absence, yet no tool in this server lists
+ * projects. "List my projects" is the first step of nearly every Test Management journey
+ * and it was unserved, which is exactly why that request did not reach any tool unless a
+ * user named one explicitly.
+ *
+ * Keep these as shared constants, not per-tool prose: the wording is a routing signal, and
+ * twenty hand-written variants drift into twenty different signals.
+ */
+/** No tool lists projects, so this genuinely has to go to the agent. */
+export const NEEDS_PROJECT_ID = " Requires a project identifier (PR-*). No tool here lists projects, so if you do not " +
+    'have one, call askBrowserstackAI with product "tm" and ask which projects exist, then ' +
+    "retry this tool with the identifier it returns.";
+/** A sibling tool can produce the id — prefer it over the agent. */
+export function needsIdFrom(idLabel, sourceTool) {
+    return ` Requires ${idLabel}. Call ${sourceTool} first if you do not have it.`;
+}
+/** A test plan id (TP-*) comes from listTestPlans. */
+export const NEEDS_TEST_PLAN_ID = needsIdFrom("a test plan identifier (TP-*)", "listTestPlans");
+/** A build id comes from either build-lookup tool. */
+export const NEEDS_BUILD_ID = needsIdFrom("a BrowserStack build id", "getBuildId or listBuildId");
+/** Session ids are not listable by any tool here. */
+export const NEEDS_SESSION_ID = " Requires a session id, which no tool here lists. If you only know the build, call " +
+    "getBuildId or listBuildId; if you have neither, call askBrowserstackAI with product " +
+    '"tra" and describe the run you mean.';
+/** A completed scan's ids come from startAccessibilityScan, or from the agent. */
+export const NEEDS_A11Y_SCAN_ID = " Requires the ids of a completed scan. They are returned by startAccessibilityScan; " +
+    'for a scan run earlier, call askBrowserstackAI with product "a11y" to locate it, since ' +
+    "no tool here lists past scans.";
+/** Auth-config ids are not listable by any tool here. */
+export const NEEDS_A11Y_CONFIG_ID = " Requires the numeric id returned by createAccessibilityAuthConfig. No tool here lists " +
+    "existing configurations, so if you do not have the id, call askBrowserstackAI with " +
+    'product "a11y".';
+/** Test ids come from listTestIds, which itself needs a build id. */
+export const NEEDS_TEST_IDS = needsIdFrom("test ids", "listTestIds");

package/package.json

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