Skip to content

Remove the pluggable asset system#2017

Merged
jwoertink merged 1 commit intomainfrom
remove_pluggable_asset_builder
Feb 25, 2026
Merged

Remove the pluggable asset system#2017
jwoertink merged 1 commit intomainfrom
remove_pluggable_asset_builder

Conversation

@jwoertink
Copy link
Member

Purpose

Reverting most of #1982

Description

We originally wanted some built-in way to tell your Lucky app which asset build system you were going to use. Then based on that, we could internally decide how to build things out.

However, once that was merged in, and we started working out Vite, things got super messy and complicated and broken real fast.

After all that, @wout came up with this absolutely brilliant idea to use bun as the default (#2015). I wanted to do this anyway, and it ended up being a much better solution anyway.

So now I'm reverting the bulk of #1982 for the time being just so we can move on and do a release without this stuff getting in the way. Eventually we can take a second stab at a pluggable system.

Copy link
Contributor

@robacarp robacarp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pluggable systems are an incredible lift, ask me how I know. It's far more sustainable to support one path deeply, and then have an escape hatch for when folks want to jump off the beaten path.

@jwoertink jwoertink merged commit d87f791 into main Feb 25, 2026
9 checks passed
@jwoertink jwoertink deleted the remove_pluggable_asset_builder branch February 25, 2026 15:57
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.

3 participants