fix: await stdout() and stderr() async functions from sandbox

[sweetmantech]

Summary

• Fix stdout and stderr not being included in task response
• The Vercel Sandbox SDK returns stdout and stderr as async functions that need to be called and awaited, not direct string properties

Test plan

• Run task with find . command - verified stdout returns file listing
• Verify exitCode, stdout, stderr, and snapshot all returned correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

• Refactor
  • Internal improvements to command execution handling with no impact to user-facing functionality.

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated runSandboxCommandTask.ts to retrieve command output asynchronously by awaiting stdout() and stderr() method calls instead of accessing them as synchronous properties, aligning the implementation with a Promise-based API.

Changes

Cohort / File(s)	Summary
Async API Update src/tasks/runSandboxCommandTask.ts	Changed from synchronous property access to asynchronous method invocation for retrieving command output; now awaits commandResult.stdout() and commandResult.stderr() with nullish-coalesce fallback to empty strings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feat: add runSandboxCommand task for background sandbox execution #17: Directly modified the same file and function to adapt to the async API for stdout/stderr retrieval.

Poem

🐰 A rabbit awaits the command's reply,
Where stdout() and stderr() dance in the sky,
No longer sync, but promises held dear,
Async and gentle—the code's crystal clear! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely describes the main change—fixing async function calls for stdout() and stderr() from the sandbox.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/include-stdout-stderr-in-response

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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