SuperAPI AI 工具集

rust-code-reviewer

编程与开发

查看项目

更新于 10-16 07:24

技能介绍

该技能用于对 Rust 代码执行严格评审,体现系统级评审模式与设计工艺原则,尤其适合 switchyard 项目的 crates/switchyard-core、crates/switchyard-translation、crates/switchyard-components、crates/switchyard-server 以及 Python/Rust FFI 表面 crates/switchyard-py;对非上述区域倾向给出建议而非阻断性意见。核心评审哲学包括:简洁优于炫技,警惕过度抽象;代码精炼、文档克制;系统级思考内存分配、异步运行时、锁竞争与延迟;先追问设计归属再机械修改;使用符合项目习惯的 Result 错误处理(anyhow/thiserror),留意非测试代码中的克隆、解包及多余的 Arc/Mutex;审慎检查 tokio、通道、取消与共享状态;命名直接明确;控制 PR 差异范围;tracing 信息要有意义而非嘈杂。评审方式:默认只评审最近编写或修改的代码,可先用 git status、git diff --stat、git diff 确定目标;根据差异信号查阅对应 references/*.md;反复多轮查找问题;输出按严重程度分组(阻断/重要/风格),尽量给出具体 file:line。通用规则包括:生产代码禁止 unwrap()/expect()(项目甚至禁止测试中使用 expect());用 tracing 而非 log;使用结构化 tracing 字段;日志级别恰当;不要条件反射地加 Arc<Mutex<…>>;不要给可克隆类型再包 Arc;去掉不必要的 clone();短临界区且不跨 await 时优先 parking_lot::RwLock;用 Drop 做资源清理;优先标准库/tokio 原语;不要因口味改动错误信息或接口;指出范围蔓延。异步重点包括跨 await 持锁、阻塞执行器线程、spawn 任务关闭与错误处理、取消行为、通道背压。同时有命名与注释规范、Switchyard 特有的 PyO3 边界不得让 panic 跨入 Python、流式翻译 SSE 事件顺序与格式兼容性,以及测试行为覆盖优先、禁止 expect 等要求。使用限制:资料未提供该技能的安装命令、收费或价格、平台支持与效果数据,暂未提供。