feat: AutoResume API takes an enabled bool instead of enum policy#2026
feat: AutoResume API takes an enabled bool instead of enum policy#2026matthewlouisbrockman merged 8 commits intomainfrom
enabled bool instead of enum policy#2026Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3b92b0af9a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
converting to draft to do this halfway; moving the API to a bool for SDK parity but still storing as an enum config to avoid requiring a orchestrator deployment |
enabled bool instead of enum policyenabled bool instead of enum policy
enabled bool instead of enum policyenabled bool instead of enum policy
Simplified the API to match the SDK inputs, autoresume will just have an enabled bool rather than a policy. Still stores as a config with a policy; for now autoResume
truemaps to ananypolicy and can change that as add settingsNote
Medium Risk
This changes the public API contract for sandbox creation auto-resume configuration, which can break existing clients and alters validation behavior (previously invalid/missing policies were rejected; now missing
enableddefaults to off). Runtime risk is otherwise limited to how paused sandboxes are resumed based on the stored policy mapping.Overview
Sandbox
autoResumeconfiguration is simplified from apolicyenum (any/off) to a single booleanenabled, with server-side creation logic mappingenabled=truetoSandboxAutoResumeAnyandfalse(or omitted) toSandboxAutoResumeOff. The OpenAPI spec, generated API clients/swagger, and integration/unit tests are updated to use the new shape, and request-time validation for unknown/empty policies is removed.Written by Cursor Bugbot for commit b4b85d4. This will update automatically on new commits. Configure here.