PR Ready

Mark a draft PR as ready for human review. This validates the PR is in a good state before transitioning it out of draft mode.

Instructions

Run pr-loop ready to (add --preserve-claude-threads if user passed "preserve" as an argument to this skill):

1. •Verify the PR is currently in draft mode
2. •Verify the PR has exactly one commit (fail with squash instructions if not)
3. •Validate that:
   • •All CI checks are passing (no failures or pending)
   • •All review threads are resolved (not just responded to)
4. •Delete resolved review threads where all comments are from Claude (unless --preserve-claude-threads is passed)
5. •Remove the LLM iteration status block from the PR description
6. •Mark the PR as ready for review (non-draft)

When to Use

Use this skill when:

• •You've finished iterating on a PR with /pr-loop or /pr-loop-unattended
• •The PR is in a good state (CI green, all threads resolved)
• •The PR has been squashed to a single commit
• •You're ready for human reviewers to look at it

Example

pr-loop ready

Output on success:

Checking PR draft status...
✓ PR is in draft mode
Checking PR commit count...
✓ PR has a single commit
Validating PR state...
✓ All threads resolved
✓ All CI checks passed
Removing status block from PR description...
✓ Status block removed
Marking PR as ready for review...
✓ PR marked as ready for review

🎉 PR is now ready for human review!

Preserving Pure-Claude Threads (Optional)

When iterating with an LLM, you may end up with review threads where all comments are from Claude (e.g., Claude talking to itself during iterations). These are typically noise for human reviewers, so by default they are deleted.

If the user specifically requests preservation (e.g., /pr-ready preserve), pass --preserve-claude-threads to keep these threads.

By default, all comments in resolved threads where every comment starts with the Claude marker will be deleted. Threads with any non-Claude comments are always preserved.

If CI Is Still Pending

If pr-loop ready fails because CI checks are still pending, use /pr-loop-unattended to wait for CI to complete. That skill has built-in waiting functionality that polls for CI status. Once CI passes, run pr-loop ready again.

Do NOT use sleep to wait for CI - always use /pr-loop-unattended which handles the waiting properly.

Important Notes

• •The PR must be in draft mode - this command is for transitioning drafts to ready
• •The PR must have exactly one commit - squash your commits before running
• •CI must be fully passing (not pending) before marking ready
• •All review threads must be actually resolved (having Claude's response as the last comment is not enough)