Rust review
Review Rust changes for correctness, ownership, API ergonomics, error handling, unsafe boundaries, async behavior, tests, performance, features, dependency choices, and greenfield conventions. Use for review-only Rust tasks.From its SKILL.md
npx -y skills add nyquistwilder/personal-pi --skill rust-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
1.7 KB, 313 tokens by cl100k_base, as published. Nobody here has run it
Rust Review
Rule
Provide actionable, evidence-backed findings prioritized by correctness, safety, public contracts, and maintainability. Do not rewrite code unless explicitly asked.
Review Focus
Check ownership design, lifetime complexity, public API ergonomics, error types, panic paths,
unsafe/FFI justification, feature flags, dependency choices, async cancellation, task
lifecycle, tests, docs, security boundaries, observability fields, and performance risks.
Antipatterns To Flag
unwrap/expecton user input, I/O, network, database, or request path failures.- Public APIs overfit to internal lifetimes or expose implementation details.
- Traits with a single implementation and no clear boundary value.
- Default features that surprise library users.
- Tokio runtime creation inside libraries.
Arc<Mutex<_>>used as a design shortcut.- Unsafe blocks without local safety comments and tests/Miri evidence where practical.
Workflow
- Inspect the diff,
Cargo.toml, features, tests, and nearby code. - Run or recommend focused checks:
cargo test,cargo clippy,cargo fmt --check,cargo deny check,cargo audit, orcargo nextestwhen configured. - Identify blocking issues before optional style suggestions.
- Tie findings to concrete code and explain impact.
- Suggest small idiomatic fixes.
Completion
Return findings by severity, validation performed or skipped, and areas not reviewed.
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.