rust-review
Installation
SKILL.md
Rust Review
Review Rust changes for correctness, clear ownership, controlled allocation, async safety, error model, maintainable APIs, and measured performance claims.
Process
- Inspect the diff and surrounding code before applying checklist items.
- Check correctness, boundaries, security, data flow, observability, tests, and deployment impact for the changed behavior.
- Report concrete findings first, ordered by severity, with file references and affected behavior.
- Call out missing tests or residual risk only when it is tied to the reviewed change.
Large PR Subagent Review
For broad or materially risky pull requests, recommend focused additional review only when independent expertise is likely to improve the verdict. Judge this from concrete risk, novelty, blast radius, reversibility, and context size—not file count or category matching alone. Small and conventional changes should remain one pragmatic pass.
When additional reviewers add value, split only by independent material surfaces. Each returns concrete evidence-based findings; the main agent removes duplicates and false positives and owns the proportional final verdict.