refactor(logging): Replace logger variable name from 'logger' to 'log'

Author: codeboyzhou

src/main/java/com/github/codeboyzhou/mcp/declarative/McpServers.java

@@ -25,7 +25,7 @@
 
 public class McpServers {
 
-  private static final Logger logger = LoggerFactory.getLogger(McpServers.class);
+  private static final Logger log = LoggerFactory.getLogger(McpServers.class);
 
   private static final McpServers INSTANCE = new McpServers();
 
@@ -71,7 +71,7 @@ public void startServer() {
 
   private void doStartServer(McpServerConfiguration configuration) {
     if (!configuration.enabled()) {
-      logger.warn("MCP server is disabled, please check your configuration file.");
+      log.warn("MCP server is disabled, please check your configuration file.");
       return;
     }
 

src/main/java/com/github/codeboyzhou/mcp/declarative/configuration/YAMLConfigurationLoader.java

@@ -12,7 +12,7 @@
 
 public record YAMLConfigurationLoader(String configFileName) {
 
-  private static final Logger logger = LoggerFactory.getLogger(YAMLConfigurationLoader.class);
+  private static final Logger log = LoggerFactory.getLogger(YAMLConfigurationLoader.class);
 
   private static final String DEFAULT_CONFIG_FILE_NAME = "mcp-server.yml";
 
@@ -24,7 +24,7 @@ public McpServerConfiguration loadConfig() {
     Path configFilePath = getConfigFilePath(configFileName);
     File file = configFilePath.toFile();
     McpServerConfiguration config = ObjectMappers.fromYaml(file, McpServerConfiguration.class);
-    logger.info("Configuration loaded successfully from file: {}", configFileName);
+    log.info("Configuration loaded successfully from file: {}", configFileName);
     return config;
   }
 

src/main/java/com/github/codeboyzhou/mcp/declarative/server/McpHttpServer.java

@@ -10,7 +10,7 @@
 
 public class McpHttpServer {
 
-  private static final Logger logger = LoggerFactory.getLogger(McpHttpServer.class);
+  private static final Logger log = LoggerFactory.getLogger(McpHttpServer.class);
 
   private static final String DEFAULT_SERVLET_CONTEXT_PATH = "/";
 
@@ -45,26 +45,26 @@ public void start() {
     try {
       httpserver.start();
       addShutdownHook(httpserver);
-      logger.info("Jetty-based HTTP server started on http://127.0.0.1:{}", port);
+      log.info("Jetty-based HTTP server started on http://127.0.0.1:{}", port);
     } catch (Exception e) {
-      logger.error("Error starting HTTP server on http://127.0.0.1:{}", port, e);
+      log.error("Error starting HTTP server on http://127.0.0.1:{}", port, e);
     }
 
     try {
       httpserver.join();
     } catch (InterruptedException e) {
-      logger.error("Error joining HTTP server", e);
+      log.error("Error joining HTTP server", e);
     }
   }
 
   private void addShutdownHook(Server httpserver) {
     Runnable runnable =
         () -> {
           try {
-            logger.info("Shutting down HTTP server and MCP server");
+            log.info("Shutting down HTTP server and MCP server");
             httpserver.stop();
           } catch (Exception e) {
-            logger.error("Error stopping HTTP server and MCP server", e);
+            log.error("Error stopping HTTP server and MCP server", e);
           }
         };
     Thread shutdownHook = new Thread(runnable);

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

@@ -12,7 +12,7 @@
 
 public abstract class AbstractMcpServerComponentFactory<T> implements McpServerComponentFactory<T> {
 
-  private static final Logger logger =
+  private static final Logger log =
       LoggerFactory.getLogger(AbstractMcpServerComponentFactory.class);
 
   private static final String RESOURCE_BUNDLE_BASE_NAME = "i18n/mcp_server_component_descriptions";
@@ -44,7 +44,7 @@ private ResourceBundle loadResourceBundle() {
     try {
       return ResourceBundle.getBundle(RESOURCE_BUNDLE_BASE_NAME, locale);
     } catch (Exception e) {
-      logger.warn(
+      log.warn(
           "Can't find resource bundle for base name: {}, locale {}, i18n will be unsupported",
           RESOURCE_BUNDLE_BASE_NAME,
           locale);

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

@@ -29,7 +29,7 @@
 public class McpServerPromptFactory
     extends AbstractMcpServerComponentFactory<McpServerFeatures.SyncPromptSpecification> {
 
-  private static final Logger logger = LoggerFactory.getLogger(McpServerPromptFactory.class);
+  private static final Logger log = LoggerFactory.getLogger(McpServerPromptFactory.class);
 
   @Inject
   protected McpServerPromptFactory(
@@ -45,7 +45,7 @@ public McpServerFeatures.SyncPromptSpecification create(Class<?> clazz, Method m
     final String description = resolveComponentAttributeValue(promptMethod.description());
     List<McpSchema.PromptArgument> promptArguments = createPromptArguments(method);
     McpSchema.Prompt prompt = new McpSchema.Prompt(name, title, description, promptArguments);
-    logger.debug("Registering prompt: {}", ObjectMappers.toJson(prompt));
+    log.debug("Registering prompt: {}", ObjectMappers.toJson(prompt));
     return new McpServerFeatures.SyncPromptSpecification(
         prompt,
         (exchange, request) -> {
@@ -56,7 +56,7 @@ public McpServerFeatures.SyncPromptSpecification create(Class<?> clazz, Method m
                 asTypedParameters(method, promptArguments, request.arguments());
             result = method.invoke(instance, typedParameters.values().toArray());
           } catch (Exception e) {
-            logger.error("Error invoking prompt method", e);
+            log.error("Error invoking prompt method", e);
             result = e + ": " + e.getMessage();
           }
           McpSchema.Content content = new McpSchema.TextContent(result.toString());

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

@@ -22,7 +22,7 @@
 public class McpServerResourceFactory
     extends AbstractMcpServerComponentFactory<McpServerFeatures.SyncResourceSpecification> {
 
-  private static final Logger logger = LoggerFactory.getLogger(McpServerResourceFactory.class);
+  private static final Logger log = LoggerFactory.getLogger(McpServerResourceFactory.class);
 
   @Inject
   protected McpServerResourceFactory(
@@ -45,7 +45,7 @@ public McpServerFeatures.SyncResourceSpecification create(Class<?> clazz, Method
             .mimeType(res.mimeType())
             .annotations(new McpSchema.Annotations(List.of(res.roles()), res.priority()))
             .build();
-    logger.debug("Registering resource: {}", ObjectMappers.toJson(resource));
+    log.debug("Registering resource: {}", ObjectMappers.toJson(resource));
     return new McpServerFeatures.SyncResourceSpecification(
         resource,
         (exchange, request) -> {
@@ -54,7 +54,7 @@ public McpServerFeatures.SyncResourceSpecification create(Class<?> clazz, Method
             Object instance = injector.getInstance(clazz);
             result = method.invoke(instance);
           } catch (Exception e) {
-            logger.error("Error invoking resource method", e);
+            log.error("Error invoking resource method", e);
             result = e + ": " + e.getMessage();
           }
           McpSchema.ResourceContents contents =

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

@@ -34,7 +34,7 @@
 public class McpServerToolFactory
     extends AbstractMcpServerComponentFactory<McpServerFeatures.SyncToolSpecification> {
 
-  private static final Logger logger = LoggerFactory.getLogger(McpServerToolFactory.class);
+  private static final Logger log = LoggerFactory.getLogger(McpServerToolFactory.class);
 
   @Inject
   protected McpServerToolFactory(
@@ -56,7 +56,7 @@ public McpServerFeatures.SyncToolSpecification create(Class<?> clazz, Method met
             .description(description)
             .inputSchema(paramSchema)
             .build();
-    logger.debug("Registering tool: {}", ObjectMappers.toJson(tool));
+    log.debug("Registering tool: {}", ObjectMappers.toJson(tool));
     return McpServerFeatures.SyncToolSpecification.builder()
         .tool(tool)
         .callHandler(
@@ -69,7 +69,7 @@ public McpServerFeatures.SyncToolSpecification create(Class<?> clazz, Method met
                 Map<String, Object> typedArgs = asTypedParameters(paramSchema, args);
                 result = method.invoke(instance, typedArgs.values().toArray());
               } catch (Exception e) {
-                logger.error("Error invoking tool method", e);
+                log.error("Error invoking tool method", e);
                 result = e + ": " + e.getMessage();
                 isError = true;
               }

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

@@ -3,10 +3,16 @@
 import com.github.codeboyzhou.mcp.declarative.server.McpHttpServer;
 import io.modelcontextprotocol.server.McpServer;
 import io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class McpSseServerFactory extends AbstractMcpServerFactory<McpSseServerInfo> {
+
+  private static final Logger log = LoggerFactory.getLogger(McpSseServerFactory.class);
+
   @Override
   public McpServer.SyncSpecification<?> sync(McpSseServerInfo info) {
+    log.warn("HTTP SSE mode has been deprecated, recommend to use Stream HTTP server instead.");
     HttpServletSseServerTransportProvider transportProvider =
         HttpServletSseServerTransportProvider.builder()
             .baseUrl(info.baseUrl())

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

@@ -5,15 +5,20 @@
 import com.github.codeboyzhou.mcp.declarative.server.McpHttpServer;
 import io.modelcontextprotocol.server.McpServer;
 import io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class ConfigurableMcpSseServerFactory extends AbstractConfigurableMcpServerFactory {
 
+  private static final Logger log = LoggerFactory.getLogger(ConfigurableMcpSseServerFactory.class);
+
   public ConfigurableMcpSseServerFactory(McpServerConfiguration configuration) {
     super(configuration);
   }
 
   @Override
   public McpServer.SyncSpecification<?> specification() {
+    log.warn("HTTP SSE mode has been deprecated, recommend to use Stream HTTP server instead.");
     McpServerSSE sse = configuration.sse();
     HttpServletSseServerTransportProvider transportProvider =
         HttpServletSseServerTransportProvider.builder()