Skip to content

Comments

Move frontend build to a separate build stage#2055

Merged
ffont merged 4 commits intomasterfrom
js-build-stage
Feb 16, 2026
Merged

Move frontend build to a separate build stage#2055
ffont merged 4 commits intomasterfrom
js-build-stage

Conversation

@alastair
Copy link
Member

@alastair alastair commented Feb 2, 2026

Allows us to build in parallel, speeding up build process Removes node from our main runtime image

@ffont
Copy link
Member

ffont commented Feb 2, 2026

Thanks for that @alastair! I think it might be great to do something like that, but we should provide an alternative to docker compose run --rm web npm run build as this will not work. This is used in local development when making changes to static src files. I guess we add a new service to the docker compose config which refers to frontend_build docker image stage? (and update the "static build" tasks in vscode workspace file)

Allows us to build in parallel, speeding up build process
Removes node from our main runtime image
@alastair
Copy link
Member Author

@ffont this is resolved - build steps are correct if you want to run manually, and there's a profile to automatically build (taking advantage of the parcel cache) whenever you make a change to a file

@ffont
Copy link
Member

ffont commented Feb 10, 2026

That looks really nice @alastair ! Although the build is failing with something uv related?

@alastair
Copy link
Member Author

yeah, this is a common build issue with pysndfile, I keep running into it. let's see if we can fix it once and for all

pysndfile has a custom setup.py which looks in some fixed locations for
the .so file. Debain puts this file in /usr/lib/x86_64-linux-gnu which
isn't found, so move it to /usr/lib
@alastair
Copy link
Member Author

this is building now, I upgraded to the latest pysndfile.
There's a bug in their build script, but I submitted a patch, hopefully we won't need our workaround in the future. We can wait until next week to release this, in case some unexpected issue comes up with processing due to the upgrade.

@ffont
Copy link
Member

ffont commented Feb 16, 2026

Now it builds in less than 2 seconds, amazing!

@ffont ffont merged commit ef99931 into master Feb 16, 2026
3 checks passed
@ffont ffont deleted the js-build-stage branch February 16, 2026 11:45
@ffont
Copy link
Member

ffont commented Feb 16, 2026

Oups,
I merged without actually testing it, and now I see that the JS has errors :S

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