diff --git a/.gitea/ai-review/findings.json b/.gitea/ai-review/findings.json index b08c396..8315689 100644 --- a/.gitea/ai-review/findings.json +++ b/.gitea/ai-review/findings.json @@ -30,29 +30,5 @@ "problem": "新增了 Codex CLI 的安裝與插件註冊過程,但沒有對應的測試來驗證安裝是否成功、插件是否正確載入。", "suggestion": "應在測試套件中加入 Docker 映像檔的整合測試,驗證容器啟動後 `codex --version` 及插件列表是否存在。", "is_new": false - }, - { - "level": "warning", - "role": "Leo", - "location": "Dockerfile:22", - "problem": "在 Dockerfile 中安裝 plugin 時,雖然使用了 `for` 迴圈處理變數,但直接將 plugin URL 硬編碼在 RUN 指令中。如果未來需要更換插件來源或增加插件,需要修改 Dockerfile,這增加了鏡像建置的耦合度。", - "suggestion": "建議將 plugin 安裝路徑抽離到環境變數或專用的配置檔案中,並在 Dockerfile 中透過讀取該清單來進行安裝,提升可維護性。", - "is_new": true - }, - { - "level": "warning", - "role": "Maya", - "location": "Dockerfile:19", - "problem": "在 Dockerfile 中安裝 Codex CLI 後未進行任何檢查,若下載失敗(例如 curl 回傳成功但內容殘缺),後續的 plugin 安裝與執行將會失敗。", - "suggestion": "在安裝指令後增加 `codex --version` 的驗證步驟,確保 CLI 已正確安裝並能正常執行。", - "is_new": true - }, - { - "level": "info", - "role": "Bard", - "location": "tests/entrypoint_test.sh:7", - "problem": "專案在 `entrypoint.sh` 中使用 `die` 函數處理致命錯誤,但測試腳本中卻定義了名稱不同的 `fail` 函數,這使得專案內的錯誤處理語彙不夠一致,略顯突兀。", - "suggestion": "建議將 `tests/entrypoint_test.sh` 中的 `fail` 函數更名為 `die`,使錯誤處理的語彙在專案各處保持一致,讓樂章的節奏更為統一。", - "is_new": true } ]