Uh oh!
There was an error while loading. Please reload this page.
refactor: simplify POST /api/sandboxes to only create sandbox - #185
Conversation
Remove prompt execution from the createSandbox endpoint. The sandbox is now created and left running so that commands can be executed separately via the runSandboxCommand task for faster API response times. - Remove prompt parameter from createSandbox function - Remove installClaudeCode and runClaudeCode calls - Remove sandbox.stop() - sandbox left running for later use - Simplify validateSandboxBody to only validate auth - Update tests to reflect new behavior (TDD) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
✨ Finishing touches🧪 Generate unit tests (beta)
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 |
Add triggerRunSandboxCommand function to trigger the Trigger.dev task with the sandboxId and prompt after the sandbox is created. This completes the async execution pattern where the API returns immediately with the sandbox info while the actual Claude Code execution runs in the background via Trigger.dev. - Add triggerRunSandboxCommand.ts to trigger the task - Add prompt validation back to validateSandboxBody - Update createSandboxPostHandler to trigger the task - Add comprehensive tests for all new functionality (44 tests pass) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
Summary
Test plan
🤖 Generated with Claude Code