Skip to content

feat(vercel-sandbox): fix bug with resuming sandbox during file read#102

Merged
marc-vercel merged 2 commits intonamed-sandboxesfrom
named-sandboxes-fix-read-file
Mar 18, 2026
Merged

feat(vercel-sandbox): fix bug with resuming sandbox during file read#102
marc-vercel merged 2 commits intonamed-sandboxesfrom
named-sandboxes-fix-read-file

Conversation

@marc-vercel
Copy link
Collaborator

Fix bug with resuming sandboxes when reading a file. I have also added an integration tests to detect if this behavior is broken in the future, as it follows a different path from the other commands (it is a stream).

TL;DR: we were missing the parseOrThrow in this method, and withResume() wrapper checks for an error before resuming (we were returning the error as string)

@vercel
Copy link

vercel bot commented Mar 17, 2026

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

Project Deployment Actions Updated (UTC)
sandbox Error Error Mar 17, 2026 5:27pm
sandbox-cli Ready Ready Preview, Comment Mar 17, 2026 5:27pm
sandbox-sdk Ready Ready Preview, Comment Mar 17, 2026 5:27pm
sandbox-sdk-ai-example Ready Ready Preview, Comment Mar 17, 2026 5:27pm

Request Review

@marc-vercel marc-vercel merged commit 74b9fe2 into named-sandboxes Mar 18, 2026
11 of 12 checks passed
@marc-vercel marc-vercel deleted the named-sandboxes-fix-read-file branch March 18, 2026 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants