Skip to content

Conversation

@stephenotalora
Copy link
Contributor

@stephenotalora stephenotalora commented Nov 14, 2025

This PR builds upon and directly depends on the updates introduced in #1407, which upgraded the project’s dependency to google/go-github@V79. The upgrade to V79 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 the go-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 latest ProjectsV2 RESTful API contracts.
  • Fixes Data Unmarshalling: Leverages improvements in the upgraded dependency to process incoming ProjectsV2 data correctly.

Dependency

Note: This PR must be merged after #1407 as it requires the fixes and new capabilities introduced in the go-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 AM Screenshot 2025-11-14 at 9 58 56 AM Screenshot 2025-11-14 at 10 16 17 AM Screenshot 2025-11-14 at 10 21 39 AM Screenshot 2025-11-14 at 10 22 16 AM Screenshot 2025-11-14 at 10 23 24 AM

User testing:

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

@stephenotalora stephenotalora changed the base branch from main to stephenotalora/mcp-go-github-v79 November 14, 2025 17:26
@stephenotalora stephenotalora self-assigned this Nov 14, 2025
@stephenotalora stephenotalora changed the title Stephenotalora/mcp server latest api updates v2 MCP server updates using google/go-github APIs Nov 14, 2025
@stephenotalora stephenotalora marked this pull request as ready for review November 14, 2025 18:39
@stephenotalora stephenotalora requested a review from a team as a code owner November 14, 2025 18:39
Copilot AI review requested due to automatic review settings November 14, 2025 18:39
Copilot finished reviewing on behalf of stephenotalora November 14, 2025 18:42
Copy link
Contributor

Copilot AI 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 native google/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
  • Updates item_id parameter handling to use RequiredBigInt for safer int64 conversion

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

Base automatically changed from stephenotalora/mcp-go-github-v79 to main November 15, 2025 21:07
SamMorrowDrums
SamMorrowDrums previously approved these changes Nov 15, 2025
Copy link
Collaborator

@SamMorrowDrums SamMorrowDrums 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 stephenotalora force-pushed the stephenotalora/mcp-server-latest-api-updates-v2 branch from 2bdd94c to a1297e1 Compare November 16, 2025 20:58
@stephenotalora
Copy link
Contributor Author

conflicts addressed ✅

@SamMorrowDrums SamMorrowDrums merged commit e95d8ee into main Nov 16, 2025
16 checks passed
@SamMorrowDrums SamMorrowDrums deleted the stephenotalora/mcp-server-latest-api-updates-v2 branch November 16, 2025 22:42
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