fix: Always use a mutable HashMap for default function arguments

google-genai-bot · copybara-github · commit 804a77704073 · 2026-01-12T06:56:19.000-08:00
PiperOrigin-RevId: 855208801
diff --git a/core/src/main/java/com/google/adk/flows/llmflows/Functions.java b/core/src/main/java/com/google/adk/flows/llmflows/Functions.java
@@ -263,8 +263,7 @@ private static Function<FunctionCall, Maybe<Event>> getFunctionCallMapper(
                 invocationContext));
       }
 
-      Map<String, Object> functionArgs =
-          functionCall.args().orElse(isLive ? new HashMap<>() : ImmutableMap.of());
+      Map<String, Object> functionArgs = functionCall.args().orElse(new HashMap<>());
 
       Maybe<Map<String, Object>> maybeFunctionResult =
           maybeInvokeBeforeToolCall(invocationContext, tool, functionArgs, toolContext)

Original file line number	Diff line number	Diff line change
`@@ -263,8 +263,7 @@ private static Function<FunctionCall, Maybe<Event>> getFunctionCallMapper(`
`263`	`263`	`invocationContext));`
`264`	`264`	`}`
`265`	`265`
`266`		`- Map<String, Object> functionArgs =`
`267`		`- functionCall.args().orElse(isLive ? new HashMap<>() : ImmutableMap.of());`
	`266`	`+ Map<String, Object> functionArgs = functionCall.args().orElse(new HashMap<>());`
`268`	`267`
`269`	`268`	`Maybe<Map<String, Object>> maybeFunctionResult =`
`270`	`269`	`maybeInvokeBeforeToolCall(invocationContext, tool, functionArgs, toolContext)`