refactor(server): Update parameter handling in McpServerToolFactory and enhance logging in test classes

Author: codeboyzhou

src/main/java/com/github/codeboyzhou/mcp/declarative/server/component/McpServerToolFactory.java

@@ -81,14 +81,14 @@ private McpSchema.JsonSchema createJsonSchema(Method method) {
       Class<?> parameterType = param.getType();
       Map<String, String> property = new HashMap<>();
 
-      if (parameterType.getAnnotation(McpJsonSchemaDefinition.class) == null) {
-        property.put("type", parameterType.getSimpleName().toLowerCase());
-        property.put("description", resolveComponentAttributeValue(toolParam.description()));
-      } else {
+      if (parameterType.isAnnotationPresent(McpJsonSchemaDefinition.class)) {
         final String parameterTypeSimpleName = parameterType.getSimpleName();
         property.put("$ref", "#/definitions/" + parameterTypeSimpleName);
         Map<String, Object> definition = createJsonSchemaDefinition(parameterType);
         definitions.put(parameterTypeSimpleName, definition);
+      } else {
+        property.put("type", parameterType.getSimpleName().toLowerCase());
+        property.put("description", resolveComponentAttributeValue(toolParam.description()));
       }
       properties.put(parameterName, property);
 

src/test/java/com/github/codeboyzhou/mcp/declarative/test/TestMcpPrompts.java

@@ -4,15 +4,21 @@
 import com.github.codeboyzhou.mcp.declarative.annotation.McpPromptParam;
 import com.github.codeboyzhou.mcp.declarative.util.ObjectMappers;
 import java.util.Map;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class TestMcpPrompts {
 
+  private static final Logger log = LoggerFactory.getLogger(TestMcpPrompts.class);
+
   @McpPrompt(name = "prompt1_name", title = "prompt1_title", description = "prompt1_description")
   public String prompt1(
       @McpPromptParam(name = "param1", title = "param1_title", description = "param1_description")
           String param1,
       @McpPromptParam(name = "param2", title = "param2_title", description = "param2_description")
-          String param2) {
+          String param2,
+      String param3) {
+    log.debug("prompt1 called with params: {}, {}, {}", param1, param2, param3);
     return ObjectMappers.toJson(Map.of("param1", param1, "param2", param2));
   }
 }

src/test/java/com/github/codeboyzhou/mcp/declarative/test/TestMcpTools.java

@@ -4,13 +4,20 @@
 import com.github.codeboyzhou.mcp.declarative.annotation.McpToolParam;
 import com.github.codeboyzhou.mcp.declarative.util.ObjectMappers;
 import java.util.Map;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class TestMcpTools {
 
+  private static final Logger log = LoggerFactory.getLogger(TestMcpTools.class);
+
   @McpTool(name = "tool1_name", title = "tool1_title", description = "tool1_description")
   public String tool1(
       @McpToolParam(name = "param1", description = "param1_description") String param1,
-      @McpToolParam(name = "param2", description = "param2_description") String param2) {
+      @McpToolParam(name = "param2", description = "param2_description", required = true)
+          String param2,
+      String param3) {
+    log.debug("tool1 called with params: {}, {}, {}", param1, param2, param3);
     return ObjectMappers.toJson(Map.of("param1", param1, "param2", param2));
   }
 }