perf(router-core): inline getOpenAndCloseBraces in parseSegment - #8071
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 (3)
📝 WalkthroughWalkthrough
ChangesRoute parser behavior
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR makes a localized parser performance change with accompanying tests and release notes; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 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 ab34f9c
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 22 bumped as dependents. 🟩 Patch bumps
|
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 regress 0 benchmarks
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Memory | mem server request-churn (solid) |
3,350.3 KB | 713.4 KB | ×4.7 |
| ⚡ | Memory | mem server error-paths unmatched (vue) |
2,726.6 KB | 592.4 KB | ×4.6 |
| ⚡ | Memory | mem server request-churn (react) |
2,254 KB | 666.6 KB | ×3.4 |
| ⚡ | Memory | mem server peak-large-page (solid) |
1.5 MB | 1.1 MB | +28.57% |
| ⚡ | Memory | mem server error-paths redirect (vue) |
474.1 KB | 405.3 KB | +16.97% |
| ⚡ | Memory | mem server error-paths redirect (react) |
338.9 KB | 300 KB | +12.95% |
| ⚡ | Memory | mem server server-fn-churn (react) |
407.1 KB | 364.5 KB | +11.7% |
| ⚡ | Memory | mem client navigation-churn (solid) |
694.8 KB | 664.7 KB | +4.54% |
| ⚡ | Simulation | client-control-flow navigation loop (react) |
177.9 ms | 171.1 ms | +3.99% |
| ⚡ | Memory | mem server aborted-requests (react) |
885.9 KB | 853.7 KB | +3.78% |
| 👁 | Memory | mem server error-paths not-found (solid) |
574.5 KB | 825.9 KB | -30.44% |
| 👁 | Simulation | client-nested-params navigation loop (react) |
212.6 ms | 228.7 ms | -7.07% |
| 👁 | Memory | mem server aborted-requests (vue) |
1,010.2 KB | 1,145.9 KB | -11.84% |
| 👁 | Memory | mem server error-paths not-found (vue) |
463.5 KB | 488.1 KB | -5.04% |
| 👁 | Memory | mem server serialization-payload (vue) |
4.2 MB | 4.4 MB | -3.23% |
| 👁 | Memory | mem server error-paths unmatched (react) |
438.8 KB | 907.6 KB | -51.65% |
| 👁 | Memory | mem server peak-large-page (react) |
1.1 MB | 1.2 MB | -5.23% |
| 👁 | Memory | mem client unique-location-churn (vue) |
489.8 KB | 540.3 KB | -9.34% |
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-parse-segment-inline-braces-extraction (ab34f9c) with main (53b63cf)
Summary by CodeRabbit
Bug Fixes
Tests
Chores