perf(vue-router): share blocker subscription logic - #7979
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 72967a3
☁️ 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 Benchmarks
The following scenarios have bundle-size changes compared with the baseline:
Current gzip tracks all emitted client JS chunks. Initial gzip tracks only the entry/import graph. Trend sparkline is historical current gzip ending with this PR measurement; lower is better. |
Merging this PR will degrade performance by 5.82%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | mem server error-paths redirect (vue) |
358.2 KB | 740.3 KB | -51.62% |
| ❌ | Memory | mem server error-paths redirect (solid) |
273.1 KB | 448.4 KB | -39.1% |
| ❌ | Memory | mem server error-paths unmatched (react) |
263.2 KB | 282.7 KB | -6.89% |
| ❌ | Memory | mem server peak-large-page (solid) |
788.1 KB | 821.5 KB | -4.06% |
| ❌ | Memory | mem server server-fn-churn (react) |
265 KB | 275.6 KB | -3.84% |
| ❌ | Simulation | ssr request loop (solid) |
157.7 ms | 163.4 ms | -3.49% |
| ⚡ | Memory | mem server request-churn (react) |
940 KB | 503.9 KB | +86.54% |
| ⚡ | Memory | mem client unique-location-churn (react) |
762.5 KB | 686.4 KB | +11.09% |
| ⚡ | Memory | mem server server-fn-churn (vue) |
274.7 KB | 263.1 KB | +4.4% |
| ⚡ | Memory | mem server server-fn-churn (solid) |
274.9 KB | 264.6 KB | +3.91% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing codex/vue-blocker-isolated (72967a3) with main (abf9b81)
Footnotes
-
6 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. ↩
Summary
Bundle impact
Fresh paired 17-scenario matrix against 697ebb6:
The minimal Vue scenario remains unchanged, confirming that the shared private implementation does not leak across the tree-shaking boundary.
Validation