feat!: Remove deprecated create method in ResponseProcessor

tilgalas · copybara-github · commit 5e1e1d434fa1 · 2026-03-09T06:57:42.000-07:00
PiperOrigin-RevId: 880834921
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
@@ -159,16 +159,15 @@ 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)
           .toList()
           .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
@@ -50,11 +50,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(transferToAgent));
+    }
+
+    public static ResponseProcessingResult create(
+        LlmResponse updatedResponse, Iterable<Event> events) {
+      return new AutoValue_ResponseProcessor_ResponseProcessingResult(
+          updatedResponse, events, /* transferToAgent= */ 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
@@ -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 {

Original file line number	Diff line number	Diff line change
`@@ -159,16 +159,15 @@ public Single<ResponseProcessor.ResponseProcessingResult> processResponse(`
`159`	`159`	`InvocationContext invocationContext, LlmResponse llmResponse) {`
`160`	`160`	`if (llmResponse.partial().orElse(false)) {`
`161`	`161`	`return Single.just(`
`162`		`- ResponseProcessor.ResponseProcessingResult.create(`
`163`		`- llmResponse, ImmutableList.of(), Optional.empty()));`
	`162`	`+ ResponseProcessor.ResponseProcessingResult.create(llmResponse, ImmutableList.of()));`
`164`	`163`	`}`
`165`	`164`	`var llmResponseBuilder = llmResponse.toBuilder();`
`166`	`165`	`return runPostProcessor(invocationContext, llmResponseBuilder)`
`167`	`166`	`.toList()`
`168`	`167`	`.map(`
`169`	`168`	`events ->`
`170`	`169`	`ResponseProcessor.ResponseProcessingResult.create(`
`171`		`- llmResponseBuilder.build(), events, Optional.empty()));`
	`170`	`+ llmResponseBuilder.build(), events));`
`172`	`171`	`}`
`173`	`172`	`}`
`174`	`173`