Skip to content

fix: clearer docker backend preflight errors#1276

Merged
teknium1 merged 4 commits intomainfrom
hermes/hermes-3702edad
Mar 14, 2026
Merged

fix: clearer docker backend preflight errors#1276
teknium1 merged 4 commits intomainfrom
hermes/hermes-3702edad

Conversation

@teknium1
Copy link
Copy Markdown
Contributor

@teknium1 teknium1 commented Mar 14, 2026

Summary

  • salvage the preflight Docker availability check from fix: clearer error when docker backend is unavailable #940 onto current main
  • preserve current find_docker() fallback behavior so Docker Desktop installs outside PATH still work
  • add a regression test proving the preflight uses the resolved docker executable path

Test plan

  • python -m pytest tests/tools/test_docker_environment.py tests/tools/test_docker_find.py -n0 -q
  • python -m pytest tests/ -n0 -q
  • python -m hermes_cli.main chat -q 'Respond with ok.'

Refs #940
teknium1 and others added 4 commits March 14, 2026 02:30
@teknium1
feat: improve context compaction handoff summaries
9874287 
Adapt PR #916 onto current main by replacing the old context summary marker
with a clearer handoff wrapper, updating the summarization prompt for
resume-oriented summaries, and preserving the current call_llm-based
compression path.
@teknium1
Merge remote-tracking branch 'origin/main' into hermes/hermes-3702edad
d1f2a34
@aydnOktay @teknium1
fix: clearer error when docker backend is unavailable
5978665
@teknium1
fix: preserve docker discovery in backend preflight
31cf535 
Follow up on salvaged PR #940 by reusing find_docker() during the new
availability check so non-PATH Docker Desktop installs still work. Add
a regression test covering the resolved executable path.
@teknium1 teknium1 merged commit 6036793 into main Mar 14, 2026
0 of 2 checks passed
@teknium1 teknium1 mentioned this pull request Mar 14, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@teknium1 @aydnOktay