Code Review Excellence Skill
本地快取版本，來源：wshobson/agents
核心原則
審查心態
- 目標：抓 bugs、確保可維護性、知識分享、執行標準
- 非目標：炫技、無意義阻擋
有效回饋
- 具體且可執行
- 教育性，非批判性
- 聚焦代碼，非個人
- 平衡（也要讚美好的部分）
- 優先級標示
審查流程
- Context Gathering (2-3 min): 理解 PR 需求與範圍
- High-Level Review (5-10 min): 架構、檔案組織、測試策略
- Line-by-Line Review (10-20 min): 邏輯、安全、性能、可維護性
- Summary & Decision (2-3 min): 明確結論與建設性摘要
優先級標籤
| 標籤 | 意義 |
|---|---|
| 🔴 [blocking] | 必須修復才能 merge |
| 🟡 [important] | 應該修復，可討論 |
| 🟢 [nit] | 可有可無，不阻擋 |
| 💡 [suggestion] | 替代方案建議 |
| 📚 [learning] | 教育性說明，無需動作 |
| 🎉 [praise] | 做得好！ |
安全審查重點
- Input validation?
- SQL injection risks?
- XSS vulnerabilities?
- Sensitive data exposure?
- Authentication/Authorization checks?
常見陷阱
- 完美主義（過度挑剔）
- 範圍蔓延（超出 PR 範圍）
- 不一致標準
- 延遲審查
- 橡皮圖章（不認真看）