react-relay 14.1.0 → 16.0.0

package/lib/relay-hooks/useBlockingPaginationFragment.js

@@ -1,102 +1,71 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @emails oncall+relay
- * 
- * @format
- */
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
-
 var _objectWithoutPropertiesLoose2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutPropertiesLoose"));
-
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
-
 var _excluded = ["disableStoreUpdates", "enableStoreUpdates"];
-
 var useLoadMoreFunction = require('./useLoadMoreFunction');
-
 var useRefetchableFragmentNode = require('./useRefetchableFragmentNode');
-
 var useStaticFragmentNodeWarning = require('./useStaticFragmentNodeWarning');
-
 var invariant = require('invariant');
-
 var _require = require('react'),
-    useCallback = _require.useCallback,
-    useEffect = _require.useEffect,
-    useRef = _require.useRef,
-    useState = _require.useState;
-
+  useCallback = _require.useCallback,
+  useEffect = _require.useEffect,
+  useRef = _require.useRef,
+  useState = _require.useState;
 var _require2 = require('relay-runtime'),
-    getFragment = _require2.getFragment,
-    getFragmentIdentifier = _require2.getFragmentIdentifier,
-    getPaginationMetadata = _require2.getPaginationMetadata;
-
+  getFragment = _require2.getFragment,
+  getFragmentIdentifier = _require2.getFragmentIdentifier,
+  getPaginationMetadata = _require2.getPaginationMetadata;
 function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
   var componentDisplayName = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 'useBlockingPaginationFragment()';
   var fragmentNode = getFragment(fragmentInput);
   useStaticFragmentNodeWarning(fragmentNode, "first argument of ".concat(componentDisplayName));
-
   var _getPaginationMetadat = getPaginationMetadata(fragmentNode, componentDisplayName),
-      connectionPathInFragmentData = _getPaginationMetadat.connectionPathInFragmentData,
-      identifierField = _getPaginationMetadat.identifierField,
-      paginationRequest = _getPaginationMetadat.paginationRequest,
-      paginationMetadata = _getPaginationMetadat.paginationMetadata,
-      stream = _getPaginationMetadat.stream;
-
+    connectionPathInFragmentData = _getPaginationMetadat.connectionPathInFragmentData,
+    paginationRequest = _getPaginationMetadat.paginationRequest,
+    paginationMetadata = _getPaginationMetadat.paginationMetadata,
+    stream = _getPaginationMetadat.stream;
   !(stream === false) ? process.env.NODE_ENV !== "production" ? invariant(false, 'Relay: @stream_connection is not compatible with `useBlockingPaginationFragment`. ' + 'Use `useStreamingPaginationFragment` instead.') : invariant(false) : void 0;
-
   var _useRefetchableFragme = useRefetchableFragmentNode(fragmentNode, parentFragmentRef, componentDisplayName),
