Improve reaction tool responses and wording

Return reaction URLs in minimal responses and clarify issue tools apply to pull requests where applicable.

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

Author: timrogers

README.md

@@ -846,14 +846,14 @@ The following sets of tools are available:
   - `owner`: Repository owner (string, required)
   - `repo`: Repository name (string, required)
 
-- **add_issue_comment_reaction** - Add Issue Comment Reaction
+- **add_issue_comment_reaction** - Add Reaction to Issue or Pull Request Comment
   - **Required OAuth Scopes**: `repo`
   - `comment_id`: The issue comment ID (number, required)
   - `content`: The emoji reaction type (string, required)
   - `owner`: Repository owner (username or organization) (string, required)
   - `repo`: Repository name (string, required)
 
-- **add_issue_reaction** - Add Issue Reaction
+- **add_issue_reaction** - Add Reaction to Issue or Pull Request
   - **Required OAuth Scopes**: `repo`
   - `content`: The emoji reaction type (string, required)
   - `issue_number`: The issue number (number, required)

pkg/github/__toolsnaps__/add_issue_comment_reaction.snap

@@ -2,9 +2,9 @@
   "annotations": {
     "destructiveHint": false,
     "openWorldHint": true,
-    "title": "Add Issue Comment Reaction"
+    "title": "Add Reaction to Issue or Pull Request Comment"
   },
-  "description": "Add a reaction to an issue comment.",
+  "description": "Add a reaction to an issue or pull request comment.",
   "inputSchema": {
     "properties": {
       "comment_id": {

pkg/github/__toolsnaps__/add_issue_reaction.snap

@@ -2,9 +2,9 @@
   "annotations": {
     "destructiveHint": false,
     "openWorldHint": true,
-    "title": "Add Issue Reaction"
+    "title": "Add Reaction to Issue or Pull Request"
   },
-  "description": "Add a reaction to an issue.",
+  "description": "Add a reaction to an issue or pull request.",
   "inputSchema": {
     "properties": {
       "content": {

pkg/github/granular_tools_test.go

@@ -2,6 +2,7 @@ package github
 
 import (
 	"context"
+	"encoding/json"
 	"net/http"
 	"strings"
 	"testing"
@@ -2091,6 +2092,11 @@ func TestAddIssueReaction(t *testing.T) {
 				assert.True(t, result.IsError)
 			} else {
 				assert.False(t, result.IsError)
+				textContent := getTextResult(t, result)
+				var response MinimalResponse
+				require.NoError(t, json.Unmarshal([]byte(textContent.Text), &response))
+				assert.Equal(t, "12345", response.ID)
+				assert.Equal(t, "https://api.github.com/repos/owner/repo/issues/42/reactions/12345", response.URL)
 			}
 		})
 	}
@@ -2146,6 +2152,11 @@ func TestAddIssueCommentReaction(t *testing.T) {
 				assert.True(t, result.IsError)
 			} else {
 				assert.False(t, result.IsError)
+				textContent := getTextResult(t, result)
+				var response MinimalResponse
+				require.NoError(t, json.Unmarshal([]byte(textContent.Text), &response))
+				assert.Equal(t, "67890", response.ID)
+				assert.Equal(t, "https://api.github.com/repos/owner/repo/issues/comments/999/reactions/67890", response.URL)
 			}
 		})
 	}
@@ -2201,6 +2212,11 @@ func TestAddPullRequestReviewCommentReaction(t *testing.T) {
 				assert.True(t, result.IsError)
 			} else {
 				assert.False(t, result.IsError)
+				textContent := getTextResult(t, result)
+				var response MinimalResponse
+				require.NoError(t, json.Unmarshal([]byte(textContent.Text), &response))
+				assert.Equal(t, "54321", response.ID)
+				assert.Equal(t, "https://api.github.com/repos/owner/repo/pulls/comments/888/reactions/54321", response.URL)
 			}
 		})
 	}

pkg/github/issues_granular.go

@@ -1199,15 +1199,15 @@ func GranularSetIssueFields(t translations.TranslationHelperFunc) inventory.Serv
 	return st
 }
 
