review-looplisted

Bounded review-apply-resolve convergence loop for a GitHub PR (draft PRs are fine). Runs up to N=5 iterations of pr-reviewer → implement-suggestion (--resolve-all) → polish simplify, converging until every review thread is resolved — through a fix OR a reply (answered question, recorded rationale) — so the PR is left with zero open threads (only genuine human-judgment flags stay open). On convergence it also refreshes the PR description to match the shipped diff and, best-effort, notes the linked Linear ticket. Use after opening a draft PR to converge the branch to a clean, review-ready state before undrafting. Also converges CI: after each iteration's push it reads the check state and delegates a red mechanical failure to ci-auto-fix, so convergence means zero open threads AND CI not red (--no-ci opts out; create-pr and autonomous-workflow pass it because they own their own CI phase). On a UI PR it also runs the committed preview-spec block against the live preview deployment once at exit (report-only, never
mthines/agent-skills · ★ 13 · Code & Development · score 79

Install: claude install-skill mthines/agent-skills

# review-loop — Bounded Review-Apply-Resolve Convergence Drive a PR from its initial draft state to a clean, review-ready state by iterating `pr-reviewer` → `implement-suggestion --resolve-all` → `polish simplify` until **every review thread is resolved** or the cap is reached, then refresh the PR description to match the shipped diff. A thread is resolved when it is **either fixed** (a code change landed) **or answered** (a reply — the answer to a question, the agent's take on a discussion, or a rationale for a declined suggestion). The only threads left open at convergence are genuine **human-judgment flags**: a real potential issue the agent will neither auto-apply nor honestly decline. That safety valve means the loop can never green-wash a PR by resolving a live finding — it surfaces it instead. This skill is an **orchestrator**. It contains no quality rules of its own. It sequences existing pieces, each owning its own domain: 1. `pr-reviewer` — finds issues (read-only; posts one `COMMENT` review; on a re-review resolves its own addressed threads). 2. `implement-suggestion --resolve-all` — applies actionable findings **and** replies-to-and-resolves the non-fix threads it can honestly close (single-shot, no `--watch`). 3. `Skill("polish", "simplify")` — applies Class M mechanical refactors behind a confidence gate. 4. `ci-auto-fix` — diagnoses and fixes a red check after the iteration's push (skipped under `--no-ci`). 5. On convergence — refreshes the PR description (