Skip to content

[UE5.7] build(react): output bundle to its own dist/ instead of SignallingWebServer/www/ (#874)#880

Merged
mcottontensor merged 1 commit intoUE5.7from
backport/UE5.7/pr-874
May 5, 2026
Merged

[UE5.7] build(react): output bundle to its own dist/ instead of SignallingWebServer/www/ (#874)#880
mcottontensor merged 1 commit intoUE5.7from
backport/UE5.7/pr-874

Conversation

@mcottontensor
Copy link
Copy Markdown
Collaborator

Backport

This will backport the following commits from master to UE5.7:

Questions ?

Please refer to the Backport tool documentation

…Server/www/ (#874)

Both reference frontend implementations were writing to
SignallingWebServer/www/ with `clean: true`, so a top-level
`npm run build --ws` ran the TS impl first and then had its output
wiped by the React impl that ran second. Wilbur was then served the
React impl's minimal page (auto-connect, no UI overlay) instead of
the TS reference frontend.

Point the React webpack at a local dist/ directory so the two
implementations no longer collide. WEBPACK_OUTPUT_PATH is honoured
for symmetry with the TS impl, and the readme documents how to have
Wilbur serve this bundle via --http_root.

(cherry picked from commit 1096245)
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 5, 2026

⚠️ No Changeset found

Latest commit: 10ed1ec

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mcottontensor mcottontensor merged commit 91b1334 into UE5.7 May 5, 2026
7 checks passed
@mcottontensor mcottontensor deleted the backport/UE5.7/pr-874 branch May 5, 2026 01:47
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