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
+1
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d650b88ab3 - Show all commits
tests/docker_image_test.sh
+1
View File
@@ -18,5 +18,6 @@ fi
docker build -t "$IMAGE_NAME" "$ROOT_DIR"
docker run --rm --entrypoint codex "$IMAGE_NAME" --version
Ghost marked this conversation as resolved
gitea-actions commented 2026-06-24 13:47:12 +00:00
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Maya
問題：目前的 Docker 冒煙測試僅驗證了 CLI 二進位檔是否存在，但尚未驗證其在容器內執行時是否能正常存取與寫入 CODEX_HOME 環境設定的目錄。
建議：建議在 docker_image_test.sh 中增加一個測試案例，執行 codex --version 之外的指令，驗證容器權限與目錄環境變數設定是否正確。

**嚴重等級**：🔵 建議 **審查員**：Maya **問題**：目前的 Docker 冒煙測試僅驗證了 CLI 二進位檔是否存在，但尚未驗證其在容器內執行時是否能正常存取與寫入 `CODEX_HOME` 環境設定的目錄。 **建議**：建議在 `docker_image_test.sh` 中增加一個測試案例，執行 `codex --version` 之外的指令，驗證容器權限與目錄環境變數設定是否正確。
docker run --rm --entrypoint sh "$IMAGE_NAME" -c 'mkdir -p "$CODEX_HOME" && test -w "$CODEX_HOME" && codex --version >/dev/null'
docker run --rm --entrypoint codex "$IMAGE_NAME" plugin list | grep -q "jsc@doc"
docker run --rm --entrypoint codex "$IMAGE_NAME" plugin list | grep -q "jsc@code-review"