Uh oh!
There was an error while loading. Please reload this page.
feat: add POST /api/sandbox endpoint - #155
Merged
sweetmantech merged 5 commits intoJan 23, 2026
Merged
Conversation
* feat: add Google Drive and Google Docs to enabled toolkits - Add googledrive and googledocs to ENABLED_TOOLKITS array - Enables Tool Router access to Google Drive and Google Docs tools * docs: update JSDoc to mention all Google integrations
Create ephemeral sandbox environments using Vercel Sandbox SDK. - Add route handler with CORS and API key authentication - Add createSandbox function using @vercel/sandbox - Returns sandboxId, status, timeout, createdAt Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Braintrust eval reportCatalog Opportunity Analysis Evaluation (HEAD-1769198199)
Catalog Songs Count Evaluation (HEAD-1769198199)
First Week Album Sales Evaluation (HEAD-1769198199)
Memory & Storage Tools Evaluation (HEAD-1769198199)
Monthly Listeners Tracking Evaluation (HEAD-1769198199)
Search Web Tool Evaluation (HEAD-1769198199)
Social Scraping Evaluation (HEAD-1769198199)
Spotify Followers Evaluation (HEAD-1769198199)
Spotify Tools Evaluation (HEAD-1769198199)
TikTok Analytics Questions Evaluation (HEAD-1769198199)
|
The Vercel Sandbox SDK includes "snapshotting" as a valid status. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Comment on lines
+12
to
+19
| * @param request - The request object | ||
| * @returns A NextResponse with sandbox data or error | ||
| */ | ||
| export async function createSandboxPostHandler(request: NextRequest): Promise<NextResponse> { | ||
| const accountIdOrError = await getApiKeyAccountId(request); | ||
| if (accountIdOrError instanceof NextResponse) { | ||
| return accountIdOrError; | ||
| } |
Contributor
There was a problem hiding this comment.
Handler extracts accountId for authentication but never uses it, making the purpose of the validation unclear and suggesting incomplete implementation
| try { | ||
| const sandbox = await createSandbox(); | ||
| return NextResponse.json(sandbox, { status: 200, headers: getCorsHeaders() }); |
Contributor
There was a problem hiding this comment.
Sandbox POST endpoint returns raw object instead of wrapped object, inconsistent with workspace and artist endpoints
| return NextResponse.json(sandbox, { status: 200, headers: getCorsHeaders() }); | ||
| } catch (error) { | ||
| const message = error instanceof Error ? error.message : "Failed to create sandbox"; | ||
| return NextResponse.json({ error: message }, { status: 400, headers: getCorsHeaders() }); |
Contributor
There was a problem hiding this comment.
Sandbox error handler returns { error: message } instead of { status: "error", error: message }, inconsistent with workspace and other handlers
DRY - reference types from @vercel/sandbox instead of duplicating. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
sweetmantech
deleted the
sweetmantech/myc-4023-api-create-sandbox-post-sandbox
branch
January 23, 2026 20:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Files Added
app/api/sandbox/route.ts- Route handler with CORS and JSDoclib/sandbox/createSandbox.ts- Sandbox creation logic using Vercel SDKlib/sandbox/createSandboxPostHandler.ts- Request handler with API key authTest plan
🤖 Generated with Claude Code