-
Notifications
You must be signed in to change notification settings - Fork 761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Filesystem Server #16
Conversation
Looks good to me! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great. We fixed the major issues.
{ | ||
name: "list_allowed_directories", | ||
description: | ||
"Returns the list of directories that this server is allowed to access. " + | ||
"Use this to understand which directories are available before trying to access files.", | ||
inputSchema: { | ||
type: "object", | ||
properties: {}, | ||
required: [], | ||
}, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIOLI: This might be a useful demonstration of a prompt provided by MCP, rather than a tool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense! I think that would be ideal if Claude were able to pull in prompts & resources automatically. This tool is primarily for the LLM to self-correct by identifying which directories are available
…mahesh/add-filesystem Add Filesystem Server
Compared to last time, this includes Zod + only allows operations on allowlisted folders.
Also includes a tool for Claude to know which folders are allowlisted. This tends to help its performance.