[FEATURE] Port bypass_multi_tools_limit for built-in search tools from adk-python
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- java
- Ambito
- backend-api-design
Direzione di ricerca
Start with AgentTool.runAsync and the related work in #1563, then trace LlmAgent.canonicalTools and BaseLlmFlow.getRequestProcessorFromTools to understand where tool replacement must apply. Review GoogleSearchTool, VertexAiSearchTool, GoogleSearchAgentTool, and VertexAiSearchAgentTool alongside the scratch JUnit/TestLlm scenario. Done means the agreed opt-in behavior, grounding propagation, API shape, and tests cover the selected Google and Vertex AI cases.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
🔴 Required Information
Is your feature request related to a specific problem?
On the Java engine, a built-in search tool has to be the only tool on its agent when the agent runs through runAsync. On main (2d4a59e), nothing handles that case for you:
GoogleSearchToolnext to a function tool, or on an agent with transfer targets (sub-agents, or a parent or peers, which addtransfer_to_agent), goes into the same request as those function declarations. google/adk-python@f3250bd reports that Gemini rejects such a request with400 INVALID_ARGUMENT: Tool use with function calling is unsupported. Documented exceptions include the Live API and, forgenerateContent, the Gemini 3 tool-combination preview (see Alternatives); ADK Java'srunAsyncpath uses neither.GoogleSearchAgentToolandVertexAiSearchAgentTool(added in #692) wrap the search in a nested agent, but nothing creates them for you. They also return only the nested agent's text:AgentTool.runAsyncturns the last event into{"result": text}, so the grounding metadata from the search (queries, sources, and for Google Search thesearchEntryPointwith the Search Suggestions) never reaches the caller.
adk-python handles both behind an opt-in flag:
GoogleSearchTool(bypass_multi_tools_limit=True)andVertexAiSearchTool(..., bypass_multi_tools_limit=True)(google/adk-python@9a6b850; off by default since google/adk-python@6da7274).- When the agent has more than one tool or any transfer target,
LlmAgent.canonical_toolsreplaces a flaggedGoogleSearchToolwithGoogleSearchAgentTool, and a flaggedVertexAiSearchToolwithDiscoveryEngineSearchTool. GoogleSearchAgentToolhas carried the nested agent's grounding metadata to the caller's next model response since it was added (google/adk-python@d3148da); google/adk-python@d689a04 moved that intoAgentToolbehind an opt-inpropagate_grounding_metadata, whichGoogleSearchAgentToolsets toTrue. The Java port of the wrapper in #692 does not carry it.- For a search tool without the flag on an agent with transfer targets, the agent-transfer processor raises a
ValueErrorthat names the flag when a sub-agent is a target, and otherwise leaves outtransfer_to_agent(google/adk-python@f3250bd).
ADK Kotlin has the same flag (GoogleSearchTool(bypassMultiToolsLimit = true), with a builder for Java callers). When the agent's tools and toolsets add up to more than one (transfer targets are not counted), it replaces flagged tools with GoogleSearchAgentTool and VertexAiSearchAgentTool, both created with propagateGroundingMetadata = true.
Describe the Solution You'd Like
The same opt-in behavior on the Java engine, in three steps that can be reviewed separately:
- Grounding propagation. An opt-in on
AgentToolthat carries the grounding metadata of the nested agent's last content event back to the caller, turned on inGoogleSearchAgentToolandVertexAiSearchAgentTool. adk-python stores it undertemp:_adk_grounding_metadataand attaches it to the caller's next model response; ADK Kotlin writes it under the same key into the state delta of the caller's function response event. - Google Search. A
bypassMultiToolsLimitoption onGoogleSearchTool, off by default, withINSTANCEunchanged. When the agent has more than one tool or any transfer target, a flaggedGoogleSearchToolis replaced withGoogleSearchAgentTool.create(model)in the tool list that requests are built from. Onmain,BaseLlmFlow.getRequestProcessorFromToolsreadsLlmAgent.toolsUnion()directly, so the replacement has to reach that path as well asLlmAgent.canonicalTools. The search agent's instruction would also change the way google/adk-python@56d3cae changed it, so that the model answers with its built-in grounding instead of emitting a client-side function call (google:search). - Vertex AI Search. The same option on
VertexAiSearchTool.
Questions before I start:
- Is the Java engine the right place for this, or is the ADK Kotlin engine through
toktthe intended path for Java apps that need it? (See Alternatives for what that path covers today.) - For Vertex AI Search, should the replacement be the existing
VertexAiSearchAgentTool(as in ADK Kotlin: no new dependency, and the search stays the model's built-in retrieval) or a port ofDiscoveryEngineSearchTool(as in adk-python)? google/adk-python#7100 lists problems with the second approach. - For a search tool without the flag on an agent with transfer targets, do you want adk-python's behavior in Java (the
ValueError, or leaving outtransfer_to_agent), or should that stay out of scope for now? - For the option itself, a builder (
GoogleSearchTool.builder().bypassMultiToolsLimit(true).build(), like the one ADK Kotlin exposes to Java) or a constructor argument?
Impact on your work
On the Java engine, agents that need Google Search or Vertex AI Search together with function tools or sub-agents have to be wired by hand today, and they lose the search sources on the way. For Google Search, that includes the searchEntryPoint that carries the Search Suggestions the Gemini API terms ask apps to show alongside grounded results. No hard timeline on my side.
Willingness to contribute
Yes. Once the questions above are settled, I would start with step 1. It also has to wait for #1563, since both change AgentTool.runAsync.
🟡 Recommended Information
Describe Alternatives You've Considered
- Wrapping by hand with
GoogleSearchAgentTool.create(model)orAgentTool.create(searchAgent). Then no request mixes the built-in tool with function declarations, but the grounding metadata is dropped (see below), and users first have to find the wrapper. #550 asked how to use Google Search in a multi-agent setup. The limitations page in the ADK docs showsAgentTool.create(...)for Java and tags thebypass_multi_tools_limitworkaround as supported in Java, which it is not yet. - Running on the ADK Kotlin engine through
tokt. Since 1.10.0 (on Maven Central from 1.10.1), thegoogle-adk-toktmodule lets a Java app run agents on the ADK Kotlin engine, where ADK Kotlin'sGoogleSearchTool.builder().bypassMultiToolsLimit(true).build()is available. That means rebuilding the agent tree on the Kotlin engine (toktadapts tools, toolsets, plugins, models and services, not whole Java agents). ADK Kotlin's replacement also does not count transfer targets, and its transfer processor addstransfer_to_agentwithout checking for built-in tools, so the sub-agent case from #550 would still send both. This request is about the Java engine. - Gemini 3 tool combination. The Gemini Developer API (
generateContent) accepts built-in tools together with function declarations for Gemini 3 models (released on 2026-03-18 and still marked Preview; it needstool_config.include_server_side_tool_invocations, and thetoolCall/toolResponseparts have to be sent back on every turn). That could replace the workaround later, but it is Preview and Gemini 3 only, and adk-python keeps the workaround: google/adk-python@deabb2a (2026-07-08) says "The built-in tools still cannot be combined with other tools, so the workaround stays", and google/adk-python@f3250bd and google/adk-python@56d3cae changed the workaround after that. Because the flag is opt-in, it would be easy to retire.
Proposed API / Implementation
LlmAgent root =
LlmAgent.builder()
.name("root")
.model("gemini-flash-latest")
.tools(
// or a constructor argument; see the questions above
GoogleSearchTool.builder().bypassMultiToolsLimit(true).build(),
FunctionTool.create(WeatherTools.class, "getWeather"))
.subAgents(bookingAgent)
.build();
// Requests from root declare google_search_agent (GoogleSearchAgentTool), getWeather and
// transfer_to_agent, with no built-in googleSearch tool. After a search, the root agent's next
// model response carries the search agent's grounding metadata.
Additional Context
What main (2d4a59e) does today, from a scratch JUnit test with TestLlm (no network):
| Setup | Result |
|---|---|
GoogleSearchTool.INSTANCE + one sub-agent |
One request with the googleSearch tool and the transfer_to_agent declaration |
GoogleSearchTool.INSTANCE + a function tool |
Both in one request; nothing is replaced |
Root agent with GoogleSearchAgentTool + a function tool; the search agent's model response has grounding metadata |
The function response is {"result": <text>}, and none of the root agent's events has grounding metadata |
| The search agent run on its own | Its final event keeps the grounding metadata, so it is lost in AgentTool |
The test covers only the sub-agent case. An agent whose only transfer targets are an LlmAgent parent or its peers gets transfer_to_agent from the same AgentTransfer.processRequest.
The grounding case from that test
GroundingMetadata grounding =
GroundingMetadata.builder().webSearchQueries(ImmutableList.of("adk java")).build();
TestLlm searchLlm =
createTestLlm(
LlmResponse.builder()
.content(Content.builder().role("model").parts(Part.fromText("ADK Java is a toolkit")).build())
.groundingMetadata(grounding)
.build());
TestLlm rootLlm =
createTestLlm(
createFunctionCallLlmResponse(
"call-1", "google_search_agent", ImmutableMap.of("request", "adk java")),
createTextLlmResponse("final answer"));
LlmAgent root =
createTestAgentBuilder(rootLlm)
.name("root")
.tools(GoogleSearchAgentTool.create(searchLlm), new EchoTool())
.build();
// Run root through an InMemoryRunner with "what is adk java?", then:
// - the function response is {"result": "ADK Java is a toolkit"}
// - no event from the root agent has groundingMetadata()
Across ADK languages: adk-python and ADK Kotlin have the flag and the replacement; adk-js has the flag on VertexAiSearchTool only, to skip its Gemini 1.x check; adk-go has neither.
Related: #550, #692, #387.
- Lingua principale
- Java
- Stelle
- 1.7k
- Fork
- 431
- Merge medio
- 2g 20h
- PR unite (30g)
- 42
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di google/adk-java
-
[spring-ai] ToolConverter silently drops enum and items from tool parameter schemasForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
google/adk-java#1609 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
[spring-ai] Streaming responses ending with CJK punctuation (。!?) are misclassified as partial and never persisted to the sessionForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
google/adk-java#1608 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
[spring-ai] Bridge drops reasoning_content (thinking) — surface it as partial events and/or persist itForse già presa @hemasekhar-p l’ha presa 1 giorno fa. Apertaneeds review
google/adk-java#1616 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
BaseLlmFlow nests each step inside the previous one and overflows the stack after a few hundred LLM callsForse già presa @hemasekhar-p l’ha presa 8 giorni fa. Apertaneeds review
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
google/adk-java#1564 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Approved tool call re-runs on every later user turn if it never got a function responseForse già presa @hemasekhar-p l’ha presa 11 giorni fa. Apertaneeds review
google/adk-java#1556 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di google/adk-java
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
-
Make branch and label autocomplete matching locale-independentForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
jenkinsci/gitlab-plugin#1950 ·
-
It's not necessary to copy the memory block in the readWrite() of org.h2.store.fs.mem.FileMemDataAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
h2database/h2database#4435 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
micronaut-projects/micronaut-core#13717 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ADORSYS-GIS/token-status-link#145 ·
I maintainer di solito rispondono entro 3 giorni