# Review Pr Comments

> Manual-only cleanup of actual feedback on an existing PR: verify comments, batch valid in-scope fixes, respond, and leave the PR merge-ready.

- Skill: `hcaiano/review-pr-comments` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add hcaiano/review-pr-comments`
- Raw SKILL.md: https://api.skillmd.com/api/skills/hcaiano/review-pr-comments/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: hcaiano (https://skillmd.com/u/hcaiano)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/hcaiano/review-pr-comments

---


# 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 delivery 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, human-requested
changes addressed, every open thread fixed or answered, and GitHub reports the
PR mergeable — bot re-reviews are never a completion requirement.

Do not merge, force-push, modify `main`, create follow-up PRs (nit-cleanup
follow-up PRs are explicitly banned), or make product and architecture
decisions without authorization.