-
Notifications
You must be signed in to change notification settings - Fork 557
Add mcp-client-go #62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
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. |
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.
