Skip to content

feat: remove setup-sandbox task call from POST /api/sandboxes#218

Merged
sweetmantech merged 2 commits intotestfrom
sweetmantech/myc-4220-api-modify-endpoint-post-apisandboxes-remove-call-to-setup
Feb 11, 2026
Merged

feat: remove setup-sandbox task call from POST /api/sandboxes#218
sweetmantech merged 2 commits intotestfrom
sweetmantech/myc-4220-api-modify-endpoint-post-apisandboxes-remove-call-to-setup

Conversation

@sweetmantech
Copy link
Contributor

Summary

  • Removed the triggerSetupSandbox call from the createSandboxPostHandler
  • The setup-sandbox task is no longer triggered when creating sandboxes
  • Command execution via triggerRunSandboxCommand still works when a command is provided
  • Updated tests: removed setup-sandbox-related test cases and mocks (10 tests still passing)

Test plan

  • All 10 existing tests pass after changes
  • Verify sandbox creation without command returns 200 with no runId
  • Verify sandbox creation with command still returns runId from the command task

🤖 Generated with Claude Code

The setup-sandbox trigger is no longer needed when creating sandboxes.
Only the run-sandbox-command task is triggered when a command is provided.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@vercel
Copy link
Contributor

vercel bot commented Feb 11, 2026

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

Project Deployment Actions Updated (UTC)
recoup-api Ready Ready Preview Feb 11, 2026 8:43pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

Warning

Rate limit exceeded

@sweetmantech has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 31 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sweetmantech/myc-4220-api-modify-endpoint-post-apisandboxes-remove-call-to-setup

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

No remaining imports after removing the setup-sandbox call from the handler.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@sweetmantech sweetmantech merged commit 0e6531b into test Feb 11, 2026
3 checks passed
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.

1 participant

Comments