skills/dev-code-reviewer/SKILL.md
自動化程式碼審查完整流程。協調 4 個審查領域:風格、安全性、效能、架構,最終產出綜合審查報告。適用於「review this code」、「看一下這段程式碼」、「程式碼審查」、「PR review」、「程式碼品質分析」、「安全性審查」、「效能審查」、「架構審查」、「程式碼風格檢查」等任務。支援指定單一領域審查。注意:CI/CD 整合、自動修正、Git commit/merge 操作不在此技能範圍內。
npx skillsauth add vincent119/ai-rules-kit dev-code-reviewerInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
系統性地從風格、安全性、效能、架構 4 個領域審查程式碼。
以下輸入會啟動此技能:
啟動後,先確認以下資訊:
請提供以下審查資訊:
1. 目標程式碼(必填):檔案路徑、PR 號碼、diff 或直接貼上程式碼
2. 語言/框架(選填):可自動偵測
3. 審查範圍(選填):全域 / 僅安全性 / 僅效能 / 僅架構 / 僅風格
4. 背景資訊(選填):PR 描述、相關 Issue、變更原因
5. 風格指南(選填):團隊特定規範
從使用者輸入提取:
建立 _workspace/ 目錄
整理輸入存為 _workspace/00_input.md
識別目標程式碼,確定審查範圍
根據請求範圍決定執行模式
| 順序 | 任務 | 負責領域 | 相依 | 產出 |
|------|------|---------|------|------|
| 1a | 風格審查 | style-inspector | 無 | _workspace/01_style_review.md |
| 1b | 安全性審查 | security-analyst | 無 | _workspace/02_security_review.md |
| 1c | 效能審查 | performance-analyst | 無 | _workspace/03_performance_review.md |
| 1d | 架構審查 | architecture-reviewer | 無 | _workspace/04_architecture_review.md |
| 2 | 綜合報告 | review-synthesizer | 1a-1d | _workspace/05_review_summary.md |
任務 1a-1d 全部並行執行。
領域間資訊傳遞:
_workspace/ 中的所有審查報告| 使用者請求模式 | 執行模式 | 部署領域 | |--------------|---------|---------| | 「review 這段程式碼」、「全面審查」 | Full Review | 全部 4 個領域 | | 「只做安全性審查」 | Security Mode | security-analyst + synthesizer | | 「分析效能」 | Performance Mode | performance-analyst + synthesizer | | 「架構審查」 | Architecture Mode | architecture-reviewer + synthesizer | | 「只檢查程式碼風格」 | Style Mode | style-inspector + synthesizer |
PR Review:提供 PR 號碼時,提取 diff 並聚焦於變更的程式碼,參考完整檔案上下文但集中審查 diff。
參考 dev-vulnerability-patterns 技能:
參考 dev-refactoring-catalog 技能:
每個領域的報告使用統一格式:
# [領域] 審查報告
## 摘要
- 整體評估:PASS / WARN / FAIL
- 發現問題數:Critical X, High X, Medium X, Low X
## 問題清單
### [嚴重度] 問題標題
- 位置:`file.py:42`
- 說明:問題描述
- 建議:修正方式
- 參考:CWE-89 / SOLID-SRP 等
# 程式碼審查報告
## 最終裁決
Approve / Request Changes / Reject
## 執行摘要
[2-3 句總結]
## 優先修正項目(Critical & High)
[跨領域排序的必修項目]
## 各領域摘要
| 領域 | 評估 | 問題數 |
|------|------|--------|
## 完整問題清單
[依優先級排序]
## 優點(What Went Well)
[值得保留的良好實踐]
| 錯誤類型 | 策略 | |---------|------| | 無法識別語言 | 從副檔名 + 程式碼模式自動偵測 | | 大型程式碼庫 | 聚焦於變更或核心檔案,在報告中說明範圍 | | 跨領域衝突 | review-synthesizer 進行取捨分析並裁決 | | 背景資訊不足 | 僅基於程式碼審查,在報告中說明限制 |
| 技能 | 目標領域 | 功能 |
|------|---------|------|
| dev-vulnerability-patterns | security-analyst | CWE 分類、語言特定漏洞模式、安全替代方案 |
| dev-refactoring-catalog | architecture-reviewer, performance-analyst | 程式碼異味到重構的對應、SOLID 違反識別、複雜度指標 |
tools
基於 SLA/SLO 量化評估事故影響的計算模型與業務影響矩陣。適用於「SLA 影響」、「SLO 違反」、「影響評估」、「營收損失估算」、「Error Budget」、「可用性計算」、「事故成本評估」等量化事故業務影響的任務。強化 impact-assessor 的評估能力。注意:事故原因分析與改善規劃不在此技能範圍內。
research
根因分析(RCA)方法論詳細指南。提供 5 Whys、Fishbone 圖、Fault Tree Analysis、變更分析等結構化 RCA 技術,以及認知偏誤防範清單。適用於「根因分析」、「RCA」、「5 Whys」、「魚骨圖」、「Fault Tree」、「原因分析方法論」、「變更分析」等事故原因分析任務。強化 root-cause-investigator 的分析能力。注意:時間軸重建與改善規劃不在此技能範圍內。
testing
事故事後分析(Postmortem)完整流程。協調 7 個執行階段:資訊收集 → 時間軸重建 → 根因分析 → 影響評估 → 改善規劃 → 報告審查 → 整合報告,最終產出完整的 Postmortem 報告。適用於「寫事故報告」、「post-incident 分析」、「RCA 報告」、「事故時間軸整理」、「建立改善措施」等請求。注意:即時 Incident Response(on-call)、監控系統設定、告警配置不在此技能範圍內。
content-media
投影片版面模式庫。提供 20 種投影片類型的最佳版面配置、格線系統、色彩與字型設計 Token。適用於「投影片版面」、「Slide Layout」、「設計系統」、「格線」、「字型」、「色彩規範」等投影片視覺設計任務。強化 visual-designer 的設計能力。注意:PPT/Keynote 檔案直接輸出不在此技能範圍內。