Skip to content

Fix transfer-desktop-to-sandbox for Helix-in-Helix mode#1585

Merged
lukemarsden merged 1 commit intomainfrom
fix/helix-in-helix-sandbox-transfer
Feb 5, 2026
Merged

Fix transfer-desktop-to-sandbox for Helix-in-Helix mode#1585
lukemarsden merged 1 commit intomainfrom
fix/helix-in-helix-sandbox-transfer

Conversation

@lukemarsden
Copy link
Collaborator

Summary

Fix image transfer to sandbox in Helix-in-Helix mode where the sandbox runs on the host Docker socket with a different container name.

Problem

In Helix-in-Helix mode:

  • The sandbox container runs on the host Docker (DOCKER_HOST_OUTER) not the inner Docker
  • The container is named helix-inner-sandbox-${USER} not helix-sandbox-nvidia-1
  • The transfer-desktop-to-sandbox function was using the wrong Docker socket and container name

This caused the error:

Error response from daemon: No such container: helix-sandbox-nvidia-1
❌ Failed to pull from registry inside sandbox

Changes

  • Add sandbox_docker() helper function to use correct Docker host in Helix-in-Helix mode
  • Use SANDBOX_CONTAINER_NAME variable for the correct container name
  • Check for sandbox existence on host Docker when in Helix-in-Helix mode
  • Use sandbox_docker for all docker exec commands targeting sandbox

Test plan

  • Run ./stack build-ubuntu in Helix-in-Helix mode
  • Verify image transfer succeeds to outer sandbox
  • Start a new session and verify desktop image is available

🤖 Generated with Claude Code

In Helix-in-Helix mode, the sandbox runs on the host Docker socket
with a different container name (helix-inner-sandbox-${USER}).

Changes:
- Add sandbox_docker() helper function to use correct Docker host
- Use SANDBOX_CONTAINER_NAME variable for the correct container name
- Check for sandbox existence on host Docker in Helix-in-Helix mode
- Use sandbox_docker for all docker exec commands targeting sandbox

This fixes "No such container: helix-sandbox-nvidia-1" errors when
running ./stack build-ubuntu or build-sway in Helix-in-Helix mode.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@lukemarsden lukemarsden merged commit 7643c7e into main Feb 5, 2026
9 checks passed
@lukemarsden lukemarsden deleted the fix/helix-in-helix-sandbox-transfer branch February 5, 2026 17:14
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