Skip to content

fix: handle unavailable hermes preflight#13

Merged
Dankosik merged 1 commit into
mainfrom
codex/hermes-unavailable-preflight
Jun 1, 2026
Merged

fix: handle unavailable hermes preflight#13
Dankosik merged 1 commit into
mainfrom
codex/hermes-unavailable-preflight

Conversation

@Dankosik
Copy link
Copy Markdown
Contributor

@Dankosik Dankosik commented Jun 1, 2026

Summary

  • add timeout-aware handling for Hermes CLI calls
  • normalize missing, non-zero, and timed-out hermes --version preflight failures into domain failures
  • cover timeout/signal paths and prevent raw Command failed: hermes --version output

Test plan

  • npm run ci
  • local TTY smoke test against an unavailable hermes --version

@Dankosik Dankosik merged commit b1bce42 into main Jun 1, 2026
2 checks passed
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.

1 participant