Add tools for one-off PR comments and replying to PR review comments#143
Merged
juruen merged 10 commits intogithub:mainfrom Apr 12, 2025
Merged
Add tools for one-off PR comments and replying to PR review comments#143juruen merged 10 commits intogithub:mainfrom
juruen merged 10 commits intogithub:mainfrom
Conversation
Contributor
ashwin-ant
commented
Apr 6, 2025
ashwin-ant
commented
- Added two new tools for working with pull request review comments:
- add_pull_request_review_comment: Create line, multi-line, or file-level review comments (not attached to a PR review). Corresponds to this endpoint.
- reply_to_pull_request_review_comment: Reply to existing PR review comments. Corresponds to this endpoint.
- Verified functionality manually in MCP Insector
Adds the ability to add review comments to pull requests with support for line, multi-line, and file-level comments, as well as replying to existing comments. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Adds a new tool to reply to existing pull request review comments using the GitHub API's comment reply endpoint. This allows for threaded discussions on pull request reviews. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
Author
|
@juruen mind taking a look? Would love to get this in if possible 😄 |
ashwin-ant
force-pushed
the
prcomments
branch
2 times, most recently
from
104ba61 to
84e6f6a
Compare
juruen
requested review from
SamMorrowDrums,
juruen,
toby and
williammartin
as code owners
Collaborator
|
I manually tested this and things worked as expected.
I'd say we are good to merge! |
Collaborator
|
@ashwin-ant although what we have now seems to work, there's something else I'd like to explore here: the possibility of using a single tool for both cases, as I've tried to reply to a comment with that tool and hasn't worked, so I'd like us to spend a bit of time on it to try to figure that out. Thanks! |
juruen
requested changes
Apr 11, 2025
Collaborator
juruen
left a comment
juruen
left a comment
There was a problem hiding this comment.
Please, see my comment about trying to use a single tool for both use cases.
Thanks!
The separate AddPullRequestReviewComment and ReplyToPullRequestReviewComment tools have been merged into a single tool that handles both creating new comments and replying to existing ones. This approach simplifies the API and provides a more consistent interface for users. - Made commit_id and path optional when using in_reply_to for replies - Updated the tests to verify both comment and reply functionality - Removed the separate ReplyToPullRequestReviewComment tool - Fixed test expectations to match how errors are returned 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
Author
|
@juruen Updated to only have a single tool. Turns out the |
Collaborator
Thank you, so much! I've manually tested the latest version and works perfectly for both cases! |
juruen
approved these changes
Apr 12, 2025
nickytonline
pushed a commit
to nickytonline/github-mcp-http
that referenced
this pull request
Oct 4, 2025
…ithub#143) * Add add_pull_request_review_comment tool for PR review comments Adds the ability to add review comments to pull requests with support for line, multi-line, and file-level comments, as well as replying to existing comments. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Add reply_to_pull_request_review_comment tool Adds a new tool to reply to existing pull request review comments using the GitHub API's comment reply endpoint. This allows for threaded discussions on pull request reviews. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Update README with new PR review comment tools * rebase * use new getClient function inadd and reply pr review tools * Unify PR review comment tools into a single consolidated tool The separate AddPullRequestReviewComment and ReplyToPullRequestReviewComment tools have been merged into a single tool that handles both creating new comments and replying to existing ones. This approach simplifies the API and provides a more consistent interface for users. - Made commit_id and path optional when using in_reply_to for replies - Updated the tests to verify both comment and reply functionality - Removed the separate ReplyToPullRequestReviewComment tool - Fixed test expectations to match how errors are returned 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * Update README to reflect the unified PR review comment tool --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Javier Uruen Val <juruen@github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.