@imenam/xls-reader 2.4.0 → 2.4.1

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:
 

package/dist/index.js

@@ -125,7 +125,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/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.1",
   "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",