Mindset
Apply this skill when reviewing code changes for style conformance. ALWAYS run before merge. NEVER skip for hotfixes.
When to Use
- A pull request is opened or updated
- A commit is pushed to a feature branch
- A config change requires style validation
When NOT to Use
- Auto-generated files (migrations, vendor/)
- Does not apply to binary file changes
- Skip when the commit is a revert with no logic changes
Procedures
- Identify the diff scope from the PR or commit
- Run the style linter against changed files
- Report violations with line numbers and fix suggestions
- Block merge if critical violations are found
Anti-Patterns
BAD: Skipping style checks for "small" commits
# BAD — even small commits can introduce style debt
git push --no-verify
GOOD: Always run checks on every commit regardless of size.