Skip to content

Commit 8711b52

Browse files
refactor: replace trackMCPSuccess function calls with direct trackMCP invocations
1 parent eced35b commit 8711b52

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

src/tools/accessibility.ts

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -119,21 +119,14 @@ async function initializeReportFetcher(
119119
return reportFetcher;
120120
}
121121

122-
function trackMCPSuccess(
123-
toolName: string,
124-
server: McpServer,
125-
config: BrowserStackConfig,
126-
): void {
127-
trackMCP(toolName, server.server.getClientVersion()!, undefined, config);
128-
}
129122

130123
async function executeAccessibilityRAG(
131124
args: { query: string },
132125
server: McpServer,
133126
config: BrowserStackConfig,
134127
): Promise<CallToolResult> {
135128
try {
136-
trackMCPSuccess("accessibilityExpert", server, config);
129+
trackMCP("accessibilityExpert", server.server.getClientVersion()!, undefined, config);
137130
return await queryAccessibilityRAG(args.query, config);
138131
} catch (error) {
139132
return handleMCPError("accessibilityExpert", server, config, error);
@@ -147,7 +140,7 @@ async function executeAccessibilityScan(
147140
config: BrowserStackConfig,
148141
): Promise<CallToolResult> {
149142
try {
150-
trackMCPSuccess("startAccessibilityScan", server, config);
143+
trackMCP("startAccessibilityScan", server.server.getClientVersion()!, undefined, config);
151144
return await runAccessibilityScan(
152145
args.name,
153146
args.pageURL,
@@ -209,7 +202,7 @@ async function executeCreateAuthConfig(
209202
config: BrowserStackConfig,
210203
): Promise<CallToolResult> {
211204
try {
212-
trackMCPSuccess("createAccessibilityAuthConfig", server, config);
205+
trackMCP("createAccessibilityAuthConfig", server.server.getClientVersion()!, undefined, config);
213206
logger.info(`Creating auth config: ${JSON.stringify(args)}`);
214207

215208
const result = await createAuthConfig(args, config);
@@ -240,7 +233,7 @@ async function executeGetAuthConfig(
240233
config: BrowserStackConfig,
241234
): Promise<CallToolResult> {
242235
try {
243-
trackMCPSuccess("getAccessibilityAuthConfig", server, config);
236+
trackMCP("getAccessibilityAuthConfig", server.server.getClientVersion()!, undefined, config);
244237

245238
const authConfig = new AccessibilityAuthConfig();
246239
const auth = setupAuth(config);

0 commit comments

Comments
 (0)