fix(router): align pending presentation across Solid and Vue - #8097
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 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 1f477f8
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 4 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 6 benchmarks
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | mem server error-paths redirect (solid) |
391.8 KB | 432 KB | -9.32% |
| ❌ | Memory | mem client navigation-churn (vue) |
1.5 MB | 1.6 MB | -6.58% |
| ❌ | Memory | mem server request-churn (vue) |
826 KB | 860.6 KB | -4.02% |
| ❌ | Simulation | client-async-pipeline navigation loop (react) |
104.5 ms | 108.6 ms | -3.75% |
| ❌ | Memory | mem server error-paths unmatched (solid) |
573.6 KB | 595.9 KB | -3.74% |
| ❌ | Simulation | ssr control-flow error 500 (react) |
227.8 ms | 235.3 ms | -3.17% |
| ⚡ | Memory | mem server error-paths unmatched (react) |
789.7 KB | 588.3 KB | +34.25% |
| ⚡ | Memory | mem server error-paths redirect (vue) |
467.9 KB | 411.3 KB | +13.77% |
| ⚡ | Memory | mem server error-paths not-found (solid) |
623.9 KB | 564.6 KB | +10.49% |
| ⚡ | Memory | mem client unique-location-churn (vue) |
575.3 KB | 528.6 KB | +8.82% |
| ⚡ | Simulation | ssr global-mw document (vue) |
970.4 ms | 906.8 ms | +7.01% |
| ⚡ | Memory | mem server peak-large-page (solid) |
1.1 MB | 1 MB | +6.63% |
| ⚡ | Memory | mem server error-paths not-found (react) |
457.9 KB | 436 KB | +5.03% |
| ⚡ | Memory | mem server aborted-requests (vue) |
983.8 KB | 950.5 KB | +3.5% |
| ⚡ | Memory | mem server error-paths unmatched (vue) |
597.9 KB | 579.9 KB | +3.1% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing opencode/pr-8084-solid-vue-test-port (1f477f8) with opencode/retained-pending-architecture-v5 (425bab3)
121fe64
into
opencode/retained-pending-architecture-v5
* fix(router-core): preserve pending UI across retained routes * fix(router-core): harden pending session ownership * fix(router): align pending presentation across Solid and Vue (#8097) fix(router): align pending presentation across frameworks * changesets * test: align hydration fake timers
Summary
Testing