Skip to content

feat: use sandbox #1075

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

feat: use sandbox #1075

wants to merge 12 commits into from

Conversation

@styfle
Copy link
Owner

@styfle styfle commented Jul 1, 2025 •
edited
Loading

https://vercel.com/docs/vercel-sandbox

TODO:

  • Measure performance of small dependency
  • Measure performance of a large dependency
  • Estimate cost
  • Decide how to calculate size (use du or run same js function?)
  • Decide how to handle case when sandbox takes longer than serverless function (add queues?)

@socket-security
Copy link

socket-security bot commented Jul 1, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Added@​vercel/​sandbox@​0.0.12771009998100
Updatednpm@​10.4.0 ⏵ 10.9.38710093 +197 +2100

View full report

@vercel
Copy link

vercel bot commented Jul 1, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
packagephobia ✅ Ready (Inspect) Visit Preview Jul 1, 2025 6:48pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@23cornellj-sys 23cornellj-sys 23cornellj-sys approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@styfle @23cornellj-sys