react-relay 17.0.0 → 18.1.0

package/relay-hooks/useFragmentInternal.js.flow

@@ -11,659 +11,35 @@
 
 'use strict';
 
-import type {QueryResult} from './QueryResource';
-import type {
-  CacheConfig,
-  FetchPolicy,
-  IEnvironment,
-  ReaderFragment,
-  ReaderSelector,
-  SelectorData,
-  Snapshot,
-} from 'relay-runtime';
-import type {
-  MissingClientEdgeRequestInfo,
-  MissingLiveResolverField,
-} from 'relay-runtime/store/RelayStoreTypes';
+import type {FragmentQueryOptions} from './useFragmentInternal_EXPERIMENTAL';
+import type {ReaderFragment, SelectorData} from 'relay-runtime';
 
-const {getQueryResourceForEnvironment} = require('./QueryResource');
-const useRelayEnvironment = require('./useRelayEnvironment');
-const invariant = require('invariant');
-const {useDebugValue, useEffect, useMemo, useRef, useState} = require('react');
-const {
-  __internal: {fetchQuery: fetchQueryInternal, getPromiseForActiveRequest},
-  RelayFeatureFlags,
-  areEqualSelectors,
-  createOperationDescriptor,
-  getPendingOperationsForFragment,
-  getSelector,
-  getVariablesFromFragment,
-  handlePotentialSnapshotErrors,
-  recycleNodesInto,
-} = require('relay-runtime');
-const warning = require('warning');
+import useFragmentInternal_CURRENT from './useFragmentInternal_CURRENT';
+import useFragmentInternal_EXPERIMENTAL from './useFragmentInternal_EXPERIMENTAL';
+import {RelayFeatureFlags} from 'relay-runtime';
 
