Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

MCP server updates usinggoogle/go-github APIs#1408

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@stephenotalora
Copy link
Contributor

@stephenotalorastephenotalora commentedNov 14, 2025
edited
Loading

This PR builds upon and directly depends on the updates introduced in#1407, which upgraded the project’s dependency togoogle/go-github@V79. The upgrade toV79 resolves fundamental issues related to data unmarshalling from Github’s ProjectsV2 RESTful API, ensuring structured and accurate decoding of API responses and laying the groundwork for the API updates in this PR.

Context

With thego-github@V79 upgrade, enables support for new and updated ProjectsV2 API endpoints, JSON unmarshalling across different resource types, and compatibility with recent API endpoints.

Key Changes

  • Updates to API Layer: Aligns MCP server tools for Projects to match the latestProjectsV2 RESTful API contracts.
  • Fixes Data Unmarshalling: Leverages improvements in the upgraded dependency to process incomingProjectsV2 data correctly.

Dependency

Note: This PR must be merged after#1407 as it requires the fixes and new capabilities introduced in thego-github@V79 upgrade.

Testing

  • No immediate test updates are required for the commits in this PR.
  • The existing tests are robust because they focus on the same data contracts that is provided by this upgrade and for this reason there are no updates to existing test suites.
  • Manual validation performed against the current ProjectsV2 REST API to confirm fidelity of data transformations and absence of regressions.
  • For more details on manual testing please follow along.

Click to expand local testing

Org testing:

Screenshot 2025-11-14 at 9 56 00 AMScreenshot 2025-11-14 at 9 58 56 AMScreenshot 2025-11-14 at 10 16 17 AMScreenshot 2025-11-14 at 10 21 39 AMScreenshot 2025-11-14 at 10 22 16 AMScreenshot 2025-11-14 at 10 23 24 AM

User testing:

Screenshot 2025-11-14 at 10 58 00 AMScreenshot 2025-11-14 at 10 59 28 AMScreenshot 2025-11-14 at 10 59 52 AMScreenshot 2025-11-14 at 11 00 33 AMScreenshot 2025-11-14 at 11 28 07 AMScreenshot 2025-11-14 at 11 34 36 AMScreenshot 2025-11-14 at 11 35 30 AMScreenshot 2025-11-14 at 11 36 21 AM

@stephenotalorastephenotalora changed the base branch frommain tostephenotalora/mcp-go-github-v79November 14, 2025 17:26
@stephenotalorastephenotalora self-assigned thisNov 14, 2025
@stephenotalorastephenotalora changed the titleStephenotalora/mcp server latest api updates v2MCP server updates usinggoogle/go-github APIsNov 14, 2025
@stephenotalorastephenotalora marked this pull request as ready for reviewNovember 14, 2025 18:39
@stephenotalorastephenotalora requested a review froma team as acode ownerNovember 14, 2025 18:39
CopilotAI review requested due to automatic review settingsNovember 14, 2025 18:39
Copilot finished reviewing on behalf ofstephenotaloraNovember 14, 2025 18:42
Copy link
Contributor

CopilotAI left a 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 updates the GitHub MCP Server to use nativegoogle/go-github v79 API methods for ProjectsV2 operations, replacing custom HTTP request handling with proper SDK integration. This change depends on the go-github v79 upgrade introduced in PR#1407, which fixed critical data unmarshalling issues.

Key changes:

  • Replaces manual HTTP requests and custom request building with go-github SDK API methods
  • Removes custom type definitions in favor of using types from the go-github library
  • Updatesitem_id parameter handling to useRequiredBigInt for safer int64 conversion

💡Add Copilot custom instructions for smarter, more guided reviews.Learn how to get started.

Base automatically changed fromstephenotalora/mcp-go-github-v79 tomainNovember 15, 2025 21:07
SamMorrowDrums
SamMorrowDrums previously approved these changesNov 15, 2025
Copy link
Collaborator

@SamMorrowDrumsSamMorrowDrums left a comment

Choose a reason for hiding this comment

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

Seems much cleaner

@SamMorrowDrums
Copy link
Collaborator

Merge conflicts prevent me from actually merging (rebase of dependency upgrade PR onto main is likely cause), and probably can be resolved by rebasing this PR against main.

stephenotalora reacted with thumbs up emoji

@stephenotalora
Copy link
ContributorAuthor

conflicts addressed ✅

@SamMorrowDrumsSamMorrowDrums merged commite95d8ee intomainNov 16, 2025
16 checks passed
@SamMorrowDrumsSamMorrowDrums deleted the stephenotalora/mcp-server-latest-api-updates-v2 branchNovember 16, 2025 22:42
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@SamMorrowDrumsSamMorrowDrumsSamMorrowDrums approved these changes

@tmelliottjrtmelliottjrAwaiting requested review from tmelliottjr

Assignees

@stephenotalorastephenotalora

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stephenotalora@SamMorrowDrums

[8]ページ先頭

©2009-2025 Movatter.jp