-      fragmentData = _useRefetchableFragme.fragmentData,
-      fragmentRef = _useRefetchableFragme.fragmentRef,
-      refetch = _useRefetchableFragme.refetch,
-      disableStoreUpdates = _useRefetchableFragme.disableStoreUpdates,
-      enableStoreUpdates = _useRefetchableFragme.enableStoreUpdates;
-
-  var fragmentIdentifier = getFragmentIdentifier(fragmentNode, fragmentRef); // Backward pagination
-
+    fragmentData = _useRefetchableFragme.fragmentData,
+    fragmentRef = _useRefetchableFragme.fragmentRef,
+    refetch = _useRefetchableFragme.refetch,
+    disableStoreUpdates = _useRefetchableFragme.disableStoreUpdates,
+    enableStoreUpdates = _useRefetchableFragme.enableStoreUpdates;
+  var fragmentIdentifier = getFragmentIdentifier(fragmentNode, fragmentRef);
   var _useLoadMore = useLoadMore({
-    componentDisplayName: componentDisplayName,
-    connectionPathInFragmentData: connectionPathInFragmentData,
-    direction: 'backward',
-    disableStoreUpdates: disableStoreUpdates,
-    enableStoreUpdates: enableStoreUpdates,
-    fragmentData: fragmentData,
-    fragmentIdentifier: fragmentIdentifier,
-    fragmentNode: fragmentNode,
-    fragmentRef: fragmentRef,
-    identifierField: identifierField,
-    paginationMetadata: paginationMetadata,
-    paginationRequest: paginationRequest
-  }),
-      loadPrevious = _useLoadMore[0],
-      hasPrevious = _useLoadMore[1],
-      disposeFetchPrevious = _useLoadMore[2]; // Forward pagination
-
-
+      componentDisplayName: componentDisplayName,
+      connectionPathInFragmentData: connectionPathInFragmentData,
+      direction: 'backward',
+      disableStoreUpdates: disableStoreUpdates,
+      enableStoreUpdates: enableStoreUpdates,
+      fragmentData: fragmentData,
+      fragmentIdentifier: fragmentIdentifier,
+      fragmentNode: fragmentNode,
+      fragmentRef: fragmentRef,
+      paginationMetadata: paginationMetadata,
+      paginationRequest: paginationRequest
+    }),
+    loadPrevious = _useLoadMore[0],
+    hasPrevious = _useLoadMore[1],
+    disposeFetchPrevious = _useLoadMore[2];
   var _useLoadMore2 = useLoadMore({
-    componentDisplayName: componentDisplayName,
-    connectionPathInFragmentData: connectionPathInFragmentData,
-    direction: 'forward',
-    disableStoreUpdates: disableStoreUpdates,
-    enableStoreUpdates: enableStoreUpdates,
-    fragmentData: fragmentData,
-    fragmentIdentifier: fragmentIdentifier,
-    fragmentNode: fragmentNode,
-    fragmentRef: fragmentRef,
-    identifierField: identifierField,
-    paginationMetadata: paginationMetadata,
-    paginationRequest: paginationRequest
-  }),
-      loadNext = _useLoadMore2[0],
-      hasNext = _useLoadMore2[1],
-      disposeFetchNext = _useLoadMore2[2];
-
+      componentDisplayName: componentDisplayName,
+      connectionPathInFragmentData: connectionPathInFragmentData,
+      direction: 'forward',
+      disableStoreUpdates: disableStoreUpdates,
+      enableStoreUpdates: enableStoreUpdates,
+      fragmentData: fragmentData,
+      fragmentIdentifier: fragmentIdentifier,
+      fragmentNode: fragmentNode,
+      fragmentRef: fragmentRef,
+      paginationMetadata: paginationMetadata,
+      paginationRequest: paginationRequest
+    }),
+    loadNext = _useLoadMore2[0],
+    hasNext = _useLoadMore2[1],
+    disposeFetchNext = _useLoadMore2[2];
   var refetchPagination = useCallback(function (variables, options) {
     disposeFetchNext();
     disposeFetchPrevious();
@@ -113,39 +82,27 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
     refetch: refetchPagination
   };
 }
