review-changes

## 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...