@j0hanz/filesystem-mcp 1.1.1 → 1.1.2

package/README.md

@@ -137,6 +137,15 @@ The server communicates via `stdio`. Ensure your MCP client is configured to run
 | `apply_patch`        | Apply unified patch                 | `path`, `patch`                               |
 | `search_and_replace` | Search & replace across files       | `filePattern`, `searchPattern`, `replacement` |
 
+### Behavioral Notes
+
+- `rm` with `recursive: false`:
+  - Deletes files and empty directories.
+  - Returns `E_INVALID_INPUT` for non-empty directories with guidance to use `recursive: true`.
+- `includeIgnored: false` (default) for navigation/search tools:
+  - Excludes common generated/vendor directories such as `node_modules`, `dist`, `.git`, and similar patterns.
+  - Set `includeIgnored: true` to include those entries.
+
 ### Resources
 
 | URI Pattern                    | Description                                      |
@@ -252,6 +261,12 @@ args = ["-y", "@j0hanz/filesystem-mcp@latest", "${workspaceFolder}"]
 - **Path Restrictions**: All file operations are strictly validated against the allowed root directories provided at startup.
 - **Path Validation**: Uses `isPathWithinDirectories` to prevent path traversal attacks.
 - **Hidden Files**: Hidden files (starting with `.`) are excluded by default in listings and searches unless explicitly requested.
+- **Ignored Directories**: Ignored directories (for example `node_modules`, `.git`, `dist`) are excluded by default unless `includeIgnored=true`.
+
+## Testing Notes
+
+- For protocol-level validation, prefer an MCP SDK client (`listTools`, `listResources`, `listPrompts`, `callTool`, `readResource`) as source-of-truth.
+- Some third-party MCP CLIs may have URI parsing limitations when reading resources; if this happens, verify resource behavior through SDK client calls.
 
 ## Development Workflow
 

package/dist/lib/constants.js

@@ -146,24 +146,42 @@ export const KNOWN_BINARY_EXTENSIONS = new Set([
     '.dat',
 ]);
 export const DEFAULT_EXCLUDE_PATTERNS = [
+    '**/node_modules',
     '**/node_modules/**',
+    '**/dist',
     '**/dist/**',
+    '**/build',
     '**/build/**',
+    '**/coverage',
     '**/coverage/**',
+    '**/.git',
     '**/.git/**',
+    '**/.vscode',
     '**/.vscode/**',
+    '**/.idea',
     '**/.idea/**',
     '**/.DS_Store',
+    '**/.next',
     '**/.next/**',
+    '**/.nuxt',
     '**/.nuxt/**',
+    '**/.output',
     '**/.output/**',
+    '**/.svelte-kit',
     '**/.svelte-kit/**',
+    '**/.cache',
     '**/.cache/**',
+    '**/.yarn',
     '**/.yarn/**',
+    '**/jspm_packages',
     '**/jspm_packages/**',
+    '**/bower_components',
     '**/bower_components/**',
+    '**/out',
     '**/out/**',
+    '**/tmp',
     '**/tmp/**',
+    '**/.temp',
     '**/.temp/**',
     '**/npm-debug.log',
     '**/yarn-debug.log',

package/dist/tools/apply-patch.js

@@ -45,7 +45,7 @@ async function handleApplyPatch(args, signal) {
         autoConvertLineEndings: args.autoConvertLineEndings,
     });
     if (patched === false) {
-        throw new McpError(ErrorCode.E_INVALID_INPUT, `Patch application failed. The file content may have changed or the patch context is insufficient. Try enable fuzzy matching.`);
+        throw new McpError(ErrorCode.E_INVALID_INPUT, 'Patch application failed. The file content may have changed or patch context is insufficient. Generate a fresh patch via diff_files against the current file, then retry. If differences are minor, enable fuzzy matching (fuzzy=true or fuzzFactor).');
     }
     if (args.dryRun) {
         return buildToolResponse('Dry run successful. Patch can be applied.', {

package/dist/tools/delete-file.js

@@ -19,10 +19,32 @@ const DELETE_FILE_TOOL = {
 };
 async function handleDeleteFile(args, signal) {
     const validPath = await validatePathForWrite(args.path, signal);
-    await withAbort(fs.rm(validPath, {
-        recursive: args.recursive,
-        force: args.ignoreIfNotExists,
-    }), signal);
+    let stats;
+    try {
+        stats = await withAbort(fs.lstat(validPath), signal);
+    }
+    catch (error) {
+        if (isNodeError(error) &&
+            error.code === 'ENOENT' &&
+            args.ignoreIfNotExists) {
+            return buildToolResponse(`Successfully deleted: ${args.path}`, {
+                ok: true,
+                path: validPath,
+            });
+        }
+        throw error;
+    }
+    if (stats.isDirectory() && !args.recursive) {
+        // Use rmdir for non-recursive directory deletes so non-empty directories
+        // consistently return ENOTEMPTY-style errors with actionable guidance.
+        await withAbort(fs.rmdir(validPath), signal);
+    }
+    else {
+        await withAbort(fs.rm(validPath, {
+            recursive: args.recursive,
+            force: args.ignoreIfNotExists,
+        }), signal);
+    }
     return buildToolResponse(`Successfully deleted: ${args.path}`, {
         ok: true,
         path: validPath,
@@ -45,6 +67,12 @@ export function registerDeleteFileTool(server, options = {}) {
             if (error.code === 'ENOTEMPTY') {
                 return buildToolErrorResponse(new Error(`Directory is not empty: ${args.path}. Use recursive: true to delete non-empty directories.`), ErrorCode.E_INVALID_INPUT, args.path);
             }
+            if (error.code === 'EISDIR') {
+                return buildToolErrorResponse(new Error(`Path is a directory: ${args.path}. Use recursive: true to delete directories.`), ErrorCode.E_INVALID_INPUT, args.path);
+            }
+            if (error.code === 'EEXIST') {
+                return buildToolErrorResponse(new Error(`Directory is not empty: ${args.path}. Use recursive: true to delete non-empty directories.`), ErrorCode.E_INVALID_INPUT, args.path);
+            }
             if (error.code === 'EPERM' || error.code === 'EACCES') {
                 return buildToolErrorResponse(error, ErrorCode.E_PERMISSION_DENIED, args.path);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/filesystem-mcp",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "mcpName": "io.github.j0hanz/filesystem-mcp",
   "description": "MCP Server that enables LLMs to interact with the local filesystem.",
   "type": "module",