Make MarkPullRequestReadyForReview consistent with other GraphQL functions

- Replace RequiredInt + manual conversion with mapstructure.Decode pattern
- Use int32 PullNumber field in params struct like other GraphQL functions
- Eliminates integer overflow concerns and maintains consistency
- Follows established pattern used by CreateAndSubmitPullRequestReview, etc.

Author: efouts

pkg/github/pullrequests.go

@@ -1630,25 +1630,15 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 			),
 		),
 		func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-			owner, err := requiredParam[string](request, "owner")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			repo, err := requiredParam[string](request, "repo")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
+			var params struct {
+				Owner      string
+				Repo       string
+				PullNumber int32
 			}
-			pullNumber, err := RequiredInt(request, "pullNumber")
-			if err != nil {
+			if err := mapstructure.Decode(request.Params.Arguments, &params); err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			// Convert int to int32 with bounds checking to avoid integer overflow
-			if pullNumber < 0 || pullNumber > 2147483647 {
-				return mcp.NewToolResultError("pullNumber out of range for int32"), nil
-			}
-			pullNumber32 := int32(pullNumber)
-
 			// Get the GraphQL client
 			client, err := getGQLClient(ctx)
 			if err != nil {
@@ -1666,9 +1656,9 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 			}
 
 			variables := map[string]any{
-				"owner": githubv4.String(owner),
-				"repo":  githubv4.String(repo),
-				"prNum": githubv4.Int(pullNumber32),
+				"owner": githubv4.String(params.Owner),
+				"repo":  githubv4.String(params.Repo),
+				"prNum": githubv4.Int(params.PullNumber),
 			}
 
 			if err := client.Query(ctx, &getPullRequestQuery, variables); err != nil {