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
+24 -2
2 changed files with 24 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 8e93f126c0 - Show all commits
action.yaml
+2
View File
2
 @@ -15,6 +15,8 @@ inputs:
outputs:
status:
description: 'Execution result status'
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 10:31:06 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Bard
問題：輸出欄位命名為 text，但賦值內容為「執行狀態（status）」，語義不符。
建議：將欄位名稱改為 status，以保持命名與意圖一致。

**嚴重等級**：🟡 警告 **審查員**：Bard **問題**：輸出欄位命名為 `text`，但賦值內容為「執行狀態（status）」，語義不符。 **建議**：將欄位名稱改為 `status`，以保持命名與意圖一致。
output:
description: 'Codex execution output'
runs:
using: 'docker'
image: 'Dockerfile'
2
entrypoint.sh
+21 -1
View File
32
 @@ -33,10 +33,30 @@ mv "$auth_file" "$CODEX_HOME/auth.json"
chmod 600 "$CODEX_HOME/auth.json"
trap - EXIT
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 10:53:11 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：使用 mkdir 作為鎖定機制若容器意外崩潰可能殘留鎖檔。
建議：考慮使用更穩健的 flock 機制來管理檔案鎖並確保自動清理。

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：使用 `mkdir` 作為鎖定機制若容器意外崩潰可能殘留鎖檔。 **建議**：考慮使用更穩健的 `flock` 機制來管理檔案鎖並確保自動清理。
codex_output="$(mktemp)"
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
問題：直接執行 codex exec 而未對其可能的執行失敗進行顯式的錯誤捕捉。若該指令失敗，腳本仍會繼續執行後續步驟（例如設定 GitHub Output），這會導致對外回報錯誤的狀態碼不一致。
建議：應對 codex exec 進行錯誤檢查（使用 if ! codex exec ...; then ... fi），確保在失敗時能正確終止腳本並輸出相關錯誤訊息。

**嚴重等級**：🟡 警告 **審查員**：Leo **問題**：直接執行 `codex exec` 而未對其可能的執行失敗進行顯式的錯誤捕捉。若該指令失敗，腳本仍會繼續執行後續步驟（例如設定 GitHub Output），這會導致對外回報錯誤的狀態碼不一致。 **建議**：應對 `codex exec` 進行錯誤檢查（使用 `if ! codex exec ...; then ... fi`），確保在失敗時能正確終止腳本並輸出相關錯誤訊息。
gitea-actions commented 2026-06-24 10:34:08 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：mv 指令未檢查目標檔案是否已存在，無條件覆蓋會導致舊有的有效設定直接遺失。
建議：在 mv 前加入檢查，若 auth.json 已存在，應決定是否備份、報錯，或確認無須保留舊檔。

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：`mv` 指令未檢查目標檔案是否已存在，無條件覆蓋會導致舊有的有效設定直接遺失。 **建議**：在 `mv` 前加入檢查，若 `auth.json` 已存在，應決定是否備份、報錯，或確認無須保留舊檔。
trap 'rm -f "$codex_output"' EXIT
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:36:02 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：執行 codex exec 的邏輯中使用了 set +e 暫時關閉錯誤退出機制，儘管後續透過 PIPESTATUS 檢查，但若 codex exec 指令本身因為環境錯誤或語法錯誤無法啟動，codex_status 可能會取得非預期的狀態碼。
建議：明確定義各個步驟的錯誤處理，或者將 codex exec 包裝在明確的函數中檢查退出碼。

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：執行 `codex exec` 的邏輯中使用了 `set +e` 暫時關閉錯誤退出機制，儘管後續透過 `PIPESTATUS` 檢查，但若 `codex exec` 指令本身因為環境錯誤或語法錯誤無法啟動，`codex_status` 可能會取得非預期的狀態碼。 **建議**：明確定義各個步驟的錯誤處理，或者將 `codex exec` 包裝在明確的函數中檢查退出碼。
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:34:08 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Mage
問題：在執行 codex exec 前便呼叫 trap - EXIT 移除了清理機制。若 codex exec 執行失敗或中斷，包含敏感憑證的 auth.json 將殘留在容器中，未能被安全刪除。
建議：應在 codex exec 完成後，或確保程式結束時（包含失敗的情況）都能正確執行刪除 auth.json 的邏輯。

**嚴重等級**：🔴 嚴重 **審查員**：Mage **問題**：在執行 `codex exec` 前便呼叫 `trap - EXIT` 移除了清理機制。若 `codex exec` 執行失敗或中斷，包含敏感憑證的 `auth.json` 將殘留在容器中，未能被安全刪除。 **建議**：應在 `codex exec` 完成後，或確保程式結束時（包含失敗的情況）都能正確執行刪除 `auth.json` 的邏輯。
set +e
codex exec \
--model "$MODEL" \
"$PROMPT"
"$PROMPT" 2>&1 | tee "$codex_output"
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 10:45:34 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：使用 install -m 600 將 auth_file 移至 auth_path。若 mktemp 產生的 auth_file 與 auth_path 不在同一個檔案系統分區（Filesystem），install 指令（底層通常是 copy + chmod/chown）可能會有短暫時間檔案權限為預設值，存在權限外洩風險。
建議：建議在確認檔案權限無誤後，於同一分區內使用 mv 進行原子性移轉，或在寫入前先明確設定 umask。

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：使用 `install -m 600` 將 `auth_file` 移至 `auth_path`。若 `mktemp` 產生的 `auth_file` 與 `auth_path` 不在同一個檔案系統分區（Filesystem），`install` 指令（底層通常是 copy + chmod/chown）可能會有短暫時間檔案權限為預設值，存在權限外洩風險。 **建議**：建議在確認檔案權限無誤後，於同一分區內使用 `mv` 進行原子性移轉，或在寫入前先明確設定 `umask`。
codex_status="${PIPESTATUS[0]}"
set -e
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:36:02 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Mage
問題：在 GitHub Actions 輸出處理中，使用了 date +%s 來產生 output delimiter。雖然發生機率極低，但在極高併發或相同執行時間下可能發生碰撞導致輸出被截斷。
建議：建議使用更具隨機性的 UUID 或確保 delimiter 包含隨機字串，避免與內容衝突。

