feat: add mcp-tools-java.md #544
Conversation
There was a problem hiding this comment.
I don't think we can just merge this as-is ... because this appears to just copy/paste a fair bit of content from https://github.com/wenhaozhao/adk-docs/blob/f2e9128c7c7fd2287c007c7a0fb598dc7c7ef0ca/docs/tools/mcp-tools.md (which is https://google.github.io/adk-docs/tools/mcp-tools/), without even adding it to the navigation. My understanding of how adk-docs are managed is that we use the same MD for Python & Java, and then have language specific sections with those tabs on them? @wenhaozhao would you be willing to update this accordingly?
|
@wenhaozhao I just noticed that (meanwhile) on https://google.github.io/adk-docs/tools/mcp-tools/ there actually is a "For Java, refer to the following sample to define an agent that initializes the MCPToolset" would you perhaps like to simply enhance that and add how to configure timeouts for MCP server, introduced in google/adk-java#272? |
|
ok, i'll update this later |
wenhaozhao
force-pushed
the
feat-mcp_tools_java
branch
from
f2e9128 to
15e0757
Compare
There was a problem hiding this comment.
LGTM. PS: @wenhaozhao you could [separately, also] contribute this to our (brand new) https://github.com/google/adk-java/tree/main/tutorials BTW, if you like ... just a thought.
cool!!! |
Add mcp-tools-java.md to Introduce How to Configure MCP-Tools in ADK-Java