Skip to content

fix: make sandbox uid configurable across all languages#75

Merged
usnavy13 merged 1 commit intodevfrom
fix/shared-sandbox-uid
Mar 13, 2026
Merged

fix: make sandbox uid configurable across all languages#75
usnavy13 merged 1 commit intodevfrom
fix/shared-sandbox-uid

Conversation

@usnavy13
Copy link
Copy Markdown
Owner

Summary

  • replace mixed hardcoded sandbox UIDs with one shared SANDBOX_UID override for all sandbox languages
  • default all sandbox languages to non-root UID 1001, including Python, Java, and D
  • update docs and tests to reflect the shared configurable sandbox UID model

Validation

  • python3 -m flake8 src/ --count --select=E9,F63,F7,F82 --show-source --statistics
  • python3 -m flake8 src/ --count --exit-zero --max-complexity=10 --statistics
  • python3 -m black src/ --check
  • python3 -m mypy src/
  • .venv/bin/python -m bandit -r src/ -s B104,B108 --severity-level high
  • pytest -q tests/unit/
  • docker build -t code-interpreter:ci-check .
  • API_BASE='http://localhost:8080' API_KEY='<repo .env key>' API_TIMEOUT=120 pytest -q tests/functional

Notes

  • The second flake8 command is the same exit-zero warning sweep used by CI; it reported existing repository style warnings but does not fail the workflow.
  • The live functional suite passed end-to-end with SANDBOX_UID=2001 set in the running host .env.

@usnavy13 usnavy13 merged commit 43ffc7c into dev Mar 13, 2026
2 checks passed
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