Avoid calling hooks conditionally

[simolus3]

In useQuery, we used to invoke the inner useWatchedQuery or useSingleQuery hooks depending on options passed to the useQuery hook.

This violates the rules of hooks, because they must be invoked unconditionally. Mixing them like this can trip up the internal state of useEffect hooks and others, causing all kinds of issues that are hard to debug.

So to avoid this, we add an active: boolean parameter to the internal hooks to conditionally disable them. Then, we can just call both hooks all the time since the inactive one won't actually do anything. I've added a test that fails with the old implementation but works with the new one.

[changeset-bot]

🦋 Changeset detected

Latest commit: b364169

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
@powersync/react	Patch
@powersync/react-native	Patch
@powersync/tanstack-react-query	Patch
@powersync/diagnostics-app	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[stevensJourney]

I initially didn't think there was much of a use-case for switching the runQueryOnce setting, but this is a good improvement which does cater for changing the behaviour (which some users do implement). I'm happy with the changes here.