Skip to content

fix: recognize Codex credential-pool auth#617

Merged
EKKOLearnAI merged 1 commit into
EKKOLearnAI:mainfrom
hanzckernel:fix/codex-credential-pool-models
May 11, 2026
Merged

fix: recognize Codex credential-pool auth#617
EKKOLearnAI merged 1 commit into
EKKOLearnAI:mainfrom
hanzckernel:fix/codex-credential-pool-models

Conversation

@hanzckernel
Copy link
Copy Markdown
Collaborator

Codex 的模型可用性和登录状态现在会兼容 Hermes auth.json 里的 credential_pool.openai-codex。旧的 providers.openai-codex.tokens 仍保留;这个 PR 不改 Hermes Agent 的写入格式。

改动

  • available-models 的 OAuth provider 授权判断同时读取 legacy providers 和新的 credential_pool
  • Codex auth status 从 credential_pool.openai-codex 解析 access / refresh token;刷新 token 时同步更新已命中的 legacy provider 和 pool entry。
  • 新增回归测试覆盖 providers 为空、credential_pool.openai-codex 有 token 时仍能列出 Codex models、并返回 authenticated。

验证

已通过:

npm test -- tests/server/codex-credential-pool-auth.test.ts tests/shared/provider-presets.test.ts tests/server/copilot-auth-controller.test.ts
npm run build

结果:15 tests passed;build passed。

已知:

  • npm test 全量目前卡在未改动的 tests/server/session-sync.test.ts,报 no such table: sessions
  • 单独运行 npm test -- tests/server/session-sync.test.ts 可复现同一个失败,和这次改动无关。

@EKKOLearnAI EKKOLearnAI merged commit 5e608ea into EKKOLearnAI:main May 11, 2026
1 check passed
@hanzckernel hanzckernel deleted the fix/codex-credential-pool-models branch May 11, 2026 20:57
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