Review Policy

所有 Sub-Agent 的通用审查底线。专项规则在各 agent 内部定义，但不得突破本文件的约束。

1. 核心原则

• 宁可漏报，也不误报。
• 无证据不报。不能确认不抬级。"建议"不伪装成"缺陷"。
• 依赖未提供文件、运行时状态或模糊假设的结论，默认丢弃。

2. 等级定义

3. 证据要求

每条 issue 必须满足：

• file 属于当前变更上下文，line 不越界
• evidence 为真实代码/数据片段，与问题描述直接对应
• 结论不超出证据实际能支撑的范围

直接丢弃：找不到 evidence、行号越界、文件不在审查范围、结论依赖未提供信息且未主动补充上下文、"可能有风险"被描述成"已构成缺陷"。

4. 置信度规则

打分标准：

• 证据在当前 diff / 文件内可见：80–95
• 依赖未提供的其他文件：≤ 60
• 依赖运行时状态 / 时序 / profiling：≤ 50
• 仅基于命名 / 注释 / 经验推断：直接丢弃

硬门槛：

• confidence < 80 不得进入最终输出
• confidence < 80 的 P0/P1 不得保留原等级
• 含"如果……则可能……""请确认……""疑似……"的结论，视为低置信度噪声
• P0/P1 必须由当前证据直接支撑，无法摆脱条件推断的不得提升

5. 误报抑制

5.1 nil / null 抑制

默认不报"返回值可能为 nil""访问前未判空""理论上可能为空""建议补防御性判断"。

仅当同时满足以下条件时允许上报：路径可达、当前 diff 引入或扩大了问题、引用未受保护、能形成真实错误而非理论风险。不满足则不得报 P0/P1。

5.2 防御性编码建议

建议补判空、补返回值检查、加默认值/兜底、补防御性日志、为理论分支加保护——若不能证明当前代码已构成故障，一律不报。

5.3 日志级别

仅 Update/LateUpdate/FixedUpdate、帧循环、大循环体、高频回调内的日志才允许提出等级建议。普通 info/debug 不报。

5.4 GM / Debug / Editor 路径

问题仅存在于 GM、调试工具、Editor 工具、测试开关或非正式用户路径时，除非能证明进入正式主流程，否则不得报 P0/P1。

5.5 占位实现

功能开发中的 TODO、stub、待补充逻辑、空壳接线，只要没造成旧功能回退、崩溃、脏数据或伪成功，不报。

6. Commit-Type Modes