docs(pr): add slow-check probe + AI re-reviewer churn tips#321
Merged
Conversation
Two additive doc improvements to the /pr skill loop (no control-flow change): - Step 4a: add a check-runs status probe to distinguish a genuinely-running slow CI check (in_progress + real started_at) from a stuck/queued one (queued >3min or no run), preventing both premature give-up and silent infinite waits. - Step 4c: note that AI re-reviewers (CodeRabbit, claude-review/robothanx CI jobs) re-review on every push and surface new incremental findings; expect 2-4 cycles to converge and don't treat a new finding after a clean round as loop-stuck — the "same failure 3+ times" guard targets recurring findings. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Address review findings on the Step 4a check-runs probe: - note the <check> placeholder must match the exact gh pr checks name (a typo yields a false "no run") - make the queued threshold measurable via created_at instead of an unanchored "~3 minutes" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two additive documentation improvements to the /pr skill loop (no control-flow change):
Co-Authored-By: Claude noreply@anthropic.com