modelcontextprotocol/rust-sdk
Watch
1
0
Fork
You've already forked rust-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: generate simple {} schema for tools with no parameters (#425)

Browse source
This commit is contained in:
Dale Seo 2025-09-08 02:40:02 -04:00 • committed by GitHub
commit b482cfc4bb
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
crates/rmcp-macros/src/tool.rs
View file

@ -186,9 +186,9 @@ pub fn tool(attr: TokenStream, input: TokenStream) -> syn::Result<TokenStream> {
rmcp::handler::server::common::cached_schema_for_type::<#params_ty>() rmcp::handler::server::common::cached_schema_for_type::<#params_ty>()
})? })?
} else { } else {
// if not found, use the default EmptyObject schema // if not found, use a simple empty JSON object
syn::parse2::<Expr>(quote! { syn::parse2::<Expr>(quote! {
rmcp::handler::server::common::cached_schema_for_type::<rmcp::model::EmptyObject>() std::sync::Arc::new(serde_json::Map::new())
})? })?
} }
}; };

2
crates/rmcp/tests/test_tool_macros.rs
View file

@ -110,7 +110,7 @@ async fn test_tool_macros() {
async fn test_tool_macros_with_empty_param() { async fn test_tool_macros_with_empty_param() {
let _attr = Server::empty_param_tool_attr(); let _attr = Server::empty_param_tool_attr();
println!("{_attr:?}"); println!("{_attr:?}");
assert_eq!(_attr.input_schema.get("type").unwrap(), "object"); assert!(_attr.input_schema.get("type").is_none());
assert!(_attr.input_schema.get("properties").is_none()); assert!(_attr.input_schema.get("properties").is_none());
} }