old-actions/codex
1
0
Fork 0
Code Pull Requests Actions Releases 2 Activity

處理 AI review findings 並改寫 Node.js entrypoint #1

Merged
jiantw83 merged 58 commits from ai-review-resolve/20260624102518 into develop 2026-06-24 14:09:27 +00:00
Conversation 34 Commits 58 Files Changed 10
+8 -4
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b63596fb09 - Show all commits
Dockerfile
+8 -4
View File
@@ -4,8 +4,9 @@ FROM alpine:latest
ENV CODEX_NON_INTERACTIVE=1
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:30:54 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：在 Dockerfile 中安裝了 git 且未清理，導致映像檔過大。
建議：建議在同一層 RUN 指令中以 --virtual 安裝 git，完成後立即刪除。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：在 Dockerfile 中安裝了 git 且未清理，導致映像檔過大。 **建議**：建議在同一層 RUN 指令中以 --virtual 安裝 git，完成後立即刪除。
ENV CODEX_INSTALL_DIR=/usr/local/bin
ENV CODEX_HOME=/root/.codex
ENV CODEX_PLUGINS="jsc@doc jsc@code-review"
ENV CODEX_PLUGIN_MARKETPLACES="https://gitea.jsc.idv.tw/plugins/doc.git https://gitea.jsc.idv.tw/plugins/code-review.git"
ARG CODEX_INSTALL_SHA256=73eb367137d151eabe89590a1d1a08a9c2fe2800a656bffac3c5707b62b42683
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:46:56 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Bard
問題：環境變數散落在檔案中間，這使得 Dockerfile 的基礎配置顯得凌亂，彷彿一首詩的標題與正文糾纏不清。
建議：建議將所有的 ENV 設定集中放置在 Dockerfile 的頂部，讓配置一目瞭然，維持檔案配置的一致性。

**嚴重等級**：🔵 建議 **審查員**：Bard **問題**：環境變數散落在檔案中間，這使得 Dockerfile 的基礎配置顯得凌亂，彷彿一首詩的標題與正文糾纏不清。 **建議**：建議將所有的 `ENV` 設定集中放置在 `Dockerfile` 的頂部，讓配置一目瞭然，維持檔案配置的一致性。
ARG CODEX_DOC_MARKETPLACE_REF=f8da961328a85f267b4402566e127310370169da
ARG CODEX_CODE_REVIEW_MARKETPLACE_REF=9e016edff016d58f4d64e0a5468220d35a0f657b
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 11:03:19 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Assassin
問題：Codex 外掛市集來源指向 gitea.jsc.idv.tw，未驗證來源的安全性。若該伺服器遭駭，將導致自動安裝惡意或篡改過的外掛，進而導致供應鏈攻擊。
建議：若可能，請將外掛來源固定在受信任的內部儲存庫或使用經簽署的外掛版本。確保來源伺服器具有嚴格的存取控管與安全性掃描。

**嚴重等級**：🟡 警告 **審查員**：Assassin **問題**：Codex 外掛市集來源指向 `gitea.jsc.idv.tw`，未驗證來源的安全性。若該伺服器遭駭，將導致自動安裝惡意或篡改過的外掛，進而導致供應鏈攻擊。 **建議**：若可能，請將外掛來源固定在受信任的內部儲存庫或使用經簽署的外掛版本。確保來源伺服器具有嚴格的存取控管與安全性掃描。
gitea-actions commented 2026-06-24 11:03:19 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：環境變數使用以空格分隔的字串，若未來名稱中包含空格將導致 shell 展開錯誤，且難以維護。
建議：建議改用換行符號（
）分隔，並在安裝迴圈中使用 IFS=$' ' 處理，以提高 shell 指令的健壯性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：環境變數使用以空格分隔的字串，若未來名稱中包含空格將導致 shell 展開錯誤，且難以維護。 **建議**：建議改用換行符號（ ）分隔，並在安裝迴圈中使用 `IFS=$' '` 處理，以提高 shell 指令的健壯性。
# 安裝必要的工具
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:27:10 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Maya
問題：新增了 Codex CLI 的安裝與插件註冊過程，但沒有對應的測試來驗證安裝是否成功、插件是否正確載入。
建議：應在測試套件中加入 Docker 映像檔的整合測試，驗證容器啟動後 codex --version 及插件列表是否存在。

