docs: correct sandbox policy prompt timing - #25724
Conversation
The get-started guide said sbx login initialized the default network policy, but login is authentication-only. Move the selector to the first sbx run, document the policy-dependent trigger and reset behavior, and sync the prompt text with the CLI. Fixes docker/sandboxes#4850. Co-Authored-By: Codex <noreply@openai.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
The PR correctly moves the sandbox policy prompt from the sbx login step to the first sbx run step in both get-started.md and governance/local.md, aligning the docs with current CLI behavior. No vendored files are touched, no redirects are needed, and there are no AI-isms or markdown issues.
One medium concern: the updated sbx policy reset description in governance/local.md changes from a deferred restart ("stops the daemon… restarts on the next command") to an immediate restart ("restarts the daemon"). If the actual behavior is still deferred, this introduces a misleading description.
2 participants
Summary
Correct the Docker Sandboxes onboarding flow so the policy selector appears with the first sandbox run instead of
sbx login. Align the local policy page and prompt example with the current CLI behavior.@netlify /ai/sandboxes/get-started/
Preview:
Closes docker/sandboxes#4850
Generated by Codex