inline isServer for proper DCE - #7996
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe React, Solid, and Vue router implementations inline ChangesSSR detection
Estimated code review effort: 2 (Simple) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
View your CI Pipeline Execution ↗ for commit 5f744df
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
Bundle Size BenchmarksThis pull request does not affect bundle size in any measured scenario. |
Merging this PR will regress 5 benchmarks
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | mem server error-paths not-found (solid) |
410.8 KB | 845.3 KB | -51.4% |
| ❌ | Memory | mem client unique-location-churn (solid) |
341.6 KB | 566.6 KB | -39.71% |
| ❌ | Memory | mem server error-paths redirect (vue) |
300.2 KB | 442.5 KB | -32.16% |
| ❌ | Memory | mem server peak-large-page (solid) |
813.2 KB | 995.8 KB | -18.34% |
| ❌ | Memory | mem client navigation-churn (solid) |
563.1 KB | 594 KB | -5.2% |
| ⚡ | Memory | mem server server-fn-churn (vue) |
3,927.3 KB | 274.8 KB | ×14 |
| ⚡ | Memory | mem server peak-large-page (react) |
2,064 KB | 952.8 KB | ×2.2 |
| ⚡ | Simulation | ssr request loop (solid) |
152.3 ms | 145.5 ms | +4.66% |
| ⚡ | Memory | mem server error-paths not-found (vue) |
333 KB | 320.5 KB | +3.92% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing bundlesize-inline-isserver (8c88574) with main (4fa1df7)
131278f to
feb9739
Compare
a28b3d3 to
5f744df
Compare
5f744df to
8c88574
Compare
Summary by CodeRabbit