add tool name filtering to mcp server implementation #1220
Conversation
| @@ -139,6 +145,9 @@ async def main(): | |||
| If you want to inherit the environment variables from the parent process, use `env=os.environ`. | |||
| """ | |||
|
|
|||
| allowed_tools: list[str] | None = None | |||
There was a problem hiding this comment.
No need to repeat this attribute definition on the subclasses
|
Would be fantastic to see this merged. Currently overriding the |
|
No further action needed here. I'll get to it at some point. |
|
@JohnUiterwyk Looks reasonable, can you please rebase on top of main? |
|
@JohnUiterwyk Looking at this and similar suggestions to modify MCP tool definitions like #1437, I'm thinking it would be better to implement this similar to the There's also #1474 which considers adding an agent-wide |
adds the ability to set a list of allowed tools when creating an MCPServer instance, such that list_tools and call_tool will be checked against the list of allowed tools if its set.
#1219