Skip to content

fix(cli): non-blocking startup update check and banner deduplication - #1386

Merged
teknium1 merged 2 commits into
mainfrom
hermes/hermes-7ef7cb6a
Mar 15, 2026
Merged

fix(cli): non-blocking startup update check and banner deduplication#1386
teknium1 merged 2 commits into
mainfrom
hermes/hermes-7ef7cb6a

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

  • cherry-pick the substantive fix from fix(cli): non-blocking startup update check, version consistency, and banner deduplication #1266 so CLI startup no longer blocks on the update check
  • deduplicate the welcome banner implementation so hermes_cli.banner.build_welcome_banner() is the single source of truth again
  • restore update-check behavior for dev/worktree installs and show update status in hermes version
  • bring over the contributor's regression tests for update-check caching, fallback behavior, and background prefetching

Test plan

  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m pytest tests/hermes_cli/test_update_check.py tests/test_cli_preloaded_skills.py -n0 -q
  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m pytest tests/ -n0 -q
  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m hermes_cli.main version

Salvages #1266 with authorship preserved for the main fix.

teknium1 and others added 2 commits March 14, 2026 21:45
- Add background thread mechanism (prefetch_update_check/get_update_result)
  so git fetch runs in parallel with skill sync and agent init
- Fix repo path fallback in check_for_updates() for dev installs
- Remove duplicate build_welcome_banner (~180 lines) and
  _format_context_length from cli.py — the banner.py version is
  now the single source of truth
- Port skin banner_hero/banner_logo support and terminal width check
  from cli.py's version into banner.py
- Add update status output to hermes version command
- Add unit tests for update check, prefetch, and version string
@teknium1
teknium1 merged commit 9e0f86c into main Mar 15, 2026
1 check passed
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
…7ef7cb6a

fix(cli): non-blocking startup update check and banner deduplication
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…7ef7cb6a

fix(cli): non-blocking startup update check and banner deduplication
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
…7ef7cb6a

fix(cli): non-blocking startup update check and banner deduplication
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.

2 participants