refactor(configuration): Improve server default configuration loading

Author: codeboyzhou

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

@@ -100,7 +100,9 @@ public void startServer(String configFileName) {
     }
 
     public void startServer() {
-        startServer("mcp-server.yml");
+        YamlConfigurationLoader configurationLoader = new YamlConfigurationLoader();
+        McpServerConfiguration configuration = configurationLoader.loadConfiguration();
+        startServerWith(configuration);
     }
 
     private void startServerWith(McpServerConfiguration configuration) {

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

@@ -10,7 +10,6 @@
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.nio.file.NoSuchFileException;
-import java.util.Objects;
 
 import static java.util.stream.Collectors.joining;
 
@@ -22,14 +21,14 @@ public class YamlConfigurationLoader {
 
     public McpServerConfiguration loadConfiguration() {
         try {
-            McpServerConfiguration configuration = load("mcp-server.yml");
-            if (configuration == null) {
-                configuration = load("mcp-server.yaml");
-            }
-            return Objects.requireNonNullElseGet(configuration, McpServerConfiguration::defaultConfiguration);
+            return load("mcp-server.yml");
         } catch (IOException e) {
-            logger.error("Error loading configuration file, will use default configuration", e);
-            return McpServerConfiguration.defaultConfiguration();
+            try {
+                return load("mcp-server.yaml");
+            } catch (IOException ex) {
+                logger.warn("The mcp-server.yml and mcp-server.yaml were not found, will use default configuration");
+                return McpServerConfiguration.defaultConfiguration();
+            }
         }
     }
 

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

@@ -102,7 +102,6 @@ void testStartServer() {
 
     @ParameterizedTest
     @ValueSource(strings = {
-        "mcp-server.yml",
         "mcp-server-async.yml",
         "mcp-server-sse-mode.yml",
         "mcp-server-not-enabled.yml"