feat: simplify staleTime for queryOptions

[Nr9]

Greptile Overview

Greptile Summary

Refactored the staleTime logic in observableQueryOptions to directly check for active RxJS subscriptions instead of inferring state from query metadata. The new implementation checks if a subscription exists in the queriesSubscriptions Map using hashKey(queryKey), which is more explicit and aligns better with the actual lifecycle of observable subscriptions.

Key Changes:

• Replaced state.dataUpdateCount and state.status checks with direct queriesSubscriptions.has() lookup
• Updated test to properly simulate an active subscription by adding it to the Map
• Improved test description to clarify it's testing active subscription behavior

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change simplifies the staleTime logic by directly checking the source of truth (the subscriptions Map) rather than indirectly inferring from query state. The implementation is clearer, more maintainable, and the test coverage validates the new behavior correctly
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
packages/react-query-observable/src/observableQueryOptions.ts	5/5	Simplified staleTime logic to check active subscriptions directly instead of relying on query state
packages/react-query-observable/src/tests/observableQueryOptions.test.ts	5/5	Updated test to properly setup subscription in Map before checking staleTime behavior

[greptile-apps]

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}