OSSF Scorecard项目中Code-Review检查的改进与优化

在开源项目质量评估工具OSSF Scorecard中，Code-Review检查项用于评估项目是否在合并代码前进行了人工代码审查。近期发现该检查项在处理FreeBSD项目的代码审查标记时存在不足，需要进行优化和改进。

问题背景

FreeBSD项目使用独特的代码审查标记方式，主要包括两种形式：

1. 在提交信息中使用"Reviewed by:"标签
2. 包含Phabricator审查链接，格式为"Differential Revision: https://reviews.freebsd.org/D###"

当前OSSF Scorecard的Code-Review检查未能完全识别这些标记，导致对FreeBSD项目的代码审查实践评估不准确。

技术分析

现有实现的问题

当前实现中，Phabricator链接的正则表达式匹配不够完善，仅能匹配"https"部分，导致：

• 多个不同的审查链接被错误地归为同一组
• 审查计数不准确
• 评估分数偏低

解决方案

经过技术讨论，确定以下改进方案：

1. 优化正则表达式模式，从： Differential Revision:[^\r\n]*(D\d+) 改为更精确的匹配模式，确保能正确提取Phabricator审查编号

2. 处理特殊情况：

   • 同一审查链接出现在多个提交中（如相关功能分多次提交）
   • 单个提交包含多个审查链接（如合并多个审查）

实现细节

在代码实现层面，审查结果会按以下逻辑分组：

1. 每个唯一的Phabricator审查编号形成一个组
2. 无审查标记的提交各自独立成组
3. 最终审查比例为"有审查的组数/总组数"

例如：

• 4个提交中：
  • 2个引用同一Phabricator审查
  • 2个无审查标记
• 将被分为3组（1审查组+2无审查组）
• 审查比例为1/3

技术影响

这一改进将：

1. 更准确地反映FreeBSD项目的代码审查实践
2. 保持对使用Phabricator的其他项目的兼容性
3. 提高评估结果的客观性和可信度

最佳实践建议

对于开源项目维护者，为确保Code-Review检查能正确识别审查记录，建议：

1. 统一使用标准的审查标记格式
2. 确保审查链接完整且可访问
3. 在提交信息中明确包含审查证据
4. 考虑使用自动化工具验证审查标记是否符合规范

这一改进不仅解决了FreeBSD项目的特定问题，也为其他使用类似审查流程的开源项目提供了更好的评估支持。