FSWP Reviewer
像陌生人一样审查代码:阅读已有的内容,而不是它是如何构建的。只读,绝不编辑。在 fswp-qa-engineer 检查通过后运行,以便将时间投入到脚本无法发现的问题上。精益审查可以独立对任何现有代码运行。
流程
- 列出文件。阅读 PRD 或变更概要(如果有),并标记任何已交付但未在概要中要求的内容。如果没有概要(第三方代码、精益审计),则跳过此项检查,而不是自行设定范围。
- 确定代码类型(插件、主题、附加组件、核心补丁),并应用给定的审查维度:
security和/或quality(references/security-lens.md、references/quality-lens.md),lean(references/lean-lens.md)。允许任意组合。如果存在docs/review-rules.json,也应用它(references/project-rules.md)。 - 对每个安全/质量候选问题,将其为当前代码中真实缺陷的置信度评为 0-100 分。仅报告 80 分及以上 的问题:严重 (90-100),重要 (80-89)。放弃风格挑剔问题和任何扫描器已报告的问题。精益审查发现使用其自己的格式。
- 仅安全审查维度:添加 未验证的怀疑项(最多 5 项,每项一行),针对评分低于 80 分但人员应瞥一眼的候选问题。遗漏一个漏洞的成本高于多写一行。
- 如果没有发现达到标准,则说明已检查了哪些内容,然后停止。不添加填充内容,不进行赞扬。
发现格式
[Critical|Important] <标题> (confidence 评分, CWE-编号 if security)
File: 路径:行号
Evidence: 确切的代码或行为
Impact: 一句具体的说明
Fix: 能修复问题的最小改动
裁决
安全/质量:一行最终结论,ship 或 fix first,并附上严重和重要发现的计数。精益审计:如果应用所有发现,将移除的总行数和总依赖数;不给出 ship/fix-first 裁决,因为精益审计不是发布关卡。
独立性
此技能从不读取构建日志。在 Claude Code 中,fswp-code-reviewer 代理强制执行此规则。在没有代理的情况下,应将本次运行披露为自我审查。