Rust Review
Review Rust changes under Cloudflare workerd’s src/rust/ with a checklist focused on CXX FFI, unsafe code, JSG resources, and error handling.
Overview
Rust-review is an agent skill for the Ship phase that reviews workerd Rust in src/rust/ using the mandatory rust-review-checklist, with CXX FFI, unsafe, JSG, and error-handling focus.
Install
npx skills add https://github.com/cloudflare/workerd --skill rust-reviewWhat is this skill?
- Mandatory use of docs/reference/rust-review-checklist.md on every Rust review
- CXX FFI safety and unsafe Rust pattern scrutiny for workerd’s Rust/C++ boundary
- JSG resource conventions and workerd-specific error-handling expectations
- Checklist lineage adapted from existing C++ review skills in the same repo
- Scoped to src/rust/—not a generic Rust style linter
Adoption & trust: 1 installs on skills.sh; 8.3k GitHub stars; 3/3 security scanners passed (skills.sh audits); trending (+100% hot-view momentum).
What problem does it solve?
Rust patches in workerd cross FFI and runtime boundaries where generic Rust lints miss CXX, JSG, and unsafe conventions that break production workers.
Who is it for?
workerd contributors reviewing src/rust/ diffs in Claude Code, Cursor, or Codex before opening or updating a PR.
Skip if: Greenfield Rust apps outside workerd, frontend-only changes, or reviews where you refuse to load the repo’s official checklist file.
When should I use this skill?
Load this skill when reviewing Rust code in src/rust/.
What do I get? / Deliverables
Review comments map to docs/reference/rust-review-checklist.md so Rust changes meet workerd’s FFI, resource, and error-handling bar before merge.
- Checklist-aligned review notes for CXX FFI and unsafe sections
- JSG resource and error-handling findings with severity implied by checklist categories
Recommended Skills
Journey fit
Ship is where contributor Rust patches are scrutinized before merge; this skill is loaded explicitly when reviewing code, not when scaffolding new features. Review subphase matches the documented trigger: reviewing Rust in src/rust/ against the project’s rust-review-checklist.
How it compares
Use instead of generic ‘review my Rust’ prompts when you need workerd-specific CXX FFI and JSG checklist gates, not a standalone Clippy pass.
Common Questions / FAQ
Who is rust-review for?
Maintainers and solo contributors working in Cloudflare workerd’s Rust tree who want agent-assisted PR review that follows the same checklist-driven process as C++ reviews in the project.
When should I use rust-review?
During Ship → review whenever you review or self-review Rust under src/rust/, especially diffs touching unsafe, CXX bridges, or JSG resource bindings.
Is rust-review safe to install?
It is read-oriented review guidance tied to a public checklist path; confirm scope in the Security Audits panel on this Prism page and avoid pasting secrets into review threads.
SKILL.md
READMESKILL.md - Rust Review
**Always** use the `docs/reference/rust-review-checklist.md` file when reviewing Rust code.