fix: correct parameter parsing in MarkPullRequestReadyForReview function

- Replace incorrect request.Params.GetString/GetInt calls with requiredParam/RequiredInt
- Fix undefined params struct reference in GraphQL variables map
- Use local variables (owner, repo, pullNumber) instead of non-existent params fields
- Standardize error reporting to match project conventions

Fixes compilation errors and ensures consistent parameter handling across the codebase.

Author: efouts

pkg/github/pullrequests.go

@@ -1630,20 +1630,19 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 			),
 		),
 		func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-			owner, err := request.Params.GetString("owner")
+			owner, err := requiredParam[string](request, "owner")
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Invalid or missing 'owner': %v", err)), nil
+				return mcp.NewToolResultError(err.Error()), nil
 			}
-
-			repo, err := request.Params.GetString("repo")
+			repo, err := requiredParam[string](request, "repo")
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Invalid or missing 'repo': %v", err)), nil
+				return mcp.NewToolResultError(err.Error()), nil
 			}
-
-			pullNumber, err := request.Params.GetInt("pullNumber")
+			pullNumber, err := RequiredInt(request, "pullNumber")
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Invalid or missing 'pullNumber': %v", err)), nil
+				return mcp.NewToolResultError(err.Error()), nil
 			}
+
 			// Get the GraphQL client
 			client, err := getGQLClient(ctx)
 			if err != nil {
@@ -1661,9 +1660,9 @@ func MarkPullRequestReadyForReview(getGQLClient GetGQLClientFn, t translations.T
 			}
 
 			variables := map[string]any{
-				"owner": githubv4.String(params.Owner),
-				"repo":  githubv4.String(params.Repo),
-				"prNum": githubv4.Int(params.PullNumber),
+				"owner": githubv4.String(owner),
+				"repo":  githubv4.String(repo),
+				"prNum": githubv4.Int(pullNumber),
 			}
 
 			if err := client.Query(ctx, &getPullRequestQuery, variables); err != nil {