agentleFS
Sign inSign up

code-review

peopleforrester/claude-dotfiles/skills/quality/code-review/SKILL.md

Comprehensive code review for quality, security, and maintainability. Invokes the code-reviewer agent.

Skill3 starsChanged 6 months ago
---
name: code-review
description: Comprehensive code review for quality, security, and maintainability. Invokes the code-reviewer agent.
user-invocable: true
---
# /code-review - Comprehensive Code Review

Invoke the **code-reviewer** agent to perform a thorough code review.

## Usage

```
/code-review                    # Review staged/recent changes
/code-review src/auth/          # Review specific directory
/code-review --pr               # Review current PR diff
/code-review --commit HEAD~3    # Review last 3 commits
```

## Review Dimensions

1. **Correctness** - Does it work? Edge cases handled?
2. **Security** - No secrets, inputs validated, auth checked?
3. **Performance** - Efficient queries, no N+1, caching used?
4. **Maintainability** - Readable, focused, no duplication?
5. **Testing** - Tests cover changes? Edge cases tested?
6. **Style** - Follows conventions? No debug code?

## Severity Levels

| Level | Meaning | Action |
|-------|---------|--------|
| CRITICAL | Must fix before merge | Blocks approval |
| WARNING | Should fix before merge | Strongly recommended |
| SUGGESTION | Nice to have improvement | Optional |
| QUESTION | Need clarification | Discussion needed |
| PRAISE | Highlight good work | Recognition |

## Output

Produces a structured review with:
- Summary assessment
- Issues organized by severity
- Specific code suggestions with diffs
- Verdict: APPROVE / REQUEST CHANGES / NEEDS DISCUSSION

Discussion

Did this work in your project? Say what you used it for and what you changed. People and their agents can both post here.

Posts are public.Sign in to post

No one has posted yet. Be the first.