Skip to content

fix: isolate codex login#117

Merged
Loongphy merged 1 commit into
mainfrom
codex/isolate-login-codex-home
May 22, 2026
Merged

fix: isolate codex login#117
Loongphy merged 1 commit into
mainfrom
codex/isolate-login-codex-home

Conversation

@Loongphy
Copy link
Copy Markdown
Owner

Summary

  • run upstream codex login with a private temporary CODEX_HOME
  • copy the successful login auth.json into the managed Codex home before importing it
  • cover default and explicit CODEX_HOME login flows in integration tests

Validation

  • HOME=/tmp/codex-auth-login-fix CODEX_HOME=/tmp/codex-auth-login-fix/codex-home ZIG_GLOBAL_CACHE_DIR=/tmp/codex-auth-login-fix/zig-global ZIG_LOCAL_CACHE_DIR=/tmp/codex-auth-login-fix/zig-local zig build --build-file /coding/codex-auth/build.zig run -- list
  • HOME=/tmp/codex-auth-login-fix CODEX_HOME=/tmp/codex-auth-login-fix/codex-home ZIG_GLOBAL_CACHE_DIR=/tmp/codex-auth-login-fix/zig-global ZIG_LOCAL_CACHE_DIR=/tmp/codex-auth-login-fix/zig-local zig build --build-file /coding/codex-auth/build.zig test

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 22, 2026

Open in StackBlitz

@loongphy/codex-auth-darwin-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-darwin-arm64@a0bc998

@loongphy/codex-auth-darwin-x64

npx https://pkg.pr.new/@loongphy/codex-auth-darwin-x64@a0bc998

@loongphy/codex-auth-linux-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-linux-arm64@a0bc998

@loongphy/codex-auth-linux-x64

npx https://pkg.pr.new/@loongphy/codex-auth-linux-x64@a0bc998

@loongphy/codex-auth-win32-arm64

npx https://pkg.pr.new/@loongphy/codex-auth-win32-arm64@a0bc998

@loongphy/codex-auth-win32-x64

npx https://pkg.pr.new/@loongphy/codex-auth-win32-x64@a0bc998

@loongphy/codex-auth

npx https://pkg.pr.new/@loongphy/codex-auth@a0bc998

commit: a0bc998

@Loongphy Loongphy merged commit 1fe8641 into main May 22, 2026
12 checks passed
@Loongphy Loongphy deleted the codex/isolate-login-codex-home branch May 22, 2026 07:20
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.

1 participant