Fix documented sandbox idle-timeout default (300s → 0 / persistent)#381
Draft
opencomputerdev[bot] wants to merge 1 commit into
Draft
Fix documented sandbox idle-timeout default (300s → 0 / persistent)#381opencomputerdev[bot] wants to merge 1 commit into
opencomputerdev[bot] wants to merge 1 commit into
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
2027 — Eval complete for commit
|
| URL | Mapping |
|---|---|
| digger.dev | → opensandbox-oc-s-762ac928075c46d2828bcb22.mintlify.app |
| opencomputer.dev | → opensandbox-oc-s-762ac928075c46d2828bcb22.mintlify.app |
2027 // complete the getting started guide at https://opencomputer.d — Did not finish
- ⚠️ Score nulled: AI judge determined the task was not completed — "Sandbox creation failed with 402 'free trial credits exhausted'; the final Hello World output step of the quickstart was never achieved."Prompt
complete the getting started guide at https://opencomputer.dev
Commit 86d854e · View report →
Evaluating agent experience using 2027.dev · View dashboard
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Verified the SDK/CLI/API surfaces against the actual code in this repo. Found one substantive, repo-wide divergence: the documented default idle timeout was 300s in ~14 places, but the real default is 0 = persistent (never auto-hibernate) in both SDKs (sandbox.ts / sandbox.py), the
ocCLI flags, and the worker's sandbox router (DefaultTimeout left unset, and 0 disables the auto-hibernate timer). The dedicated docs/sandboxes/timeout.mdx page already documented the correct default — only the reference tables/ParamFields, the HTTP API reference, and several prose claims (how-it-works, sandboxes/overview, troubleshooting, cli/sandbox) were stale.Fixes: updated all timeout-default values from 300 → 0 and reworded the prose to say sandboxes are persistent by default and only auto-hibernate when a positive timeout is set. Did NOT touch example code that explicitly passes
timeout: 300— that's valid usage, not a default claim.Also spot-checked and confirmed ACCURATE (no change): default template
base(SDK) vs README's explicitdefault(a real seeded template per migration 008); env vars OPENCOMPUTER_API_KEY/OPENCOMPUTER_API_URL;oc config set api-key;exec.run()60s default; and the getAllowedHostsbaseSecretStore/union-allowlist behavior (matches internal/api/sandbox_allowed_hosts.go).Unverified (no access): the managed-agents product surfaces under docs/agents/, docs/agents-api/, and docs/sessions-api/ describe services that AGENTS.md states are not implemented in this repo — left untouched.
Findings (4)
timeout: opts.timeout ?? 0); sdks/python/opencomputer/sandbox.py:94 (timeout: int = 0)Int("timeout", 0, "... 0 = never hibernate")) and :301 (wake flag default 0)🤖 Opened by the OpenComputer docs-reconciler — a scheduled backstop. Review and merge, or push your own commits and it will leave this PR alone.