Skip to content

fix(ui,sdk-bridge): 'codex --login' -> 'codex login' in loginCommand prompts#329

Merged
chadbyte merged 1 commit intochadbyte:mainfrom
MukundaKatta:codex/fix-codex-login-command-typo
Apr 22, 2026
Merged

fix(ui,sdk-bridge): 'codex --login' -> 'codex login' in loginCommand prompts#329
chadbyte merged 1 commit intochadbyte:mainfrom
MukundaKatta:codex/fix-codex-login-command-typo

Conversation

@MukundaKatta
Copy link
Copy Markdown
Contributor

Summary

Per #327, Codex CLI uses a subcommand (`codex login`), not a flag (`codex --login`). Two sites suggested the invalid flag form:

  • `lib/sdk-bridge.js:924` — `loginCommand: "codex --login"` (sent to UI)
  • `lib/public/modules/app-rendering.js:499` — fallback `loginCmd` default

Users who followed the suggestion got 'unknown argument --login' and stayed stuck in the auth-required loop.

This PR is scoped to the login-command typo. The larger Darwin-only hardcoded binary path issue in the same issue is a separate refactor and left for a follow-up.

Refs #327

Testing

Two one-word replacements in two JS files; verified no `codex --login` references remain.

…prompts

Codex CLI uses a subcommand ('codex login'), not a flag. The UI
prompt in sdk-bridge.js (line 924) and the fallback prompt in
app-rendering.js (line 499) suggested the invalid 'codex --login'.
Following a bad suggestion leaves users stuck in the auth-required
loop.

This PR is scoped to the login-command typo only. The Darwin
hard-coded binary path issue (also in chadbyte#327) is a separate, larger
refactor and left for a follow-up.

Refs chadbyte#327
@chadbyte chadbyte merged commit 2519403 into chadbyte:main Apr 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This issue has been resolved in version 2.32.1-beta.1 (main).

To update, run:

npx clay-server@2.32.1-beta.1

-- Clay Deploy Bot

Build anything, with anyone, in one place.

@github-actions
Copy link
Copy Markdown
Contributor

This issue has been resolved in version 2.33.0 (stable).

To update, run:

npx clay-server@2.33.0

-- Clay Deploy Bot

Build anything, with anyone, in one place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants