docs(config.json): register missing svelte API reference functions in sidebar - #11192
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan includes up to 10 reviews per rolling hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe Svelte API reference navigation now includes entries for ChangesSvelte API navigation
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized documentation change adds missing Svelte API reference links without changing published code or runtime behavior; no actionable merge-blocking risk remains. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 e4c58e7
☁️ 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. |
size-limit report 📦
|
🎯 Changes
The Svelte API Reference sidebar (
docs/config.json) was missing links to two functions that exist underdocs/framework/svelte/reference/functions/(generated viapnpm run generate-docs) and are publicly exported from@tanstack/svelte-query:useQueryClient— already registered for React, Vue, and Preact; Svelte was the outlier missing ituseHydrate— Svelte's function-based equivalent of theHydrationBoundary/hydration API that other frameworks expose in their reference docsOther candidates found in the same gap analysis were intentionally excluded:
useIsRestoring— not registered in React's reference sidebar either; it's documented inline in thepersistQueryClientplugin guide instead, which Svelte doesn't currently havegetQueryClientContext/setQueryClientContext/getIsRestoringContext/setIsRestoringContext— internal plumbing used byuseQueryClientandQueryClientProvider.sveltethemselves, not user-facing APIs, with no usage in any guide or plugin docNeither of the two added functions is marked
@deprecatedin source or struck through inframework/svelte/reference/index.md.✅ Checklist
pnpm run test:pr. (docs-only change, no code/tests affected)🚀 Release Impact
Summary by CodeRabbit
useQueryClientanduseHydrate.