feat(ask): askBrowserstackAI — mid-run permission relay, with production defaults

package.json

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

src/index.ts

@@ -43,9 +43,12 @@ async function main() {
 
 main().catch(console.error);
 
-// Ensure logs are flushed before exit
+// Ensure logs are flushed before exit. Optional-called because an exit handler must not
+// be the thing that throws: `flush` is absent from every `vi.mock("src/logger")` in the
+// suite, so any test file that loads this module used to fail the run with
+// `default.flush is not a function` depending on how vitest scheduled workers.
 process.on("exit", () => {
-  logger.flush();
+  logger.flush?.();
 });
 
 export { setLogger } from "./logger.js";

src/server-factory.ts

@@ -20,6 +20,7 @@ import addBuildInsightsTools from "./tools/build-insights.js";
 import { setupOnInitialized } from "./oninitialized.js";
 import { BrowserStackConfig } from "./lib/types.js";
 import addRCATools from "./tools/rca-agent.js";
+import addAskBrowserStackAITool from "./tools/ask-browserstack/register.js";
 
 /**
  * Wrapper class for BrowserStack MCP Server
@@ -61,6 +62,10 @@ export class BrowserStackMcpServer {
       addSelfHealTools,
       addBuildInsightsTools,
       addRCATools,
+      // Hands a plain-language task to BrowserStack's agent and relays its mid-run
+      // permission asks back to this client, so a write can be confirmed by the human
+      // sitting in front of it rather than refused for want of anyone to ask.
+      addAskBrowserStackAITool,
     ];
 
     toolAdders.forEach((adder) => {

src/tools/accessibility.ts

@@ -14,6 +14,7 @@ import { getBrowserStackAuth } from "../lib/get-auth.js";
 import { BrowserStackConfig } from "../lib/types.js";
 import { elicitCredentialsIfSupported } from "../lib/elicit-credentials.js";
 import logger from "../logger.js";
+import { NEEDS_A11Y_CONFIG_ID, NEEDS_A11Y_SCAN_ID } from "./tool-handoff.js";
 
 interface AuthCredentials {
   username: string;
@@ -575,7 +576,8 @@ export default function addAccessibilityTools(
 
   tools.getAccessibilityAuthConfig = server.tool(
     "getAccessibilityAuthConfig",
-    "Retrieve an existing authentication configuration by ID.",
+    "Retrieve an existing authentication configuration by ID." +
+      NEEDS_A11Y_CONFIG_ID,
     {
       configId: z.number().describe("ID of the auth configuration to retrieve"),
     },
@@ -593,7 +595,8 @@ export default function addAccessibilityTools(
 
   tools.fetchAccessibilityIssues = server.tool(
     "fetchAccessibilityIssues",
-    "Fetch accessibility issues from a completed scan with pagination support. Use cursor parameter to get subsequent pages of results.",
+    "Fetch accessibility issues from a completed scan with pagination support. Use cursor parameter to get subsequent pages of results." +
+      NEEDS_A11Y_SCAN_ID,
     {
       scanId: z
         .string()