Persisted Query Link: improve memory management

[phryneas]

This PR is ready for review & merging :)

Checklist:

• If this PR contains changes to the library itself (not necessary for e.g. docs updates), please include a changeset (see CONTRIBUTING.md)
• If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
• Make sure all of the significant new logic is covered by tests

[changeset-bot]

🦋 Changeset detected

Latest commit: 6d2ec4e

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

This PR includes changesets to release 1 package

Name	Type
@apollo/client	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

[github-actions]

size-limit report 📦

Path	Size
dist/apollo-client.min.cjs	37.63 KB (0%)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs"	44.2 KB (0%)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/main.cjs" (production)	42.67 KB (0%)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js"	32.86 KB (0%)
import { ApolloClient, InMemoryCache, HttpLink } from "dist/index.js" (production)	31.55 KB (0%)
import { ApolloProvider } from "dist/react/index.js"	1.24 KB (0%)
import { ApolloProvider } from "dist/react/index.js" (production)	1.22 KB (0%)
import { useQuery } from "dist/react/index.js"	4.33 KB (0%)
import { useQuery } from "dist/react/index.js" (production)	4.15 KB (0%)
import { useLazyQuery } from "dist/react/index.js"	4.64 KB (0%)
import { useLazyQuery } from "dist/react/index.js" (production)	4.45 KB (0%)
import { useMutation } from "dist/react/index.js"	2.61 KB (0%)
import { useMutation } from "dist/react/index.js" (production)	2.59 KB (0%)
import { useSubscription } from "dist/react/index.js"	2.29 KB (0%)
import { useSubscription } from "dist/react/index.js" (production)	2.25 KB (0%)
import { useSuspenseQuery } from "dist/react/index.js"	4.29 KB (0%)
import { useSuspenseQuery } from "dist/react/index.js" (production)	3.7 KB (0%)
import { useBackgroundQuery } from "dist/react/index.js"	3.87 KB (0%)
import { useBackgroundQuery } from "dist/react/index.js" (production)	3.29 KB (0%)
import { useLoadableQuery } from "dist/react/index.js"	4.11 KB (0%)
import { useLoadableQuery } from "dist/react/index.js" (production)	3.52 KB (0%)
import { useReadQuery } from "dist/react/index.js"	3.01 KB (0%)
import { useReadQuery } from "dist/react/index.js" (production)	2.95 KB (0%)
import { useFragment } from "dist/react/index.js"	2.11 KB (0%)
import { useFragment } from "dist/react/index.js" (production)	2.06 KB (0%)

[plausible-phry]

I have a feeling that .toBeGarbageCollected works locally, but gets flimsy in CI - I'll take another look at that tomorrow.

[phryneas]

The tests failed because we had some kind of concurrency problem - by importing from another test file, all the tests from that other file were added to this file, and then failed for reasons I'll never understand.

That's two hours wasted debugging I'll never get back 🤦

[alessbell]

LGTM!