`data-source` breaks React Query default behavior: manual `refetch()` for queries with `enabled: false` no longer works as expected

[arteria32]

In React Query, a query with enabled: false does not fetch data automatically, but can still be refetched manually.

The @gravity-ui/data-source wrapper overrides this behavior — see this change.

I suggest either reverting this change or documenting the breaking change more clearly in the minor release notes.

[DakEnviy]

It is a bad practice to call refetch for disabled queries. Quote from the tanstack documentation:

Permanently disabling a query opts out of many great features that TanStack Query has to offer (like background refetches), and it's also not the idiomatic way. It takes you from the declarative approach (defining dependencies when your query should run) into an imperative mode (fetch whenever I click here)

So, current behaviour is expected and intentionally overridden because this approach leads to bugs. If you want to fetch data imperatively, you should not use react-query or data-source. Just call your fetch function without any wrapper or hook.

[arteria32]

It is a bad practice to call refetch for disabled queries. Quote from the tanstack documentation:

Permanently disabling a query opts out of many great features that TanStack Query has to offer (like background refetches), and it's also not the idiomatic way. It takes you from the declarative approach (defining dependencies when your query should run) into an imperative mode (fetch whenever I click here)

So, current behaviour is expected and intentionally overridden because this approach leads to bugs. If you want to fetch data imperatively, you should not use react-query or data-source. Just call your fetch function without any wrapper or hook.

Ok, thank you for your answer.
Lets add a clearer breaking-change entry to the changelog to help library users avoid bugs

[DakEnviy]

Done here #54