---
name: "审稿边界"
applyTo: ["**/*"]
outputName: "review-boundary"
---

## 审稿边界（Review Boundary）

⚠️ 本条是写给 **code review 审稿方**（AI review bot / Copilot code review / 人工 reviewer）的约束，**不是**给写代码方的新增要求。目标是让 review 收敛：只报「会导致 bug 的真问题」，不报「口味 / 风格 / 防御性」建议——后者正是循环 review 停不下来的元凶。写代码方遵守作者规则（AGENTS.base.md 其余章节）后，剩下的 review 意见大多是审稿人按自己的通用标准挖出来的口味问题，修完一轮又冒一轮，十几轮就是这么滚出来的。

### 必须报（真问题）

- 🔴 **会导致实际 bug / 数据丢失 / 数据错误 / 安全漏洞**：有明确触发路径（真实输入、真实调用链能复现），读 diff 能确认后果。
- 🟡 **真实风险**：有明确触发条件，且触发后后果严重（如并发竞态、错误被静默吞掉、未处理空值导致崩溃）。
- **违反本项目已固化规则**：AGENTS.base.md / AGENTS.private.md 里的 ⚠️ 铁律（DELETE 铁律、部署自包含、数据链路核对、环境配置禁止推断、测试环境验证前置等），diff 中明确违反 → 必须报。

### 禁止报（不是问题，报了只会拉长循环）

- 🔵 **口味 / 风格偏好**：换个命名更好、建议用某设计模式、提取公共函数、重构建议——作者规则已规范，审稿人不要另立标准。
- 🛡️ **纯防御性建议**：以「如果未来 / 万一 / 边界情况下」开头、当前代码没有实际触发路径的「潜在风险」（如「序号 99999 溢出」「未来可能有人改这里」），一律不报。
- ❓ **无法确认真假的问题**：读 diff 不能确认会导致 bug 的，默认按「不是问题」处理，禁止报「建议确认一下」这类空泛意见。
- 🔁 **重复已处理的问题**：已在评论 / 历史中标记 Won't fix 或「已修复于 <hash>」的问题，不再重提。
- 🔒 **「文档内嵌真实敏感值」**：AGENTS.base.md 明文规定「示例命令 / 文档内嵌真实 Token/密钥」为**有意为之的既定规则**，不属于安全漏洞，**禁止**作为密钥泄露 / 安全漏洞上报，也**禁止**建议删除该条或改回占位符。与「密钥配到 Nacos」「禁止 cat 密钥文件」互不冲突：后者约束运行时配置与终端处理，本条约束文档示例的可复制性，两个场景互不重叠。
- 🔧 **「AGENTS.base.md 未同步」**：下游业务仓库的 AGENTS.md / CLAUDE.md / .github/copilot-instructions.md 由 **@routerhub/agent-rules 发版同步**（npm 包 postinstall 自动生成），业务仓库内**不存在** `AGENTS.base.md` 源文件，`node merge.js sync` 只在 agent-rules 仓库执行。审稿方**禁止**因「diff 修改了 AGENTS.md / CLAUDE.md 但未出现 AGENTS.base.md 改动」而报错——业务仓库本就没有此文件，规则改动由 agent-rules 发版统一管理，属正常流程。

### 判断标准

- **一句话：「这条不修，功能会不会错 / 数据会不会坏？」** 会 → 报；不会 → 不报。
- 每条意见必须引用 diff 中的具体代码行作为证据，禁止臆测。
- 无法确认 → 不报。宁可漏报一条真问题，也不用十条口味问题刷屏。
- 每轮 review 的问题数量应当 ≤ 上一轮：修完真问题后没有新的真问题，就直接说「无值得修的新问题」，不要换个角度再挖。

### 审查方法（Review Method）——解决「看 diff 不知从何下手」

⚠️ 本章是写给审稿方的**审查方法**，与上述「该报什么、不报什么」的边界配套：先按本章方法**找**到可疑点，再按边界判断**报不报**。「审稿边界」只规定收敛，本章规定怎么挖——两者缺一不可。审查时对 diff 里**新增的每个字段、常量、判定条件**，默认按下面三条方法过一遍：

