react-relay 14.0.0 → 14.1.0

package/relay-hooks/HooksImplementation.js.flow

@@ -11,19 +11,17 @@
 
 'use strict';
 
-// flowlint ambiguous-object-type:error
-
 import typeof useFragment from './useFragment';
-import type {UseLazyLoadQueryHookType} from './useLazyLoadQuery';
-import type {UsePreloadedQueryHookType} from './usePreloadedQuery';
+import type {UsePaginationFragmentType} from './usePaginationFragment';
+import type {UseRefetchableFragmentType} from './useRefetchableFragment';
 
 const warning = require('warning');
 
-type HooksImplementation = {|
+type HooksImplementation = {
   useFragment: useFragment,
-  useLazyLoadQuery: UseLazyLoadQueryHookType,
-  usePreloadedQuery: UsePreloadedQueryHookType,
-|};
+  usePaginationFragment: UsePaginationFragmentType,
+  useRefetchableFragment: UseRefetchableFragmentType,
+};
 
 let implementation: HooksImplementation | null = null;
 

package/relay-hooks/InternalLogger.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 type LogEvent = (eventData: string) => void;

package/relay-hooks/LRUCache.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 const invariant = require('invariant');

package/relay-hooks/LazyLoadEntryPointContainer_DEPRECATED.react.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {
@@ -27,13 +25,13 @@ const React = require('react');
 const {useContext, useEffect, useMemo} = require('react');
 const {stableCopy} = require('relay-runtime');
 
-type PreloadedEntryPoint<TEntryPointComponent> = $ReadOnly<{|
+type PreloadedEntryPoint<TEntryPointComponent> = $ReadOnly<{
   entryPoints: React.ElementConfig<TEntryPointComponent>['entryPoints'],
   extraProps: React.ElementConfig<TEntryPointComponent>['extraProps'],
   getComponent: () => TEntryPointComponent,
   queries: React.ElementConfig<TEntryPointComponent>['queries'],
   rootModuleID: string,
-|}>;
+}>;
 
 type EntryPointContainerProps<
   TEntryPointParams,
@@ -42,7 +40,7 @@ type EntryPointContainerProps<
   TRuntimeProps,
   TExtraProps,
 > = $ReadOnly<
-  $ReadOnly<{|
+  $ReadOnly<{
     entryPoint: EntryPoint<
       TEntryPointParams,
       EntryPointComponent<
@@ -55,7 +53,7 @@ type EntryPointContainerProps<
     entryPointParams: TEntryPointParams,
     environmentProvider?: IEnvironmentProvider<EnvironmentProviderOptions>,
     props: TRuntimeProps,
-  |}>,
+  }>,
 >;
 
 function stableStringify(value: mixed): string {

package/relay-hooks/MatchContainer.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 const React = require('react');
@@ -87,7 +85,7 @@ const {useMemo} = React;
 
 // Note: this type is intentionally non-exact, it is expected that the
 // object may contain sibling fields.
-type TypenameOnlyPointer = {|+__typename: string|};
+type TypenameOnlyPointer = {+__typename: string};
 export type MatchPointer = {
   +__fragmentPropName?: ?string,
   +__module_component?: mixed,
@@ -95,12 +93,12 @@ export type MatchPointer = {
   ...
 };
 
-export type MatchContainerProps<TProps: {...}, TFallback: React.Node> = {|
+export type MatchContainerProps<TProps: {...}, TFallback: React.Node> = {
   +fallback?: ?TFallback,
   +loader: (module: mixed) => React.AbstractComponent<TProps>,
   +match: ?MatchPointer | ?TypenameOnlyPointer,
   +props?: TProps,
-|};
+};
 
 function MatchContainer<TProps: {...}, TFallback: React.Node | null>({
   fallback,
@@ -150,11 +148,11 @@ function MatchContainer<TProps: {...}, TFallback: React.Node | null>({
     // output of subscriptions already has a stable identity.
     if (__fragmentPropName != null && __id != null && __fragments != null) {
       const fragProps: {
-        [string]: {|
+        [string]: {
           __fragmentOwner: $FlowFixMe,
           __fragments: $FlowFixMe,
           __id: string,
-        |},
+        },
       } = {};
       fragProps[__fragmentPropName] = {__id, __fragments, __fragmentOwner};
       return fragProps;

package/relay-hooks/ProfilerContext.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 // This contextual profiler can be used to wrap a react sub-tree. It will bind
 // the RelayProfiler during the render phase of these components. Allows
 // collecting metrics for a specific part of your application.

package/relay-hooks/QueryResource.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {Cache} from './LRUCache';
@@ -41,7 +39,7 @@ const DEFAULT_FETCH_POLICY = 'store-or-network';
 export type QueryResource = QueryResourceImpl;
 
 type QueryResourceCache = Cache<QueryResourceCacheEntry>;
-type QueryResourceCacheEntry = {|
+type QueryResourceCacheEntry = {
   +id: number,
   +cacheIdentifier: string,
   +operationAvailability: ?OperationAvailability,
@@ -56,17 +54,17 @@ type QueryResourceCacheEntry = {|
   temporaryRetain(environment: IEnvironment): Disposable,
   permanentRetain(environment: IEnvironment): Disposable,
   releaseTemporaryRetain(): void,
-|};
+};
 export opaque type QueryResult: {
   fragmentNode: ReaderFragment,
   fragmentRef: mixed,
   ...
-} = {|
+} = {
   cacheIdentifier: string,
   fragmentNode: ReaderFragment,
   fragmentRef: mixed,
   operation: OperationDescriptor,
-|};
+};
 
 const WEAKMAP_SUPPORTED = typeof WeakMap === 'function';
 interface IMap<K, V> {

package/relay-hooks/RelayEnvironmentProvider.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {IEnvironment} from 'relay-runtime';
@@ -24,13 +22,13 @@ const ReactRelayContext = require('react-relay/ReactRelayContext');
 
 const {useMemo} = React;
 
-type Props = $ReadOnly<{|
+type Props = $ReadOnly<{
   children: React.Node,
   environment: IEnvironment,
   getEnvironmentForActor?: ?(
     actorIdentifier: ActorIdentifier,
   ) => IActorEnvironment,
-|}>;
+}>;
 
 function RelayEnvironmentProvider(props: Props): React.Node {
   const {children, environment, getEnvironmentForActor} = props;

package/relay-hooks/SuspenseResource.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {Disposable, IEnvironment} from 'relay-runtime';

package/relay-hooks/__flowtests__/EntryPointTypes/EntryPointElementConfig-flowtest.js.flow

@@ -17,15 +17,15 @@ import type {
   EntryPointProps,
 } from '../../EntryPointTypes.flow';
 
-type MyComponentOtherProps = $ReadOnly<{|
+type MyComponentOtherProps = $ReadOnly<{
   foo: string,
-|}>;
+}>;
 
 type MyComponentProps = EntryPointProps<{}, {}, MyComponentOtherProps, {}>;
 
 const MyComponent = (_props: MyComponentProps) => null;
 
-type PreloadParams = $ReadOnly<{||}>;
+type PreloadParams = $ReadOnly<{}>;
 
 type MyComponentEntryPointType = EntryPoint<PreloadParams, typeof MyComponent>;
 

package/relay-hooks/__flowtests__/EntryPointTypes/NestedEntrypoints-flowtest.js.flow

@@ -27,9 +27,9 @@ type NestedEntrypointComponentProps = EntryPointProps<{}>;
 const NestedEntrypointComponent = (_props: NestedEntrypointComponentProps) =>
   null;
 
-type NestedEntrypointPreloadParams = $ReadOnly<{|
+type NestedEntrypointPreloadParams = $ReadOnly<{
   subEntrypointPreloadParam: string,
-|}>;
+}>;
 
 type NestedEntryPointType = EntryPoint<
   NestedEntrypointPreloadParams,

package/relay-hooks/__flowtests__/useBlockingPaginationFragment-flowtest.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 import type {LoadMoreFn} from '../useLoadMoreFunction';
 import type {
   FetchFn,
@@ -36,14 +34,14 @@ type ExpectedReturnType<
   TQuery: OperationType,
   TQueryVariables,
   TFragmentData,
-> = {|
+> = {
   data: TFragmentData,
   loadNext: LoadMoreFn<TQuery>,
   loadPrevious: LoadMoreFn<TQuery>,
   hasNext: boolean,
   hasPrevious: boolean,
   refetch: FetchFn<TQueryVariables>,
-|};
+};
 
 /* eslint-disable react-hooks/rules-of-hooks */
 
@@ -103,8 +101,8 @@ refetch(variables, {
 });
 
 // LoadMore options
-declare var extraVariables: {|nickname: string|};
-declare var invalidVariables: {|foo: string|};
+declare var extraVariables: {nickname: string};
+declare var invalidVariables: {foo: string};
 
 const {loadNext} = useBlockingPaginationFragment<QueryOperation, _>(
   fragmentInput,

package/relay-hooks/__flowtests__/useFragment-flowtest.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 import type {
   useFragmentFlowtest_user$data,
   useFragmentFlowtest_user$key,

package/relay-hooks/__flowtests__/usePaginationFragment-flowtest.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 import type {LoadMoreFn} from '../useLoadMoreFunction';
 import type {
   FetchFn,
@@ -36,7 +34,7 @@ type ExpectedReturnType<
   TQuery: OperationType,
   TQueryVariables,
   TFragmentData,
-> = {|
+> = {
   data: TFragmentData,
   loadNext: LoadMoreFn<TQuery>,
   loadPrevious: LoadMoreFn<TQuery>,
@@ -45,7 +43,7 @@ type ExpectedReturnType<
   isLoadingNext: boolean,
   isLoadingPrevious: boolean,
   refetch: FetchFn<TQueryVariables>,
-|};
+};
 
 /* eslint-disable react-hooks/rules-of-hooks */
 
@@ -105,8 +103,8 @@ refetch(variables, {
 });
 
 // LoadMore options
-declare var extraVariables: {|nickname: string|};
-declare var invalidVariables: {|foo: string|};
+declare var extraVariables: {nickname: string};
+declare var invalidVariables: {foo: string};
 
 const {loadNext} = usePaginationFragment<QueryOperation, _>(
   fragmentInput,

package/relay-hooks/__flowtests__/useRefetchableFragment-flowtest.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 import type {
   FetchFn,
   NonNullableData,

package/relay-hooks/__flowtests__/utils.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {Disposable, Fragment, FragmentType} from 'relay-runtime';
@@ -28,18 +26,18 @@ export type Example_user$key = {
   ...
 };
 
-export type NonNullableData = {|
+export type NonNullableData = {
   +id: string,
   +count: number,
-|};
+};
 export type NullableData = ?NonNullableData;
 export type NonNullablePluralData = $ReadOnlyArray<NonNullableData>;
 export type NullablePluralData = ?$ReadOnlyArray<NonNullableData>;
 
-export type AnotherNonNullableData = {|
+export type AnotherNonNullableData = {
   +name: ?string,
   +friends: ?number,
-|};
+};
 
 declare export var keyNonNullable: Example_user$key;
 
@@ -66,16 +64,16 @@ declare export var fragmentData: {
   ...
 };
 
-export type QueryOperation = {|
+export type QueryOperation = {
   +variables: QueryVariables,
   +response: {...},
-|};
+};
 
-export type QueryVariables = {|
+export type QueryVariables = {
   id: string,
   nickname: ?string,
   name: string,
-|};
+};
 
 export type QueryVariablesSubset = {
   id: string,

package/relay-hooks/preloadQuery_DEPRECATED.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {
@@ -51,7 +49,7 @@ const pendingQueriesByEnvironment = WEAKMAP_SUPPORTED
   : new Map();
 
 type PendingQueryEntry =
-  | $ReadOnly<{|
+  | $ReadOnly<{
       cacheKey: string,
       fetchKey: ?string | ?number,
       fetchPolicy: PreloadFetchPolicy,
@@ -61,8 +59,8 @@ type PendingQueryEntry =
       status: PreloadQueryStatus,
       subject: ReplaySubject<GraphQLResponse>,
       subscription: Subscription,
-    |}>
-  | $ReadOnly<{|
+    }>
+  | $ReadOnly<{
       cacheKey: string,
       fetchKey: ?string | ?number,
       fetchPolicy: PreloadFetchPolicy,
@@ -70,7 +68,7 @@ type PendingQueryEntry =
       id: ?string,
       name: string,
       status: PreloadQueryStatus,
-    |}>;
+    }>;
 
 function preloadQuery<TQuery: OperationType, TEnvironmentProviderOptions>(
   environment: IEnvironment,

package/relay-hooks/prepareEntryPoint_DEPRECATED.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {

package/relay-hooks/react-cache/RelayReactCache.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 const invariant = require('invariant');

package/relay-hooks/react-cache/getQueryResultOrFetchQuery_REACT_CACHE.js.flow

@@ -9,8 +9,6 @@
  * @format
  */
 
-// flowlint ambiguous-object-type:error
-
 'use strict';
 
 import type {
@@ -34,10 +32,10 @@ const warning = require('warning');
 
 type QueryCacheCommitable = () => () => void;
 
-type QueryResult = {|
+type QueryResult = {
   fragmentNode: ReaderFragment,
   fragmentRef: mixed,
-|};
+};
 
 // Note that the status of a cache entry will be 'resolved' when partial
 // rendering is allowed, even if a fetch is ongoing. The pending status
@@ -49,34 +47,42 @@ type QueryResult = {|
 // before any components have mounted. It is unused when Legacy Timeouts
 // mode is off.
 type QueryCacheEntryStatus =
-  | {|
+  | {
       status: 'resolved',
       result: QueryResult,
-    |}
-  | {|
+    }
+  | {
       status: 'pending',
       promise: Promise<void>,
-    |}
-  | {|
+    }
+  | {
       status: 'rejected',
       error: Error,
-    |};
+    };
 
-type QueryCacheEntry = {|
+type QueryCacheEntry = {
   ...QueryCacheEntryStatus,
   onCommit: QueryCacheCommitable,
   suspenseResource: SuspenseResource | null,
-|};
+};
 
 const DEFAULT_FETCH_POLICY = 'store-or-network';
 
+const WEAKMAP_SUPPORTED = typeof WeakMap === 'function';
+
+interface IMap<K, V> {
+  delete(key: K): boolean;
+  get(key: K): V | void;
+  set(key: K, value: V): IMap<K, V>;
+}
+
 type QueryCacheKey = string;
 
 class QueryCache {
-  _map: Map<IEnvironment, Map<QueryCacheKey, QueryCacheEntry>>;
+  _map: IMap<IEnvironment, Map<QueryCacheKey, QueryCacheEntry>>;
 
   constructor() {
-    this._map = new Map();
+    this._map = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
   }
 
   get(environment: IEnvironment, key: QueryCacheKey): QueryCacheEntry | void {
@@ -160,12 +166,12 @@ function makeInitialCacheEntry() {
 function getQueryResultOrFetchQuery_REACT_CACHE(
   environment: IEnvironment,
   queryOperationDescriptor: OperationDescriptor,
-  options?: {|
+  options?: {
     fetchPolicy?: FetchPolicy,
     renderPolicy?: RenderPolicy,
     fetchKey?: ?string | ?number,
     fetchObservable?: Observable<GraphQLResponse>,
-  |},
+  },
 ): [QueryResult, QueryCacheCommitable] {
   const fetchPolicy = options?.fetchPolicy ?? DEFAULT_FETCH_POLICY;
   const renderPolicy =