**嚴重等級**：🔴 嚴重 **審查員**：Maya **問題**：新增了 Codex CLI 的安裝與插件註冊過程，但沒有對應的測試來驗證安裝是否成功、插件是否正確載入。 **建議**：應在測試套件中加入 Docker 映像檔的整合測試，驗證容器啟動後 `codex --version` 及插件列表是否存在。
RUN apk add --no-cache --no-check-certificate bash ca-certificates curl git jq util-linux
3
 @@ -15,13 +16,16 @@ RUN install_script="$(mktemp)" \
&& curl -fsSL --retry 3 --retry-delay 2 --max-time 120 \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:27:10 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：從未經驗證的遠端 git 儲存庫安裝插件，若儲存庫內容遭竄改，將導致任意程式碼執行風險。
建議：必須鎖定特定 commit hash (SHA) 來引用插件，確保所安裝內容符合預期且未遭篡改。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：從未經驗證的遠端 git 儲存庫安裝插件，若儲存庫內容遭竄改，將導致任意程式碼執行風險。 **建議**：必須鎖定特定 commit hash (SHA) 來引用插件，確保所安裝內容符合預期且未遭篡改。
gitea-actions commented 2026-06-24 10:30:53 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：從未經驗證的遠端 git 儲存庫安裝插件，或以不安全方式下載腳本，將導致任意程式碼執行風險及構建不一致。
建議：必須鎖定特定 commit hash (SHA) 來引用插件，確保所安裝內容符合預期且未遭篡改；並應對下載腳本進行哈希驗證。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：從未經驗證的遠端 git 儲存庫安裝插件，或以不安全方式下載腳本，將導致任意程式碼執行風險及構建不一致。 **建議**：必須鎖定特定 commit hash (SHA) 來引用插件，確保所安裝內容符合預期且未遭篡改；並應對下載腳本進行哈希驗證。
gitea-actions commented 2026-06-24 10:45:34 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：從未經驗證的遠端 git 儲存庫安裝插件，若儲存庫內容遭竄改，將導致任意程式碼執行風險。同時，直接執行從外部網路下載的腳本，且未經過 Hash 檢查或簽章驗證，是嚴重的供應鏈攻擊破口。
建議：必須鎖定特定 commit hash (SHA) 來引用插件，並對下載的腳本進行 SHA-256 雜湊驗證。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：從未經驗證的遠端 git 儲存庫安裝插件，若儲存庫內容遭竄改，將導致任意程式碼執行風險。同時，直接執行從外部網路下載的腳本，且未經過 Hash 檢查或簽章驗證，是嚴重的供應鏈攻擊破口。 **建議**：必須鎖定特定 commit hash (SHA) 來引用插件，並對下載的腳本進行 SHA-256 雜湊驗證。
gitea-actions commented 2026-06-24 11:03:19 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：直接從網址下載並執行安裝腳本，未鎖定版本或驗證 checksum，且從未經驗證的遠端儲存庫安裝插件，存在嚴重供應鏈攻擊風險。
建議：必須鎖定特定 commit hash (SHA) 來引用插件，並將安裝腳本下載至本地，進行 SHA-256 雜湊驗證。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：直接從網址下載並執行安裝腳本，未鎖定版本或驗證 checksum，且從未經驗證的遠端儲存庫安裝插件，存在嚴重供應鏈攻擊風險。 **建議**：必須鎖定特定 commit hash (SHA) 來引用插件，並將安裝腳本下載至本地，進行 SHA-256 雜湊驗證。
https://chatgpt.com/codex/install.sh \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:27:10 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Mage
問題：在 Docker 構建階段直接執行 codex plugin marketplace add 並嘗試同時安裝與啟用，若網路不穩或遠端 Gitea 服務暫時不可用，構建會直接失敗，導致整個鏡像無法建立。
建議：建議將插件安裝邏輯移至 entrypoint.sh，在執行時才確認環境並安裝，或者在 Dockerfile 中增加重試機制與錯誤捕獲。

