vibecoding-review

description	VibeCoding 审查与质量评估。由 vibecoding-workflow 编排调度到测试验证(T)阶段时使用。也可在用户直接要求'审查代码''review''检查质量'时独立触发。包含测试验证、Quality Gate 四维评估。
name	VibeCoding Review

审查阶段

T 测试验证

Path A (快速，含简化 Quality Gate)

1. 验证 bug 已修复或配置已生效
2. 确认没有引入新问题
3. 快速检查：功能正确 + 无明显安全问题 + 无语法错误
4. 直接判定 PASS 或 CONCERNS（不需要完整四维评估）

Path B+ (完整)

1. 对照 R0 阶段的验收标准逐项检查
2. 运行项目现有测试：
   • 先搜索项目中的测试配置（package.json scripts / pytest.ini / Makefile 等）
   • 执行测试命令，报告结果
   • 如果没有现有测试，声明"项目无现有测试"并跳过
3. 自我代码审查：
   • 边界条件处理
   • 错误处理完整性
   • 类型安全（如适用）
   • 性能隐患（N+1 查询、不必要的循环等）

Path D (增强)

以上全部 + 建议用户使用 /best-of-n 对关键逻辑做多模型交叉验证

产出：测试结果 + 审查意见 + 发现的问题清单

Quality Gate 四维评估

T 阶段完成后，执行 Quality Gate：

评估维度

1. 功能完整性

• 所有 MUST 需求是否已实现？
• 验收标准是否全部满足？
• 是否有遗漏的边界情况？

2. 代码质量

• 无 TODO / FIXME / HACK 遗留？（CI/CD 的 TODO 除外）
• 无硬编码魔法数字或字符串？
• 错误处理完整（不吞异常、有降级方案）？
• 函数职责单一？

3. 一致性

• 符合项目现有命名风格？
• 文件结构和组织方式与项目一致？
• 导入方式和模块引用方式一致？

4. 安全性

• 无明文密码 / 密钥 / token？
• 无 SQL 注入 / XSS 风险？
• 无不受限的用户输入直接使用？
• 无不安全的 eval / exec？

判定

判定	条件	动作
PASS	全部通过	→ V 归档阶段
CONCERNS	有小问题不影响核心功能	列出 → 修复 → 重新评估
REWORK	有结构性问题	→ 回到 E 阶段重做对应任务
FAIL	方案有根本缺陷	→ 回到 D 阶段重新设计

状态写入

如 .ai_state/ 存在，评估完成后写入 quality.json：

{
  "verdict": "PASS",
  "dimensions": {
    "completeness": "pass",
    "quality": "pass",
    "consistency": "pass",
    "security": "pass"
  },
  "issues": [],
  "timestamp": "2026-04-05T10:00:00Z"
}

如果判定非 PASS，issues 数组列出每个问题：

{
  "issues": [
    {"dimension": "quality", "severity": "minor", "desc": "utils.js 第 42 行有 TODO"},
    {"dimension": "security", "severity": "major", "desc": "API key 硬编码在 config.js"}
  ]
}