Skip to content

Fix local bucket watches outside workspace - #887

Open
scuffi wants to merge 1 commit into
mainfrom
fix/883-local-watch-main
Open

Fix local bucket watches outside workspace#887
scuffi wants to merge 1 commit into
mainfrom
fix/883-local-watch-main

Conversation

@scuffi

@scuffi scuffi commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Local bucket mounts outside /workspace could download files but could not watch for container changes, leaving synchronization one-way.

Add an internal mount-watch path that only accepts registered local mount roots and recovers after remounts.

Resolves #883.

@changeset-bot

changeset-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 06ae76b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudflare/sandbox Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

devin-ai-integration[bot]

This comment was marked as resolved.

@pkg-pr-new

pkg-pr-new Bot commented Aug 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@cloudflare/sandbox@887

commit: 06ae76b

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

🐳 Docker Images Published

Variant Image
Default cloudflare/sandbox:0.0.0-pr-887-06ae76bb
Python cloudflare/sandbox:0.0.0-pr-887-06ae76bb-python
OpenCode cloudflare/sandbox:0.0.0-pr-887-06ae76bb-opencode
Musl cloudflare/sandbox:0.0.0-pr-887-06ae76bb-musl

Usage:

FROM cloudflare/sandbox:0.0.0-pr-887-06ae76bb

Version: 0.0.0-pr-887-06ae76bb


📦 Standalone Binary

For arbitrary Dockerfiles:

COPY --from=cloudflare/sandbox:0.0.0-pr-887-06ae76bb /container-server/sandbox /sandbox
ENTRYPOINT ["/sandbox"]

Download via GitHub CLI:

gh run download 33642692132 -n sandbox-binary

Extract from Docker:

docker run --rm cloudflare/sandbox:0.0.0-pr-887-06ae76bb cat /container-server/sandbox > sandbox && chmod +x sandbox

@scuffi
scuffi force-pushed the fix/883-local-watch-main branch from 091afa3 to 6e358b4 Compare August 28, 2026 14:00
@scuffi
scuffi force-pushed the fix/883-local-watch-main branch from 6e358b4 to 06ae76b Compare September 2, 2026 14:32
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.

Local R2 mount at /data repeatedly fails reverse-sync watcher because watch() requires /workspace

1 participant