From ebbd8cbff0dbb14507a5fe33f0077ef55cf3898f Mon Sep 17 00:00:00 2001
From: Savio Dias <savio@browserstack.com>
Date: Thu, 17 Sep 2026 12:48:35 +0530
Subject: [PATCH 1/2] fix: existing failure telemetry

---
 src/tools/bstack-sdk.ts     | 10 ++++++++--
 src/tools/build-insights.ts |  1 +
 src/tools/percy-sdk.ts      | 18 ++++++++++++++----
 3 files changed, 23 insertions(+), 6 deletions(-)

diff --git a/src/tools/bstack-sdk.ts b/src/tools/bstack-sdk.ts
index 9e2a3a5c..c7337529 100644
--- a/src/tools/bstack-sdk.ts
+++ b/src/tools/bstack-sdk.ts
@@ -26,13 +26,19 @@ export function registerRunBrowserStackTestsTool(
     async (args) => {
       try {
         trackMCP(
-          "runTestsOnBrowserStack",
+          "setupBrowserStackAutomateTests",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return await runTestsOnBrowserStackHandler(args, config);
       } catch (error) {
-        return handleMCPError("runTestsOnBrowserStack", server, config, error);
+        return handleMCPError(
+          "setupBrowserStackAutomateTests",
+          server,
+          config,
+          error,
+        );
       }
     },
   );
diff --git a/src/tools/build-insights.ts b/src/tools/build-insights.ts
index 2f90a4f5..2ea00617 100644
--- a/src/tools/build-insights.ts
+++ b/src/tools/build-insights.ts
@@ -147,6 +147,7 @@ export default function addBuildInsightsTools(
         trackMCP(
           "fetchBuildInsights",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return await fetchBuildInsightsTool(args, config);
diff --git a/src/tools/percy-sdk.ts b/src/tools/percy-sdk.ts
index 012ccd20..3c385a2e 100644
--- a/src/tools/percy-sdk.ts
+++ b/src/tools/percy-sdk.ts
@@ -69,14 +69,15 @@ export function registerPercyTools(
     async (args) => {
       try {
         trackMCP(
-          "VisualTestIntegrationAgent",
+          "percyVisualTestIntegrationAgent",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return simulatePercyChangeHandler(args, config);
       } catch (error) {
         return handleMCPError(
-          "VisualTestIntegrationAgent",
+          "percyVisualTestIntegrationAgent",
           server,
           config,
           error,
@@ -99,13 +100,19 @@ export function registerPercyTools(
     async (args) => {
       try {
         trackMCP(
-          "setupPercyVisualTesting",
+          "expandPercyVisualTesting",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return setUpPercyHandler(args, config);
       } catch (error) {
-        return handleMCPError("setupPercyVisualTesting", server, config, error);
+        return handleMCPError(
+          "expandPercyVisualTesting",
+          server,
+          config,
+          error,
+        );
       }
     },
   );
@@ -126,6 +133,7 @@ export function registerPercyTools(
         trackMCP(
           "addPercySnapshotCommands",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return await updateTestsWithPercyCommands(args);
@@ -198,6 +206,7 @@ export function registerPercyTools(
         trackMCP(
           "fetchPercyChanges",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return await fetchPercyChanges(args, config);
@@ -223,6 +232,7 @@ export function registerPercyTools(
         trackMCP(
           "managePercyBuildApproval",
           server.server.getClientVersion()!,
+          undefined,
           config,
         );
         return await approveOrDeclinePercyBuild(args, config);

From a4cf8e18a9fc24b367f526986e453c11c126386b Mon Sep 17 00:00:00 2001
From: Savio Dias <savio@browserstack.com>
Date: Thu, 17 Sep 2026 14:32:01 +0530
Subject: [PATCH 2/2] fix(percy): pass undefined error arg in listTestFiles and
 runPercyScan trackMCP calls

Same misuse as the other seven call sites, missed because these two were
single-line calls. Sweep of src/ confirms no trackMCP call now passes
config in the error slot.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
---
 src/tools/percy-sdk.ts | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/tools/percy-sdk.ts b/src/tools/percy-sdk.ts
index 3c385a2e..4c70431c 100644
--- a/src/tools/percy-sdk.ts
+++ b/src/tools/percy-sdk.ts
@@ -161,7 +161,12 @@ export function registerPercyTools(
     },
     async () => {
       try {
-        trackMCP("listTestFiles", server.server.getClientVersion()!, config);
+        trackMCP(
+          "listTestFiles",
+          server.server.getClientVersion()!,
+          undefined,
+          config,
+        );
         return addListTestFiles();
       } catch (error) {
         return handleMCPError("listTestFiles", server, config, error);
@@ -182,7 +187,12 @@ export function registerPercyTools(
     },
     async (args) => {
       try {
-        trackMCP("runPercyScan", server.server.getClientVersion()!, config);
+        trackMCP(
+          "runPercyScan",
+          server.server.getClientVersion()!,
+          undefined,
+          config,
+        );
         return runPercyScan(args);
       } catch (error) {
         return handleMCPError("runPercyScan", server, config, error);