Skip to content

Conversation

huangjeff5
Copy link
Contributor

@huangjeff5 huangjeff5 commented Sep 25, 2025

Adding a Go example with official Github MCP server. This demo shows that you can easily build an agent that leverages domain-specific tools through an MCP server.

Also fixed Gemini plugin issues along the way:

  1. MCP server returns tool input JSON schema where minItems, maxItems, minimum, maximum => are already numbers, but gemini plugin expects strings in the JSON schema and casts to int64/float64. Current plugin implementation is brittle and panics, we handle all possible cases explicitly and use the most precise type.
  2. Related issue was happening to "required", "propertyOrdering", "enum" fields. gemini plugin expects []any, but they are actually typed as []string. Handle both cases explicitly.

@huangjeff5 huangjeff5 requested review from apascal07 and hugoaguirre and removed request for apascal07 September 25, 2025 18:52
@huangjeff5 huangjeff5 marked this pull request as ready for review September 25, 2025 18:52
@github-actions github-actions bot added docs Improvements or additions to documentation go fix labels Sep 25, 2025
Added error logging for missing GITHUB_MCP_CMD and PR number.
@huangjeff5
Copy link
Contributor Author

Updated pull request flag description and refined instructions.
@apascal07 apascal07 changed the title feat(go): Add Go MCP PR explainer example and safeguard Gemini fix(go): add Go MCP PR explainer example and safeguard Gemini Sep 29, 2025
@apascal07 apascal07 merged commit 292c839 into main Sep 29, 2025
6 checks passed
@apascal07 apascal07 deleted the jh-mcp-fixes branch September 29, 2025 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation fix go
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants