Skill: pr-monitor
What I do
I help you manage and track the progress of pull requests. I focus on monitoring CI/CD status, review comments, and approval states. I ensure that PRs are moved through the pipeline efficiently and that all feedback is addressed promptly.
When to use me
- When you've submitted a PR and need to track its progress.
- When you're waiting for reviews from teammates.
- When you need to check why a CI build failed.
- When you're preparing to merge a PR.
Core principles
- Continuous monitoring, check the status of your PRs regularly to avoid delays.
- Proactive communication, respond to review comments quickly and notify reviewers when changes are made.
- CI-first approach, always fix CI failures before asking for a review.
- Draft by default, use draft PRs for work-in-progress to signal that it's not ready for final review.
Patterns & examples
Checking PR status with GitHub CLI
Use the gh command to stay updated.
gh pr status, See a summary of your PRs.
gh pr view, See details of a specific PR, including reviews and checks.
gh pr checks, List the status of all CI checks.
Responding to feedback
Address all comments before re-requesting a review.
- Pattern, Fix the issue, push the change, and then reply to the comment confirming the fix. If you disagree, explain your reasoning clearly and politely.
Resolving review threads
After addressing a review comment and replying, resolve the thread via the GraphQL API.
# Get thread IDs
gh api graphql -f query='{
repository(owner: "OWNER", name: "REPO") {
pullRequest(number: NUM) {
reviewThreads(first: 50) {
nodes {
id
isResolved
comments(first: 1) {
nodes {
databaseId
body
}
}
}
}
}
}
}'
# Resolve thread
gh api graphql -f query='mutation {
resolveReviewThread(input: {threadId: "THREAD_ID"}) {
thread {
isResolved
}
}
}'
Monitoring for conflicts
Keep your branch up to date with the base branch.
- Action, Regularly rebase or merge the base branch (e.g.,
main) into your PR branch to catch conflicts early.
Quality PR descriptions
Help reviewers by providing context.
- Good, Include a summary of changes, why they were made, and how to test them. Link to related issues.
Anti-patterns to avoid
- ❌ The "Ghost" PR, leaving a PR unattended for days while CI is failing or reviewers are waiting.
- ❌ Merging with failed checks, never merge a PR if CI/CD checks have failed, unless there is an exceptional and documented reason.
- ❌ Ignoring negative reviews, merging a PR without addressing a "Request Changes" review from a teammate.
- ❌ Too many commits, avoid pushing dozens of tiny "fix typo" commits. Squash or clean up your history before the final merge.
- ❌ Leaving threads unresolved after addressing them. Addressed threads should always be resolved to clear them for the reviewer.
KB Reference
~/vaults/baphled/3. Resources/Knowledge Base/AI Development System/Skills/Delivery/PR Monitor.md
Related skills
github-expert, for advanced CLI usage.
release-management, for coordinating merges.
documentation-writing, for better PR descriptions.
git-master, for branch management.
1---2name: pr-monitor3description: Monitor PR for CI status, reviews, and coordinate response workflow4---5
6# Skill: pr-monitor
7
8## What I do
9
10I help you manage and track the progress of pull requests. I focus on monitoring CI/CD status, review comments, and approval states. I ensure that PRs are moved through the pipeline efficiently and that all feedback is addressed promptly.
11
12## When to use me
13
14- When you've submitted a PR and need to track its progress.
15- When you're waiting for reviews from teammates.
16- When you need to check why a CI build failed.
17- When you're preparing to merge a PR.
18
19## Core principles
20
211. **Continuous monitoring**, check the status of your PRs regularly to avoid delays.
222. **Proactive communication**, respond to review comments quickly and notify reviewers when changes are made.
233. **CI-first approach**, always fix CI failures before asking for a review.
244. **Draft by default**, use draft PRs for work-in-progress to signal that it's not ready for final review.
25
26## Patterns & examples
27
28### Checking PR status with GitHub CLI
29Use the `gh` command to stay updated.
30- `gh pr status`, See a summary of your PRs.
31- `gh pr view`, See details of a specific PR, including reviews and checks.
32- `gh pr checks`, List the status of all CI checks.
33
34### Responding to feedback
35Address all comments before re-requesting a review.
36- **Pattern**, Fix the issue, push the change, and then reply to the comment confirming the fix. If you disagree, explain your reasoning clearly and politely.
37
38### Resolving review threads
39After addressing a review comment and replying, resolve the thread via the GraphQL API.
40
41```bash
42# Get thread IDs
43gh api graphql -f query='{
44 repository(owner: "OWNER", name: "REPO") {
45 pullRequest(number: NUM) {
46 reviewThreads(first: 50) {
47 nodes {
48 id
49 isResolved
50 comments(first: 1) {
51 nodes {
52 databaseId
53 body
54 }
55 }
56 }
57 }
58 }
59 }
60}'
61
62# Resolve thread
63gh api graphql -f query='mutation {
64 resolveReviewThread(input: {threadId: "THREAD_ID"}) {
65 thread {
66 isResolved
67 }
68 }
69}'
70```
71
72### Monitoring for conflicts
73Keep your branch up to date with the base branch.
74- **Action**, Regularly rebase or merge the base branch (e.g., `main`) into your PR branch to catch conflicts early.
75
76### Quality PR descriptions
77Help reviewers by providing context.
78- **Good**, Include a summary of changes, why they were made, and how to test them. Link to related issues.
79
80## Anti-patterns to avoid
81
82- ❌ **The "Ghost" PR**, leaving a PR unattended for days while CI is failing or reviewers are waiting.
83- ❌ **Merging with failed checks**, never merge a PR if CI/CD checks have failed, unless there is an exceptional and documented reason.
84- ❌ **Ignoring negative reviews**, merging a PR without addressing a "Request Changes" review from a teammate.
85- ❌ **Too many commits**, avoid pushing dozens of tiny "fix typo" commits. Squash or clean up your history before the final merge.
86- ❌ **Leaving threads unresolved after addressing them**. Addressed threads should always be resolved to clear them for the reviewer.
87
88## KB Reference
89
90`~/vaults/baphled/3. Resources/Knowledge Base/AI Development System/Skills/Delivery/PR Monitor.md`
91
92## Related skills
93
94- `github-expert`, for advanced CLI usage.
95- `release-management`, for coordinating merges.
96- `documentation-writing`, for better PR descriptions.
97- `git-master`, for branch management.