Add tools for one-off PR comments and replying to PR review comments #143
Conversation
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>
|
@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
juruen,
SamMorrowDrums,
williammartin and
toby
as code owners
|
I manually tested this and things worked as expected.
I'd say we are good to merge! |
|
@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! |
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>
|
@juruen Updated to only have a single tool. Turns out the |
Thank you, so much! I've manually tested the latest version and works perfectly for both cases! |
