-
Notifications
You must be signed in to change notification settings - Fork 9.1k
bug(github): Add missing pull request operations to MCP server #728
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
bug(github): Add missing pull request operations to MCP server #728
Conversation
Add support for all pull request operations documented in README.md that were previously missing from the MCP server implementation. This includes: - get_pull_request: Get details of a specific pull request - list_pull_requests: List and filter repository pull requests - create_pull_request_review: Create a review on a pull request - merge_pull_request: Merge a pull request - get_pull_request_files: Get the list of files changed in a pull request - get_pull_request_status: Get the combined status of all status checks - update_pull_request_branch: Update a PR branch with latest base branch changes - get_pull_request_comments: Get the review comments on a pull request - get_pull_request_reviews: Get the reviews on a pull request
kushparikh01
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.
Was looking for this, can this get merged please!
|
Also hoping this can be resolved quickly since it's quite straight forward. Thank you! |
|
Everyone and their dog is using MCP now how do we not have an executive order to get this merged?? |
|
Looks like there's a lot of support for this change, and it seems straightforward. Merging this soon would really help keep things moving smoothly. Appreciate your help in getting this across the finish line! |
|
here to beg for a merge |
Add support for all pull request operations documented in README.md that were previously included in the MCP server implementation but omitted from the list of advertised functions. The bug here is that the index.ts doesn't list all of the tools that are present.
This includes:
Description
Server Details
Motivation and Context
Solves a frequently reported bug, namely that the GitHub MCP Server does not advertise its full capabilities due to an accidental omission in the original release. See bugs #541 and #493.
How Has This Been Tested?
This has been tested with an MCP client (cursor) and should work identically with all clients.
Breaking Changes
No breaking changes.
Types of changes
Checklist