Skip to content

fix(cli): correct memory-mb help default and use non-deprecated pause#1510

Open
mishushakov wants to merge 1 commit into
mainfrom
mishushakov/cli-pause-memory-help-fixes
Open

fix(cli): correct memory-mb help default and use non-deprecated pause#1510
mishushakov wants to merge 1 commit into
mainfrom
mishushakov/cli-pause-memory-help-fixes

Conversation

@mishushakov

Copy link
Copy Markdown
Member

Fixes two small CLI issues. The e2b template create --memory-mb help text claimed a default of 512 MB, but the real default is 1024 MB — the help now reflects that. The e2b sandbox pause command was calling the deprecated Sandbox.betaPause() alias and now calls Sandbox.pause() directly.

Usage

e2b template create --help   # --memory-mb now shows "The default value is 1024."
e2b sandbox pause <sandboxID>  # behaves the same, no longer uses the deprecated method

🤖 Generated with Claude Code

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-bot

changeset-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6c12667

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@e2b/cli Patch

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

@cursor

cursor Bot commented Jun 30, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Help text and a deprecated SDK alias swap only; no behavior or default logic changes beyond using the supported pause API.

Overview
The e2b template create --memory-mb option help said the default was 512 MB; it now says 1024 MB to match the actual default. e2b sandbox pause now calls Sandbox.pause() instead of the deprecated Sandbox.betaPause(). A patch changeset documents both fixes for @e2b/cli.

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

@github-actions

Copy link
Copy Markdown
Contributor

Package Artifacts

Built from 583f8b4. Download artifacts from this workflow run.

JS SDK (e2b@2.31.1-mishushakov-cli-pause-memory-help-fixes.0):

npm install ./e2b-2.31.1-mishushakov-cli-pause-memory-help-fixes.0.tgz

CLI (@e2b/cli@2.13.1-mishushakov-cli-pause-memory-help-fixes.0):

npm install ./e2b-cli-2.13.1-mishushakov-cli-pause-memory-help-fixes.0.tgz

Python SDK (e2b==2.30.0+mishushakov-cli-pause-memory-help-fixes):

pip install ./e2b-2.30.0+mishushakov.cli.pause.memory.help.fixes-py3-none-any.whl

@claude claude 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.

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.

@mishushakov mishushakov requested a review from huv1k June 30, 2026 11:28
@mishushakov mishushakov enabled auto-merge (squash) June 30, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant