[3.8 beta] useBackgroundQuery/useReadQuery compatibility with @apollo/experimental-nextjs-app-support

[alessbell]

Issue Description

See discussion: #10755 (comment)

Link to Reproduction

N/A

Reproduction Steps

No response

[alessbell]

• hide queryref in a Symbol that AC declares -> might rename existing QueryRef to InternalQueryRef
• PR to experimental package

[alessbell]

• we can only flush data over to the client when a Suspense boundary finishes
• for these two hooks, some scenarios where this could cause UI bugs:
  • conditionally rendering a Suspense boundary - the data would never be transported over (the data is in the array on the server but is never flushed)
  • timing issues: the client receives an update to some data in the query that's running on the server before the server data is flushed to the client. This could cause "old" data to overwrite "new" data.
• server and client components should not have overlapping queries because the RSC render static HTML and will not update (unless you call router refresh - which is what Patrick does in his RSC example)
• useBackgroundQuery would run once on the server and once on the client

[alessbell]

The majority of this work was completed in apollographql/apollo-client-nextjs#38 and released in v0.3.0 of @apollo/experimental-nextjs-app-support.

Leaving this issue open to track the remaining work needed in Apollo Client (hide queryRef) which I should be able to get to today :)

[alessbell]

Closed via #11010.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
For general questions, we recommend using StackOverflow or our discord server.