-// AddIssueReaction adds a reaction to an issue.
+// AddIssueReaction adds a reaction to an issue or pull request.
 func AddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataIssues,
 		mcp.Tool{
 			Name:        "add_issue_reaction",
-			Description: t("TOOL_ADD_ISSUE_REACTION_DESCRIPTION", "Add a reaction to an issue."),
+			Description: t("TOOL_ADD_ISSUE_REACTION_DESCRIPTION", "Add a reaction to an issue or pull request."),
 			Annotations: &mcp.ToolAnnotations{
-				Title:           t("TOOL_ADD_ISSUE_REACTION_USER_TITLE", "Add Issue Reaction"),
+				Title:           t("TOOL_ADD_ISSUE_REACTION_USER_TITLE", "Add Reaction to Issue or Pull Request"),
 				ReadOnlyHint:    false,
 				DestructiveHint: jsonschema.Ptr(false),
 				OpenWorldHint:   jsonschema.Ptr(true),
@@ -1268,7 +1268,8 @@ func AddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool
 			defer func() { _ = resp.Body.Close() }()
 
 			r, err := json.Marshal(MinimalResponse{
-				ID: fmt.Sprintf("%d", reaction.GetID()),
+				ID:  fmt.Sprintf("%d", reaction.GetID()),
+				URL: fmt.Sprintf("%srepos/%s/%s/issues/%d/reactions/%d", client.BaseURL(), owner, repo, issueNumber, reaction.GetID()),
 			})
 			if err != nil {
 				return utils.NewToolResultErrorFromErr("failed to marshal response", err), nil, nil
@@ -1279,15 +1280,15 @@ func AddIssueReaction(t translations.TranslationHelperFunc) inventory.ServerTool
 	return st
 }
 
-// AddIssueCommentReaction adds a reaction to an issue comment.
+// AddIssueCommentReaction adds a reaction to an issue or pull request comment.
 func AddIssueCommentReaction(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataIssues,
 		mcp.Tool{
 			Name:        "add_issue_comment_reaction",
-			Description: t("TOOL_ADD_ISSUE_COMMENT_REACTION_DESCRIPTION", "Add a reaction to an issue comment."),
+			Description: t("TOOL_ADD_ISSUE_COMMENT_REACTION_DESCRIPTION", "Add a reaction to an issue or pull request comment."),
 			Annotations: &mcp.ToolAnnotations{
-				Title:           t("TOOL_ADD_ISSUE_COMMENT_REACTION_USER_TITLE", "Add Issue Comment Reaction"),
+				Title:           t("TOOL_ADD_ISSUE_COMMENT_REACTION_USER_TITLE", "Add Reaction to Issue or Pull Request Comment"),
 				ReadOnlyHint:    false,
 				DestructiveHint: jsonschema.Ptr(false),
 				OpenWorldHint:   jsonschema.Ptr(true),
@@ -1348,7 +1349,8 @@ func AddIssueCommentReaction(t translations.TranslationHelperFunc) inventory.Ser
 			defer func() { _ = resp.Body.Close() }()
 
 			r, err := json.Marshal(MinimalResponse{
-				ID: fmt.Sprintf("%d", reaction.GetID()),
+				ID:  fmt.Sprintf("%d", reaction.GetID()),
+				URL: fmt.Sprintf("%srepos/%s/%s/issues/comments/%d/reactions/%d", client.BaseURL(), owner, repo, commentID, reaction.GetID()),
 			})
 			if err != nil {
 				return utils.NewToolResultErrorFromErr("failed to marshal response", err), nil, nil

pkg/github/pullrequests_granular.go

@@ -827,7 +827,8 @@ func AddPullRequestReviewCommentReaction(t translations.TranslationHelperFunc) i
 			defer func() { _ = resp.Body.Close() }()
 
 			r, err := json.Marshal(MinimalResponse{
-				ID: fmt.Sprintf("%d", reaction.GetID()),
+				ID:  fmt.Sprintf("%d", reaction.GetID()),
+				URL: fmt.Sprintf("%srepos/%s/%s/pulls/comments/%d/reactions/%d", client.BaseURL(), owner, repo, commentID, reaction.GetID()),
 			})
 			if err != nil {
 				return utils.NewToolResultErrorFromErr("failed to marshal response", err), nil, nil