Skip to content

Fix sandbox remapping and plugin version resolution#664

Merged
ahmedhesham6 merged 2 commits intomainfrom
fix/sandbox-user-mapping-plugin-resolution
Mar 26, 2026
Merged

Fix sandbox remapping and plugin version resolution#664
ahmedhesham6 merged 2 commits intomainfrom
fix/sandbox-user-mapping-plugin-resolution

Conversation

@ahmedhesham6
Copy link
Copy Markdown
Collaborator

@ahmedhesham6 ahmedhesham6 commented Mar 23, 2026

Description

Improve sandbox startup behavior for host-mounted files and make plugin resolution use stable, explicit versions.

Related Issues

None

Changes Made

  • add a container entrypoint + gosu flow so sandbox containers can remap to the host UID/GID without mapping root into the sandbox
  • update autopilot/systemd startup to handle docker-group access via sg and add quoting/round-trip test coverage
  • pin plugin downloads to resolved versions, prefer the server version for warden, and update plugin configs to pass the new resolution flag
  • treat local provider credentials as valid auth so onboarding does not run unnecessarily when provider keys are already configured

Testing

  • cargo fmt --all -- cli/src/commands/auto_update.rs cli/src/commands/autopilot/mod.rs cli/src/commands/board.rs cli/src/commands/browser.rs cli/src/commands/warden.rs cli/src/main.rs cli/src/utils/plugins.rs libs/server/src/sandbox.rs
  • cargo clippy -p stakpak -p stakpak-server --all-targets -- -D warnings
  • sh -n scripts/entrypoint.sh
  • Tested on Linux/macOS/Windows (specify which)

Screenshots (if applicable)

N/A

Breaking Changes

None

- remap sandbox containers through an entrypoint + gosu flow so bind-mounted files stay writable without mapping root into the sandbox\n- wrap systemd autopilot startup with sg docker when needed and add coverage for the quoting paths\n- pin resolved plugin download versions, prefer server versions for warden, and treat local provider credentials as valid auth for onboarding checks
@ahmedhesham6 ahmedhesham6 force-pushed the fix/sandbox-user-mapping-plugin-resolution branch from c95b2be to 0a32cac Compare March 26, 2026 16:39
@ahmedhesham6 ahmedhesham6 merged commit 0c2466a into main Mar 26, 2026
1 check passed
@ahmedhesham6 ahmedhesham6 deleted the fix/sandbox-user-mapping-plugin-resolution branch March 26, 2026 17:42
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