審查
使用此技能評估已完成的工作。審查從不撰寫它正在評斷的變更。
這是標準的審查流程。merge-readiness 會路由至此處，而 ai-slop-cleaner 則是其中的選用通道。
目標
找出真正有問題的地方，按嚴重性排序，並提供足夠的細節以便採取行動。
流程
- 確定變更內容及其原本的意圖。
- 根據該意圖閱讀變更。
- 先檢查正確性，再檢查風險，最後檢查簡化程度。
- 回報前驗證每個候選發現。
- 以最嚴重者優先回報發現。
檢查內容
- 正確性 — 邏輯缺陷、邊界情況、錯誤路徑、並發問題
- 風險 — 安全邊界、破壞性操作、資料完整性
- 重複使用 — 應使用但未使用的現有工具或模式
- 簡化 — 可以刪除或合併的程式碼
- 涵蓋範圍 — 未經測試即上線的行為
規則
- 分離的通道：審查者不得與作者處於同一個活躍上下文。
- 回報前請先驗證。一個聽起來合理但無法重現的發現只是噪音。
- 誠實說明嚴重性；不要為了看起來詳盡而虛增列表。
- 「無發現」在工作無問題時是有效的結果。
- 預設為建議性質 — 審查提供資訊，但不作為關卡。嚴格的關卡（發佈、安全性、破壞性操作）應獨立處理，且預設為失敗。
輸出
- 發現項目，依嚴重性排序（最嚴重在前），每個項目包含檔案、行號與具體的失敗情境
- 已檢查且無發現的項目
- 無法評估的項目