fix: consistent HERMES_HOME and .env path resolution across all entry points - #51
Merged
teknium1 merged 4 commits intoFeb 26, 2026
Merged
Conversation
…py pattern Load ~/.hermes/.env first with project root as dev fallback, and remove redundant second load_dotenv call inside load_cli_config(). Also sets MSWEA_GLOBAL_CONFIG_DIR so mini-swe-agent shares the same config. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Both entry points hardcoded Path.home() / ".hermes" for .env, config.yaml, logs, and lock files. Now uses _hermes_home which reads HERMES_HOME env var with ~/.hermes as default, matching cli.py and run_agent.py. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Consistent with other entry points: use _hermes_home from HERMES_HOME env var, and add UTF-8 → latin-1 encoding fallback on load_dotenv. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Line 184 hardcoded Path.home() / ".hermes" instead of using the existing HERMES_HOME variable which already respects the env var. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
deankerr
marked this pull request as ready for review
February 26, 2026 08:26
This was referenced Mar 11, 2026
10 tasks
angelburgosrosado
pushed a commit
to angelburgosrosado/hermes-agent
that referenced
this pull request
Apr 27, 2026
…olution fix: consistent HERMES_HOME and .env path resolution across all entry points
Benriez
pushed a commit
to Benriez/hermes-agent
that referenced
this pull request
Jun 4, 2026
Records issue NousResearch#51 workflow lessons in reusable Kanban skills and Agent Garden adapter policy. Encodes: - parent orchestration cards are dependencies and must be completed after decomposition - review work must transition to review status, not ready - minimax-implementer must use the formal review path - sdlc-review availability gap is a known failure mode - direct SQLite writes are rescue-only and forbidden for normal workflow Adds issue NousResearch#51 learning examples with run 86/87/88/89 outcomes and REVIEW_PASS evidence.
dizhaky
referenced
this pull request
in dizhaky/hermes-agent
Jun 23, 2026
…AN-1385) (#26) baileys 7.0.0-rc.9->rc13 (clears #43 critical) + protobufjs 7.6.4 (#59,#58) in whatsapp-bridge; esbuild+tsx (#46,#61) in ui-tui; @babel/core (#84) in web; joi+http-proxy-middleware (#51,#89) in website. js-yaml #86 accepted (no in-range fix via gray-matter). Bridge startup verified. #93/#94 (code fixes) separate. Co-Authored-By: Claude <noreply@anthropic.com>
eriksilver
added a commit
to eriksilver/hermes-agent
that referenced
this pull request
Jun 29, 2026
Original scraper-suite PR NousResearch#9 (added both jeffco entries 2026-06-09) was parked the same week and never merged. Resurrection PR NousResearch#51 (2026-06-29) ships sheriff-only — tax-sale was dropped per Erik (low signal density: 345 leads/yr of mostly-rural 3+yr tax-delinquent parcels, mostly LLC owners, not worth the SMS reach-out). Drops the jefferson_tax_sale entry from the watch list + remediation hints; keeps jefferson_foreclosure (still expected daily, 36h tolerance). Atlas count back to 7. Also tightens the jefferson_foreclosure hint to note the lumpy-cadence reality (empty for weeks between 5-20 sale waves — not a bug).
waefrebeorn
pushed a commit
to waefrebeorn/slermes
that referenced
this pull request
Jul 2, 2026
…olution fix: consistent HERMES_HOME and .env path resolution across all entry points
Meraniya
pushed a commit
to Meraniya/hermes-agent
that referenced
this pull request
Aug 6, 2026
choi138
added a commit
to choi138/hermes-agent
that referenced
this pull request
Aug 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
~/.hermes/.envfirst (respectingHERMES_HOME), project.envas fallback. Removed redundant secondload_dotenvcall insideload_cli_config(). AddedMSWEA_GLOBAL_CONFIG_DIRsetup.Path.home() / ".hermes"with_hermes_home(.env,config.yaml, logs, prefill messages, personality/reasoning config).~/.hermespaths with_hermes_homefor.envreload,config.yaml, and lock directory.HERMES_HOME, add UTF-8 → latin-1 encoding fallback onload_dotenv.HERMES_HOMEconstant instead of hardcodedPath.home()in directory check.Test plan
py_compilesyntax check~/.hermes, customHERMES_HOMEoverride, derived pathshermes status/hermes doctorwith default configHERMES_HOME=/tmp/test-hermesto verify override🤖 Generated with Claude Code