Conversation
|
Hi @a-akimov, do you plan to review this PR? If not, I’ll close it. |
|
thanks @a-akimov |
|
@claude review this PR and provide your comments. |
|
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 |
|
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? |
|
@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. |
|
Hi @maciej-kisiel, let me merge this one, and if you have some further suggestions how to simplify it, please feel free to submit subsequent PRs to this repo. Thank you very much for looking into this, highly appreciated! |
|
Hi @amikai, thank you very much for your contribution! I am merging it now, and please feel free to submit a doc guide to the main MCP repo. |
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.
