perf(router-core): cache lightweight route matches#7601
Conversation
📝 WalkthroughWalkthroughThis PR introduces a WeakMap-based result cache to ChangesLightweight Route Matching Cache
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 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 docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 91b3196
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview3 package(s) bumped directly, 10 bumped as dependents. 🟩 Patch bumps
|
Bundle Size Benchmarks
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 improve performance by 5.07%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | client-side navigation loop (vue) |
85.2 ms | 79 ms | +7.78% |
| ⚡ | Simulation | ssr request loop (react) |
81.1 ms | 77.9 ms | +4.17% |
| ⚡ | Simulation | client-side navigation loop (solid) |
116.9 ms | 113.2 ms | +3.33% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing perf/router-core-lightweight-cache (91b3196) with main (36ed574)
Summary
Testing
Summary by CodeRabbit