**嚴重等級**：🔴 嚴重 **審查員**：Mage **問題**：在 Docker 構建階段直接執行 `codex plugin marketplace add` 並嘗試同時安裝與啟用，若網路不穩或遠端 Gitea 服務暫時不可用，構建會直接失敗，導致整個鏡像無法建立。 **建議**：建議將插件安裝邏輯移至 `entrypoint.sh`，在執行時才確認環境並安裝，或者在 Dockerfile 中增加重試機制與錯誤捕獲。
gitea-actions commented 2026-06-24 10:27:10 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：每個 RUN 指令都會建立一個新的 Docker 映像層，在此處拆分為兩個獨立的 RUN 指令安裝插件，會產生不必要的額外層，增加了映像檔體積並浪費了建置與拉取映像檔的時間。
建議：請將所有安裝插件的 RUN 指令透過 && 合併為單一個 RUN 指令，以優化映像層數並提升效能。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：每個 `RUN` 指令都會建立一個新的 Docker 映像層，在此處拆分為兩個獨立的 `RUN` 指令安裝插件，會產生不必要的額外層，增加了映像檔體積並浪費了建置與拉取映像檔的時間。 **建議**：請將所有安裝插件的 `RUN` 指令透過 `&&` 合併為單一個 `RUN` 指令，以優化映像層數並提升效能。
gitea-actions commented 2026-06-24 10:30:53 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Mage
問題：在 Docker 構建階段直接執行插件安裝並嘗試同時啟用，若網路不穩或遠端服務不可用，構建會失敗，且可能導致環境處於損壞狀態。
建議：建議將插件安裝邏輯移至 entrypoint.sh，或在 Dockerfile 中增加重試機制與錯誤捕獲（如 set -o pipefail）。

**嚴重等級**：🔴 嚴重 **審查員**：Mage **問題**：在 Docker 構建階段直接執行插件安裝並嘗試同時啟用，若網路不穩或遠端服務不可用，構建會失敗，且可能導致環境處於損壞狀態。 **建議**：建議將插件安裝邏輯移至 `entrypoint.sh`，或在 Dockerfile 中增加重試機制與錯誤捕獲（如 `set -o pipefail`）。
gitea-actions commented 2026-06-24 10:30:53 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：拆分為多個獨立的 RUN 指令安裝插件，會產生不必要的映像層，增加了體積與建置時間。
建議：將相關安裝指令透過 && 合併為單一個 RUN 指令，以優化映像層數並提升效能。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：拆分為多個獨立的 `RUN` 指令安裝插件，會產生不必要的映像層，增加了體積與建置時間。 **建議**：將相關安裝指令透過 `&&` 合併為單一個 `RUN` 指令，以優化映像層數並提升效能。
gitea-actions commented 2026-06-24 10:31:06 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：將安裝插件拆分為多個獨立的 RUN 指令，產生不必要的映像層，增加體積並減慢建置速度。
建議：請將所有安裝插件的 RUN 指令透過 && 合併為單一個 RUN 指令。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：將安裝插件拆分為多個獨立的 `RUN` 指令，產生不必要的映像層，增加體積並減慢建置速度。 **建議**：請將所有安裝插件的 `RUN` 指令透過 `&&` 合併為單一個 `RUN` 指令。
gitea-actions commented 2026-06-24 10:34:08 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在 Dockerfile 中直接使用 curl 下載並執行安裝腳本 (sh "$install_script") 存在潛在的安全風險與不確定性，且沒有檢查腳本的完整性或簽章。若腳本內容在未來變更，可能導致映像檔建置失敗或植入非預期的內容，增加維護與安全風險。
建議：建議將安裝腳本改為明確的版本化下載，或者如果可能，將安裝邏輯整合進 Dockerfile 自身，以確保建置過程的冪等性與安全性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在 Dockerfile 中直接使用 `curl` 下載並執行安裝腳本 (`sh "$install_script"`) 存在潛在的安全風險與不確定性，且沒有檢查腳本的完整性或簽章。若腳本內容在未來變更，可能導致映像檔建置失敗或植入非預期的內容，增加維護與安全風險。 **建議**：建議將安裝腳本改為明確的版本化下載，或者如果可能，將安裝邏輯整合進 Dockerfile 自身，以確保建置過程的冪等性與安全性。
gitea-actions commented 2026-06-24 10:45:34 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在 Dockerfile 中直接使用 curl 下載並執行安裝腳本存在潛在的安全風險與不確定性，且沒有檢查腳本的完整性或簽章。若腳本內容在未來變更，可能導致映像檔建置失敗或植入非預期的內容。
建議：建議將安裝腳本改為明確的版本化下載，或者將安裝邏輯整合進 Dockerfile 自身。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在 Dockerfile 中直接使用 `curl` 下載並執行安裝腳本存在潛在的安全風險與不確定性，且沒有檢查腳本的完整性或簽章。若腳本內容在未來變更，可能導致映像檔建置失敗或植入非預期的內容。 **建議**：建議將安裝腳本改為明確的版本化下載，或者將安裝邏輯整合進 Dockerfile 自身。
gitea-actions commented 2026-06-24 11:03:19 +00:00
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：Dockerfile 直接從外部連結下載並執行腳本，且未驗證其雜湊值，若來源遭攔截或篡改，將導致任意程式碼執行。
建議：下載指令碼後，務必使用 sha256sum 或其他雜湊函數驗證其完整性，確保與預期內容一致後再執行。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：Dockerfile 直接從外部連結下載並執行腳本，且未驗證其雜湊值，若來源遭攔截或篡改，將導致任意程式碼執行。 **建議**：下載指令碼後，務必使用 sha256sum 或其他雜湊函數驗證其完整性，確保與預期內容一致後再執行。
gitea-actions commented 2026-06-24 11:03:19 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Bard
問題：將安裝腳本邏輯直接寫在 RUN 指令中顯得冗長，且使用 $(mktemp) 容易產生難以追蹤的臨時檔案。
建議：考慮將安裝邏輯封裝成一個獨立的 script 檔案，讓 Dockerfile 更簡潔優雅。

**嚴重等級**：🟡 警告 **審查員**：Bard **問題**：將安裝腳本邏輯直接寫在 `RUN` 指令中顯得冗長，且使用 `$(mktemp)` 容易產生難以追蹤的臨時檔案。 **建議**：考慮將安裝邏輯封裝成一個獨立的 script 檔案，讓 `Dockerfile` 更簡潔優雅。
-o "$install_script" \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:45:34 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Leo
問題：在單一行指令中安裝多個插件且缺乏說明，未來若需新增、移除或維修特定插件時，維護者難以快速理解該插件的用途與相依性。
建議：建議將插件安裝拆分為獨立的 RUN 指令，並為每個關鍵插件加上註解說明其用途。

**嚴重等級**：🔵 建議 **審查員**：Leo **問題**：在單一行指令中安裝多個插件且缺乏說明，未來若需新增、移除或維修特定插件時，維護者難以快速理解該插件的用途與相依性。 **建議**：建議將插件安裝拆分為獨立的 RUN 指令，並為每個關鍵插件加上註解說明其用途。
gitea-actions commented 2026-06-24 13:11:54 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：若 sh "$install_script" 安裝過程失敗，rm -f 由於被 && 連接，不會被執行，導致 build 過程中殘留不必要的暫存檔案，雖然 Docker Layer 會自動清除，但仍屬不佳的清理實作。
建議：建議改用 trap 進行清理，或確保 rm -f 指令在 failure 情境下也能執行，例如：sh "$install_script" || { rm -f "$install_script"; exit 1; }

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：若 sh "$install_script" 安裝過程失敗，rm -f 由於被 && 連接，不會被執行，導致 build 過程中殘留不必要的暫存檔案，雖然 Docker Layer 會自動清除，但仍屬不佳的清理實作。 **建議**：建議改用 trap 進行清理，或確保 rm -f 指令在 failure 情境下也能執行，例如：sh "$install_script" || { rm -f "$install_script"; exit 1; }
&& echo "${CODEX_INSTALL_SHA256} ${install_script}" | sha256sum -c - \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:58:17 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Maya
問題：在 Dockerfile 中安裝 Codex CLI 後未進行任何檢查，若下載失敗（例如 curl 回傳成功但內容殘缺），後續的 plugin 安裝與執行將會失敗。
建議：在安裝指令後增加 codex --version 的驗證步驟，確保 CLI 已正確安裝並能正常執行。

