fix: make platform optional and remove unrelated test

Author: kashishhora

src/modules/tools.ts

@@ -64,10 +64,7 @@ export function setupMCPCatTools(server: MCPServerLike): void {
         writeToLog(
           `Warning: Original list tools handler failed, this suggests an error MCPCat did not cause - ${error}`,
         );
-        event.error = {
-          message: getMCPCompatibleErrorMessage(error),
-          platform: "javascript",
-        };
+        event.error = { message: getMCPCompatibleErrorMessage(error) };
         event.isError = true;
         event.duration =
           (event.timestamp &&
@@ -88,10 +85,7 @@ export function setupMCPCatTools(server: MCPServerLike): void {
         writeToLog(
           "Warning: No tools found in the original list. This is likely due to the tools not being registered before MCPCat.track().",
         );
-        event.error = {
-          message: "No tools were sent to MCP client.",
-          platform: "javascript",
-        };
+        event.error = { message: "No tools were sent to MCP client." };
         event.isError = true;
         event.duration =
           (event.timestamp &&

src/modules/tracing.ts

@@ -74,10 +74,7 @@ export function setupListToolsTracing(
         writeToLog(
           `Warning: Original list tools handler failed, this suggests an error MCPCat did not cause - ${error}`,
         );
-        event.error = {
-          message: getMCPCompatibleErrorMessage(error),
-          platform: "javascript",
-        };
+        event.error = { message: getMCPCompatibleErrorMessage(error) };
         event.isError = true;
         event.duration =
           (event.timestamp &&
@@ -98,10 +95,7 @@ export function setupListToolsTracing(
         writeToLog(
           "Warning: No tools found in the original list. This is likely due to the tools not being registered before MCPCat.track().",
         );
-        event.error = {
-          message: "No tools were sent to MCP client.",
-          platform: "javascript",
-        };
+        event.error = { message: "No tools were sent to MCP client." };
         event.isError = true;
         event.duration =
           (event.timestamp &&
@@ -264,7 +258,6 @@ export function setupToolCallTracing(server: MCPServerLike): void {
           event.isError = true;
           event.error = {
             message: `Tool call handler not found for ${request.params?.name || "unknown"}`,
-            platform: "javascript",
           };
           event.duration =
             (event.timestamp &&

src/tests/protocol-validation-errors.test.ts

@@ -195,5 +195,5 @@ export interface ErrorData {
   stack?: string; // Full stack trace string
   frames?: StackFrame[]; // Parsed stack frames
   chained_errors?: ChainedErrorData[];
-  platform: string; // Platform identifier (e.g., "javascript", "node")
+  platform?: string; // Platform identifier (e.g., "javascript", "node")
 }