Skip to content

Whitelist trace2 socket in Claude sandboxes - #2092

Open
svarlamov wants to merge 1 commit into
codex/agent-sandbox-feature-flagfrom
codex/claude-agent-sandbox-feature-flag
Open

Whitelist trace2 socket in Claude sandboxes#2092
svarlamov wants to merge 1 commit into
codex/agent-sandbox-feature-flagfrom
codex/claude-agent-sandbox-feature-flag

Conversation

@svarlamov

@svarlamov svarlamov commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • extend the shared whitelist_agent_sandboxes opt-in to Claude Code
  • add only the active trace2 socket to sandbox.network.allowUnixSockets
  • preserve existing sandbox settings, remain idempotent, and remove only git-ai's socket on uninstall
  • fail safely on unexpected sandbox config shapes and never enable broad allowAllUnixSockets access

Stack

Depends on #2091.

Claude Code documentation

Tests

  • task fmt
  • task lint
  • task test TEST_FILTER=claude_sandbox::
  • task test TEST_FILTER=mdm::agents::claude_code::

Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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