@skyramp/mcp 0.0.63-rc.7 → 0.0.63

package/build/prompts/testbot/testbot-prompts.js

@@ -170,8 +170,8 @@ Call \`skyramp_submit_report\` with \`summaryOutputFile\`: "${summaryOutputFile}
 
 **nextSteps** — actionable next steps when test failures suggest misconfiguration.
    Each entry must be a single-line string (no embedded newlines).
-   If multiple tests fail with 404 NOT_FOUND or connection refused on endpoints that ARE defined in the diff, add: "Some endpoints returned 404 — verify your \`target_setup_command\` deploys the PR branch and \`target_ready_check_command\` confirms the service is healthy."
-   If tests fail with 401/403 on endpoints that require auth, add a step about \`auth_token_command\`.
+   If multiple tests fail with 404 NOT_FOUND or connection refused on endpoints that ARE defined in the diff, add: "Some endpoints returned 404 — verify your \`targetSetupCommand\` deploys the PR branch and \`targetReadyCheckCommand\` confirms the service is healthy."
+   If tests fail with 401/403 on endpoints that require auth, add a step about \`authTokenCommand\`.
    Only add next steps for systemic patterns (3+ tests with the same error class), not individual failures.
 
 **businessCaseAnalysis** — 1-2 sentences describing what user-facing interactions this PR

package/build/services/TestExecutionService.js

@@ -8,7 +8,7 @@ import { logger } from "../utils/logger.js";
 import { buildContainerEnv } from "./containerEnv.js";
 const DEFAULT_TIMEOUT = 300000; // 5 minutes
 const MAX_CONCURRENT_EXECUTIONS = 5;
-export const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.14";
+export const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.15";
 const DOCKER_PLATFORM = "linux/amd64";
 const EXECUTION_PROGRESS_INTERVAL = 10000; // 10 seconds between progress updates during execution
 // Temp file with valid empty JSON — used instead of /dev/null for .json config files

package/build/tools/generate-tests/generateContractRestTool.js

@@ -1,3 +1,4 @@
+import path from "path";
 import { z } from "zod";
 import { baseTestSchema, TestType } from "../../types/TestTypes.js";
 import { TestGenerationService, } from "../../services/TestGenerationService.js";
@@ -30,11 +31,31 @@ const contractTestSchema = {
     providerOutput: z
         .string()
         .optional()
-        .describe("Absolute file path for the generated provider contract test file"),
+        .refine((val) => {
+        if (!val || val === "")
+            return true;
+        const validExtensions = [".py", ".ts", ".js", ".java"];
+        return validExtensions.some((ext) => val.endsWith(ext));
+    }, { message: "Output file must have one of these extensions: .py, .ts, .js, .java" })
+        .describe("Output file for the generated provider contract test. " +
+        "For Playwright framework, the file must match *.spec.ts so Playwright can discover it " +
+        "(e.g. contract_provider_orders.spec.ts). " +
+        "For other frameworks (pytest, junit), standard extensions are fine (.py, .java). " +
+        "Optional — when omitted, Skyramp generates a default name."),
     consumerOutput: z
         .string()
         .optional()
-        .describe("Absolute file path for the generated consumer contract test file"),
+        .refine((val) => {
+        if (!val || val === "")
+            return true;
+        const validExtensions = [".py", ".ts", ".js", ".java"];
+        return validExtensions.some((ext) => val.endsWith(ext));
+    }, { message: "Output file must have one of these extensions: .py, .ts, .js, .java" })
+        .describe("Output file for the generated consumer contract test. " +
+        "For Playwright framework, the file must match *.spec.ts so Playwright can discover it " +
+        "(e.g. contract_consumer_orders.spec.ts). " +
+        "For other frameworks (pytest, junit), standard extensions are fine (.py, .java). " +
+        "Optional — when omitted, Skyramp generates a default name."),
     parentRequestData: z
         .record(z.string(), z.string())
         .optional()
@@ -55,6 +76,13 @@ const contractTestSchema = {
         "The value is the expected HTTP status code string for the provisioning call that creates that parent resource. " +
         "For example: {\"product_id\": \"201\"}. " +
         "Requires apiSchema. Not allowed with consumerMode or skipProvisionParents."),
+    parentFormParams: z
+        .record(z.string(), z.string())
+        .optional()
+        .describe("Map of form parameters for provisioning parent resources. " +
+        "Key is the path param variable name (e.g. 'product_id'); " +
+        "value is comma-separated key=value pairs (e.g. 'field1=val1,field2=val2'). " +
+        "Requires apiSchema. Not allowed with consumerMode or skipProvisionParents."),
     skipProvisionParents: z
         .boolean()
         .default(false)
@@ -126,9 +154,9 @@ ${sections.join("\n")}
         if (errList.isError)
             return errList;
         const errors = [];
-        if ((params.parentRequestData || params.parentStatusCode) &&
+        if ((params.parentRequestData || params.parentStatusCode || params.parentFormParams) &&
             !params.apiSchema) {
-            errors.push("parentRequestData and parentStatusCode are only allowed when apiSchema is provided.");
+            errors.push("parentRequestData, parentStatusCode, and parentFormParams are only allowed when apiSchema is provided.");
         }
         if (params.providerOutput && !params.providerMode) {
             errors.push("providerOutput is only valid when providerMode is enabled.");
@@ -136,15 +164,34 @@ ${sections.join("\n")}
         if (params.consumerOutput && !params.consumerMode) {
             errors.push("consumerOutput is only valid when consumerMode is enabled.");
         }
-        if (params.consumerMode && (params.parentRequestData || params.parentStatusCode)) {
-            errors.push("parentRequestData and parentStatusCode are not allowed when consumerMode is enabled.");
+        if (params.consumerMode && (params.parentRequestData || params.parentStatusCode || params.parentFormParams)) {
+            errors.push("parentRequestData, parentStatusCode, and parentFormParams are not allowed when consumerMode is enabled.");
+        }
+        const fw = (params.framework ?? "").toLowerCase();
+        if (fw === "playwright") {
+            const specPattern = /\.(spec|test)\.[tj]s$/;
+            for (const [field, value] of [
+                ["providerOutput", params.providerOutput],
+                ["consumerOutput", params.consumerOutput],
+            ]) {
+                if (value && value !== "") {
+                    if (!specPattern.test(value)) {
+                        const parsed = path.parse(value);
+                        const suggested = /\.[tj]s$/.test(parsed.ext)
+                            ? value.replace(/\.[tj]s$/, ".spec.ts")
+                            : value + ".spec.ts";
+                        errors.push(`Error: Playwright requires test files to match *.{spec}.{ts,js} (got "${value}" for ${field}). ` +
+                            `Rename to e.g. ${suggested} so Playwright can discover it.`);
+                    }
+                }
+            }
         }
         if (params.skipProvisionParents) {
             if (!params.providerMode) {
                 errors.push("skipProvisionParents requires providerMode to be enabled.");
             }
-            if (params.parentRequestData || params.parentStatusCode) {
-                errors.push("parentRequestData and parentStatusCode are not allowed when skipProvisionParents is enabled.");
+            if (params.parentRequestData || params.parentStatusCode || params.parentFormParams) {
+                errors.push("parentRequestData, parentStatusCode, and parentFormParams are not allowed when skipProvisionParents is enabled.");
             }
         }
         if (errors.length > 0) {
@@ -166,6 +213,7 @@ ${sections.join("\n")}
             consumerOutput: params.consumerOutput,
             parentRequestData: params.parentRequestData,
             parentStatusCode: params.parentStatusCode,
+            parentFormParams: params.parentFormParams,
             skipProvisionParents: params.skipProvisionParents,
         };
     }

package/build/tools/generate-tests/generateMockRestTool.js

@@ -128,7 +128,10 @@ This file must:
             method: params.method,
             apiSchema: params.apiSchema ? [params.apiSchema] : [],
             language: params.language ?? "python",
-            framework: params.framework ?? "",
+            framework: params.framework ??
+                (params.language === "typescript" || params.language === "javascript"
+                    ? "playwright"
+                    : ""),
             output: params.output,
             outputDir: params.outputDir,
             force: params.force ?? true,

package/build/tools/submitReportTool.js

@@ -88,7 +88,7 @@ export function registerSubmitReportTool(server) {
                 .optional()
                 .default([])
                 .describe("Actionable next steps for the user. Populate when test failures suggest misconfiguration " +
-                "(e.g. 404s on endpoints that exist in the diff → check target_setup_command)."),
+                "(e.g. 404s on endpoints that exist in the diff → check targetSetupCommand)."),
             commitMessage: z
                 .string()
                 .optional()

package/build/tools/submitReportTool.test.js

@@ -170,12 +170,12 @@ describe("registerSubmitReportTool", () => {
         const outputFile = path.join(tmpDir, "report.json");
         const result = await handler({
             ...sampleReportParams(outputFile),
-            nextSteps: ["Check your target_setup_command — endpoints returned 404"],
+            nextSteps: ["Check your targetSetupCommand — endpoints returned 404"],
         });
         expect(result.isError).toBeUndefined();
         const written = JSON.parse(await fs.readFile(outputFile, "utf-8"));
         expect(written.nextSteps).toEqual([
-            "Check your target_setup_command — endpoints returned 404",
+            "Check your targetSetupCommand — endpoints returned 404",
         ]);
     });
     it("defaults nextSteps to empty array when omitted", async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.0.63-rc.7",
+  "version": "0.0.63",
   "main": "build/index.js",
   "type": "module",
   "bin": {
@@ -46,7 +46,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.24.3",
     "@playwright/test": "^1.55.0",
-    "@skyramp/skyramp": "1.3.14",
+    "@skyramp/skyramp": "1.3.15",
     "dockerode": "^4.0.6",
     "fast-glob": "^3.3.3",
     "simple-git": "^3.30.0",