refactor(server): Replace constructor calls with builder pattern

Author: codeboyzhou

src/main/java/com/github/codeboyzhou/mcp/declarative/server/factory/ConfigurableMcpHttpSseServerFactory.java

@@ -4,7 +4,6 @@
 import com.github.codeboyzhou.mcp.declarative.configuration.McpServerConfiguration;
 import com.github.codeboyzhou.mcp.declarative.configuration.McpServerSSE;
 import com.github.codeboyzhou.mcp.declarative.server.McpHttpServer;
-import com.github.codeboyzhou.mcp.declarative.util.JsonHelper;
 import io.modelcontextprotocol.server.McpAsyncServer;
 import io.modelcontextprotocol.server.McpServer;
 import io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider;
@@ -24,8 +23,11 @@ public HttpServletSseServerTransportProvider transportProvider() {
     final String baseUrl = sse.baseUrl();
     final String messageEndpoint = sse.messageEndpoint();
     final String sseEndpoint = sse.endpoint();
-    return new HttpServletSseServerTransportProvider(
-        JsonHelper.MAPPER, baseUrl, messageEndpoint, sseEndpoint);
+    return HttpServletSseServerTransportProvider.builder()
+        .baseUrl(baseUrl)
+        .sseEndpoint(sseEndpoint)
+        .messageEndpoint(messageEndpoint)
+        .build();
   }
 
   @Override

src/main/java/com/github/codeboyzhou/mcp/declarative/server/factory/McpHttpSseServerFactory.java

@@ -3,7 +3,6 @@
 import com.github.codeboyzhou.mcp.declarative.common.NamedThreadFactory;
 import com.github.codeboyzhou.mcp.declarative.server.McpHttpServer;
 import com.github.codeboyzhou.mcp.declarative.server.McpSseServerInfo;
-import com.github.codeboyzhou.mcp.declarative.util.JsonHelper;
 import io.modelcontextprotocol.server.McpAsyncServer;
 import io.modelcontextprotocol.server.McpServer;
 import io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider;
@@ -17,8 +16,11 @@ public HttpServletSseServerTransportProvider transportProvider(McpSseServerInfo
     final String baseUrl = serverInfo.baseUrl();
     final String messageEndpoint = serverInfo.messageEndpoint();
     final String sseEndpoint = serverInfo.sseEndpoint();
-    return new HttpServletSseServerTransportProvider(
-        JsonHelper.MAPPER, baseUrl, messageEndpoint, sseEndpoint);
+    return HttpServletSseServerTransportProvider.builder()
+        .baseUrl(baseUrl)
+        .sseEndpoint(sseEndpoint)
+        .messageEndpoint(messageEndpoint)
+        .build();
   }
 
   @Override

src/main/java/com/github/codeboyzhou/mcp/declarative/server/factory/McpServerResourceFactory.java

@@ -38,11 +38,15 @@ public McpServerFeatures.AsyncResourceSpecification create(Class<?> clazz, Metho
     final String name = StringHelper.defaultIfBlank(res.name(), method.getName());
     final String title = resolveComponentAttributeValue(res.title());
     final String description = resolveComponentAttributeValue(res.description());
-    McpSchema.Annotations annotations =
-        new McpSchema.Annotations(List.of(res.roles()), res.priority());
     McpSchema.Resource resource =
-        new McpSchema.Resource(
-            res.uri(), name, title, description, res.mimeType(), null, annotations);
+        McpSchema.Resource.builder()
+            .uri(res.uri())
+            .name(name)
+            .title(title)
+            .description(description)
+            .mimeType(res.mimeType())
+            .annotations(new McpSchema.Annotations(List.of(res.roles()), res.priority()))
+            .build();
     logger.debug("Registering resource: {}", JsonHelper.toJson(resource));
     return new McpServerFeatures.AsyncResourceSpecification(
         resource,

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

@@ -59,25 +59,28 @@ public McpServerFeatures.AsyncToolSpecification create(Class<?> clazz, Method me
             .inputSchema(paramSchema)
             .build();
     logger.debug("Registering tool: {}", JsonHelper.toJson(tool));
-    return new McpServerFeatures.AsyncToolSpecification(
-        tool,
-        (exchange, params) ->
-            Mono.fromSupplier(
-                () -> {
-                  Object result;
-                  boolean isError = false;
-                  try {
-                    Object instance = injector.getInstance(clazz);
-                    Map<String, Object> typedParameters = asTypedParameters(paramSchema, params);
-                    result = method.invoke(instance, typedParameters.values().toArray());
-                  } catch (Exception e) {
-                    logger.error("Error invoking tool method", e);
-                    result = e + ": " + e.getMessage();
-                    isError = true;
-                  }
-                  McpSchema.Content content = new McpSchema.TextContent(result.toString());
-                  return new McpSchema.CallToolResult(List.of(content), isError);
-                }));
+    return McpServerFeatures.AsyncToolSpecification.builder()
+        .tool(tool)
+        .callHandler(
+            (exchange, request) ->
+                Mono.fromSupplier(
+                    () -> {
+                      Object result;
+                      boolean isError = false;
+                      try {
+                        Object instance = injector.getInstance(clazz);
+                        Map<String, Object> args = request.arguments();
+                        Map<String, Object> typedArgs = asTypedParameters(paramSchema, args);
+                        result = method.invoke(instance, typedArgs.values().toArray());
+                      } catch (Exception e) {
+                        logger.error("Error invoking tool method", e);
+                        result = e + ": " + e.getMessage();
+                        isError = true;
+                      }
+                      McpSchema.Content content = new McpSchema.TextContent(result.toString());
+                      return new McpSchema.CallToolResult(List.of(content), isError);
+                    }))
+        .build();
   }
 
   @Override