starred/servers-modelcontextprotocol-1
1
0
Fork 0
mirror of https://github.com/modelcontextprotocol/servers.git synced 2026-04-17 15:53:23 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3230 from nulone/fix/filesystem-move-file-destructive-hint

Browse Source 
fix(filesystem): mark move_file as destructive operation
This commit is contained in:
Ola Hungerford
2026-02-28 14:17:21 -07:00
committed by GitHub
parent 04cce79b4c 86fd903d5f
commit 72efc42976
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/filesystem/README.md
View File

@@ -200,7 +200,7 @@ The mapping for filesystem tools is:
| `create_directory` | `false` | `true` | `false` | Re‑creating the same dir is a no‑op |
| `write_file` | `false` | `true` | `true` | Overwrites existing files |
| `edit_file` | `false` | `false` | `true` | Re‑applying edits can fail or double‑apply |
| `move_file` | `false` | `false` | `false` | Move/rename only; repeat usually errors |
| `move_file` | `false` | `false` | `true` | Deletes source file |
> Note: `idempotentHint` and `destructiveHint` are meaningful only when `readOnlyHint` is `false`, as defined by the MCP spec.

2
src/filesystem/index.ts
View File

@@ -608,7 +608,7 @@ server.registerTool(
destination: z.string()
},
outputSchema: { content: z.string() },
annotations: { readOnlyHint: false, idempotentHint: false, destructiveHint: false }
annotations: { readOnlyHint: false, idempotentHint: false, destructiveHint: true }
},
async (args: z.infer<typeof MoveFileArgsSchema>) => {
const validSourcePath = await validatePath(args.source);