Skip to content

fix(codex): prefer npm binary over WindowsApps#2583

Open
somewherelostt wants to merge 1 commit into
mainfrom
fix/codex-windowsapps-auth
Open

fix(codex): prefer npm binary over WindowsApps#2583
somewherelostt wants to merge 1 commit into
mainfrom
fix/codex-windowsapps-auth

Conversation

@somewherelostt

Copy link
Copy Markdown
Collaborator

Fixes #2582

Summary

  • Prefer the npm-installed Codex shim/native binary on Windows before probing PATH.
  • Skip the packaged WindowsApps OpenAI Codex executable when it appears in PATH, because it can shadow the authenticated npm install and return false auth failures.
  • Add a Windows regression test for the npm-vs-WindowsApps resolver ordering.

Test

  • cd backend && go test ./internal/adapters/agent/codex

@somewherelostt

Copy link
Copy Markdown
Collaborator Author
image fixed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows Codex auth probe reports Needs auth when WindowsApps codex.exe shadows npm codex.cmd

1 participant