review-pr-comments

Solid

On-demand review cleanup for an existing PR that has actual feedback: verify each comment, apply valid in-scope fixes in one batched round, respond where useful, and leave the PR merge-ready. Only runs when the user explicitly invokes review-pr-comments.

Code & Development 3 stars 0 forks Updated today MIT

Install

View on GitHub

Quality Score: 80/100

Stars 20%
20
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
92
Issue Health 10%
80
License 10%
100
Description 5%
100

Skill Content

# Review PR Comments Handle real review feedback on an existing PR. This is an on-demand, exception path: most PRs have no comments because cloud auto-review is disabled and the heavy review already happened locally (ship-it's dual-review gate). Do not summon bot reviews (`@codex review`, `@coderabbitai review`), and never treat a bot's silence as something to wait for. Process, at most TWO rounds (a round = collect → triage → one batched commit → push → required CI green on the new head): 1. Collect all open feedback: human reviews, bot comments, unresolved and outdated threads, failing checks. 2. Triage each finding against the code: - **Fix in the batch:** real correctness, security, or data-integrity issues, and anything a human reviewer explicitly requested. - **Reply, don't push:** style nits, defensive hardening for invariants that already hold, severity-inflated or duplicate findings. Answer the thread, resolve it, move on. A nit is never a reason for a push. - **Surface to the user:** product/architecture decisions, scope expansions, conflicting reviewer guidance. 3. Validate the complete head (original changes plus fixes), commit and push once, and wait for required CI (poll at 60–120 s intervals, never tight loops). After two rounds, stop pushing: report remaining items with your triage and recommendation. Feedback arriving after your final push does not reopen the loop. Success: required checks green on the final head, hum...

Details

Author
hcaiano
Repository
hcaiano/skills
Created
2 months ago
Last Updated
today
Language
JavaScript
License
MIT

Similar Skills

Semantically similar based on skill content — not just same category