feat: logging fixes

Author: tomkosm

src/index.ts

@@ -19,6 +19,7 @@ import { StreamableHTTPServerTransport } from '@modelcontextprotocol/sdk/server/
 import express, { Request, Response } from 'express';
 import dotenv from 'dotenv';
 import { randomUUID } from 'node:crypto';
+import { safeLog } from './utils/log.js';
 
 dotenv.config();
 
@@ -933,29 +934,6 @@ function delay(ms: number): Promise<void> {
 
 let isStdioTransport = false;
 
-function safeLog(
-  level:
-    | 'error'
-    | 'debug'
-    | 'info'
-    | 'notice'
-    | 'warning'
-    | 'critical'
-    | 'alert'
-    | 'emergency',
-  data: any
-): void {
-  try {
-    // Always log to stderr to avoid relying on MCP logging capability
-    const message = `[${level}] ${
-      typeof data === 'object' ? JSON.stringify(data) : String(data)
-    }`;
-    console.error(message);
-  } catch (_) {
-    // ignore
-  }
-}
-
 // Add retry logic with exponential backoff
 async function withRetry<T>(
   operation: () => Promise<T>,

src/server-v1.ts

@@ -16,6 +16,7 @@ import FirecrawlApp, {
 
 // import express, { Request, Response } from 'express';
 import dotenv from 'dotenv';
+import { safeLog } from './utils/log.js';
 
 dotenv.config();
 
@@ -875,24 +876,7 @@ export function createV1Server() {
     return new Promise((resolve) => setTimeout(resolve, ms));
   }
 
-  function safeLog(
-    level:
-      | 'error'
-      | 'debug'
-      | 'info'
-      | 'notice'
-      | 'warning'
-      | 'critical'
-      | 'alert'
-      | 'emergency',
-    data: any
-  ): void {
-    // Always log to stderr to avoid relying on MCP logging capability
-    const message = `[V1][${level}] ${
-      typeof data === 'object' ? JSON.stringify(data) : String(data)
-    }`;
-    console.error(message);
-  }
+  // use shared safeLog via import (scope V1)
 
   // Add retry logic with exponential backoff
   async function withRetry<T>(

src/server-v2.ts

@@ -15,6 +15,7 @@ import FirecrawlApp, {
 
 // import express, { Request, Response } from 'express';
 import dotenv from 'dotenv';
+import { safeLog } from './utils/log.js';
 
 dotenv.config();
 
@@ -869,24 +870,7 @@ export function createV2Server() {
     return new Promise((resolve) => setTimeout(resolve, ms));
   }
 
-  function safeLog(
-    level:
-      | 'error'
-      | 'debug'
-      | 'info'
-      | 'notice'
-      | 'warning'
-      | 'critical'
-      | 'alert'
-      | 'emergency',
-    data: any
-  ): void {
-    // Always log to stderr to avoid relying on MCP logging capability
-    const message = `[V2][${level}] ${
-      typeof data === 'object' ? JSON.stringify(data) : String(data)
-    }`;
-    console.error(message);
-  }
+  // use shared safeLog via import (scope V2 if desired)
 
   // Add retry logic with exponential backoff
   async function withRetry<T>(

src/utils/log.ts

@@ -0,0 +1,32 @@
+export type LogLevel =
+  | 'error'
+  | 'debug'
+  | 'info'
+  | 'notice'
+  | 'warning'
+  | 'critical'
+  | 'alert'
+  | 'emergency';
+
+export function safeLog(level: LogLevel, data: any, scope?: string): void {
+  try {
+    const prefix = scope ? `[${scope}]` : '';
+    const message = `${prefix}[${level}] ${
+      typeof data === 'object' ? JSON.stringify(data) : String(data)
+    }`;
+    if (level === 'warning') {
+      console.warn(message);
+    } else if (
+      level === 'error' ||
+      level === 'critical' ||
+      level === 'alert' ||
+      level === 'emergency'
+    ) {
+      console.error(message);
+    } else {
+      console.log(message);
+    }
+  } catch (_) {
+    // ignore logging failures
+  }
+}