fix: close reaction removal responses on errors

Register nil-safe response body cleanup before handling go-github errors for all reaction removal tools, and cover every API-error path.

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

Copilot-Session: 9acf9c83-49aa-41af-a42d-ad75de34d132

Author: timrogers

pkg/github/granular_tools_test.go

@@ -2633,6 +2633,19 @@ func TestGranularRemoveIssueCommentReaction(t *testing.T) {
 			},
 			expectedErrMsg: "missing required parameter: comment_id",
 		},
+		{
+			name: "API error",
+			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
+				DeleteReposIssuesCommentsReactionsByOwnerByRepoByCommentID: mockResponse(t, http.StatusNotFound, `{"message":"Not Found"}`),
+			}),
+			args: map[string]any{
+				"owner":       "owner",
+				"repo":        "repo",
+				"comment_id":  float64(999),
+				"reaction_id": float64(67890),
+			},
+			expectedErrMsg: "failed to remove reaction from issue comment",
+		},
 	}
 
 	for _, tc := range tests {
@@ -2746,6 +2759,19 @@ func TestGranularRemovePullRequestReviewCommentReaction(t *testing.T) {
 			},
 			expectedErrMsg: "missing required parameter: repo",
 		},
+		{
+			name: "API error",
+			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
+				DeleteReposPullsCommentsReactionsByOwnerByRepoByCommentID: mockResponse(t, http.StatusNotFound, `{"message":"Not Found"}`),
+			}),
+			args: map[string]any{
+				"owner":       "owner",
+				"repo":        "repo",
+				"comment_id":  float64(888),
+				"reaction_id": float64(54321),
+			},
+			expectedErrMsg: "failed to remove reaction from pull request review comment",
+		},
 	}
 
 	for _, tc := range tests {

pkg/github/issues_granular.go

@@ -1729,10 +1729,12 @@ func GranularRemoveIssueReaction(t translations.TranslationHelperFunc) inventory
 			}
 
 			resp, err := client.Reactions.DeleteIssueReaction(ctx, owner, repo, issueNumber, reactionID)
+			if resp != nil && resp.Body != nil {
+				defer func() { _ = resp.Body.Close() }()
+			}
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx, "failed to remove reaction from issue", resp, err), nil, nil
 			}
-			defer func() { _ = resp.Body.Close() }()
 
 			return utils.NewToolResultText("reaction successfully removed from issue"), nil, nil
 		},
@@ -1886,10 +1888,12 @@ func GranularRemoveIssueCommentReaction(t translations.TranslationHelperFunc) in
 			}
 
 			resp, err := client.Reactions.DeleteIssueCommentReaction(ctx, owner, repo, commentID, reactionID)
+			if resp != nil && resp.Body != nil {
+				defer func() { _ = resp.Body.Close() }()
+			}
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx, "failed to remove reaction from issue comment", resp, err), nil, nil
 			}
-			defer func() { _ = resp.Body.Close() }()
 
 			return utils.NewToolResultText("reaction successfully removed from issue comment"), nil, nil
 		},

pkg/github/pullrequests_granular.go

@@ -960,10 +960,12 @@ func GranularRemovePullRequestReviewCommentReaction(t translations.TranslationHe
 			}
 
 			resp, err := client.Reactions.DeletePullRequestCommentReaction(ctx, owner, repo, commentID, reactionID)
+			if resp != nil && resp.Body != nil {
+				defer func() { _ = resp.Body.Close() }()
+			}
 			if err != nil {
 				return ghErrors.NewGitHubAPIErrorResponse(ctx, "failed to remove reaction from pull request review comment", resp, err), nil, nil
 			}
-			defer func() { _ = resp.Body.Close() }()
 
 			return utils.NewToolResultText("reaction successfully removed from pull request review comment"), nil, nil
 		},