Skip to content

Commit 43829a3

Browse files
committed
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
1 parent ba15eb1 commit 43829a3

1 file changed

Lines changed: 0 additions & 7 deletions

File tree

src/server.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,6 @@ class YepCodeMcpServer extends Server {
180180
this.runCodeCleanup = runCodeCleanup;
181181
this.skipCodingRules = skipCodingRules;
182182
this.setupHandlers();
183-
this.setupErrorHandling();
184183

185184
try {
186185
this.yepCodeRun = new YepCodeRun(config);
@@ -206,12 +205,6 @@ class YepCodeMcpServer extends Server {
206205
}
207206
}
208207

209-
private setupErrorHandling(): void {
210-
this.onerror = (error) => {
211-
this.logger.error("[MCP Error]", error);
212-
};
213-
}
214-
215208
private setupHandlers(): void {
216209
this.setupToolHandlers();
217210
this.setupResourceHandlers();

0 commit comments

Comments
 (0)