Skip to content

Conversation

@amikai
Copy link
Contributor

@amikai amikai commented Oct 25, 2025

Add the go MCP server example in quickstart-resources

How Has This Been Tested?

Use the MCP inspector to verify it.

list_tool:
list_tool

get_alert tool:
CleanShot 2025-10-26 at 00 22 18@2x

get_forecast tool:
CleanShot 2025-10-26 at 00 25 33@2x

@amikai amikai force-pushed the weather-server-go branch from 4991352 to 80ba22e Compare October 25, 2025 16:36
@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!

@a-akimov a-akimov self-requested a review January 25, 2026 15:13
Copy link

@maciej-kisiel maciej-kisiel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from the Go SDK perspective!

req.Header.Set("User-Agent", UserAgent)
req.Header.Set("Accept", "application/geo+json")

client := &http.Client{}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: use http.DefaultClient?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx, done

@a-akimov
Copy link
Contributor

@amikai @maciej-kisiel thank you! Let me merge it now, and if you have more things to improve in this example, please submit them in further PRs.

@a-akimov a-akimov merged commit e0ab645 into modelcontextprotocol:main Jan 28, 2026
1 check passed
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