chore: update ai-review findings [ai-review-bot][failure]

.gitea/ai-review/findings.json

@@ -31,37 +31,36 @@
     "suggestion": "應在測試套件中加入 Docker 映像檔的整合測試，驗證容器啟動後 `codex --version` 及插件列表是否存在。",
     "is_new": false
   },
-  {
-    "level": "warning",
-    "role": "Leo",
-    "location": "Dockerfile:17",
-    "problem": "在 Dockerfile 中直接使用 `curl` 下載並執行安裝腳本存在潛在的安全風險與不確定性，且沒有檢查腳本的完整性或簽章。若腳本內容在未來變更，可能導致映像檔建置失敗或植入非預期的內容。",
-    "suggestion": "建議將安裝腳本改為明確的版本化下載，或者將安裝邏輯整合進 Dockerfile 自身。",
-    "is_new": false
-  },
-  {
-    "level": "warning",
-    "role": "Leo",
-    "location": "Dockerfile:20",
-    "problem": "在建置期透過網路安裝插件，增加建置複雜度、時間，且對外部網路依賴過高，不可重現。",
-    "suggestion": "建議將插件預先打包在映像檔中，或使用特定版本的 manifest 檔案管理相依性。",
-    "is_new": false
-  },
   {
     "level": "warning",
     "role": "Leo",
     "location": "Dockerfile:23",
     "problem": "插件安裝邏輯重複，直接寫死在指令中。未來若需要安裝更多插件，維護成本會隨數量線性增加，且難以管理插件版本。",
     "suggestion": "將插件清單抽離為環境變數或專用的設定檔，在 Dockerfile 中使用迴圈讀取並安裝，提升擴充性。",
+    "is_new": false
+  },
+  {
+    "level": "warning",
+    "role": "Bard",
+    "location": "entrypoint.sh:11",
+    "problem": "清理函式 `cleanup` 內的變數命名 (`auth_file`, `auth_path`, `codex_output`, `auth_lock`) 雖清楚，但寫法稍顯瑣碎，且 `trap` 的慣用語法建議確保在變數未定義時也能安全執行。",
+    "suggestion": "在 Shell 腳本中，建議統一使用 `unset` 或更簡潔的方式管理環境變數清理，並確保變數引用更具彈性。",
     "is_new": true
   },
   {
-    "level": "info",
-    "role": "Leo",
-    "location": "Dockerfile:18",
-    "problem": "在單一行指令中安裝多個插件且缺乏說明，未來若需新增、移除或維修特定插件時，維護者難以快速理解該插件的用途與相依性。",
-    "suggestion": "建議將插件安裝拆分為獨立的 RUN 指令，並為每個關鍵插件加上註解說明其用途。",
-    "is_new": false
+    "level": "warning",
+    "role": "Mage",
+    "location": "entrypoint.sh:35",
+    "problem": "使用 `mkdir` 作為鎖定機制若容器意外崩潰可能殘留鎖檔。",
+    "suggestion": "考慮使用更穩健的 `flock` 機制來管理檔案鎖並確保自動清理。"
+  },
+  {
+    "level": "warning",
+    "role": "Bard",
+    "location": "entrypoint.sh:65",
+    "problem": "在執行 `codex` 的函式中，使用 `set +e` 與 `set -e` 的開關切換來處理回傳值，雖然正確但破壞了程式碼的流暢閱讀感，像是在樂譜中頻繁變調。",
+    "suggestion": "建議嘗試使用 `if ! codex ...; then status=$?; fi` 的方式，避免在函式內頻繁切換 `set -e` 狀態，保持程式邏輯的單純性。",
+    "is_new": true
   },
   {
     "level": "info",
@@ -69,6 +68,6 @@
     "location": "Dockerfile:7",
     "problem": "環境變數散落在檔案中間，這使得 Dockerfile 的基礎配置顯得凌亂，彷彿一首詩的標題與正文糾纏不清。",
     "suggestion": "建議將所有的 `ENV` 設定集中放置在 `Dockerfile` 的頂部，讓配置一目瞭然，維持檔案配置的一致性。",
-    "is_new": true
+    "is_new": false
   }
 ]