Skip to content

chore: remove @griffel/webpack-extraction-plugin and @griffel/webpack-loader - #1074

Draft
Oleksandr Fediashov (layershifter) with Copilot wants to merge 2 commits into
mainfrom
copilot/rebase-cleanup-prepare-merge
Draft

chore: remove @griffel/webpack-extraction-plugin and @griffel/webpack-loader#1074
Oleksandr Fediashov (layershifter) with Copilot wants to merge 2 commits into
mainfrom
copilot/rebase-cleanup-prepare-merge

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes the legacy Webpack integration packages, which are superseded by @griffel/webpack-plugin:

  • Deleted @griffel/webpack-extraction-plugin and @griffel/webpack-loader — sources, tests, __fixtures__, project/tsconfig files, READMEs and changelogs.
  • Dropped their paths entries from tsconfig.base.json and their workspace entries from yarn.lock.
  • Removed the pending change/@griffel-webpack-extraction-plugin/* and change/@griffel-webpack-loader/* files, and added a change file for @griffel/webpack-plugin.

Code changes

  • packages/webpack-plugin/src/GriffelPlugin.mts: the Rspack optimization.splitChunks error message now refers to @griffel/webpack-plugin instead of the removed @griffel/webpack-extraction-plugin.

E2E (Rspack)

  • e2e/rspack: the legacy-rspack-1 / legacy-css-extract-rspack-1 scenarios are replaced by modern-rspack-1 / modern-css-extract-rspack-1 running on @griffel/webpack-plugin; legacy-rspack-2 (and its snapshot) is deleted, snapshots and Rspack configs renamed accordingly.
  • The scenario package list is consolidated into a single GRIFFEL_PACKAGES constant, and the npmResolutions / LEGACY_BABEL_RESOLUTIONS workaround (needed only for the Babel-based legacy chain, see Not compatible with Babel 8 #993) is gone.

Docs

  • README.md, .github/copilot-instructions.md, e2e/rspack/README.md, and the babel-preset, webpack-plugin, css-extraction-utils, sort-css-queries READMEs now point at @griffel/webpack-plugin; CSS extraction is no longer labelled "experimental".

Continues the work started in #809.

Copilot AI and others added 2 commits August 4, 2026 16:01
…-loader

Co-authored-by: layershifter <14183168+layershifter@users.noreply.github.com>
…babel-preset READMEs

Co-authored-by: layershifter <14183168+layershifter@users.noreply.github.com>
@layershifter

Copy link
Copy Markdown
Contributor

Copilot PR's description is not accurate to changes

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

📊 Bundle size report

✅ No changes found

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Copilot PR's description is not accurate to changes

Updated the PR description to reflect what the branch actually does: removal of @griffel/webpack-extraction-plugin and @griffel/webpack-loader (sources, fixtures, configs, changelogs, tsconfig paths, pending change files), the GriffelPlugin Rspack error message fix, the migration of the Rspack e2e scenarios from legacy-* to modern-* on @griffel/webpack-plugin, and the doc updates.

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