核心用法
rust-code-review 是一个面向 Rust 生态系统的深度代码审查技能,采用结构化审查流程确保审查质量:
1. 前置检查:首先读取 Cargo.toml 确认 Rust Edition(2018/2021/2024)和 MSRV,识别关键依赖(thiserror/anyhow/tokio/serde 等)
2. 上下文感知审查:读取完整函数而非仅 diff,避免断章取义
3. 分类检查清单:覆盖 9 大核心领域,每个领域配有独立参考文档
4. 分级严重性评估:Critical/Major/Minor/Informational 四级分类
5. 验证协议:最终加载 beagle-rust:review-verification-protocol 完成交叉验证
显著优点
- Edition 2024 完整支持:涵盖 RPIT 精确生命周期捕获、
unsafe_op_in_unsafe_fn、r#gen保留关键字、!类型回退等破坏性变更 - 系统化审查框架:独创 Gates 机制(Crate context → Expanded read → Severity match → Verification protocol),强制避免"脑内验证"
- 领域深度细分:9 份独立参考文档覆盖所有权借用、生命周期方差、错误处理、异步并发、并发原语、类型布局、unsafe 深潜等
- 上下文敏感规则:明确区分生产代码与测试代码(如
unwrap()在测试中可接受),避免无效告警 - 性能与安全并重:既关注
clone()陷阱、collect()冗余等性能问题,也识别 use-after-free、数据竞争、UB 等内存安全问题
潜在局限
- 外部依赖:需配合
beagle-rust:rust-best-practices技能获取性能优化、类型状态模式、Clippy 配置等扩展内容 - 验证协议开销:强制性的四步 Gates 机制在小型审查中可能显得繁琐
- 参考文档未内联:核心判断依据分散在独立 markdown 文件中,离线场景需预加载
- MSRV 敏感:部分建议(如
LazyLock替代once_cell)需要 Rust 1.80+,低 MSRV 项目需手动过滤
适合人群
- Rust 代码审查者:需要系统性检查清单避免遗漏关键问题
- Rust 2024 迁移团队:需要精确识别 Edition 变更点
- unsafe 代码作者:需要安全契约文档化与 soundness 验证流程
- 开源项目维护者:需要标准化 PR 审查流程
- 异步/并发代码开发者:需要 Send/Sync 边界与运行时阻塞检查
常规风险
- 误报可能:虽然设计了上下文敏感规则,但复杂的生命周期推导仍可能导致过度警告
- 严重性误判:依赖使用者正确执行 Severity Calibration,跳过 Gates 可能导致 Critical 问题被降级
- Edition 检测依赖:若
Cargo.toml未正确识别,后续所有 Edition 特定检查可能失效 - 参考文档版本漂移:外部引用的 markdown 文件若未及时更新,可能包含过时建议