← ClaudeAtlas

review-changeslisted

Review branch changes or a PR for code quality, tests, documentation, and commit hygiene. Routes to the `review-loop` skill for a bounded review-apply-simplify convergence loop, or to `pr-reviewer` directly for a one-shot read-only review. Invoke with /review-changes.
mthines/agent-skills · ★ 13 · Code & Development · score 79
Install: claude install-skill mthines/agent-skills
## Routing Choose the path based on the argument shape: | Argument shape | Path | Reason | | --- | --- | --- | | no arg / `--report` | `Skill("review-loop", "<pr-url> [--critical\|--no-ci\|--external-review\|--interval S]")` on the current branch's open PR, or `pr-reviewer` directly if `--report` (one-shot, no apply) | Own PR convergence loop, or one-shot report. | | PR URL or `#<n>` | `Skill("review-loop", "<pr-url> [--critical\|--no-ci\|--external-review\|--interval S]")` — runs the bounded review-apply-simplify loop | Converges the PR regardless of who authored it (`pr-reviewer` detects `REVIEW_RELATION` itself). | `--no-ci`, `--external-review` and `--interval S` are pass-throughs: this skill never sets them itself, it only forwards what the user passed (see the note below). ```bash # --report routes straight to pr-reviewer, which --external-review exists to avoid. # Refuse the pair rather than accepting a flag this path cannot honour, mirroring # review-loop's refusal of --no-feedback --external-review. case "$ARGUMENTS" in *--report*) case "$ARGUMENTS" in *--external-review*) echo "--report needs pr-reviewer; drop --external-review or drop --report." exit 1 ;; esac ;; esac # Resolve the current branch's PR whenever no PR was named. Test for the ABSENCE of a # PR reference, never for an exact flag string: every flag in the table above is # combinable, so `= "--report"` left `--external-review`, `--critical`, `--no-ci`, and # `--interval S` with n