feat: add PR CI watch skill#2447
Conversation
|
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughTwo new documentation files are added to define and instruct the PR CI watch subagent skill. The first file provides maintenance guidelines, while the second specifies the skill's purpose, control flow, terminal outcomes, and operational requirements for monitoring GitHub PR checks. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 291f549fc3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| Preferred command pattern: | ||
|
|
||
| ```bash | ||
| gh pr checks <pr> --watch |
There was a problem hiding this comment.
Scope watch command to required checks
The skill defines success as "all required checks passed," but the recommended command gh pr checks <pr> --watch watches all checks by default. In repositories with optional checks, a non-required failure will be reported as CI failure even when required branch-protection checks are green, so the main agent can incorrectly tell users the PR failed. Either add --required to the watch command or change the documented success criteria to match all checks.
Useful? React with 👍 / 👎.
Summary
gh pr checks <pr> --watchso the main agent only resumes when CI succeeds or failsSummary by CodeRabbit