@skyramp/mcp 0.4.1 → 0.4.2-rc.1

package/build/tools/test-management/analyzeChangesTool.js

@@ -4,18 +4,19 @@ import * as fs from "fs";
 import * as path from "path";
 import yaml from "js-yaml";
 import { logger } from "../../utils/logger.js";
-import { parseWorkspaceAuthType, getDefaultAuthHeader, WorkspaceAuthType, readWorkspaceConfigRaw, resolveWorkspaceApiService, findWorkspaceAuthDisagreement } from "../../utils/workspaceAuth.js";
+import { parseWorkspaceAuthType, getDefaultAuthHeader, WorkspaceAuthType, readWorkspaceConfigRaw, resolveWorkspaceApiService, findWorkspaceAuthDisagreement, } from "../../utils/workspaceAuth.js";
 import { AnalyticsService } from "../../services/AnalyticsService.js";
 import { makeProgressReporter } from "../../utils/progress.js";
-import { StateManager, registerSession, storeSessionData, rememberTestsRepoDir, getActiveRunStatePath, setActiveRunStatePath, } from "../../utils/AnalysisStateManager.js";
+import { StateManager, registerSession, storeSessionData, rememberTestsRepoDir, rememberPrimaryRepository, getPrimaryRepository, getActiveRunStatePath, setActiveRunStatePath, } from "../../utils/AnalysisStateManager.js";
+import { resolveRealPath } from "../../utils/repositorySlug.js";
 import { isTestFile } from "../../utils/testFileClassification.js";
 import { isPlanOnlyMode } from "../../utils/planOnlyMode.js";
 import { TestDiscoveryService } from "../../services/TestDiscoveryService.js";
-import { AnalysisScope } from "../../types/RepositoryAnalysis.js";
+import { AnalysisScope, } from "../../types/RepositoryAnalysis.js";
 import { computeBranchDiff, changedFileNames, } from "../../utils/branchDiff.js";
 import { detectProjectMetadata } from "../../utils/projectMetadata.js";
 import { diffBlocksByPath } from "../../utils/branchDiff.js";
-import { collectUiIdentifierChanges } from "../../utils/removedUiElements.js";
+import { collectUiIdentifierChanges, } from "../../utils/removedUiElements.js";
 import { parseTraceFile, discoverTraceFiles, discoverPlaywrightZips, } from "../../utils/trace-parser.js";
 import { TestSource } from "../../types/TestAnalysis.js";
 /** Exported for testing: maps a parsed trace result to a TraceFile. */
