From bb2d411043cf48129998809643e44bd7e728d242 Mon Sep 17 00:00:00 2001 From: Michael Bolin Date: Thu, 24 Apr 2025 16:38:57 -0700 Subject: [PATCH] fix: update scripts/build_container.sh to use pnpm instead of npm (#631) I suspect this is why some contributors kept accidentally including a new `codex-cli/package-lock.json` in their PRs. Note the `Dockerfile` still uses `npm` instead of `pnpm`, but that appears to be fine. (Probably nicer to globally install as few things as possible in the image.) --- codex-cli/scripts/build_container.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/codex-cli/scripts/build_container.sh b/codex-cli/scripts/build_container.sh index fd4c8f5a..d4d29f6b 100755 --- a/codex-cli/scripts/build_container.sh +++ b/codex-cli/scripts/build_container.sh @@ -8,9 +8,9 @@ pushd "$SCRIPT_DIR/.." >> /dev/null || { echo "Error: Failed to change directory to $SCRIPT_DIR/.." exit 1 } -npm install -npm run build +pnpm install +pnpm run build rm -rf ./dist/openai-codex-*.tgz -npm pack --pack-destination ./dist +pnpm pack --pack-destination ./dist mv ./dist/openai-codex-*.tgz ./dist/codex.tgz docker build -t codex -f "./Dockerfile" .