| name | review-change |
| description | Use this skill when the user asks for a code review, a correctness pass, a check before PR, or asks whether a Symfony/API Platform change follows project conventions. Trigger on "review this", "check my changes", or "is this implementation correct?". |
| allowed-tools | Read, Grep, Glob |
Review Change
Use this skill to review a change for correctness, architecture, and test completeness.
Read first:
.claude/rules/architecture.md.claude/rules/testing.md.claude/rules/security.md
Read as needed:
.claude/patterns.md- the relevant diff and nearby files
Workflow:
- Inspect the diff or the requested scope.
- Check architecture and layer ownership.
- Check validation, persistence boundaries, and error handling consistency.
- Check test coverage for the changed behavior.
- Check whether the change is likely to pass the repository quality gates.
Response format:
- findings first, ordered by severity
- then assumptions or open questions
- then a short summary only if useful
Rules:
- Focus on correctness, security, regressions, and missing tests.
- Do not spend time on style nits already covered by tooling unless they expose real risk.
