# NousResearch/hermes-agent · OAuth refresh token rotation race condition fix

## Open now

- Nothing open on GitHub.

These items address a bug where a stale credential pool could overwrite a newer rotated refresh token with a spent one, causing lost logins. The work involves binding credential verdicts to token generation to prevent stale processes from undoing a valid token rotation.

## TL;DR

- 4 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 2 PR(s) closed without merging.
- A fix was merged to main in PR #121257. Verify it addresses your case.
- **Recommended:** read PR #121257 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `resolved` · deciding PR(s): `pr-121257`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.67`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Closed / duplicate issues

- #120815: \[Bug\]: a stale credential pool writes a spent OAuth refresh token over a newer rotation and the login is lost

## Prior & competing PRs

- **#121257: Stale processes can no longer undo a credential-pool token rotation (#120815, salvage #120943)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `agent/credential_pool.py`, `agent/credential_pool_admin.py`, `hermes_cli/auth.py`, `tests/agent/test_credential_pool_anthropic_refresh_race.py`, `tests/agent/test_credential_pool_profile_oauth_fork.py`, `tests/agent/test_credential_pool_terminal_refresh_visibility.py`, `website/docs/user-guide/features/credential-pools.md`
- **#120816: fix(auth): a stale credential pool never writes a spent refresh token over a newer rotation**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `f847ef8cf7`, `214c01f013`, `f24a1d7f92`, `#120815`, `#121257`
- **#120943: fix(auth): bind credential verdicts to token generation**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `95ef81cea0`, `214c01f013`, `f24a1d7f92`, `#121257`

## Fix status

- Merged to main in PR #121257 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-09-24`.