fix: scope rspack overrides to preserve rsbuild2 dependencies#2497
fix: scope rspack overrides to preserve rsbuild2 dependencies#2497colinaaa merged 2 commits intolynx-family:mainfrom
Conversation
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdjusts pnpm workspace override rules in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Merging this PR will degrade performance by 77.36%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ⚡ | 002-hello-reactLynx-destroyBackground |
894.7 µs | 670.1 µs | +33.51% |
| ❌ | basic-performance-large-css |
15.3 ms | 67.6 ms | -77.36% |
Comparing colinaaa:fix/rspack-overrides-v1-scope (562e504) with main (02be891)
Footnotes
-
26 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
React External#591 Bundle Size — 583.27KiB (0%).562e504(current) vs 02be891 main#571(baseline) Bundle metrics
|
| Current #591 |
Baseline #571 |
|
|---|---|---|
0B |
0B |
|
0B |
0B |
|
0% |
0% |
|
0 |
0 |
|
3 |
3 |
|
17 |
17 |
|
5 |
5 |
|
8.59% |
8.59% |
|
0 |
0 |
|
0 |
0 |
Bundle analysis report Branch colinaaa:fix/rspack-overrides-v1... Project dashboard
Generated by RelativeCI Documentation Report issue
React Example#7473 Bundle Size — 224.41KiB (0%).562e504(current) vs 02be891 main#7453(baseline) Bundle metrics
|
| Current #7473 |
Baseline #7453 |
|
|---|---|---|
0B |
0B |
|
0B |
0B |
|
0% |
0% |
|
0 |
0 |
|
4 |
4 |
|
179 |
179 |
|
69 |
69 |
|
44.5% |
44.5% |
|
2 |
2 |
|
0 |
0 |
Bundle size by type no changes
| Current #7473 |
Baseline #7453 |
|
|---|---|---|
145.76KiB |
145.76KiB |
|
78.65KiB |
78.65KiB |
Bundle analysis report Branch colinaaa:fix/rspack-overrides-v1... Project dashboard
Generated by RelativeCI Documentation Report issue
React MTF Example#606 Bundle Size — 195.57KiB (0%).562e504(current) vs 02be891 main#586(baseline) Bundle metrics
|
| Current #606 |
Baseline #586 |
|
|---|---|---|
0B |
0B |
|
0B |
0B |
|
0% |
0% |
|
0 |
0 |
|
3 |
3 |
|
173 |
173 |
|
66 |
66 |
|
43.99% |
43.99% |
|
2 |
2 |
|
0 |
0 |
Bundle size by type no changes
| Current #606 |
Baseline #586 |
|
|---|---|---|
111.23KiB |
111.23KiB |
|
84.34KiB |
84.34KiB |
Bundle analysis report Branch colinaaa:fix/rspack-overrides-v1... Project dashboard
Generated by RelativeCI Documentation Report issue
Web Explorer#9047 Bundle Size — 898.15KiB (0%).562e504(current) vs 02be891 main#9028(baseline) Bundle metrics
|
| Current #9047 |
Baseline #9028 |
|
|---|---|---|
44.47KiB |
44.47KiB |
|
2.22KiB |
2.22KiB |
|
0% |
0% |
|
9 |
9 |
|
11 |
11 |
|
229 |
229 |
|
11 |
11 |
|
27.22% |
27.22% |
|
10 |
10 |
|
0 |
0 |
Bundle size by type no changes
| Current #9047 |
Baseline #9028 |
|
|---|---|---|
494.3KiB |
494.3KiB |
|
401.63KiB |
401.63KiB |
|
2.22KiB |
2.22KiB |
Bundle analysis report Branch colinaaa:fix/rspack-overrides-v1... Project dashboard
Generated by RelativeCI Documentation Report issue
Summary
@rspack/core@^1.7.0and@rsbuild/core@1instead of overriding all@rspack/coreresolutions@rsdoctor/*pinned to the workspace@rspack/corewhile allowing rsbuild2/rspress to resolve their required rspack v2 dependencyVerification
pnpm dedupe --checkpnpm turbo buildSummary by CodeRabbit