fix(mcp): register mint and pull under their documented tool names - #81
Open
RobinOppenstam wants to merge 1 commit into
Open
fix(mcp): register mint and pull under their documented tool names#81RobinOppenstam wants to merge 1 commit into
RobinOppenstam wants to merge 1 commit into
Conversation
FastMCP derives the tool name from the function's __name__, so the conditional registrations exposed _sema_mint/_sema_pull while install.md and the server instructions document sema_mint/sema_pull. Clients calling the documented names get 'Unknown tool'. Register with explicit names. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Register the conditionally-exposed mint/pull MCP tools with explicit public names, so they appear as
sema_mint/sema_pullinstead of_sema_mint/_sema_pull.Why
FastMCP derives a tool's name from the wrapped function's
__name__. The conditional registrations wrap the underscore-prefixed functions directly, so the server currently advertises_sema_mint/_sema_pull— while install.md, the server's own workflow instructions, and every published client snippet documentsema_mint/sema_pull. Clients calling the documented names getUnknown tool. Found while driving the server end-to-end over stdio.How to test
pytest— full suite passes (409 passed locally on 3.12).tools/list— the tool names includesema_mintandsema_pull, with no underscore-prefixed leftovers, andtools/callonsema_mintworks.🤖 Generated with Claude Code