fix(vertex-ai): Invert proxyToolCalls flag to match internal tool execution behavior

tzolov · tzolov · commit e4d8a9c3ef7c · 2025-02-14T00:20:52.000+01:00
The proxyToolCalls flag was incorrectly mapped to internalToolExecutionEnabled. This change inverts the boolean value to
properly reflect that proxy tool calls are the opposite of internal tool execution.

Signed-off-by: Christian Tzolov &lt;christian.tzolov@broadcom.com&gt;
diff --git a/models/spring-ai-vertex-ai-gemini/src/main/java/org/springframework/ai/vertexai/gemini/VertexAiGeminiChatOptions.java b/models/spring-ai-vertex-ai-gemini/src/main/java/org/springframework/ai/vertexai/gemini/VertexAiGeminiChatOptions.java
@@ -491,7 +491,7 @@ public Builder safetySettings(List<VertexAiGeminiSafetySetting> safetySettings)
 
 		@Deprecated
 		public Builder proxyToolCalls(boolean proxyToolCalls) {
-			return this.internalToolExecutionEnabled(proxyToolCalls);
+			return this.internalToolExecutionEnabled(!proxyToolCalls);
 		}
 
 		public Builder internalToolExecutionEnabled(boolean internalToolExecutionEnabled) {

Original file line number	Diff line number	Diff line change
`@@ -491,7 +491,7 @@ public Builder safetySettings(List<VertexAiGeminiSafetySetting> safetySettings)`
`491`	`491`
`492`	`492`	`@Deprecated`
`493`	`493`	`public Builder proxyToolCalls(boolean proxyToolCalls) {`
`494`		`- return this.internalToolExecutionEnabled(proxyToolCalls);`
	`494`	`+ return this.internalToolExecutionEnabled(!proxyToolCalls);`
`495`	`495`	`}`
`496`	`496`
`497`	`497`	`public Builder internalToolExecutionEnabled(boolean internalToolExecutionEnabled) {`