Rust Code Review

Review the actual diff and its callers, not an isolated snippet or a generic checklist. Prioritize defects that can change behavior, violate invariants, expose data, deadlock, panic unexpectedly, or break public APIs.

Scope and Routing

Use this skill to review:

• ownership, borrowing, lifetime, and drop behavior;
• unsafe preconditions and safe-wrapper soundness;
• error propagation, panic paths, partial updates, and rollback;
• concurrency, cancellation, lock ordering, and task lifecycle;
• public API, semver, feature, target, and MSRV compatibility;
• allocation, cloning, blocking, serialization, and hot-path costs;
• tests, documentation, dependencies, and operational failure paths.

Route format and lint configuration to rust-style-clippy, deep unsafe or ABI analysis to rust-unsafe-ffi, concurrency design to rust-concurrency, dependency resolution to rust-cargo-build, test implementation to rust-testing, and API shape decisions (trait sealing, newtype design, error taxonomy, builder patterns) to rust-api-design.

Workflow