fix(sandbox): sanitize @ in container names for WhatsApp LID chat IDs#1031
Open
kamushadenes wants to merge 1 commit into
Open
fix(sandbox): sanitize @ in container names for WhatsApp LID chat IDs#1031kamushadenes wants to merge 1 commit into
kamushadenes wants to merge 1 commit into
Conversation
WhatsApp LID-format chat IDs contain @ (e.g. 551152861098:5@s.whatsapp.net) which is invalid in Docker container names, causing sandbox creation to fail for any WhatsApp-triggered agent session. Add @ to the sanitizeKey replacer alongside the existing : / . and space characters. Adds a test case with a realistic WhatsApp LID key. Fixes nextlevelbuilder#1029 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
5 tasks
kamushadenes
added a commit
to kamushadenes/nix
that referenced
this pull request
Apr 24, 2026
Re-enable the Docker sandbox overlay (docker-compose.sandbox.yml) now that the @ container naming bug has a fix (kamushadenes/goclaw fork, upstream PR nextlevelbuilder/goclaw#1031). - Custom sandbox image (goclaw-sandbox:custom) extends bookworm-slim with pre-installed CLIs: gh, gcloud, vt, fleetctl - Override GOCLAW_SANDBOX_NETWORK=true so sandboxed CLIs can reach external APIs (GitHub, Google Cloud, VirusTotal, Fleet) - ExecStartPre builds the custom image on first boot (cached after) - Remove the manual docker-socket overlay — sandbox.yml already mounts it - Point goclaw-repo to kamushadenes fork until #1031 merges upstream Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
kamushadenes
added a commit
to kamushadenes/goclaw
that referenced
this pull request
Apr 24, 2026
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
1 participant
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
sanitizeKey()ininternal/sandbox/docker.goreplaces:,/,.with-but misses@. WhatsApp LID-format chat IDs like551152861098:5@s.whatsapp.netproduce container names containing@, which Docker rejects as invalid."@", "-"to thestrings.NewReplacercallTest plan
go test ./internal/sandbox/ -run TestSanitizeKeypassesFixes #1029