41 lines
1.1 KiB
Markdown
41 lines
1.1 KiB
Markdown
# 程式碼審查 (Code Review)
|
||
|
||
對未提交的變更進行全面的安全與品質審查：
|
||
|
||
1. 獲取已變更的檔案：`git diff --name-only HEAD`
|
||
|
||
2. 針對每個變更的檔案，檢查以下項目：
|
||
|
||
**安全問題 (極高優先級 - CRITICAL):**
|
||
- 硬編碼的憑據、API 金鑰、Token
|
||
- SQL 注入漏洞
|
||
- XSS 漏洞
|
||
- 缺少輸入驗證
|
||
- 不安全的依賴項
|
||
- 路徑遍歷 (Path traversal) 風險
|
||
|
||
**程式碼品質 (高優先級 - HIGH):**
|
||
- 函式超過 50 行
|
||
- 檔案超過 800 行
|
||
- 嵌套深度超過 4 層
|
||
- 缺少錯誤處理
|
||
- `console.log` 語句
|
||
- TODO/FIXME 註釋
|
||
- 公共 API 缺少 JSDoc
|
||
|
||
**最佳實踐 (中優先級 - MEDIUM):**
|
||
- 可變模式 (Mutation patterns，應優先使用不可變資料結構)
|
||
- 程式碼/註釋中的 Emoji 使用
|
||
- 新程式碼缺少測試
|
||
- 無障礙設計 (a11y) 問題
|
||
|
||
3. 產生報告，包含：
|
||
- 嚴重程度：CRITICAL, HIGH, MEDIUM, LOW
|
||
- 檔案位置與行號
|
||
- 問題描述
|
||
- 建議的修復方案
|
||
|
||
4. 如果發現 CRITICAL 或 HIGH 層級的問題，則禁止提交 (Block commit)
|
||
|
||
絕對不要核准帶有安全漏洞的程式碼！
|