mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-05-27 06:11:40 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

fix(banner): resolve update-check repo from running code, not profile-scoped path

Browse source 
check_for_updates() and _resolve_repo_dir() were preferring
$HERMES_HOME/hermes-agent/ over Path(__file__).parent.parent.resolve()
when looking for a .git checkout.  For profiles created with
--clone-all, $HERMES_HOME/hermes-agent/ points to a stale copy
with a frozen HEAD, causing persistent "N commits behind" banners
that never resolved.

Flip the resolution order: prefer the running code's location first,
fall back to $HERMES_HOME/hermes-agent/ only when the live checkout
doesn't have a .git (system-wide pip installs, distro packages).

The embedded-rev branch (HERMES_REVISION env var, set by nix builds)
is unaffected — it uses git ls-remote against upstream, never reads
the local checkout's HEAD.

Based on PR #21728 by @fahdad
This commit is contained in:
fahdad 2026-05-08 00:01:58 -07:00 • committed by kshitij
parent f7e514d4ad
commit cca2869d78
1 changed files with 14 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

20
hermes_cli/banner.py
View file

@ -206,9 +206,12 @@ def check_for_updates() -> Optional[int]:
if embedded_rev: if embedded_rev:
behind = _check_via_rev(embedded_rev) behind = _check_via_rev(embedded_rev)
else: else:
repo_dir = hermes_home / "hermes-agent" # Prefer the running code's location over the profile-scoped path.
# $HERMES_HOME/hermes-agent/ may be a stale copy from --clone-all;
# Path(__file__) always resolves to the actual installed checkout.
repo_dir = Path(__file__).parent.parent.resolve()
if not (repo_dir / ".git").exists(): if not (repo_dir / ".git").exists():
repo_dir = Path(__file__).parent.parent.resolve() repo_dir = hermes_home / "hermes-agent"
if not (repo_dir / ".git").exists(): if not (repo_dir / ".git").exists():
return None return None
behind = _check_via_local_git(repo_dir) behind = _check_via_local_git(repo_dir)
@ -222,11 +225,16 @@ def check_for_updates() -> Optional[int]:
def _resolve_repo_dir() -> Optional[Path]: def _resolve_repo_dir() -> Optional[Path]:
"""Return the active Hermes git checkout, or None if this isn't a git install.""" """Return the active Hermes git checkout, or None if this isn't a git install.
hermes_home = get_hermes_home()
repo_dir = hermes_home / "hermes-agent" Prefers the running code's location over the profile-scoped path
because ``$HERMES_HOME/hermes-agent/`` may be a stale copy carried
over by ``--clone-all``.
"""
repo_dir = Path(__file__).parent.parent.resolve()
if not (repo_dir / ".git").exists(): if not (repo_dir / ".git").exists():
repo_dir = Path(__file__).parent.parent.resolve() hermes_home = get_hermes_home()
repo_dir = hermes_home / "hermes-agent"
return repo_dir if (repo_dir / ".git").exists() else None return repo_dir if (repo_dir / ".git").exists() else None