What problem does it solve?
This Skill provides a structured pre-merge review checklist to ensure code quality and security, saving time and reducing the likelihood of post-merge issues.
Core Features & Use Cases
- Comprehensive Checklist: A detailed list of items to check before merging code, based on common issues caught by automated reviewers.
- Database Operations: Ensures multi-step database operations are wrapped in transactions and migrations are atomic.
- Security & Data Safety: Validates that sensitive data is handled securely and securely, with no secrets in logs or error messages.
- String Safety: Ensures string operations are safe and secure, with no byte-index slicing or case-sensitive path comparisons.
- Trait Wrappers & Decorator Chain: Checks that new trait methods are properly delegated and tested.
- Tests: Ensures tests are written correctly, with no mutation of global statics and no real network requests.
- Comments & Documentation: Validates that code comments and documentation are up-to-date and accurate.
Quick Start
To use the review-checklist skill, activate it with the command "run review-checklist" and follow the checklist provided.