perf(router-core): skip impossible JSON parses during search parsing - #8013
perf(router-core): skip impossible JSON parses during search parsing#8013Sheraff wants to merge 1 commit into
Conversation
|
View your CI Pipeline Execution ↗ for commit bfb5233
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview2 package(s) bumped directly, 21 bumped as dependents. 🟩 Patch bumps
|
|
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)
📝 WalkthroughWalkthrough
ChangesSearch parameter parsing
Estimated code review effort: 2 (Simple) | ~15 minutes Possibly related PRs
🚥 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 |
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 10.71%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | mem server error-paths unmatched (react) |
272.3 KB | 849.3 KB | -67.93% |
| ❌ | Memory | mem client unique-location-churn (solid) |
352.4 KB | 457.4 KB | -22.96% |
| ❌ | Memory | mem client navigation-churn (solid) |
575 KB | 629 KB | -8.58% |
| ❌ | Memory | mem client unique-location-churn (vue) |
1.2 MB | 1.3 MB | -6.16% |
| ❌ | Memory | mem server server-fn-churn (vue) |
263.2 KB | 274.8 KB | -4.21% |
| ⚡ | Memory | mem server error-paths not-found (solid) |
728.6 KB | 417 KB | +74.72% |
| ⚡ | Memory | mem client preload-churn (vue) |
805.5 KB | 736.6 KB | +9.35% |
| ⚡ | Memory | mem server error-paths not-found (react) |
260.8 KB | 250.3 KB | +4.18% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing perf/router-core-search-parse-fast-path (bfb5233) with main (44a8c3e)
Summary
jsonStartfast path toparseSearchWithwhen usingJSON.parseTesting
CI=1 NX_DAEMON=false pnpm nx run @tanstack/router-core:test:unit --outputStyle=stream --skipRemoteCache -- tests/searchParams.test.tsCI=1 NX_DAEMON=false pnpm nx run @tanstack/router-core:test:types --outputStyle=stream --skipRemoteCacheCI=1 NX_DAEMON=false pnpm nx run @tanstack/router-core:test:eslint --outputStyle=stream --skipRemoteCacheCI=1 NX_DAEMON=false pnpm nx run @tanstack/router-core:test:unit --outputStyle=stream --skipRemoteCache -- bench tests/searchParams.bench.ts --runSummary by CodeRabbit
Bug Fixes
Performance
Tests