Add in Go snippets for MCP tools #901
Conversation
koverholt
left a comment
koverholt
left a comment
There was a problem hiding this comment.
Thanks for adding the Go samples and reformatting existing Python / Java samples.
Agree that this page needs refactoring to show permutations of MCP config rather than the longer serial explanations w/ tutorial steps that are there now. So thanks for getting the samples in place even in this existing format.
Future
Future versions of this page and our third-party tools page (example) should show more densely nested code tabs in a much more concise way without having to scroll, such as:
- Python
- Local MCP (stdio)
- Remote MCP (http)
- Remote MCP (sse)
- Java
- Local MCP (stdio)
- Remote MCP (http)
- Remote MCP (sse)
- Go
- Local MCP (stdio)
- Remote MCP (http)
- Remote MCP (sse)
while splitting the rest of the content on this page into separate pages under Custom Tools > MCP > .... That way users can quickly get to the sample code they need for a given MCP scenario via the nested tabs, or optionally dive deeper into more detailed topics around MCP on other pages.
Now
Approving this as a solid intermediate step to get the Go samples up! 😁 And would love to have an additional review by @joefernandez.
docs/tools/mcp-tools.md
Outdated
|
|
||
| * **Set up ADK:** Follow the standard ADK [setup instructions](../get-started/quickstart.md/#venv-install) in the quickstart. | ||
| * **Install/update Python/Java:** MCP requires Python version of 3.9 or higher for Python or Java 17 or higher. | ||
| * **Install/update Python/Java/Go:** MCP requires Python version of 3.9 or higher for Python, Java 17 or higher, or Go 1.22 or higher. |
There was a problem hiding this comment.
Order: Move Go between Py and Java.
| For Go, you can create an `LlmAgent` with an `MCPToolset` to connect to the file system server. This example shows a self-contained application that initializes the agent and runs it with a sample prompt. | ||
|
|
||
| === "Go" | ||
| ```go |
There was a problem hiding this comment.
Can you move to a go file so we can verify compilation and functionality?
|
Approved. Feel free to merge this and handle the comments in a follow-on change. |
There was a problem hiding this comment.
HEADS-UP: this file is MOVING:
docs/tools/mcp-tools.md to--> docs/tools-custom/mcp-tools.md
4 participants
Updates:
Note: @joefernandez this page likely needs a much larger refactoring, if we want to migrate towards the tab structure we have on other pages. PTAL at how I modified the existing Python/Java and see if you'd prefer that updated differently.