@imenam/xls-reader 2.4.1 → 2.4.3

package/README.md

@@ -144,7 +144,7 @@ Or point to a local build:
 
 ## Configuration
 
-No environment variables are required. Optionally, set `MCP_LOG_DIR` to change where logs are written (default: `C:\var\log\xls-reader` on Windows, `/var/log/xls-reader` otherwise).
+No environment variables are required. Optionally, set `MCP_LOG_DIR` to change where logs are written (default: `.xls-reader/logs` in the home directory of the user running the server). When the log file cannot be written, each log line is reported on stderr as `Critical: Failed to write to log file <path>: <reason>` and the server keeps running.
 
 ```env
 # .env

package/dist/index.js

@@ -10,7 +10,7 @@ import { listSheets, readSheet, readWorkbook } from "./excel.js";
 import { deliverFile, RELAY_SHAPE_DESCRIPTIONS } from "./relay.js";
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const rootDir = path.resolve(__dirname, "..");
-dotenv.config({ path: path.join(rootDir, ".env"), override: true });
+dotenv.config({ path: path.join(rootDir, ".env"), override: true, quiet: true });
 setupLogging("MASTER");
 const server = new McpServer({
     name: "xls-reader",
@@ -107,9 +107,6 @@ server.tool("get_sheet", [
     }
 });
 // ─── export_workbook ──────────────────────────────────────────────────────────
-// L'export est un outil à part, et non un `format` de get_sheet : le relais rend
-// obligatoire tout `<x>_path` dont il voit la paire, donc porter `destination_path` dans
-// get_sheet forcerait une destination sur chaque simple lecture de feuille.
 server.tool("export_workbook", [
     "Export an Excel workbook as a JSON file written on the agent machine, with no size limit.",
     "Exports the whole workbook, or only sheet_name when provided.",

package/dist/logger.js

@@ -1,11 +1,9 @@
 import fs from "fs";
+import os from "os";
 import path from "path";
 let currentPrefix = "";
 export function logToFile(message, prefix = "") {
-    const defaultLogDir = process.platform === "win32"
-        ? "C:\\var\\log\\xls-reader"
-        : "/var/log/xls-reader";
-    const logDir = process.env.MCP_LOG_DIR ?? defaultLogDir;
+    const logDir = process.env.MCP_LOG_DIR ?? path.join(os.homedir(), ".xls-reader", "logs");
     const logFile = path.join(logDir, "server.log");
     const now = new Date().toLocaleString("fr-FR", {
         year: "numeric",
@@ -24,8 +22,9 @@ export function logToFile(message, prefix = "") {
         }
         fs.appendFileSync(logFile, logEntry);
     }
-    catch {
-        process.stderr.write("Critical: Failed to write to log file\n");
+    catch (error) {
+        const reason = error instanceof Error ? error.message : String(error);
+        process.stderr.write(`Critical: Failed to write to log file ${logFile}: ${reason}\n`);
     }
 }
 export function setupLogging(prefix) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imenam/xls-reader",
-  "version": "2.4.1",
+  "version": "2.4.3",
   "description": "MCP server that lets AI agents read Excel files: list sheets and render any sheet as a Markdown table.",
   "type": "module",
   "main": "dist/index.js",