審查關卡:合併前的一份清單
審查關卡(review gate)就是一份在改動上線前必須為真的簡短清單。自己過一遍,或者讓 AI 逐項確認:
- 我讀了 diff,並理解改了什麼、為什麼改
- 改動被限定在我實際要求的範圍內
- 測試通過(且新行為有對應的測試)
- 型別檢查和 linter 都是乾淨的
- 沒有遺留金鑰、除錯日誌或被註釋掉的程式碼
- 沒有安全檢查被悄悄移除
- 我自己試過這個改動,至少跑過 happy path
如果有哪一項你勾不上,那你就還沒做完。
這份清單刻意保持簡短,因為一個你會跳過的關卡,根本就不算關卡。尤其是當你跑得很快、改動"看起來沒問題"時,會有一種直接放行的誘惑。對任何要面向使用者的東西,請抵抗它。一個有用的小技巧,是讓 AI 替你來填——把清單貼過去,然後說:"針對你剛做的改動,把這份審查關卡逐條走一遍。每一項,給出 PASS 或 FAIL,並附上一行證據。" AI 沒執行過測試就沒法誠實地寫下"測試通過",而強迫它逐行給出證據,會把一句含糊的"看著不錯"變成一個你可以審計的東西。