nicklaw5 / helix / 24290219257

93%

push

github

fix: Resolve infinite token refresh loop on missing scope 401s (#257)

* Initial plan

* Fix infinite token refresh loop when scope is missing

When a 401 response is due to missing scopes (e.g., "Missing scope:
moderator:read:followers"), the token refresh would not resolve the
issue since refreshing preserves the same scopes. Previously, this
caused an infinite loop of refresh-retry cycles.

Changes:
- Parse the 401 error response body before attempting token refresh
- Skip token refresh when error message indicates missing scopes
- Limit token refresh to one attempt per request to prevent infinite
  loops for any persistent 401 error
- Add tests for both missing scope and persistent 401 scenarios

Agent-Logs-Url: https://github.com/nicklaw5/helix/sessions/acc538d2-8063-4c51-a449-a200add24a75

Co-authored-by: nicklaw5 <7908386+nicklaw5@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nicklaw5 <7908386+nicklaw5@users.noreply.github.com>

7 of 10 new or added lines in 1 file covered. (70.0%)

1821 of 1957 relevant lines covered (93.05%)

5601.46 hits per line