Skip to content

vanilla-extract: Inject filescopes into bundled Vanilla Extract files#247

Merged
askoufis merged 1 commit intomasterfrom
bundle-ve-with-filescopes
Nov 25, 2025
Merged

vanilla-extract: Inject filescopes into bundled Vanilla Extract files#247
askoufis merged 1 commit intomasterfrom
bundle-ve-with-filescopes

Conversation

@askoufis
Copy link
Copy Markdown
Contributor

@askoufis askoufis commented Nov 25, 2025

In situations where a Vanilla Extract plugin is not configured to process files inside node_modules, Styles were unable to be assigned to a file errors can be thrown.

This change uses a new feature in the VE rollup plugin to inject filescopes into VE modules. Before #239, crackle handled injecting these filescopes.

See the snapshot files here (e.g. dist/capsize.css.mjs).

@askoufis askoufis requested a review from a team as a code owner November 25, 2025 03:30
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Nov 25, 2025

🦋 Changeset detected

Latest commit: a5f8cbe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@capsizecss/vanilla-extract Patch

Not sure what this means? Click here to learn what changesets are.

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

@askoufis askoufis changed the title Bundle @capsizecss/vanilla-extract with filescopes `vanilla-extract: Inject filescopes into bundled Vanilla Extract files Nov 25, 2025
@askoufis askoufis changed the title `vanilla-extract: Inject filescopes into bundled Vanilla Extract files vanilla-extract: Inject filescopes into bundled Vanilla Extract files Nov 25, 2025
@askoufis
Copy link
Copy Markdown
Contributor Author

Tested against an internal repo and it worked 🎉

@askoufis askoufis merged commit 76519cd into master Nov 25, 2025
8 checks passed
@askoufis askoufis deleted the bundle-ve-with-filescopes branch November 25, 2025 05:44
@seek-oss-ci seek-oss-ci mentioned this pull request Nov 25, 2025
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