# Review

> Review code changes for quality, security, and best practices

- Skill: `martinacostadev/review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add martinacostadev/review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/martinacostadev/review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Security
- Author: martinacostadev (https://skillmd.com/u/martinacostadev)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/martinacostadev/review

---


Review code changes following senior developer standards.

## Review Process

1. **Gather Context**
   - If PR number provided, run `gh pr view $ARGUMENTS --json files,body`
   - If file path provided, read the file
   - If nothing provided, run `git diff` for unstaged changes

2. **Analyze Changes**
   - Check for SOLID principle violations
   - Look for security issues
   - Identify performance concerns
   - Review TypeScript types
   - Check test coverage

3. **Generate Report**

## Review Checklist

### Code Quality
- [ ] Clear, descriptive naming
- [ ] Functions under 50 lines
- [ ] No code duplication
- [ ] Proper error handling

### TypeScript
- [ ] No `any` types
- [ ] Explicit return types
- [ ] Proper null handling

### React Patterns
- [ ] Hooks rules followed
- [ ] Proper dependency arrays
- [ ] No unnecessary re-renders

### Security
- [ ] Input validation
- [ ] No XSS vulnerabilities
- [ ] No exposed secrets

## Output Format

```markdown
## Code Review Summary

### Overview
[Brief description of changes]

### Strengths
- [What's done well]

### Issues
#### Critical
- [Must fix before merge]

#### Suggestions
- [Recommended improvements]

### Security
[Any security concerns]
```

Target: $ARGUMENTS