review

审查

一项已完成的变更，只有通过三个方面的检查，才能算作“完成”：结构、差异和文档。每个方面都有专门的技能负责。本技能只负责它们无法涵盖的内容——运行顺序、它们之间的迭代，以及最终结论。

工作流

1. 范围。 修正待审查的差异，并确认它确实有实质内容；对于琐碎的修改，跳过整个检查流程。
2. 结构 — refactor-clean。 首先运行此项，趁重构成本还低时进行，并在审查即将变动的差异之前完成。
3. 差异 — code-review。 审查已稳定的结构。如果发现的问题会导致结构变更，则返回第 2 步，而不是继续往下。
4. 文档 — write-docs。 更新此变更涉及的文档。然后从根 README 开始，沿链接链逐一追踪到每篇文档：路径上的每个链接都仍然有效，且从中心页面到叶子页面的阅读顺序依然清晰。
5. 报告。 汇总三项检查的结论——各自发现了什么、你改了什么、留下了什么以及原因。只有所有检查都干净无误或问题均已解决，才算完成。

规则

• 开始某项检查时，先阅读对应技能——规则在技能中，不在这里。
• 顺序决定呈现方式，而非披露时机：一旦发现问题就立即指出，即使该问题归后续检查负责。
• 迭代，不要层层推进：后续修复若重新影响较早的检查，就返回该检查。完整检查一轮且没有新增问题后，即可停止。