Solid router main merge#7628
Conversation
ci: changeset release Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* fix(start-rsbuild): compile node_modules for RSC directives * chore: add comments * chore: add changeset
Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com>
…7559) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
* fix(benchmarks): run SSR setup imports natively * fix(benchmarks): update codspeed vitest plugin * fix(benchmarks): avoid SSR CodSpeed setup hooks * ci: apply automated fixes * fix(benchmarks): preload SSR handlers before bench * ci: apply automated fixes * ensure benchmarks build are NODE_ENV=production * fail codspeed on error, to avoid future skips like this one --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
|
View your CI Pipeline Execution ↗ for commit 545866d
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview3 package(s) bumped directly, 19 bumped as dependents. 🟩 Patch bumps
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
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. |
There was a problem hiding this comment.
Important
At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.
Nx Cloud is proposing a fix for your failed CI:
We fixed the solid SSR benchmark type-check failures by updating jsxImportSource from "solid-js" to "@solidjs/web" in all 14 solid scenario tsconfigs, aligning them with how @tanstack/solid-router and @tanstack/solid-start already configure TypeScript for solid-js 2.x. We also replaced the Suspense import and JSX usage in the streaming benchmark with Loading, which is the solid-js 2.x equivalent.
Tip
✅ We verified this fix by re-running tanstack-router-e2e-vue-basic-vue-query-file-based:test:e2e, @benchmarks/ssr-assets-solid:test:types:ssr.
Warning
The suggested diff is too large to display here, but you can view it on Nx Cloud ↗
Or Apply changes locally with:
npx nx-cloud apply-locally YXEb-J3MP
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
Merging this PR will improve performance by 65.52%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | ssr request loop (react) |
330.5 ms | 81.1 ms | ×4.1 |
| ⚡ | Simulation | ssr request loop (vue) |
420.3 ms | 286.7 ms | +46.57% |
| ⚡ | Simulation | client-side navigation loop (solid) |
72.6 ms | 62.7 ms | +15.85% |
| ⚡ | Simulation | ssr request loop (solid) |
174.6 ms | 161 ms | +8.45% |
| 🆕 | Simulation | ssr global-mw server-fn (vue) |
N/A | 60 ms | N/A |
| 🆕 | Simulation | ssr not-found (vue) |
N/A | 168.8 ms | N/A |
| 🆕 | Simulation | ssr redirect (vue) |
N/A | 68.4 ms | N/A |
| 🆕 | Simulation | ssr head (vue) |
N/A | 325.1 ms | N/A |
| 🆕 | Simulation | ssr loaders (vue) |
N/A | 136.6 ms | N/A |
| 🆕 | Simulation | ssr selective (vue) |
N/A | 139.9 ms | N/A |
| 🆕 | Simulation | ssr server-fn GET (vue) |
N/A | 74.5 ms | N/A |
| 🆕 | Simulation | ssr server-fn POST (vue) |
N/A | 71.4 ms | N/A |
| 🆕 | Simulation | ssr server-route middleware (vue) |
N/A | 63.3 ms | N/A |
| 🆕 | Simulation | ssr server-route (vue) |
N/A | 60.9 ms | N/A |
| 🆕 | Simulation | ssr streaming deferred (vue) |
N/A | 105 ms | N/A |
| 🆕 | Simulation | ssr assets inline-css cdn (vue) |
N/A | 187.4 ms | N/A |
| 🆕 | Simulation | ssr assets linked-css control (vue) |
N/A | 214.3 ms | N/A |
| 🆕 | Simulation | ssr before-load chain (vue) |
N/A | 184.2 ms | N/A |
| 🆕 | Simulation | ssr control-flow error 500 (vue) |
N/A | 198.1 ms | N/A |
| 🆕 | Simulation | ssr control-flow route headers (vue) |
N/A | 213.4 ms | N/A |
| ... | ... | ... | ... | ... | ... |
ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.
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 solid-router-main-merge2 (3fe770a) with solid-router-v2-pre (67a9040)1
Footnotes
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| '@tanstack/react-router': patch | |||
There was a problem hiding this comment.
we shouldn't bring changesets from the main branch to this pre branch. It'll trigger react-router betas
No description provided.