Merge pull request #9504 from apollographql/avoid-full-reobservation-…

…after-fetchMore

Avoid full reapplication of `cache-and-network` and `network-only` fetch policies after successful `fetchMore`

CHANGELOG.md

@@ -14,6 +14,14 @@
 - Provide additional context to `nextFetchPolicy` functions to assist with `fetchPolicy` transitions. More details can be found in the [`nextFetchPolicy` documentation](https://deploy-preview-9067--apollo-client-docs.netlify.app/data/queries/#nextfetchpolicy). <br/>
   [@benjamn](https://github.com/benjamn) in [#9222](https://github.com/apollographql/apollo-client/pull/9222)
 
+- Remove nagging deprecation warning about passing an `options.updateQuery` function to `fetchMore`. <br/>
+  [@benjamn](https://github.com/benjamn) in [#9504](https://github.com/apollographql/apollo-client/pull/9504)
+
+### Potentially disruptive changes
+
+- Calling `fetchMore` for queries using the `cache-and-network` or `network-only` fetch policies should no longer trigger additional network requests when cache results are complete. <br/>
+  [@benjamn](https://github.com/benjamn) in [#9504](https://github.com/apollographql/apollo-client/pull/9504)
+
 ### Postponed to v3.7
 
 - Tentatively reimplement `useQuery` and `useLazyQuery` to use the [proposed `useSyncExternalStore` API](https://github.com/reactwg/react-18/discussions/86) from React 18. <br/>

package.json

@@ -56,7 +56,7 @@
     {
       "name": "apollo-client",
       "path": "./dist/apollo-client.min.cjs",
-      "maxSize": "29kB"
+      "maxSize": "29.2kB"
     }
   ],
   "engines": {

src/__tests__/client.ts

@@ -3374,12 +3374,9 @@ describe('@connection', () => {
         } else if (handleCount === 4) {
           expect(result.data).toEqual({ count: "secondary" });
           expect(nextFetchPolicyCallCount).toBe(3);
+          client.cache.evict({ fieldName: "count" });
         } else if (handleCount === 5) {
           expect(result.data).toEqual({ count: 1 });
-          expect(nextFetchPolicyCallCount).toBe(3);
-          client.cache.evict({ fieldName: "count" });
-        } else if (handleCount === 6) {
-          expect(result.data).toEqual({ count: 2 });
           expect(nextFetchPolicyCallCount).toBe(4);
           expect(obs.options.fetchPolicy).toBe("cache-first");
           setTimeout(resolve, 50);