Skip to content

fix: start Docker daemon after install in sandbox mode#3129

Merged
AhmedTMM merged 1 commit intomainfrom
fix/docker-daemon-start
Apr 1, 2026
Merged

fix: start Docker daemon after install in sandbox mode#3129
AhmedTMM merged 1 commit intomainfrom
fix/docker-daemon-start

Conversation

@la14-1
Copy link
Copy Markdown
Member

@la14-1 la14-1 commented Apr 1, 2026

Summary

  • After brew install orbstack on macOS, runs open -a OrbStack to launch the app
  • After apt-get install docker.io on Linux, runs sudo systemctl start docker
  • Waits up to 30s polling docker info for the daemon to be ready
  • Clear error message with guidance if it times out

Fixes the issue where --beta sandbox installs OrbStack/Docker successfully but then fails with "Docker installed but not responding" because the daemon was never started.

Test plan

  • Biome lint passes
  • Full test suite passes (1962 tests)
  • Manual: spawn claude local --beta sandbox on macOS without Docker — should install OrbStack, launch it, wait for daemon, then proceed

🤖 Generated with Claude Code

The sandbox mode now starts the Docker daemon whenever it's not running,
not only after a fresh install. This handles the common case where
OrbStack/Docker is installed but the daemon isn't started yet.

Flow: check daemon → if down, check binary → if missing, install →
start daemon (open -a OrbStack / systemctl start docker) → poll up to 30s

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@la14-1 la14-1 force-pushed the fix/docker-daemon-start branch from b699ea0 to bb0dd36 Compare April 1, 2026 00:46
Copy link
Copy Markdown
Collaborator

@AhmedTMM AhmedTMM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AhmedTMM AhmedTMM merged commit 426ebc9 into main Apr 1, 2026
5 checks passed
@AhmedTMM AhmedTMM deleted the fix/docker-daemon-start branch April 1, 2026 00:50
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