code-review

SKILL.md

| 基礎檢查失敗 | | | | | typecheck 失敗（每個錯誤） | -20 | | | lint 錯誤（每個錯誤） | -10 | | | lint 警告（每個警告） | -5 | | | 測試失敗（每個失敗） | -30 | | | 使用 any（每處） | -15 | | | 使用 @ts-ignore（每處） | -20 | | | 使用 as unknown as（每處） | -15 | | 致命缺陷 | | | | | 安全漏洞（SQL注入、XSS） | -50 | | | 硬編碼密鑰/Token | -50 | | | 資料遺失風險（事務不完整） | -40 | | | 競態條件 | -40 | | | 函數名稱與實作不符 | -30 | | 嚴重缺陷 | | |

| | 未處理 Supabase/API error | -20 | | | 邊界值未處理（null/undefined） | -15 | | | 空陣列未處理 | -15 | | | 隱式假設未驗證 | -15 | | 中等缺陷 | | | | | 錯誤訊息顯示技術細節給用戶 | -10 | | | 缺少 console.error 日誌 | -10 | | | 魔術數字未抽成常數 | -5 | | | 函數超過 50 行 | -5 | | | 複雜邏輯無註解 | -5 | | 輕微缺陷 | | | | | 命名不清楚 | -3 | | | 可抽成 helper 但沒抽 | -3 | | | 未使用的 import/變數 | -2 |

| 90-100 | 🟢 優秀 | 可合併 | | 80-89 | 🟡 良好 | 修正後可合併 | | 60-79 | 🟠 需改進 | 必須修正 | | 0-59 | 🔴 不合格 | 重寫 | | <0 | ⛔ 災難 | 拒絕，需嚴肅檢討 |

Original anzeigen