Update README to reflect the unified PR review comment tool

ashwin-ant · ashwin-ant · commit c2b2dc111d3d · 2025-04-11T08:17:48.000-07:00
diff --git a/README.md b/README.md
@@ -288,28 +288,20 @@ export GITHUB_MCP_TOOL_ADD_ISSUE_COMMENT_DESCRIPTION="an alternative description
   - `draft`: Create as draft PR (boolean, optional)
   - `maintainer_can_modify`: Allow maintainer edits (boolean, optional)
 
-- **add_pull_request_review_comment** - Add a review comment to a pull request
+- **add_pull_request_review_comment** - Add a review comment to a pull request or reply to an existing comment
 
   - `owner`: Repository owner (string, required)
   - `repo`: Repository name (string, required)
   - `pull_number`: Pull request number (number, required)
   - `body`: The text of the review comment (string, required)
-  - `commit_id`: The SHA of the commit to comment on (string, required)
-  - `path`: The relative path to the file that necessitates a comment (string, required)
+  - `commit_id`: The SHA of the commit to comment on (string, required unless using in_reply_to)
+  - `path`: The relative path to the file that necessitates a comment (string, required unless using in_reply_to)
   - `line`: The line of the blob in the pull request diff that the comment applies to (number, optional)
   - `side`: The side of the diff to comment on (LEFT or RIGHT) (string, optional)
   - `start_line`: For multi-line comments, the first line of the range (number, optional)
   - `start_side`: For multi-line comments, the starting side of the diff (LEFT or RIGHT) (string, optional)
   - `subject_type`: The level at which the comment is targeted (line or file) (string, optional)
-  - `in_reply_to`: The ID of the review comment to reply to (number, optional)
-
-- **reply_to_pull_request_review_comment** - Reply to an existing review comment on a pull request
-
-  - `owner`: Repository owner (string, required)
-  - `repo`: Repository name (string, required)
-  - `pull_number`: Pull request number (number, required)
-  - `comment_id`: The unique identifier of the comment to reply to (number, required)
-  - `body`: The text of the reply comment (string, required)
+  - `in_reply_to`: The ID of the review comment to reply to (number, optional). When specified, only body is required and other parameters are ignored.
 
 - **update_pull_request** - Update an existing pull request in a GitHub repository
 
diff --git a/pkg/github/pullrequests_test.go b/pkg/github/pullrequests_test.go
@@ -1890,7 +1890,6 @@ func Test_AddPullRequestReviewComment(t *testing.T) {
 			assert.NotNil(t, result)
 			require.Len(t, result.Content, 1)
 
-			// Check for error message in the result
 			textContent := getTextResult(t, result)
 			if tc.expectedErrMsg != "" {
 				assert.Contains(t, textContent.Text, tc.expectedErrMsg)