@@ -50,8 +51,8 @@ export async function isGraphQLFile(filePath, repositoryPath) {
     try {
         const absPath = path.join(repositoryPath, filePath);
         const fileContent = await fs.promises.readFile(absPath, "utf-8");
-        return GRAPHQL_SCHEMA_CONTENT_PATTERN.test(fileContent) ||
-            GRAPHQL_IMPLEMENTATION_CONTENT_PATTERN.test(fileContent);
+        return (GRAPHQL_SCHEMA_CONTENT_PATTERN.test(fileContent) ||
+            GRAPHQL_IMPLEMENTATION_CONTENT_PATTERN.test(fileContent));
     }
     catch {
         return false;
@@ -59,9 +60,12 @@ export async function isGraphQLFile(filePath, repositoryPath) {
 }
 // ── Seed and fixture data ───────────────────────────────────────────────────
 export const analyzeChangesInputSchema = {
-    repositoryPath: z
+    repositoryPath: z.string().describe("Absolute path to the repository root"),
+    repository: z
         .string()
-        .describe("Absolute path to the repository root"),
+        .trim()
+        .min(1)
+        .describe("The 'owner/repo' of the repository at repositoryPath (e.g. 'letsramp/api-insight'). For the primary repo it is the value the task instructions name; for a related repo it is that repo's own value from the <related_repositories> block. It keys this analysis' section of the run-scoped state file."),
     scope: z
         .enum(["full_repo", "branch_diff"])
         .default("branch_diff")
@@ -82,17 +86,11 @@ export const analyzeChangesInputSchema = {
         .number()
         .optional()
         .describe("GitHub PR number. Accepted and applied nowhere: nothing reads previous Testbot comments on the pull request any more. Pass it if the caller supplies it."),
-    repository: z
-        .string()
-        .optional()
-        .describe("The owner/repo this call analyzes (e.g. 'letsramp/api-insight'). In a multi-repo run, ALWAYS set it — for the primary AND every related repo. All calls in one Testbot run share ONE state file automatically (you do NOT pass a file path); each repo's analysis is stored as its own section within it. The primary's analysis is kept at the root and tagged with its owner/repo; related repos are stored as keyed sections. Omit only for a single-repo run."),
-    primaryRepository: z
-        .string()
-        .optional()
-        .describe("The owner/repo of the run's PRIMARY repository. In a multi-repo run set it on EVERY skyramp_analyze_changes call, for the primary and each related repo, so the state file's root is the primary whichever call comes first. Omit in a single-repo run."),
     testsRepoDir: z
         .string()
-        .refine((v) => path.isAbsolute(v), { message: "testsRepoDir must be an absolute path" })
+        .refine((v) => path.isAbsolute(v), {
+        message: "testsRepoDir must be an absolute path",
+    })
         .optional()
         .describe("Absolute path to a separate test repository clone. When set, existing test discovery scans this directory instead of repositoryPath. Used in cross-repo test delivery mode where tests live in a separate repo."),
     includeUncommitted: z
@@ -117,7 +115,11 @@ const _authFindingShapeMatchesType = {};
 void _authFindingShapeMatchesType;
 /** Both halves of `uiContext` carry the same record, so the shape is declared once.
  *  It mirrors `RemovedUiElement`; the assignment below fails to compile if either drifts. */
-const uiIdentifierShape = z.object({ attribute: z.string(), value: z.string(), file: z.string() });
+const uiIdentifierShape = z.object({
+    attribute: z.string(),
+    value: z.string(),
+    file: z.string(),
+});
 const _uiIdentifierShapeMatchesType = {};
 void _uiIdentifierShapeMatchesType;
 /** SKYR-4182: structuredContent needs a declared outputSchema, and the schema
@@ -128,7 +130,7 @@ export const analyzeChangesOutputSchema = {
     stateFile: z.string().optional(),
     repositoryPath: z.string(),
     repositoryName: z.string(),
-    repository: z.string().optional(),
+    repository: z.string(),
     relatedRepositories: z.array(z.string()).optional(),
     analysisScope: z.string(),
     planOnly: z.boolean(),
@@ -251,7 +253,9 @@ export function registerAnalyzeChangesTool(server) {
             // changedSelectors derivation and the removed-element collection below,
             // so a diff that reaches both is not sliced twice.
             let _diffByFile = null;
-            const getDiffByFile = () => (_diffByFile ??= diffData?.diffContent ? diffBlocksByPath(diffData.diffContent) : new Map());
+            const getDiffByFile = () => (_diffByFile ??= diffData?.diffContent
+                ? diffBlocksByPath(diffData.diffContent)
+                : new Map());
             // No resource tokens are derived from the changed files. Deriving them
             // guessed which existing tests were relevant from filenames and symbol
             // names; `TestDiscoveryService` now takes the external tests uncut, up to
@@ -280,7 +284,8 @@ export function registerAnalyzeChangesTool(server) {
                 }));
                 // Use the authoritative list from discovery — includes relevant files even
                 // when regex endpoint extraction failed (e.g. helper-abstracted HTTP calls).
-                discoveredRelevantExternalPaths = discoveryResult.relevantExternalTestPaths;
+                discoveredRelevantExternalPaths =
+                    discoveryResult.relevantExternalTestPaths;
                 logger.info("Test discovery completed", {
                     totalTests: existingTests.length,
                     relevantExternal: discoveredRelevantExternalPaths.length,
@@ -348,11 +353,14 @@ export function registerAnalyzeChangesTool(server) {
             let specFetchSucceeded = false;
             if (wsSchemaPath) {
                 try {
-                    const isUrl = wsSchemaPath.startsWith("http://") || wsSchemaPath.startsWith("https://");
+                    const isUrl = wsSchemaPath.startsWith("http://") ||
+                        wsSchemaPath.startsWith("https://");
                     const SPEC_FETCH_TIMEOUT_MS = 10_000;
                     let specText;
                     if (isUrl) {
-                        const specRes = await fetch(wsSchemaPath, { signal: AbortSignal.timeout(SPEC_FETCH_TIMEOUT_MS) });
+                        const specRes = await fetch(wsSchemaPath, {
+                            signal: AbortSignal.timeout(SPEC_FETCH_TIMEOUT_MS),
+                        });
                         if (!specRes.ok) {
                             throw new Error(`HTTP ${specRes.status} ${specRes.statusText} fetching spec at ${wsSchemaPath}`);
                         }
@@ -365,10 +373,16 @@ export function registerAnalyzeChangesTool(server) {
                     spec = yaml.load(specText);
                     specPaths = new Set(Object.keys((spec && typeof spec === "object" ? spec.paths : null) ?? {}));
                     // Only treat spec as authoritative when it actually has usable path entries
-                    specFetchSucceeded = spec && typeof spec === "object" &&
-                        spec.paths !== null && typeof spec.paths === "object" &&
-                        specPaths.size > 0;
-                    logger.info("Loaded OpenAPI spec paths", { count: specPaths.size, source: wsSchemaPath });
+                    specFetchSucceeded =
+                        spec &&
+                            typeof spec === "object" &&
+                            spec.paths !== null &&
+                            typeof spec.paths === "object" &&
+                            specPaths.size > 0;
+                    logger.info("Loaded OpenAPI spec paths", {
+                        count: specPaths.size,
+                        source: wsSchemaPath,
+                    });
                 }
                 catch (err) {
                     logger.warning("Could not load OpenAPI spec — continuing without path validation", {
@@ -462,7 +476,7 @@ export function registerAnalyzeChangesTool(server) {
             // regex extraction failed but path/name scoring identified as PR-relevant.
             // Relativize against repositoryPath to avoid leaking absolute machine paths
             // into the prompt, state file, and PR reports.
-            const relevantExternalTestPaths = discoveredRelevantExternalPaths.map(p => path.relative(params.repositoryPath, p));
+            const relevantExternalTestPaths = discoveredRelevantExternalPaths.map((p) => path.relative(params.repositoryPath, p));
             // Build the full RepositoryAnalysis object — same structure as
             // analyzeRepositoryTool. It is stored in the in-process session store for
             // the MCP resources and registerRecommendTestsPrompt; the tool result
@@ -472,11 +486,13 @@ export function registerAnalyzeChangesTool(server) {
             // the diff, and the agent reads the diff directly instead.
             // No `summary`: the server used to write a sentence about the diff here
             // and it has written an empty string since. Nothing reads the field.
-            const diffContext = diffData ? {
-                currentBranch: diffData.currentBranch,
-                baseBranch: diffData.baseBranch,
-                changedFiles: diffData.changedFiles,
-            } : undefined;
+            const diffContext = diffData
+                ? {
+                    currentBranch: diffData.currentBranch,
+                    baseBranch: diffData.baseBranch,
+                    changedFiles: diffData.changedFiles,
+                }
+                : undefined;
             // No `routeDiscovery`. Its three parts — a candidate route-file list, a
             // router entry-point grep and the OpenAPI path list — were the server's
             // guess at the repository's surface. Nothing downstream read them.
@@ -506,9 +522,21 @@ export function registerAnalyzeChangesTool(server) {
                     integrationPatterns: [],
                 },
                 artifacts: {
-                    openApiSpecs: wsSchemaPath ? [{ path: wsSchemaPath, version: "from-workspace-config", endpointCount: 0, baseUrl: wsBaseUrl, authType: wsAuthMethod }] : [],
-                    playwrightRecordings: discoverPlaywrightZips(params.repositoryPath).map(p => ({ path: p, description: "" })),
-                    traceFiles: traceResult ? [buildTraceFileEntry(traceFiles[0], traceResult)] : [],
+                    openApiSpecs: wsSchemaPath
+                        ? [
+                            {
+                                path: wsSchemaPath,
+                                version: "from-workspace-config",
+                                endpointCount: 0,
+                                baseUrl: wsBaseUrl,
+                                authType: wsAuthMethod,
+                            },
+                        ]
+                        : [],
+                    playwrightRecordings: discoverPlaywrightZips(params.repositoryPath).map((p) => ({ path: p, description: "" })),
+                    traceFiles: traceResult
+                        ? [buildTraceFileEntry(traceFiles[0], traceResult)]
+                        : [],
                     notFound: [],
                 },
                 workspace: { baseUrl: wsBaseUrl },
@@ -525,8 +553,18 @@ export function registerAnalyzeChangesTool(server) {
                     hasCiCd: projectMeta.hasCiCd,
                 },
                 existingTests: {
-                    frameworks: [...new Set(existingTests.map((t) => t.framework).filter(Boolean))],
-                    coverage: { unit: 0, integration: 0, e2e: 0, ui: 0, load: 0, contract: 0, smoke: 0 },
+                    frameworks: [
+                        ...new Set(existingTests.map((t) => t.framework).filter(Boolean)),
+                    ],
+                    coverage: {
+                        unit: 0,
+                        integration: 0,
+                        e2e: 0,
+                        ui: 0,
+                        load: 0,
+                        contract: 0,
+                        smoke: 0,
+                    },
                     testLocations: testLocationsByType,
                     hasCoverageReports: false,
                     relevantExternalTestPaths,
@@ -554,18 +592,19 @@ export function registerAnalyzeChangesTool(server) {
                 let retiredElements = [];
                 if (diffData?.diffContent) {
                     try {
-                        ({ removed: removedElements, retired: retiredElements } = collectUiIdentifierChanges({
-                            // Every changed file that is not a test. A removed `data-testid`
-                            // is found by reading the diff, not by the file's extension.
-                            changedFrontendFiles: (diffData.changedFiles ?? []).filter((f) => !isTestFile(f)),
-                            deletedFiles: diffData.deletedFiles ?? [],
-                            diffByFile: getDiffByFile(),
-                            existingTestFiles: [
-                                ...existingTests.map((t) => t.testFile),
-                                ...discoveredRelevantExternalPaths,
-                            ],
-                            discoveryFailed: testDiscoveryFailed,
-                        }));
+                        ({ removed: removedElements, retired: retiredElements } =
+                            collectUiIdentifierChanges({
+                                // Every changed file that is not a test. A removed `data-testid`
+                                // is found by reading the diff, not by the file's extension.
+                                changedFrontendFiles: (diffData.changedFiles ?? []).filter((f) => !isTestFile(f)),
+                                deletedFiles: diffData.deletedFiles ?? [],
+                                diffByFile: getDiffByFile(),
+                                existingTestFiles: [
+                                    ...existingTests.map((t) => t.testFile),
+                                    ...discoveredRelevantExternalPaths,
+                                ],
+                                discoveryFailed: testDiscoveryFailed,
+                            }));
                     }
                     catch (err) {
                         logger.warning("removedUiElements: collection failed, continuing without removal-guard grounding", {
@@ -606,7 +645,10 @@ export function registerAnalyzeChangesTool(server) {
                     // skyramp_register_test_plan still recomputes the listing at verify time
                     // from `baseBranch`, so the two cannot drift apart.
                     diff: diffData
-                        ? { baseBranch: diffData.baseBranch, changedFiles: changedNames.map((file) => file.path) }
+                        ? {
+                            baseBranch: diffData.baseBranch,
+                            changedFiles: changedNames.map((file) => file.path),
+                        }
                         : undefined,
                     sessionId,
                 },
@@ -618,14 +660,26 @@ export function registerAnalyzeChangesTool(server) {
             catch (error) {
                 logger.warning(`Failed to cleanup old state files: ${error.message}`);
             }
-            // Resolve the state file. Multi-repo (`repository` set): all calls in one
-            // Testbot run share ONE state file — the first call creates it and records
-            // the run anchor; later related-repo calls reuse it (no agent-provided path,
-            // which previously caused filename hallucinations). Single-repo (`repository`
-            // absent): always a fresh per-call file — byte-identical to prior behavior.
-            const activeRunPath = params.repository
-                ? getActiveRunStatePath()
-                : undefined;
+            // Resolve the state file. All calls in one Testbot run share ONE state
+            // file — the first call creates it and records the run anchor; later
+            // calls reuse it (no agent-provided path, which previously caused
+            // filename hallucinations). Single-repo runs take the same path now
+            // that `repository` is required: one run, one state file.
+            // Real paths, because a symlinked or relative spelling of the primary
+            // checkout must not read as a different repository.
+            const ownPath = await resolveRealPath(params.repositoryPath);
+            const ownRepo = params.repository;
+            // The run declares its primary when the prompt renders. Entry without a
+            // prompt render — IDE use, a bare analyze call — has no declaration, so
+            // this first call declares its own checkout, which is a statement at the
+            // run's entry point rather than a write-order accident.
+            rememberPrimaryRepository(ownPath, ownRepo);
+            // Set-if-unset above, so this is the prompt's declaration when there was
+            // one and this call's own checkout when there was not.
+            const primary = getPrimaryRepository();
+            const primaryPath = await resolveRealPath(primary.repositoryPath);
+            const primaryRepo = primary.repo;
+            const activeRunPath = getActiveRunStatePath();
             const stateManager = activeRunPath
                 ? StateManager.fromStatePath(activeRunPath)
                 : new StateManager("analysis", sessionId);
@@ -634,15 +688,14 @@ export function registerAnalyzeChangesTool(server) {
             // stored at the root; a related repo is stored as a keyed section within the
             // same run-scoped stateFile.
             await stateManager.writeRepoData(unifiedState, {
-                repo: params.repository,
-                primaryRepo: params.primaryRepository,
+                repo: ownRepo,
                 repositoryPath: params.repositoryPath,
+                primary: { repositoryPath: primaryPath, repo: primaryRepo },
                 step: "analyze_changes",
             });
             const stateFile = stateManager.getStatePath();
-            // Anchor the run's state file so subsequent related-repo calls reuse it.
-            if (params.repository)
-                setActiveRunStatePath(stateFile);
+            // Anchor the run's state file so subsequent calls reuse it.
+            setActiveRunStatePath(stateFile);
             registerSession(sessionId, stateFile);
             // A related repo reads its OWN .skyramp/workspace.yml, so it can disagree
             // with the primary's unified entry for the same service and nothing would
@@ -694,7 +747,9 @@ export function registerAnalyzeChangesTool(server) {
             const skyrampTestFiles = [];
             const externalTestFiles = [];
             for (const test of existingTests) {
-                const target = test.source === TestSource.External ? externalTestFiles : skyrampTestFiles;
+                const target = test.source === TestSource.External
+                    ? externalTestFiles
+                    : skyrampTestFiles;
                 target.push(displayPath(test.testFile));
             }
             return buildAnalyzeChangesResult({
@@ -702,7 +757,7 @@ export function registerAnalyzeChangesTool(server) {
                 stateFile,
                 repositoryPath: params.repositoryPath,
                 repositoryName: path.basename(params.repositoryPath),
-                ...(params.repository ? { repository: params.repository } : {}),
+                repository: ownRepo,
                 ...(relatedRepositories.length ? { relatedRepositories } : {}),
                 analysisScope,
                 planOnly: isPlanOnlyMode(),
@@ -728,7 +783,9 @@ export function registerAnalyzeChangesTool(server) {
                 },
                 // Removed half only. No prompt describes `retiredElements`, and beside the guard
                 // list it reads as more guards to record; the plan checks read the state file.
-                ...(uiContext?.removedElements.length ? { uiContext: { removedElements: uiContext.removedElements } } : {}),
+                ...(uiContext?.removedElements.length
+                    ? { uiContext: { removedElements: uiContext.removedElements } }
+                    : {}),
                 nextStep: "skyramp_analyze_test_health, with this stateFile",
             });
         }
@@ -749,7 +806,9 @@ export function registerAnalyzeChangesTool(server) {
         finally {
             AnalyticsService.pushMCPToolEvent(TOOL_NAME, errorResult, {
                 repositoryPath: params.repositoryPath,
-            }).catch((err) => { logger.warning("Analytics event failed", { error: String(err) }); });
+            }).catch((err) => {
+                logger.warning("Analytics event failed", { error: String(err) });
+            });
         }
     });
 }

package/build/tools/test-management/analyzeTestHealthTool.js

@@ -13,13 +13,20 @@ const TOOL_NAME = "skyramp_analyze_test_health";
 // variants detected when the file uses the page fixture (not just the request fixture).
 // Plain "playwright"/"cypress" without the -ui suffix = REST API usage — excluded.
 const UI_COMPONENT_FRAMEWORKS = new Set([
-    "rtl", "react-testing-library",
-    "vue-test-utils", "angular-testbed", "svelte-testing-library",
+    "rtl",
+    "react-testing-library",
+    "vue-test-utils",
+    "angular-testbed",
+    "svelte-testing-library",
     "enzyme",
-    "flutter-widget", "flutter-test",
-    "xcuitest", "espresso", "appium",
+    "flutter-widget",
+    "flutter-test",
+    "xcuitest",
+    "espresso",
+    "appium",
     "selenium",
-    "playwright-ui", "cypress-ui",
+    "playwright-ui",
+    "cypress-ui",
 ]);
 // No REST API test is ever written in TSX/JSX — extension alone is sufficient.
 const UI_TEST_EXTENSIONS = /\.(test|spec)\.(tsx|jsx)$/;
@@ -27,7 +34,9 @@ const UI_SNAPSHOT_EXT = /\.snap$/;
 function isUiTest(t) {
     const fw = t.framework?.toLowerCase() ?? "";
     const isUiSnap = UI_SNAPSHOT_EXT.test(t.testFile);
-    return UI_COMPONENT_FRAMEWORKS.has(fw) || UI_TEST_EXTENSIONS.test(t.testFile) || isUiSnap;
+    return (UI_COMPONENT_FRAMEWORKS.has(fw) ||
+        UI_TEST_EXTENSIONS.test(t.testFile) ||
+        isUiSnap);
 }
 /**
  * Names of services whose workspace.yml records a runnable external-test suite
@@ -69,11 +78,14 @@ export function registerAnalyzeTestHealthTool(server) {
                 .describe("Set it to true only when you have already captured a blueprint: browser_navigate to the route the diff affects, then browser_blueprint to capture the current DOM, then call this tool with blueprintCaptured: true. If you captured nothing, call this tool directly and omit this parameter. Passing true activates blueprint-diff analysis for browser UI tests (Playwright, Cypress, Selenium). Component/widget tests always use source-only analysis regardless of this flag."),
             repository: z
                 .string()
-                .optional()
-                .describe("The owner/repo whose analysis section to assess (e.g. 'letsramp/api-insight'). In a multi-repo run, ALWAYS set it — the primary's owner/repo for the primary repo, or a related repo's owner/repo for that repo's section within the run-scoped stateFile. Omit only for a single-repo run."),
+                .trim()
+                .min(1)
+                .describe("The owner/repo whose analysis section to assess (e.g. 'letsramp/api-insight'). Set it on every call — the primary's owner/repo for the primary repo, or a related repo's owner/repo for that repo's section within the run-scoped stateFile."),
         },
         outputSchema: {
-            prompt: z.string().describe("LLM drift analysis prompt and assessment instructions."),
+            prompt: z
+                .string()
+                .describe("LLM drift analysis prompt and assessment instructions."),
         },
     }, async (args) => {
         let errorResult;
@@ -157,7 +169,10 @@ export function registerAnalyzeTestHealthTool(server) {
             // verdict about the diff is what this removes.
             logger.info(`Including UI drift analysis for ${uiTests.length} UI test(s), ${apiTests.length} API test(s)`);
             const uiDriftParams = {
-                tests: uiTests.map(t => ({ testFile: t.testFile, source: t.source })),
+                tests: uiTests.map((t) => ({
+                    testFile: t.testFile,
+                    source: t.source,
+                })),
                 blueprintCaptured: args.blueprintCaptured,
             };
             // Collect all checked-out repo paths so the grep instruction covers
@@ -169,12 +184,17 @@ export function registerAnalyzeTestHealthTool(server) {
             // repo (SKYR-4246). Dedupe by root because a legacy untagged primary can also
             // appear as a related section holding the same path.
             const allRepoPaths = [
-                ...new Set((await stateManager.listRepoCheckouts()).map(c => c.root)),
+                ...new Set((await stateManager.listRepoCheckouts()).map((c) => c.root)),
             ];
             const promptText = buildDriftAnalysisPrompt(stateManager.getStatePath(), apiTests.map((t) => ({ testFile: t.testFile, source: t.source })), uiDriftParams, allRepoPaths, stateData?.externalTestResults);
             return {
                 structuredContent: { prompt: promptText },
-                content: [{ type: "text", text: "Drift analysis prompt generated. Follow the prompt field to assess each test." }],
+                content: [
+                    {
+                        type: "text",
+                        text: "Drift analysis prompt generated. Follow the prompt field to assess each test.",
+                    },
+                ],
             };
         }
         catch (error) {
@@ -183,7 +203,9 @@ export function registerAnalyzeTestHealthTool(server) {
             return errorResult;
         }
         finally {
-            AnalyticsService.pushMCPToolEvent(TOOL_NAME, errorResult, {}).catch((err) => { logger.warning("Analytics event failed", { error: String(err) }); });
+            AnalyticsService.pushMCPToolEvent(TOOL_NAME, errorResult, {}).catch((err) => {
+                logger.warning("Analytics event failed", { error: String(err) });
+            });
         }
     });
 }