react-relay 13.1.0 → 14.0.0

package/ReactRelayContext.js

@@ -1,5 +1,5 @@
 /**
- * Relay v13.1.0
+ * Relay v14.0.0
  *
  * Copyright (c) Meta Platforms, Inc. and affiliates.
  *

package/ReactRelayFragmentContainer.js.flow

@@ -11,11 +11,11 @@
 // flowlint ambiguous-object-type:error
 
 'use strict';
-
 import type {GeneratedNodeMap, RelayProp, $RelayProps} from './ReactRelayTypes';
 import type {
   FragmentMap,
   FragmentSpecResolver,
+  IEnvironment,
   RelayContext,
 } from 'relay-runtime';
 
@@ -58,7 +58,7 @@ function createContainerWithFragments<
 
   return class extends React.Component<ContainerProps, ContainerState> {
     static displayName = containerName;
-    constructor(props) {
+    constructor(props: $FlowFixMe) {
       super(props);
       const relayContext = assertRelayContext(props.__relayContext);
       const rootIsQueryRenderer = props.__rootIsQueryRenderer ?? false;
@@ -163,7 +163,10 @@ function createContainerWithFragments<
       this.state.resolver.dispose();
     }
 
-    shouldComponentUpdate(nextProps, nextState): boolean {
+    shouldComponentUpdate(
+      nextProps: ContainerProps,
+      nextState: ContainerState,
+    ): boolean {
       // Short-circuit if any Relay-related data has changed
       if (nextState.data !== this.state.data) {
         return true;
@@ -247,7 +250,7 @@ function createContainerWithFragments<
   };
 }
 
-function getRelayProp(environment) {
+function getRelayProp(environment: IEnvironment) {
   return {
     environment,
   };

package/ReactRelayLocalQueryRenderer.js.flow

@@ -40,7 +40,7 @@ const queryRendererContext: ReactRelayQueryRendererContextType = {
   rootIsQueryRenderer: true,
 };
 
-function useDeepCompare<T: interface {}>(value: T): T {
+function useDeepCompare<T: {...}>(value: T): T {
   const latestValue = React.useRef(value);
   if (!areEqual(latestValue.current, value)) {
     if (__DEV__) {

package/ReactRelayPaginationContainer.js.flow

@@ -235,7 +235,7 @@ function createGetConnectionFromProps(metadata: ReactConnectionMetadata) {
     'ReactRelayPaginationContainer: Unable to synthesize a ' +
       'getConnectionFromProps function.',
   );
-  return props => {
+  return (props: any) => {
     let data = props[metadata.fragmentName];
     for (let i = 0; i < path.length; i++) {
       if (!data || typeof data !== 'object') {
@@ -264,7 +264,9 @@ function createGetFragmentVariables(
 
 type ReactConnectionMetadata = ConnectionMetadata & {fragmentName: string, ...};
 
-function findConnectionMetadata(fragments): ReactConnectionMetadata {
+function findConnectionMetadata(
+  fragments: FragmentMap,
+): ReactConnectionMetadata {
   let foundConnectionMetadata = null;
   let isRelayModern = false;
   for (const fragmentName in fragments) {
@@ -308,7 +310,7 @@ function toObserver(observerOrCallback: ?ObserverOrCallback): Observer<void> {
     ? {
         error: observerOrCallback,
         complete: observerOrCallback,
-        unsubscribe: subscription => {
+        unsubscribe: (subscription: Subscription) => {
           typeof observerOrCallback === 'function' && observerOrCallback();
         },
       }
@@ -356,7 +358,7 @@ function createContainerWithFragments<
     _isUnmounted: boolean;
     _hasFetched: boolean;
 
-    constructor(props) {
+    constructor(props: any) {
       super(props);
       const relayContext = assertRelayContext(props.__relayContext);
       const rootIsQueryRenderer = props.__rootIsQueryRenderer ?? false;
@@ -415,7 +417,7 @@ function createContainerWithFragments<
      * for updates. Props may be the same in which case previous data and
      * subscriptions can be reused.
      */
