Skip to content

feat(sandbox): SandboxProcessRunner OS process manager (#300) - #307

Merged
ZhuchkaTriplesix merged 2 commits into
devfrom
issue/300-sandbox-process-runner
Jul 10, 2026
Merged

feat(sandbox): SandboxProcessRunner OS process manager (#300)#307
ZhuchkaTriplesix merged 2 commits into
devfrom
issue/300-sandbox-process-runner

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Summary

  • Add SandboxScratchDirectory for isolated /tmp/querya_sandbox/<plugin>_<token>/ workspaces with delete + orphan cleanup
  • Add SandboxLaunchCommand builders: Linux bwrap --unshare-all --share-net --ro-bind / /, macOS sandbox-exec Seatbelt profile, Windows direct spawn (AppContainer/Job Object deferred to native helper)
  • Add SandboxProcessRunner / SandboxProcessHandle with injectable Process.start, sanitized environment (no parent env / no secrets on argv), and kill() / dispose() that always remove the scratch tree

Closes #300

Test plan

  • flutter test test/core/extensions/sandbox/ — unit + soft bwrap integration
  • CI analyze / full suite

Implements Block E M1 OS process sandbox manager: scratch directories under
querya_sandbox/, Linux bwrap and macOS sandbox-exec launch commands,
Windows soft-start, and kill/dispose cleanup (issue #300).
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit 4e360cb into dev Jul 10, 2026
4 checks passed
@ZhuchkaTriplesix ZhuchkaTriplesix self-assigned this Jul 12, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/300-sandbox-process-runner branch July 26, 2026 14:39
ZhuchkaTriplesix added a commit that referenced this pull request Jul 27, 2026
feat(sandbox): SandboxProcessRunner OS process manager (#300)
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