SuperAPI AI 工具集

review-hog-validation-criteria

安全与审计

查看项目

更新于 10-16 03:45

技能介绍

用途:这是 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,也不提出新问题;格式问题不在其关注范围;具体实现细节、安装方式与配置暂未提供。