From 798f62f63a61b7580ef408ccf404d1427781111b Mon Sep 17 00:00:00 2001 From: "Maxime Roy (new.blacc)" Date: Mon, 6 Apr 2026 10:02:27 +0200 Subject: [PATCH] fix(logger): make file transport opt-in via LOG_FILE_PATH (#9) The file transport defaulted to writing excalidraw.log relative to process.cwd(), polluting every project directory where the MCP server started. Console transport already handles warn+error to stderr. File logging is now opt-in: set LOG_FILE_PATH to enable it. Co-authored-by: Claude Opus 4.6 (1M context) --- src/utils/logger.ts | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/utils/logger.ts b/src/utils/logger.ts index a971d73..67d55b2 100644 --- a/src/utils/logger.ts +++ b/src/utils/logger.ts @@ -1,7 +1,5 @@ import winston from 'winston'; -const LOG_FILE_PATH = process.env.LOG_FILE_PATH || 'excalidraw.log'; - const logger: winston.Logger = winston.createLogger({ level: process.env.LOG_LEVEL || 'info', @@ -21,13 +19,15 @@ const logger: winston.Logger = winston.createLogger({ new winston.transports.Console({ level: 'warn', // only warn+error to stderr stderrLevels: ['warn','error'] - }), - - new winston.transports.File({ - filename: LOG_FILE_PATH, // all levels to file - level: 'debug' }) ] }); +if (process.env.LOG_FILE_PATH) { + logger.add(new winston.transports.File({ + filename: process.env.LOG_FILE_PATH, + level: 'debug' + })); +} + export default logger; \ No newline at end of file