# Reviewing Code

> Code review workflow for evaluating changes against standards. Use when reviewing pull requests, commits, or diffs. Use when this capability is needed.

- Skill: `tomevault-io/reviewing-code-2` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add tomevault-io/reviewing-code-2`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tomevault-io/reviewing-code-2/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Productivity
- Author: tomevault-io (https://skillmd.com/u/tomevault-io)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/tomevault-io/reviewing-code-2

---


# Code Review Standards

Systematic review for quality and consistency.

## Review Checklist

### Type Safety
- [ ] No `any` types
- [ ] Minimal `as` casting
- [ ] E2E type coverage

### Clarity
- [ ] Descriptive names
- [ ] Early returns
- [ ] Flat code structure
- [ ] No magic strings/numbers

### React (if applicable)
- [ ] Pure components
- [ ] React Query for data
- [ ] Suspense/Error boundaries

### Security
- [ ] Input validation
- [ ] Query builders (no raw SQL)
- [ ] No exposed secrets

### Accessibility
- [ ] Semantic HTML
- [ ] ARIA labels
- [ ] Keyboard navigation

### Testing
- [ ] Tests behavior
- [ ] Bug fixes have tests

## Output Format

```
## Summary
[What changed]

## Issues
### Critical (blocks merge)
- [Issue]: [Fix]

### Suggested (improve quality)
- [Issue]: [Fix]

## Status
✅ Approved | ⚠️ Minor changes | ❌ Needs work
```

## Priority

1. **Critical**: Security, type-safety, breaking changes
2. **High**: Maintainability, readability
3. **Medium**: Naming, minor refactoring
4. **Low**: Style preferences

---
> Converted and distributed by [TomeVault](https://tomevault.io/claim/x-tri) — claim your Tome and manage your conversions.
<!-- tomevault:4.0:skill_md:2026-04-15 -->