Code Review 规则的数目是否有限制？

[caiyuchao]

open code review是如何保证的rules都被模型加载的，比如c.md里写了很多条规则，怎么保证规则不被模型忽略？ 或者建议多少条目。

[lizhengfeng101]

感谢提问！这是一个很好的问题。

设计思路

OpenCodeReview 的规则系统是在语言驱动的 AI Agent 设计之上增加了一层规则引擎，做到按文件类型精准加载对应的规则。

具体来说：

1. 规则按文件类型精准匹配：系统内置了一套 glob pattern 匹配机制（见 system_rules.json），当审查 .java 文件时只加载 Java 相关规则，审查 .rs 文件时只加载 Rust 相关规则，审查 pom.xml 时只加载 Maven 相关规则。这样每次审查只注入与当前文件直接相关的规则，而不是把所有语言的规则一股脑塞给模型。

2. 规则以结构化 Prompt 形式注入：匹配到的规则会作为 system prompt 的一部分发送给模型，引导模型在审查时关注这些维度。

关于"能否保证规则不被忽略"

需要坦诚地说：没有办法 100% 确保模型严格遵守每一条规则。这是 LLM 驱动的代码审查与传统静态分析工具的本质区别——静态分析工具是确定性的程序执行，而 LLM 是概率性的。

这套规则引擎的定位是：在 LLM 驱动的审查场景下，提供一个表现更好的方案，而非传统程序意义上的百分百满分答案。通过精准匹配 + 结构化的 prompt 注入，实测下来规则的遵循率表现良好，但不能承诺绝对。

规则条数建议

规则不需要像《Java 开发手册》那样写几百条面面俱到。建议参考项目内置的规则写法：

• 通用规则（default.md）：约 20 行，聚焦 5 个维度（正确性、安全、性能、可维护性、测试覆盖）
• Java 规则（java.md）：约 40 行，覆盖拼写错误、死代码、逻辑错误、性能问题、线程安全 5 个类别
• Rust 规则（rust.md）：约 50 行，针对所有权、错误处理、unsafe、并发等语言特有维度

核心原则是：聚焦高价值、容易被忽视的审查维度，每个类别列出关键的检查点即可。规则越精炼，模型的遵循效果越好。

如果你有自定义规则的需求，可以通过项目级 .opencodereview/rule.json 配置，支持按 glob pattern 指定不同文件的审查规则，也支持与系统内置规则合并（merge_system_rule: true）。