Review

Name: Review
Author: mag123c

ASecurity

Negative perspective code review (convention, quality, bugs, performance)

192 stars
0 votes
0 copies
4 views
Added February 8, 2026

toolsgoperformance

Security analysis

A100/100

Scanned February 12, 2026

npx -y skills add mag123c/toktrack --skill review --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Review?

Add the live security badge to your README. It updates with every re-scan.

[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/mag123c-review/badge)](https://www.skillsdirectory.com/skills/mag123c-review)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

Download with Pro

SKILL.md

---
name: review
description: Negative perspective code review (convention, quality, bugs, performance)
required_context:
  - .claude/ai-context/architecture.md
  - .claude/ai-context/conventions.md
---

# Review

## Purpose
Review implementation from a **negative perspective**. Goal is to find issues.

## Checklist

### 1. Convention Violations
- [ ] CLAUDE.md rule violations
- [ ] Naming convention violations
- [ ] Architecture layer violations

### 2. Code Quality
- [ ] Unnecessary complexity
- [ ] Duplicate code
- [ ] Magic numbers/strings
- [ ] Missing error handling

### 3. Potential Bugs
- [ ] Unhandled edge cases
- [ ] Off-by-one errors
- [ ] Missing null/None checks
- [ ] Race conditions

### 4. Performance
- [ ] Unnecessary allocations/copies
- [ ] O(n²)+ complexity
- [ ] Cache not utilized

### 5. Test Coverage
- [ ] Missing test cases
- [ ] Boundary value tests
- [ ] Error path tests

## Output
```markdown
## Review Result

### Issues Found
1. [CRITICAL/WARNING/INFO] Description

### Recommendations
- Suggested fixes

### Verdict
- [ ] PASS: Ready to commit → auto-call /wrap
- [ ] FAIL: Needs fixes → return to /implement
```

## Rules
- Goal is to find issues (no praise)
- Fix discovered issues before proceeding
- On FAIL → return to /implement

## Next Step
- **PASS**: immediately call `/wrap`
- **FAIL**: return to `/implement`, fix, restart chain