feat(time): add time_ prefix to tool names for consistency #3274
+6
−6
Conversation
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
Adds 'time_' prefix to the Time MCP server tool names: - get_current_time → time_get_current - convert_time → time_convert This follows the naming convention used by other MCP servers (e.g., git_status, git_commit) and makes it easier to identify related tools when multiple MCP servers are registered. Closes modelcontextprotocol#3201
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.
Summary
Adds a
time_prefix to the Time MCP server tool names for consistency with other MCP servers:get_current_time→time_get_currentconvert_time→time_convertMotivation
This follows the naming convention used by other MCP servers (e.g.,
git_status,git_commit,git_diff) and makes it easier to identify related tools when multiple MCP servers are registered with a client.As noted in issue #3201, users integrating multiple MCP servers see tool names like:
MCP_DOCKER_convert_timeMCP_DOCKER_get_current_timeWith this change, the tools become:
MCP_DOCKER_time_get_currentMCP_DOCKER_time_convertThis makes it immediately clear these tools belong to the Time server.
Changes
server.pyBreaking Change Notice
get_current_time,convert_time) will need to be updated.Closes #3201