docs(start): troubleshoot loader serialization failures - #8395
tannerlinsley wants to merge 1 commit into
Conversation
|
View your CI Pipeline Execution ↗ for commit 92bf3f7
☁️ Nx Cloud last updated this comment at |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe change adds loader serialization guidance, a React Start example route, generated route wiring, and tests for SSR dehydration, direct requests, client navigation, hydration, and client interaction. ChangesLoader serialization
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Other Sequence Diagram(s)sequenceDiagram
participant Browser
participant LoaderSerializationRoute
participant ServerRouter
participant LoaderSerializationComponent
Browser->>LoaderSerializationRoute: request /loader-serialization
LoaderSerializationRoute->>ServerRouter: load route data
ServerRouter-->>LoaderSerializationRoute: serialize loader values
LoaderSerializationRoute-->>LoaderSerializationComponent: provide loader data
LoaderSerializationComponent-->>Browser: render values and client counter
Merge Risk: ⚪ Minimal · up to The new loader-serialization example, coverage, and guidance are consistent and ready to merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 5 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 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 |
🚀 Changeset Version Preview3 package(s) bumped directly, 21 bumped as dependents. 🟩 Patch bumps
|
Bundle Size BenchmarksThis pull request does not affect bundle size in any measured scenario. |
🎯 Changes
Explain why a loader returning a plain function can work during browser navigation but fail on an SSR refresh, and show the data-only return value that fixes it. This adds troubleshooting to the existing hydration-errors guide.
A Router SSR regression exercises the failing function value and the accepted string result. A Start browser fixture verifies Date, Map, Set, BigInt, and undefined through server rendering, hydration, and client navigation. The navigation test also checks that the document stays loaded, and an interactive counter verifies hydration.
This complements #8162, which updates the serialization reference and documents adapters. It does not duplicate that reference edit. Related report: #7843.
Validation: 2 focused serializer tests and 8 browser cases across Vite/Rsbuild SSR/SPA passed. Required lint, type, and unit checks and formatting passed.
✅ Checklist
🚀 Release Impact
Summary by CodeRabbit
Documentation
Examples & Tests