feat: Support instance methods for automatic function calling in Google GenAI Java SDK. - #1199
Merged
Merged
Conversation
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
2 times, most recently
from
August 24, 2026 16:37
2965fb4 to
6af6b61
Compare
copybara-service
Bot
changed the title
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
6 times, most recently
from
August 25, 2026 19:52
36ed0c9 to
b4848bf
Compare
copybara-service
Bot
changed the title
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
19 times, most recently
from
August 31, 2026 16:53
b18dd6a to
4913311
Compare
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
14 times, most recently
from
September 4, 2026 21:43
4d284c8 to
7442369
Compare
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
5 times, most recently
from
September 15, 2026 10:11
4309c7e to
ebc3650
Compare
…le GenAI Java SDK.
Automatic Function Calling (AFC) previously restricted function declarations and invocations to public static methods only, throwing an `IllegalArgumentException` ("Instance methods are not supported. Please use static methods.") if an instance method was provided. This forced users to encapsulate state in static fields or create static wrappers when calling methods on service objects.
Support passing an instance or instance mapping along with `Method` references when configuring tools for Automatic Function Calling:
- Remove static modifier validation in `FunctionDeclaration.fromMethod(...)` so schemas can be generated from instance methods.
- Add `function_instances` mapping field (`Map<Method, Object>`) to `Tool` definition.
- Add validation in `AfcUtil.getFunctionMap(...)` to ensure that any non-static method has a corresponding instance in `functionInstances`, failing fast with a descriptive `IllegalArgumentException`.
- Update `AfcUtil` to extract function instances from `GenerateContentConfig` and invoke target methods with their bound instance (`method.invoke(instance, args)`).
- Safely handle optional `name` and `args` in `FunctionCall`, preventing `NoSuchElementException` and handling null map entries in `AfcUtil.getFunctionResponseParts` and `getFunctionResponse`.
- Pass `functionInstanceMap` across synchronous and asynchronous model generation loops (`Models`, `AsyncModels`, `Chat`, and `AsyncChat`).
- Add unit tests in `FunctionDeclarationTest`, `AfcUtilTest`, and `PrivateChatAfcTest`.
Closes #807
PiperOrigin-RevId: 981705233
copybara-service
Bot
force-pushed
the
copybara_964733917
branch
from
September 15, 2026 10:29
ebc3650 to
a372539
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat: Support instance methods for automatic function calling in Google GenAI Java SDK.
Automatic Function Calling (AFC) previously restricted function declarations and invocations to public static methods only, throwing an
IllegalArgumentException("Instance methods are not supported. Please use static methods.") if an instance method was provided. This forced users to encapsulate state in static fields or create static wrappers when calling methods on service objects.Support passing an instance or instance mapping along with
Methodreferences when configuring tools for Automatic Function Calling:FunctionDeclaration.fromMethod(...)so schemas can be generated from instance methods.function_instancesmapping field (Map<Method, Object>) toTooldefinition.AfcUtil.getFunctionMap(...)to ensure that any non-static method has a corresponding instance infunctionInstances, failing fast with a descriptiveIllegalArgumentException.AfcUtilto extract function instances fromGenerateContentConfigand invoke target methods with their bound instance (method.invoke(instance, args)).nameandargsinFunctionCall, preventingNoSuchElementExceptionand handling null map entries inAfcUtil.getFunctionResponsePartsandgetFunctionResponse.functionInstanceMapacross synchronous and asynchronous model generation loops (Models,AsyncModels,Chat, andAsyncChat).FunctionDeclarationTest,AfcUtilTest, andPrivateChatAfcTest.Closes #807