Skip to content

fix: export the container default shell - #153

Open
BrettKinny wants to merge 1 commit into
mainfrom
agent/default-shell-env
Open

fix: export the container default shell#153
BrettKinny wants to merge 1 commit into
mainfrom
agent/default-shell-env

Conversation

@BrettKinny

Copy link
Copy Markdown
Collaborator

What changed

  • Export SHELL=/bin/bash in the image, matching the default command.
  • Add CI coverage that asserts the variable survives container startup.

Why

Some container-launched tools choose their child shell from SHELL. Docker does not guarantee that variable is exported when using docker exec, so the image could fall back to /bin/sh even though Squarebox's default command and shell configuration are Bash-based. That bypasses .bashrc and Starship initialization.

Validation

  • docker build -t squarebox:shell-env-test .
  • docker run --rm squarebox:shell-env-test env asserts SHELL=/bin/bash.
  • git diff --check

@BrettKinny
BrettKinny marked this pull request as ready for review August 5, 2026 10:15
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