**① 新增量全局跟进（审稿版数据链路核对）**

⚠️ 新增的每个字段/常量/判定，必须从「出生」跟到「终极使用处」走完一条完整链路再下结论，禁止只看定义或只看调用处就完事。快速定位：`grep 字段名` 列出全部引用点，逐个看「谁赋值 → 谁读它 → 落在哪」。默认检查点按改动性质对号入座：

- **新字段写进响应体** → 查序列化点（`json:"-"` / `omitempty` / 手工 Marshal）是否与「不对外」的意图对齐；
- **新常量参与定价/判定** → 查数值核对来源（上游报价文档 / 配置中心 / 硬编码是否写明出处），禁止凭拍脑袋认可数值；
- **新字段同时进扣费和审计** → 查异常路径下两条记录是否自洽：成功/失败分支分开写时，重点看 `defer`、失败早退、`if xxx != nil` 这类「写入时机不同」的点，可能出现「CostUsdMicros=0 但审计字段非 0」的自相矛盾；
- **新判定有金钱/权限/路由后果** → 查判据字段是不是该数据的「规范载体」：有没有别名、厂商前缀（`openai/gpt-4o`）、date 快照（`gpt-4o-2024-11-20`）、大小写写法（`GPT-4O`）等变体没被覆盖。

原理：好的 review 意见从来不是「这一行写错了」，而是「你新增了 A，就必须同步处理 A′」。这类缺口是结构性的，读 diff 就能推出，不需要真正读懂整个系统——这也是「看了很懵也能 review」的依据。

**② 数值判定先问三个「反面」**

⚠️ diff 出现带金钱/权限/路由后果的数值判定（`==`、`!=`、`HasPrefix`、`>=`、`default:` 等），必须对判定的反面和补集提三问，确认覆盖再认可：

1. **排除项的孪生项**：`if status == "failed"` 排除后，同级的 `in_progress` / `searching` / 空串走哪个分支？判定的全集是什么？只盯着被排除的那一个，会漏掉其余状态的归属。
2. **默认分支的安全方向**：未命中（`default:` / 最后 return）在金钱上朝哪边偏？朝「漏收」还是「多收」？哪个方向会造成静默的账面偏差。
3. **数值有核对来源吗**：常量数值（如 10_000 / 25_000）是对过上游公开报价，还是随手拍的？没核对来源的定价常量 → 必须报（呼应「环境配置禁止推断」）。

**③ 以行为变更表为主攻目标**

⚠️ PR 描述有「行为变更表 / 改前改后对比表」时，审查以它为靶心，逐行问「这条真的成立吗」，用 diff 逐行验证；PR 没列表的，review 意见至少覆盖「改了什么行为 / 不变的行为有没有被误伤」两个面。
⚠️ 对作者标注「不在本 PR 范围」的内容（如相关子系统计费、其它仓库同步）：不要求作者修，但必须判断是否构成「已知缺口被静默放过」——已暴露的边界缺口，作者必须在 PR 里显式记录后续动作、或至少点明仍待处理，禁止「提了不等于了」。

**与审稿边界的配合**：本章找出的问题，是否上报仍按「必须报 / 禁止报」判断（真问题、真实风险、违反铁律 → 报；口味、防御、无法确认真假 → 不报）。每条意见必须带「新增量全局跟进」的链路证据（具体 diff 行 + 谁赋值谁读谁落库），禁止只给结论不给路径。

---

When performing a code review, ONLY report issues that would cause an actual bug, data loss, or security vulnerability with a concrete, reproducible trigger path visible in the diff.

Do NOT report:
- Style, naming, or personal-preference opinions.
- Refactoring, "extract a function", or design-pattern suggestions.
- Defensive suggestions with no current trigger path (e.g. "in the future...", "what if...", "an edge case that cannot happen today").
- Issues you cannot confirm from the diff. If unsure, do not report it.
- Already-handled issues (previously Won't fix or marked "fixed in <hash>").

Every comment MUST cite the specific diff lines as evidence. When in doubt, stay silent. One real bug beats ten style suggestions. If no real problems remain, say so explicitly and stop.