-
Notifications
You must be signed in to change notification settings - Fork 77
feat: add tool selection by feature param #172
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
Conversation
jirispilka
left a comment
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.
Looks good to me. The only change I would do is rename things and remove the beta flag. What do you think?
…gory, remove beta flag param
Agree, renamed and moved the beta tools to |
MichalKalita
left a comment
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.
👍
closes #166
Only
docstools category is currently enabled by default (always)This PR introduces the
tools(--toolsor?tools) comma separated list param so users can choose which tools are loaded into their MCP session.Possible values:
Example:
Also vibe refactored the tool loading logic so it's not duplicated in the
stdio.ts