← Back to list

code-review
by rcmiller01
⭐ 0🍴 0📅 Jan 15, 2026
SKILL.md
name: code-review description: Review code for quality, security, and correctness with severity-based feedback
Code Review Skill
Review code and artifacts for quality, security, performance, and correctness.
When to Use
- Reviewing code before merge
- Security audit of implementation
- Performance review
- Checking code quality and best practices
Input Format
Review Type: [code | security | performance | general]
Code:
```[language]
[Your code here]
Context: [Optional - what this code does, requirements]
### Examples
**Code review:**
Review Type: code Code:
def process_payment(user_id, amount):
user = db.query(f"SELECT * FROM users WHERE id = {user_id}")
if user.balance >= amount:
db.execute(f"UPDATE users SET balance = balance - {amount}")
return True
return False
Context: Payment processing function for e-commerce checkout
**Security review:**
Review Type: security Code: [paste code]
## Output Format
```markdown
## Review Summary
**Decision**: [APPROVED | CHANGES REQUESTED | BLOCKED]
### Issues Found
#### 🔴 Critical (X)
| Issue | Location | Fix |
|-------|----------|-----|
| SQL Injection | line 2 | Use parameterized queries |
#### 🟠 High (X)
...
#### 🟡 Medium (X)
...
#### 🔵 Low (X)
...
### Positive Notes
- Good use of...
- Clean separation of...
### Recommendations
1. First priority fix
2. Second priority fix
Issue Categories
| Category | Description |
|---|---|
| security | Vulnerabilities, injection, auth issues |
| performance | N+1 queries, memory leaks, inefficiency |
| correctness | Bugs, logic errors, edge cases |
| style | Naming, formatting, conventions |
| documentation | Missing docs, unclear comments |
Severity Levels
| Level | Action | Criteria |
|---|---|---|
| 🔴 Critical | Must fix | Security vulnerabilities, data loss risk |
| 🟠 High | Should fix | Bugs, significant issues |
| 🟡 Medium | Recommended | Performance, maintainability |
| 🔵 Low | Nice to have | Style, minor improvements |
| ℹ️ Info | FYI | Suggestions, alternatives |
Decision Criteria
- APPROVED: No critical/high issues
- CHANGES REQUESTED: Has medium+ issues that should be addressed
- BLOCKED: Has critical security or correctness issues
Related Skills
- coder - Generate fixes for identified issues
- architect - Review architectural implications
- pm-validator - Validate against requirements
Score
Total Score
50/100
Based on repository quality metrics
✓SKILL.md
SKILL.mdファイルが含まれている
+20
○LICENSE
ライセンスが設定されている
0/10
○説明文
100文字以上の説明がある
0/10
○人気
GitHub Stars 100以上
0/15
○最近の活動
3ヶ月以内に更新がある
0/10
○フォーク
10回以上フォークされている
0/5
✓Issue管理
オープンIssueが50未満
+5
✓言語
プログラミング言語が設定されている
+5
○タグ
1つ以上のタグが設定されている
0/5
Reviews
💬
Reviews coming soon