Skip to content

Conversation

@NathanFlurry
Copy link
Member

Summary

  • Add Node.js build stage to all release Dockerfiles to build the inspector frontend
  • Copy pre-built inspector assets into Rust build stages so they get embedded in binaries
  • Remove SANDBOX_AGENT_SKIP_INSPECTOR=1 from all Dockerfiles and the release workflow

Test plan

  • Build linux-x86_64 locally: docker/release/build.sh x86_64-unknown-linux-musl
  • Run the binary and verify inspector is embedded (should show "inspector ui available")
  • Access the inspector UI at http://localhost:2468/ui/

🤖 Generated with Claude Code

@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-34 January 30, 2026 06:17 Destroyed
@railway-app
Copy link

railway-app bot commented Jan 30, 2026

🚅 Deployed to the sandbox-agent-pr-34 environment in sandbox-agent

Service Status Web Updated (UTC)
website ✅ Success (View Logs) Web Jan 30, 2026 at 8:07 am
inspector ✅ Success (View Logs) Web Jan 30, 2026 at 8:07 am

@claude
Copy link

claude bot commented Jan 30, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@NathanFlurry NathanFlurry changed the base branch from main to graphite-base/34 January 30, 2026 07:49
@NathanFlurry NathanFlurry force-pushed the feat/include-inspector-ui-in-release branch from 73a0f5a to 272d454 Compare January 30, 2026 07:49
@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-34 January 30, 2026 07:49 Destroyed
@NathanFlurry NathanFlurry changed the base branch from graphite-base/34 to fix/r2-binary-download January 30, 2026 07:49
Copy link
Member Author

NathanFlurry commented Jan 30, 2026

Merge activity

  • Jan 30, 7:57 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 30, 8:06 AM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 30, 8:07 AM UTC: @NathanFlurry merged this pull request with Graphite.

@NathanFlurry NathanFlurry force-pushed the feat/include-inspector-ui-in-release branch from 272d454 to d39b910 Compare January 30, 2026 08:00
@NathanFlurry NathanFlurry force-pushed the fix/r2-binary-download branch from f2ab924 to f73d884 Compare January 30, 2026 08:00
@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-34 January 30, 2026 08:00 Destroyed
@NathanFlurry NathanFlurry changed the base branch from fix/r2-binary-download to graphite-base/34 January 30, 2026 08:04
@NathanFlurry NathanFlurry changed the base branch from graphite-base/34 to main January 30, 2026 08:05
@NathanFlurry NathanFlurry force-pushed the feat/include-inspector-ui-in-release branch from d39b910 to a7b8882 Compare January 30, 2026 08:06
@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-34 January 30, 2026 08:06 Destroyed
@NathanFlurry NathanFlurry merged commit 8aab9e3 into main Jan 30, 2026
4 of 6 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.

2 participants