Update x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/external/unified/UnifiedChatCompletionRequestEntity.java

jaybcee · timgrein · web-flow · commit b03ee46b8213 · 2024-12-19T13:34:35.000-05:00
Co-authored-by: Tim Grein &lt;tim.grein@elastic.co&gt;
diff --git a/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/external/unified/UnifiedChatCompletionRequestEntity.java b/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/external/unified/UnifiedChatCompletionRequestEntity.java
@@ -135,7 +135,9 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
                 builder.endObject();
             }
         }
-        if (unifiedRequest.tools() != null && unifiedRequest.tools().isEmpty() == false) {
+       boolean usesTools = unifiedRequest.tools() != null && unifiedRequest.tools().isEmpty() == false
+
+        if (usesTools) {
             builder.startArray(TOOL_FIELD);
             for (UnifiedCompletionRequest.Tool t : unifiedRequest.tools()) {
                 builder.startObject();

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,9 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws`
`135`	`135`	`builder.endObject();`
`136`	`136`	`}`
`137`	`137`	`}`
`138`		`- if (unifiedRequest.tools() != null && unifiedRequest.tools().isEmpty() == false) {`
	`138`	`+ boolean usesTools = unifiedRequest.tools() != null && unifiedRequest.tools().isEmpty() == false`
	`139`	`+`
	`140`	`+ if (usesTools) {`
`139`	`141`	`builder.startArray(TOOL_FIELD);`
`140`	`142`	`for (UnifiedCompletionRequest.Tool t : unifiedRequest.tools()) {`
`141`	`143`	`builder.startObject();`