refactor: improve code consistency in MarkPullRequestReadyForReview

- Rename 'variables' to 'vars' for consistency with other GraphQL functions
- Improve comment precision for GraphQL schema requirements

These changes address code style nitpicks while maintaining functionality.

Author: efouts

pkg/github/pullrequests.go

@@ -1655,13 +1655,13 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 				} `graphql:"repository(owner: $owner, name: $repo)"`
 			}
 
-			variables := map[string]any{
+			vars := map[string]any{
 				"owner": githubv4.String(params.Owner),
 				"repo":  githubv4.String(params.Repo),
 				"prNum": githubv4.Int(params.PullNumber),
 			}
 
-			if err := client.Query(ctx, &getPullRequestQuery, variables); err != nil {
+			if err := client.Query(ctx, &getPullRequestQuery, vars); err != nil {
 				return mcp.NewToolResultError(fmt.Sprintf("failed to get pull request: %v", err)), nil
 			}
 
@@ -1674,7 +1674,7 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 			var markReadyForReviewMutation struct {
 				MarkPullRequestReadyForReview struct {
 					PullRequest struct {
-						ID githubv4.ID // We don't need this, but a selector is required or GQL complains
+						ID githubv4.ID // Required by GraphQL schema, but not used in response
 					}
 				} `graphql:"markPullRequestReadyForReview(input: $input)"`
 			}