用途
specsfy-specialist-code-review 是一套代码审查技能，用于在合并或发布之前评估具体的 diff、分支或已提交的 PR。它也适用于对已经写好的改动给出“第二意见”，即使尚未开 PR。
它不用于从零设计新的结构性决策；如果需求是架构层面的新决策，应使用 $specsfy-specialist-software-architecture，因为审查只评估已经决定并写出的内容，不替代决策本身。
当 diff 涉及认证、授权、敏感数据或外部输入时，可与 $specsfy-specialist-application-security 组合；当发现的问题涉及命名、不变量或领域边界混乱时，可与 $specsfy-specialist-domain-modeling 组合。
主要能力
该技能围绕以下审查视角展开：契约、正确性、安全、架构、测试与运行风险。其工作流程包括：
- 固定对比基线与 diff 的确切范围，避免在没有清晰基线的情况下审查出与本次改动无关的问题。
- 在做出判断前读取适用的规格、issue、验收标准和仓库说明，避免把“正确”变成个人意见。
- 把每个被改文件映射到它影响的行为和边界（数据、权限、API 契约、配置、依赖）。
- 在风格之前评估正确性、边界情况、安全、并发和运行影响；风格只在自动化未覆盖时成为阻塞项。
- 从测试所提供的证据出发检查测试：去掉修复后测试是否会失败，还是只覆盖了代码行而没有证明行为。
- 报告每个可疑发现前，阅读真实代码及调用方/消费方以确认，减少误报。
- 按严重程度报告，附精确位置（文件:行）、触发失败的条件、影响和可能的修复。
使用方式
触发时机是存在具体的 diff、分支或 PR 需要评估时，或需要对已写好的改动给出第二意见时。技能内部按“固定基线—读规格—映射变更范围—按视角评估—检查测试证据—确认发现—按严重度报告”的顺序推进，并在最后总结本次审查覆盖范围与残余风险。
其报告规则要求：优先处理 bug 和具体风险，不把风格偏好变成阻塞项；每个发现描述输入条件、可观察后果和证据（行、测试、日志）；考虑与现有客户端的兼容性、并发、回滚计划和可观测性，而不只看顺利路径；区分 PR 中缺失的范围（合并阻塞项）与可选的未来改进（评论而非阻塞）；不重复 lint、formatter 或类型检查器已覆盖的发现；当某一视角无发现时应明确声明“此视角无发现”，而不暗示超出观察范围之外没有风险。
适用场景
适合在合并或发布前审查 diff、分支和 PR，或对已有改动做独立复核。涵盖的典型风险包括契约变更、正确性与边界情况、安全边界、架构耦合、测试有效性和运行层面影响。
限制
该技能明确不用于从零设计新的结构性决策。资料未提供该技能的收费、价格、平台支持或具体安装命令等信息，这些暂未提供。此外，其报告规范要求在缺少证据时不得宣称某个 diff“安全”或“正确”，并禁止使用绝对化措辞。