fix(cli): correct memory-mb help default and use non-deprecated pause#1510
fix(cli): correct memory-mb help default and use non-deprecated pause#1510mishushakov wants to merge 1 commit into
Conversation
Fix `template create --memory-mb` help text to state the real default of 1024 MB (was 512), and switch `sandbox pause` to call the non-deprecated `Sandbox.pause()` instead of the deprecated `Sandbox.betaPause()` alias. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: 6c12667 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 6c12667. Bugbot is set up for automated code reviews on this repo. Configure here. |
Package ArtifactsBuilt from 583f8b4. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.31.1-mishushakov-cli-pause-memory-help-fixes.0.tgzCLI ( npm install ./e2b-cli-2.13.1-mishushakov-cli-pause-memory-help-fixes.0.tgzPython SDK ( pip install ./e2b-2.30.0+mishushakov.cli.pause.memory.help.fixes-py3-none-any.whl |
There was a problem hiding this comment.
LGTM, both fixes are correct: Sandbox.pause() is the non-deprecated entry point and the template build default is indeed 1024 MB.
Extended reasoning...
Overview
This PR makes two trivial CLI fixes in packages/cli: the e2b template create --memory-mb help text is updated from "512" to "1024" to match the real default, and e2b sandbox pause is switched from the deprecated Sandbox.betaPause() to Sandbox.pause(). A patch-level changeset is included.
Security risks
None. No auth, crypto, permission, or input-parsing logic is touched — only a help string and a method-name swap to the non-deprecated alias.
Level of scrutiny
Very low. Both changes are mechanical and verifiable against the SDK: SandboxApi.pause is defined at packages/js-sdk/src/sandbox/sandboxApi.ts:982 while betaPause at line 1021 carries @deprecated Use SandboxApi.pause instead, and the memory default of 1024 is hard-coded at packages/js-sdk/src/template/index.ts:1138 and documented as @default 1024 in types.ts:42.
Other factors
The bug hunting system found no issues. Changeset is properly labeled patch. No outstanding reviewer comments. Safe to auto-approve.
Fixes two small CLI issues. The
e2b template create --memory-mbhelp text claimed a default of 512 MB, but the real default is 1024 MB — the help now reflects that. Thee2b sandbox pausecommand was calling the deprecatedSandbox.betaPause()alias and now callsSandbox.pause()directly.Usage
🤖 Generated with Claude Code