[Fix] packages/react issue with useQuery not supporting dynamic query parameters

[Chriztiaan]

Original issue reported at #323.
Dynamic query dependencies would warn with the following if the query parameter array changed in size:
Warning: The final argument passed to useMemo changed size between renders. The order and size of this array must remain constant. Previous: [] Incoming: [xxxxxxxxxxxxx]

Tested with the following to verify that it works now:

const [params, setParams] = React.useState<string[]>([]);
const { data } = useQuery(`select * from lists where id=?`, params);

onclick:: setParams(["some-uuid"])

The change from React.useMemo(() => {}, [...array]); to React.useMemo(() => {},[JSON.stringify(array)]) should not introduce any issues, as valid query parameters are expected to be serialisable.

[changeset-bot]

🦋 Changeset detected

Latest commit: af14633

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

This PR includes changesets to release 3 packages

Name	Type
@powersync/react	Patch
@powersync/react-native	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

[DominicGBauer]

We may want to look at something like this in the future https://github.com/sandiiarov/use-deep-compare#readme.

[guillempuche]

Very cool Christian!

[fooware]

I just got hit with this today in our process of porting over our codebase from Electric. Spent quite some time trying to figure out which useMemo it was, until I found this. Thanks for fixing it @Chriztiaan!

Any idea when this could be merged and released? Is there any canary / nightly one could test after it is merged?

[Chriztiaan]

@fooware, I'll release this soonest and ping you.

[fooware]

I can confirm that the console warnings are gone the latest release and the filtering is working as expected.