-
Notifications
You must be signed in to change notification settings - Fork 2
Add docs for new thv run --tools-override flag.
#179
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
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Pull Request Overview
This change adds documentation for the new --tools-override flag in the ToolHive CLI, which allows users to modify tool names and descriptions when running MCP servers. This helps users customize how tools are exposed to clients, which can be useful for network policy compliance or to influence agent behavior.
- Adds a new "Override tool names and descriptions" section explaining the feature
- Provides a concrete JSON configuration example using the
fetchMCP server - Includes important guidance on combining tool filtering with tool overrides
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.
Thanks @blkt. I ran it through Claude for a review and it had some good suggestions for clarity, I also had some ideas I applied; it seems Copilot has over-rotated on its instructions and is only flagging simple typos now in its reviews 🤦
|
@danbarr thank you for your thorough review, I should have addressed all comments. |
f9e15c0 to
1333313
Compare
| "Fetches a URL from the internet and optionally extracts its contents as markdown." | ||
| ``` | ||
|
|
||
| You can now change it by supplying a file like the following |
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.
I forgot I had a question about this, I assume the format is one entry under toolsOverride per tool? If so:
| You can now change it by supplying a file like the following | |
| To override this, create a configuration file with one entry under | |
| `toolsOverride` for each tool you want to modify: |
This change adds a small section under the "Run MCP servers" page documenting how to override filter names and descriptions. Fixes #127
1333313 to
7e8e80d
Compare
danbarr
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.
Thanks @blkt, let's merge after the release happens.
Description
This change adds a small section under the "Run MCP servers" page documenting how to override filter names and descriptions.
Fixes #127
Related issues/PRs
stacklok/toolhive#1841
Merge checklist
Reviews