feat: Add mark_pr_ready_for_review tool

[efouts]

This commit introduces a new tool, mark_pr_ready_for_review, which allows changing a GitHub pull request from a draft state to ready for review.

This tool utilizes the GitHub GraphQL API's markPullRequestReadyForReview mutation, as the REST API does not support this specific state transition for existing draft PRs.

The implementation includes the tool logic, its registration in the pull_requests toolset, and comprehensive tests.

[Copilot]

Pull Request Overview

Introduce a new tool to transition draft GitHub pull requests to ready-for-review state via GraphQL, including command registration, implementation, and tests.

• Register the mark_pr_ready_for_review tool in the server toolset.
• Implement the GraphQL query and mutation for marking a PR as ready for review.
• Add comprehensive unit tests for success and failure scenarios.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
pkg/github/tools.go	Register new mark_pr_ready_for_review tool.
pkg/github/pullrequests.go	Implement GraphQL logic for marking draft PRs as ready for review.
pkg/github/pullrequests_test.go	Add unit tests covering various success/failure cases for the new tool.

Comments suppressed due to low confidence (2)

pkg/github/pullrequests_test.go:2295

• Add a test case for failure to obtain the GraphQL client (e.g., have getGQLClient return an error) to verify the handler surfaces this as a tool error instead of returning a raw error.

func TestMarkPullRequestReadyForReview(t *testing.T) {

pkg/github/pullrequests.go:1638

• Add a test to cover parameter decoding failures (e.g., missing or wrong-type owner/repo/pullNumber) so you can confirm those errors are returned as tool errors.

if err := mapstructure.Decode(request.Params.Arguments, &params); err != nil {

[Copilot]

Pull Request Overview

Adds a new CLI tool (mark_pr_ready_for_review) that transitions a draft GitHub pull request to ready-for-review via the GraphQL API.

• Registers the new tool in the existing toolset.
• Implements the GraphQL query/mutation logic in pullrequests.go.
• Provides comprehensive unit tests for success and failure scenarios.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
pkg/github/tools.go	Registers MarkPullRequestReadyForReview in toolset
pkg/github/pullrequests.go	Implements the new GraphQL-based ready-for-review tool
pkg/github/pullrequests_test.go	Adds tests covering all new tool behaviors

Comments suppressed due to low confidence (1)

pkg/github/pullrequests.go:1664

• The local variables owner, repo, and pullNumber are parsed above, but the code mistakenly references an undefined params struct here. Update the variables map to use the parsed locals, e.g.: "owner": githubv4.String(owner), "repo": githubv4.String(repo), and "prNum": githubv4.Int(pullNumber).

"owner": githubv4.String(params.Owner),

[Copilot]

Pull Request Overview

This PR introduces a new tool, mark_pr_ready_for_review, to change a draft pull request into a ready for review state by using the GitHub GraphQL API.

• Registers the new tool in the toolset.
• Implements the GraphQL mutation to mark PRs as ready.
• Adds comprehensive tests to validate the tool's behavior.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
pkg/github/tools.go	Registers the new MarkPullRequestReadyForReview tool
pkg/github/pullrequests_test.go	Adds tests covering success and various failure scenarios for the new tool
pkg/github/pullrequests.go	Implements the GraphQL logic for marking a pull request ready for review

[SamMorrowDrums]

Thanks very much for the contribution @efouts I was thinking about this PR and wondering if it might make sense to convert it to set_pr_state and provide an enum for setting the PR to draft, as well as marking as ready tor review. WDYT?

[efouts]

Thanks very much for the contribution @efouts I was thinking about this PR and wondering if it might make sense to convert it to set_pr_state and provide an enum for setting the PR to draft, as well as marking as ready tor review. WDYT?

@SamMorrowDrums happy to do that or add another tool mirroring the convertpullrequesttodraft mutation. which is most in line with the project?

(edit) I went ahead and updated to be set_pr_state with the guess that is the preference.

[Copilot]

Pull Request Overview

This PR adds a new tool, mark_pr_ready_for_review, to allow changing a GitHub pull request's state from draft to ready-for-review (and vice versa), using the GitHub GraphQL API.

• Added a new tool registration (SetPRStatus) in pkg/github/tools.go.
• Implemented the tool logic in pkg/github/pullrequests.go with corresponding tests in pkg/github/pullrequests_test.go.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
pkg/github/tools.go	Registers the new SetPRStatus tool in the toolset.
pkg/github/pullrequests_test.go	Adds comprehensive tests for both state transition paths.
pkg/github/pullrequests.go	Implements the GraphQL logic for setting the pull request status.