Skip to content

fix: rename deferred tools to comply with MCP naming standard#6

Merged
abidlabs merged 2 commits intogradio-app:mainfrom
Mandark-droid:fix/mcp-tool-naming-standard
Feb 27, 2026
Merged

fix: rename deferred tools to comply with MCP naming standard#6
abidlabs merged 2 commits intogradio-app:mainfrom
Mandark-droid:fix/mcp-tool-naming-standard

Conversation

@Mandark-droid
Copy link
Copy Markdown
Contributor

Summary

  • Renamed "Search Deferred Tools" to "search_deferred_tools"
  • Renamed "Call Deferred Tool" to "call_deferred_tool"

Fixes #5

Problem

The MCP tool naming standard (SEP-986) only allows: A-Z, a-z, 0-9, underscore (_), dash (-), and dot (.)

Spaces in tool names cause validation warnings and potential compatibility issues with MCP clients.

Changes

File Changes
toolsets/mcp_server.py Tool definitions and handler checks
toolsets/gradio_ui.py UI display and schema references
toolsets/toolset.py Docstring reference
README.md Documentation

Reference

https://modelcontextprotocol.io/specification/2025-11-25/server/tools#tool-names

@abidlabs
Copy link
Copy Markdown
Member

Ah makes sense, this looks good except that it includes the commits from #4. Could you separate the changes so this PR only includes the tool name change?

Renamed tool names to use underscores instead of spaces per MCP SEP-986:
- "Search Deferred Tools" -> "search_deferred_tools"
- "Call Deferred Tool" -> "call_deferred_tool"

The MCP specification requires tool names to only contain:
A-Z, a-z, 0-9, underscore (_), dash (-), and dot (.)

Spaces in tool names cause validation warnings and potential parsing issues.

See: https://modelcontextprotocol.io/specification/2025-11-25/server/tools#tool-names
@Mandark-droid Mandark-droid force-pushed the fix/mcp-tool-naming-standard branch from 1683c30 to cc5dc42 Compare February 23, 2026 21:38
@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Feb 23, 2026

🪼 branch checks and previews

Name Status URL
🦄 Changes failed! Workflow log

@abidlabs
Copy link
Copy Markdown
Member

🦄 change detected

This Pull Request includes changes to the following packages.

Package Version
toolsets minor

  • fix: rename deferred tools to comply with MCP naming standard

‼️ Changeset not approved. Ensure the version bump is appropriate for all packages before approving.

  • Maintainers can approve the changeset by checking this checkbox.

Something isn't right?

  • Maintainers can change the version label to modify the version bump.
  • If the bot has failed to detect any changes, or if this pull request needs to update multiple packages to different versions or requires a more comprehensive changelog entry, maintainers can update the changelog file directly.

@Mandark-droid
Copy link
Copy Markdown
Contributor Author

Hi @abidlabs, good catch! I've rebased this branch onto main so it no longer includes the commits from PR #4. This PR now only contains the tool name changes.

Copy link
Copy Markdown
Member

@abidlabs abidlabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thanks @Mandark-droid!

@abidlabs abidlabs merged commit 7f60512 into gradio-app:main Feb 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deferred tool names violate MCP naming standard (SEP-986)

3 participants