@skyramp/mcp 0.3.8 → 0.4.0-rc.1

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

@@ -66,7 +66,7 @@ export function registerAnalyzeTestHealthTool(server) {
             blueprintCaptured: z
                 .boolean()
                 .optional()
-                .describe("If hasFrontendChanges is true in the skyramp_analyze_changes output: call browser_navigate to the primary route affected by the diff, then call browser_blueprint to capture the current DOM, then call this tool with blueprintCaptured: true. If hasFrontendChanges is false: 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."),
+                .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()
@@ -144,33 +144,22 @@ export function registerAnalyzeTestHealthTool(server) {
             catch (error) {
                 logger.warning(`Failed to cleanup old diff files: ${error.message}`);
             }
-            const uiContext = stateData.uiContext;
-            const hasUiFrontend = !!(uiContext?.changedFrontendFiles?.length);
+            // A UI test goes to UI drift because it is a UI test. This used to ask
+            // first whether the server had classified any changed file as frontend,
+            // and on a "no" every UI test fell through to API drift.
             const uiTests = [];
             const apiTests = [];
             for (const t of existingTests) {
-                // Only route to UI drift when frontend files changed — otherwise UI tests
-                // (playwright-ui, RTL, etc.) fall through to API drift so they are not
-                // silently dropped on API-only PRs.
-                (hasUiFrontend && isUiTest(t) ? uiTests : apiTests).push(t);
-            }
-            let uiDriftParams;
-            if (hasUiFrontend) {
-                logger.info(`Including UI drift analysis for ${uiTests.length} UI test(s), ${apiTests.length} API test(s)`);
-                // Shared components = changed frontend files imported by 2+ production files
-                // (server-computed importer graph, SKYR-3855). Surfacing the concrete importer
-                // list makes the "shared component → cannot IGNORE unread" drift rule enforceable
-                // against real data instead of the agent's qualitative guess.
-                const sharedComponents = (uiContext.frontendFileIntegration ?? [])
-                    .filter((f) => f.importers.length >= 2)
-                    .map((f) => ({ file: f.file, importers: f.importers }));
-                uiDriftParams = {
-                    tests: uiTests.map(t => ({ testFile: t.testFile, source: t.source })),
-                    changedFrontendFiles: uiContext.changedFrontendFiles,
-                    blueprintCaptured: args.blueprintCaptured,
-                    sharedComponents,
-                };
+                (isUiTest(t) ? uiTests : apiTests).push(t);
             }
+            // Always included, empty test list and all: the UI drift prompt tells the
+            // agent to grep for component tests itself, and gating it on a 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 })),
+                blueprintCaptured: args.blueprintCaptured,
+            };
             // Collect all checked-out repo paths so the grep instruction covers
             // every repo in a multi-repo run (endpoints in Repo A, tests in Repo B).
             // TODO(multi-repo): existingTests is scoped to the current repo only — related

package/build/tools/test-management/index.d.ts

@@ -2,3 +2,4 @@ export { registerAnalyzeChangesTool } from "./analyzeChangesTool.js";
 export { registerAnalyzeTestHealthTool } from "./analyzeTestHealthTool.js";
 export { registerActionsTool } from "./actionsTool.js";
 export { registerRegisterTestPlanTool } from "./registerTestPlanTool.js";
+export { registerResolveScreenTool } from "./resolveScreenTool.js";

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

@@ -2,3 +2,4 @@ export { registerAnalyzeChangesTool } from "./analyzeChangesTool.js";
 export { registerAnalyzeTestHealthTool } from "./analyzeTestHealthTool.js";
 export { registerActionsTool } from "./actionsTool.js";
 export { registerRegisterTestPlanTool } from "./registerTestPlanTool.js";
+export { registerResolveScreenTool } from "./resolveScreenTool.js";