From 43829a3c06050cc099954ce25e22064c58e058dc Mon Sep 17 00:00:00 2001
From: alvaro <agomez@yepcode.io>
Date: Tue, 17 Mar 2026 10:07:46 +0100
Subject: [PATCH] Remove default MCP transport error handler

Let consuming projects decide how to handle transport errors by not
setting server.onerror by default. They can assign server.onerror
after instantiation if they need custom handling (logging, Sentry, etc).

Made-with: Cursor
---
 src/server.ts | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/src/server.ts b/src/server.ts
index bc70dd0..53450e3 100644
--- a/src/server.ts
+++ b/src/server.ts
@@ -180,7 +180,6 @@ class YepCodeMcpServer extends Server {
     this.runCodeCleanup = runCodeCleanup;
     this.skipCodingRules = skipCodingRules;
     this.setupHandlers();
-    this.setupErrorHandling();
 
     try {
       this.yepCodeRun = new YepCodeRun(config);
@@ -206,12 +205,6 @@ class YepCodeMcpServer extends Server {
     }
   }
 
-  private setupErrorHandling(): void {
-    this.onerror = (error) => {
-      this.logger.error("[MCP Error]", error);
-    };
-  }
-
   private setupHandlers(): void {
     this.setupToolHandlers();
     this.setupResourceHandlers();