react-relay 15.0.0 → 16.0.0

package/lib/relay-hooks/useFetchTrackingRef.js

@@ -1,33 +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 useUnsafeRef_DEPRECATED = require('./useUnsafeRef_DEPRECATED');
 var _require = require('react'),
   useCallback = _require.useCallback,
   useEffect = _require.useEffect;
-
-/**
- * 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.
- */
 function useFetchTrackingRef() {
   var subscriptionRef = useUnsafeRef_DEPRECATED(null);
   var isFetchingRef = useUnsafeRef_DEPRECATED(false);
@@ -46,8 +22,6 @@ function useFetchTrackingRef() {
     subscriptionRef.current = null;
     isFetchingRef.current = false;
   }, []);
-
-  // Dipose of ongoing fetch on unmount
   useEffect(function () {
     return disposeFetch;
   }, [disposeFetch]);

package/lib/relay-hooks/useFragment.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 HooksImplementation = require('./HooksImplementation');
@@ -21,15 +10,12 @@ var _require2 = require('react'),
 var _require3 = require('relay-runtime'),
   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;
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       fragment: fragmentNode.name,
       data: data
@@ -40,11 +26,8 @@ function useFragment_LEGACY(fragment, key) {
 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)
   } else {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     return useFragment_LEGACY(fragment, key);
   }
 }

package/lib/relay-hooks/useFragmentNode.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('./FragmentResource'),
@@ -19,6 +8,7 @@ var _require2 = require('react'),
   useEffect = _require2.useEffect,
   useState = _require2.useState;
 var _require3 = require('relay-runtime'),
