Skip to content

fix: exclude bash backend from hex package#49

Merged
mikehostetler merged 4 commits into
mainfrom
fix/issue-48-required-bash
Jun 10, 2026
Merged

fix: exclude bash backend from hex package#49
mikehostetler merged 4 commits into
mainfrom
fix/issue-48-required-bash

Conversation

@mikehostetler

@mikehostetler mikehostetler commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Keep the Bash backend source unguarded in source/Git builds, avoiding guarded Bash.Interop compatibility code.
  • Keep the pinned upstream tv-labs/bash Git dependency for dev/test so the backend can compile against Bash.Session.signal/3 before the next Bash Hex release.
  • Exclude lib/jido_shell/backend/bash.ex and lib/jido_shell/backend/bash/* from the Hex package until upstream bash publishes the required API.
  • Document the temporary source/Git-only Bash backend availability while waiting for the upstream release.

Closes #48.

Validation

  • mix compile --warnings-as-errors
  • mix test (622 passed, 3 excluded)
  • mix hex.build --unpack -o /tmp/jido_shell_hex and confirmed Bash backend files are excluded
  • MIX_ENV=prod mix compile --warnings-as-errors from the unpacked package
  • mix hex.build

@mikehostetler mikehostetler changed the title fix: require bash dependency for backend compile fix: guard optional bash backend dependency Jun 10, 2026
@mikehostetler mikehostetler changed the title fix: guard optional bash backend dependency fix: exclude bash backend from hex package Jun 10, 2026
@mikehostetler mikehostetler merged commit 4a2e232 into main Jun 10, 2026
18 checks passed
@mikehostetler mikehostetler deleted the fix/issue-48-required-bash branch June 10, 2026 13:28
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.

CI compile fail in dependabot jido_harness PR: Bash.Interop module missing from optional dependency

1 participant