Skip to content

Rust SDK: Add sandbox config - #2371

Open
jcoc611-microsoft wants to merge 1 commit into
mainfrom
user/juosori/Sandbox-RustSDK
Open

Rust SDK: Add sandbox config#2371
jcoc611-microsoft wants to merge 1 commit into
mainfrom
user/juosori/Sandbox-RustSDK

Conversation

@jcoc611-microsoft

Copy link
Copy Markdown

Summary

Add sandbox configuration support to Rust session creation and resume APIs.

  • Expose SandboxConfig and related policy types through the Rust public API.
  • Forward sandbox_config as sandboxConfig in create and resume requests.
  • Omit the field when it is not configured.
  • Add serialization coverage for nested sandbox network and proxy settings.

The Java SDK’s generated RPC models already include SandboxConfig in session options. This change makes the same protocol capability available to Rust SDK consumers.

Testing

  • Focused sandbox serialization test
  • Clippy with all features and warnings denied
  • Rust formatting check

I also tested sandboxing config e2e in a separate repo that uses the Rust SDK.

@jcoc611-microsoft
jcoc611-microsoft requested review from jmoseley and stephentoub and a balanced review from Copilot and removed request for stephentoub August 21, 2026 21:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Rust SDK sandbox configuration support for session creation and resume.

Changes:

  • Exposes sandbox policy types publicly.
  • Forwards optional sandbox configuration in create/resume requests.
  • Adds nested serialization and omission coverage.
Show a summary per file
File Description
rust/src/wire.rs Adds optional sandbox configuration to wire payloads.
rust/src/types.rs Extends public configs, conversions, exports, redacted debug output, and tests.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@jcoc611-microsoft
jcoc611-microsoft marked this pull request as ready for review August 21, 2026 21:40
@jcoc611-microsoft
jcoc611-microsoft requested a review from a team as a code owner August 21, 2026 21:40
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.

2 participants