fix: array-to-List bug and clarify synchronous usage in “Async” classes

kimsunghyun1995 · kimsunghyun1995 · commit f8dc0d2bfd1f · 2025-04-21T15:27:02.000+09:00
Signed-off-by: kimsunghyun &lt;tjdgus2319@gmail.com&gt;
diff --git a/mcp/common/src/main/java/org/springframework/ai/mcp/McpToolUtils.java b/mcp/common/src/main/java/org/springframework/ai/mcp/McpToolUtils.java
@@ -16,6 +16,7 @@
 
 package org.springframework.ai.mcp;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -323,7 +324,7 @@ public static List<ToolCallback> getToolCallbacksFromSyncClients(List<McpSyncCli
 		if (CollectionUtils.isEmpty(mcpClients)) {
 			return List.of();
 		}
-		return List.of((new SyncMcpToolCallbackProvider(mcpClients).getToolCallbacks()));
+		return Arrays.asList(new SyncMcpToolCallbackProvider(mcpClients).getToolCallbacks());
 	}
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@`
`16`	`16`
`17`	`17`	`package org.springframework.ai.mcp;`
`18`	`18`
	`19`	`+import java.util.Arrays;`
`19`	`20`	`import java.util.List;`
`20`	`21`	`import java.util.Map;`
`21`	`22`	`import java.util.Optional;`
`@@ -323,7 +324,7 @@ public static List<ToolCallback> getToolCallbacksFromSyncClients(List<McpSyncCli`
`323`	`324`	`if (CollectionUtils.isEmpty(mcpClients)) {`
`324`	`325`	`return List.of();`
`325`	`326`	`}`
`326`		`- return List.of((new SyncMcpToolCallbackProvider(mcpClients).getToolCallbacks()));`
	`327`	`+ return Arrays.asList(new SyncMcpToolCallbackProvider(mcpClients).getToolCallbacks());`
`327`	`328`	`}`
`328`	`329`
`329`	`330`	`/**`