Skip to content

feat(settings): backport OS sandbox block + security deny hardening - #18

Merged
robercano merged 1 commit into
mainfrom
chore/backport-sandbox-hardening
Jun 30, 2026
Merged

feat(settings): backport OS sandbox block + security deny hardening#18
robercano merged 1 commit into
mainfrom
chore/backport-sandbox-hardening

Conversation

@robercano-ghbot

Copy link
Copy Markdown
Collaborator

Backports the OS-sandbox + permission hardening that proved out downstream. The framework's settings.json had permissions.allow/deny but no sandbox block and only a minimal deny list.

What

  • New sandbox block — OS sandbox (bubblewrap on Linux/WSL2) confining Bash writes to repo+temp and blocking reads of Windows /mnt and credential dirs (~/.ssh, ~/.aws, ~/.config/gcloud, ~/.kube, ~/.gnupg, ~/.npmrc, ~/.docker/config.json). allowUnsandboxedCommands: true so it degrades gracefully when bwrap isn't installed.
  • Expanded permissions.deny (defense-in-depth, always enforced even when the sandbox is off): privilege escalation (sudo/doas/su), docker, network fetchers (curl/wget/nc/telnet), Windows shells (cmd.exe/powershell/wsl.exe), /mnt, package publishes, gh auth token/gh secret, force-push, rm -rf, and Read denies for .env* + credential dirs + /etc writes.

Scope

  • The stack-agnostic permissions.allow list is unchanged — projects still add their own build/test prefixes per the file's _README.
  • This is the generic security layer only; nothing project-specific.

Note: created via the contents API off current main; single-file change.

🤖 Generated with Claude Code

@robercano robercano left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good!

@robercano
robercano merged commit 2da7d7a into main Jun 30, 2026
6 checks passed
@robercano
robercano deleted the chore/backport-sandbox-hardening branch June 30, 2026 11:41
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