Skip to content

fix(svelte-scoped): replace build hook with buildEnd to remove setAss…#5159

Merged
zyyv merged 1 commit intounocss:mainfrom
henrikvilhelmberglund:fix-remove-sveltescoped-remove-setassetsource
Apr 7, 2026
Merged

fix(svelte-scoped): replace build hook with buildEnd to remove setAss…#5159
zyyv merged 1 commit intounocss:mainfrom
henrikvilhelmberglund:fix-remove-sveltescoped-remove-setassetsource

Conversation

@henrikvilhelmberglund
Copy link
Copy Markdown
Contributor

…etSource causing failed builds in vite 8

setAssetSource was removed (well, not implemented) in Rolldown which caused failed builds in Svelte-scoped when upgrading to Vite 8. It is unclear how willing they are to reimplement it since Vite 8 was released anyway, but in this case there was no real need for it in the first place.

This PR moves everything into buildEnd so the on-demand theme functionality in presetWind4 continues working without problems.

fixes #5158

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 18, 2026

Deploy Preview for unocss ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 94262dc
🔍 Latest deploy log https://app.netlify.com/projects/unocss/deploys/69bb1e7dcccd7d0008a8a2e1
😎 Deploy Preview https://deploy-preview-5159--unocss.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 18, 2026

Open in StackBlitz

commit: 94262dc

@zyyv zyyv added this pull request to the merge queue Apr 7, 2026
Merged via the queue into unocss:main with commit 17513da Apr 7, 2026
11 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.

Svelte-scoped building broken in Vite 8

2 participants