{"repo":"NousResearch/hermes-agent","public_id":"7026","display_label":"Auth errors incorrectly marked retryable in message classifier","summary":"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.","subsystem":null,"counts":{"members":3,"open_issues":0,"open_issues_distinct":0,"open_prs":0,"closed_issues":1,"merged_prs":1,"attempt_prs":1},"landed_fix":{"status":"merged_to_main","basis":"structural","pr":7027,"merged_at":"2026-04-10T09:48:46+00:00"},"resolution":{"state":"resolved","by":"pr-7027"},"freshness":{"state":"stale","last_verified_at":"2026-04-10T09:48:46+00:00","is_stale":true},"grouping":{"backbone":"STRUCTURAL_SPANNING","link_density":0.6666666666666666},"fix_edges":[{"issue_number":7026,"pr_number":7027,"tier":"verified","basis":"closing_ref","confidence":0.95,"recoverable_from_timeline":true,"pr_merged_at":"2026-04-10T09:48:46+00:00"}],"members":[{"kind":"issue","number":7026,"title":"bug: message-based auth errors use retryable=True in _classify_by_message() causing retry loops","url":null,"state":"closed"},{"kind":"pr","number":7027,"title":"fix: set retryable=False for message-based auth errors in _classify_by_message()","url":null,"state":"merged","close_reason":{"category":"merged"},"merged":true,"is_fix_pr":true,"fix_tier":"verified","files_changed":["agent/error_classifier.py"],"fix_hunks":[{"file":"agent/error_classifier.py","hunks":["@@ -725,10 +725,14 @@ def _classify_by_message(\n         )\n \n     # Auth patterns\n+    # Auth errors should NOT be retried directly — the credential is invalid and\n+    # retrying with the same key will always fail.  Set retryable=False so the\n+    # caller triggers credential rotation (should_rotate_credential=True) or\n+    # provider fallback rather than an immediate retry loop.\n     if any(p in error_msg for p in _AUTH_PATTERNS):\n         return result_fn(\n             FailoverReason.auth,\n-            retryable=True,\n+            retryable=False,\n             should_rotate_credential=True,\n         )\n \n"]}]},{"kind":"pr","number":7033,"title":"fix(agent): set retryable=False for auth errors in _classify_by_message","url":null,"state":"closed","close_reason":{"refs":[{"kind":"pr","value":"#7027","number":7027}],"category":"superseded_by_pr","implication":"superseded"},"merged":false}],"trust":{"default_tier":"verified","includes_suggested":false}}