Skip to content

refactor(e2e-rspack): switch to @griffel/webpack-plugin#806

Draft
layershifter wants to merge 2 commits intomicrosoft:mainfrom
layershifter:refactor/e2e-rspack-webpack-plugin
Draft

refactor(e2e-rspack): switch to @griffel/webpack-plugin#806
layershifter wants to merge 2 commits intomicrosoft:mainfrom
layershifter:refactor/e2e-rspack-webpack-plugin

Conversation

@layershifter
Copy link
Member

Summary

  • Switch e2e-rspack from deprecated @griffel/webpack-extraction-plugin + @griffel/webpack-loader to the unified @griffel/webpack-plugin
  • Remove unnecessary IS_RSPACK conditionals in the plugin and loader now that Rspack supports NormalModule.getCompilationHooks and namedChunks.get()
  • Replace resolve aliases with relative imports in e2e test assets (oxc-resolver doesn't see Rspack aliases)

Test plan

  • Webpack unit tests pass (46/46)
  • Rspack e2e test passes (CSS snapshot matches)

🤖 Generated with Claude Code

layershifter and others added 2 commits March 11, 2026 17:21
…o @nx/js:tsc

Replace @nx/vite:build with @nx/js:tsc executor for these three packages.
This uses the TypeScript compiler directly instead of bundling via Rollup,
producing individual output files matching the source structure.

- Update project.json build targets to use @nx/js:tsc
- Update package.json exports to use "default" condition (compatible with bundler resolution)
- Add module: "ESNext" + moduleResolution: "bundler" to tsconfig.lib.json
- Delete vite.config.ts (was build+test), create vitest.config.mts (test-only)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…_RSPACK branches

Switch e2e-rspack from deprecated @griffel/webpack-extraction-plugin + @griffel/webpack-loader
to the unified @griffel/webpack-plugin. Remove unnecessary IS_RSPACK conditionals in the plugin
and loader now that Rspack supports NormalModule.getCompilationHooks and namedChunks.get().

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

📊 Bundle size report

✅ No changes found

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.

1 participant