Skip to content

fix: share Codex auth refresh across profiles - #43415

Closed
ablott976 wants to merge 1 commit into
NousResearch:mainfrom
ablott976:fix/shared-codex-auth
Closed

fix: share Codex auth refresh across profiles#43415
ablott976 wants to merge 1 commit into
NousResearch:mainfrom
ablott976:fix/shared-codex-auth

Conversation

@ablott976

Copy link
Copy Markdown

Summary

  • Let profile-isolated Hermes processes borrow root/global OpenAI Codex auth when no local Codex provider state exists
  • Refresh borrowed Codex tokens back into the owning/root auth store under that store's lock instead of cloning rotated tokens into profile auth.json
  • Extend Codex pool fallback to shared/root pool-only entries and expose auth_scope/auth_store in Codex status
  • Document the feature in funciones.txt

Test Plan

  • python3 -m py_compile hermes_cli/auth.py
  • python3 -m pytest tests/hermes_cli/test_auth_commands.py -q -o 'addopts='

Notes

  • Existing profiles with explicit local providers.openai-codex still shadow shared auth by design
  • Set HERMES_CODEX_SHARED_AUTH=0 to force strict per-profile Codex auth

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/cli CLI entry point, hermes_cli/, setup wizard area/auth Authentication, OAuth, credential pools provider/openai OpenAI / Codex Responses API duplicate This issue or pull request already exists labels Jun 10, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #28277 (earliest open PR for sharing Codex OAuth across profiles, addressing #6653). Competing with #34121 and #39796 in the same saturated cluster. Note: this PR bundles a funciones.txt doc artifact that looks fork-specific. Marking duplicate; maintainers can select the canonical implementation.

@liuhao1024

Copy link
Copy Markdown
Contributor

Accidental commit: funciones.txt in repo root

The diff adds a funciones.txt file at the repository root containing internal implementation notes in Spanish (OpenViking memory controls, Telegram attachment delivery, shared Codex auth). This appears to be a local notes file that was accidentally included in the commit.

funciones.txt | 17 +++++++++++++++++

This file should be removed before merge — it doesn't belong in the repo root and contains no runnable code.

@ablott976

Copy link
Copy Markdown
Author

Closing to avoid unintended upstream notifications. Will continue in fork/local branch before re-opening if needed.

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

Labels

area/auth Authentication, OAuth, credential pools comp/cli CLI entry point, hermes_cli/, setup wizard duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists provider/openai OpenAI / Codex Responses API type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants