AI Review Bot
31 lines
1.6 KiB
JSON
31 lines
1.6 KiB
JSON
[
|
|
{
|
|
"level": "warning",
|
|
"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
|
|
},
|
|
{
|
|
"level": "info",
|
|
"role": "Maya",
|
|
"location": "app/json.test.js",
|
|
"suggestion": "在 `validateJSONArrayFile` 函數中，寫入修復後的 JSON 時，有判斷是否需要添加換行符 (`repaired.endsWith('\\n') ? repaired : `${repaired}\\n``)。目前的測試案例只驗證了最終結果包含換行符，但沒有明確測試兩種情況：當 AI 回傳的內容已經包含換行符時，以及不包含換行符時，都能正確處理。建議增加一個測試案例來覆蓋這兩種情況。",
|
|
"is_new": false
|
|
}
|
|
]
|