Skip to content

filesystem: add descriptions to all tool parameters#4547

Open
m9ter1a wants to merge 1 commit into
modelcontextprotocol:mainfrom
m9ter1a:filesystem-param-descriptions
Open

filesystem: add descriptions to all tool parameters#4547
m9ter1a wants to merge 1 commit into
modelcontextprotocol:mainfrom
m9ter1a:filesystem-param-descriptions

Conversation

@m9ter1a

@m9ter1a m9ter1a commented Jul 22, 2026

Copy link
Copy Markdown

Several tools in the filesystem server expose parameters — path, content, source, destination, pattern, excludePatterns — with no description. When an agent fills these in it has only the parameter name to go on; path, for instance, doesn't convey that it must resolve inside one of the server's allowed directories.

This PR adds a short .describe(...) to each previously-undescribed parameter (18 in total). It's a metadata-only change:

  • no behavior change; all 152 existing tests pass (vitest run)
  • descriptions are brief and factual — path fields note the allowed-directory constraint, and so on

I noticed the gap while auditing MCP servers' tool schemas for agent-friendliness with mcpcheck.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant