Skip to content

fix(mcp_tool): Temporarily disable Bound Token usage for mcp_tool#5372

Closed
andyrzhao wants to merge 3 commits intogoogle:mainfrom
andyrzhao:main
Closed

fix(mcp_tool): Temporarily disable Bound Token usage for mcp_tool#5372
andyrzhao wants to merge 3 commits intogoogle:mainfrom
andyrzhao:main

Conversation

@andyrzhao
Copy link
Copy Markdown

Link to Issue or Description of Change

#5361

Testing Plan

Manual End-to-End (E2E) Tests:

We plan to manually verify this fix with an mcp tool deployment on Agent Engine. (An identical temporary patch was made for Gen-AI SDK previously and efficacy verified.)

@adk-bot
Copy link
Copy Markdown
Collaborator

adk-bot commented Apr 17, 2026

Response from ADK Triaging Agent

Hello @andyrzhao, thank you for creating this PR!

I've noticed that the pyink-check has failed. Could you please run the auto-formatter script to fix the code formatting? You can do this by running the following command from the root of the repository:

./autoformat.sh

This will help ensure that the code adheres to the project's coding style. Thanks!

@adk-bot adk-bot added the mcp [Component] Issues about MCP support label Apr 17, 2026
@rohityan rohityan self-assigned this Apr 20, 2026
@rohityan
Copy link
Copy Markdown
Collaborator

Hi @andyrzhao, Thank you for your contribution through this pull request! This PR has merge conflicts that require changes from your end. Could you please rebase your branch with the latest main branch to address these? Once this is complete, please let us know so we can proceed with the review.

@rohityan rohityan added the request clarification [Status] The maintainer need clarification or more information from the author label Apr 20, 2026
@sasha-gitg sasha-gitg closed this May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mcp [Component] Issues about MCP support request clarification [Status] The maintainer need clarification or more information from the author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants