@skyramp/mcp 0.0.51 → 0.0.53

package/build/services/TestExecutionService.js

@@ -6,7 +6,7 @@ import { stripVTControlCharacters } from "util";
 import { logger } from "../utils/logger.js";
 const DEFAULT_TIMEOUT = 300000; // 5 minutes
 const MAX_CONCURRENT_EXECUTIONS = 5;
-const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.3";
+const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.5";
 const DOCKER_PLATFORM = "linux/amd64";
 const EXECUTION_PROGRESS_INTERVAL = 10000; // 10 seconds between progress updates during execution
 // Files and directories to exclude when mounting workspace to Docker container

package/build/tools/trace/startTraceCollectionTool.js

@@ -3,7 +3,7 @@ import { SkyrampClient } from "@skyramp/skyramp";
 import openProxyTerminalTracked from "../../utils/proxy-terminal.js";
 import { TELEMETRY_entrypoint_FIELD_NAME } from "../../utils/utils.js";
 import { logger } from "../../utils/logger.js";
-import { basePlaywrightSchema, baseSchema, DEFAULT_SESSION_STORAGE_PATH, SESSION_STORAGE_FILENAME, } from "../../types/TestTypes.js";
+import { basePlaywrightSchema, baseSchema, } from "../../types/TestTypes.js";
 import { AnalyticsService } from "../../services/AnalyticsService.js";
 import path from "path";
 const TOOL_NAME = "skyramp_start_trace_collection";
@@ -94,11 +94,13 @@ For detailed documentation visit: https://www.skyramp.dev/docs/load-test/advance
             outputDir: params.outputDir,
             prompt: params.prompt,
         });
-        // Resolve playwrightSaveStoragePath to outputDir if it's the default relative path
         let saveStoragePath = params.playwrightSaveStoragePath;
-        if (params.outputDir &&
-            saveStoragePath === DEFAULT_SESSION_STORAGE_PATH) {
-            saveStoragePath = path.join(params.outputDir, SESSION_STORAGE_FILENAME);
+        if (saveStoragePath) {
+            // If saveStoragePath is just a filename (no directory separators), use outputDir
+            if (params.outputDir && !saveStoragePath.includes(path.sep) && !path.isAbsolute(saveStoragePath)) {
+                saveStoragePath = path.join(params.outputDir, saveStoragePath);
+            }
+            logger.info("Session storage will be saved to:", { saveStoragePath });
         }
         try {
             // Send initial progress
@@ -118,6 +120,9 @@ For detailed documentation visit: https://www.skyramp.dev/docs/load-test/advance
                 playwrightViewportSize: params.playwrightViewportSize,
                 entrypoint: TELEMETRY_entrypoint_FIELD_NAME,
             };
+            if (saveStoragePath) {
+                generateOptions.playwrightSaveStoragePath = saveStoragePath;
+            }
             // Send progress for configuration
             const traceMode = params.playwright ? "UI + Backend" : "Backend-only";
             await sendProgress(30, 100, `Configuring ${traceMode} trace collection...`);

package/build/types/TestTypes.js

@@ -1,6 +1,5 @@
 import { z } from "zod";
 export const SESSION_STORAGE_FILENAME = "skyramp_session_storage.json";
-export const DEFAULT_SESSION_STORAGE_PATH = `./${SESSION_STORAGE_FILENAME}`;
 export var TestType;
 (function (TestType) {
     TestType["SMOKE"] = "smoke";
@@ -88,11 +87,11 @@ export const basePlaywrightSchema = z.object({
     playwrightStoragePath: z
         .string()
         .optional()
-        .describe("Path to playwright session storage to load for trace collection with playwright enabled. THE PATH MUST BE AN ABSOLUTE PATH LIKE /Users/<path>/<filename>.json"),
+        .describe("Path to LOAD existing playwright session storage for trace collection with playwright enabled. THE PATH MUST BE AN ABSOLUTE PATH LIKE /Users/<path>/<filename>.json"),
     playwrightSaveStoragePath: z
         .string()
-        .default(DEFAULT_SESSION_STORAGE_PATH)
-        .describe(`Path to SAVE Playwright session storage after trace collection. When user says 'with session storage' or 'save session' without specifying a path, use the default '${DEFAULT_SESSION_STORAGE_PATH}'. This SAVES authentication state (cookies, localStorage, sessionStorage) when the browser closes. To LOAD existing auth state, use playwrightStoragePath instead. Can be relative (e.g., 'auth.json') or absolute path.`),
+        .optional()
+        .describe(`Path to SAVE Playwright session storage after trace collection. ONLY provide this when user explicitly says 'with session storage', 'save session', or similar. If user specifies this without a path, defaults to '${SESSION_STORAGE_FILENAME}' in the outputDir. This SAVES authentication state (cookies, localStorage, sessionStorage) when the browser closes. To LOAD existing auth state, use playwrightStoragePath instead. Can be relative (e.g., 'auth.json') or absolute path.`),
     playwrightViewportSize: z
         .string()
         .default("")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.0.51",
+  "version": "0.0.53",
   "main": "build/index.js",
   "type": "module",
   "bin": {
@@ -45,7 +45,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.24.3",
-    "@skyramp/skyramp": "^1.3.3",
+    "@skyramp/skyramp": "1.3.5",
     "@playwright/test": "^1.55.0",
     "dockerode": "^4.0.6",
     "fast-glob": "^3.3.3",