react-relay 15.0.0 → 16.0.0

package/lib/relay-hooks/react-cache/useLazyLoadQuery_REACT_CACHE.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _require = require('../loadQuery'),
@@ -26,8 +15,6 @@ function useLazyLoadQuery_REACT_CACHE(gqlQuery, variables, options) {
   var queryOperationDescriptor = useMemoOperationDescriptor(gqlQuery, variables, (_options$networkCache = options === null || options === void 0 ? void 0 : options.networkCacheConfig) !== null && _options$networkCache !== void 0 ? _options$networkCache : {
     force: true
   });
-
-  // Get the query going if needed -- this may suspend.
   var _getQueryResultOrFetc = getQueryResultOrFetchQuery(environment, queryOperationDescriptor, {
       fetchPolicy: options === null || options === void 0 ? void 0 : options.fetchPolicy,
       renderPolicy: options === null || options === void 0 ? void 0 : options.UNSTABLE_renderPolicy,
@@ -36,11 +23,8 @@ function useLazyLoadQuery_REACT_CACHE(gqlQuery, variables, options) {
     queryResult = _getQueryResultOrFetc[0],
     effect = _getQueryResultOrFetc[1];
   useEffect(effect);
-
-  // Read the query's root fragment -- this may suspend.
   var fragmentNode = queryResult.fragmentNode,
     fragmentRef = queryResult.fragmentRef;
-  // $FlowExpectedError[incompatible-return] Is this a fixable incompatible-return?
   return useFragmentInternal(fragmentNode, fragmentRef, 'useLazyLoadQuery()', {
     fetchPolicy: options === null || options === void 0 ? void 0 : options.fetchPolicy,
     networkCacheConfig: options === null || options === void 0 ? void 0 : options.networkCacheConfig

package/lib/relay-hooks/react-cache/usePaginationFragment_REACT_CACHE.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -32,15 +21,12 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
   var _getPaginationMetadat = getPaginationMetadata(fragmentNode, componentDisplayName),
     connectionPathInFragmentData = _getPaginationMetadat.connectionPathInFragmentData,
     paginationRequest = _getPaginationMetadat.paginationRequest,
-    paginationMetadata = _getPaginationMetadat.paginationMetadata,
-    identifierField = _getPaginationMetadat.identifierField;
+    paginationMetadata = _getPaginationMetadat.paginationMetadata;
   var _useRefetchableFragme = useRefetchableFragmentInternal(fragmentNode, parentFragmentRef, componentDisplayName),
     fragmentData = _useRefetchableFragme.fragmentData,
     fragmentRef = _useRefetchableFragme.fragmentRef,
     refetch = _useRefetchableFragme.refetch;
   var fragmentIdentifier = getFragmentIdentifier(fragmentNode, fragmentRef);
-
-  // Backward pagination
   var _useLoadMore = useLoadMore({
       componentDisplayName: componentDisplayName,
       connectionPathInFragmentData: connectionPathInFragmentData,
@@ -49,7 +35,6 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
@@ -57,8 +42,6 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
     hasPrevious = _useLoadMore[1],
     isLoadingPrevious = _useLoadMore[2],
     disposeFetchPrevious = _useLoadMore[3];
-
-  // Forward pagination
   var _useLoadMore2 = useLoadMore({
       componentDisplayName: componentDisplayName,
       connectionPathInFragmentData: connectionPathInFragmentData,
@@ -67,7 +50,6 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
@@ -83,7 +65,6 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
     }));
   }, [disposeFetchNext, disposeFetchPrevious, refetch]);
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       fragment: fragmentNode.name,
       data: fragmentData,
@@ -109,9 +90,6 @@ function useLoadMore(args) {
   var _useState = useState(false),
     isLoadingMore = _useState[0],
     reallySetIsLoadingMore = _useState[1];
-  // Schedule this update since it must be observed by components at the same
-  // batch as when hasNext changes. hasNext is read from the store and store
-  // updates are scheduled, so this must be scheduled too.
   var setIsLoadingMore = function setIsLoadingMore(value) {
     var _environment$getSched;
     var schedule = (_environment$getSched = environment.getScheduler()) === null || _environment$getSched === void 0 ? void 0 : _environment$getSched.schedule;

package/lib/relay-hooks/react-cache/usePreloadedQuery_REACT_CACHE.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _require = require('../loadQuery'),
@@ -54,29 +43,14 @@ function usePreloadedQuery_REACT_CACHE(gqlQuery, preloadedQuery, options) {
     process.env.NODE_ENV !== "production" ? warning(preloadedQuery.isDisposed === false, 'usePreloadedQuery(): Expected preloadedQuery to not be disposed yet. ' + 'This is because disposing the query marks it for future garbage ' + 'collection, and as such query results may no longer be present in the Relay ' + 'store. In the future, this will become a hard error.') : void 0;
     var fallbackFetchObservable = fetchQuery(environment, operation);
     if (source != null && environment === preloadedQuery.environment) {
-      // If the source observable exists and the environments match, reuse
-      // the source observable.
-      // If the source observable happens to be empty, we need to fall back
-      // and re-execute and de-dupe the query (at render time).
       fetchObservable = source.ifEmpty(fallbackFetchObservable);
     } else if (environment !== preloadedQuery.environment) {
-      // If a call to loadQuery is made with a particular environment, and that
-      // preloaded query is passed to usePreloadedQuery in a different environment
-      // context, we cannot re-use the existing preloaded query.
-      // Instead, we need to fall back and re-execute and de-dupe the query with
-      // the new environment (at render time).
-      // TODO T68036756 track occurences of this warning and turn it into a hard error
       process.env.NODE_ENV !== "production" ? warning(false, 'usePreloadedQuery(): usePreloadedQuery was passed a preloaded query ' + 'that was created with a different environment than the one that is currently ' + 'in context. In the future, this will become a hard error.') : void 0;
       fetchObservable = fallbackFetchObservable;
     } else {
-      // if (source == null)
-      // If the source observable does not exist, we need to
-      // fall back and re-execute and de-dupe the query (at render time).
       fetchObservable = fallbackFetchObservable;
     }
   }
-
-  // Get the query going if needed -- this may suspend.
   var _getQueryResultOrFetc = getQueryResultOrFetchQuery(environment, operation, {
       fetchPolicy: fetchPolicy,
       renderPolicy: options === null || options === void 0 ? void 0 : options.UNSTABLE_renderPolicy,
@@ -86,8 +60,6 @@ function usePreloadedQuery_REACT_CACHE(gqlQuery, preloadedQuery, options) {
     queryResult = _getQueryResultOrFetc[0],
     effect = _getQueryResultOrFetc[1];
   useEffect(effect);
-
-  // Read the query's root fragment -- this may suspend.
   var fragmentNode = queryResult.fragmentNode,
     fragmentRef = queryResult.fragmentRef;
   var data = useFragmentInternal(fragmentNode, fragmentRef, 'usePreloadedQuery()', {
@@ -95,7 +67,6 @@ function usePreloadedQuery_REACT_CACHE(gqlQuery, preloadedQuery, options) {
     networkCacheConfig: networkCacheConfig
   });
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       query: preloadedQuery.name,
       variables: preloadedQuery.variables,

package/lib/relay-hooks/react-cache/useRefetchableFragmentInternal_REACT_CACHE.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -71,7 +60,7 @@ function useRefetchableFragmentNode(fragmentNode, parentFragmentRef, componentDi
   var _getRefetchMetadata = getRefetchMetadata(fragmentNode, componentDisplayName),
     refetchableRequest = _getRefetchMetadata.refetchableRequest,
     fragmentRefPathInResponse = _getRefetchMetadata.fragmentRefPathInResponse,
-    identifierField = _getRefetchMetadata.identifierField;
+    identifierInfo = _getRefetchMetadata.identifierInfo;
   var fragmentIdentifier = getFragmentIdentifier(fragmentNode, parentFragmentRef);
   var _useReducer = useReducer(reducer, {
       fetchPolicy: undefined,
@@ -108,47 +97,18 @@ function useRefetchableFragmentNode(fragmentNode, parentFragmentRef, componentDi
     });
     disposeQuery();
   } else if (refetchQuery != null && queryRef != null) {
-    // If refetch was called, we expect to have a refetchQuery and queryRef
-    // in state, since both state updates to set the refetchQuery and the
-    // queryRef occur simultaneously.
-    // In this case, we need to read the refetched query data (potentially
-    // suspending if it's in flight), and extract the new fragment ref
-    // from the query in order read the current @refetchable fragment
-    // with the updated fragment owner as the new refetchQuery.
-
-    // Before observing the refetch, record the current ID and typename
-    // so that, if we are refetching existing data on
-    // a field that implements Node, after refetching we
-    // can validate that the received data is consistent
     var debugPreviousIDAndTypename;
     if (process.env.NODE_ENV !== "production") {
-      debugPreviousIDAndTypename = debugFunctions.getInitialIDAndType(refetchQuery.request.variables, fragmentRefPathInResponse, environment);
+      debugPreviousIDAndTypename = debugFunctions.getInitialIDAndType(refetchQuery.request.variables, fragmentRefPathInResponse, identifierInfo === null || identifierInfo === void 0 ? void 0 : identifierInfo.identifierQueryVariableName, environment);
     }
     var handleQueryCompleted = function handleQueryCompleted(maybeError) {
       onComplete && onComplete(maybeError !== null && maybeError !== void 0 ? maybeError : null);
     };
-
-    // The queryRef.source obtained from useQueryLoader will be
-    // an observable we can consume /if/ a network request was
-    // started. Otherwise, given that QueryResource.prepare
-    // always expects an observable we fall back to a new network
-    // observable. Note however that if loadQuery did not make a network
-    // request, we don't expect to make one here, unless the state of
-    // the cache has changed between the call to refetch and this
-    // render.
     var fetchObservable = queryRef.source != null ? queryRef.source : fetchQuery(environment, refetchQuery);
-
-    // Now wwe can we read the refetch query here using the
-    // queryRef provided from useQueryLoader. Note that the
-    // network request is started during the call to refetch,
-    // but if the refetch query is still in flight, we will suspend
-    // at this point:
     var queryResult = profilerContext.wrapPrepareQueryResource(function () {
       return QueryResource.prepare(refetchQuery, fetchObservable, fetchPolicy, renderPolicy, {
         error: handleQueryCompleted,
         complete: function complete() {
-          // Validate that the type of the object we got back matches the type
-          // of the object already in the store
           if (process.env.NODE_ENV !== "production") {
             debugFunctions.checkSameTypeAfterRefetch(debugPreviousIDAndTypename, environment, fragmentNode, componentDisplayName);
           }
@@ -158,40 +118,24 @@ function useRefetchableFragmentNode(fragmentNode, parentFragmentRef, componentDi
     });
     var queryData = readFragmentInternal(environment, queryResult.fragmentNode, queryResult.fragmentRef, componentDisplayName).data;
     !(queryData != null) ? process.env.NODE_ENV !== "production" ? invariant(false, 'Relay: Expected to be able to read refetch query response. ' + "If you're seeing this, this is likely a bug in Relay.") : invariant(false) : void 0;
-
-    // After reading/fetching the refetch query, we extract from the
-    // refetch query response the new fragment ref we need to use to read
-    // the fragment. The new fragment ref will point to the refetch query
-    // as its fragment owner.
     var refetchedFragmentRef = getValueAtPath(queryData, fragmentRefPathInResponse);
     fragmentRef = refetchedFragmentRef;
     if (process.env.NODE_ENV !== "production") {
-      // Validate that the id of the object we got back matches the id
-      // we queried for in the variables.
-      // We do this during render instead of onComplete to make sure we are
-      // only validating the most recent refetch.
       debugFunctions.checkSameIDAfterRefetch(debugPreviousIDAndTypename, fragmentRef, fragmentNode, componentDisplayName);
     }
   }
-
-  // We read and subscribe to the fragment using useFragmentNode.
-  // If refetch was called, we read the fragment using the new computed
-  // fragment ref from the refetch query response; otherwise, we use the
-  // fragment ref passed by the caller as normal.
   var fragmentData = useFragmentInternal(fragmentNode, fragmentRef, componentDisplayName);
-  var refetch = useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragmentData, fragmentIdentifier, fragmentNode, fragmentRefPathInResponse, identifierField, loadQuery, parentFragmentRef, refetchableRequest);
+  var refetch = useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragmentData, fragmentIdentifier, fragmentNode, fragmentRefPathInResponse, identifierInfo, loadQuery, parentFragmentRef, refetchableRequest);
   return {
     fragmentData: fragmentData,
     fragmentRef: fragmentRef,
     refetch: refetch
   };
 }
-function useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragmentData, fragmentIdentifier, fragmentNode, fragmentRefPathInResponse, identifierField, loadQuery, parentFragmentRef, refetchableRequest) {
+function useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragmentData, fragmentIdentifier, fragmentNode, fragmentRefPathInResponse, identifierInfo, loadQuery, parentFragmentRef, refetchableRequest) {
   var isMountedRef = useIsMountedRef();
-  var identifierValue = identifierField != null && fragmentData != null && typeof fragmentData === 'object' ? fragmentData[identifierField] : null;
+  var identifierValue = (identifierInfo === null || identifierInfo === void 0 ? void 0 : identifierInfo.identifierField) != null && fragmentData != null && typeof fragmentData === 'object' ? fragmentData[identifierInfo.identifierField] : null;
   return useCallback(function (providedRefetchVariables, options) {
-    // Bail out and warn if we're trying to refetch after the component
-    // has unmounted
     if (isMountedRef.current !== true) {
       process.env.NODE_ENV !== "production" ? warning(false, 'Relay: Unexpected call to `refetch` on unmounted component for fragment ' + '`%s` in `%s`. It looks like some instances of your component are ' + 'still trying to fetch data but they already unmounted. ' + 'Please make sure you clear all timers, intervals, ' + 'async calls, etc that may trigger a fetch.', fragmentNode.name, componentDisplayName) : void 0;
       return {
@@ -219,35 +163,16 @@ function useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragme
       parentVariables = fragmentSelector.owner.variables;
       fragmentVariables = fragmentSelector.variables;
     }
-
-    // A user of `useRefetchableFragment()` may pass a subset of
-    // all variables required by the fragment when calling `refetch()`.
-    // We fill in any variables not passed by the call to `refetch()` with the
-    // variables from the original parent fragment owner.
     var refetchVariables = (0, _objectSpread2["default"])((0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, parentVariables), fragmentVariables), providedRefetchVariables);
-
-    // If the query needs an identifier value ('id' or similar) and one
-    // was not explicitly provided, read it from the fragment data.
-    if (identifierField != null && !providedRefetchVariables.hasOwnProperty('id')) {
-      // @refetchable fragments are guaranteed to have an `id` selection
-      // if the type is Node, implements Node, or is @fetchable. Double-check
-      // that there actually is a value at runtime.
+    if (identifierInfo != null && !providedRefetchVariables.hasOwnProperty(identifierInfo.identifierQueryVariableName)) {
       if (typeof identifierValue !== 'string') {
-        process.env.NODE_ENV !== "production" ? warning(false, 'Relay: Expected result to have a string  ' + '`%s` in order to refetch, got `%s`.', identifierField, identifierValue) : void 0;
+        process.env.NODE_ENV !== "production" ? warning(false, 'Relay: Expected result to have a string  ' + '`%s` in order to refetch, got `%s`.', identifierInfo.identifierField, identifierValue) : void 0;
       }
-      refetchVariables.id = identifierValue;
+      refetchVariables[identifierInfo.identifierQueryVariableName] = identifierValue;
     }
     var refetchQuery = createOperationDescriptor(refetchableRequest, refetchVariables, {
       force: true
     });
-
-    // We call loadQuery which will start a network request if necessary
-    // and update the querRef from useQueryLoader.
-    // Note the following:
-    // - loadQuery will dispose of any previously refetched queries.
-    // - We use the variables extracted off the OperationDescriptor
-    // so that they have been filtered out to include only the
-    // variables actually declared in the query.
     loadQuery(refetchQuery.request.variables, {
       fetchPolicy: fetchPolicy,
       __environment: refetchEnvironment,
@@ -264,29 +189,21 @@ function useRefetchFunction(componentDisplayName, dispatch, disposeQuery, fragme
     return {
       dispose: disposeQuery
     };
-  },
-  // NOTE: We disable react-hooks-deps warning because:
-  //   - We know fragmentRefPathInResponse is static, so it can be omitted from
-  //     deps
-  //   - We know fragmentNode is static, so it can be omitted from deps.
-  //   - fragmentNode and parentFragmentRef are also captured by including
-  //     fragmentIdentifier
-  // eslint-disable-next-line react-hooks/exhaustive-deps
-  [fragmentIdentifier, dispatch, disposeQuery, identifierValue, loadQuery]);
+  }, [fragmentIdentifier, dispatch, disposeQuery, identifierValue, loadQuery]);
 }
 var debugFunctions;
 if (process.env.NODE_ENV !== "production") {
   debugFunctions = {
-    getInitialIDAndType: function getInitialIDAndType(memoRefetchVariables, fragmentRefPathInResponse, environment) {
+    getInitialIDAndType: function getInitialIDAndType(memoRefetchVariables, fragmentRefPathInResponse, identifierQueryVariableName, environment) {
       var _require4 = require('relay-runtime'),
         Record = _require4.Record;
-      var id = memoRefetchVariables === null || memoRefetchVariables === void 0 ? void 0 : memoRefetchVariables.id;
+      var id = memoRefetchVariables === null || memoRefetchVariables === void 0 ? void 0 : memoRefetchVariables[identifierQueryVariableName !== null && identifierQueryVariableName !== void 0 ? identifierQueryVariableName : 'id'];
       if (fragmentRefPathInResponse.length !== 1 || fragmentRefPathInResponse[0] !== 'node' || id == null) {
         return null;
       }
       var recordSource = environment.getStore().getSource();
       var record = recordSource.get(id);
-      var typename = record && Record.getType(record);
+      var typename = record == null ? null : Record.getType(record);
       if (typename == null) {
         return null;
       }
@@ -314,7 +231,6 @@ if (process.env.NODE_ENV !== "production") {
       }
       var _require6 = require('relay-runtime'),
         ID_KEY = _require6.ID_KEY;
-      // $FlowExpectedError[incompatible-use]
       var resultID = refetchedFragmentRef[ID_KEY];
       if (resultID != null && resultID !== previousIDAndTypename.id) {
         process.env.NODE_ENV !== "production" ? warning(false, 'Relay: Call to `refetch` returned a different id, expected ' + '`%s`, got `%s`, on `%s` in `%s`. ' + 'Please make sure the server correctly implements ' + 'unique id requirement.', resultID, previousIDAndTypename.id, fragmentNode.name, componentDisplayName) : void 0;

package/lib/relay-hooks/react-cache/useRefetchableFragment_REACT_CACHE.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var useStaticFragmentNodeWarning = require('../useStaticFragmentNodeWarning');
@@ -24,14 +13,11 @@ function useRefetchableFragment(fragmentInput, fragmentRef) {
     fragmentData = _useRefetchableFragme.fragmentData,
     refetch = _useRefetchableFragme.refetch;
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       fragment: fragmentNode.name,
       data: fragmentData
     });
   }
-  /* $FlowExpectedError[prop-missing] : Exposed options is a subset of internal
-   * options */
   return [fragmentData, refetch];
 }
 module.exports = useRefetchableFragment;

package/lib/relay-hooks/useBlockingPaginationFragment.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -34,7 +23,6 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
   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;
@@ -46,8 +34,6 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
     disableStoreUpdates = _useRefetchableFragme.disableStoreUpdates,
     enableStoreUpdates = _useRefetchableFragme.enableStoreUpdates;
   var fragmentIdentifier = getFragmentIdentifier(fragmentNode, fragmentRef);
-
-  // Backward pagination
   var _useLoadMore = useLoadMore({
       componentDisplayName: componentDisplayName,
       connectionPathInFragmentData: connectionPathInFragmentData,
@@ -58,15 +44,12 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
     loadPrevious = _useLoadMore[0],
     hasPrevious = _useLoadMore[1],
     disposeFetchPrevious = _useLoadMore[2];
-
-  // Forward pagination
   var _useLoadMore2 = useLoadMore({
       componentDisplayName: componentDisplayName,
       connectionPathInFragmentData: connectionPathInFragmentData,
@@ -77,7 +60,6 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
@@ -92,7 +74,6 @@ function useBlockingPaginationFragment(fragmentInput, parentFragmentRef) {
     }));
   }, [disposeFetchNext, disposeFetchPrevious, refetch]);
   return {
-    // $FlowFixMe[incompatible-cast]
     data: fragmentData,
     loadNext: loadNext,
     loadPrevious: loadPrevious,
@@ -121,13 +102,7 @@ function useLoadMore(args) {
   };
   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 () {
@@ -138,14 +113,7 @@ 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), {}, {
@@ -155,23 +123,13 @@ function useLoadMore(args) {
     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.
   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];
 }

package/lib/relay-hooks/useClientQuery.js

@@ -1,27 +1,9 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 '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'

package/lib/relay-hooks/useEntryPointLoader.js

@@ -1,14 +1,3 @@
-/**
- * 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.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -27,26 +16,6 @@ var initialNullEntryPointReferenceState = {
 };
 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;
@@ -78,53 +47,17 @@ 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),
@@ -149,8 +82,6 @@ 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;
       try {