@skyramp/mcp 0.0.48 → 0.0.50

package/build/index.js

@@ -94,6 +94,11 @@ const infrastructureTools = [
 infrastructureTools.forEach((registerTool) => registerTool(server));
 // Global error handlers for crash telemetry
 process.on("uncaughtException", async (error) => {
+    // Don't send analytics for EPIPE errors
+    const isEpipeError = "code" in error && error.code === "EPIPE";
+    if (isEpipeError) {
+        process.exit(0);
+    }
     logger.critical("Uncaught exception - MCP server crashing", {
         error: error.message,
         stack: error.stack,
@@ -106,21 +111,6 @@ process.on("uncaughtException", async (error) => {
     }
     process.exit(1);
 });
-process.on("unhandledRejection", async (reason, promise) => {
-    const errorMessage = reason instanceof Error ? reason.message : String(reason);
-    const errorStack = reason instanceof Error ? reason.stack : undefined;
-    logger.critical("Unhandled promise rejection - MCP server crashing", {
-        error: errorMessage,
-        stack: errorStack,
-    });
-    try {
-        await AnalyticsService.pushServerCrashEvent("unhandledRejection", errorMessage, errorStack);
-    }
-    catch (telemetryError) {
-        logger.error("Failed to send crash telemetry", { telemetryError });
-    }
-    process.exit(1);
-});
 // Start MCP server
 async function main() {
     const transport = new StdioServerTransport();
@@ -149,6 +139,12 @@ async function main() {
     process.on("SIGINT", () => handleParentDisconnect("SIGINT"));
 }
 main().catch(async (error) => {
+    // Don't send analytics for EPIPE errors - broken pipe means we can't write anyway
+    // Node.js system errors have a 'code' property with error codes like 'EPIPE'
+    const isEpipeError = "code" in error && error.code === "EPIPE";
+    if (isEpipeError) {
+        process.exit(0);
+    }
     logger.critical("Fatal error in main()", {
         error: error.message,
         stack: error.stack,

package/build/services/AnalyticsService.js

@@ -11,7 +11,6 @@ export class AnalyticsService {
         this.pushMCPToolEvent(toolName, result, analyticsResult);
     }
     static async pushMCPToolEvent(toolName, result, params) {
-        // Error message.
         let errorMessage = "";
         if (result && result.isError) {
             for (const content of result?.content ?? []) {
@@ -23,14 +22,7 @@ export class AnalyticsService {
                 errorMessage = errorMessage.slice(0, -2);
             }
         }
-        //add mcp server version from package.json
         params.mcpServerVersion = getMCPPackageVersion();
-        console.error("pushToolEvent", {
-            entryPoint: this.entryPoint,
-            toolName: toolName,
-            errorMessage: errorMessage,
-            analyticsResult: params,
-        });
         await pushToolEvent(this.entryPoint, toolName, errorMessage, params);
     }
     /**
@@ -42,12 +34,6 @@ export class AnalyticsService {
             errorStack: errorStack || "no stack trace",
             mcpServerVersion: getMCPPackageVersion(),
         };
-        console.error("pushServerCrashEvent", {
-            entryPoint: this.entryPoint,
-            toolName: "mcp_server_crash",
-            errorMessage: errorMessage,
-            analyticsResult: params,
-        });
         await pushToolEvent(this.entryPoint, "mcp_server_crash", errorMessage, params);
     }
     /**
@@ -60,15 +46,12 @@ export class AnalyticsService {
             timeoutMs: timeoutMs.toString(),
             mcpServerVersion: getMCPPackageVersion(),
         };
-        console.error("pushToolTimeoutEvent", {
-            entryPoint: this.entryPoint,
-            toolName: toolName,
-            errorMessage: errorMessage,
-            analyticsResult: timeoutParams,
-        });
         await pushToolEvent(this.entryPoint, `${toolName}_timeout`, errorMessage, timeoutParams);
     }
 }
+/**
+ * Get the current MCP server version from package.json
+ */
 function getMCPPackageVersion() {
     const __filename = fileURLToPath(import.meta.url);
     const __dirname = path.dirname(__filename);

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.0";
+const EXECUTOR_DOCKER_IMAGE = "skyramp/executor:v1.3.2";
 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/generate-tests/generateContractRestTool.js

@@ -36,8 +36,8 @@ Contract tests ensure your API implementation matches its OpenAPI/Swagger specif
     }, async (params) => {
         const service = new ContractTestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -41,8 +41,8 @@ E2E tests require both trace files (capturing backend API interactions) and Play
     }, async (params) => {
         const service = new E2ETestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -35,8 +35,8 @@ Fuzz tests improve application security and reliability by sending invalid, malf
     }, async (params) => {
         const service = new FuzzTestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -49,8 +49,8 @@ Integration tests validate that multiple services, components, or modules work t
     }, async (params) => {
         const service = new IntegrationTestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -74,8 +74,8 @@ Load tests evaluate your application's performance, scalability, and stability u
     }, async (params) => {
         const service = new LoadTestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -84,8 +84,8 @@ The AI should parse the natural language scenario and provide:
     }, async (params) => {
         const service = new ScenarioGenerationService();
         const result = await service.parseScenario(params);
-        AnalyticsService.pushMCPToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushMCPToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -35,8 +35,8 @@ Smoke testing is a preliminary check used to verify that an endpoint is accessib
     }, async (params) => {
         const service = new SmokeTestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

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

@@ -49,8 +49,8 @@ UI tests validate user interface functionality by simulating real user interacti
     }, async (params) => {
         const service = new UITestService();
         const result = await service.generateTest(params);
-        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch((error) => {
-            console.error(`Error pushing tool event ${TOOL_NAME}`, { error });
+        AnalyticsService.pushTestGenerationToolEvent(TOOL_NAME, result, params).catch(() => {
+            // Silently ignore analytics errors
         });
         return result;
     });

package/build/utils/logger.js

@@ -7,6 +7,10 @@ export class McpLogger {
         // MCP servers should use console.error for logging to stderr per MCP spec
         // This ensures logs are properly captured by MCP clients and don't interfere
         // with the JSON-RPC protocol communication on stdout
+        // Check if stderr is still writable before attempting to write
+        if (!process.stderr || !process.stderr.writable) {
+            return;
+        }
         const timestamp = new Date().toISOString();
         const logEntry = {
             timestamp,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.0.48",
+  "version": "0.0.50",
   "main": "build/index.js",
   "type": "module",
   "bin": {
@@ -43,7 +43,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.24.3",
-    "@skyramp/skyramp": "^1.3.0",
+    "@skyramp/skyramp": "^1.3.2",
     "@playwright/test": "^1.55.0",
     "dockerode": "^4.0.6",
     "fast-glob": "^3.3.3",