kDiT Code Review
适用场景
- Review PR / MR
- Review 自己或他人的代码改动
- 用户要求
/kdit-code-review或 "帮我 review 一下"
评审维度(按优先级)
| 优先级 | 维度 | 说明 |
|---|---|---|
| P0 | 架构合规 | 基于 .skills/02_architecture/ 和 .skills/03_standards/ 检查 |
| P1 | 逻辑正确性 | 边界条件、资源泄漏、并发安全、数据流完整 |
| P1 | 单元测试覆盖 | kdit/ 核心代码变更必须包含对应测试,否则阻塞 |
| P2 | 安全漏洞 | OWASP Top 10 |
| P3 | 代码质量 | 格式规范、复杂度 |
详见 → code_review.md
评审流程
- 获取改动范围 —
git diff或 PR diff,确认涉及的模块 - 判定测试要求 — 检查 diff 是否涉及
kdit/核心代码;若涉及,diff 中必须包含tests/kdit/下的测试变更,否则直接标记为 P1 阻塞项 - 按优先级逐项检查 — P0 架构合规 → P1 逻辑正确性 + 单元测试覆盖 → P2 安全漏洞 → P3 代码质量
- 查阅关联规范 — 根据改动模块查阅对应的架构和编码规范文档
- 输出评审结果 — 按优先级列出发现的问题,附带具体文件/行号和修复建议
关联规范
| 改动模块 | 必查文档 |
|---|---|
| Node 相关 | node.md、pin-hub.md、node-and-tensor.md |
| Pipeline 相关 | pipeline.md |
| Generator 相关 | generator.md |
| Engine/Executor | overview.md |
| Adapter 相关 | adapter.md |
| Key/Pool 相关 | key-system.md、pool-key.md |
相关 skill
| Skill | 何时调用 |
|---|---|
/kdit-architecture |
P0 架构合规检查:查阅依赖方向、Node 签名、PinHub 机制等设计约束 |
/kdit-standards |
P0 编码规范检查:查阅 Import/类型注解/异常处理/命名规则 |
/kdit-quality |
P3 质量检查:查阅测试覆盖要求和格式规范 |