@makeshkumar/mcp-xl-reader 1.0.1 → 1.0.2

package/dist/index.js

@@ -133,13 +133,12 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             },
             {
                 name: "find_excel_files",
-                description: "Finds all .xlsx files in a given directory.",
+                description: "Finds all .xlsx files in a given directory. If directoryPath is omitted, it automatically searches the allowed directories or current working directory.",
                 inputSchema: {
                     type: "object",
                     properties: {
-                        directoryPath: { type: "string", description: "Absolute path to the directory to search." }
-                    },
-                    required: ["directoryPath"]
+                        directoryPath: { type: "string", description: "Absolute path to the directory to search (optional)." }
+                    }
                 }
             },
             {
@@ -195,15 +194,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             content: [{ type: "text", text: "Invalid arguments provided." }]
         };
     }
-    if (name === "find_excel_files") {
-        if (!args.directoryPath) {
-            return {
-                isError: true,
-                content: [{ type: "text", text: "Missing required directoryPath parameter." }]
-            };
-        }
-    }
-    else {
+    if (name !== "find_excel_files") {
         if (!args.filePath) {
             return {
                 isError: true,
@@ -211,12 +202,9 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             };
         }
     }
-    let absolutePath;
+    let absolutePath = "";
     try {
-        if (name === "find_excel_files") {
-            absolutePath = await validateDirectoryPath(args.directoryPath);
-        }
-        else {
+        if (name !== "find_excel_files") {
             absolutePath = await validateFilePath(args.filePath);
         }
     }
@@ -230,18 +218,52 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         const workbook = new ExcelJS.Workbook();
         switch (name) {
             case "find_excel_files": {
-                // Read all files recursively in the given directory
-                const files = await fs.readdir(absolutePath, { recursive: true, withFileTypes: true });
-                const excelFiles = files
-                    .filter(dirent => dirent.isFile() && dirent.name.toLowerCase().endsWith('.xlsx'))
-                    // fs.readdir with recursive=true returns relative paths in dirent.name or dirent.parentPath
-                    .map(dirent => {
-                    const d = dirent;
-                    const parentPath = d.parentPath || d.path || absolutePath;
-                    return path.resolve(parentPath, dirent.name);
-                });
+                let directoriesToSearch = [];
+                if (args && args.directoryPath) {
+                    directoriesToSearch.push(await validateDirectoryPath(args.directoryPath));
+                }
+                else {
+                    const allowedDirectories = process.env.ALLOWED_DIRECTORIES ? process.env.ALLOWED_DIRECTORIES.split(',') : [];
+                    if (allowedDirectories.length > 0) {
+                        for (const dir of allowedDirectories) {
+                            try {
+                                directoriesToSearch.push(await validateDirectoryPath(dir.trim()));
+                            }
+                            catch (e) {
+                                // ignore invalid ones
+                            }
+                        }
+                    }
+                    else {
+                        // fallback to current working directory
+                        directoriesToSearch.push(process.cwd());
+                    }
+                }
+                if (directoriesToSearch.length === 0) {
+                    return {
+                        isError: true,
+                        content: [{ type: "text", text: "No valid directories to search." }]
+                    };
+                }
+                const allExcelFiles = [];
+                for (const dir of directoriesToSearch) {
+                    try {
+                        const files = await fs.readdir(dir, { recursive: true, withFileTypes: true });
+                        const excelFiles = files
+                            .filter(dirent => dirent.isFile() && dirent.name.toLowerCase().endsWith('.xlsx'))
+                            .map(dirent => {
+                            const d = dirent;
+                            const parentPath = d.parentPath || d.path || dir;
+                            return path.resolve(parentPath, dirent.name);
+                        });
+                        allExcelFiles.push(...excelFiles);
+                    }
+                    catch (e) {
+                        // Skip if one directory fails
+                    }
+                }
                 return {
-                    content: [{ type: "text", text: JSON.stringify(excelFiles, null, 2) }]
+                    content: [{ type: "text", text: JSON.stringify(allExcelFiles, null, 2) }]
                 };
             }
             case "list_sheets": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@makeshkumar/mcp-xl-reader",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "main": "dist/index.js",
   "type": "module",
   "scripts": {