examples(coding-agent): bump sandbox runtime#1536
examples(coding-agent): bump sandbox runtime#1536goncalossilva wants to merge 1 commit intobadlogic:mainfrom
Conversation
Update the sandbox extension example to use\n@anthropic-ai/sandbox-runtime@^0.0.37.\n\nAlso pass through all SandboxRuntimeConfig fields from the merged config\ninstead of whitelisting only a subset. This keeps new runtime options\nworking as the sandbox-runtime API evolves.
|
Hi @goncalossilva, thanks for your interest in contributing! We ask new contributors to open an issue first before submitting a PR. This helps us discuss the approach and avoid wasted effort. Next steps:
This PR will be closed automatically. See https://github.com/badlogic/pi-mono/blob/main/CONTRIBUTING.md for more details. |
There was a problem hiding this comment.
Pull request overview
This PR bumps the @anthropic-ai/sandbox-runtime dependency from version 0.0.26 to 0.0.37 in the sandbox example extension. The update refactors configuration handling to be more maintainable and forward-compatible with newer runtime versions.
Changes:
- Upgraded
@anthropic-ai/sandbox-runtimefrom ^0.0.26 to ^0.0.37 - Simplified the
deepMergefunction to use spread operators instead of explicit field assignments - Refactored
SandboxManager.initialize()call to pass the full runtime config via destructuring
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Bumped sandbox-runtime dependency to ^0.0.37 |
| package-lock.json | Updated lock file with new sandbox-runtime version and transitive dependency lodash-es to 4.17.23 |
| index.ts | Refactored config merging and initialization to be more maintainable and forward-compatible with new runtime config fields |
Files not reviewed (1)
- packages/coding-agent/examples/extensions/sandbox/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Fair! It was a better idea all along, as I don't fully understand why some existing fields were excluded originally, and that may inform the approach here. I'll be back if/once OSS vacation is off. 😄 |
Bumps sandbox example extension to
@anthropic-ai/sandbox-runtime@^0.0.37and updates config merging to preserve allSandboxRuntimeConfigfields.Newer versions add optional fields (e.g.,
network.mitmProxyandenableWeakerNetworkIsolation). The extension currently allows only a subset of fields, so other options were dropped.