Skip to content

Commit a55316c

Browse files
committed
Remove name assertion from ToolResponseMessage validation
- Removed the assertion checking for a non-null 'name' in ToolResponseMessage - This change affects multiple AI model implementations: AzureOpenAiChatModel, MiniMaxChatModel, MistralAiChatModel, MoonshotChatModel, OpenAiChatModel, and ZhiPuAiChatModel Resolves #1410
1 parent c5f07e5 commit a55316c

File tree

6 files changed

+0
-6
lines changed

6 files changed

+0
-6
lines changed

models/spring-ai-azure-openai/src/main/java/org/springframework/ai/azure/openai/AzureOpenAiChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -356,7 +356,6 @@ private List<ChatRequestMessage> fromSpringAiMessage(Message message) {
356356

357357
toolMessage.getResponses().forEach(response -> {
358358
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
359-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
360359
});
361360

362361
return toolMessage.getResponses()

models/spring-ai-minimax/src/main/java/org/springframework/ai/minimax/MiniMaxChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,6 @@ else if (message.getMessageType() == MessageType.TOOL) {
373373

374374
toolMessage.getResponses().forEach(response -> {
375375
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
376-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
377376
});
378377

379378
return toolMessage.getResponses()

models/spring-ai-mistral-ai/src/main/java/org/springframework/ai/mistralai/MistralAiChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,6 @@ else if (message instanceof ToolResponseMessage toolResponseMessage) {
345345

346346
toolResponseMessage.getResponses().forEach(response -> {
347347
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
348-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
349348
});
350349

351350
return toolResponseMessage.getResponses()

models/spring-ai-moonshot/src/main/java/org/springframework/ai/moonshot/MoonshotChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,6 @@ else if (message.getMessageType() == MessageType.TOOL) {
323323

324324
toolMessage.getResponses().forEach(response -> {
325325
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
326-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
327326
});
328327

329328
return toolMessage.getResponses()

models/spring-ai-openai/src/main/java/org/springframework/ai/openai/OpenAiChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -459,7 +459,6 @@ else if (message.getMessageType() == MessageType.TOOL) {
459459

460460
toolMessage.getResponses().forEach(response -> {
461461
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
462-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
463462
});
464463

465464
return toolMessage.getResponses()

models/spring-ai-zhipuai/src/main/java/org/springframework/ai/zhipuai/ZhiPuAiChatModel.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,6 @@ else if (message.getMessageType() == MessageType.TOOL) {
340340

341341
toolMessage.getResponses().forEach(response -> {
342342
Assert.isTrue(response.id() != null, "ToolResponseMessage must have an id");
343-
Assert.isTrue(response.name() != null, "ToolResponseMessage must have a name");
344343
});
345344

346345
return toolMessage.getResponses()

0 commit comments

Comments
 (0)