update function name to match tool name

LuluBeatson · LuluBeatson · commit 4bdb40af9c83 · 2025-07-18T09:17:56.000+01:00
diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go
@@ -1151,8 +1151,8 @@ func CreatePendingPullRequestReview(getGQLClient GetGQLClientFn, t translations.
 		}
 }
 
-// AddPullRequestReviewCommentToPendingReview creates a tool to add a comment to a pull request review.
-func AddPullRequestReviewCommentToPendingReview(getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {
+// AddCommentToPendingReview creates a tool to add a comment to a pull request review.
+func AddCommentToPendingReview(getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {
 	return mcp.NewTool("add_comment_to_pending_review",
 			mcp.WithDescription(t("TOOL_ADD_COMMENT_TO_PENDING_REVIEW_DESCRIPTION", "Add a review comment to the requester's latest pending pull request review, a pending review needs to already exist to call this (check with the user if not sure).")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
diff --git a/pkg/github/pullrequests_test.go b/pkg/github/pullrequests_test.go
@@ -2137,7 +2137,7 @@ func TestAddPullRequestReviewCommentToPendingReview(t *testing.T) {
 
 	// Verify tool definition once
 	mockClient := githubv4.NewClient(nil)
-	tool, _ := AddPullRequestReviewCommentToPendingReview(stubGetGQLClientFn(mockClient), translations.NullTranslationHelper)
+	tool, _ := AddCommentToPendingReview(stubGetGQLClientFn(mockClient), translations.NullTranslationHelper)
 	require.NoError(t, toolsnaps.Test(tool.Name, tool))
 
 	assert.Equal(t, "add_comment_to_pending_review", tool.Name)
@@ -2222,7 +2222,7 @@ func TestAddPullRequestReviewCommentToPendingReview(t *testing.T) {
 
 			// Setup client with mock
 			client := githubv4.NewClient(tc.mockedClient)
-			_, handler := AddPullRequestReviewCommentToPendingReview(stubGetGQLClientFn(client), translations.NullTranslationHelper)
+			_, handler := AddCommentToPendingReview(stubGetGQLClientFn(client), translations.NullTranslationHelper)
 
 			// Create call request
 			request := createMCPRequest(tc.requestArgs)
diff --git a/pkg/github/tools.go b/pkg/github/tools.go
@@ -89,7 +89,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
 			// Reviews
 			toolsets.NewServerTool(CreateAndSubmitPullRequestReview(getGQLClient, t)),
 			toolsets.NewServerTool(CreatePendingPullRequestReview(getGQLClient, t)),
-			toolsets.NewServerTool(AddPullRequestReviewCommentToPendingReview(getGQLClient, t)),
+			toolsets.NewServerTool(AddCommentToPendingReview(getGQLClient, t)),
 			toolsets.NewServerTool(SubmitPendingPullRequestReview(getGQLClient, t)),
 			toolsets.NewServerTool(DeletePendingPullRequestReview(getGQLClient, t)),
 		)

Original file line number	Diff line number	Diff line change
`@@ -1151,8 +1151,8 @@ func CreatePendingPullRequestReview(getGQLClient GetGQLClientFn, t translations.`
`1151`	`1151`	`}`
`1152`	`1152`	`}`
`1153`	`1153`
`1154`		`-// AddPullRequestReviewCommentToPendingReview creates a tool to add a comment to a pull request review.`
`1155`		`-func AddPullRequestReviewCommentToPendingReview(getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {`
	`1154`	`+// AddCommentToPendingReview creates a tool to add a comment to a pull request review.`
	`1155`	`+func AddCommentToPendingReview(getGQLClient GetGQLClientFn, t translations.TranslationHelperFunc) (mcp.Tool, server.ToolHandlerFunc) {`
`1156`	`1156`	`return mcp.NewTool("add_comment_to_pending_review",`
`1157`	`1157`	`mcp.WithDescription(t("TOOL_ADD_COMMENT_TO_PENDING_REVIEW_DESCRIPTION", "Add a review comment to the requester's latest pending pull request review, a pending review needs to already exist to call this (check with the user if not sure).")),`
`1158`	`1158`	`mcp.WithToolAnnotation(mcp.ToolAnnotation{`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG`
`89`	`89`	`// Reviews`
`90`	`90`	`toolsets.NewServerTool(CreateAndSubmitPullRequestReview(getGQLClient, t)),`
`91`	`91`	`toolsets.NewServerTool(CreatePendingPullRequestReview(getGQLClient, t)),`
`92`		`- toolsets.NewServerTool(AddPullRequestReviewCommentToPendingReview(getGQLClient, t)),`
	`92`	`+ toolsets.NewServerTool(AddCommentToPendingReview(getGQLClient, t)),`
`93`	`93`	`toolsets.NewServerTool(SubmitPendingPullRequestReview(getGQLClient, t)),`
`94`	`94`	`toolsets.NewServerTool(DeletePendingPullRequestReview(getGQLClient, t)),`
`95`	`95`	`)`