-type FragmentQueryOptions = {
-  fetchPolicy?: FetchPolicy,
-  networkCacheConfig?: ?CacheConfig,
-};
-
-type FragmentState = $ReadOnly<
-  | {kind: 'bailout'}
-  | {kind: 'singular', snapshot: Snapshot, epoch: number}
-  | {kind: 'plural', snapshots: $ReadOnlyArray<Snapshot>, epoch: number},
->;
-
-type StateUpdaterFunction<T> = ((T) => T) => void;
-
-function isMissingData(state: FragmentState): boolean {
-  if (state.kind === 'bailout') {
-    return false;
-  } else if (state.kind === 'singular') {
-    return state.snapshot.isMissingData;
-  } else {
-    return state.snapshots.some(s => s.isMissingData);
-  }
-}
-
-function getMissingClientEdges(
-  state: FragmentState,
-): $ReadOnlyArray<MissingClientEdgeRequestInfo> | null {
-  if (state.kind === 'bailout') {
-    return null;
-  } else if (state.kind === 'singular') {
-    return state.snapshot.missingClientEdges ?? null;
-  } else {
-    let edges: null | Array<MissingClientEdgeRequestInfo> = null;
-    for (const snapshot of state.snapshots) {
-      if (snapshot.missingClientEdges) {
-        edges = edges ?? [];
-        for (const edge of snapshot.missingClientEdges) {
-          edges.push(edge);
-        }
-      }
-    }
-    return edges;
-  }
-}
-
-function getSuspendingLiveResolver(
-  state: FragmentState,
-): $ReadOnlyArray<MissingLiveResolverField> | null {
-  if (state.kind === 'bailout') {
-    return null;
-  } else if (state.kind === 'singular') {
-    return state.snapshot.missingLiveResolverFields ?? null;
-  } else {
-    let missingFields: null | Array<MissingLiveResolverField> = null;
-    for (const snapshot of state.snapshots) {
-      if (snapshot.missingLiveResolverFields) {
-        missingFields = missingFields ?? [];
-        for (const edge of snapshot.missingLiveResolverFields) {
-          missingFields.push(edge);
-        }
-      }
-    }
-    return missingFields;
-  }
-}
-
-function handlePotentialSnapshotErrorsForState(
-  environment: IEnvironment,
-  state: FragmentState,
-): void {
-  if (state.kind === 'singular') {
-    handlePotentialSnapshotErrors(
-      environment,
-      state.snapshot.missingRequiredFields,
-      state.snapshot.relayResolverErrors,
-      state.snapshot.errorResponseFields,
-      state.snapshot.selector.node.metadata?.throwOnFieldError ?? false,
-    );
-  } else if (state.kind === 'plural') {
-    for (const snapshot of state.snapshots) {
-      handlePotentialSnapshotErrors(
-        environment,
-        snapshot.missingRequiredFields,
-        snapshot.relayResolverErrors,
-        snapshot.errorResponseFields,
-        snapshot.selector.node.metadata?.throwOnFieldError ?? false,
-      );
-    }
-  }
-}
-
-/**
- * Check for updates to the store that occurred concurrently with rendering the given `state` value,
- * returning a new (updated) state if there were updates or null if there were no changes.
- */
-function handleMissedUpdates(
-  environment: IEnvironment,
-  state: FragmentState,
-): null | [/* has data changed */ boolean, FragmentState] {
-  if (state.kind === 'bailout') {
-    return null;
-  }
-  // FIXME this is invalid if we've just switched environments.
-  const currentEpoch = environment.getStore().getEpoch();
-  if (currentEpoch === state.epoch) {
-    return null;
-  }
-  // The store has updated since we rendered (without us being subscribed yet),
-  // so check for any updates to the data we're rendering:
-  if (state.kind === 'singular') {
-    const currentSnapshot = environment.lookup(state.snapshot.selector);
-    const updatedData = recycleNodesInto(
-      state.snapshot.data,
-      currentSnapshot.data,
-    );
-    const updatedCurrentSnapshot: Snapshot = {
-      data: updatedData,
-      isMissingData: currentSnapshot.isMissingData,
-      missingClientEdges: currentSnapshot.missingClientEdges,
-      missingLiveResolverFields: currentSnapshot.missingLiveResolverFields,
-      seenRecords: currentSnapshot.seenRecords,
-      selector: currentSnapshot.selector,
-      missingRequiredFields: currentSnapshot.missingRequiredFields,
-      relayResolverErrors: currentSnapshot.relayResolverErrors,
-      errorResponseFields: currentSnapshot.errorResponseFields,
-    };
-    return [
-      updatedData !== state.snapshot.data,
-      {
-        kind: 'singular',
-        snapshot: updatedCurrentSnapshot,
-        epoch: currentEpoch,
-      },
-    ];
-  } else {
-    let didMissUpdates = false;
-    const currentSnapshots = [];
-    for (let index = 0; index < state.snapshots.length; index++) {
-      const snapshot = state.snapshots[index];
-      const currentSnapshot = environment.lookup(snapshot.selector);
-      const updatedData = recycleNodesInto(snapshot.data, currentSnapshot.data);
-      const updatedCurrentSnapshot: Snapshot = {
-        data: updatedData,
-        isMissingData: currentSnapshot.isMissingData,
-        missingClientEdges: currentSnapshot.missingClientEdges,
-        missingLiveResolverFields: currentSnapshot.missingLiveResolverFields,
-        seenRecords: currentSnapshot.seenRecords,
-        selector: currentSnapshot.selector,
-        missingRequiredFields: currentSnapshot.missingRequiredFields,
-        relayResolverErrors: currentSnapshot.relayResolverErrors,
-        errorResponseFields: currentSnapshot.errorResponseFields,
-      };
-      if (updatedData !== snapshot.data) {
-        didMissUpdates = true;
-      }
-      currentSnapshots.push(updatedCurrentSnapshot);
-    }
-    invariant(
-      currentSnapshots.length === state.snapshots.length,
-      'Expected same number of snapshots',
-    );
-    return [
-      didMissUpdates,
-      {
-        kind: 'plural',
-        snapshots: currentSnapshots,
-        epoch: currentEpoch,
-      },
-    ];
-  }
-}
-
-function handleMissingClientEdge(
-  environment: IEnvironment,
-  parentFragmentNode: ReaderFragment,
-  parentFragmentRef: mixed,
-  missingClientEdgeRequestInfo: MissingClientEdgeRequestInfo,
-  queryOptions?: FragmentQueryOptions,
-): [QueryResult, ?Promise<mixed>] {
-  const originalVariables = getVariablesFromFragment(
-    parentFragmentNode,
-    parentFragmentRef,
-  );
-  const variables = {
-    ...originalVariables,
-    id: missingClientEdgeRequestInfo.clientEdgeDestinationID, // TODO should be a reserved name
-  };
-  const queryOperationDescriptor = createOperationDescriptor(
-    missingClientEdgeRequestInfo.request,
-    variables,
-    queryOptions?.networkCacheConfig,
-  );
-  // This may suspend. We don't need to do anything with the results; all we're
-  // doing here is started the query if needed and retaining and releasing it
-  // according to the component mount/suspense cycle; QueryResource
-  // already handles this by itself.
-  const QueryResource = getQueryResourceForEnvironment(environment);
-  const queryResult = QueryResource.prepare(
-    queryOperationDescriptor,
-    fetchQueryInternal(environment, queryOperationDescriptor),
-    queryOptions?.fetchPolicy,
-  );
-
-  return [
-    queryResult,
-    getPromiseForActiveRequest(environment, queryOperationDescriptor.request),
-  ];
-}
-
-function subscribeToSnapshot(
-  environment: IEnvironment,
-  state: FragmentState,
-  setState: StateUpdaterFunction<FragmentState>,
-  hasPendingStateChanges: {current: boolean},
-): () => void {
-  if (state.kind === 'bailout') {
-    return () => {};
-  } else if (state.kind === 'singular') {
-    const disposable = environment.subscribe(state.snapshot, latestSnapshot => {
-      setState(prevState => {
-        // In theory a setState from a subscription could be batched together
-        // with a setState to change the fragment selector. Guard against this
-        // by bailing out of the state update if the selector has changed.
-        if (
-          prevState.kind !== 'singular' ||
-          prevState.snapshot.selector !== latestSnapshot.selector
-        ) {
-          const updates = handleMissedUpdates(environment, prevState);
-          if (updates != null) {
-            const [dataChanged, nextState] = updates;
-            environment.__log({
-              name: 'useFragment.subscription.missedUpdates',
-              hasDataChanges: dataChanged,
-            });
-            hasPendingStateChanges.current = dataChanged;
-            return dataChanged ? nextState : prevState;
-          } else {
-            return prevState;
-          }
-        }
-
-        hasPendingStateChanges.current = true;
-        return {
-          kind: 'singular',
-          snapshot: latestSnapshot,
-          epoch: environment.getStore().getEpoch(),
-        };
-      });
-    });
-    return () => {
-      disposable.dispose();
-    };
-  } else {
-    const disposables = state.snapshots.map((snapshot, index) =>
-      environment.subscribe(snapshot, latestSnapshot => {
-        setState(prevState => {
-          // In theory a setState from a subscription could be batched together
-          // with a setState to change the fragment selector. Guard against this
-          // by bailing out of the state update if the selector has changed.
-          if (
-            prevState.kind !== 'plural' ||
-            prevState.snapshots[index]?.selector !== latestSnapshot.selector
-          ) {
-            const updates = handleMissedUpdates(environment, prevState);
-            if (updates != null) {
-              const [dataChanged, nextState] = updates;
-              environment.__log({
-                name: 'useFragment.subscription.missedUpdates',
-                hasDataChanges: dataChanged,
-              });
-              hasPendingStateChanges.current =
-                hasPendingStateChanges.current || dataChanged;
-              return dataChanged ? nextState : prevState;
-            } else {
-              return prevState;
-            }
-          }
-          const updated = [...prevState.snapshots];
-          updated[index] = latestSnapshot;
-          hasPendingStateChanges.current = true;
-          return {
-            kind: 'plural',
-            snapshots: updated,
-            epoch: environment.getStore().getEpoch(),
-          };
-        });
-      }),
-    );
-    return () => {
-      for (const d of disposables) {
-        d.dispose();
-      }
-    };
-  }
-}
-
-function getFragmentState(
-  environment: IEnvironment,
-  fragmentSelector: ?ReaderSelector,
-): FragmentState {
-  if (fragmentSelector == null) {
-    return {kind: 'bailout'};
-  } else if (fragmentSelector.kind === 'PluralReaderSelector') {
-    // Note that if fragmentRef is an empty array, fragmentSelector will be null so we'll hit the above case.
-    // Null is returned by getSelector if fragmentRef has no non-null items.
-    return {
-      kind: 'plural',
-      snapshots: fragmentSelector.selectors.map(s => environment.lookup(s)),
-      epoch: environment.getStore().getEpoch(),
-    };
-  } else {
-    return {
-      kind: 'singular',
-      snapshot: environment.lookup(fragmentSelector),
-      epoch: environment.getStore().getEpoch(),
-    };
-  }
-}
-
-// fragmentNode cannot change during the lifetime of the component, though fragmentRef may change.
 hook useFragmentInternal(
   fragmentNode: ReaderFragment,
   fragmentRef: mixed,
   hookDisplayName: string,
   queryOptions?: FragmentQueryOptions,
 ): ?SelectorData | Array<?SelectorData> {
-  const fragmentSelector = useMemo(
-    () => getSelector(fragmentNode, fragmentRef),
-    [fragmentNode, fragmentRef],
-  );
-
-  const isPlural = fragmentNode?.metadata?.plural === true;
-
-  if (isPlural) {
-    invariant(
-      fragmentRef == null || Array.isArray(fragmentRef),
-      'Relay: Expected fragment pointer%s for fragment `%s` to be ' +
-        'an array, instead got `%s`. Remove `@relay(plural: true)` ' +
-        'from fragment `%s` to allow the prop to be an object.',
-      fragmentNode.name,
-      typeof fragmentRef,
-      fragmentNode.name,
-    );
-  } else {
-    invariant(
-      !Array.isArray(fragmentRef),
-      'Relay: Expected fragment pointer%s for fragment `%s` not to be ' +
-        'an array, instead got `%s`. Add `@relay(plural: true)` ' +
-        'to fragment `%s` to allow the prop to be an array.',
-      fragmentNode.name,
-      typeof fragmentRef,
-      fragmentNode.name,
-    );
-  }
-  invariant(
-    fragmentRef == null ||
-      (isPlural && Array.isArray(fragmentRef) && fragmentRef.length === 0) ||
-      fragmentSelector != null,
-    'Relay: Expected to receive an object where `...%s` was spread, ' +
-      'but the fragment reference was not found`. This is most ' +
-      'likely the result of:\n' +
-      "- Forgetting to spread `%s` in `%s`'s parent's fragment.\n" +
-      '- Conditionally fetching `%s` but unconditionally passing %s prop ' +
-      'to `%s`. If the parent fragment only fetches the fragment conditionally ' +
-      '- with e.g. `@include`, `@skip`, or inside a `... on SomeType { }` ' +
-      'spread  - then the fragment reference will not exist. ' +
-      'In this case, pass `null` if the conditions for evaluating the ' +
-      'fragment are not met (e.g. if the `@include(if)` value is false.)',
-    fragmentNode.name,
-    fragmentNode.name,
+  if (RelayFeatureFlags.ENABLE_ACTIVITY_COMPATIBILITY) {
+    // $FlowFixMe[react-rule-hook] - the condition is static
+    return useFragmentInternal_EXPERIMENTAL(
+      fragmentNode,
+      fragmentRef,
+      hookDisplayName,
+      queryOptions,
+    );
+  }
+  // $FlowFixMe[react-rule-hook] - the condition is static
+  return useFragmentInternal_CURRENT(
+    fragmentNode,
+    fragmentRef,
     hookDisplayName,
-    fragmentNode.name,
-    hookDisplayName,
-  );
-
-  const environment = useRelayEnvironment();
-  const [_state, setState] = useState<FragmentState>(() =>
-    getFragmentState(environment, fragmentSelector),
+    queryOptions,
   );
-  let state = _state;
-
-  // This copy of the state we only update when something requires us to
-  // unsubscribe and re-subscribe, namely a changed environment or
-  // fragment selector.
-  const [_subscribedState, setSubscribedState] = useState(state);
-  // FIXME since this is used as an effect dependency, it needs to be memoized.
-  let subscribedState = _subscribedState;
-
-  const [previousFragmentSelector, setPreviousFragmentSelector] =
-    useState(fragmentSelector);
-  const [previousEnvironment, setPreviousEnvironment] = useState(environment);
-  if (
-    !areEqualSelectors(fragmentSelector, previousFragmentSelector) ||
-    environment !== previousEnvironment
-  ) {
-    // Enqueue setState to record the new selector and state
-    setPreviousFragmentSelector(fragmentSelector);
-    setPreviousEnvironment(environment);
-    const newState = getFragmentState(environment, fragmentSelector);
-    setState(newState);
-    setSubscribedState(newState); // This causes us to form a new subscription
-    // But render with the latest state w/o waiting for the setState. Otherwise
-    // the component would render the wrong information temporarily (including
-    // possibly incorrectly triggering some warnings below).
-    state = newState;
-    subscribedState = newState;
-  }
-
-  // The purpose of this is to detect whether we have ever committed, because we
-  // don't suspend on store updates, only when the component either is first trying
-  // to mount or when the our selector changes. The selector change in particular is
-  // how we suspend for pagination and refetch. Also, fragment selector can be null
-  // or undefined, so we use false as a special value to distinguish from all fragment
-  // selectors; false means that the component hasn't mounted yet.
-  const committedFragmentSelectorRef = useRef<false | ?ReaderSelector>(false);
-  useEffect(() => {
-    committedFragmentSelectorRef.current = fragmentSelector;
-  }, [fragmentSelector]);
-
-  // Handle the queries for any missing client edges; this may suspend.
-  // FIXME handle client edges in parallel.
-  if (fragmentNode.metadata?.hasClientEdges === true) {
-    // The fragment is validated to be static (in useFragment) and hasClientEdges is
-    // a static (constant) property of the fragment. In practice, this effect will
-    // always or never run for a given invocation of this hook.
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    // $FlowFixMe[react-rule-hook]
-    const [clientEdgeQueries, activeRequestPromises] = useMemo(() => {
-      const missingClientEdges = getMissingClientEdges(state);
-      // eslint-disable-next-line no-shadow
-      let clientEdgeQueries;
-      const activeRequestPromises = [];
-      if (missingClientEdges?.length) {
-        clientEdgeQueries = ([]: Array<QueryResult>);
-        for (const edge of missingClientEdges) {
-          const [queryResult, requestPromise] = handleMissingClientEdge(
-            environment,
-            fragmentNode,
-            fragmentRef,
-            edge,
-            queryOptions,
-          );
-          clientEdgeQueries.push(queryResult);
-          if (requestPromise != null) {
-            activeRequestPromises.push(requestPromise);
-          }
-        }
-      }
-      return [clientEdgeQueries, activeRequestPromises];
-    }, [state, environment, fragmentNode, fragmentRef, queryOptions]);
-
-    if (activeRequestPromises.length) {
-      throw Promise.all(activeRequestPromises);
-    }
-
-    // See above note
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    // $FlowFixMe[react-rule-hook]
-    useEffect(() => {
-      const QueryResource = getQueryResourceForEnvironment(environment);
-      if (clientEdgeQueries?.length) {
-        const disposables = [];
-        for (const query of clientEdgeQueries) {
-          disposables.push(QueryResource.retain(query));
-        }
-        return () => {
-          for (const disposable of disposables) {
-            disposable.dispose();
-          }
-        };
-      }
-    }, [environment, clientEdgeQueries]);
-  }
-
-  if (isMissingData(state)) {
-    // Suspend if a Live Resolver within this fragment is in a suspended state:
-    const suspendingLiveResolvers = getSuspendingLiveResolver(state);
-    if (suspendingLiveResolvers != null && suspendingLiveResolvers.length > 0) {
-      throw Promise.all(
-        suspendingLiveResolvers.map(({liveStateID}) => {
-          // $FlowFixMe[prop-missing] This is expected to be a LiveResolverStore
-          return environment.getStore().getLiveResolverPromise(liveStateID);
-        }),
-      );
-    }
-    // Suspend if an active operation bears on this fragment, either the
-    // fragment's owner or some other mutation etc. that could affect it.
-    // We only suspend when the component is first trying to mount or changing
-    // selectors, not if data becomes missing later:
-    if (
-      RelayFeatureFlags.ENABLE_RELAY_OPERATION_TRACKER_SUSPENSE ||
-      environment !== previousEnvironment ||
-      !committedFragmentSelectorRef.current ||
-      // $FlowFixMe[react-rule-unsafe-ref]
-      !areEqualSelectors(committedFragmentSelectorRef.current, fragmentSelector)
-    ) {
-      invariant(fragmentSelector != null, 'refinement, see invariants above');
-      const fragmentOwner =
-        fragmentSelector.kind === 'PluralReaderSelector'
-          ? fragmentSelector.selectors[0].owner
-          : fragmentSelector.owner;
-      const pendingOperationsResult = getPendingOperationsForFragment(
-        environment,
-        fragmentNode,
-        fragmentOwner,
-      );
-      if (pendingOperationsResult) {
-        throw pendingOperationsResult.promise;
-      }
-    }
-  }
-
-  // Report required fields only if we're not suspending, since that means
-  // they're missing even though we are out of options for possibly fetching them:
-  handlePotentialSnapshotErrorsForState(environment, state);
-
-  const hasPendingStateChanges = useRef<boolean>(false);
-
-  useEffect(() => {
-    // Check for updates since the state was rendered
-    let currentState = subscribedState;
-    const updates = handleMissedUpdates(environment, subscribedState);
-    if (updates !== null) {
-      const [didMissUpdates, updatedState] = updates;
-      // TODO: didMissUpdates only checks for changes to snapshot data, but it's possible
-      // that other snapshot properties may have changed that should also trigger a re-render,
-      // such as changed missing resolver fields, missing client edges, etc.
-      // A potential alternative is for handleMissedUpdates() to recycle the entire state
-      // value, and return the new (recycled) state only if there was some change. In that
-      // case the code would always setState if something in the snapshot changed, in addition
-      // to using the latest snapshot to subscribe.
-      if (didMissUpdates) {
-        setState(updatedState);
-      }
-      currentState = updatedState;
-    }
-    return subscribeToSnapshot(
-      environment,
-      currentState,
-      setState,
-      hasPendingStateChanges,
-    );
-  }, [environment, subscribedState]);
-
-  if (hasPendingStateChanges.current) {
-    const updates = handleMissedUpdates(environment, state);
-    if (updates != null) {
-      const [hasStateUpdates, updatedState] = updates;
-      if (hasStateUpdates) {
-        setState(updatedState);
-        state = updatedState;
-      }
-    }
-    // $FlowFixMe[react-rule-unsafe-ref]
-    hasPendingStateChanges.current = false;
-  }
-
-  let data: ?SelectorData | Array<?SelectorData>;
-  if (isPlural) {
-    // Plural fragments require allocating an array of the snapshot data values,
-    // which has to be memoized to avoid triggering downstream re-renders.
-    //
-    // Note that isPlural is a constant property of the fragment and does not change
-    // for a particular useFragment invocation site
-    const fragmentRefIsNullish = fragmentRef == null; // for less sensitive memoization
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    // $FlowFixMe[react-rule-hook]
-    data = useMemo(() => {
-      if (state.kind === 'bailout') {
-        // Bailout state can happen if the fragmentRef is a plural array that is empty or has no
-        // non-null entries. In that case, the compatible behavior is to return [] instead of null.
-        return fragmentRefIsNullish ? null : [];
-      } else {
-        invariant(
-          state.kind === 'plural',
-          'Expected state to be plural because fragment is plural',
-        );
-        return state.snapshots.map(s => s.data);
-      }
-    }, [state, fragmentRefIsNullish]);
-  } else if (state.kind === 'bailout') {
-    // This case doesn't allocate a new object so it doesn't have to be memoized
-    data = null;
-  } else {
-    // This case doesn't allocate a new object so it doesn't have to be memoized
-    invariant(
-      state.kind === 'singular',
-      'Expected state to be singular because fragment is singular',
-    );
-    data = state.snapshot.data;
-  }
-
-  if (RelayFeatureFlags.LOG_MISSING_RECORDS_IN_PROD || __DEV__) {
-    if (
-      fragmentRef != null &&
-      (data === undefined ||
-        (Array.isArray(data) &&
-          data.length > 0 &&
-          data.every(d => d === undefined)))
-    ) {
-      warning(
-        false,
-        'Relay: Expected to have been able to read non-null data for ' +
-          'fragment `%s` declared in ' +
-          '`%s`, since fragment reference was non-null. ' +
-          "Make sure that that `%s`'s parent isn't " +
-          'holding on to and/or passing a fragment reference for data that ' +
-          'has been deleted.',
-        fragmentNode.name,
-        hookDisplayName,
-        hookDisplayName,
-      );
-    }
-  }
-
-  if (__DEV__) {
-    // eslint-disable-next-line react-hooks/rules-of-hooks
-    // $FlowFixMe[react-rule-hook]
-    useDebugValue({fragment: fragmentNode.name, data});
-  }
-
-  return data;
 }
 
 module.exports = useFragmentInternal;