You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The MCP protocol allows for tool collection to change over time (see the specs here).
With the current state of this project users will have to override the behaviour of the ListToolsHandler and also write code to keep the list of tools consistent. This will introduce a lot of overhead when the goal is to allow for some tool to be removed from the capability list. It would be also required to write code to notify clients about the tool availability changes.
The message flow to implement will be up to the tool developer.
The McpServer could implement the message flow and listing functionality in a consistent way for all tool implementation.
A developer would need to use this type only when requiring advanced functionalities for tool availability changes, in any other cases the current offer via custom attributes would be fine.
The text was updated successfully, but these errors were encountered:
Originally posted by @colombod in PederHP/mcpdotnet#100.
The MCP protocol allows for tool collection to change over time (see the specs here).
With the current state of this project users will have to override the behaviour of the
ListToolsHandler
and also write code to keep the list of tools consistent. This will introduce a lot of overhead when the goal is to allow for some tool to be removed from the capability list. It would be also required to write code to notify clients about the tool availability changes.The message flow to implement will be up to the tool developer.
Using a more specific type
The McpServer could implement the message flow and listing functionality in a consistent way for all tool implementation.
A developer would need to use this type only when requiring advanced functionalities for tool availability changes, in any other cases the current offer via custom attributes would be fine.
The text was updated successfully, but these errors were encountered: