Rename sandbox setup to create and add sandbox update - #11
Merged
Merged
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
Removes the Test Sandbox and the cached docker/sandbox-templates Claude Code images, then recreates the sandbox so sbx pulls the latest template. Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
sbx template ls can list one image under several IDs. Removing the first ID removes them all, and the next rm fails with a 404 'no template image'. Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skillwalker sandbox setuptoskillwalker sandbox create. The old spelling now fails withUnknown argument. The hint strings in@testdouble/sandbox-integrationand the docs name the new command.skillwalker sandbox update. It deletes the Test Sandbox and recreates it from the latest Claude Code sandbox template.How
sandbox updategets the latest templatesbxhas nopullcommand, andsbx runreuses whatever template image it already has cached. The newupdateSandbox(repoRoot)works in three steps:sbx rm --force claude-skills-skillwalker, if the sandbox exists.sbx template lsand removes everydocker/sandbox-templatesimage whose tag starts withclaude-code, usingsbx template rm <image id>.createSandbox, sosbx runfetches the current template and runs the OAuth login again.sbx template lscan list one image under several IDs, and removing the first ID removes them all. A laterrmthat reportsno template imagetherefore counts as already removed. Any other failure stops the command with aSkillwalkerError.Notes for review
make sandbox-setuptarget keeps its name, to match the earlier consolidation decision. It now callssandbox create.sandbox updatehas not been run against realsbx. It is covered by unit tests with a mockedBun.spawn. Thesbx template lscolumn layout was checked againstsbxv0.29.0.Test plan
make test(77 files, 1016 tests passing)bun packages/cli/index.ts sandboxlistscreate,update,clean,shell./build/skillwalker sandbox updateagainst a real sandbox and confirm a fresh template is pulled🤖 Generated with Claude Code