Skip to content

fix(hindsight): add missing get_hermes_home import#6093

Closed
Archerouyang wants to merge 1 commit into
NousResearch:mainfrom
Archerouyang:fix/hindsight-import
Closed

fix(hindsight): add missing get_hermes_home import#6093
Archerouyang wants to merge 1 commit into
NousResearch:mainfrom
Archerouyang:fix/hindsight-import

Conversation

@Archerouyang

Copy link
Copy Markdown
Contributor

Problem

get_hermes_home was only imported inside _load_config(), but _start_daemon() (called when mode=local) also references it, causing:

NameError: get_hermes_home is not defined

Fix

  • Move the import to module level so it is available everywhere in the file
  • Remove the now-redundant import from _load_config()

Verification

  • python -m py_compile plugins/memory/hindsight/__init__.py passes
  • The local mode daemon start path no longer references an undefined name

Fixes #5993

Import hermes_constants.get_hermes_home at module level so it is
available in _start_daemon() when local mode starts the embedded
daemon. Previously the import was only inside _load_config(), causing
NameError when _start_daemon() referenced get_hermes_home().

Fixes NousResearch#5993

Co-Authored-By: 史官 <historian@slock.team>
@teknium1

teknium1 commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Merged via PR #6098. Your commit was cherry-picked onto current main with your authorship preserved in git log. Thanks @Archerouyang!

@teknium1 teknium1 closed this Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: ~/.hermes/hermes-agent/plugins/memory/hindsight/-init-.py

3 participants