[FEATURE] Port bypass_multi_tools_limit for built-in search tools from adk-python
メンテナーはふだん 1 日以内に返信
@hirematha がすでに取り組んでいます。
2026年10月5日 から。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 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
- 平均マージ
- 3日 13時間
- マージ済み PR(30日)
- 42
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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 が 3 日前に担当しました。 オープンneeds review
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
google/adk-java#1628 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
[spring-ai] ToolConverter silently drops enum and items from tool parameter schemas対応中かも @hirematha が 5 日前に担当しました。 オープンneeds review
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
google/adk-java#1609 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
[spring-ai] Streaming responses ending with CJK punctuation (。!?) are misclassified as partial and never persisted to the session対応中かも @hirematha が 5 日前に担当しました。 オープンneeds review
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
google/adk-java#1608 · コメント 3 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
FirestoreSessionService compares event timestamps as text, reordering or skipping same-second eventsオープン
難易度 3/5 1〜2日 初心者へのやさしさ 50/100
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
メンテナーはふだん 1 日以内に返信
google/adk-java の issue をすべて見る
似ている issue
-
Clock.MakeDate continues execution and returns a rolled-over instant after dispatching error on invalid date対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
mit-cml/appinventor-sources#4155 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 62/100
Hira-shi/PW1-DAI-Carrel-Egal-Eyer#28 ·
メンテナーはふだん 1 日以内に返信
-
`GET /v1/event/token/{uuid}` can report a BOM upload as done before policy evaluation and metrics have finished対応中かも @Zargath が今日担当しました。 オープンdefect in triage
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
DependencyTrack/dependency-track#7646 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
floci-io/floci#5425 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
objectionary/eo-graphs#80 ·