Code Review:把 PR 審查做成系統化流程的 Skill

前言

代碼審查的質量很不穩定:同一份 diff,有人只看命名和格式,有人能指出 SQL 注入和競態條件。差距通常不在經驗,而在有沒有一套覆蓋全面的檢查清單和固定的審查流程。這個 Skill 把安全、性能、正確性、可維護性、測試等維度的審查要點整理成可直接使用的 prompt 模式,讓審查者按維度系統過一遍,而不是隨手掃。

這是什麼

Code Review 是一個基於通用 SKILL.md 格式的代碼審查 Skill,由 wpank 發佈在 ClawHub,座標 @wpank/code-review,版本 1.0.0,來源 clawhub,分類 dev-programming。SkillHub 頁面顯示評分 100000、下載 49793、安裝 1972,標籤 requires_api_key=false,即不需要 API key 即可使用。

它解決的問題是:審查 PR 時缺乏統一標準、遺漏安全項、反饋零散。Skill 內置嚴重等級劃分、結構化反饋指南、三遍審查流程和需避免的反模式,適用於審查 PR、建立審查標準或提升審查質量。

核心功能

審查維度與優先級

Skill 把審查拆成七個維度,每個維度有明確優先級:

  1. Security:漏洞、認證、數據暴露,Critical
  2. Performance:速度、內存、擴展瓶頸,High
  3. Correctness:邏輯錯誤、邊界情況、數據完整性,High
  4. Maintainability:可讀性、結構、可維護性,Medium
  5. Testing:覆蓋率、測試質量、可靠性,Medium
  6. Accessibility:WCAG 合規、鍵盤導航、屏幕閱讀器,Medium
  7. Documentation:註釋、API 文檔、changelog,Low

分維度檢查清單

每個維度下是一組可直接逐條覈對的具體檢查項,例如 Security 維度包含 SQL 注入、XSS、CSRF、認證、授權、輸入校驗、密鑰管理、依賴安全、敏感數據、限流、文件上傳安全、HTTP 安全頭共 12 項。Performance 維度覆蓋 N+1 查詢、不必要的重渲染、內存泄漏、bundle 體積、懶加載、緩存策略、數據庫索引、分頁、異步任務、圖片優化。Correctness、Maintainability、Testing 維度同樣各有 10 條左右的清單。

嚴重等級與結構化反饋

Skill 要求按嚴重等級標註問題,並提供結構化的反饋格式,避免“這個不太行”式的模糊意見。具體等級分級和反饋模板來自 Skill 正文,使用前建議通讀一遍。

三遍審查流程

Skill 明確反對一遍讀完試圖抓全所有問題,改爲三遍:

  1. 第一遍(2-5 分鐘):讀 PR 描述和關聯 issue,掃文件列表確認改動範圍合理,檢查整體方案是否對症,確認沒有架構漂移。
  2. 第二遍(主要時間):逐文件逐行讀 diff,對照上述清單檢查每個函數改動,覈對每個 I/O 邊界的錯誤處理。
  3. 第三遍(約 5 分鐘):專注邊界值、失敗模式、併發和缺失測試。

反模式清單

Skill 內置一份審查中要避免的反模式,比如把風格問題和阻塞問題混在一起提、對每個細節都長篇大論導致重點被淹沒。這部分用於校準反饋的表達方式。

安裝與啓用

官方頁面給出兩種安裝方式。OpenClaw / Moltbot / Clawbot 環境:

npx clawhub@latest install code-review

或使用 skills CLI:

$ openclaw skills install @wpank/code-review

安裝後 Skill 通過 SKILL.md 描述自身用途,由 Agent 在審查 PR、建立審查標準等場景下自動加載。不需要配置 API key。

典型用法

安裝後直接以審查任務的形式使用,例如:

  1. 讓 Agent 審查當前分支相對主幹的 diff:說明“審查這次改動”,Agent 會加載該 Skill 的維度清單和流程。
  2. 建立審查標準:把 Skill 作爲團隊的審查基線,要求所有 PR 按七個維度逐項過。
  3. 專項複查:針對安全敏感的改動,讓 Agent 只跑 Security 清單並按嚴重等級輸出結構化反饋。

Skill 正文是純 prompt 文本,也可以直接複製到對話裏手動使用,不依賴安裝環境。

適用場景與注意

適合需要系統性審查 PR、想統一團隊審查口徑、或希望補齊安全和性能檢查項的開發者和團隊。它的清單偏通用,條目是原則性的,落地到具體語言和框架時需要結合項目實際裁剪,比如 .NET 的依賴注入問題不在內置清單裏,需要自己補充。

注意兩點:一是 Skill 是審查指南,不能替代自動化工具,靜態掃描、測試和 CI 仍要保留;二是清單覆蓋面廣,逐條全跑會拖慢審查節奏,建議按改動類型選維度,而不是每次都七維度全開。

結尾

這個 Skill 的價值在於把“系統化審查”從個人習慣變成可複製的清單和流程:維度、嚴重等級、三遍流程、反模式都寫好了,裝完即可用。官方地址:https://www.skillhub.cn/skills/clawhub_wpank/code-review ,上游倉庫:https://clawhub.ai/wpank/code-review 。

羽毛球分组比赛记分
小程序二维码

欢迎使用《羽毛球分组比赛记分》微信小程序

小夜