← ClaudeAtlas

review-changelisted

Use to review a branch, pull request, diff, or uncommitted change for correctness, architecture, security, tests, migrations, and maintainability.
johnoconnor0/johns-os · ★ 0 · Code & Development · score 73
Install: claude install-skill johnoconnor0/johns-os
# Review Change ## Trigger Use when the user asks for a code review, PR review, diff review, architecture review of changes, or risk assessment. ## When To Use - Before merge or release. - After implementation. - When a change touches shared behavior, security, data, or deployment. ## Inputs Inspected - Git diff, PR context, or changed files. - Implementation plan and relevant lifecycle artifacts. - Tests and existing conventions. ## Workflow 1. Inspect the actual diff or changed files before making claims. 2. Compare behavior against the implementation plan, architecture decisions, and test strategy where available. 3. Review correctness, security, data/migration impact, API compatibility, operational risk, and maintainability. 4. Put findings first, ordered by severity, with concrete file references. 5. State tests actually run separately from recommended tests. 6. When the review surfaces a high-stakes, irreversible, or cross-cutting decision, recommend `run-engineering-council` before merge. 7. Write the review artifact and validate it with `python "${CLAUDE_PLUGIN_ROOT}/scripts/validate-artifact.py" <review artifact>`. ## Edge Handling - **Empty input** (no diff, PR, branch, or files named): ask the user for the diff, PR, or branch to review before proceeding (`empty-argument`) — do not invent a change to review. - **No changes to inspect** (the diff/PR/branch resolves to nothing): report that there is nothing to review rather than fabricating findings (`no-dif