Address review comments

Signed-off-by: Christian Tzolov <[email protected]>

Author: tzolov

auto-configurations/spring-ai-mcp-client/src/main/java/org/springframework/ai/autoconfigure/mcp/client/McpClientAutoConfiguration.java

@@ -110,6 +110,18 @@
 		matchIfMissing = true)
 public class McpClientAutoConfiguration {
 
+	/**
+	 * Create a dynamic client name based on the client name and the name of the server
+	 * connection.
+	 * @param clientName the client name as defined by the configuration
+	 * @param serverConnectionName the name of the server connection being used by the
+	 * client
+	 * @return the connected client name
+	 */
+	private String connectedClientName(String clientName, String serverConnectionName) {
+		return clientName + " - " + serverConnectionName;
+	}
+
 	/**
 	 * Creates a list of {@link McpSyncClient} instances based on the available
 	 * transports.
@@ -145,7 +157,8 @@ public List<McpSyncClient> mcpSyncClients(McpSyncClientConfigurer mcpSyncClientC
 			for (NamedClientMcpTransport namedTransport : namedTransports) {
 
 				McpSchema.Implementation clientInfo = new McpSchema.Implementation(
-						commonProperties.getName() + " - " + namedTransport.name(), commonProperties.getVersion());
+						this.connectedClientName(commonProperties.getName(), namedTransport.name()),
+						commonProperties.getVersion());
 
 				McpClient.SyncSpec syncSpec = McpClient.sync(namedTransport.transport())
 					.clientInfo(clientInfo)
@@ -257,7 +270,8 @@ public List<McpAsyncClient> mcpAsyncClients(McpAsyncClientConfigurer mcpSyncClie
 			for (NamedClientMcpTransport namedTransport : namedTransports) {
 
 				McpSchema.Implementation clientInfo = new McpSchema.Implementation(
-						commonProperties.getName() + " - " + namedTransport.name(), commonProperties.getVersion());
+						this.connectedClientName(commonProperties.getName(), namedTransport.name()),
+						commonProperties.getVersion());
 
 				McpClient.AsyncSpec syncSpec = McpClient.async(namedTransport.transport())
 					.clientInfo(clientInfo)