-    UNSAFE_componentWillReceiveProps(nextProps) {
+    UNSAFE_componentWillReceiveProps(nextProps: any) {
       const relayContext = assertRelayContext(nextProps.__relayContext);
       const rootIsQueryRenderer = nextProps.__rootIsQueryRenderer ?? false;
       const prevIDs = getDataIDsFromObject(fragments, this.props);
@@ -479,7 +481,10 @@ function createContainerWithFragments<
       this._cleanup();
     }
 
-    shouldComponentUpdate(nextProps, nextState): boolean {
+    shouldComponentUpdate(
+      nextProps: Props,
+      nextState: ContainerState,
+    ): boolean {
       // Short-circuit if any Relay-related data has changed
       if (
         nextState.data !== this.state.data ||
@@ -722,7 +727,7 @@ function createContainerWithFragments<
       return this._queryFetcher;
     }
 
-    _canFetchPage(method): boolean {
+    _canFetchPage(method: 'loadMore' | 'refetchConnection'): boolean {
       if (this._isUnmounted) {
         warning(
           false,
@@ -813,7 +818,7 @@ function createContainerWithFragments<
       }
       this._hasFetched = true;
 
-      const onNext = (payload, complete) => {
+      const onNext = (payload: mixed, complete: () => void) => {
         const prevData = this._resolver.resolve();
         this._resolver.setVariables(
           getFragmentVariables(

package/ReactRelayQueryFetcher.js.flow

@@ -194,6 +194,7 @@ class ReactRelayQueryFetcher {
       onDataChange &&
       this._fetchOptions.onDataChangeCallbacks.indexOf(onDataChange) === -1
     ) {
+      // $FlowFixMe[incompatible-use]
       this._fetchOptions.onDataChangeCallbacks.push(onDataChange);
     }
 

package/ReactRelayQueryRenderer.js.flow

@@ -29,7 +29,6 @@ const ReactRelayQueryRendererContext = require('./ReactRelayQueryRendererContext
 const areEqual = require('areEqual');
 const React = require('react');
 const {
-  RelayFeatureFlags,
   createOperationDescriptor,
   deepFreeze,
   getRequest,
@@ -56,7 +55,12 @@ export type RenderProps<T> = {|
  * constructor. If a request is already in flight from a previous call to the
  * constructor, just reuse the query fetcher and wait for the response.
  */
-const requestCache = {};
+const requestCache: {
+  [string]: void | {|
+    queryFetcher: ReactRelayQueryFetcher,
+    snapshot: ?Snapshot,
+  |},
+} = {};
 
 const queryRendererContext: ReactRelayQueryRendererContextType = {
   rootIsQueryRenderer: true,
@@ -157,10 +161,7 @@ class ReactRelayQueryRenderer extends React.Component<Props, State> {
   }
 
   componentDidMount() {
-    if (
-      RelayFeatureFlags.ENABLE_QUERY_RENDERER_OFFSCREEN_SUPPORT === true &&
-      this._maybeHiddenOrFastRefresh === true
-    ) {
+    if (this._maybeHiddenOrFastRefresh === 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

package/ReactRelayRefetchContainer.js.flow

@@ -24,6 +24,7 @@ import type {
   Disposable,
   FragmentMap,
   GraphQLTaggedNode,
+  IEnvironment,
   RelayContext,
   Subscription,
   Variables,
@@ -86,7 +87,7 @@ function createContainerWithFragments<
     _queryFetcher: ?ReactRelayQueryFetcher;
     _isUnmounted: boolean;
 
-    constructor(props) {
+    constructor(props: any) {
       super(props);
       const relayContext = assertRelayContext(props.__relayContext);
       const rootIsQueryRenderer = props.__rootIsQueryRenderer ?? false;
@@ -217,7 +218,10 @@ function createContainerWithFragments<
       this._refetchSubscription && this._refetchSubscription.unsubscribe();
     }
 
-    shouldComponentUpdate(nextProps, nextState): boolean {
+    shouldComponentUpdate(
+      nextProps: ContainerProps,
+      nextState: ContainerState,
+    ): boolean {
       // Short-circuit if any Relay-related data has changed
       if (
         nextState.data !== this.state.data ||
@@ -465,7 +469,10 @@ function createContainerWithFragments<
   };
 }
 
-function getRelayProp(environment, refetch): RelayRefetchProp {
+function getRelayProp(
+  environment: IEnvironment,
+  refetch: RelayRefetchProp['refetch'],
+): RelayRefetchProp {
   return {
     environment,
     refetch,

package/__flowtests__/__generated__/ReactRelayFragmentContainerFlowtest_viewer.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<56e1daf6db87bf9886b3627a3e0d76f9>>
+ * @generated SignedSource<<23bcef30afc22a42d79dd52e0cfe899c>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/__flowtests__/__generated__/ReactRelayFragmentContainerFlowtest_viewer2.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<c4fb5411e941bc7c5102499b684cf810>>
+ * @generated SignedSource<<af6c4570779066ea564051c9c71c494f>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/__flowtests__/__generated__/ReactRelayPaginationContainerFlowtestQuery.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<059d6a6d94da6f9b3cf43a0418ecc7b8>>
+ * @generated SignedSource<<6d173a357c286b417fdc586a839384d4>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep
@@ -27,8 +27,8 @@ export type ReactRelayPaginationContainerFlowtestQuery$data = {|
   |},
 |};
 export type ReactRelayPaginationContainerFlowtestQuery = {|
-  variables: ReactRelayPaginationContainerFlowtestQuery$variables,
   response: ReactRelayPaginationContainerFlowtestQuery$data,
+  variables: ReactRelayPaginationContainerFlowtestQuery$variables,
 |};
 */
 

package/__flowtests__/__generated__/ReactRelayPaginationContainerFlowtest_viewer.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<86f990ee88078c49934d0f28c6c31da4>>
+ * @generated SignedSource<<b3cddee250d0ff22e89a7f0331e85df6>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep
@@ -23,7 +23,7 @@ export type ReactRelayPaginationContainerFlowtest_viewer$data = {|
     +friends: ?{|
       +edges: ?$ReadOnlyArray<?{|
         +node: ?{|
-          +__typename: string,
+          +__typename: "User",
         |},
       |}>,
     |},

package/__flowtests__/__generated__/ReactRelayRefetchContainerFlowtestQuery.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<a07bf128347274d353ab379de5eaef1f>>
+ * @generated SignedSource<<3b1b16b5d5ec1c94783d6161ae61bc3d>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep
@@ -27,8 +27,8 @@ export type ReactRelayRefetchContainerFlowtestQuery$data = {|
   |},
 |};
 export type ReactRelayRefetchContainerFlowtestQuery = {|
-  variables: ReactRelayRefetchContainerFlowtestQuery$variables,
   response: ReactRelayRefetchContainerFlowtestQuery$data,
+  variables: ReactRelayRefetchContainerFlowtestQuery$variables,
 |};
 */
 

package/__flowtests__/__generated__/ReactRelayRefetchContainerFlowtest_viewer.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<6d9922010e1474b1214c383c394daa24>>
+ * @generated SignedSource<<6d04d7f7bf097d5869a1fdf0dea1e659>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep
@@ -23,7 +23,7 @@ export type ReactRelayRefetchContainerFlowtest_viewer$data = {|
     +friends: ?{|
       +edges: ?$ReadOnlyArray<?{|
         +node: ?{|
-          +__typename: string,
+          +__typename: "User",
         |},
       |}>,
     |},

package/__flowtests__/__generated__/RelayModernFlowtest_badref.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<050146f21476e7ed6edd2243d4c6b65d>>
+ * @generated SignedSource<<d3c71cb965d8374d40fb30f3c5ae9fd4>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/__flowtests__/__generated__/RelayModernFlowtest_notref.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<d8eae7df6faa43d95f6140dc3c9ad3ba>>
+ * @generated SignedSource<<bd60df80c19d8248426865abbc23562e>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/__flowtests__/__generated__/RelayModernFlowtest_user.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<a7ba31d9d2381b0c8b700175be271b75>>
+ * @generated SignedSource<<527809017626cb43378afb92fed67feb>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/__flowtests__/__generated__/RelayModernFlowtest_users.graphql.js.flow

@@ -1,10 +1,10 @@
 /**
  * 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.
  *
- * @generated SignedSource<<d032f6e721c9c9cdd034cdd93dfcbad6>>
+ * @generated SignedSource<<8a94ce305c03af7280da9b95a665349f>>
  * @flow
  * @lightSyntaxTransform
  * @nogrep

package/buildReactRelayContainer.js.flow

@@ -11,7 +11,7 @@
 // flowlint ambiguous-object-type:error
 
 'use strict';
-
+import type {ReaderFragment} from '../relay-runtime/util/ReaderNode';
 import type {GeneratedNodeMap} from './ReactRelayTypes';
 import type {FragmentMap} from 'relay-runtime';
 
@@ -45,7 +45,7 @@ function buildReactRelayContainer<TBase: React$ComponentType<any>>(
   const containerName = getContainerName(ComponentClass);
   assertFragmentMap(getComponentName(ComponentClass), fragmentSpec);
 
-  const fragments = {};
+  const fragments: {[string]: ReaderFragment} = {};
   for (const key in fragmentSpec) {
     fragments[key] = getFragment(fragmentSpec[key]);
   }

package/hooks.js

@@ -1,5 +1,5 @@
 /**
- * Relay v13.1.0
+ * Relay v14.0.0
  *
  * Copyright (c) Meta Platforms, Inc. and affiliates.
  *

package/index.js

@@ -1,5 +1,5 @@
 /**
- * Relay v13.1.0
+ * Relay v14.0.0
  *
  * Copyright (c) Meta Platforms, Inc. and affiliates.
  *

package/jest-react/internalAct.js.flow

@@ -36,7 +36,7 @@ interface Thenable<+R> {
 
 let actingUpdatesScopeDepth = 0;
 
-function act(scope: () => Thenable<mixed> | void) {
+function act<T>(scope: () => Thenable<T> | T): Thenable<T> {
   if (Scheduler.unstable_flushAllWithoutAsserting === undefined) {
     throw Error(
       'This version of `act` requires a special mock build of Scheduler.',
@@ -49,10 +49,19 @@ function act(scope: () => Thenable<mixed> | void) {
     );
   }
 
+  const previousIsActEnvironment = global.IS_REACT_ACT_ENVIRONMENT;
   const previousActingUpdatesScopeDepth = actingUpdatesScopeDepth;
   actingUpdatesScopeDepth++;
+  if (__DEV__ && actingUpdatesScopeDepth === 1) {
+    // Because this is not the "real" `act`, we set this to `false` so React
+    // knows not to fire `act` warnings.
+    global.IS_REACT_ACT_ENVIRONMENT = false;
+  }
 
   const unwind = () => {
+    if (__DEV__ && actingUpdatesScopeDepth === 1) {
+      global.IS_REACT_ACT_ENVIRONMENT = previousIsActEnvironment;
+    }
     actingUpdatesScopeDepth--;
 
     if (__DEV__) {
@@ -71,20 +80,21 @@ function act(scope: () => Thenable<mixed> | void) {
   // returned and 2) we could use async/await. Since it's only our used in
   // our test suite, we should be able to.
   try {
-    const thenable = scope();
+    const result = scope();
     if (
-      typeof thenable === 'object' &&
-      thenable !== null &&
-      typeof thenable.then === 'function'
+      typeof result === 'object' &&
+      result !== null &&
+      typeof result.then === 'function'
     ) {
+      const thenableResult: Thenable<T> = (result: any);
       return {
-        then(resolve: () => void, reject: (error: mixed) => void) {
-          thenable.then(
-            () => {
+        then(resolve, reject) {
+          thenableResult.then(
+            returnValue => {
               flushActWork(
                 () => {
                   unwind();
-                  resolve();
+                  resolve(returnValue);
                 },
                 error => {
                   unwind();
@@ -100,6 +110,7 @@ function act(scope: () => Thenable<mixed> | void) {
         },
       };
     } else {
+      const returnValue: T = (result: any);
       try {
         // TODO: Let's not support non-async scopes at all in our tests. Need to
         // migrate existing tests.
@@ -107,6 +118,11 @@ function act(scope: () => Thenable<mixed> | void) {
         do {
           didFlushWork = Scheduler.unstable_flushAllWithoutAsserting();
         } while (didFlushWork);
+        return {
+          then(resolve, reject) {
+            resolve(returnValue);
+          },
+        };
       } finally {
         unwind();
       }

package/legacy.js

@@ -1,5 +1,5 @@
 /**
- * Relay v13.1.0
+ * Relay v14.0.0
  *
  * Copyright (c) Meta Platforms, Inc. and affiliates.
  *

package/lib/ReactRelayQueryFetcher.js

@@ -153,6 +153,7 @@ var ReactRelayQueryFetcher = /*#__PURE__*/function () {
     };
 
     if (onDataChange && this._fetchOptions.onDataChangeCallbacks.indexOf(onDataChange) === -1) {
+      // $FlowFixMe[incompatible-use]
       this._fetchOptions.onDataChangeCallbacks.push(onDataChange);
     }
 

package/lib/ReactRelayQueryRenderer.js

@@ -31,7 +31,6 @@ var areEqual = require("fbjs/lib/areEqual");
 var React = require('react');
 
 var _require = require('relay-runtime'),
-    RelayFeatureFlags = _require.RelayFeatureFlags,
     createOperationDescriptor = _require.createOperationDescriptor,
     deepFreeze = _require.deepFreeze,
     getRequest = _require.getRequest;
@@ -132,7 +131,7 @@ var ReactRelayQueryRenderer = /*#__PURE__*/function (_React$Component) {
   _proto.componentDidMount = function componentDidMount() {
     var _this2 = this;
 
-    if (RelayFeatureFlags.ENABLE_QUERY_RENDERER_OFFSCREEN_SUPPORT === true && this._maybeHiddenOrFastRefresh === true) {
+    if (this._maybeHiddenOrFastRefresh === 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

package/lib/jest-react/internalAct.js

@@ -36,10 +36,21 @@ function act(scope) {
     throw Error("This version of `act` requires Jest's timer mocks " + '(i.e. jest.useFakeTimers).');
   }
 
+  var previousIsActEnvironment = global.IS_REACT_ACT_ENVIRONMENT;
   var previousActingUpdatesScopeDepth = actingUpdatesScopeDepth;
   actingUpdatesScopeDepth++;
 
+  if (process.env.NODE_ENV !== "production" && actingUpdatesScopeDepth === 1) {
+    // Because this is not the "real" `act`, we set this to `false` so React
+    // knows not to fire `act` warnings.
+    global.IS_REACT_ACT_ENVIRONMENT = false;
+  }
+
   var unwind = function unwind() {
+    if (process.env.NODE_ENV !== "production" && actingUpdatesScopeDepth === 1) {
+      global.IS_REACT_ACT_ENVIRONMENT = previousIsActEnvironment;
+    }
+
     actingUpdatesScopeDepth--;
 
     if (process.env.NODE_ENV !== "production") {
@@ -55,15 +66,16 @@ function act(scope) {
 
 
   try {
-    var thenable = scope();
+    var result = scope();
 
-    if (typeof thenable === 'object' && thenable !== null && typeof thenable.then === 'function') {
+    if (typeof result === 'object' && result !== null && typeof result.then === 'function') {
+      var thenableResult = result;
       return {
         then: function then(resolve, reject) {
-          thenable.then(function () {
+          thenableResult.then(function (returnValue) {
             flushActWork(function () {
               unwind();
-              resolve();
+              resolve(returnValue);
             }, function (error) {
               unwind();
               reject(error);
@@ -75,6 +87,8 @@ function act(scope) {
         }
       };
     } else {
+      var returnValue = result;
+
       try {
         // TODO: Let's not support non-async scopes at all in our tests. Need to
         // migrate existing tests.
@@ -83,6 +97,12 @@ function act(scope) {
         do {
           didFlushWork = Scheduler.unstable_flushAllWithoutAsserting();
         } while (didFlushWork);
+
+        return {
+          then: function then(resolve, reject) {
+            resolve(returnValue);
+          }
+        };
       } finally {
         unwind();
       }

package/lib/readContext.js

@@ -10,7 +10,8 @@
 // flowlint ambiguous-object-type:error
 'use strict';
 
-var React = require('react');
+var React = require('react'); // $FlowFixMe[incompatible-use]
+
 
 var _React$__SECRET_INTER =
 /* $FlowFixMe[prop-missing] Flow doesn't know about React's internals for