feat: add structured error tracking with stack trace parsing (#11)

Author: kashishhora

src/modules/exceptions.ts

@@ -411,8 +411,6 @@ export class SentryExporter implements Exporter {
       } else if (typeof event.error === "object" && event.error !== null) {
         if ("message" in event.error) {
           errorMessage = String(event.error.message);
-        } else if ("error" in event.error) {
-          errorMessage = String(event.error.error);
         } else {
           errorMessage = JSON.stringify(event.error);
         }

src/modules/exporters/sentry.ts

@@ -1,5 +1,6 @@
 export * from "./compatibility.js";
 export * from "./context-parameters.js";
+export * from "./exceptions.js";
 export * from "./internal.js";
 export * from "./logging.js";
 export * from "./session.js";

src/modules/index.ts

@@ -7,9 +7,9 @@ import {
 } from "../types.js";
 import { PublishEventRequestEventTypeEnum } from "mcpcat-api";
 import { publishEvent } from "./eventQueue.js";
-import { getMCPCompatibleErrorMessage } from "./compatibility.js";
 import { writeToLog } from "./logging.js";
 import { INACTIVITY_TIMEOUT_IN_MINUTES } from "./constants.js";
+import { captureException } from "./exceptions.js";
 
 /**
  * Simple LRU cache for session identities.
@@ -325,9 +325,7 @@ export async function handleIdentify(
         new Date().getTime() - identifyEvent.timestamp.getTime()) ||
       undefined;
     identifyEvent.isError = true;
-    identifyEvent.error = {
-      message: getMCPCompatibleErrorMessage(error),
-    };
+    identifyEvent.error = captureException(error);
     publishEvent(server, identifyEvent);
   }
 }

src/modules/internal.ts

@@ -16,6 +16,7 @@ import { getServerSessionId } from "./session.js";
 import { PublishEventRequestEventTypeEnum } from "mcpcat-api";
 import { publishEvent } from "./eventQueue.js";
 import { getMCPCompatibleErrorMessage } from "./compatibility.js";
+import { captureException } from "./exceptions.js";
 
 function isToolResultError(result: any): boolean {
   return result && typeof result === "object" && result.isError === true;
@@ -269,19 +270,15 @@ export function setupToolCallTracing(server: MCPServerLike): void {
         // Check if the result indicates an error
         if (isToolResultError(result)) {
           event.isError = true;
-          event.error = {
-            message: getMCPCompatibleErrorMessage(result),
-          };
+          event.error = captureException(result);
         }
 
         event.response = result;
         publishEvent(server, event);
         return result;
       } catch (error) {
         event.isError = true;
-        event.error = {
-          message: getMCPCompatibleErrorMessage(error),
-        };
+        event.error = captureException(error);
         publishEvent(server, event);
         throw error;
       }

src/modules/tracing.ts

@@ -12,10 +12,10 @@ import { getServerTrackingData, handleIdentify } from "./internal.js";
 import { getServerSessionId } from "./session.js";
 import { PublishEventRequestEventTypeEnum } from "mcpcat-api";
 import { publishEvent } from "./eventQueue.js";
-import { getMCPCompatibleErrorMessage } from "./compatibility.js";
 import { addContextParameterToTool } from "./context-parameters.js";
 import { handleReportMissing } from "./tools.js";
 import { setupInitializeTracing, setupListToolsTracing } from "./tracing.js";
+import { captureException } from "./exceptions.js";
 
 // WeakMap to track which callbacks have already been wrapped
 const wrappedCallbacks = new WeakMap<Function, boolean>();
@@ -359,9 +359,7 @@ function addTracingToToolCallback(
         // Check if the result indicates an error
         if (isToolResultError(result)) {
           event.isError = true;
-          event.error = {
-            message: getMCPCompatibleErrorMessage(result),
-          };
+          event.error = captureException(result);
         }
 
         event.response = result;
@@ -373,9 +371,7 @@ function addTracingToToolCallback(
         return result;
       } catch (error) {
         event.isError = true;
-        event.error = {
-          message: getMCPCompatibleErrorMessage(error),
-        };
+        event.error = captureException(error);
         event.duration =
           (event.timestamp &&
             new Date().getTime() - event.timestamp.getTime()) ||