Skip to content

fix(auth): guard shared Codex store consumers - #4

Open
cooper-oai wants to merge 4 commits into
fix/codex-oauth-refresh-ownership-4-linked-aliasesfrom
fix/codex-oauth-refresh-ownership-5-consumer-guards
Open

fix(auth): guard shared Codex store consumers#4
cooper-oai wants to merge 4 commits into
fix/codex-oauth-refresh-ownership-4-linked-aliasesfrom
fix/codex-oauth-refresh-ownership-5-consumer-guards

Conversation

@cooper-oai

@cooper-oai cooper-oai commented May 29, 2026

Copy link
Copy Markdown
Owner

Summary

Part 5 of 8. Depends on part 4.

  • protect media delivery and model-cache consumers of the shared Codex store
  • fail closed for pre-upgrade unclaimed refresh families
  • remove refresh-linked aliases when the canonical family is removed
  • sanitize merged pool payloads and prune borrowed stale rows

Why

All consumers must observe the canonical Hermes-owned state, and ambiguous or borrowed token rows must not reach refresh or reappear through merge paths.

This is a fork-local stacked draft. Promote it upstream after part 4 lands.

Validation

  • focused suite at this layer: 433 passed, 2 skipped
  • ruff, py_compile, Windows footgun scan, and git diff --check: clean

Stack

Priority order, highest to lowest:

  1. NousResearch/hermes-agent#34820 — prevent Hermes Codex OAuth refresh-token reuse
  2. cooper-oai/hermes-agent#1 — share canonical Codex state across profiles
  3. cooper-oai/hermes-agent#2 — reconcile stale Codex pool snapshots
  4. cooper-oai/hermes-agent#3 — migrate linked Codex aliases safely
  5. cooper-oai/hermes-agent#4 — guard shared Codex store consumers
  6. cooper-oai/hermes-agent#5 — quarantine Codex families by composite identity
  7. cooper-oai/hermes-agent#6 — harden Codex migration and fallback routing
  8. cooper-oai/hermes-agent#7 — avoid Codex refresh contention and fix guidance

Part 1 targets upstream main. Parts 2-8 are fork-local dependent drafts so each diff stays reviewable; promote each part upstream after the part below it lands.

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