Skip to content

API spec update, sandbox spec from basetenlabs/sandbox, and sandbox network removal - #41

Merged
cretz merged 1 commit into
mainfrom
cretz/remove-sandbox-network
Nov 9, 2026
Merged

cretz merged 1 commit into
mainfrom
cretz/remove-sandbox-network

Conversation

@cretz

@cretz cretz commented Nov 9, 2026

Copy link
Copy Markdown
Collaborator

🚀 What

  • Update management and sandbox API specs
  • Sandbox API spec now comes from basetenlabs/sandbox instead of blaxel-ai/sandbox
  • Remove sandbox network configuration (SandboxNetwork, SandboxNetworkProxy, SandboxNetworkProxyRoute, network on create and info), which the API no longer supports

💻 How

  • Point SANDBOX_SPEC_URL at basetenlabs/sandbox and regenerate with --update-specs
  • The fork's spec is older than Blaxel's, so the low-level sandbox client loses process wait, filesystem copy, and filesystem tree params; no high-level SDK or CLI code uses them

🔬 Testing

  • Existing unit tests updated; typecheck and unit tests pass

@cretz
cretz requested review from cploujoux and a balanced review from Copilot November 9, 2026 14:12

Copilot AI 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.

🟢 Approval recommended

No active review issues are reported for the SDK and API alignment changes.

No open issues 🎉

What changed in this PR

This PR aligns the Python SDK with updated management APIs and Baseten’s sandbox fork, removing unsupported sandbox network configuration from both generated clients and the high-level SDK.

Changes:

  • Switch sandbox spec generation to basetenlabs/sandbox, removing low-level process-wait, filesystem-copy, and tree-query support absent from that fork.
  • Remove sandbox network models, exports, creation parameters, and response mappings; update related tests.
  • Refresh management models for intelligent-route usage and harness naming, and clarify billing date-range documentation.
File Description
tests/​sandbox/​test_parity.py Remove parity checks for deleted network models.
tests/​sandbox/​test_info.py Remove network mapping fixtures and assertions.
tests/​sandbox/​test_client.py Update creation and info tests without network configuration.
scripts/​apigen/​specs/​sandbox.yml Replace sandbox spec with the Baseten fork’s API definitions.
scripts/​apigen/​specs/​management.json Refresh management schemas and remove sandbox network configuration.
scripts/​apigen/​__main__.py Point sandbox spec downloads to Baseten’s fork.
baseten/​sandbox/​_info.py Remove network dataclasses and conversion helpers.
baseten/​sandbox/​_client.py Remove network parameters from synchronous and asynchronous creation.
baseten/​sandbox/​__init__.py Remove public network-model exports.
baseten/​client/​sandboxapi/​_models.py Regenerate models to match the fork’s supported features.
baseten/​client/​sandboxapi/​_client.py Regenerate methods, parameters, and error mappings.
baseten/​client/​sandboxapi/​__init__.py Remove exports for deleted sandbox API models.
baseten/​client/​managementapi/​_models.py Update usage and harness models; remove network models.
baseten/​client/​managementapi/​__init__.py Align exports with regenerated management models.

Review effort: Balanced

@cretz
cretz requested a review from rdingwall November 9, 2026 14:41
@cretz
cretz merged commit 3f1347c into main Nov 9, 2026
9 of 10 checks passed
@cretz
cretz deleted the cretz/remove-sandbox-network branch November 9, 2026 15: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.

4 participants