reviewing-for-substancelisted

# Reviewing for Substance ## Core Rule Progress is measured on the deliverable, never on the documents about the deliverable. Two tests, applied relentlessly: - **For any review finding**: 这条不改，什么会坏、什么会返工？ No answer → not a finding. - **For any round of work**: 这一轮结束，交付物本身前进了什么？ No answer → not progress, whatever got edited. AI has two ways of looking busy without being useful: manufacturing review findings to seem thorough, and polishing meta-artifacts (formats, structures, phrasings, documents about documents) to seem productive. Both produce motion without movement. This skill bans both. ## Finding Triage Every review finding lands in exactly one class: | Class | Definition | Handling | |---|---|---| | **Blocking** | Wrong behavior, broken contract, missed requirement, data loss — rework guaranteed if shipped | Must close before proceeding | | **Material** | Works, but creates real cost later: unhandled state, a field an implementer must guess at, silent assumption | Close, or consciously defer with a note | | **Polish** | Style, phrasing, structure taste | Author's discretion; never blocking; may be omitted entirely | ## The Nitpick Ban Rejected as findings, regardless of how they're phrased: synonym preferences, formatting taste, 「我会换个说法」, restating content in the reviewer's own voice, heading-level opinions, 总分总 structure advice. The reviewer unsure whether something qualifies runs the test: *what breaks, or what gets rebuilt, if this ships exactly as-is?* Thr