hermes - 💡(How to fix) Fix [Bug]: auxiliary_client.py uses DEFAULT_CONFIG without importing it [2 pull requests]

Official PRs (…)
ON THIS PAGE

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Error Message

Compression should read auxiliary config from DEFAULT_CONFIG.get("auxiliary", {}) without error, and summary generation should proceed normally.

Fix Action

Fixed

Code Example

from hermes_cli.config import get_hermes_home

---

⚠️ Summary generation failed (name 'DEFAULT_CONFIG' is not defined).

---

from hermes_cli.config import DEFAULT_CONFIG, get_hermes_home
RAW_BUFFERClick to expand / collapse

Bug Description

agent/auxiliary_client.py references DEFAULT_CONFIG.get("auxiliary", {}) at lines 3874 and 4249, but the import statement on line 103 only pulls in get_hermes_home:

from hermes_cli.config import get_hermes_home

DEFAULT_CONFIG is never imported, causing a NameError: name "DEFAULT_CONFIG" is not defined whenever compression calls call_llm() — which triggers during context compaction. This produces the warning:

⚠️ Summary generation failed (name 'DEFAULT_CONFIG' is not defined).

Steps to Reproduce

  1. Run Hermes Agent with a model that has limited context (e.g., 65K) and compression enabled.
  2. Trigger context compaction by exceeding the compression threshold.
  3. Observe the warning in logs: Summary generation failed (name 'DEFAULT_CONFIG' is not defined).

Expected Behavior

Compression should read auxiliary config from DEFAULT_CONFIG.get("auxiliary", {}) without error, and summary generation should proceed normally.

Actual Behavior

A NameError is raised when the compression code path attempts to access DEFAULT_CONFIG, causing summary generation to fail with a warning. This affects title generation and context compaction for any model whose context window triggers the auxiliary LLM call during compression.

Local Fix

Add DEFAULT_CONFIG to the import on line 103 of agent/auxiliary_client.py:

from hermes_cli.config import DEFAULT_CONFIG, get_hermes_home

DEFAULT_CONFIG is defined in hermes_cli/config.py at line 395 and exported from that module.

Environment

  • Repo: NousResearch/hermes-agent
  • Commit: fef1a4124 (HEAD of main)
  • File: agent/auxiliary_client.py, lines 103, 3874, 4249

Notes

Git history confirms this has been a missing import since the file was refactored — line 103 has only ever imported get_hermes_home from hermes_cli.config. The usages of DEFAULT_CONFIG at lines 3874 and 4249 were added without updating the import. This is the same class of bug as #8173 (RedactingFormatter missing import in gateway/run.py).

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

hermes - 💡(How to fix) Fix [Bug]: auxiliary_client.py uses DEFAULT_CONFIG without importing it [2 pull requests]