Skip to content

Conversation

@bashmish
Copy link
Member

@bashmish bashmish commented Dec 23, 2025

Closes #2152

What I did

  1. Enabled CSS assets extraction by default (as a result, removed configuration option bundleAssetsFromCss).
  2. Made extraction of assets from all link rel types.
  3. Fixed "assetFileNames" behavior.
  4. Refactored all tests, added tests for more corner cases.
  5. Added legacy modes for old 2.x.x behavior.

See MIGRATION.md for migration notes.

@changeset-bot
Copy link

changeset-bot bot commented Dec 23, 2025

🦋 Changeset detected

Latest commit: b00f8fb

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

This PR includes changesets to release 3 packages
Name Type
@web/rollup-plugin-html Major
@web/dev-server-storybook Patch
@web/storybook-builder 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

@bashmish bashmish requested a review from thepassle December 23, 2025 09:49
rollupPluginHTML({
input: { html: await generateIframeHtml(options), name: 'iframe.html' },
extractAssets: true,
bundleAssetsFromCss: true,
Copy link
Member Author

Choose a reason for hiding this comment

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

I think I can't make a release of storybook-builder in this PR, because it's unrelated, but due to tight coupling in the monorepo, I had to remove this option in here to prevent TS errors

therefore I'll make sure to create a follow-up PR to update the storybook-builder right after this PR is merged and 3.x.x is released

@bashmish bashmish marked this pull request as draft December 23, 2025 09:58
@bashmish bashmish marked this pull request as ready for review December 24, 2025 09:30
@bashmish bashmish force-pushed the feat/improve-assets-handling branch from 26e4e0d to a07c36e Compare December 24, 2025 12:04
@bashmish bashmish force-pushed the feat/improve-assets-handling branch from 051ad6e to 72ea214 Compare December 24, 2025 12:40
@bashmish bashmish merged commit d5ee691 into master Dec 24, 2025
9 of 14 checks passed
@bashmish bashmish deleted the feat/improve-assets-handling branch December 24, 2025 13:55
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.

Erratic treatment of assets

3 participants