Revert change of type

Signed-off-by: Filip Hrisafov <[email protected]>

Author: filiphr

spring-ai-client-chat/src/main/java/org/springframework/ai/chat/client/DefaultChatClient.java

@@ -574,7 +574,7 @@ public static class DefaultChatClientRequestSpec implements ChatClientRequestSpe
 
 		private final List<Media> media = new ArrayList<>();
 
-		private final Set<String> toolNames = new LinkedHashSet<>();
+		private final List<String> toolNames = new ArrayList<>();
 
 		private final List<ToolCallback> toolCallbacks = new ArrayList<>();
 
@@ -610,9 +610,9 @@ public static class DefaultChatClientRequestSpec implements ChatClientRequestSpe
 
 		public DefaultChatClientRequestSpec(ChatModel chatModel, @Nullable String userText,
 				Map<String, Object> userParams, @Nullable String systemText, Map<String, Object> systemParams,
-				List<ToolCallback> toolCallbacks, List<Message> messages, Collection<String> toolNames,
-				List<Media> media, @Nullable ChatOptions chatOptions, List<Advisor> advisors,
-				Map<String, Object> advisorParams, ObservationRegistry observationRegistry,
+				List<ToolCallback> toolCallbacks, List<Message> messages, List<String> toolNames, List<Media> media,
+				@Nullable ChatOptions chatOptions, List<Advisor> advisors, Map<String, Object> advisorParams,
+				ObservationRegistry observationRegistry,
 				@Nullable ChatClientObservationConvention observationConvention, Map<String, Object> toolContext,
 				@Nullable TemplateRenderer templateRenderer) {
 
@@ -689,7 +689,7 @@ public List<Media> getMedia() {
 			return this.media;
 		}
 
-		public Set<String> getToolNames() {
+		public List<String> getToolNames() {
 			return this.toolNames;
 		}
 

spring-ai-client-chat/src/main/java/org/springframework/ai/chat/client/DefaultChatClientUtils.java

@@ -98,7 +98,7 @@ static ChatClientRequest toChatClientRequest(DefaultChatClient.DefaultChatClient
 			if (processedChatOptions == null) {
 				ToolCallingChatOptions.Builder builder = ToolCallingChatOptions.builder();
 				if (!inputRequest.getToolNames().isEmpty()) {
-					builder.toolNames(inputRequest.getToolNames());
+					builder.toolNames(new HashSet<>(inputRequest.getToolNames()));
 				}
 				if (!inputRequest.getToolCallbacks().isEmpty()) {
 					List<ToolCallback> toolCallbacks = inputRequest.getToolCallbacks();