-
 function useLoadMore(args) {
   var disableStoreUpdates = args.disableStoreUpdates,
-      enableStoreUpdates = args.enableStoreUpdates,
-      loadMoreArgs = (0, _objectWithoutPropertiesLoose2["default"])(args, _excluded);
-
+    enableStoreUpdates = args.enableStoreUpdates,
+    loadMoreArgs = (0, _objectWithoutPropertiesLoose2["default"])(args, _excluded);
   var _useState = useState(null),
-      requestPromise = _useState[0],
-      setRequestPromise = _useState[1];
-
+    requestPromise = _useState[0],
+    setRequestPromise = _useState[1];
   var requestPromiseRef = useRef(null);
   var promiseResolveRef = useRef(null);
-
   var promiseResolve = function promiseResolve() {
     if (promiseResolveRef.current != null) {
       promiseResolveRef.current();
       promiseResolveRef.current = null;
     }
   };
-
   var handleReset = function handleReset() {
     promiseResolve();
   };
-
   var observer = {
     complete: promiseResolve,
-    // NOTE: loadMore is a no-op if a request is already in flight, so we
-    // can safely assume that `start` will only be called once while a
-    // request is in flight.
     start: function start() {
-      // NOTE: We disable store updates when we suspend to ensure
-      // that higher-pri updates from the Relay store don't disrupt
-      // any Suspense timeouts passed via withSuspenseConfig.
       disableStoreUpdates();
       var promise = new Promise(function (resolve) {
         promiseResolveRef.current = function () {
@@ -156,44 +113,24 @@ function useLoadMore(args) {
       requestPromiseRef.current = promise;
       setRequestPromise(promise);
     },
-    // NOTE: Since streaming is disallowed with this hook, this means that the
-    // first payload will always contain the entire next page of items,
-    // while subsequent paylaods will contain @defer'd payloads.
-    // This allows us to unsuspend here, on the first payload, and allow
-    // descendant components to suspend on their respective @defer payloads
     next: promiseResolve,
-    // TODO: Handle error; we probably don't want to throw an error
-    // and blow away the whole list of items.
     error: promiseResolve
   };
-
   var _useLoadMoreFunction = useLoadMoreFunction((0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, loadMoreArgs), {}, {
-    observer: observer,
-    onReset: handleReset
-  })),
-      loadMore = _useLoadMoreFunction[0],
-      hasMore = _useLoadMoreFunction[1],
-      disposeFetch = _useLoadMoreFunction[2]; // NOTE: To determine if we need to suspend, we check that the promise in
-  // state is the same as the promise on the ref, which ensures that we
-  // wont incorrectly suspend on other higher-pri updates before the update
-  // to suspend has committed.
-
-
+      observer: observer,
+      onReset: handleReset
+    })),
+    loadMore = _useLoadMoreFunction[0],
+    hasMore = _useLoadMoreFunction[1],
+    disposeFetch = _useLoadMoreFunction[2];
   if (requestPromise != null && requestPromise === requestPromiseRef.current) {
     throw requestPromise;
   }
-
   useEffect(function () {
     if (requestPromise !== requestPromiseRef.current) {
-      // NOTE: After suspense pagination has resolved, we re-enable store updates
-      // for this fragment. This may cause the component to re-render if
-      // we missed any updates to the fragment data other than the pagination update.
       enableStoreUpdates();
-    } // NOTE: We know the identity of enableStoreUpdates wont change
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-
+    }
   }, [requestPromise]);
   return [loadMore, hasMore, disposeFetch];
 }
-
 module.exports = useBlockingPaginationFragment;

package/lib/relay-hooks/useClientQuery.js

@@ -1,33 +1,12 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @emails oncall+relay
- * 
- * @format
- */
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
-
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
-
 var useLazyLoadQuery = require('./useLazyLoadQuery');
-/**
- * This hook can be used to render client-only queries.
- * These queries are consist of queries for client-only data,
- * schematized via local schema extensions and/or Relay resolvers.
- */
-
-
 function useClientQuery(gqlQuery, variables, options) {
-  // $FlowFixMe[incompatible-type] client queries can be used with useLazyLoadQuery, but only with `store-only` policy.
   var query = gqlQuery;
   return useLazyLoadQuery(query, variables, (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, options), {}, {
     fetchPolicy: 'store-only'
   }));
 }
-
 module.exports = useClientQuery;

package/lib/relay-hooks/useEntryPointLoader.js

@@ -1,72 +1,32 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @emails oncall+relay
- * 
- * @format
- */
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
-
 var _createForOfIteratorHelper2 = _interopRequireDefault(require("@babel/runtime/helpers/createForOfIteratorHelper"));
-
 var loadEntryPoint = require('./loadEntryPoint');
-
 var _require = require('./loadQuery'),
-    useTrackLoadQueryInRender = _require.useTrackLoadQueryInRender;
-
+  useTrackLoadQueryInRender = _require.useTrackLoadQueryInRender;
 var useIsMountedRef = require('./useIsMountedRef');
