# code-review

> Comprehensive code review checklist for pull requests

- Author: Angie Jones
- Repository: block/agent-skills
- Version: 20260206120252
- Stars: 2
- Forks: 0
- Last Updated: 2026-02-06
- Source: https://github.com/block/agent-skills
- Web: https://mule.run/skillshub/@@block/agent-skills~code-review:20260206120252

---

---
name: code-review
description: Comprehensive code review checklist for pull requests
author: goose
version: "1.0"
tags:
  - development
  - quality
  - review
---

# Code Review Checklist

When reviewing code, check each of these areas:

## Functionality

- [ ] Code does what the PR description claims
- [ ] Edge cases are handled
- [ ] Error handling is appropriate

## Code Quality

- [ ] Follows project style guide
- [ ] No hardcoded values that should be configurable
- [ ] Functions are focused and well-named

## Testing

- [ ] New functionality has tests
- [ ] Tests are meaningful, not just for coverage
- [ ] Existing tests still pass

## Security

- [ ] No credentials or secrets in code
- [ ] User input is validated
- [ ] SQL queries are parameterized

## Performance

- [ ] No obvious performance issues
- [ ] Database queries are optimized
- [ ] No unnecessary loops or iterations

## Documentation

- [ ] Public APIs are documented
- [ ] Complex logic has comments explaining "why"
- [ ] README updated if needed