From 17ee690fffee409b33a6d46ca6b92674541c2c14 Mon Sep 17 00:00:00 2001
From: Maciej Szwaja <mszwaja@google.com>
Date: Fri, 6 Mar 2026 11:47:44 -0800
Subject: [PATCH] feat!: Remove deprecated create method in ResponseProcessor

PiperOrigin-RevId: 879742182
---
 .../com/google/adk/flows/llmflows/CodeExecution.java |  5 ++---
 .../google/adk/flows/llmflows/ResponseProcessor.java | 12 +++++++++---
 .../google/adk/flows/llmflows/BaseLlmFlowTest.java   |  9 ++-------
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/core/src/main/java/com/google/adk/flows/llmflows/CodeExecution.java b/core/src/main/java/com/google/adk/flows/llmflows/CodeExecution.java
index f7c3c51ef..f2cbe967e 100644
--- a/core/src/main/java/com/google/adk/flows/llmflows/CodeExecution.java
+++ b/core/src/main/java/com/google/adk/flows/llmflows/CodeExecution.java
@@ -159,8 +159,7 @@ public Single<ResponseProcessor.ResponseProcessingResult> processResponse(
         InvocationContext invocationContext, LlmResponse llmResponse) {
       if (llmResponse.partial().orElse(false)) {
         return Single.just(
-            ResponseProcessor.ResponseProcessingResult.create(
-                llmResponse, ImmutableList.of(), Optional.empty()));
+            ResponseProcessor.ResponseProcessingResult.create(llmResponse, ImmutableList.of()));
       }
       var llmResponseBuilder = llmResponse.toBuilder();
       return runPostProcessor(invocationContext, llmResponseBuilder)
@@ -168,7 +167,7 @@ public Single<ResponseProcessor.ResponseProcessingResult> processResponse(
           .map(
               events ->
                   ResponseProcessor.ResponseProcessingResult.create(
-                      llmResponseBuilder.build(), events, Optional.empty()));
+                      llmResponseBuilder.build(), events));
     }
   }
 
diff --git a/core/src/main/java/com/google/adk/flows/llmflows/ResponseProcessor.java b/core/src/main/java/com/google/adk/flows/llmflows/ResponseProcessor.java
index 4baa29523..b95815383 100644
--- a/core/src/main/java/com/google/adk/flows/llmflows/ResponseProcessor.java
+++ b/core/src/main/java/com/google/adk/flows/llmflows/ResponseProcessor.java
@@ -19,6 +19,7 @@
 import com.google.adk.agents.InvocationContext;
 import com.google.adk.events.Event;
 import com.google.adk.models.LlmResponse;
+import com.google.api.client.util.Preconditions;
 import com.google.auto.value.AutoValue;
 import io.reactivex.rxjava3.core.Single;
 import java.util.Optional;
@@ -50,11 +51,16 @@ public abstract static class ResponseProcessingResult {
      */
     public abstract Optional<String> transferToAgent();
 
-    /** Creates a new {@link ResponseProcessingResult}. */
     public static ResponseProcessingResult create(
-        LlmResponse updatedResponse, Iterable<Event> events, Optional<String> transferToAgent) {
+        LlmResponse updatedResponse, Iterable<Event> events, String transferToAgent) {
       return new AutoValue_ResponseProcessor_ResponseProcessingResult(
-          updatedResponse, events, transferToAgent);
+          updatedResponse, events, Optional.of(Preconditions.checkNotNull(transferToAgent)));
+    }
+
+    public static ResponseProcessingResult create(
+        LlmResponse updatedResponse, Iterable<Event> events) {
+      return new AutoValue_ResponseProcessor_ResponseProcessingResult(
+          updatedResponse, events, Optional.empty());
     }
   }
 
diff --git a/core/src/test/java/com/google/adk/flows/llmflows/BaseLlmFlowTest.java b/core/src/test/java/com/google/adk/flows/llmflows/BaseLlmFlowTest.java
index ff151a0b2..4a0b345c6 100644
--- a/core/src/test/java/com/google/adk/flows/llmflows/BaseLlmFlowTest.java
+++ b/core/src/test/java/com/google/adk/flows/llmflows/BaseLlmFlowTest.java
@@ -524,19 +524,14 @@ private static RequestProcessor createRequestProcessor(
 
   private static ResponseProcessor createResponseProcessor() {
     return (context, response) ->
-        Single.just(
-            ResponseProcessingResult.create(
-                response, ImmutableList.of(), /* transferToAgent= */ Optional.empty()));
+        Single.just(ResponseProcessingResult.create(response, ImmutableList.of()));
   }
 
   private static ResponseProcessor createResponseProcessor(
       Function<LlmResponse, LlmResponse> responseUpdater) {
     return (context, response) ->
         Single.just(
-            ResponseProcessingResult.create(
-                responseUpdater.apply(response),
-                ImmutableList.of(),
-                /* transferToAgent= */ Optional.empty()));
+            ResponseProcessingResult.create(responseUpdater.apply(response), ImmutableList.of()));
   }
 
   private static class TestTool extends BaseTool {