@skyramp/mcp 0.2.8 → 0.2.150-rc.ldw

package/build/tools/one-click/oneClickTool.test.js

@@ -0,0 +1,172 @@
+import { jest, describe, it, expect, beforeEach, afterEach } from "@jest/globals";
+import { execFileSync } from "child_process";
+import * as fs from "fs";
+import * as os from "os";
+import * as path from "path";
+jest.unstable_mockModule("../../services/AnalyticsService.js", () => ({
+    AnalyticsService: { pushMCPToolEvent: jest.fn() },
+}));
+const { registerOneClickTool } = await import("./oneClickTool.js");
+function registerToolForTest() {
+    let registered;
+    const server = {
+        registerTool: jest.fn((name, config, handler) => {
+            registered = { name, config, handler };
+        }),
+    };
+    registerOneClickTool(server);
+    if (!registered)
+        throw new Error("tool was not registered");
+    return registered;
+}
+describe("skyramp_one_click_tool", () => {
+    let tmpDir;
+    let originalLocalDevEnv;
+    beforeEach(() => {
+        originalLocalDevEnv = process.env.SKYRAMP_FEATURE_LOCAL_DEV;
+        process.env.SKYRAMP_FEATURE_LOCAL_DEV = "1";
+        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-one-click-"));
+        execFileSync("git", ["init"], { cwd: tmpDir, stdio: "ignore" });
+    });
+    afterEach(() => {
+        if (originalLocalDevEnv === undefined) {
+            delete process.env.SKYRAMP_FEATURE_LOCAL_DEV;
+        }
+        else {
+            process.env.SKYRAMP_FEATURE_LOCAL_DEV = originalLocalDevEnv;
+        }
+        fs.rmSync(tmpDir, { recursive: true, force: true });
+    });
+    it("routes local-dev requests and installs local-dev skill files", async () => {
+        const tool = registerToolForTest();
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+            callingAgent: "cursor",
+            baseBranch: "checkr-prod-flow",
+        });
+        const text = result.content[0].text;
+        expect(result.isError).toBe(false);
+        expect(text).toContain('skyramp_local_dev baseCommit: checkr-prod-flow');
+        const cursorSkill = fs.readFileSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"), "utf-8");
+        expect(cursorSkill).toContain("mock only true third-party downstream services");
+        expect(fs.existsSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"))).toBe(true);
+        expect(fs.existsSync(path.join(tmpDir, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+    });
+    it("installs only the Claude local-dev skill file", async () => {
+        const tool = registerToolForTest();
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+            callingAgent: "claude",
+        });
+        expect(result.isError).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".claude/skills/local-dev/SKILL.md"))).toBe(true);
+        expect(fs.existsSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+    });
+    it("skips skill creation for agents without repo-local skill support", async () => {
+        const tool = registerToolForTest();
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+            callingAgent: "vscode",
+        });
+        expect(result.isError).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+    });
+    it("does not install a skill file without a recognized calling agent", async () => {
+        const tool = registerToolForTest();
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+            callingAgent: "unknown",
+        });
+        expect(result.isError).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+    });
+    it("surfaces local-dev mock routing parameters in workflow text", async () => {
+        const tool = registerToolForTest();
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+            callingAgent: "cursor",
+            baseBranch: "checkr-prod-flow",
+            servicesToMock: "stripe",
+            realServices: "partner-accounts,redpanda",
+            mockProtocols: "rest",
+        });
+        const text = result.content[0].text;
+        expect(result.isError).toBe(false);
+        expect(text).toContain("Services to mock: stripe");
+        expect(text).toContain("Real services: partner-accounts,redpanda");
+        expect(text).toContain("Mock protocols: rest");
+        expect(text).toContain("<SERVICE_ROUTING>");
+        expect(text).toContain("<MOCK_SERVICES>stripe</MOCK_SERVICES>");
+        expect(text).toContain("<REAL_SERVICES>partner-accounts,redpanda</REAL_SERVICES>");
+        expect(text).toContain("<MOCK_PROTOCOLS>rest</MOCK_PROTOCOLS>");
+    });
+    it("does not install local-dev skill files outside an existing git repository", async () => {
+        const tool = registerToolForTest();
+        const nonRepo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-one-click-nonrepo-"));
+        try {
+            const result = await tool.handler({
+                workflow: "local_dev_test_changes",
+                repositoryPath: nonRepo,
+                callingAgent: "cursor",
+            });
+            expect(result.isError).toBe(true);
+            expect(result.content[0].text).toContain("repositoryPath must point to a git repository");
+            expect(fs.existsSync(path.join(nonRepo, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+            expect(fs.existsSync(path.join(nonRepo, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+            expect(fs.existsSync(path.join(nonRepo, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+        }
+        finally {
+            fs.rmSync(nonRepo, { recursive: true, force: true });
+        }
+    });
+    it("validates repositoryPath even when skill creation is skipped", async () => {
+        const tool = registerToolForTest();
+        const nonRepo = fs.mkdtempSync(path.join(os.tmpdir(), "skyramp-one-click-nonrepo-"));
+        try {
+            const result = await tool.handler({
+                workflow: "local_dev_test_changes",
+                repositoryPath: nonRepo,
+                callingAgent: "vscode",
+            });
+            expect(result.isError).toBe(true);
+            expect(result.content[0].text).toContain("repositoryPath must point to a git repository");
+            expect(fs.existsSync(path.join(nonRepo, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+            expect(fs.existsSync(path.join(nonRepo, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+            expect(fs.existsSync(path.join(nonRepo, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+        }
+        finally {
+            fs.rmSync(nonRepo, { recursive: true, force: true });
+        }
+    });
+    it("hides local-dev workflow when the feature flag is disabled", async () => {
+        delete process.env.SKYRAMP_FEATURE_LOCAL_DEV;
+        const tool = registerToolForTest();
+        expect(String(tool.config.description)).not.toContain("local_dev_test_changes");
+        expect(String(tool.config.inputSchema)).not.toContain("local_dev_test_changes");
+        expect(Object.keys(tool.config.inputSchema)).not.toContain("baseBranch");
+        expect(Object.keys(tool.config.inputSchema)).not.toContain("servicesToMock");
+        expect(Object.keys(tool.config.inputSchema)).not.toContain("realServices");
+        expect(Object.keys(tool.config.inputSchema)).not.toContain("mockProtocols");
+        expect(Object.keys(tool.config.inputSchema)).not.toContain("callingAgent");
+        const result = await tool.handler({
+            workflow: "local_dev_test_changes",
+            repositoryPath: tmpDir,
+        });
+        expect(result.isError).toBe(true);
+        expect(result.content[0].text).toContain('Unknown workflow: "local_dev_test_changes"');
+        expect(fs.existsSync(path.join(tmpDir, ".cursor/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".claude/skills/local-dev/SKILL.md"))).toBe(false);
+        expect(fs.existsSync(path.join(tmpDir, ".vscode/skills/local-dev/SKILL.md"))).toBe(false);
+    });
+});

package/build/tools/preflightMockCheckTool.d.ts

@@ -0,0 +1,2 @@
+import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+export declare function registerPreflightMockCheckTool(server: McpServer): void;

package/build/tools/preflightMockCheckTool.js

@@ -0,0 +1,96 @@
+import { z } from "zod";
+import { validateMockCompatibility, MockProtocol, } from "../utils/mockCompatibility.js";
+const warningSchema = z.object({
+    code: z.string().describe("Warning code identifier (e.g. REST_LOOPBACK_URL, GRPC_PERSISTENT_CONN)."),
+    message: z.string().describe("Human-readable description of the issue."),
+    suggestion: z.string().describe("Recommended action to resolve or work around the issue."),
+    affectedMocks: z.array(z.string()).describe("Mock identifiers (service name or endpoint URL) affected by this warning."),
+    blocking: z.boolean().optional().describe("When true, the affected mocks cannot intercept traffic and must not be generated or deployed."),
+});
+const preflightOutputSchema = {
+    hasBlockers: z.boolean().describe("True when at least one warning is blocking. Blocking mocks must be removed from the generation list before proceeding."),
+    warnings: z.array(warningSchema).describe("All compatibility warnings. Inspect the blocking field on each entry; blocking ones must be skipped, advisory ones may proceed with caution."),
+};
+const TOOL_NAME = "skyramp_preflight_mock_check";
+const mockSpecSchema = z.object({
+    protocol: z.nativeEnum(MockProtocol).describe("The protocol of the mock"),
+    endpointURL: z.string().describe("The endpoint URL for the mock"),
+    method: z.string().optional().describe("HTTP method (for REST mocks)"),
+    grpcServiceName: z.string().optional().describe("gRPC service name (for gRPC mocks)"),
+    kafkaTopic: z.string().optional().describe("Kafka topic (for Kafka mocks)"),
+});
+const inputSchema = {
+    mocks: z
+        .array(mockSpecSchema)
+        .min(1)
+        .describe("Array of mock specifications to validate for compatibility"),
+    dockerLocalDev: z
+        .boolean()
+        .default(true)
+        .describe("When true, apply Docker local-dev checks such as REST loopback URL blockers. Set false for non-Docker/local runtimes."),
+};
+function formatWarning(warning) {
+    const icon = warning.blocking ? "🚫" : "⚠️ ";
+    return [
+        `${icon} [${warning.code}]${warning.blocking ? " BLOCKING — do not generate or deploy these mocks" : ""}`,
+        `   ${warning.message}`,
+        `   Suggestion: ${warning.suggestion}`,
+        `   Affected: ${warning.affectedMocks.join(", ")}`,
+    ].join("\n");
+}
+export function registerPreflightMockCheckTool(server) {
+    server.registerTool(TOOL_NAME, {
+        description: "Pre-flight compatibility check for mock deployments. Validates that a set of mocks " +
+            "can be deployed together without known protocol conflicts (e.g., REST loopback URLs in Docker local-dev " +
+            "and gRPC persistent connection caveats). Returns two classes of " +
+            "issues: advisory warnings (mocks are deployable but may exhibit runtime issues) and " +
+            "blocking warnings (mocks cannot intercept traffic and must not be generated or deployed). " +
+            "Check hasBlockers in the result and the blocking flag on each warning before proceeding.",
+        inputSchema,
+        outputSchema: preflightOutputSchema,
+    }, async (params) => {
+        const mocks = params.mocks;
+        const dockerLocalDev = params.dockerLocalDev !== false;
+        const result = validateMockCompatibility(mocks, { dockerLocalDev });
+        let text;
+        if (result.warnings.length === 0) {
+            text = [
+                "✅ Pre-flight check passed — all clear.",
+                "",
+                `${mocks.length} mock(s) validated. No known compatibility issues detected.`,
+                "Safe to deploy in a single apply_mock() call.",
+            ].join("\n");
+        }
+        else {
+            const blockingCount = result.warnings.filter((w) => w.blocking).length;
+            const advisoryCount = result.warnings.length - blockingCount;
+            const headerParts = [];
+            if (blockingCount > 0)
+                headerParts.push(`${blockingCount} blocking`);
+            if (advisoryCount > 0)
+                headerParts.push(`${advisoryCount} advisory`);
+            text = [
+                `⚠️  Pre-flight check: ${result.warnings.length} issue(s) detected (${headerParts.join(", ")}).`,
+                "",
+                ...result.warnings.map(formatWarning),
+                "",
+                result.hasBlockers
+                    ? "🚫 Blocking mocks must be removed from the generation list — they cannot intercept traffic. Advisory mocks may still be deployed with the caveats noted above."
+                    : "Advisory warnings only — mocks are deployable, but consider the suggestions above to avoid runtime issues.",
+            ].join("\n");
+        }
+        return {
+            structuredContent: {
+                hasBlockers: result.hasBlockers,
+                warnings: result.warnings.map((w) => ({
+                    code: w.code,
+                    message: w.message,
+                    suggestion: w.suggestion,
+                    affectedMocks: w.affectedMocks,
+                    ...(w.blocking !== undefined ? { blocking: w.blocking } : {}),
+                })),
+            },
+            content: [{ type: "text", text }],
+        };
+    });
+}

package/build/tools/submitReportTool.d.ts

@@ -16,7 +16,7 @@ export declare const targetElementSchema: z.ZodObject<{
     stableId: string | null;
     contextText: string[] | null;
     mutability?: "unknown" | "mutable" | "immutable" | undefined;
-    widgetType?: "custom" | "unknown" | "native" | undefined;
+    widgetType?: "unknown" | "custom" | "native" | undefined;
 }, {
     role: string;
     accessibleName: string;
@@ -24,7 +24,7 @@ export declare const targetElementSchema: z.ZodObject<{
     stableId: string | null;
     contextText: string[] | null;
     mutability?: "unknown" | "mutable" | "immutable" | undefined;
-    widgetType?: "custom" | "unknown" | "native" | undefined;
+    widgetType?: "unknown" | "custom" | "native" | undefined;
 }>;
 export declare const pageContextSchema: z.ZodObject<{
     url: z.ZodString;
@@ -63,7 +63,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }, {
         role: string;
         accessibleName: string;
@@ -71,7 +71,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }>, "many">>>;
     pageContext: z.ZodOptional<z.ZodObject<{
         url: z.ZodString;
@@ -102,7 +102,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -127,7 +127,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -152,7 +152,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -177,7 +177,7 @@ export declare const newTestSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -234,7 +234,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }, {
         role: string;
         accessibleName: string;
@@ -242,7 +242,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }>, "many">>>;
     pageContext: z.ZodOptional<z.ZodObject<{
         url: z.ZodString;
@@ -279,7 +279,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -313,7 +313,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -347,7 +347,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;
@@ -381,7 +381,7 @@ export declare const additionalRecommendationSchema: z.ZodEffects<z.ZodObject<{
         stableId: string | null;
         contextText: string[] | null;
         mutability?: "unknown" | "mutable" | "immutable" | undefined;
-        widgetType?: "custom" | "unknown" | "native" | undefined;
+        widgetType?: "unknown" | "custom" | "native" | undefined;
     }[] | null | undefined;
     pageContext?: {
         url: string;

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

@@ -15,11 +15,12 @@ export declare const analyzeChangesInputSchema: {
     scope: z.ZodOptional<z.ZodDefault<z.ZodEnum<["full_repo", "branch_diff"]>>>;
     baseBranch: z.ZodOptional<z.ZodString>;
     testDirectory: z.ZodOptional<z.ZodString>;
-    topN: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+    topN: z.ZodDefault<z.ZodOptional<z.ZodNumber>>;
     maxGenerate: z.ZodOptional<z.ZodNumber>;
     prNumber: z.ZodOptional<z.ZodNumber>;
     repository: z.ZodOptional<z.ZodString>;
     testsRepoDir: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
+    includeUncommitted: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
 };
 export declare const NO_UI_INSTRUCTIONS = "No UI changes detected \u2014 no blueprint capture needed.";
 export declare const NO_RESOLVABLE_URLS_INSTRUCTIONS = "Frontend changes detected but no candidate URLs could be resolved (workspace baseUrl missing or no router files matched). UI recommendations will be source-grounded only.";

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

@@ -352,8 +352,8 @@ export const analyzeChangesInputSchema = {
         .describe("Directory containing existing tests (auto-detected if omitted)"),
     topN: z
         .number()
-        .default(MAX_RECOMMENDATIONS)
         .optional()
+        .default(MAX_RECOMMENDATIONS)
         .describe(`Number of ranked test recommendations to generate. Defaults to ${MAX_RECOMMENDATIONS}.`),
     maxGenerate: z
         .number()
@@ -374,6 +374,11 @@ export const analyzeChangesInputSchema = {
         .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
+        .boolean()
+        .optional()
+        .default(false)
+        .describe("When true, diffs the base ref against the working tree (captures uncommitted and unstaged changes). Use for local-dev workflows. Defaults to false (CI mode — committed changes only)."),
 };
 // ── UI blueprint-capture instructions ──
 // Moved here from the former skyramp_ui_analyze_changes pre-flight tool. These
@@ -441,7 +446,7 @@ export function registerAnalyzeChangesTool(server) {
             if (analysisScope === AnalysisScope.CurrentBranchDiff) {
                 await sendProgress(10, 100, "Computing branch diff...");
                 try {
-                    diffData = await computeBranchDiff(params.repositoryPath, params.baseBranch);
+                    diffData = await computeBranchDiff(params.repositoryPath, params.baseBranch, params.includeUncommitted ?? false);
                     logger.info("Branch diff computed", {
                         currentBranch: diffData.currentBranch,
                         baseBranch: diffData.baseBranch,
@@ -1106,6 +1111,56 @@ export function registerAnalyzeChangesTool(server) {
                 affectedServices: classifiedEndpoints.affectedServices,
                 summary: "",
             } : undefined;
+            // ── Route discovery context for LLM grounding and state persistence ──
+            // fullAnalysis lives only in inMemorySessionStore (for MCP resources
+            // and registerRecommendTestsPrompt). The disk state carries only the
+            // slim fields that downstream tools (health, execute, actions) need.
+            // routerMountContext and candidateRouteFiles are computed here so they
+            // can be persisted to the state file for downstream tools (health, drift).
+            // Without them, analyzeTestHealth would work only off the static catalog
+            // which has wrong paths for nested resources and unsupported frameworks.
+            const routeLikeUnmatchedFiles = [];
+            for (const file of classifiedEndpoints?.unmatchedFiles ?? []) {
+                const routeLike = SOURCE_EXTS.test(file) &&
+                    (ROUTE_FILE_PATTERN.test(file) || ROUTE_FILE_BASENAME_PATTERN.test(path.basename(file)));
+                if (routeLike && !(await isGraphQLFile(file, params.repositoryPath))) {
+                    routeLikeUnmatchedFiles.push(file);
+                }
+            }
+            const shouldIncludeCandidateRouteFiles = analysisScope !== AnalysisScope.CurrentBranchDiff ||
+                rawRelatedEndpointCount === 0 ||
+                scannedEndpoints.length === 0 ||
+                routeLikeUnmatchedFiles.length > 0;
+            let candidateRouteFiles;
+            if (shouldIncludeCandidateRouteFiles) {
+                candidateRouteFiles = [];
+                for (const file of findCandidateRouteFiles(params.repositoryPath)) {
+                    if (!(await isGraphQLFile(file, params.repositoryPath))) {
+                        candidateRouteFiles.push(file);
+                    }
+                }
+            }
+            // Write the full diff to a temp file before building state so the path
+            // can be persisted and read by analyzeTestHealthTool for per-line detection.
+            let diffFilePath;
+            if (diffData?.diffContent) {
+                diffFilePath = path.join(os.tmpdir(), `skyramp-diff-${sessionId}.diff`);
+                await fs.promises.writeFile(diffFilePath, diffData.diffContent, { encoding: "utf-8", mode: 0o600 });
+            }
+            const routeDiscovery = {
+                candidateFiles: [
+                    ...(diffData?.changedFiles ?? []),
+                    ...(candidateRouteFiles ?? []),
+                ].filter((file, index, files) => files.indexOf(file) === index),
+                staticHints: scannedEndpoints.map((ep) => ({
+                    path: ep.path,
+                    methods: ep.methods,
+                    sourceFile: ep.sourceFile,
+                })),
+                openApiPaths: specPaths ? [...specPaths] : [],
+                routerMountContext,
+                ...(diffFilePath ? { diffFilePath } : {}),
+            };
             const fullAnalysis = {
                 metadata: {
                     repositoryName: path.basename(params.repositoryPath),
@@ -1162,6 +1217,7 @@ export function registerAnalyzeChangesTool(server) {
                     hasCoverageReports: false,
                     relevantExternalTestPaths,
                 },
+                routeDiscovery,
                 ...(diffContext ? { branchDiffContext: diffContext } : {}),
             };
             // Store RecommendationState in memory so it's compatible with skyramp_recommend_tests if needed
@@ -1171,42 +1227,6 @@ export function registerAnalyzeChangesTool(server) {
                 analysis: fullAnalysis,
             };
             storeSessionData(sessionId, recommendationState);
-            // ── Step 11: Build UnifiedAnalysisState and save ──
-            // fullAnalysis lives only in inMemorySessionStore (for MCP resources
-            // and registerRecommendTestsPrompt). The disk state carries only the
-            // slim fields that downstream tools (health, execute, actions) need.
-            // routerMountContext and candidateRouteFiles are computed here so they
-            // can be persisted to the state file for downstream tools (health, drift).
-            // Without them, analyzeTestHealth would work only off the static catalog
-            // which has wrong paths for nested resources and unsupported frameworks.
-            const routeLikeUnmatchedFiles = [];
-            for (const file of classifiedEndpoints?.unmatchedFiles ?? []) {
-                const routeLike = SOURCE_EXTS.test(file) &&
-                    (ROUTE_FILE_PATTERN.test(file) || ROUTE_FILE_BASENAME_PATTERN.test(path.basename(file)));
-                if (routeLike && !(await isGraphQLFile(file, params.repositoryPath))) {
-                    routeLikeUnmatchedFiles.push(file);
-                }
-            }
-            const shouldIncludeCandidateRouteFiles = analysisScope !== AnalysisScope.CurrentBranchDiff ||
-                rawRelatedEndpointCount === 0 ||
-                scannedEndpoints.length === 0 ||
-                routeLikeUnmatchedFiles.length > 0;
-            let candidateRouteFiles;
-            if (shouldIncludeCandidateRouteFiles) {
-                candidateRouteFiles = [];
-                for (const file of findCandidateRouteFiles(params.repositoryPath)) {
-                    if (!(await isGraphQLFile(file, params.repositoryPath))) {
-                        candidateRouteFiles.push(file);
-                    }
-                }
-            }
-            // Write the full diff to a temp file before building state so the path
-            // can be persisted and read by analyzeTestHealthTool for per-line detection.
-            let diffFilePath;
-            if (diffData?.diffContent) {
-                diffFilePath = path.join(os.tmpdir(), `skyramp-diff-${sessionId}.diff`);
-                await fs.promises.writeFile(diffFilePath, diffData.diffContent, { encoding: "utf-8", mode: 0o600 });
-            }
             // Compute UI context from the diff's changed files using the shared
             // `isFrontendFile` classifier. Persisting this in the stateFile lets
             // skyramp_analyze_test_health and the recommendation prompt consume the
@@ -1299,6 +1319,7 @@ export function registerAnalyzeChangesTool(server) {
                     sessionId,
                     routerMountContext,
                     candidateRouteFiles,
+                    routeDiscovery,
                     relevantExternalTestPaths,
                 },
             };

package/build/tools/trace/startTraceCollectionTool.js

@@ -5,7 +5,7 @@ import { SkyrampClient } from "@skyramp/skyramp";
 import openProxyTerminalTracked from "../../utils/proxy-terminal.js";
 import { getEntryPoint } from "../../utils/telemetry.js";
 import { logger } from "../../utils/logger.js";
-import { basePlaywrightSchema, baseSchema, SESSION_STORAGE_FILENAME, } from "../../types/TestTypes.js";
+import { basePlaywrightSchema, baseSchema, RuntimeEnvironment, SESSION_STORAGE_FILENAME, } from "../../types/TestTypes.js";
 import { AnalyticsService } from "../../services/AnalyticsService.js";
 import { resolveSessionPaths } from "./resolveSessionPaths.js";
 import { setSavedSessionPath } from "./sessionState.js";
@@ -66,8 +66,8 @@ For detailed documentation visit: https://www.skyramp.dev/docs/load-test/advance
             playwrightSaveStoragePath: basePlaywrightSchema.shape.playwrightSaveStoragePath,
             playwrightViewportSize: basePlaywrightSchema.shape.playwrightViewportSize,
             runtime: z
-                .string()
-                .default("docker")
+                .literal(RuntimeEnvironment.DOCKER)
+                .default(RuntimeEnvironment.DOCKER)
                 .describe("Runtime environment for trace collection. Currently only 'docker' is supported and is used as the default."),
             include: z
                 .array(z.string())

package/build/tools/workspace/initializeWorkspaceTool.js

@@ -1,6 +1,5 @@
 import { z } from "zod";
-import { WorkspaceConfigManager, serviceSchema } from "../../workspace/workspace.js";
-import { createRequire } from "module";
+import { WorkspaceConfigManager, serviceSchema, } from "../../workspace/workspace.js";
 import fs from "fs/promises";
 import yaml from "js-yaml";
 import { logger } from "../../utils/logger.js";
@@ -8,8 +7,6 @@ import { AnalyticsService } from "../../services/AnalyticsService.js";
 import { SKYRAMP_IMAGE_VERSION } from "../../utils/versions.js";
 import { validateAndConsumeScanToken } from "./initScanWorkspaceTool.js";
 import { upsertServicesByRepo } from "./serviceUpsert.js";
-const require = createRequire(import.meta.url);
-const MCP_VERSION = require("../../../package.json").version || "";
 function getExecutorVersion() {
     return SKYRAMP_IMAGE_VERSION;
 }
@@ -20,9 +17,7 @@ const TOOL_NAME = "skyramp_init_workspace";
 // set (incl. cookie/session/token) and the multi-repo `repository` field, so input
 // services are validated against it directly — no local schema extension needed.
 const initializeWorkspaceSchema = z.object({
-    workspacePath: z
-        .string()
-        .describe("Path to workspace directory"),
+    workspacePath: z.string().describe("Path to workspace directory"),
     services: z
         .array(serviceSchema)
         .describe("Array of services discovered by skyramp_init_scan. Must contain at least one service."),
@@ -123,14 +118,12 @@ export function registerInitializeWorkspaceTool(server) {
             if (params.merge && alreadyExists) {
                 config = await manager.read();
                 config = await manager.updateMetadata({
-                    mcpVersion: MCP_VERSION,
                     executorVersion: getExecutorVersion(),
                 });
             }
             else {
                 config = await manager.initialize();
                 config = await manager.updateMetadata({
-                    mcpVersion: MCP_VERSION,
                     executorVersion: getExecutorVersion(),
                 });
             }