fix(react-query-next-experimental): replace deprecated 'isServer' with 'environmentManager.isServer()'#10857
Conversation
…h 'environmentManager.isServer()'
📝 WalkthroughWalkthroughThis PR migrates server environment detection in ChangesDeprecation Migration: isServer → environmentManager.isServer()
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 79db658
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview4 package(s) bumped directly, 21 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
🎯 Changes
Migrate
react-query-next-experimentalfrom the deprecatedisServerexport toenvironmentManager.isServer(), matching the migration already done forreact-query/preact-queryin #10199 and forvue-queryin #10826.Updated files:
packages/react-query-next-experimental/src/HydrationStreamProvider.tsxpackages/react-query-next-experimental/src/ReactQueryStreamedHydration.tsxNo behavior change —
environmentManager.isServer()calls the same underlying check by default while allowing host environments (frameworks, SSR runtimes, tests) to override the server detection.✅ Checklist
🚀 Release Impact
Summary by CodeRabbit
@tanstack/react-query-next-experimentalto use the current environment detection method instead of the deprecated approach, ensuring better long-term compatibility and alignment with package standards.