Conversation
Signed-off-by: pnkcaht <samzoovsk19@gmail.com>
|
Basically, it was just an added comment. Depending on what it is, I might do more about the issue mentioned, but I don't think there's much more I can do. |
|
@Pnkcaht Claude Code recently started processing the listChanged tool notification. Are you using any clients that are not refreshing the tool list today? I think this PR is a good idea - I'm just wondering which clients still rely on mcp-exec/client_restart. |
@slimslenderslacks I understand your question and thank you for asking. As far as I know, all currently active clients process the listChanged notification automatically, therefore none of our current clients depend on mcp-exec/client_restart. This pull request is primarily aimed at ensuring the safety of legacy or custom clients that may not update the list of tools automatically. It ensures that the message to the user correctly informs them about the need to restart the MCP_DOCKER connection, if necessary, without affecting modern clients :) |
What I did
Updated the
mcp-addtool response to avoid implying immediate tool availability after adding a server.The response now reflects the actual behavior of MCP clients that do not automatically refresh the tool list, and clearly communicates how newly added tools can be accessed in the current session.
Related issue
Fixes #316
What was the problem?
The
mcp-addtool reported success in a way that implied the server and its tools were immediately available to the client.In practice:
/mcpor tool listings may not update until the client reconnectsHow this change fixes it
The success message was updated to be explicit and accurate:
mcp-execin the current sessionThis keeps server behavior unchanged while aligning user-facing output with real client behavior.
Before / After
Before
After