diff --git a/.gitea/ai-review/findings.json b/.gitea/ai-review/findings.json index 61089d6..7977839 100644 --- a/.gitea/ai-review/findings.json +++ b/.gitea/ai-review/findings.json @@ -4,6 +4,20 @@ "role": "Maya", "location": "app/json.test.js", "suggestion": "在 `readJSONText` 相關的測試中,除了測試檔案過大的情況,也建議增加一個測試案例,驗證當檔案大小剛好等於 `MAX_JSON_BYTES` 時,檔案能夠被成功讀取且不會拋出錯誤。這能確保邊界條件的處理是正確的。", + "is_new": false + }, + { + "level": "warning", + "role": "Aria", + "location": "app/gitea.js:14", + "suggestion": "陣列字面量過長,建議將每個元素放在獨立一行,以提高可讀性。例如:\n`return filterDiff(resp.data, [\n '.gitea/',\n '.amazonq/',\n '.claude/',\n '.codex/',\n '.gemini/',\n '.github/',\n 'CLAUDE.md',\n 'GEMINI.md',\n 'TODO.md',\n 'README.md'\n]);`", + "is_new": true + }, + { + "level": "warning", + "role": "Aria", + "location": "app/gitea.test.js:64", + "suggestion": "`describe` 區塊的回呼函數不應使用 `async` 關鍵字。`describe` 區塊應同步執行,而異步操作應在 `it` 或 `beforeEach` 等鉤子函數中處理。", "is_new": true }, { @@ -11,6 +25,6 @@ "role": "Maya", "location": "app/json.test.js", "suggestion": "在 `validateJSONArrayFile` 函數中,寫入修復後的 JSON 時,有判斷是否需要添加換行符 (`repaired.endsWith('\\n') ? repaired : `${repaired}\\n``)。目前的測試案例只驗證了最終結果包含換行符,但沒有明確測試兩種情況:當 AI 回傳的內容已經包含換行符時,以及不包含換行符時,都能正確處理。建議增加一個測試案例來覆蓋這兩種情況。", - "is_new": true + "is_new": false } ]