---
name: choutos/pr-review
source: https://app.decimal.ai/s/choutos-pr-review@1/SKILL.md
source_sha256: a265a7fd5135
---

# PR Review

## Process

1. Read the PR description and linked issue (if any)
2. Review the diff file by file
3. Provide feedback privately first (never post to GitHub/GitLab without explicit approval)

## What to Check

### Correctness
- Does the code do what the PR claims?
- Are edge cases handled?
- Could this break existing functionality?

### Quality
- Clear naming and structure?
- Appropriate error handling?
- Tests added or updated?
- No hardcoded values that should be config?

### Security
- No secrets in code?
- Input validation present?
- SQL injection / XSS risks?

### Operations
- Will this affect deployment?
- Database migrations needed?
- Config changes required?

## Feedback Format

For each issue found:
```
**[severity]** file:line — description

Suggestion: ...
```

Severity levels: `critical` (blocks merge), `important` (should fix), `nit` (optional improvement).

## Rules

- **Never post reviews publicly without explicit approval**
- Acknowledge what's done well, not just problems
- Be specific: "line 42 has X" beats "the code has issues"
- Suggest fixes, don't just point out problems