Skip to content

feat: add CODEX_HOME support#179

Merged
SaladDay merged 2 commits into
SaladDay:mainfrom
unive3sal:main
May 15, 2026
Merged

feat: add CODEX_HOME support#179
SaladDay merged 2 commits into
SaladDay:mainfrom
unive3sal:main

Conversation

@unive3sal
Copy link
Copy Markdown
Collaborator

@unive3sal unive3sal commented May 12, 2026

add CODEX_HOME support for codex.

@unive3sal unive3sal requested a review from SaladDay May 12, 2026 13:42
@SaladDay
Copy link
Copy Markdown
Owner

SaladDay commented May 13, 2026

看了一下这个 PR,谢谢贡献。

我觉得这里需要先明确一点:当前实现实际支持的是 CODEX_HOME,还不是真正的 XDG Base Directory support。代码现在会在 get_codex_config_dir() 里优先读取 CODEX_HOME,然后才 fallback 到 cc-switch 的 codex_config_dir 设置和默认 ~/.codex

这能解决一部分场景:如果用户通过 CODEX_HOME 把 Codex home 放到了非默认目录,cc-switch 也能跟着读写同一个目录。对 repo-level / isolated Codex home 这类高级用法是有价值的。

但如果目标是解决 XDG 目录问题,比如 $XDG_CONFIG_HOME/codex,这个 PR 还没有覆盖。

建议二选一:

  1. 如果只做当前能力,把标题/描述改成类似 “respect CODEX_HOME for Codex config dir”,并在 UI/文档里说明优先级:CODEX_HOME > cc-switch settings > ~/.codex

  2. 如果确实要做 XDG support,需要额外实现 $XDG_CONFIG_HOME/codex 的解析和 fallback 规则。

还有一个细节:OpenAI Codex 自己的 find_codex_home() 会要求 CODEX_HOME 存在且是目录,并规范化;当前 PR 只要环境变量非空就直接使用。这个不一定阻塞,但如果想和 Codex runtime 行为更一致,可以考虑对无效目录给出更清晰的错误。

@unive3sal
Copy link
Copy Markdown
Collaborator Author

OK,我有空改一下标题和具体的处理细节,和 codex 的处理保持一致

@unive3sal
Copy link
Copy Markdown
Collaborator Author

我修改了一版

@unive3sal unive3sal changed the title feat: add codex XDG support feat: add CODEX_HOME support May 14, 2026
@SaladDay SaladDay merged commit d3c240c into SaladDay:main May 15, 2026
2 checks passed
@SaladDay
Copy link
Copy Markdown
Owner

LGTM, THANKS!

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.

2 participants