@apollo/client 3.11.4 → 3.11.6

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @apollo/client
 
+## 3.11.6
+
+### Patch Changes
+
+- [#12049](https://github.com/apollographql/apollo-client/pull/12049) [`9c26892`](https://github.com/apollographql/apollo-client/commit/9c268927b1f8e5921b9440a53c9979a37f594e75) Thanks [@phryneas](https://github.com/phryneas) and [@maciesielka](https://github.com/maciesielka)! - Fix a bug where `useFragment` did not re-render as expected
+
+- [#12044](https://github.com/apollographql/apollo-client/pull/12044) [`04462a2`](https://github.com/apollographql/apollo-client/commit/04462a274ad39b392142385a2f052abbf3014749) Thanks [@DoctorJohn](https://github.com/DoctorJohn)! - Cache the `useSubscription` hook's `restart` function definition between re-renders.
+
+## 3.11.5
+
+### Patch Changes
+
+- [#12027](https://github.com/apollographql/apollo-client/pull/12027) [`eb3e21b`](https://github.com/apollographql/apollo-client/commit/eb3e21b9f7fa6a3161705c2c7270129c17b65095) Thanks [@JavaScriptBach](https://github.com/JavaScriptBach)! - Type `MutationResult.reset` as an arrow function
+
+- [#12020](https://github.com/apollographql/apollo-client/pull/12020) [`82d8cb4`](https://github.com/apollographql/apollo-client/commit/82d8cb4255be497748829f12eb25ac87c11ee5e4) Thanks [@jerelmiller](https://github.com/jerelmiller)! - Better conform to Rules of React by avoiding write of ref in render for `useFragment`.
+
 ## 3.11.4
 
 ### Patch Changes

package/apollo-client.cjs

@@ -31,7 +31,7 @@ function _interopNamespace(e) {
 var equal__default = /*#__PURE__*/_interopDefaultLegacy(equal);
 var React__namespace = /*#__PURE__*/_interopNamespace(React);
 
-var version = "3.11.4";
+var version = "3.11.6";
 
 function maybe(thunk) {
     try {
@@ -7481,15 +7481,6 @@ function useRenderGuard() {
     }, []);
 }
 
-var INIT = {};
-function useLazyRef(getInitialValue) {
-    var ref = React__namespace.useRef(INIT);
-    if (ref.current === INIT) {
-        ref.current = getInitialValue();
-    }
-    return ref;
-}
-
 var useKey = "use";
 var realHook = React__namespace[useKey];
 var __use = realHook ||
@@ -8145,10 +8136,11 @@ function useSubscription(subscription, options) {
             observable.__.result
             : fallbackResult;
     }, function () { return fallbackResult; });
-    return React__namespace.useMemo(function () { return (tslib.__assign(tslib.__assign({}, ret), { restart: function () {
-            invariant(!optionsRef.current.skip, 57);
-            setObservable(recreateRef.current());
-        } })); }, [ret]);
+    var restart = React__namespace.useCallback(function () {
+        invariant(!optionsRef.current.skip, 57);
+        setObservable(recreateRef.current());
+    }, [optionsRef, recreateRef]);
+    return React__namespace.useMemo(function () { return (tslib.__assign(tslib.__assign({}, ret), { restart: restart })); }, [ret, restart]);
 }
 function createSubscription(client, query, variables, fetchPolicy, errorPolicy, context, extensions) {
     var options = {
@@ -8193,25 +8185,23 @@ function useFragment(options) {
 }
 function _useFragment(options) {
     var cache = useApolloClient(options.client).cache;
-    var diffOptions = useDeepMemo(function () {
-        var fragment = options.fragment, fragmentName = options.fragmentName, from = options.from, _a = options.optimistic, optimistic = _a === void 0 ? true : _a, rest = tslib.__rest(options, ["fragment", "fragmentName", "from", "optimistic"]);
-        return tslib.__assign(tslib.__assign({}, rest), { returnPartialData: true, id: typeof from === "string" ? from : cache.identify(from), query: cache["getFragmentDoc"](fragment, fragmentName), optimistic: optimistic });
-    }, [options]);
-    var resultRef = useLazyRef(function () {
-        return diffToResult(cache.diff(diffOptions));
-    });
-    var stableOptions = useDeepMemo(function () { return options; }, [options]);
-    React__namespace.useMemo(function () {
-        resultRef.current = diffToResult(cache.diff(diffOptions));
-    }, [diffOptions, cache]);
-    var getSnapshot = React__namespace.useCallback(function () { return resultRef.current; }, []);
+    var from = options.from, rest = tslib.__rest(options, ["from"]);
+    var id = React__namespace.useMemo(function () { return (typeof from === "string" ? from : cache.identify(from)); }, [cache, from]);
+    var stableOptions = useDeepMemo(function () { return (tslib.__assign(tslib.__assign({}, rest), { from: id })); }, [rest, id]);
+    var diff = React__namespace.useMemo(function () {
+        var fragment = stableOptions.fragment, fragmentName = stableOptions.fragmentName, from = stableOptions.from, _a = stableOptions.optimistic, optimistic = _a === void 0 ? true : _a;
+        return {
+            result: diffToResult(cache.diff(tslib.__assign(tslib.__assign({}, stableOptions), { returnPartialData: true, id: from, query: cache["getFragmentDoc"](fragment, fragmentName), optimistic: optimistic }))),
+        };
+    }, [stableOptions, cache]);
+    var getSnapshot = React__namespace.useCallback(function () { return diff.result; }, [diff]);
     return useSyncExternalStore(React__namespace.useCallback(function (forceUpdate) {
         var lastTimeout = 0;
         var subscription = cache.watchFragment(stableOptions).subscribe({
             next: function (result) {
-                if (equal__default(result, resultRef.current))
+                if (equal__default(result, diff.result))
                     return;
-                resultRef.current = result;
+                diff.result = result;
                 clearTimeout(lastTimeout);
                 lastTimeout = setTimeout(forceUpdate);
             },
@@ -8220,7 +8210,7 @@ function _useFragment(options) {
             subscription.unsubscribe();
             clearTimeout(lastTimeout);
         };
-    }, [cache, stableOptions]), getSnapshot, getSnapshot);
+    }, [cache, stableOptions, diff]), getSnapshot, getSnapshot);
 }
 function diffToResult(diff) {
     var result = {