[FEATURE] Port bypass_multi_tools_limit for built-in search tools from adk-python
维护者通常 1 天内回复
@hirematha 已经在做这个了。
开始于 2026年10月5日。
评估
- 难度
- 5/5
- 预计耗时
- 一周以上
- 新手友好度
- 35/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- java
调研方向
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.
由索引模型根据 Issue 内容生成。
描述
🔴 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.
- 主要语言
- Java
- 星标
- 1.7k
- 派生
- 433
- 平均合并
- 4 天 14 小时
- 30 天内合并 PR
- 62
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
google/adk-java 的其他 Issue
-
GeminiUtil placeholder user turn ("Continue output. DO NOT look at this line ...") is flagged by prompt injection filters可能已有人在做 @hemasekhar-p 于 8 天前认领。 未关闭needs review
难度 2/5 1-3 小时 新手友好度 76/100
google/adk-java#1628 · 1 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
[spring-ai] Streaming responses ending with CJK punctuation (。!?) are misclassified as partial and never persisted to the session可能已有人在做 @hirematha 于 11 天前认领。 未关闭needs review
难度 2/5 1-3 小时 新手友好度 84/100
google/adk-java#1608 · 3 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
VertexAiSessionService: Event.id differs between the streamed event and getSession (id is not sent to Vertex and the appendEvent response is empty)可能已有人在做 @hemasekhar-p 于 3 天前认领。 未关闭waiting on reporter
难度 4/5 3-5 天 新手友好度 45/100
google/adk-java#1651 · 2 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
runLive passes partial events to appendEvent and never aggregates text chunks, unlike ADK Python可能已有人在做 @hemasekhar-p 于 4 天前认领。 未关闭needs review
难度 4/5 3-5 天 新手友好度 45/100
google/adk-java#1648 · 3 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
FirestoreSessionService compares event timestamps as text, reordering or skipping same-second events可能已有人在做 @hemasekhar-p 于 4 天前认领。 未关闭needs review
难度 3/5 1-2 天 新手友好度 50/100
google/adk-java#1646 · 3 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
相似的 Issue
-
Triage
难度 2/5 1-3 小时 新手友好度 75/100
MinecraftForge/MinecraftForge#10902 · 1 条评论 ·
-
bug
难度 2/5 1-3 小时 新手友好度 74/100
refinedmods/refinedstorage2#1421 ·
-
StepExamBenchmarkTest runs a loop and prints a timing but asserts nothing可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭jules
难度 1/5 1 小时以内 新手友好度 82/100
open-learning-exchange/myplanet#18199 · 1 条评论 ·
维护者通常 4 天内回复
-
难度 2/5 1-3 小时 新手友好度 65/100
维护者通常 1 天内回复
-
Scheme-less stream URLs throw NullPointerException instead of URISyntaxException可能已有人在做 @blihio 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 82/100
pedroSG94/RootEncoder#2218 ·
维护者通常 3 天内回复