Skip to content

fix(sandbox): expose resource mounts to file tools - #135

Merged
yanpgwang merged 1 commit into
mainfrom
codex/fix-mounted-resource-file-tools
Aug 11, 2026
Merged

fix(sandbox): expose resource mounts to file tools#135
yanpgwang merged 1 commit into
mainfrom
codex/fix-mounted-resource-file-tools

Conversation

@yanpgwang

Copy link
Copy Markdown
Owner

Summary

  • allow standard file tools to access explicitly provisioned Session File and Memory Store mounts
  • preserve read-only boundaries for uploads, Skills, and read-only Memory Stores
  • keep workspace traversal and unrelated absolute paths rejected
  • restore the same authority after Docker provider reattachment

Why

Managed Agents advertises mounted Files and Memory Stores at absolute sandbox paths, but Docker ReadFile and WriteFile previously rebased every path beneath /workspace. As a result, Read/Write/Edit could silently target /workspace/mnt/... while Bash saw the real mount.

Verification

  • make verify
  • make test-service
  • focused Docker tests covering File Resources, read-write and read-only Memory Stores, traversal rejection, and provider reattachment

@yanpgwang
yanpgwang merged commit 0c53841 into main Aug 11, 2026
6 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