Skip to content

feat(named-sandboxes): fix 422 error when resuming after a snapshot#126

Merged
marc-vercel merged 2 commits into
named-sandboxesfrom
sanboxes-fix-resume-error
Apr 1, 2026
Merged

feat(named-sandboxes): fix 422 error when resuming after a snapshot#126
marc-vercel merged 2 commits into
named-sandboxesfrom
sanboxes-fix-resume-error

Conversation

@marc-vercel
Copy link
Copy Markdown
Collaborator

Fix an error where we are failing with a 422 when we resume after a snapshot. This is because we were not handling the 422 Snapshotting error properly and we did not retry.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
sandbox Ready Ready Preview, Comment, Open in v0 Apr 1, 2026 1:23pm
sandbox-cli Ready Ready Preview, Comment Apr 1, 2026 1:23pm
sandbox-sdk Ready Ready Preview, Comment Apr 1, 2026 1:23pm
sandbox-sdk-ai-example Ready Ready Preview, Comment Apr 1, 2026 1:23pm
workflow-code-runner Error Error Apr 1, 2026 1:23pm

Request Review

@marc-vercel marc-vercel changed the base branch from main to named-sandboxes April 1, 2026 13:01
@marc-vercel marc-vercel changed the title Sanboxes fix resume error feat(named-sandboxes): fix 422 error when resuming after a snapshot Apr 1, 2026
@marc-vercel marc-vercel marked this pull request as ready for review April 1, 2026 13:02
@marc-vercel marc-vercel requested review from AndyW22 and QuiiBz April 1, 2026 13:02
Copy link
Copy Markdown
Contributor

@vercel vercel Bot left a comment

Choose a reason for hiding this comment

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

Additional Suggestion:

The update method's catch block doesn't handle isSandboxSnapshottingError, causing network policy update errors to propagate when the sandbox is snapshotting.

Fix on Vercel

@marc-vercel marc-vercel merged commit 90dfaa5 into named-sandboxes Apr 1, 2026
13 of 14 checks passed
@marc-vercel marc-vercel deleted the sanboxes-fix-resume-error branch April 1, 2026 13:37
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
LukeSheard pushed a commit that referenced this pull request May 11, 2026
…126)

Fix an error where we are failing with a 422 when we resume after a
snapshot. This is because we were not handling the 422 Snapshotting
error properly and we did not retry.
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