**嚴重等級**：🟡 警告 **審查員**：Maya **問題**：在 Dockerfile 中安裝 Codex CLI 後未進行任何檢查，若下載失敗（例如 curl 回傳成功但內容殘缺），後續的 plugin 安裝與執行將會失敗。 **建議**：在安裝指令後增加 `codex --version` 的驗證步驟，確保 CLI 已正確安裝並能正常執行。
&& sh "$install_script" \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:27:10 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在建置映像檔時直接透過 codex plugin marketplace add 安裝插件，會增加映像檔建置的時間與複雜度，且若插件來源 Gitea 無法連線，映像檔就無法成功建置。
建議：建議將常用的插件預先打包在映像檔中，或是透過環境變數配置插件路徑，減少建置期的網絡依賴。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在建置映像檔時直接透過 `codex plugin marketplace add` 安裝插件，會增加映像檔建置的時間與複雜度，且若插件來源 Gitea 無法連線，映像檔就無法成功建置。 **建議**：建議將常用的插件預先打包在映像檔中，或是透過環境變數配置插件路徑，減少建置期的網絡依賴。
gitea-actions commented 2026-06-24 10:30:53 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在建置映像檔時直接安裝插件，會增加建置時間與複雜度，並引入對遠端 Gitea 的網路依賴。
建議：建議將常用的插件預先打包在映像檔中，或透過環境變數配置，減少建置期的網絡依賴。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在建置映像檔時直接安裝插件，會增加建置時間與複雜度，並引入對遠端 Gitea 的網路依賴。 **建議**：建議將常用的插件預先打包在映像檔中，或透過環境變數配置，減少建置期的網絡依賴。
gitea-actions commented 2026-06-24 10:31:06 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在建置期透過網路安裝插件，增加建置複雜度、時間，且對外部網路依賴過高，不可重現。
建議：建議將插件預先打包在映像檔中，或使用特定版本的 manifest 檔案管理相依性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在建置期透過網路安裝插件，增加建置複雜度、時間，且對外部網路依賴過高，不可重現。 **建議**：建議將插件預先打包在映像檔中，或使用特定版本的 manifest 檔案管理相依性。
gitea-actions commented 2026-06-24 10:34:08 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Bard
問題：插件安裝指令全部擠在同一個 RUN 指令中，缺乏視覺上的區隔，讓整個安裝流程顯得雜亂無章。
建議：建議使用反斜線將指令換行，並在市場加入與插件安裝之間加入空行或註解，以增強指令的可讀性。

**嚴重等級**：🔵 建議 **審查員**：Bard **問題**：插件安裝指令全部擠在同一個 `RUN` 指令中，缺乏視覺上的區隔，讓整個安裝流程顯得雜亂無章。 **建議**：建議使用反斜線將指令換行，並在市場加入與插件安裝之間加入空行或註解，以增強指令的可讀性。
gitea-actions commented 2026-06-24 10:45:34 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Assassin
問題：自動化從外部 Gitea 儲存庫安裝插件，若該儲存庫被竄改，將導致攻擊者可以在執行環境中安裝並執行惡意程式碼。這是極高風險的行為。
建議：應限制插件來源，並在安裝前進行原始碼審查。若非必要，請將插件程式碼打包至 Docker Image 內部，避免動態安裝。

