Sync main to solidl v2 pre#7522
Conversation
Put Start project setup in a single onboarding path for React and Solid while preserving old quick-start URLs with redirects.
Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com>
Co-authored-by: schiller-manuel <schiller-manuel@users.noreply.github.com> Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com>
Co-authored-by: schiller-manuel <6340397+schiller-manuel@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
ci: changeset release Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* ci: update all actions * disable package manager cache
* fix: fix primitive beforeLoad errors * ci: apply automated fixes --------- 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>
|
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 Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 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 cf29c6e
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
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 not alter performance
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ❌ | ssr request loop (vue) |
420.3 ms | 482 ms | -12.8% |
| ⚡ | client-side navigation loop (solid) |
72.6 ms | 66.3 ms | +9.62% |
| ⚡ | ssr request loop (solid) |
174.6 ms | 161.9 ms | +7.84% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing sync-main-to-solidl-v2-pre (cf29c6e) with solid-router-v2-pre (67a9040)2
Footnotes
-
1 benchmark was skipped, so the baseline result was used instead. If it was deleted from the codebase, click here and archive it to remove it from the performance reports. ↩
-
No successful run was found on
solid-router-v2-pre(cf29c6e) during the generation of this report, so 67a9040 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
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 fix the failing @tanstack/solid-start-client:test:eslint task by moving the misplaced eslint-disable-next-line comment in GenericHydrate.tsx one line down, so it directly precedes the (isServer ?? typeof window === 'undefined') expression that triggers @typescript-eslint/no-unnecessary-condition. Previously the comment applied to the outer ( on the next line — an innocuous character — leaving the actual ESLint violation uncovered and generating both an "unused directive" warning and the original error.
Tip
✅ We verified this fix by re-running @tanstack/solid-start-client:test:eslint.
diff --git a/packages/solid-start-client/src/GenericHydrate.tsx b/packages/solid-start-client/src/GenericHydrate.tsx
index b959f161..3e6bb08b 100644
--- a/packages/solid-start-client/src/GenericHydrate.tsx
+++ b/packages/solid-start-client/src/GenericHydrate.tsx
@@ -355,8 +355,8 @@ export function GenericHydrate(props: InternalHydrateProps) {
Solid.createRenderEffect(
() =>
!(
- // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
(
+ // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
(isServer ?? typeof window === 'undefined') ||
gate.resolved ||
initialHydrateStrategy._t === 'never' ||
Or Apply changes locally with:
npx nx-cloud apply-locally s89H-1h8Q
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
merge main to solid-router-v2-pre