fix(macros): add error deal

1.Check whether the parameters include aggregated parameters and individual parameters
2.Check if the toolbox attribute is default

Signed-off-by: jokemanfire <[email protected]>

Author: jokemanfire

crates/rmcp-macros/src/tool.rs

@@ -3,8 +3,7 @@ use std::collections::HashSet;
 use proc_macro2::TokenStream;
 use quote::{ToTokens, quote};
 use syn::{
-    Expr, FnArg, Ident, ItemFn, ItemImpl, MetaList, PatType, Token, Type, Visibility, parse::Parse,
-    parse_quote,
+    parse::Parse, parse_quote, spanned::Spanned, Expr, FnArg, Ident, ItemFn, ItemImpl, MetaList, PatType, Token, Type, Visibility
 };
 
 #[derive(Default)]
@@ -224,6 +223,11 @@ pub(crate) fn tool_impl_item(attr: TokenStream, mut input: ItemImpl) -> syn::Res
                     rmcp::tool_box!(@derive #ident);
                 ));
             }
+        } else {
+            return Err(syn::Error::new(
+                proc_macro2::Span::call_site(),
+                "tool_box attribute is required for trait implementation",
+            ));
         }
     } else if let Some(ident) = tool_box_ident {
         // if it is a normal impl block
@@ -367,6 +371,12 @@ pub(crate) fn tool_fn_item(attr: TokenStream, mut input_fn: ItemFn) -> syn::Resu
                         unextractable_args_indexes.insert(index);
                     }
                     Some(Caught::Aggregated(rust_type)) => {
+                        if let ToolParams::Params { .. } = tool_macro_attrs.params {
+                            return Err(syn::Error::new(
+                                rust_type.span(),
+                                "cannot mix aggregated and individual parameters",
+                            ));
+                        }
                         tool_macro_attrs.params = ToolParams::Aggregated { rust_type };
                         unextractable_args_indexes.insert(index);
                     }