Skip to content

feat: AutoResume API takes an enabled bool instead of enum policy#2026

Merged
matthewlouisbrockman merged 8 commits intomainfrom
normalize-autoresume-api-to-sdk
Mar 2, 2026
Merged

feat: AutoResume API takes an enabled bool instead of enum policy#2026
matthewlouisbrockman merged 8 commits intomainfrom
normalize-autoresume-api-to-sdk

Conversation

@matthewlouisbrockman
Copy link
Contributor

@matthewlouisbrockman matthewlouisbrockman commented Feb 28, 2026

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 true maps to an any policy and can change that as add settings


Note

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 enabled defaults to off). Runtime risk is otherwise limited to how paused sandboxes are resumed based on the stored policy mapping.

Overview
Sandbox autoResume configuration is simplified from a policy enum (any/off) to a single boolean enabled, with server-side creation logic mapping enabled=true to SandboxAutoResumeAny and false (or omitted) to SandboxAutoResumeOff. 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.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

@matthewlouisbrockman matthewlouisbrockman marked this pull request as draft March 2, 2026 18:46
@matthewlouisbrockman
Copy link
Contributor Author

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

@matthewlouisbrockman matthewlouisbrockman changed the title AutoResume takes an enabled bool instead of enum policy feat: AutoResume takes an enabled bool instead of enum policy Mar 2, 2026
@matthewlouisbrockman matthewlouisbrockman changed the title feat: AutoResume takes an enabled bool instead of enum policy feat: AutoResume API takes an enabled bool instead of enum policy Mar 2, 2026
@matthewlouisbrockman matthewlouisbrockman marked this pull request as ready for review March 2, 2026 19:16
@matthewlouisbrockman matthewlouisbrockman merged commit 9931031 into main Mar 2, 2026
37 checks passed
@matthewlouisbrockman matthewlouisbrockman deleted the normalize-autoresume-api-to-sdk branch March 2, 2026 22:09
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.

3 participants