@makeshkumar/mcp-xl-reader 1.0.0 → 1.0.1

package/dist/index.js

@@ -42,6 +42,33 @@ async function validateFilePath(filePath) {
     }
     return absolutePath;
 }
+// Helper for security and directory existence check
+async function validateDirectoryPath(directoryPath) {
+    const absolutePath = path.resolve(directoryPath);
+    try {
+        const stats = await fs.stat(absolutePath);
+        if (!stats.isDirectory()) {
+            throw new Error(`Path exists but is not a directory: ${absolutePath}`);
+        }
+    }
+    catch (error) {
+        if (error.code === 'ENOENT') {
+            throw new Error(`Directory Not Found: ${absolutePath}`);
+        }
+        else if (error.code === 'EACCES') {
+            throw new Error(`Permission Denied: ${absolutePath}`);
+        }
+        throw error;
+    }
+    const allowedDirectories = process.env.ALLOWED_DIRECTORIES ? process.env.ALLOWED_DIRECTORIES.split(',') : [];
+    if (allowedDirectories.length > 0) {
+        const isAllowed = allowedDirectories.some(dir => absolutePath.startsWith(path.resolve(dir)));
+        if (!isAllowed) {
+            throw new Error(`Security Violation: Access to ${absolutePath} is not within allowed directories.`);
+        }
+    }
+    return absolutePath;
+}
 // Helper to format cell values properly
 function formatCellValue(cell) {
     if (cell.value === null || cell.value === undefined) {
@@ -104,6 +131,17 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     required: ["filePath"]
                 }
             },
+            {
+                name: "find_excel_files",
+                description: "Finds all .xlsx files in a given directory.",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        directoryPath: { type: "string", description: "Absolute path to the directory to search." }
+                    },
+                    required: ["directoryPath"]
+                }
+            },
             {
                 name: "read_spreadsheet",
                 description: "Returns JSON representation of rows. Streams using worksheet.eachRow for memory efficiency.",
@@ -151,16 +189,36 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
 });
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
     const { name, arguments: args } = request.params;
-    if (!args || typeof args !== 'object' || !args.filePath) {
+    if (!args || typeof args !== 'object') {
         return {
             isError: true,
-            content: [{ type: "text", text: "Missing required filePath parameter." }]
+            content: [{ type: "text", text: "Invalid arguments provided." }]
         };
     }
-    const { filePath } = args;
+    if (name === "find_excel_files") {
+        if (!args.directoryPath) {
+            return {
+                isError: true,
+                content: [{ type: "text", text: "Missing required directoryPath parameter." }]
+            };
+        }
+    }
+    else {
+        if (!args.filePath) {
+            return {
+                isError: true,
+                content: [{ type: "text", text: "Missing required filePath parameter." }]
+            };
+        }
+    }
     let absolutePath;
     try {
-        absolutePath = await validateFilePath(filePath);
+        if (name === "find_excel_files") {
+            absolutePath = await validateDirectoryPath(args.directoryPath);
+        }
+        else {
+            absolutePath = await validateFilePath(args.filePath);
+        }
     }
     catch (error) {
         return {
@@ -171,6 +229,21 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     try {
         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);
+                });
+                return {
+                    content: [{ type: "text", text: JSON.stringify(excelFiles, null, 2) }]
+                };
+            }
             case "list_sheets": {
                 await workbook.xlsx.readFile(absolutePath);
                 const sheets = workbook.worksheets.map(ws => ws.name);

package/package.json

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