Fixed incorrect variable usage in MethodToolCallback.java

yuchaozhou · web-flow · commit a2a6dc3b3fc5 · 2025-04-12T19:35:53.000+08:00
Signed-off-by: DaliangZhou &lt;zoyuchao@foxmail.com&gt;
diff --git a/spring-ai-model/src/main/java/org/springframework/ai/tool/method/MethodToolCallback.java b/spring-ai-model/src/main/java/org/springframework/ai/tool/method/MethodToolCallback.java
@@ -119,7 +119,7 @@ private void validateToolContextSupport(@Nullable ToolContext toolContext) {
 		var isNonEmptyToolContextProvided = toolContext != null && !CollectionUtils.isEmpty(toolContext.getContext());
 		var isToolContextAcceptedByMethod = Stream.of(toolMethod.getParameterTypes())
 			.anyMatch(type -> ClassUtils.isAssignable(type, ToolContext.class));
-		if (isToolContextAcceptedByMethod && !isNonEmptyToolContextProvided) {
+		if (isNonEmptyToolContextProvided && !isToolContextAcceptedByMethod) {
 			throw new IllegalArgumentException("ToolContext is required by the method as an argument");
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ private void validateToolContextSupport(@Nullable ToolContext toolContext) {`
`119`	`119`	`var isNonEmptyToolContextProvided = toolContext != null && !CollectionUtils.isEmpty(toolContext.getContext());`
`120`	`120`	`var isToolContextAcceptedByMethod = Stream.of(toolMethod.getParameterTypes())`
`121`	`121`	`.anyMatch(type -> ClassUtils.isAssignable(type, ToolContext.class));`
`122`		`- if (isToolContextAcceptedByMethod && !isNonEmptyToolContextProvided) {`
	`122`	`+ if (isNonEmptyToolContextProvided && !isToolContextAcceptedByMethod) {`
`123`	`123`	`throw new IllegalArgumentException("ToolContext is required by the method as an argument");`
`124`	`124`	`}`
`125`	`125`	`}`