WIP: Upgrade to solidjs beta.12#7415
Conversation
|
Important Review skippedDraft detected. 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 |
|
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.
|
|
View your CI Pipeline Execution ↗ for commit 927fa5a
☁️ 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
Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better. |
Merging this PR will improve performance by 63.33%
Performance Changes
Tip Curious why this is faster? Comment Comparing Footnotes |
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 a cache-corruption bug introduced by the new matchedRoutesCache in router-core/src/router.ts. The cached routeParams object was being mutated in-place by Object.assign(routeParams, strictParams) in matchRoute, causing subsequent cache hits to return already-parsed params (e.g. { postId: '2', status: 'parsed' }) as the raw input to parseParams instead of the original URL params ({ postId: '2' }). Returning a fresh copy of routeParams on cache hits prevents this mutation from propagating back into the cache, fixing the failing unit tests across @tanstack/react-router, @tanstack/vue-router, and @tanstack/solid-router.
Tip
✅ We verified this fix by re-running @tanstack/react-router:test:unit, @tanstack/solid-router:test:unit, @tanstack/vue-router:test:unit.
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 yJni-iCyH
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
No description provided.