Add codex-cli to installer - #22
Conversation
Co-authored-by: bobbravo2 <348865+bobbravo2@users.noreply.github.com>
Co-authored-by: bobbravo2 <348865+bobbravo2@users.noreply.github.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1d543a006a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # 15. codex-cli — OpenAI Codex CLI agent | ||
| if ! command -v codex &>/dev/null; then | ||
| echo "🤖 codex-cli not found. The Board has authorised autonomous coding on the floor. Installing..." | ||
| npm install -g @openai/codex |
There was a problem hiding this comment.
When node is present but npm is not—such as with a minimal or custom Node installation—the earlier Node.js block skips brew install node, and this command then fails with exit 127 under set -e, aborting the installer before Codexbar and the completion message. Check for npm alongside node or otherwise install it before using it.
Useful? React with 👍 / 👎.
Provisions
@openai/codexCLI as part of the standard Severed Floor environment.Changes
install.sh— Section 15: idempotentcommand -v codexguard; installs vianpm install -g @openai/codextest/install.bats— Install/skip test pair;npmmock added tosetup()(needed since install path callsnpmdirectly);codexstub added to the all-tools E2E testREADME.md— New table row, ordered before Codexbar to match install sequence.github/workflows/ci.yml—codex --versionadded to E2E verify step