Skip to content

chore: update e2b sdk#359

Merged
ben-fornefeld merged 4 commits into
mainfrom
chore/update-e2b-sdk
Jun 6, 2026
Merged

chore: update e2b sdk#359
ben-fornefeld merged 4 commits into
mainfrom
chore/update-e2b-sdk

Conversation

@ben-fornefeld
Copy link
Copy Markdown
Member

@ben-fornefeld ben-fornefeld commented Jun 6, 2026

Updated E2B SDK and how we pass auth headers

Copilot AI review requested due to automatic review settings June 6, 2026 06:53
@ben-fornefeld ben-fornefeld requested a review from drankou as a code owner June 6, 2026 06:53
@cla-bot cla-bot Bot added the cla-signed label Jun 6, 2026
@cursor
Copy link
Copy Markdown

cursor Bot commented Jun 6, 2026

PR Summary

Medium Risk
Touches all dashboard sandbox create/connect flows; a missed call site or SDK mismatch would break sandbox auth or connections.

Overview
Bumps the e2b dependency from 2.27.1 to 2.28.0 and aligns dashboard sandbox usage with the SDK’s renamed connect/create option: headersapiHeaders.

Supabase team/token auth is unchanged; only the option key updates for new sandbox (sbx/new), filesystem inspect (Sandbox.connect), and dashboard terminal create/connect paths. Unit tests for terminal sandbox calls were updated to assert apiHeaders.

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

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment Jun 6, 2026 7:01am
web-juliett Ready Ready Preview, Comment Jun 6, 2026 7:01am

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the e2b SDK dependency and adapts dashboard sandbox connection code to the SDK’s updated Sandbox.connect option naming.

Changes:

  • Bump e2b from ^2.27.1 to ^2.28.0.
  • Update Sandbox.connect(...) options to use apiHeaders instead of headers in terminal + sandbox inspect flows.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/features/dashboard/terminal/sandbox-session.ts Updates terminal sandbox connection options to pass auth via apiHeaders.
src/features/dashboard/sandbox/inspect/context.tsx Updates inspect sandbox connection options to pass auth via apiHeaders.
package.json Bumps e2b dependency to ^2.28.0.
bun.lock Locks e2b to 2.28.0.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/features/dashboard/terminal/sandbox-session.ts
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 47008d983a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/features/dashboard/terminal/sandbox-session.ts
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6fd0581. Configure here.

Comment thread tests/unit/dashboard-terminal.test.ts Outdated
Comment thread tests/unit/dashboard-terminal.test.ts Outdated
@ben-fornefeld ben-fornefeld merged commit f9a911a into main Jun 6, 2026
15 checks passed
@ben-fornefeld ben-fornefeld deleted the chore/update-e2b-sdk branch June 6, 2026 07:03
Comment thread src/features/dashboard/terminal/sandbox-session.ts
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.

4 participants