$npx -y skills add Geeksfino/openskills --skill code-reviewReviews code for quality, best practices, and potential issues. Use when asked to review, audit, or check code for problems.
| 1 | # Code Review Skill |
| 2 | |
| 3 | Perform thorough code reviews following this methodology. |
| 4 | |
| 5 | ## Review Checklist |
| 6 | |
| 7 | ### 1. Correctness |
| 8 | - Does the code do what it's supposed to do? |
| 9 | - Are there any logic errors? |
| 10 | - Are edge cases handled? |
| 11 | |
| 12 | ### 2. Security |
| 13 | - Input validation and sanitization |
| 14 | - Authentication and authorization |
| 15 | - Sensitive data handling |
| 16 | - SQL injection, XSS, and other vulnerabilities |
| 17 | |
| 18 | ### 3. Performance |
| 19 | - Algorithm complexity |
| 20 | - Unnecessary computations |
| 21 | - Memory leaks or inefficient memory usage |
| 22 | - Database query optimization |
| 23 | |
| 24 | ### 4. Maintainability |
| 25 | - Code readability and clarity |
| 26 | - Appropriate naming conventions |
| 27 | - Single responsibility principle |
| 28 | - DRY (Don't Repeat Yourself) |
| 29 | |
| 30 | ### 5. Testing |
| 31 | - Test coverage |
| 32 | - Edge case testing |
| 33 | - Integration tests where appropriate |
| 34 | |
| 35 | ## Output Format |
| 36 | |
| 37 | Structure your review as: |
| 38 | |
| 39 | ``` |
| 40 | ## Summary |
| 41 | [One paragraph overview] |
| 42 | |
| 43 | ## Critical Issues |
| 44 | [Must fix before merge] |
| 45 | |
| 46 | ## Suggestions |
| 47 | [Nice to have improvements] |
| 48 | |
| 49 | ## Positive Observations |
| 50 | [What's done well] |
| 51 | ``` |
| 52 | |
| 53 | ## Guidelines |
| 54 | |
| 55 | - Be constructive, not critical |
| 56 | - Explain *why* something is an issue |
| 57 | - Suggest specific fixes when possible |
| 58 | - Acknowledge good patterns and practices |