Align granular reaction tool constructors

Rename the standalone reaction tool constructors to match the granular tool naming convention and clarify issue comment reaction IDs for pull request comments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: SamMorrowDrums

docs/feature-flags.md

@@ -100,7 +100,7 @@ runtime behavior (such as output formatting) won't appear here.
 
 - **add_issue_comment_reaction** - Add Reaction to Issue or Pull Request Comment
   - **Required OAuth Scopes**: `repo`
-  - `comment_id`: The issue comment ID (number, required)
+  - `comment_id`: The issue or pull request comment ID (number, required)
   - `content`: The emoji reaction type (string, required)
   - `owner`: Repository owner (username or organization) (string, required)
   - `repo`: Repository name (string, required)

pkg/github/__toolsnaps__/add_issue_comment_reaction.snap

@@ -8,7 +8,7 @@
   "inputSchema": {
     "properties": {
       "comment_id": {
-        "description": "The issue comment ID",
+        "description": "The issue or pull request comment ID",
         "minimum": 1,
         "type": "number"
       },

pkg/github/granular_tools_test.go

@@ -44,8 +44,8 @@ func TestGranularToolSnaps(t *testing.T) {
 		GranularRemoveSubIssue,
 		GranularReprioritizeSubIssue,
 		GranularSetIssueFields,
-		AddIssueReaction,
-		AddIssueCommentReaction,
+		GranularAddIssueReaction,
+		GranularAddIssueCommentReaction,
 		GranularUpdatePullRequestTitle,
 		GranularUpdatePullRequestBody,
 		GranularUpdatePullRequestState,
@@ -57,7 +57,7 @@ func TestGranularToolSnaps(t *testing.T) {
 		GranularAddPullRequestReviewComment,
 		GranularResolveReviewThread,
 		GranularUnresolveReviewThread,
-		AddPullRequestReviewCommentReaction,
+		GranularAddPullRequestReviewCommentReaction,
 	}
 
 	for _, constructor := range toolConstructors {
@@ -2035,7 +2035,7 @@ func TestGranularSetIssueFields(t *testing.T) {
 
 // --- Reaction granular tool handler tests ---
 
-func TestAddIssueReaction(t *testing.T) {
+func TestGranularAddIssueReaction(t *testing.T) {
 	mockReaction := &gogithub.Reaction{
 		ID:      gogithub.Ptr(int64(12345)),
 		Content: gogithub.Ptr("+1"),
@@ -2086,7 +2086,7 @@ func TestAddIssueReaction(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 			client := mustNewGHClient(t, tc.mockedClient)
 			deps := BaseDeps{Client: client}
-			serverTool := AddIssueReaction(translations.NullTranslationHelper)
+			serverTool := GranularAddIssueReaction(translations.NullTranslationHelper)
 			handler := serverTool.Handler(deps)
 			request := createMCPRequest(tc.args)
 			result, err := handler(ContextWithDeps(context.Background(), deps), &request)
@@ -2105,7 +2105,7 @@ func TestAddIssueReaction(t *testing.T) {
 	}
 }
 
-func TestAddIssueCommentReaction(t *testing.T) {
+func TestGranularAddIssueCommentReaction(t *testing.T) {
 	mockReaction := &gogithub.Reaction{
 		ID:      gogithub.Ptr(int64(67890)),
 		Content: gogithub.Ptr("heart"),
@@ -2146,7 +2146,7 @@ func TestAddIssueCommentReaction(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 			client := mustNewGHClient(t, tc.mockedClient)
 			deps := BaseDeps{Client: client}
-			serverTool := AddIssueCommentReaction(translations.NullTranslationHelper)
+			serverTool := GranularAddIssueCommentReaction(translations.NullTranslationHelper)
 			handler := serverTool.Handler(deps)
 			request := createMCPRequest(tc.args)
 			result, err := handler(ContextWithDeps(context.Background(), deps), &request)
@@ -2165,7 +2165,7 @@ func TestAddIssueCommentReaction(t *testing.T) {
 	}
 }
 
-func TestAddPullRequestReviewCommentReaction(t *testing.T) {
+func TestGranularAddPullRequestReviewCommentReaction(t *testing.T) {
 	mockReaction := &gogithub.Reaction{
 		ID:      gogithub.Ptr(int64(54321)),
 		Content: gogithub.Ptr("rocket"),
@@ -2206,7 +2206,7 @@ func TestAddPullRequestReviewCommentReaction(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 			client := mustNewGHClient(t, tc.mockedClient)
 			deps := BaseDeps{Client: client}
-			serverTool := AddPullRequestReviewCommentReaction(translations.NullTranslationHelper)
+			serverTool := GranularAddPullRequestReviewCommentReaction(translations.NullTranslationHelper)
 			handler := serverTool.Handler(deps)
 			request := createMCPRequest(tc.args)
 			result, err := handler(ContextWithDeps(context.Background(), deps), &request)

pkg/github/issues_granular.go

@@ -1199,8 +1199,8 @@ func GranularSetIssueFields(t translations.TranslationHelperFunc) inventory.Serv
 	return st
 }
 
-// AddIssueReaction adds a reaction to an issue or pull request.
-func AddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
+// GranularAddIssueReaction adds a reaction to an issue or pull request.
+func GranularAddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataIssues,
 		mcp.Tool{
@@ -1281,8 +1281,8 @@ func AddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool
 	return st
 }
 
-// AddIssueCommentReaction adds a reaction to an issue or pull request comment.
-func AddIssueCommentReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
+// GranularAddIssueCommentReaction adds a reaction to an issue or pull request comment.
+func GranularAddIssueCommentReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataIssues,
 		mcp.Tool{
@@ -1307,7 +1307,7 @@ func AddIssueCommentReaction(t translations.TranslationHelperFunc) inventory.Ser
 					},
 					"comment_id": {
 						Type:        "number",
-						Description: "The issue comment ID",
+						Description: "The issue or pull request comment ID",
 						Minimum:     jsonschema.Ptr(1.0),
 					},
 					"content": {

pkg/github/pullrequests_granular.go

@@ -758,8 +758,8 @@ func GranularUnresolveReviewThread(t translations.TranslationHelperFunc) invento
 	return st
 }
 
-// AddPullRequestReviewCommentReaction adds a reaction to a pull request review comment.
-func AddPullRequestReviewCommentReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
+// GranularAddPullRequestReviewCommentReaction adds a reaction to a pull request review comment.
+func GranularAddPullRequestReviewCommentReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataPullRequests,
 		mcp.Tool{

pkg/github/tools.go

@@ -314,8 +314,8 @@ func AllTools(t translations.TranslationHelperFunc) []inventory.ServerTool {
 		GranularRemoveSubIssue(t),
 		GranularReprioritizeSubIssue(t),
 		GranularSetIssueFields(t),
-		AddIssueReaction(t),
-		AddIssueCommentReaction(t),
+		GranularAddIssueReaction(t),
+		GranularAddIssueCommentReaction(t),
 
 		// Granular pull request tools (feature-flagged, replace consolidated update_pull_request/pull_request_review_write)
 		GranularUpdatePullRequestTitle(t),
@@ -329,7 +329,7 @@ func AllTools(t translations.TranslationHelperFunc) []inventory.ServerTool {
 		GranularAddPullRequestReviewComment(t),
 		GranularResolveReviewThread(t),
 		GranularUnresolveReviewThread(t),
-		AddPullRequestReviewCommentReaction(t),
+		GranularAddPullRequestReviewCommentReaction(t),
 	})
 }