Skip to content

Add sandbox apis - #32

Merged
herber merged 1 commit into
mainfrom
feat/add-sandbox-apis
Oct 16, 2026
Merged

herber merged 1 commit into
mainfrom
feat/add-sandbox-apis

Conversation

@herber

@herber herber commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Note

Medium Risk
Large generated surface with breaking path/parameter changes for firewalls and integration networking; callers using removed networks/enclaves/bindings endpoints must migrate.

Overview
Adds sandbox support to the Python SDK: generated Magnetar clients for machines (lifecycle, access tokens, usage), disks, images/builds, regions, and specs, exposed via new client.sandbox (and public exports) alongside chats / chat groups.

Also syncs other API changes: chat command respond, consumer provider reconnect, and a networking model rename—instance networks and firewall-bindings / generic enclaves endpoints are replaced by integration enclaves, integration firewall assignments, and integration internet gateway (including network logs). Firewall create/list no longer use network_id / inline bindings; listing can filter by sandbox machine_id. Several tool_filters request shapes pick up an additional allowed variant in generated types.

Reviewed by Cursor Bugbot for commit df75714. Bugbot is set up for automated code reviews on this repo. Configure here.

@herber herber self-assigned this Oct 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit df75714. Configure here.

# Build query parameters from keyword arguments
query_dict = {}
query_dict["from"] = from_
query_dict["to"] = to

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Datetime query params not serialized

Medium Severity

timeline and summary put from_ and to datetime values into the query dict unchanged. The HTTP client stringifies them as 2026-10-16 08:25:00 instead of ISO-8601, so the required range is sent in the wrong format and usage requests fail or return an empty window. Similar log endpoints already take these bounds as strings.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit df75714. Configure here.

@herber
herber merged commit df75714 into main Oct 16, 2026
12 checks passed
@herber
herber deleted the feat/add-sandbox-apis branch October 16, 2026 09:16
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.

1 participant