@imenam/xls-reader 2.4.0 → 2.4.3

package/README.md

@@ -100,7 +100,7 @@ Exports a workbook as a JSON file, with **no size limit**, and writes it **on th
 | `destination_overwrite` | `boolean` | ❌ | Replace the destination file if it exists (default: `false`) |
 | `destination_relay` | `boolean` | ❌ | Filled in by the relay, never by the agent |
 
-The exported file never passes through the conversation. When this MCP runs next to the agent (stdio), it writes the file itself. Behind [`@imenam/mcp-http-gateway`](https://www.npmjs.com/package/@imenam/mcp-http-gateway) it runs on the gateway's machine and cannot reach the agent's disk: it then returns the bytes, and the relay — which *does* run on the agent machine — writes them at `destination_path`. That path requires `GATEWAY_INLINE_WRITE_ROOTS` in the relay's `env` block; without it the call fails explicitly rather than dropping the file on the wrong machine.
+The exported file never passes through the conversation. When this MCP runs next to the agent (stdio), it writes the file itself. Behind [`@imenam/mcp-http-gateway`](https://www.npmjs.com/package/@imenam/mcp-http-gateway) it runs on the gateway's machine and cannot reach the agent's disk: it then returns the bytes, and the relay — which *does* run on the agent machine — writes them at `destination_path`. The relay keeps that path to itself and never forwards it, which is why the tool schema declares it optional; the tool rejects a call that lacks it when it has to write the file itself. That path requires `GATEWAY_INLINE_WRITE_ROOTS` in the relay's `env` block; without it the call fails explicitly rather than dropping the file on the wrong machine.
 
 The file content is:
 
@@ -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.",
@@ -125,7 +122,12 @@ server.tool("export_workbook", [
         .boolean()
         .optional()
         .describe("Treat the first row as a column header (default: true)"),
-    destination_path: z.string().describe(RELAY_SHAPE_DESCRIPTIONS.destination_path),
+    // Optionnel au niveau du schéma : le relais retient ce chemin et ne le transmet pas.
+    // deliverFile l'exige quand l'écriture a lieu ici.
+    destination_path: z
+        .string()
+        .optional()
+        .describe(`${RELAY_SHAPE_DESCRIPTIONS.destination_path}. Required`),
     filename: z.string().describe(RELAY_SHAPE_DESCRIPTIONS.filename),
     destination_overwrite: z
         .boolean()

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/dist/relay.js

@@ -50,6 +50,9 @@ export function deliverFile(data, mimeType, target) {
         ];
     }
     // Pas de relais : le MCP tourne à côté de l'agent, son disque est le bon.
+    if (!target.destination_path) {
+        throw new Error("destination_path is required: the absolute directory where the file is written.");
+    }
     if (!path.isAbsolute(target.destination_path)) {
         throw new Error(`destination_path must be an absolute directory path: ${target.destination_path}`);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imenam/xls-reader",
-  "version": "2.4.0",
+  "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",