**嚴重等級**：🔵 建議 **審查員**：Mage **問題**：在 GitHub Actions 輸出處理中，使用了 `date +%s` 來產生 output delimiter。雖然發生機率極低，但在極高併發或相同執行時間下可能發生碰撞導致輸出被截斷。 **建議**：建議使用更具隨機性的 UUID 或確保 delimiter 包含隨機字串，避免與內容衝突。
gitea-actions commented 2026-06-24 11:03:19 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Bard
問題：使用靜態路徑檢查 auth.json 是否存在，若前次執行中斷導致檔案未清除，會導致後續執行失敗（Self-inflicted DoS）。
建議：移除對既有檔案的檢查，改為在執行前確保該檔案為最新且受控狀態，或使用唯一的隨機臨時檔。

**嚴重等級**：🟡 警告 **審查員**：Bard **問題**：使用靜態路徑檢查 `auth.json` 是否存在，若前次執行中斷導致檔案未清除，會導致後續執行失敗（Self-inflicted DoS）。 **建議**：移除對既有檔案的檢查，改為在執行前確保該檔案為最新且受控狀態，或使用唯一的隨機臨時檔。
if [[ -n "${GITHUB_OUTPUT:-}" ]]; then
output_delimiter="CODEX_OUTPUT_$(date +%s)_$$"
if [[ "$codex_status" -eq 0 ]]; then
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 13:20:25 +00:00
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Mage
問題：在執行 install -m 600 時，若 codex 進程已經在嘗試讀取 auth.json，會發生檔案存取競態（Race Condition）。雖然使用了 flock，但這僅在同一個 shell 腳本實例中有效，無法保護跨容器或跨執行環境的檔案存取一致性。
建議：建議將 auth.json 放置於唯讀且受限的目錄中，並通過環境變數直接傳遞路徑給 codex，而非在執行時進行檔案寫入與複製。

**嚴重等級**：🔴 嚴重 **審查員**：Mage **問題**：在執行 install -m 600 時，若 codex 進程已經在嘗試讀取 auth.json，會發生檔案存取競態（Race Condition）。雖然使用了 flock，但這僅在同一個 shell 腳本實例中有效，無法保護跨容器或跨執行環境的檔案存取一致性。 **建議**：建議將 auth.json 放置於唯讀且受限的目錄中，並通過環境變數直接傳遞路徑給 codex，而非在執行時進行檔案寫入與複製。
echo "status=completed" >> "$GITHUB_OUTPUT"
else
echo "status=failed" >> "$GITHUB_OUTPUT"
fi
{
echo "output<<$output_delimiter"
cat "$codex_output"
echo "$output_delimiter"
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-24 10:45:34 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔴 嚴重
審查員：Mage
問題：在併發環境下，若多個執行緒或過程同時嘗試建立 auth.json，可能會因為檢查檔案是否存在（Line 35）與建立檔案之間的競態條件，導致 die 錯誤甚至意外地驗證失敗。雖然目前看起來是單一容器環境，但在 GitHub Actions 或其他 Runner 中，安全起見應使用原子操作。
建議：建議使用 mkdir 的原子性或檔案鎖定機制，或是確保 auth_path 在容器初始化階段就已經是唯讀且受保護的，避免檢查與寫入之間的延遲風險。

**嚴重等級**：🔴 嚴重 **審查員**：Mage **問題**：在併發環境下，若多個執行緒或過程同時嘗試建立 auth.json，可能會因為檢查檔案是否存在（Line 35）與建立檔案之間的競態條件，導致 `die` 錯誤甚至意外地驗證失敗。雖然目前看起來是單一容器環境，但在 GitHub Actions 或其他 Runner 中，安全起見應使用原子操作。 **建議**：建議使用 `mkdir` 的原子性或檔案鎖定機制，或是確保 `auth_path` 在容器初始化階段就已經是唯讀且受保護的，避免檢查與寫入之間的延遲風險。
gitea-actions commented 2026-06-24 13:20:25 +00:00
Review
Copy Link
Copy Source

嚴重等級：🟡 警告
審查員：Mage
問題：在容器化環境（通常是 ephemeral 的）中，auth.json 寫入後立刻被刪除，這會導致 codex 在後續執行中因找不到驗證檔案而無法運作。另外，trap 的清理機制會導致該檔案在 codex 完成工作前被刪除，這對於長效執行或需要多次存取的應用場景是錯誤的設計。
建議：評估 codex 是否需要該檔案在執行期間持續存在。若需要，請調整清理時機，或考慮使用記憶體中的臨時檔案系統（tmpfs）來提升安全性，而非直接移除檔案。

**嚴重等級**：🟡 警告 **審查員**：Mage **問題**：在容器化環境（通常是 ephemeral 的）中，auth.json 寫入後立刻被刪除，這會導致 codex 在後續執行中因找不到驗證檔案而無法運作。另外，trap 的清理機制會導致該檔案在 codex 完成工作前被刪除，這對於長效執行或需要多次存取的應用場景是錯誤的設計。 **建議**：評估 codex 是否需要該檔案在執行期間持續存在。若需要，請調整清理時機，或考慮使用記憶體中的臨時檔案系統（tmpfs）來提升安全性，而非直接移除檔案。
} >> "$GITHUB_OUTPUT"
fi
exit "$codex_status"