review-process
Installation
SKILL.md
Review Process
Apply the execution contract before this procedure.
Standardized code review process for Ruby code changesets.
Quick Reference
| Severity | Definition | Target Action |
|---|---|---|
| Critical | Security issue, data corruption risk, crash/unhandled exception | Must resolve; blocks merge |
| Major | Logical flaw, structural issue, design smell, missing tests | High priority to fix before merge |
| Minor | Inefficient query, duplicate code, suboptimal naming, missing YARD | Optional/nice-to-have in this changeset |
| Nitpick | Formatting, style guides, purely cosmetic | Acknowledge; do not block merge |