Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

ChatCompletionsRequest drops MCP tool parameter schemas from parametersJsonSchema

未关闭
#1,426 2 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

维护者通常 1 天内回复

@hemasekhar-p 已经在做这个了。

开始于 2026年8月13日。

  • #1427 来自 @dangzitou —— 未关闭

评估

这个 Issue 还没有评估数据。

描述

needs review

🔴 Required Information

Describe the Bug:

ADK's native McpToolset is unusable with the native chat-completions connector for MCP tools that take parameters.

AbstractMcpTool declares MCP tool input schemas through FunctionDeclaration.parametersJsonSchema(...), but com.google.adk.models.chat.ChatCompletionsRequest reads only FunctionDeclaration.parameters().

When only the raw JSON schema is populated, the chat-completions path falls back to an empty parameter schema:

{
  "type": "object",
  "properties": {}
}

As a result, the model receives a parameterless tool declaration for an MCP tool that actually requires arguments and commonly calls the tool with {}.

This appears to be an incompatibility between two first-party core/ ADK components rather than a consumer configuration issue.

Steps to Reproduce:

  1. Create an MCP tool with a required parameter, for example:

    {
      "type": "object",
      "properties": {
        "jobId": {
          "type": "string"
        }
      },
      "required": ["jobId"]
    }
    
  2. Expose the MCP tool through ADK's native McpToolset / AbstractMcpTool.

  3. Use the tool with a model backed by ChatCompletionsHttpClient.

  4. Inspect the outbound /chat/completions request.

  5. Observe that the generated tool declaration contains an empty parameter schema instead of the MCP tool's JSON schema.

Expected Behavior:

The outbound OpenAI-compatible tool declaration should preserve the MCP tool's parameter schema, including its properties and required fields:

{
  "type": "function",
  "function": {
    "name": "analyze_premerge_failures_by_job",
    "parameters": {
      "type": "object",
      "properties": {
        "jobId": {
          "type": "string"
        }
      },
      "required": ["jobId"]
    }
  }
}

The model should therefore be able to identify jobId as a required argument and provide it when calling the tool.

Observed Behavior:

The outbound declaration contains an empty schema:

{
  "type": "function",
  "function": {
    "name": "analyze_premerge_failures_by_job",
    "parameters": {
      "type": "object",
      "properties": {}
    }
  }
}

The model cannot see that jobId exists and commonly emits an empty argument object:

{}

This prevents parameterized MCP tools from being used correctly through the chat-completions connector.


🟡 Optional Information

Regression:

No known regression has been established. The issue is observed with ADK 1.7.1.

Logs:

The relevant evidence is the outbound /chat/completions request. The tool declaration contains the following schema instead of the MCP-provided schema:

{
  "type": "object",
  "properties": {}
}

Screenshots / Video:

N/A.

Additional Context:

FunctionDeclaration supports two representations for function parameters:

  • typed parameters(Schema)
  • raw parametersJsonSchema(...)

AbstractMcpTool populates the raw parametersJsonSchema form for MCP tool input schemas. ChatCompletionsRequest currently consumes only the typed parameters(Schema) form.

We also observed the same raw-only declaration shape through an application-side MCP adapter, but the native McpToolset reproduction demonstrates that the issue does not depend on that adapter.

A closely related fix already exists for the Claude connector in PR #1342 / commit 760c8da. That fix forwards the raw schema directly, preserving JSON Schema constructs such as $ref and $defs.

OpenAI-compatible chat-completions APIs also accept raw JSON Schema, so forwarding parametersJsonSchema directly appears preferable to converting it into the narrower typed Schema representation.

Impact:

Any parameterized MCP tool is affected when used through the chat-completions connector. In our case, this blocks:

  • a tool accepting jobId
  • parameterized architecture-analysis MCP tools

Zero-argument tools continue to work because their empty schema is valid.

Temporary Workaround:

Until ADK is fixed, we materialize the raw schemas into typed parameters(Schema) in our BaseLlm implementation before delegating to ChatCompletionsHttpClient.

This workaround is connector-local and is covered by a wire-level regression test.

Minimal Reproduction Code:

A minimal reproduction should consist of:

// Create an MCP tool whose input schema contains a required parameter,
// expose it through the native McpToolset, and invoke it through
// ChatCompletionsHttpClient.
//
// The outbound /chat/completions request contains:
// {
//   "type": "object",
//   "properties": {}
// }
// instead of the MCP tool's declared schema.

The important distinction is that the MCP tool declaration populates parametersJsonSchema(...), while ChatCompletionsRequest currently reads parameters().

How often has this issue occurred?:

  • Always (100%)
主要语言
Java
星标
1.7k
派生
433
平均合并
3 天 13 小时
30 天内合并 PR
42

环境准备

在 Codespaces 中打开

在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

google/adk-java 的其他 Issue

查看 google/adk-java 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。