Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

[FEATURE] Port bypass_multi_tools_limit for built-in search tools from adk-python

Ouverte
#1,598 1 commentaire 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Active
Stack technique
java

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

needs review

🔴 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:

  • GoogleSearchTool next to a function tool, or on an agent with transfer targets (sub-agents, or a parent or peers, which add transfer_to_agent), goes into the same request as those function declarations. google/adk-python@f3250bd reports that Gemini rejects such a request with 400 INVALID_ARGUMENT: Tool use with function calling is unsupported. Documented exceptions include the Live API and, for generateContent, the Gemini 3 tool-combination preview (see Alternatives); ADK Java's runAsync path uses neither.
  • GoogleSearchAgentTool and VertexAiSearchAgentTool (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.runAsync turns the last event into {"result": text}, so the grounding metadata from the search (queries, sources, and for Google Search the searchEntryPoint with the Search Suggestions) never reaches the caller.

adk-python handles both behind an opt-in flag:

  • GoogleSearchTool(bypass_multi_tools_limit=True) and VertexAiSearchTool(..., 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_tools replaces a flagged GoogleSearchTool with GoogleSearchAgentTool, and a flagged VertexAiSearchTool with DiscoveryEngineSearchTool.
  • GoogleSearchAgentTool has 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 into AgentTool behind an opt-in propagate_grounding_metadata, which GoogleSearchAgentTool sets to True. 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 ValueError that names the flag when a sub-agent is a target, and otherwise leaves out transfer_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:

  1. Grounding propagation. An opt-in on AgentTool that carries the grounding metadata of the nested agent's last content event back to the caller, turned on in GoogleSearchAgentTool and VertexAiSearchAgentTool. adk-python stores it under temp:_adk_grounding_metadata and 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.
  2. Google Search. A bypassMultiToolsLimit option on GoogleSearchTool, off by default, with INSTANCE unchanged. When the agent has more than one tool or any transfer target, a flagged GoogleSearchTool is replaced with GoogleSearchAgentTool.create(model) in the tool list that requests are built from. On main, BaseLlmFlow.getRequestProcessorFromTools reads LlmAgent.toolsUnion() directly, so the replacement has to reach that path as well as LlmAgent.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).
  3. 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 tokt the 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 of DiscoveryEngineSearchTool (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 out transfer_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) or AgentTool.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 shows AgentTool.create(...) for Java and tags the bypass_multi_tools_limit workaround 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), the google-adk-tokt module lets a Java app run agents on the ADK Kotlin engine, where ADK Kotlin's GoogleSearchTool.builder().bypassMultiToolsLimit(true).build() is available. That means rebuilding the agent tree on the Kotlin engine (tokt adapts tools, toolsets, plugins, models and services, not whole Java agents). ADK Kotlin's replacement also does not count transfer targets, and its transfer processor adds transfer_to_agent without 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 needs tool_config.include_server_side_tool_invocations, and the toolCall/toolResponse parts 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.

Langage dominant
Java
Étoiles
1.7k
Forks
431
Merge moyen
2 j 20 h
PR mergées (30 j)
42

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de google/adk-java

Toutes les issues de google/adk-java

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.