+  RelayFeatureFlags = _require3.RelayFeatureFlags,
   getFragmentIdentifier = _require3.getFragmentIdentifier;
 var warning = require("fbjs/lib/warning");
 function useFragmentNode(fragmentNode, fragmentRef, componentDisplayName) {
@@ -28,8 +18,6 @@ function useFragmentNode(fragmentNode, fragmentRef, componentDisplayName) {
   var _useState = useState(0),
     forceUpdate = _useState[1];
   var fragmentIdentifier = getFragmentIdentifier(fragmentNode, fragmentRef);
-
-  // Read fragment data; this might suspend.
   var fragmentResult = FragmentResource.readWithIdentifier(fragmentNode, fragmentRef, fragmentIdentifier, componentDisplayName);
   var isListeningForUpdatesRef = useUnsafeRef_DEPRECATED(true);
   function enableStoreUpdates() {
@@ -46,36 +34,19 @@ function useFragmentNode(fragmentNode, fragmentRef, componentDisplayName) {
     if (isMountedRef.current === false || isListeningForUpdatesRef.current === false) {
       return;
     }
-
-    // React bails out on noop state updates as an optimization.
-    // If we want to force an update via setState, we need to pass an value.
-    // The actual value can be arbitrary though, e.g. an incremented number.
     forceUpdate(function (count) {
       return count + 1;
     });
   }
-
-  // Establish Relay store subscriptions in the commit phase, only if
-  // rendering for the first time, or if we need to subscribe to new data
-  // If the fragment identifier changes, it means that the variables on the
-  // fragment owner changed, or the fragment ref points to different records.
-  // In this case, we need to resubscribe to the Relay store.
   useEffect(function () {
     isMountedRef.current = true;
     var disposable = FragmentResource.subscribe(fragmentResult, handleDataUpdate);
     return function () {
-      // When unmounting or resubscribing to new data, clean up current
-      // subscription. This will also make sure fragment data is no longer
-      // cached so that next time it its read, it will be freshly read from
-      // the Relay store
       isMountedRef.current = false;
       disposable.dispose();
     };
-    // NOTE: We disable react-hooks-deps warning because environment and fragmentIdentifier
-    // is capturing all information about whether the effect should be re-ran.
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [environment, fragmentIdentifier]);
-  if (process.env.NODE_ENV !== "production") {
+  if (RelayFeatureFlags.LOG_MISSING_RECORDS_IN_PROD || process.env.NODE_ENV !== "production") {
     if (fragmentRef != null && (fragmentResult.data === undefined || Array.isArray(fragmentResult.data) && fragmentResult.data.length > 0 && fragmentResult.data.every(function (data) {
       return data === undefined;
     }))) {
@@ -83,7 +54,6 @@ function useFragmentNode(fragmentNode, fragmentRef, componentDisplayName) {
     }
   }
   return {
-    // $FlowFixMe[incompatible-return]
     data: fragmentResult.data,
     disableStoreUpdates: disableStoreUpdates,
     enableStoreUpdates: enableStoreUpdates

package/lib/relay-hooks/useIsMountedRef.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('react'),

package/lib/relay-hooks/useIsOperationNodeActive.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 useRelayEnvironment = require('./useRelayEnvironment');

package/lib/relay-hooks/useIsParentQueryActive.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 useIsOperationNodeActive = require('./useIsOperationNodeActive');

package/lib/relay-hooks/useLazyLoadQuery.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'),
@@ -18,14 +7,7 @@ var useMemoOperationDescriptor = require('./useMemoOperationDescriptor');
 var useRelayEnvironment = require('./useRelayEnvironment');
 var _require2 = require('relay-runtime'),
   fetchQuery = _require2.__internal.fetchQuery;
-
-// This separate type export is only needed as long as we are injecting
-// a separate hooks implementation in ./HooksImplementation -- it can
-// be removed after we stop doing that.
-
 function useLazyLoadQuery(gqlQuery, variables, options) {
-  // We need to use this hook in order to be able to track if
-  // loadQuery was called during render
   useTrackLoadQueryInRender();
   var environment = useRelayEnvironment();
   var query = useMemoOperationDescriptor(gqlQuery, variables, options && options.networkCacheConfig ? options.networkCacheConfig : {

package/lib/relay-hooks/useLazyLoadQueryNode.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 ProfilerContext = require('./ProfilerContext');
@@ -51,26 +40,11 @@ function useLazyLoadQueryNode(_ref) {
   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 cached resource will have been disposed
-      // by the previous cleanup, so instead of attempting to
-      // do our regular commit setup, which would incorrectly attempt to
-      // retain a cached query resource that was disposed, we need to force
-      // a re-render so that the cache entry for this query is re-intiliazed and
-      // and re-evaluated (and potentially cause a refetch).
       maybeHiddenOrFastRefresh.current = false;
       forceUpdate(function (n) {
         return n + 1;
@@ -81,19 +55,10 @@ function useLazyLoadQueryNode(_ref) {
     return function () {
       disposable.dispose();
     };
-    // NOTE: We disable react-hooks-deps warning because the `environment`
-    // and `cacheIdentifier` identities are capturing all information about whether
-    // the effect should be re-executed and the query re-retained.
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [environment, cacheIdentifier]);
   useEffect(function () {
-    // Release any temporary retain that's not released. At this point, if the
-    // cacheIdentifier doesn't change, the query is still permanently retained,
-    // and the temporary retain is redundant.
     QueryResource.releaseTemporaryRetain(preparedQueryResult);
-    // This effect is intended to run on every commit, thus no dependency
   });
-
   var fragmentNode = preparedQueryResult.fragmentNode,
     fragmentRef = preparedQueryResult.fragmentRef;
   var _useFragmentNode = useFragmentNode(fragmentNode, fragmentRef, componentDisplayName),

package/lib/relay-hooks/useLoadMoreFunction.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,6 +16,7 @@ var _require2 = require('relay-runtime'),
   ConnectionInterface = _require2.ConnectionInterface,
   createOperationDescriptor = _require2.createOperationDescriptor,
   getPaginationVariables = _require2.getPaginationVariables,
+  getRefetchMetadata = _require2.getRefetchMetadata,
   getSelector = _require2.getSelector,
   getValueAtPath = _require2.getValueAtPath;
 var warning = require("fbjs/lib/warning");
@@ -41,15 +31,16 @@ function useLoadMoreFunction(args) {
     paginationMetadata = args.paginationMetadata,
     componentDisplayName = args.componentDisplayName,
     observer = args.observer,
-    onReset = args.onReset,
-    identifierField = args.identifierField;
+    onReset = args.onReset;
   var environment = useRelayEnvironment();
   var _useFetchTrackingRef = useFetchTrackingRef(),
     isFetchingRef = _useFetchTrackingRef.isFetchingRef,
     startFetch = _useFetchTrackingRef.startFetch,
     disposeFetch = _useFetchTrackingRef.disposeFetch,
     completeFetch = _useFetchTrackingRef.completeFetch;
-  var identifierValue = identifierField != null && fragmentData != null && typeof fragmentData === 'object' ? fragmentData[identifierField] : null;
+  var _getRefetchMetadata = getRefetchMetadata(fragmentNode, componentDisplayName),
+    identifierInfo = _getRefetchMetadata.identifierInfo;
+  var identifierValue = (identifierInfo === null || identifierInfo === void 0 ? void 0 : identifierInfo.identifierField) != null && fragmentData != null && typeof fragmentData === 'object' ? fragmentData[identifierInfo.identifierField] : null;
   var isMountedRef = useIsMountedRef();
   var _useState = useState(environment),
     mirroredEnvironment = _useState[0],
@@ -68,20 +59,14 @@ function useLoadMoreFunction(args) {
   var _getConnectionState = getConnectionState(direction, fragmentNode, fragmentData, connectionPathInFragmentData),
     cursor = _getConnectionState.cursor,
     hasMore = _getConnectionState.hasMore;
-
-  // Dispose of pagination requests in flight when unmounting
   useEffect(function () {
     return function () {
       disposeFetch();
     };
   }, [disposeFetch]);
   var loadMore = useCallback(function (count, options) {
-    // TODO(T41131846): Fetch/Caching policies for loadMore
-
     var onComplete = options === null || options === void 0 ? void 0 : options.onComplete;
     if (isMountedRef.current !== true) {
-      // Bail out and warn if we're trying to paginate after the component
-      // has unmounted
       process.env.NODE_ENV !== "production" ? warning(false, 'Relay: Unexpected fetch 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 {
         dispose: function dispose() {}
@@ -105,17 +90,11 @@ function useLoadMoreFunction(args) {
     var extraVariables = options === null || options === void 0 ? void 0 : options.UNSTABLE_extraVariables;
     var baseVariables = (0, _objectSpread2["default"])((0, _objectSpread2["default"])({}, parentVariables), fragmentVariables);
     var paginationVariables = getPaginationVariables(direction, count, cursor, baseVariables, (0, _objectSpread2["default"])({}, extraVariables), paginationMetadata);
-
-    // 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) {
-      // @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) {
       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;
       }
-      paginationVariables.id = identifierValue;
+      paginationVariables[identifierInfo.identifierQueryVariableName] = identifierValue;
     }
     var paginationQuery = createOperationDescriptor(paginationRequest, paginationVariables, {
       force: true
@@ -139,11 +118,7 @@ function useLoadMoreFunction(args) {
     return {
       dispose: disposeFetch
     };
-  },
-  // NOTE: We disable react-hooks-deps warning because all values
-  // inside paginationMetadata are static
-  // eslint-disable-next-line react-hooks/exhaustive-deps
-  [environment, identifierValue, direction, cursor, startFetch, disposeFetch, completeFetch, isFetchingRef, isParentQueryActive, fragmentData, fragmentNode.name, fragmentRef, componentDisplayName]);
+  }, [environment, identifierValue, direction, cursor, startFetch, disposeFetch, completeFetch, isFetchingRef, isParentQueryActive, fragmentData, fragmentNode.name, fragmentRef, componentDisplayName]);
   return [loadMore, hasMore, disposeFetch];
 }
 function getConnectionState(direction, fragmentNode, fragmentData, connectionPathInFragmentData) {

package/lib/relay-hooks/useMemoOperationDescriptor.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 useMemoVariables = require('./useMemoVariables');

package/lib/relay-hooks/useMemoVariables.js

@@ -1,25 +1,8 @@
-/**
- * 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 areEqual = require("fbjs/lib/areEqual");
 var _require = require('react'),
   useState = _require.useState;
-
-/**
- * Memoizes the passed in `variables` object based on `areEqual` equality.
- * This is useful when a `variables` object is used as a value in a depencency
- * array as it might often be constructed during render.
- */
 function useMemoVariables(variables) {
   var _useState = useState(variables),
     mirroredVariables = _useState[0],

package/lib/relay-hooks/useMutation.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"];

package/lib/relay-hooks/usePaginationFragment.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"];
@@ -25,11 +14,6 @@ var _require2 = require('relay-runtime'),
   getFragment = _require2.getFragment,
   getFragmentIdentifier = _require2.getFragmentIdentifier,
   getPaginationMetadata = _require2.getPaginationMetadata;
-
-// This separate type export is only needed as long as we are injecting
-// a separate hooks implementation in ./HooksImplementation -- it can
-// be removed after we stop doing that.
-
 function usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef) {
   var fragmentNode = getFragment(fragmentInput);
   useStaticFragmentNodeWarning(fragmentNode, 'first argument of usePaginationFragment()');
@@ -37,15 +21,12 @@ function usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef) {
   var _getPaginationMetadat = getPaginationMetadata(fragmentNode, componentDisplayName),
     connectionPathInFragmentData = _getPaginationMetadat.connectionPathInFragmentData,
     paginationRequest = _getPaginationMetadat.paginationRequest,
-    paginationMetadata = _getPaginationMetadat.paginationMetadata,
-    identifierField = _getPaginationMetadat.identifierField;
+    paginationMetadata = _getPaginationMetadat.paginationMetadata;
   var _useRefetchableFragme = useRefetchableFragmentNode(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,
@@ -54,7 +35,6 @@ function usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
@@ -62,8 +42,6 @@ function usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef) {
     hasPrevious = _useLoadMore[1],
     isLoadingPrevious = _useLoadMore[2],
     disposeFetchPrevious = _useLoadMore[3];
-
-  // Forward pagination
   var _useLoadMore2 = useLoadMore({
       componentDisplayName: componentDisplayName,
       connectionPathInFragmentData: connectionPathInFragmentData,
@@ -72,7 +50,6 @@ function usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef) {
       fragmentIdentifier: fragmentIdentifier,
       fragmentNode: fragmentNode,
       fragmentRef: fragmentRef,
-      identifierField: identifierField,
       paginationMetadata: paginationMetadata,
       paginationRequest: paginationRequest
     }),
@@ -88,7 +65,6 @@ function usePaginationFragment_LEGACY(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,
@@ -141,7 +117,6 @@ function usePaginationFragment(fragmentInput, parentFragmentRef) {
   if (impl) {
     return impl.usePaginationFragment(fragmentInput, parentFragmentRef);
   } else {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     return usePaginationFragment_LEGACY(fragmentInput, parentFragmentRef);
   }
 }

package/lib/relay-hooks/usePreloadedQuery.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'),
@@ -25,8 +14,6 @@ var _require3 = require('relay-runtime'),
   fetchQuery = _require3$__internal.fetchQuery;
 var warning = require("fbjs/lib/warning");
 function usePreloadedQuery(gqlQuery, preloadedQuery, options) {
-  // We need to use this hook in order to be able to track if
-  // loadQuery was called during render
   useTrackLoadQueryInRender();
   var environment = useRelayEnvironment();
   var fetchKey = preloadedQuery.fetchKey,
@@ -62,24 +49,11 @@ function usePreloadedQuery(gqlQuery, preloadedQuery, options) {
     var fallbackFetchObservable = fetchQuery(environment, operation);
     var fetchObservable;
     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;
     }
     useLazyLoadQueryNodeParams = {
@@ -93,7 +67,6 @@ function usePreloadedQuery(gqlQuery, preloadedQuery, options) {
   }
   var data = useLazyLoadQueryNode(useLazyLoadQueryNodeParams);
   if (process.env.NODE_ENV !== "production") {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     useDebugValue({
       query: preloadedQuery.name,
       variables: preloadedQuery.variables,