code-review

Code Review Skill

當進行程式碼審查時，請遵循以下步驟：

審查清單 (Review checklist)

1. 正確性 (Correctness)：程式碼是否符合預期功能？
2. 邊界情況 (Edge cases)：是否處理了錯誤條件？
3. 風格 (Style)：是否符合專案規範？
4. 效能 (Performance)：是否存在明顯的效率低落？

Chrome Extension 專屬檢查

由於這是一個 Chrome 擴充功能專案，額外檢查以下項目：

1. Manifest V3 相容性：確認使用的 API 符合 Manifest V3 規範
2. Chrome API 使用：確認正確使用 chrome.tabs、chrome.bookmarks 等 API
3. 權限最小化：確認只請求必要的權限

如何提供回饋

• 不需要給予太多浮誇的稱讚來滿足 PR 提交者的行緒價值
• 具體指出需要修改的地方
• 解釋「為什麼」，而不僅僅是「什麼」
• 儘可能提供替代方案

自動化工具

在開始人工審查之前，先執行 lint 檢查腳本以快速發現常見問題：

.agent/skills/code-review/scripts/lint-check.sh .

何時使用

• PR 審查開始時：在閱讀程式碼之前先跑一次，快速掌握潛在問題
• 提交前自檢：開發者可在提交 PR 前自行檢查
• CI 整合：可加入 CI pipeline 作為自動化檢查

腳本檢查項目

參考資料

詳細的審查標準請參考：

• 安全檢查清單 - 權限、CSP、API 安全
• 效能模式 - DOM 操作、事件處理最佳實踐
• 程式碼風格指南 - 命名慣例、模組結構