NousResearch/hermes-agent

Auth errors incorrectly marked retryable in message classifier

Cluster 7026. Data as of

Nothing open on GitHub. Fix merged in #7027 on 10 Apr 2026. Check it covers your case.

The short answer

0open issues
0open PRs
3items grouped
Known?
Yes. 3 related items grouped here, 1 issue already closed.
Fix event
A fix was merged to main in PR #7027. Check it covers your case. No release data is tracked.
Tracker
The maintainers' tracker marks this resolved, deciding PR #7027. That is separate from the fix event.
Still open
Nothing open on GitHub.

These issues and PRs address a bug where message-based authentication errors are classified with retryable=True in _classify_by_message(), causing unintended retry loops. The fixes set retryable=False for these auth error cases.

Open issues0

No open issues.

Open PRs0

No open pull requests.

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

  • #7027 fix: set retryable=False for message-based auth errors in _classify_by_message()

    merged to main

    files: agent/error_classifier.py

    Show the diff
    • agent/error_classifier.py
      @@ -725,10 +725,14 @@ def _classify_by_message(
               )
       
           # Auth patterns
      +    # Auth errors should NOT be retried directly — the credential is invalid and
      +    # retrying with the same key will always fail.  Set retryable=False so the
      +    # caller triggers credential rotation (should_rotate_credential=True) or
      +    # provider fallback rather than an immediate retry loop.
           if any(p in error_msg for p in _AUTH_PATTERNS):
               return result_fn(
                   FailoverReason.auth,
      -            retryable=True,
      +            retryable=False,
                   should_rotate_credential=True,
               )
       

1 PR closed without merging

From the close comments: superseded by another PR 1

  • #7033 fix(agent): set retryable=False for auth errors in _classify_by_message · superseded by another PR → #7027
Closed issues · 1
  • Closed#7026bug: message-based auth errors use retryable=True in _classify_by_message() causing retry loops
How this cluster was grouped
  • Link evidence (structural links throughout): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
  • Pairwise link density 0.67: the share of item 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.

Pick up a Lab task

Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.

Open the Lab

At a glance

Merged PRs
#7027
Closed issues
1
Attempts
1 PR closed without merging
Fix link checked
10 Apr 2026. This confirmation is old and may be out of date (a revert or reopen would not have invalidated it here); re-verify.

For your agent

The same facts, typed.