代码整齐，为什么还不能合并？

导出按钮按规范写完，结果只下载当前页。需求要的是全部筛选结果。 也可能结果正确，却绕过了仓库约定的权限入口。一个总分无法解释这两种失败。

先预测：类型检查全绿，能证明导出范围正确吗？它只检查自己覆盖的性质。 本页的小目标，是让另一位评审者看得出需求、规范和风险分别处于什么状态。

需求与规范分别评审 / Review requirements and standards as two independent dimensions.需求与规范分别评审 / Review requirements and standards as two independent dimensions.
需求与规范分别评审。

两条轴，各自给结论

固定本次比较的起点与终点，找到工单、规格和仓库规范。 需求轴逐项核对行为；规范轴逐条核对约定。没有规格就标记缺失，先补确认依据。

把明确违规和可能的坏味道分开。前者能指向规则，后者需要解释修改收益。 即使同一处代码同时涉及两轴，也保留两条结论，避免规范通过掩盖功能缺口。

评审轴依据位置问题证据
需求工单：导出全部结果导出入口只导出当前页42 条结果只得到 20 行
规范仓库权限规则下载处理器已走统一权限入口未登录请求被拒绝

此双轴结构根据 Matt Pocock 的 skills 仓库 改编，MIT。 来源文件：skills/engineering/code-review/SKILL.md。表内场景和文字由本站编写。

PR 要让别人能复核

变化概览说明谁在什么操作下得到新行为；前后证据使用同一输入，注明环境与结果。 合并风险写明回滚是否可行、影响哪些调用者，以及尚未验证什么。

PR 的变化、证据与风险 / A PR explains the change, before-and-after evidence and merge risks.PR 的变化、证据与风险 / A PR explains the change, before-and-after evidence and merge risks.
PR 的变化、证据与风险。

showing hard evidence that the changes work and assessing merge risk

— Matt Pocock

不要把「模型检查过」填在证据栏。贴出可定位的日志、截图或测试记录， 并说明它们证明哪项行为。没有运行的检查应写成待验证，不能算通过。

三块 PR 结构根据 skills 的 skills/engineering/pr/SKILL.md 改编，MIT。

该文件致谢 Dex Horthy / HumanLayer 的 show-me；这里仅保留致谢，不取其内容。

练习：补齐能判断的说明

动手练习

给导出修复补一条证据和一个回滚条件

  • 描述 A：修复导出，检查通过，可以合并。
  • 描述 B：改用完整筛选结果生成 CSV；同一筛选在修复前后分别得到 20、42 行。
  • 任务：选较可复核的一份，再补验证记录位置、失败路径和回滚条件。
参考思路

B 更可复核。注明测试数据与运行记录；验证无权限时拒绝下载。若只改应用代码且 CSV 接口兼容，可回退版本；若改了持久数据，须另查恢复路径。

完成标准

另一位读者能定位比较范围、两轴依据和行为证据，分别复述两条结论。 他也能指出影响范围、回滚条件与未验证项；缺少任何一项，就补说明或继续验证。

接下来用 交接页 把这些状态带进新会话。

速查

01

需求轴

按验收行为找证据，缺规格就标明缺失。

02

规范轴

明确违规与判断性建议分别列出。

03

PR 三块

变化概览、前后证据、合并风险。

来源与归属

skills MIT 许可声明

本页改编的结构来自 mattpocock/skills，MIT 许可，Copyright (c) 2026 Matt Pocock。完整许可附后，与原仓库 LICENSE一字不差。

完整许可：mattpocock/skills

MIT License

Copyright (c) 2026 Matt Pocock

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

非官方、非商业的学习整理，与 Matt Pocock 无隶属或背书关系。所引作品版权归原作者；MIT 复用内容另附版权与许可声明。