@makeshkumar/mcp-xl-reader 1.0.0 → 1.0.2

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,16 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     required: ["filePath"]
                 }
             },
+            {
+                name: "find_excel_files",
+                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 (optional)." }
+                    }
+                }
+            },
             {
                 name: "read_spreadsheet",
                 description: "Returns JSON representation of rows. Streams using worksheet.eachRow for memory efficiency.",
@@ -151,16 +188,25 @@ 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;
-    let absolutePath;
+    if (name !== "find_excel_files") {
+        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 validateFilePath(args.filePath);
+        }
     }
     catch (error) {
         return {
@@ -171,6 +217,55 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     try {
         const workbook = new ExcelJS.Workbook();
         switch (name) {
+            case "find_excel_files": {
+                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(allExcelFiles, 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.2",
   "main": "dist/index.js",
   "type": "module",
   "scripts": {