用途:对当前分支的改动进行本地代码审查,对照 Materialize 项目标准检查。触发语包括“review my code”“review my changes”“check my diff”“does this look ok”等。
主要能力:解析参数(PR 编号、基准分支或空);按可用方式获取 diff:给定 PR 编号用 gh pr diff 123,有 git 用 git diff <base>...HEAD(默认 main),有 jj 用 jj diff -r <revset>(默认从 trunk);同时获取文件列表(git/jj 加 --stat,PR 用 gh pr diff 123 --stat)。按检查清单审查,结果分为 Blocking、Strong suggestions、Nits。
检查清单覆盖:测试(行为变更需有新增或修改测试,SQL 行为看 test/sqllogictest/ 的 .slt,wire/protocol 看 test/pgtest/ 的 .pt,Rust 逻辑加单元测试并用 cargo test -p mz-<crate> 运行);Rust 代码风格(导入顺序 std→外部 crate→crate::,错误用 thiserror,异步用 ore::task::spawn,测试用 #[mz_ore::test]);SQL 风格(关键字大写、标识符小写、函数名与括号间无空格);错误消息规范;敏感数据处理(使用 mz_ore::secure::{SecureString, SecureVec} 或 zeroize::Zeroizing<T>,敏感类型不派生 Clone/Debug);架构(简洁、无特殊分支、封装边界、新依赖需论证);收尾(无遗留 XXX/FIXME/dbg!/println!、无无关格式改动、新公开项有文档注释);发布说明;单一语义变更原则。
使用方式:作为技能调用,传入 PR 编号、基准分支或留空。审查为只读,不修改任何代码。
限制:仅适用于 Materialize 项目及其规范;具体安装与集成方式资料未提供;不执行代码修改。