@skyramp/mcp 0.4.4-rc.1 → 0.4.4

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

@@ -1,7 +1,6 @@
-import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import { McpServer, CallToolResult } from "@modelcontextprotocol/server";
 import { z } from "zod";
 import { AuthFinding } from "../../utils/workspaceAuth.js";
-import { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
 import { TraceFile } from "../../types/RepositoryAnalysis.js";
 import { ChangedFileName } from "../../utils/branchDiff.js";
 import { RemovedUiElement } from "../../utils/removedUiElements.js";
@@ -12,11 +11,14 @@ export declare function isGraphQLFile(filePath: string, repositoryPath: string):
 export declare const analyzeChangesInputSchema: {
     repositoryPath: z.ZodString;
     repository: z.ZodString;
-    scope: z.ZodOptional<z.ZodDefault<z.ZodEnum<["full_repo", "branch_diff"]>>>;
+    scope: z.ZodOptional<z.ZodDefault<z.ZodEnum<{
+        full_repo: "full_repo";
+        branch_diff: "branch_diff";
+    }>>>;
     baseBranch: z.ZodOptional<z.ZodString>;
     testDirectory: z.ZodOptional<z.ZodString>;
     prNumber: z.ZodOptional<z.ZodNumber>;
-    testsRepoDir: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
+    testsRepoDir: z.ZodOptional<z.ZodString>;
     includeUncommitted: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
 };
 /**
@@ -86,7 +88,7 @@ export declare const analyzeChangesOutputSchema: {
     repositoryPath: z.ZodString;
     repositoryName: z.ZodString;
     repository: z.ZodString;
-    relatedRepositories: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
+    relatedRepositories: z.ZodOptional<z.ZodArray<z.ZodString>>;
     analysisScope: z.ZodString;
     planOnly: z.ZodBoolean;
     currentBranch: z.ZodOptional<z.ZodString>;
@@ -94,26 +96,12 @@ export declare const analyzeChangesOutputSchema: {
     changedFiles: z.ZodArray<z.ZodObject<{
         path: z.ZodString;
         deleted: z.ZodBoolean;
-    }, "strip", z.ZodTypeAny, {
-        path: string;
-        deleted: boolean;
-    }, {
-        path: string;
-        deleted: boolean;
-    }>, "many">;
+    }, z.core.$strip>>;
     existingTestFiles: z.ZodOptional<z.ZodObject<{
-        skyramp: z.ZodArray<z.ZodString, "many">;
-        external: z.ZodArray<z.ZodString, "many">;
-        relevantExternal: z.ZodArray<z.ZodString, "many">;
-    }, "strip", z.ZodTypeAny, {
-        skyramp: string[];
-        external: string[];
-        relevantExternal: string[];
-    }, {
-        skyramp: string[];
-        external: string[];
-        relevantExternal: string[];
-    }>>;
+        skyramp: z.ZodArray<z.ZodString>;
+        external: z.ZodArray<z.ZodString>;
+        relevantExternal: z.ZodArray<z.ZodString>;
+    }, z.core.$strip>>;
     workspace: z.ZodOptional<z.ZodObject<{
         baseUrl: z.ZodString;
         authMethod: z.ZodString;
@@ -124,113 +112,29 @@ export declare const analyzeChangesOutputSchema: {
             kind: z.ZodLiteral<"workspace-disagreement">;
             message: z.ZodString;
             serviceName: z.ZodString;
-            field: z.ZodEnum<["authType", "authHeader", "authScheme"]>;
+            field: z.ZodEnum<{
+                authHeader: "authHeader";
+                authScheme: "authScheme";
+                authType: "authType";
+            }>;
             primary: z.ZodString;
             analysed: z.ZodString;
             primaryWorkspaceFile: z.ZodString;
             analysedWorkspaceFile: z.ZodString;
-        }, "strip", z.ZodTypeAny, {
-            message: string;
-            serviceName: string;
-            kind: "workspace-disagreement";
-            field: "authHeader" | "authScheme" | "authType";
-            primaryWorkspaceFile: string;
-            analysedWorkspaceFile: string;
-            primary: string;
-            analysed: string;
-        }, {
-            message: string;
-            serviceName: string;
-            kind: "workspace-disagreement";
-            field: "authHeader" | "authScheme" | "authType";
-            primaryWorkspaceFile: string;
-            analysedWorkspaceFile: string;
-            primary: string;
-            analysed: string;
-        }>, "many">>;
-    }, "strip", z.ZodTypeAny, {
-        baseUrl: string;
-        authMethod: string;
-        openApiSpecLoaded: boolean;
-        authHeader?: string | undefined;
-        openApiSpecPath?: string | undefined;
-        authFindings?: {
-            message: string;
-            serviceName: string;
-            kind: "workspace-disagreement";
-            field: "authHeader" | "authScheme" | "authType";
-            primaryWorkspaceFile: string;
-            analysedWorkspaceFile: string;
-            primary: string;
-            analysed: string;
-        }[] | undefined;
-    }, {
-        baseUrl: string;
-        authMethod: string;
-        openApiSpecLoaded: boolean;
-        authHeader?: string | undefined;
-        openApiSpecPath?: string | undefined;
-        authFindings?: {
-            message: string;
-            serviceName: string;
-            kind: "workspace-disagreement";
-            field: "authHeader" | "authScheme" | "authType";
-            primaryWorkspaceFile: string;
-            analysedWorkspaceFile: string;
-            primary: string;
-            analysed: string;
-        }[] | undefined;
-    }>>;
+        }, z.core.$strip>>>;
+    }, z.core.$strip>>;
     uiContext: z.ZodOptional<z.ZodObject<{
         removedElements: z.ZodOptional<z.ZodArray<z.ZodObject<{
             attribute: z.ZodString;
             value: z.ZodString;
             file: z.ZodString;
-        }, "strip", z.ZodTypeAny, {
-            value: string;
-            file: string;
-            attribute: string;
-        }, {
-            value: string;
-            file: string;
-            attribute: string;
-        }>, "many">>;
+        }, z.core.$strip>>>;
         retiredElements: z.ZodOptional<z.ZodArray<z.ZodObject<{
             attribute: z.ZodString;
             value: z.ZodString;
             file: z.ZodString;
-        }, "strip", z.ZodTypeAny, {
-            value: string;
-            file: string;
-            attribute: string;
-        }, {
-            value: string;
-            file: string;
-            attribute: string;
-        }>, "many">>;
-    }, "strip", z.ZodTypeAny, {
-        removedElements?: {
-            value: string;
-            file: string;
-            attribute: string;
-        }[] | undefined;
-        retiredElements?: {
-            value: string;
-            file: string;
-            attribute: string;
-        }[] | undefined;
-    }, {
-        removedElements?: {
-            value: string;
-            file: string;
-            attribute: string;
-        }[] | undefined;
-        retiredElements?: {
-            value: string;
-            file: string;
-            attribute: string;
-        }[] | undefined;
-    }>>;
+        }, z.core.$strip>>>;
+    }, z.core.$strip>>;
     nextStep: z.ZodOptional<z.ZodString>;
 };
 /**

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

@@ -187,11 +187,11 @@ export function registerAnalyzeChangesTool(server) {
         description: `Read the repository and the current change, and discover the existing tests — first step of the unified Test Health Analysis Flow. Returns DATA only, no instructions: a stateFile path, the branch and its base, the changed files with their status, the existing test files, the workspace base URL and auth, the OpenAPI spec path, the repository's seed and fixture data files, and the identifying attributes the change removed from surviving files. It proposes no tests and it tells you nothing about the diff — read the diff and decide what the change warrants. Pass stateFile to skyramp_register_test_plan (required before you write any test), skyramp_analyze_test_health and skyramp_actions.`,
         // SKYR-4182: the payload is returned via structuredContent, which requires a
         // declared outputSchema.
-        outputSchema: analyzeChangesOutputSchema,
-        inputSchema: analyzeChangesInputSchema,
-    }, async (params, extra) => {
+        outputSchema: z.object(analyzeChangesOutputSchema),
+        inputSchema: z.object(analyzeChangesInputSchema),
+    }, async (params, ctx) => {
         let errorResult;
-        const sendProgress = makeProgressReporter(extra);
+        const sendProgress = makeProgressReporter(ctx);
         try {
             const analysisScope = params.scope === "branch_diff" || !params.scope
                 ? AnalysisScope.CurrentBranchDiff

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

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

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

@@ -44,7 +44,7 @@ export function registerAnalyzeTestHealthTool(server) {
             openWorldHint: false,
         },
         description: `Assess existing tests for drift against the current branch diff and generate maintenance instructions (UPDATE, REGENERATE, DELETE, VERIFY, or IGNORE) for each test. Covers both API tests and UI tests (Playwright, Cypress, RTL, component tests). Requires a stateFile from skyramp_analyze_changes.`,
-        inputSchema: {
+        inputSchema: z.object({
             stateFile: z
                 .string()
                 .describe("Path to the analyze-changes-state.json file returned by skyramp_analyze_changes (the stateFile field in its output). Do NOT pass testbot-result.txt here."),
@@ -57,12 +57,12 @@ export function registerAnalyzeTestHealthTool(server) {
                 .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: {
+        }),
+        outputSchema: z.object({
             prompt: z
                 .string()
                 .describe("LLM drift analysis prompt and assessment instructions."),
-        },
+        }),
     }, async (args) => {
         let errorResult;
         try {