-
 var _require2 = require('react'),
-    useCallback = _require2.useCallback,
-    useEffect = _require2.useEffect,
-    useRef = _require2.useRef,
-    useState = _require2.useState;
-
+  useCallback = _require2.useCallback,
+  useEffect = _require2.useEffect,
+  useRef = _require2.useRef,
+  useState = _require2.useState;
 var initialNullEntryPointReferenceState = {
   kind: 'NullEntryPointReference'
 };
-
 function useLoadEntryPoint(environmentProvider, entryPoint, options) {
   var _options$TEST_ONLY__i, _options$TEST_ONLY__i2, _options$TEST_ONLY__i3, _options$TEST_ONLY__i4;
-
-  /**
-   * We want to always call `entryPointReference.dispose()` for every call to
-   * `setEntryPointReference(loadEntryPoint(...))` so that no leaks of data in Relay
-   * stores will occur.
-   *
-   * However, a call to `setState(newState)` is not always followed by a commit where
-   * this value is reflected in the state. Thus, we cannot reliably clean up each ref
-   * with `useEffect(() => () => entryPointReference.dispose(), [entryPointReference])`.
-   *
-   * Instead, we keep track of each call to `loadEntryPoint` in a ref.
-   * Relying on the fact that if a state change commits, no state changes that were
-   * initiated prior to the currently committing state change will ever subsequently
-   * commit, we can safely dispose of all preloaded entry point references
-   * associated with state changes initiated prior to the currently committing state
-   * change.
-   *
-   * Finally, when the hook unmounts, we also dispose of all remaining uncommitted
-   * entry point references.
-   */
   useTrackLoadQueryInRender();
   var initialEntryPointReferenceInternal = (_options$TEST_ONLY__i = options === null || options === void 0 ? void 0 : (_options$TEST_ONLY__i2 = options.TEST_ONLY__initialEntryPointData) === null || _options$TEST_ONLY__i2 === void 0 ? void 0 : _options$TEST_ONLY__i2.entryPointReference) !== null && _options$TEST_ONLY__i !== void 0 ? _options$TEST_ONLY__i : initialNullEntryPointReferenceState;
   var initialEntryPointParamsInternal = (_options$TEST_ONLY__i3 = options === null || options === void 0 ? void 0 : (_options$TEST_ONLY__i4 = options.TEST_ONLY__initialEntryPointData) === null || _options$TEST_ONLY__i4 === void 0 ? void 0 : _options$TEST_ONLY__i4.entryPointParams) !== null && _options$TEST_ONLY__i3 !== void 0 ? _options$TEST_ONLY__i3 : null;
   var isMountedRef = useIsMountedRef();
   var undisposedEntryPointReferencesRef = useRef(new Set([initialEntryPointReferenceInternal]));
-
   var _useState = useState(initialEntryPointReferenceInternal),
-      entryPointReference = _useState[0],
-      setEntryPointReference = _useState[1];
-
+    entryPointReference = _useState[0],
+    setEntryPointReference = _useState[1];
   var _useState2 = useState(initialEntryPointParamsInternal),
-      entryPointParams = _useState2[0],
-      setEntryPointParams = _useState2[1];
-
+    entryPointParams = _useState2[0],
+    setEntryPointParams = _useState2[1];
   var disposeEntryPoint = useCallback(function () {
     if (isMountedRef.current) {
       var nullEntryPointReference = {
@@ -87,71 +47,28 @@ function useLoadEntryPoint(environmentProvider, entryPoint, options) {
   var maybeHiddenOrFastRefresh = useRef(false);
   useEffect(function () {
     return function () {
-      // Attempt to detect if the component was
-      // hidden (by Offscreen API), or fast refresh occured;
-      // Only in these situations would the effect cleanup
-      // for "unmounting" run multiple times, so if
-      // we are ever able to read this ref with a value
-      // of true, it means that one of these cases
-      // has happened.
       maybeHiddenOrFastRefresh.current = true;
     };
   }, []);
   useEffect(function () {
     if (maybeHiddenOrFastRefresh.current === true) {
-      // This block only runs if the component has previously "unmounted"
-      // due to it being hidden by the Offscreen API, or during fast refresh.
-      // At this point, the current entryPointReference will have been disposed
-      // by the previous cleanup, so instead of attempting to
-      // do our regular commit setup, which would incorrectly leave our
-      // current entryPointReference disposed, we need to load the entryPoint again
-      // and force a re-render by calling entryPointLoaderCallback again,
-      // so that the entryPointReference's queries are correctly re-retained, and
-      // potentially refetched if necessary.
       maybeHiddenOrFastRefresh.current = false;
-
       if (entryPointReference.kind !== 'NullEntryPointReference' && entryPointParams != null) {
         entryPointLoaderCallback(entryPointParams);
       }
-
       return;
-    } // When a new entryPointReference is committed, we iterate over all
-    // entrypoint refs in undisposedEntryPointReferences and dispose all of
-    // the refs that aren't the currently committed one. This ensures
-    // that we don't leave any dangling entrypoint references for the
-    // case that loadEntryPoint is called multiple times before commit; when
-    // this happens, multiple state updates will be scheduled, but only one
-    // will commit, meaning that we need to keep track of and dispose any
-    // query references that don't end up committing.
-    // - We are relying on the fact that sets iterate in insertion order, and we
-    // can remove items from a set as we iterate over it (i.e. no iterator
-    // invalidation issues.) Thus, it is safe to loop through
-    // undisposedEntryPointReferences until we find entryPointReference, and
-    // remove and dispose all previous references.
-    // - We are guaranteed to find entryPointReference in the set, because if a
-    // state change results in a commit, no state changes initiated prior to that
-    // one will be committed, and we are disposing and removing references
-    // associated with commits that were initiated prior to the currently
-    // committing state change. (A useEffect callback is called during the commit
-    // phase.)
-
-
+    }
     var undisposedEntryPointReferences = undisposedEntryPointReferencesRef.current;
-
     if (isMountedRef.current) {
       var _iterator = (0, _createForOfIteratorHelper2["default"])(undisposedEntryPointReferences),
-          _step;
-
+        _step;
       try {
         for (_iterator.s(); !(_step = _iterator.n()).done;) {
           var undisposedEntryPointReference = _step.value;
-
           if (undisposedEntryPointReference === entryPointReference) {
             break;
           }
-
           undisposedEntryPointReferences["delete"](undisposedEntryPointReference);
-
           if (undisposedEntryPointReference.kind !== 'NullEntryPointReference') {
             undisposedEntryPointReference.dispose();
           }
@@ -165,15 +82,11 @@ function useLoadEntryPoint(environmentProvider, entryPoint, options) {
   }, [entryPointReference, entryPointParams, entryPointLoaderCallback, isMountedRef]);
   useEffect(function () {
     return function disposeAllRemainingEntryPointReferences() {
-      // undisposedEntryPointReferences.current is never reassigned
-      // eslint-disable-next-line react-hooks/exhaustive-deps
       var _iterator2 = (0, _createForOfIteratorHelper2["default"])(undisposedEntryPointReferencesRef.current),
-          _step2;
-
+        _step2;
       try {
         for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
           var unhandledStateChange = _step2.value;
-
           if (unhandledStateChange.kind !== 'NullEntryPointReference') {
             unhandledStateChange.dispose();
           }
@@ -187,5 +100,4 @@ function useLoadEntryPoint(environmentProvider, entryPoint, options) {
   }, []);
   return [entryPointReference.kind === 'NullEntryPointReference' ? null : entryPointReference, entryPointLoaderCallback, disposeEntryPoint];
 }
-
 module.exports = useLoadEntryPoint;

package/lib/relay-hooks/useFetchTrackingRef.js

@@ -1,42 +1,17 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @emails oncall+relay
- * 
- * @format
- */
 'use strict';
 
+var useUnsafeRef_DEPRECATED = require('./useUnsafeRef_DEPRECATED');
 var _require = require('react'),
-    useCallback = _require.useCallback,
-    useEffect = _require.useEffect,
-    useRef = _require.useRef;
-/**
- * This hook returns a mutable React ref that holds the value of whether a
- * fetch request is in flight. The reason this is a mutable ref instead of
- * state is because we don't actually want to trigger an update when this
- * changes, but instead synchronously keep track of whether the network request
- * is in flight, for example in order to bail out of a request if one is
- * already in flight. If this was state, due to the nature of concurrent
- * updates, this value wouldn't be in sync with when the request is actually
- * in flight.
- * The additional functions returned by this Hook can be used to mutate
- * the ref.
- */
-
-
+  useCallback = _require.useCallback,
+  useEffect = _require.useEffect;
 function useFetchTrackingRef() {
-  var subscriptionRef = useRef(null);
-  var isFetchingRef = useRef(false);
+  var subscriptionRef = useUnsafeRef_DEPRECATED(null);
+  var isFetchingRef = useUnsafeRef_DEPRECATED(false);
   var disposeFetch = useCallback(function () {
     if (subscriptionRef.current != null) {
       subscriptionRef.current.unsubscribe();
       subscriptionRef.current = null;
     }
-
     isFetchingRef.current = false;
   }, []);
   var startFetch = useCallback(function (subscription) {
@@ -46,8 +21,7 @@ function useFetchTrackingRef() {
   var completeFetch = useCallback(function () {
     subscriptionRef.current = null;
     isFetchingRef.current = false;
-  }, []); // Dipose of ongoing fetch on unmount
-
+  }, []);
   useEffect(function () {
     return disposeFetch;
   }, [disposeFetch]);
@@ -58,5 +32,4 @@ function useFetchTrackingRef() {
     completeFetch: completeFetch
   };
 }
-
 module.exports = useFetchTrackingRef;

package/lib/relay-hooks/useFragment.js

@@ -1,61 +1,34 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @emails oncall+relay
- * 
- * @format
- */
 'use strict';
 
 var HooksImplementation = require('./HooksImplementation');
-
 var _require = require('./loadQuery'),
-    useTrackLoadQueryInRender = _require.useTrackLoadQueryInRender;
-
+  useTrackLoadQueryInRender = _require.useTrackLoadQueryInRender;
 var useFragmentNode = require('./useFragmentNode');
-
 var useStaticFragmentNodeWarning = require('./useStaticFragmentNodeWarning');
-
 var _require2 = require('react'),
-    useDebugValue = _require2.useDebugValue;
-
+  useDebugValue = _require2.useDebugValue;
 var _require3 = require('relay-runtime'),
-    getFragment = _require3.getFragment;
-
+  getFragment = _require3.getFragment;
 function useFragment_LEGACY(fragment, key) {
-  // We need to use this hook in order to be able to track if
-  // loadQuery was called during render
   useTrackLoadQueryInRender();
   var fragmentNode = getFragment(fragment);
   useStaticFragmentNodeWarning(fragmentNode, 'first argument of useFragment()');
-
   var _useFragmentNode = useFragmentNode(fragmentNode, key, 'useFragment()'),
-      data = _useFragmentNode.data;
-
+    data = _useFragmentNode.data;
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       fragment: fragmentNode.name,
       data: data
     });
   }
-
   return data;
 }
-
 function useFragment(fragment, key) {
   var impl = HooksImplementation.get();
-
   if (impl) {
-    // $FlowFixMe This is safe because impl.useFragment has the type of useFragment...
-    return impl.useFragment(fragment, key); // (i.e. type declared above, but not the supertype used in this function definition)
+    return impl.useFragment(fragment, key);
   } else {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     return useFragment_LEGACY(fragment, key);
   }
 }
-
 module.exports = useFragment;