**嚴重等級**：🔴 嚴重 **審查員**：Assassin **問題**：自動化從外部 Gitea 儲存庫安裝插件，若該儲存庫被竄改，將導致攻擊者可以在執行環境中安裝並執行惡意程式碼。這是極高風險的行為。 **建議**：應限制插件來源，並在安裝前進行原始碼審查。若非必要，請將插件程式碼打包至 Docker Image 內部，避免動態安裝。
&& codex --version \
&& rm -f "$install_script"
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:34:08 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：將插件安裝邏輯分散在多個 RUN 指令中，若其中一個插件安裝失敗，後續的建置流程仍會嘗試執行，且可能會留下殘留狀態，導致建置結果不可預測。
建議：建議將相關插件安裝指令整合到單一的 RUN 指令中，並加入錯誤處理與清理機制，確保安裝過程的原子性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：將插件安裝邏輯分散在多個 `RUN` 指令中，若其中一個插件安裝失敗，後續的建置流程仍會嘗試執行，且可能會留下殘留狀態，導致建置結果不可預測。 **建議**：建議將相關插件安裝指令整合到單一的 `RUN` 指令中，並加入錯誤處理與清理機制，確保安裝過程的原子性。
gitea-actions commented 2026-06-24 10:58:17 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：在 Dockerfile 中安裝 plugin 時，雖然使用了 for 迴圈處理變數，但直接將 plugin URL 硬編碼在 RUN 指令中。如果未來需要更換插件來源或增加插件，需要修改 Dockerfile，這增加了鏡像建置的耦合度。
建議：建議將 plugin 安裝路徑抽離到環境變數或專用的配置檔案中，並在 Dockerfile 中透過讀取該清單來進行安裝，提升可維護性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：在 Dockerfile 中安裝 plugin 時，雖然使用了 `for` 迴圈處理變數，但直接將 plugin URL 硬編碼在 RUN 指令中。如果未來需要更換插件來源或增加插件，需要修改 Dockerfile，這增加了鏡像建置的耦合度。 **建議**：建議將 plugin 安裝路徑抽離到環境變數或專用的配置檔案中，並在 Dockerfile 中透過讀取該清單來進行安裝，提升可維護性。
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:46:56 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：插件安裝邏輯重複，直接寫死在指令中。未來若需要安裝更多插件，維護成本會隨數量線性增加，且難以管理插件版本。
建議：將插件清單抽離為環境變數或專用的設定檔，在 Dockerfile 中使用迴圈讀取並安裝，提升擴充性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：插件安裝邏輯重複，直接寫死在指令中。未來若需要安裝更多插件，維護成本會隨數量線性增加，且難以管理插件版本。 **建議**：將插件清單抽離為環境變數或專用的設定檔，在 Dockerfile 中使用迴圈讀取並安裝，提升擴充性。
# 安裝技能
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 13:11:54 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Leo
問題：插件 URL 與版本參照 (REF) 硬編碼在 Dockerfile 中，未來若需更新插件或更換來源，需重新編譯整個 Docker 映像檔，維護成本較高。
建議：建議將這些插件清單與版本資訊移至外部設定檔 (如 plugins.json)，並在 Dockerfile 中讀取該檔案進行安裝，增加彈性。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：插件 URL 與版本參照 (REF) 硬編碼在 Dockerfile 中，未來若需更新插件或更換來源，需重新編譯整個 Docker 映像檔，維護成本較高。 **建議**：建議將這些插件清單與版本資訊移至外部設定檔 (如 plugins.json)，並在 Dockerfile 中讀取該檔案進行安裝，增加彈性。
RUN for marketplace in $CODEX_PLUGIN_MARKETPLACES; do codex plugin marketplace add "$marketplace"; done \
&& for plugin in $CODEX_PLUGINS; do codex plugin add "$plugin"; done
RUN codex plugin marketplace add "https://gitea.jsc.idv.tw/plugins/doc.git" --ref "$CODEX_DOC_MARKETPLACE_REF" \
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:34:08 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：插件安裝指令採取連續序列執行，導致多次獨立的網路 I/O 等待，嚴重浪費建置時間（估計浪費數秒）。
建議：檢查 codex CLI 是否支援批次安裝，若不支援，考慮將指令改為並行處理（例如：command1 & command2 & wait）以同時發送網路請求。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：插件安裝指令採取連續序列執行，導致多次獨立的網路 I/O 等待，嚴重浪費建置時間（估計浪費數秒）。 **建議**：檢查 codex CLI 是否支援批次安裝，若不支援，考慮將指令改為並行處理（例如：command1 & command2 & wait）以同時發送網路請求。
gitea-actions commented 2026-06-24 10:36:02 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Rogue
問題：插件安裝指令採取連續序列執行，導致多次獨立的網路 I/O 等待，嚴重浪費建置時間。
建議：檢查 codex CLI 是否支援批次安裝，若不支援，考慮將指令改為並行處理。

**嚴重等級**：🟡 警告 **審查員**：Rogue **問題**：插件安裝指令採取連續序列執行，導致多次獨立的網路 I/O 等待，嚴重浪費建置時間。 **建議**：檢查 codex CLI 是否支援批次安裝，若不支援，考慮將指令改為並行處理。
&& codex plugin marketplace add "https://gitea.jsc.idv.tw/plugins/code-review.git" --ref "$CODEX_CODE_REVIEW_MARKETPLACE_REF" \
&& codex plugin add "jsc@doc" \
&& codex plugin add "jsc@code-review"
COPY entrypoint.sh /entrypoint.sh
1