-
Notifications
You must be signed in to change notification settings - Fork 3k
MCP google/go-github@V79 update
#1407
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
Conversation
google/go-github@V79 update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR upgrades the google/go-github dependency from v77 to v79 to incorporate upstream ProjectsV2 API fixes that resolve field-selection and unmarshalling issues.
- Updated all import paths from
github.com/google/go-github/v77tov79 - Refreshed
go.modandgo.sumwith v79.0.0 checksums - Updated third-party license references across all platform-specific license files
Reviewed Changes
Copilot reviewed 46 out of 48 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Updated dependency from v77.0.0 to v79.0.0 |
| go.sum | Updated checksums for v79.0.0 |
| third-party/github.com/google/go-github/v79/github/LICENSE | Added LICENSE file for v79 |
| third-party-licenses.darwin.md | Updated license reference to v79 |
| third-party-licenses.linux.md | Updated license reference to v79 |
| third-party-licenses.windows.md | Updated license reference to v79 |
| pkg/raw/raw.go | Updated import path to v79 |
| pkg/raw/raw_test.go | Updated import path to v79 |
| pkg/github/tools.go | Updated import path to v79 |
| pkg/github/server.go | Updated import path to v79 |
| pkg/github/server_test.go | Updated import path to v79 |
| pkg/github/security_advisories.go | Updated import path to v79 |
| pkg/github/security_advisories_test.go | Updated import path to v79 |
| pkg/github/secret_scanning.go | Updated import path to v79 |
| pkg/github/secret_scanning_test.go | Updated import path to v79 |
| pkg/github/search_utils.go | Updated import path to v79 |
| pkg/github/search.go | Updated import path to v79 |
| pkg/github/search_test.go | Updated import path to v79 |
| pkg/github/repository_resource.go | Updated import path to v79 |
| pkg/github/repository_resource_test.go | Updated import path to v79 |
| pkg/github/repositories.go | Updated import path to v79 |
| pkg/github/repositories_test.go | Updated import path to v79 |
| pkg/github/pullrequests.go | Updated import path to v79 |
| pkg/github/pullrequests_test.go | Updated import path to v79 |
| pkg/github/projects.go | Updated import path to v79 |
| pkg/github/projects_test.go | Updated import path to v79 |
| pkg/github/notifications.go | Updated import path to v79 |
| pkg/github/notifications_test.go | Updated import path to v79 |
| pkg/github/minimal_types.go | Updated import path to v79 |
| pkg/github/issues.go | Updated import path to v79 |
| pkg/github/issues_test.go | Updated import path to v79 |
| pkg/github/git.go | Updated import path to v79 |
| pkg/github/gists.go | Updated import path to v79 |
| pkg/github/gists_test.go | Updated import path to v79 |
| pkg/github/discussions.go | Updated import path to v79 |
| pkg/github/discussions_test.go | Updated import path to v79 |
| pkg/github/dependabot.go | Updated import path to v79 |
| pkg/github/dependabot_test.go | Updated import path to v79 |
| pkg/github/context_tools_test.go | Updated import path to v79 |
| pkg/github/code_scanning.go | Updated import path to v79 |
| pkg/github/code_scanning_test.go | Updated import path to v79 |
| pkg/github/actions.go | Updated import path to v79 |
| pkg/github/actions_test.go | Updated import path to v79 |
| pkg/errors/error.go | Updated import path to v79 |
| pkg/errors/error_test.go | Updated import path to v79 |
| internal/ghmcp/server.go | Updated import path to v79 |
| e2e/e2e_test.go | Updated import path to v79 |
| cmd/github-mcp-server/generate_docs.go | Updated import path to v79 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tmelliottjr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks good to me! Thanks for taking care of this. I'll defer approving for codeowners.
SamMorrowDrums
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good.
Towards: https://github.com/github/memex/issues/20995
This PR updates our
google/go-githubdependency from v77 to v79 to incorporate the latest ProjectsV2 fixes released in v79.0.0 which includes the following bug fixes:Fieldproperty inUpdateProjectItemOptionshas changed from a single value to:[]*ProjectV2FieldUpdateThis aligns the client with the actual API contracts. As part of the update I have also taken care of:
github.com/google/go-github/v79.-
go.modandgo.sumhave been refreshed to pull in the correctedProjectsV2models.UpdateProjectItemOptions.Fieldhas been updated or validated to ensure compatibility with the new slice-based type.Why this upgrade matters
The upstream fixes directly address decoding issues seen when interacting with ProjectsV2 APIs. Moving to v79 ensures that the MCP server correctly parses the Project item payloads returned from GitHub and supports the updated mutation format for field updates.
Local Testing
The update has been tested against projects APIs already integrated with google/go-github, for reference see screenshots as part of this PR.
Click to expand image samples