Conversation
|
View your CI Pipeline Execution ↗ for commit 31fd3bc
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 24 bumped as dependents. 🟩 Patch bumps
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: TanStack/query/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe devtools resolves disabled-query visibility from an explicit local-storage value or, when none is set, from the ChangesDisabled-query visibility
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The disabled-query setting appears to behave as intended. No merge-blocking issue is identified. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The change affects which queries appear in the devtools list, not query execution or access controls. No security issue was established, but review coverage does not justify a minimal-risk assessment. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 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 |
size-limit report 📦
|
🎯 Changes
The
hideDisabledQueriesprop is accepted by the framework devtools (ReactQueryDevtools,PreactQueryDevtools, etc.) and forwarded toquery-devtools, but the query list inDevtools.tsxonly read thehideDisabledQueriessetting from local storage, so passing the prop had no effect.ContentViewnow resolves the effective value with ahideDisabledQueriesmemo: the local storage setting (changed from the settings menu) if set, otherwise the prop, otherwisefalse. This follows the same precedence aspositionandinitialIsOpen. Both the query list filter and the settings menu's Show/Hide selection use this value.Adds tests that the prop hides disabled queries, and that the local storage setting takes precedence over the prop.
✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit
hideDisabledQueriesprop. If no valid saved preference is available, the prop controls visibility; when neither setting is provided, disabled queries remain visible.