diff --git a/.gitea/ai-review/findings.json b/.gitea/ai-review/findings.json index fe51488..abcb0eb 100644 --- a/.gitea/ai-review/findings.json +++ b/.gitea/ai-review/findings.json @@ -1 +1,34 @@ -[] +[ + { + "level": "critical", + "role": "Mage", + "location": "app/usage.test.js:254", + "problem": "測試情境雖處理了 NaN,但未涵蓋 `limit` 為 0 或負數等極端數值(例如除以零導致的錯誤或異常百分比計算)。若 `formatUsageStatsLine` 內部有計算 `(used / limit) * 100` 或類似邏輯,傳入 0 會產生 `Infinity`,傳入負數則會產生荒謬的邏輯結果。", + "suggestion": "建議新增對 `limit: 0` 以及負數邊界值的測試案例,確保函式對這些非預期的數值輸入同樣能輸出「無法計算」或相應的錯誤處理訊息,而非輸出 `Infinity` 或負百分比。", + "is_new": true + }, + { + "level": "warning", + "role": "Leo", + "location": "app/usage.test.js:250", + "problem": "在測試案例中直接引用外部定義的 `usage` 變數,若該變數在其他測試中被意外修改,會導致測試間的隱性耦合,使得測試結果難以預測,降低測試的可維護性。", + "suggestion": "建議直接在測試函式內定義該案例所需的完整 `usage` 物件,或是使用工廠函式(Factory function)來生成所需的測試資料,確保測試案例的獨立性與確定性。", + "is_new": true + }, + { + "level": "warning", + "role": "Maya", + "location": "app/usage.test.js:250", + "problem": "目前的測試僅針對 NaN 的情況,但未考慮到 Infinity、null 或 undefined 等其他同樣可能導致格式錯誤或輸出異常的「無效數字」邊界條件。", + "suggestion": "建議補充針對 Infinity、null 與 undefined 的測試案例,確保這些值在 formatUsageStatsLine 中皆能正確處理並輸出友善的錯誤訊息,而非損壞的文字。", + "is_new": true + }, + { + "level": "info", + "role": "Maya", + "location": "app/usage.test.js:250", + "problem": "目前的測試案例將多個欄位同時設為 NaN,測試邏輯較為單一,未能明確驗證「單一欄位無效」與「組合欄位無效」時的具體行為差異。", + "suggestion": "建議將測試拆分為更細緻的案例,分別驗證 quota.limit 與 rate.limit 欄位在單獨無效,以及兩者同時無效的情況,以確保各欄位的防禦性邏輯皆有被完整覆蓋。", + "is_new": true + } +]