Skip to content

fix: support frozen sandbox worker entry#59

Merged
F16shen merged 1 commit intoAI-Shell-Team:mainfrom
F16shen:fix/sandbox_worker-not-found
Mar 18, 2026
Merged

fix: support frozen sandbox worker entry#59
F16shen merged 1 commit intoAI-Shell-Team:mainfrom
F16shen:fix/sandbox_worker-not-found

Conversation

@F16shen
Copy link
Copy Markdown
Collaborator

@F16shen F16shen commented Mar 18, 2026

Summary

  • route frozen sandbox worker launches through an internal daemon entrypoint instead of sys.executable -m ...
  • keep source runs on the existing module entrypoint and include aish.security.sandbox_worker in the PyInstaller sandbox binary
  • add regression coverage for source and frozen worker launch paths

Problem

PyInstaller-built aish-sandbox binaries use the daemon executable as sys.executable. Launching the worker with sys.executable -m aish.security.sandbox_worker causes the daemon CLI to parse -m as its own argument and fail.

Testing

  • /home/lixin/workspace/aish/.venv/bin/python -m pytest tests/security/sandbox/test_daemon_isolation.py tests/security/sandbox/test_sandboxd.py tests/security/sandbox/test_worker.py

@F16shen F16shen merged commit 2ba42c0 into AI-Shell-Team:main Mar 18, 2026
10 checks passed
@F16shen F16shen deleted the fix/sandbox_worker-not-found branch March 18, 2026 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant