Skip to content

[build-tools] Add sandbox MCP connection step - #4333

Open
sjchmiela wants to merge 1 commit into
stanley/mcp-server-url-contextfrom
stanley/sandbox-mcp-daemon
Open

[build-tools] Add sandbox MCP connection step#4333
sjchmiela wants to merge 1 commit into
stanley/mcp-server-url-contextfrom
stanley/sandbox-mcp-daemon

Conversation

@sjchmiela

@sjchmiela sjchmiela commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Why

See https://github.com/expo/universe/pull/30766.

How

Added a simple "sandbox daemon" that connects to MCP and an eas/start_sandbox EAS function.

This sets up the basic structure, planning to take care of protocol, handling messages and executing commands will come in future PRs.

Test Plan

Tested with https://github.com/expo/universe/pull/30766 and https://github.com/expo/universe/pull/30763.

<-- GET /sandbox/connect
New sandbox daemon - sandboxId[01a0692f-b464-77c8-8ab0-e2143eec9872] ipAddress[::1]
--> GET /sandbox/connect 101 23ms
[wrangler:info] GET /sandbox/connect 101 Switching Protocols (50ms)

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.65385% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.54%. Comparing base (51cc35e) to head (7fe8c92).

Files with missing lines Patch % Lines
...es/build-tools/src/steps/functions/startSandbox.ts 77.36% 12 Missing ⚠️
...kages/build-tools/src/steps/utils/sandboxDaemon.ts 90.00% 5 Missing ⚠️
Additional details and impacted files
@@                        Coverage Diff                         @@
##           stanley/mcp-server-url-context    #4333      +/-   ##
==================================================================
+ Coverage                           65.50%   65.54%   +0.04%     
==================================================================
  Files                                1061     1063       +2     
  Lines                               49338    49442     +104     
  Branches                            10504    10464      -40     
==================================================================
+ Hits                                32313    32400      +87     
- Misses                              15602    16926    +1324     
+ Partials                             1423      116    -1307     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sjchmiela
sjchmiela force-pushed the stanley/sandbox-mcp-daemon branch from a860d22 to d0b5a8f Compare September 3, 2026 20:38
@sjchmiela
sjchmiela changed the base branch from main to stanley/mcp-server-url-context September 3, 2026 20:39
@sjchmiela
sjchmiela force-pushed the stanley/sandbox-mcp-daemon branch 2 times, most recently from d7401e8 to d0f408b Compare September 3, 2026 21:18
@sjchmiela
sjchmiela force-pushed the stanley/sandbox-mcp-daemon branch 4 times, most recently from c67dbb1 to 7c1c971 Compare September 3, 2026 21:27
@sjchmiela
sjchmiela force-pushed the stanley/sandbox-mcp-daemon branch from 7c1c971 to 7fe8c92 Compare September 3, 2026 21:31
@sjchmiela sjchmiela added the no changelog PR that doesn't require a changelog entry label Sep 4, 2026
@sjchmiela
sjchmiela force-pushed the stanley/sandbox-mcp-daemon branch from 29c5674 to 7fe8c92 Compare September 4, 2026 11:14
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant