Build the browser UI in release CLI builds so studio ui ships working - #4257
Merged
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Collaborator
📊 Performance Test ResultsComparing 0b4cff7 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
… on Windows Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
sejas
approved these changes
Jul 21, 2026
sejas
left a comment
Member
There was a problem hiding this comment.
I tested it and I confirm I was able to see apps/cli/dist/cli/ui/index.local.html after running rm -rf apps/ui/dist-local && npm run cli:build:npm. It also started the UI correctly in the browser.
I noticed that a warning appears when running npm run cli:build. I think the warning already existed, but now is more visible. I don't see any issue or delay when loading the UI.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related issues
studio uicommand shipping broken in published CLIs (no issue filed).How AI was used in this PR
Claude Code investigated the release pipeline, confirmed the gap by inspecting the published
wp-studio@1.15.0registry tarball, wrote the fix, and verified the builds end-to-end (build outputs,npm packcontents, andstudio uiserving the UI).Proposed Changes
The published
wp-studionpm package ships thestudio uicommand but not the browser UI it serves:wp-studio@1.15.0on npm contains noui/assets, so runningstudio uifrom an npm install responds with "Cannot GET /". The root cause is that nothing in the release pipeline ever buildsapps/ui— the CLI build only copiesdist-localif it happens to already exist, and silently skips it otherwise.studio uican no longer ship silently.npm run cli:build:uiconvenience script that chains the UI build and the CLI build for anyone who wants the fullstudio uiexperience from a dev build.Testing Instructions
rm -rf apps/ui/dist-local && npm run cli:build:npm— the build should build the UI itself, andapps/cli/dist/cli/ui/index.local.htmlshould exist afterwards.npm pack -w wp-studio --dry-run | grep dist/cli/ui— the tarball file list should include the UI assets.npm run cli:build:ui && node apps/cli/dist/cli/main.mjs ui— the browser should show the Studio UI at/(not "Cannot GET /").rm -rf apps/ui/dist-local apps/cli/dist && npm run cli:buildshould succeed without building the UI.Pre-merge Checklist
🤖 Generated with Claude Code