.gitea/ai-review/exclusions.json

@@ -64,5 +64,23 @@
     "role": "Maya",
     "original_finding": "應在測試案例中模擬空 MODEL 輸入，並驗證腳本是否正確拋出錯誤並以 exit 1 終止。",
     "reason": "AI 對話收斂判定為誤報（問題在最新程式碼中不成立或不適用）"
+  },
+  {
+    "location": "Dockerfile:17",
+    "role": "Bard",
+    "original_finding": "將安裝腳本邏輯直接寫在 `RUN` 指令中顯得冗長，且使用 `$(mktemp)` 容易產生難以追蹤的臨時檔案。",
+    "reason": "安裝流程需在同一 Docker layer 中下載、驗證 SHA-256、執行並清理暫存腳本；留在 Dockerfile 可讓供應鏈驗證步驟與安裝命令緊鄰，mktemp 產生的檔案也已於同一 RUN 中刪除。"
+  },
+  {
+    "location": "entrypoint.sh:45",
+    "role": "Bard",
+    "original_finding": "使用靜態路徑檢查 `auth.json` 是否存在，若前次執行中斷導致檔案未清除，會導致後續執行失敗（Self-inflicted DoS）。",
+    "reason": "拒絕覆寫既有 auth.json 是刻意的 secret 安全保護，避免覆蓋呼叫端掛載或殘留的認證檔；Docker Action 預期每次執行使用獨立容器，前次中斷造成殘留的風險低於靜默覆寫 secret 的風險。"
+  },
+  {
+    "location": "tests/entrypoint_test.sh:17",
+    "role": "Bard",
+    "original_finding": "在測試中建立 `codex` 指令時，Here-document 與命令混雜，可讀性較低。",
+    "reason": "測試 helper 需要動態產生可執行的假 codex 指令，here-document 在此處比額外 fixture 檔更直觀且維持測試自含；此項屬風格偏好，不影響行為正確性。"
   }
 ]

.gitea/ai-review/findings.json

@@ -1,83 +1 @@
-[
-  {
-    "level": "critical",
-    "role": "Assassin",
-    "location": "Dockerfile:13",
-    "problem": "直接將網際網路的腳本經由 pipe 傳送給 shell 執行 (curl | sh) 極度危險，且缺乏驗證，極易導致容器建置階段遭劫持或執行惡意代碼。",
-    "suggestion": "應先將腳本下載至本地，透過 sha256sum 或 gpg 驗證其完整性與來源可靠性後，再執行安裝。",
-    "is_new": false
-  },
-  {
-    "level": "critical",
-    "role": "Assassin",
-    "location": "Dockerfile:16",
-    "problem": "直接從網址下載並執行安裝腳本，未鎖定版本或驗證 checksum，且從未經驗證的遠端儲存庫安裝插件，存在嚴重供應鏈攻擊風險。",
-    "suggestion": "必須鎖定特定 commit hash (SHA) 來引用插件，並將安裝腳本下載至本地，進行 SHA-256 雜湊驗證。"
-  },
-  {
-    "level": "critical",
-    "role": "Assassin",
-    "location": "Dockerfile:17",
-    "problem": "Dockerfile 直接從外部連結下載並執行腳本，且未驗證其雜湊值，若來源遭攔截或篡改，將導致任意程式碼執行。",
-    "suggestion": "下載指令碼後，務必使用 sha256sum 或其他雜湊函數驗證其完整性，確保與預期內容一致後再執行。"
-  },
-  {
-    "level": "critical",
-    "role": "Assassin",
-    "location": "Dockerfile:20",
-    "problem": "自動化從外部 Gitea 儲存庫安裝插件，若該儲存庫被竄改，將導致攻擊者可以在執行環境中安裝並執行惡意程式碼。這是極高風險的行為。",
-    "suggestion": "應限制插件來源，並在安裝前進行原始碼審查。若非必要，請將插件程式碼打包至 Docker Image 內部，避免動態安裝。",
-    "is_new": false
-  },
-  {
-    "level": "critical",
-    "role": "Maya",
-    "location": "Dockerfile:11",
-    "problem": "新增了 Codex CLI 的安裝與插件註冊過程，但沒有對應的測試來驗證安裝是否成功、插件是否正確載入。",
-    "suggestion": "應在測試套件中加入 Docker 映像檔的整合測試，驗證容器啟動後 `codex --version` 及插件列表是否存在。",
-    "is_new": false
-  },
-  {
-    "level": "warning",
-    "role": "Assassin",
-    "location": "Dockerfile:10",
-    "problem": "Codex 外掛市集來源指向 `gitea.jsc.idv.tw`，未驗證來源的安全性。若該伺服器遭駭，將導致自動安裝惡意或篡改過的外掛，進而導致供應鏈攻擊。",
-    "suggestion": "若可能，請將外掛來源固定在受信任的內部儲存庫或使用經簽署的外掛版本。確保來源伺服器具有嚴格的存取控管與安全性掃描。",
-    "is_new": true
-  },
-  {
-    "level": "warning",
-    "role": "Leo",
-    "location": "Dockerfile:10",
-    "problem": "環境變數使用以空格分隔的字串，若未來名稱中包含空格將導致 shell 展開錯誤，且難以維護。",
-    "suggestion": "建議改用換行符號（\n）分隔，並在安裝迴圈中使用 `IFS=$'\n'` 處理，以提高 shell 指令的健壯性。"
-  },
-  {
-    "level": "warning",
-    "role": "Bard",
-    "location": "Dockerfile:17",
-    "problem": "將安裝腳本邏輯直接寫在 `RUN` 指令中顯得冗長，且使用 `$(mktemp)` 容易產生難以追蹤的臨時檔案。",
-    "suggestion": "考慮將安裝邏輯封裝成一個獨立的 script 檔案，讓 `Dockerfile` 更簡潔優雅。"
-  },
-  {
-    "level": "warning",
-    "role": "Bard",
-    "location": "entrypoint.sh:45",
-    "problem": "使用靜態路徑檢查 `auth.json` 是否存在，若前次執行中斷導致檔案未清除，會導致後續執行失敗（Self-inflicted DoS）。",
-    "suggestion": "移除對既有檔案的檢查，改為在執行前確保該檔案為最新且受控狀態，或使用唯一的隨機臨時檔。"
-  },
-  {
-    "level": "warning",
-    "role": "Bard",
-    "location": "tests/entrypoint_test.sh:17",
-    "problem": "在測試中建立 `codex` 指令時，Here-document 與命令混雜，可讀性較低。",
-    "suggestion": "建議使用更整齊的縮排格式，或將其抽離為獨立的測試輔助檔案。"
-  },
-  {
-    "level": "info",
-    "role": "Maya",
-    "location": "tests/entrypoint_test.sh:65",
-    "problem": "測試案例對於輸入參數的邊界測試（例如 `PROMPT` 為空字串、極長字串）不足。",
-    "suggestion": "增加針對 `PROMPT` 輸入為空字串或是包含特殊 Shell 跳脫字元的測試案例。"
-  }
-]
+[]