Skip to content

Conversation

@amikai
Copy link
Contributor

@amikai amikai commented Oct 28, 2025

Add rust mcp-client-go example in quickstart-resources.

How Has This Been Tested?

Directly run the client example with server example in the repo.
CleanShot 2025-10-29 at 01 05 25@2x

@amikai
Copy link
Contributor Author

amikai commented Jan 19, 2026

Hi @a-akimov, do you plan to review this PR? If not, I’ll close it.

@a-akimov
Copy link
Contributor

Hi @a-akimov, do you plan to review this PR? If not, I’ll close it.

Hi @amikai , apologies for the delay. Please give me a few days to review and approve it, with a deadline by the end of this week. Thank you for your contribution!

@amikai
Copy link
Contributor Author

amikai commented Jan 19, 2026

thanks @a-akimov

@a-akimov a-akimov self-assigned this Jan 25, 2026
@a-akimov
Copy link
Contributor

@claude review this PR and provide your comments.

@maciej-kisiel
Copy link

From the Go SDK perspective: is this code going to be published at https://modelcontextprotocol.io/docs/develop/build-client? If so, I think it could be polished a bit. In particular, the processQuery function has a slightly different structure than in other languages (processing all tool calls at the end instead of one at a time). In some places it also checks additional edge cases (e.g. "is query empty?") which would be good for production code, but for examples it may give an impression that Go is much more verbose than other languages and I believe we should strive for simplicity here.

@amikai
Copy link
Contributor Author

amikai commented Jan 28, 2026

Hi @maciej-kisiel, I’ll use the code to create the PR to publish it at https://modelcontextprotocol.io/docs/develop/build-client. My opinion is that although the code may become more verbose, it’s still better than surprising the user. @a-akimov Do you think this makes sense?

@a-akimov
Copy link
Contributor

@maciej-kisiel thank you for looking into this! Could you perhaps give a few examples what you would recommend simpliyfing? Personally I agree that we should try to keep quick start examples simple for educational purposes, so that people are focused on learning the basics of MCP; however there is also a fine line between what's enough and what's not enough. So, some concrete examples from you can be very helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants