crawlforge-mcp-server 4.2.6 → 4.2.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "crawlforge-mcp-server",
-  "version": "4.2.6",
+  "version": "4.2.8",
   "description": "CrawlForge MCP Server - Professional Model Context Protocol server with 23 web scraping, crawling, and content processing tools. Defaults to local Ollama for LLM extraction (no API key needed); OpenAI/Anthropic available as opt-in. v4.0 adds Markdown-first output, pre-built site templates, Camoufox stealth engine, and cost transparency.",
   "main": "server.js",
   "bin": {

package/server.js

@@ -1163,12 +1163,18 @@ async function gracefulShutdown(signal) {
 process.on('SIGINT', () => gracefulShutdown('SIGINT'));
 process.on('SIGTERM', () => gracefulShutdown('SIGTERM'));
 process.on('uncaughtException', (error) => {
+  // Keep the long-running stdio server alive: a single uncaught error in one
+  // request path should not tear down the session for every other tool. We log
+  // and continue rather than exiting. (Node considers the process state
+  // technically undefined after this; acceptable trade-off for a resilient MCP
+  // server, vs. disconnecting the client on any stray throw.)
   console.error('Uncaught Exception:', error);
-  gracefulShutdown('uncaughtException');
 });
 process.on('unhandledRejection', (reason, promise) => {
+  // A stray rejection — typically background async work inside a single tool —
+  // must NOT terminate the whole stdio MCP server, which would disconnect every
+  // other tool mid-session. Log it and keep serving.
   console.error('Unhandled Rejection at:', promise, 'reason:', reason);
-  gracefulShutdown('unhandledRejection');
 });
 
 // Memory monitoring (development only)

package/src/utils/Logger.js

@@ -391,8 +391,9 @@ export class Logger {
 
     this.winston.error(message, errorContext);
 
-    // Track error for analysis
-    if (this.enableErrorTracking) {
+    // Track error for analysis (only when an Error object was actually passed —
+    // logger.error(message) with no error must not reach trackError's error.name).
+    if (this.enableErrorTracking && error) {
       this.trackError(error, context, requestId);
     }
   }
@@ -425,11 +426,13 @@ export class Logger {
   trackError(error, context, requestId) {
     // Could be extended to send to error tracking service
     // For now, just log structured error data
+    // Null-safe: a shared logger must never throw, even if called with a
+    // non-Error (or null) value.
     this.winston.error('Error tracking', {
       errorTracking: {
-        type: error.name,
-        message: error.message,
-        stack: error.stack,
+        type: error?.name ?? 'UnknownError',
+        message: error?.message ?? String(error ?? ''),
+        stack: error?.stack,
         context,
         requestId,
         timestamp: new Date().toISOString()