用途:作为 PR 审查的最终判断关口。此前各专家视角已标记候选问题,本技能不重新审查 PR、也不发明新问题,而是针对每个已标记问题,结合真实代码库调查后给出保留或丢弃的结论(is_valid)。
主要能力:判断问题是否值得上报。保留(is_valid = true)的标准是代码按实际执行路径会引发以下之一:正确性缺陷(错误结果、逻辑错误、边界/差一错误、真实输入会触发的边界情况、错误的数据转换或状态变更);安全问题(注入、认证/权限缺口、IDOR/租户隔离漏洞、密钥泄露、不安全反序列化、路径遍历、SSRF);数据丢失或损坏(破坏性或非幂等操作、写入丢失、迁移丢数据或损坏数据、破坏共享状态的竞态);契约破坏(不兼容的 API/schema/签名变更、调用方损坏、其他代码依赖的不变量被改);真实规模下的性能问题(N+1 查询、真实输入下的无界循环/内存、热路径缺索引、异步路径上的阻塞 I/O、意外的二次复杂度);资源/可靠性缺陷(连接或文件句柄泄漏、未释放锁、吞掉错误掩盖失败、缺少对必然发生的失败模式的处理)。合格的保留应能同时说出具体触发条件和具体后果。
丢弃(is_valid = false)的标准:过度设计(提取抽象、加接口、可配置化、为不在范围内的场景做未来兼容);推测性“如果”(依赖调用点、类型或已有校验下不可能发生的输入);防御性偏执(上游类型或不变量已排除的 None/错误防护、框架或父调用方已做的冗余检查);几乎不可能发生的边界情况;纯风格/口味(命名、格式、注释措辞、导入顺序,无行为差异;格式不属于 PostHog Review 关注点);已被处理(父调用方、默认值、框架保证、已有校验已阻止,需读周边代码确认);错误或不可复现(调查实际代码发现前提有误)。
使用方式:完整阅读被标记文件及其周边代码,不只凭片段判断;追踪问题是否真能到达,检查调用点、类型、校验与输入流向;权衡真实影响;拿不准时丢弃(精确优先于召回)。记录聚焦的论证,说明结论的具体理由,并将 category 设为问题所属类型。
适用场景:PostHog Review 流程中对候选 PR 问题的最终筛选。
限制:不重新审查 PR,不发明新问题;格式问题不在关注范围;资料未提供具体安装命令、收费或平台信息。