fix: prevent server shutdown on uncaught errors

CRITICAL FIX: Uncaught exceptions and unhandled promise rejections
no longer kill the entire server. This was causing catastrophic
failures where a single tool error would shut down the server.

Changes:
- uncaughtException handler: LOG only, do not call gracefulShutdown()
- unhandledRejection handler: LOG only, do not call gracefulShutdown()
- Added comments explaining that tool errors are already caught
- Version bump to 1.2.3

WHY THIS IS CRITICAL:
A production server should NEVER crash because of a single tool error.
Tool errors are caught in the try/catch at line 208, but if any
promise rejection escapes that catch, the old code would kill the
entire server and all active sessions.

NOW:
- Server logs the error and continues running
- Only SIGTERM/SIGINT trigger graceful shutdown
- Individual tool errors return error responses, not crash the server

This is essential for production stability.

Author: PhilflowIO

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsdav-mcp-server",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "Model Context Protocol (MCP) server for CalDAV/CardDAV/VTODO integration via tsdav - enables AI systems to manage calendars, contacts, and tasks",
   "type": "module",
   "main": "src/index.js",

src/index.js

@@ -426,15 +426,17 @@ async function gracefulShutdown(signal) {
 process.on('SIGTERM', () => gracefulShutdown('SIGTERM'));
 process.on('SIGINT', () => gracefulShutdown('SIGINT'));
 
-// Handle uncaught errors
+// Handle uncaught errors - LOG but DO NOT kill the server
+// Tool errors are caught in the tool handler try/catch (line 208)
+// These handlers are only for truly unexpected errors that escape our error handling
 process.on('uncaughtException', (error) => {
-  logger.error({ error: error.message, stack: error.stack }, 'Uncaught exception');
-  gracefulShutdown('uncaughtException');
+  logger.error({ error: error.message, stack: error.stack }, 'Uncaught exception - server continuing');
+  // DO NOT call gracefulShutdown() - let the server continue running
 });
 
 process.on('unhandledRejection', (reason, promise) => {
-  logger.error({ reason, promise }, 'Unhandled promise rejection');
-  gracefulShutdown('unhandledRejection');
+  logger.error({ reason, promise }, 'Unhandled promise rejection - server continuing');
+  // DO NOT call gracefulShutdown() - let the server continue running
 });
 
 let server;