relay-runtime 20.1.0 → 21.0.0

package/util/RelayReplaySubject.js.flow

@@ -64,7 +64,7 @@ class RelayReplaySubject<T> {
             sink.next(event.data);
             break;
           default:
-            (event.kind: empty);
+            event.kind as empty;
             invariant(
               false,
               'RelayReplaySubject: Unknown event kind `%s`.',
@@ -96,8 +96,8 @@ class RelayReplaySubject<T> {
     }
     this._complete = true;
     this._events.push({
-      kind: 'error',
       error,
+      kind: 'error',
     });
     this._sinks.forEach(sink => sink.error(error));
   }
@@ -107,8 +107,8 @@ class RelayReplaySubject<T> {
       return;
     }
     this._events.push({
-      kind: 'next',
       data,
+      kind: 'next',
     });
     this._sinks.forEach(sink => sink.next(data));
   }

package/util/RelayRuntimeTypes.js.flow

@@ -40,11 +40,11 @@ export type Variables = {+[string]: $FlowFixMe};
 export type OperationType = {
   // TODO(T33395812) Make this an open object type
   +variables: Variables,
-  +response: mixed,
+  +response: unknown,
   +rawResponse?: {...},
 };
 
-export type VariablesOf<T: OperationType> = T['variables'];
+export type VariablesOf<T extends OperationType> = T['variables'];
 
 /**
  * Settings for how a query response may be cached.
@@ -68,7 +68,7 @@ export type CacheConfig = {
   +onSubscribe?: () => void,
   +onResume?: (pauseTimeMs: number) => void,
   +onPause?: (mqttConnectionIsOk: boolean, internetIsOk: boolean) => void,
-  +metadata?: {+[key: string]: mixed, ...},
+  +metadata?: {+[key: string]: unknown, ...},
   +transactionId?: ?string,
 };
 
@@ -86,7 +86,7 @@ export type RenderPolicy = 'full' | 'partial';
  * Return type of graphql tag literals for all operations.
  */
 declare export opaque type Operation<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
   TRawResponse,
 >: ConcreteRequest;
@@ -95,7 +95,7 @@ declare export opaque type Operation<
  * Return type of graphql tag literals for updatable queries.
  */
 declare export opaque type UpdatableQuery<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
 >: ConcreteUpdatableQuery;
 
@@ -111,7 +111,7 @@ declare export opaque type UpdatableFragment<
  * Return type of graphql tag literals for queries.
  */
 declare export opaque type Query<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
   TRawResponse = void,
 >: Operation<TVariables, TData, TRawResponse>;
@@ -120,7 +120,7 @@ declare export opaque type Query<
  * Return type of graphql tag literals for client-only queries.
  */
 declare export opaque type ClientQuery<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
   TRawResponse = void,
 >: ClientRequest;
@@ -129,7 +129,7 @@ declare export opaque type ClientQuery<
  * Return type of graphql tag literals for mutations.
  */
 declare export opaque type Mutation<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
   TRawResponse = {...},
 >: Operation<TVariables, TData, TRawResponse>;
@@ -141,7 +141,7 @@ declare export opaque type Mutation<
  *       `RelayObservable`'s `Subscription` type.
  */
 declare export opaque type GraphQLSubscription<
-  -TVariables: Variables,
+  -TVariables extends Variables,
   +TData,
   TRawResponse = void,
 >: Operation<TVariables, TData, TRawResponse>;
@@ -166,7 +166,7 @@ declare export opaque type Fragment<TFragmentType, +TData>: ReaderFragment;
 declare export opaque type RefetchableFragment<
   TFragmentType,
   +TData,
-  TVariables: Variables,
+  TVariables extends Variables,
 >: Fragment<TFragmentType, TData>;
 
 /**
@@ -177,5 +177,5 @@ declare export opaque type PrefetchableRefetchableFragment<
   TFragmentType,
   +TData,
   +TEdgeData,
-  TVariables: Variables,
+  TVariables extends Variables,
 >: Fragment<TFragmentType, TData>;

package/util/createPayloadFor3DField.js.flow

@@ -21,15 +21,19 @@ const {
 
 export opaque type Local3DPayload<
   // eslint-disable-next-line no-unused-vars
-  +DocumentName: string,
-  +Response: {...},
+  +DocumentName extends string,
+  +Response extends {...},
 > = Response;
 
-// $FlowFixMe[unsupported-variance-annotation]
-function createPayloadFor3DField<+DocumentName: string, +Response: {...}>(
+function createPayloadFor3DField<
+  // $FlowFixMe[unsupported-variance-annotation]
+  +DocumentName extends string,
+  // $FlowFixMe[unsupported-variance-annotation]
+  +Response extends {...},
+>(
   name: DocumentName,
   operation: JSResourceReference<NormalizationSplitOperation>,
-  component: JSResourceReference<mixed>,
+  component: JSResourceReference<unknown>,
   response: Response,
 ): Local3DPayload<DocumentName, Response> {
   const data = {

package/util/deepFreeze.js.flow

@@ -17,7 +17,7 @@
  * For convenience, and for consistency with the behavior of `Object.freeze`,
  * returns the now-frozen original object.
  */
-function deepFreeze<T: {...}>(object: T): T {
+function deepFreeze<T extends {...}>(object: T): T {
   if (!shouldBeFrozen(object)) {
     return object;
   }
@@ -35,7 +35,7 @@ function deepFreeze<T: {...}>(object: T): T {
   return object;
 }
 
-function shouldBeFrozen(value: mixed): boolean {
+function shouldBeFrozen(value: unknown): boolean {
   // Only freeze plain JS arrays and objects
   return (
     value != null &&

package/util/getFragmentIdentifier.js.flow

@@ -25,7 +25,7 @@ const {intern} = require('./StringInterner');
 
 function getFragmentIdentifier(
   fragmentNode: ReaderFragment,
-  fragmentRef: mixed,
+  fragmentRef: unknown,
 ): string {
   const selector = getSelector(fragmentNode, fragmentRef);
   const fragmentOwnerIdentifier =

package/util/getPaginationMetadata.js.flow

@@ -21,7 +21,7 @@ function getPaginationMetadata(
   fragmentNode: ReaderFragment,
   componentDisplayName: string,
 ): {
-  connectionPathInFragmentData: $ReadOnlyArray<string | number>,
+  connectionPathInFragmentData: ReadonlyArray<string | number>,
   identifierField: ?string,
   paginationRequest: ConcreteRequest,
   paginationMetadata: ReaderPaginationMetadata,

package/util/getPaginationVariables.js.flow

@@ -26,7 +26,7 @@ function getPaginationVariables(
   baseVariables: Variables,
   extraVariables: Variables,
   paginationMetadata: ReaderPaginationMetadata,
-): {[string]: mixed, ...} {
+): {[string]: unknown, ...} {
   const {backward: backwardMetadata, forward: forwardMetadata} =
     paginationMetadata;
 

package/util/getPendingOperationsForFragment.js.flow

@@ -22,9 +22,9 @@ function getPendingOperationsForFragment(
   fragmentOwner: RequestDescriptor,
 ): {
   promise: Promise<void>,
-  pendingOperations: $ReadOnlyArray<RequestDescriptor>,
+  pendingOperations: ReadonlyArray<RequestDescriptor>,
 } | null {
-  let pendingOperations: $ReadOnlyArray<RequestDescriptor> = [];
+  let pendingOperations: ReadonlyArray<RequestDescriptor> = [];
   let promise = getPromiseForActiveRequest(environment, fragmentOwner);
 
   if (promise != null) {

package/util/getRefetchMetadata.js.flow

@@ -24,7 +24,7 @@ function getRefetchMetadata(
   fragmentNode: ReaderFragment,
   componentDisplayName: string,
 ): {
-  fragmentRefPathInResponse: $ReadOnlyArray<string | number>,
+  fragmentRefPathInResponse: ReadonlyArray<string | number>,
   identifierInfo: ?RefetchableIdentifierInfo,
   refetchableRequest: ConcreteRequest,
   refetchMetadata: ReaderRefetchMetadata,
@@ -50,10 +50,11 @@ function getRefetchMetadata(
   );
 
   // handle both commonjs and es modules
-  const refetchableRequest: ConcreteRequest | string =
-    (refetchMetadata: $FlowFixMe).operation.default
-      ? (refetchMetadata: $FlowFixMe).operation.default
-      : refetchMetadata.operation;
+  const refetchableRequest: ConcreteRequest | string = (
+    refetchMetadata as $FlowFixMe
+  ).operation.default
+    ? (refetchMetadata as $FlowFixMe).operation.default
+    : refetchMetadata.operation;
   const fragmentRefPathInResponse = refetchMetadata.fragmentPathInResult;
   invariant(
     typeof refetchableRequest !== 'string',

package/util/getValueAtPath.js.flow

@@ -14,9 +14,9 @@
 const invariant = require('invariant');
 
 function getValueAtPath(
-  data: mixed,
-  path: $ReadOnlyArray<string | number>,
-): mixed {
+  data: unknown,
+  path: ReadonlyArray<string | number>,
+): unknown {
   let result = data;
   for (const key of path) {
     if (result == null) {

package/util/handlePotentialSnapshotErrors.js.flow

@@ -22,7 +22,7 @@ const invariant = require('invariant');
 function handleFieldErrors(
   environment: IEnvironment,
   fieldErrors: FieldErrors,
-  loggingContext: mixed | void,
+  loggingContext: unknown | void,
 ) {
   for (const fieldError of fieldErrors) {
     // First we log all events. Note that the logger may opt to throw its own
@@ -67,7 +67,7 @@ function handleFieldErrors(
           // can't assert this via types.
           break;
         default:
-          (fieldError.kind: empty);
+          fieldError.kind as empty;
           invariant(false, 'Relay: Unexpected event kind: %s', fieldError.kind);
       }
     }
@@ -86,7 +86,7 @@ function eventShouldThrow(event: FieldError): boolean {
     case 'missing_expected_data.log':
       return false;
     default:
-      (event.kind: empty);
+      event.kind as empty;
       throw new Error('Relay: Unexpected event kind');
   }
 }
@@ -94,7 +94,7 @@ function eventShouldThrow(event: FieldError): boolean {
 function handlePotentialSnapshotErrors(
   environment: IEnvironment,
   fieldErrors: ?FieldErrors,
-  loggingContext: mixed | void,
+  loggingContext: unknown | void,
 ) {
   /**
    * Inside handleFieldErrors, we check for throwOnFieldError - but this fn logs the error anyway by default
@@ -106,6 +106,6 @@ function handlePotentialSnapshotErrors(
 }
 
 module.exports = {
-  handlePotentialSnapshotErrors,
   eventShouldThrow,
+  handlePotentialSnapshotErrors,
 };

package/util/isEmptyObject.js.flow

@@ -14,7 +14,7 @@
 // $FlowFixMe[method-unbinding] added when improving typing for this parameters
 const hasOwnProperty = Object.prototype.hasOwnProperty;
 
-function isEmptyObject(obj: {+[key: string]: mixed}): boolean {
+function isEmptyObject(obj: {+[key: string]: unknown}): boolean {
   for (const key in obj) {
     if (hasOwnProperty.call(obj, key)) {
       return false;

package/util/isPromise.js.flow

@@ -11,9 +11,9 @@
 
 'use strict';
 
-declare function isPromise<T>(p: mixed): p is Promise<T>;
+declare function isPromise<T>(p: unknown): p is Promise<T>;
 
-function isPromise(p: mixed) {
+function isPromise(p: unknown) {
   return p != null && typeof p === 'object' && typeof p.then === 'function';
 }
 

package/util/isScalarAndEqual.js.flow

@@ -18,7 +18,7 @@
  * - returns false for complex values, since these cannot be cheaply tested for
  *   equality (use `areEquals` instead)
  */
-function isScalarAndEqual(valueA: mixed, valueB: mixed): boolean {
+function isScalarAndEqual(valueA: unknown, valueB: unknown): boolean {
   return valueA === valueB && (valueA === null || typeof valueA !== 'object');
 }
 

package/util/recycleNodesInto.js.flow

@@ -38,8 +38,8 @@ function recycleNodesIntoImpl<T>(
   let canRecycle = false;
 
   // Assign local variables to preserve Flow type refinement.
-  const prevArray: ?Array<mixed> = Array.isArray(prevData) ? prevData : null;
-  const nextArray: ?Array<mixed> = Array.isArray(nextData) ? nextData : null;
+  const prevArray: ?Array<unknown> = Array.isArray(prevData) ? prevData : null;
+  const nextArray: ?Array<unknown> = Array.isArray(nextData) ? nextData : null;
   if (prevArray && nextArray) {
     const canMutateNext = canMutate && !Object.isFrozen(nextArray);
     canRecycle =

package/util/registerEnvironmentWithDevTools.js.flow

@@ -28,7 +28,7 @@ function registerEnvironmentWithDevTools(environment: IEnvironment): void {
   // $FlowFixMe[incompatible-use] D61394600
   const devToolsHook = _global && _global.__RELAY_DEVTOOLS_HOOK__;
   if (devToolsHook) {
-    // $FlowFixMe[incompatible-call] D61394600
+    // $FlowFixMe[incompatible-type] D61394600
     devToolsHook.registerEnvironment(environment);
   }
 }

package/util/shallowFreeze.js.flow

@@ -12,7 +12,7 @@
 'use strict';
 
 // Shallow freeze to prevent Relay from mutating the value in recycleNodesInto or deepFreezing the value
-module.exports = function shallowFreeze(value: mixed) {
+module.exports = function shallowFreeze(value: unknown) {
   if (
     typeof value === 'object' &&
     value != null &&

package/util/stableCopy.js.flow

@@ -16,7 +16,7 @@
  * keys sorted such that equivalent values would have identical JSON.stringify
  * results.
  */
-function stableCopy<T: mixed>(value: T): T {
+function stableCopy<T extends unknown>(value: T): T {
   if (!value || typeof value !== 'object') {
     return value;
   }
@@ -24,17 +24,17 @@ function stableCopy<T: mixed>(value: T): T {
     return value.map(stableCopy);
   }
   const keys = Object.keys(value).sort();
-  const stable: {[string]: mixed} = {};
+  const stable: {[string]: unknown} = {};
   for (let i = 0; i < keys.length; i++) {
     stable[keys[i]] = stableCopy(value[keys[i]]);
   }
-  return (stable: any);
+  return stable as any;
 }
 
 // Detect if a data structure contains cycles. The logic here mirrors
 // `stableCopy` above and is intended to detect cycles early before they get
 // passed to `stableCopy` which would result in a stack overflow.
-function hasCycle(value: mixed, parents: Set<mixed> = new Set()): boolean {
+function hasCycle(value: unknown, parents: Set<unknown> = new Set()): boolean {
   // $FlowFixMe[sketchy-null-mixed]
   if (!value || typeof value !== 'object') {
     return false;
@@ -52,6 +52,6 @@ function hasCycle(value: mixed, parents: Set<mixed> = new Set()): boolean {
 }
 
 module.exports = {
-  stableCopy,
   hasCycle,
+  stableCopy,
 };

package/util/withProvidedVariables.js.flow

@@ -19,17 +19,19 @@ const warning = require('warning');
 
 const WEAKMAP_SUPPORTED = typeof WeakMap === 'function';
 let debugCache:
-  | Map<mixed, mixed>
-  | Map<() => mixed, mixed>
-  | WeakMap<interface {} | $ReadOnlyArray<mixed>, mixed>
-  | WeakMap<() => mixed, mixed> = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
+  | Map<unknown, unknown>
+  | Map<() => unknown, unknown>
+  | WeakMap<interface {} | ReadonlyArray<unknown>, unknown>
+  | WeakMap<() => unknown, unknown> = WEAKMAP_SUPPORTED
+  ? new WeakMap()
+  : new Map();
 
 function withProvidedVariables(
   userSuppliedVariables: Variables,
   providedVariables: ?ProvidedVariablesType,
 ): Variables {
   if (providedVariables != null) {
-    const operationVariables: {[string]: mixed} = {};
+    const operationVariables: {[string]: unknown} = {};
     // $FlowFixMe[unsafe-object-assign]
     Object.assign(operationVariables, userSuppliedVariables);
     Object.keys(providedVariables).forEach((varName: string) => {
@@ -64,10 +66,12 @@ function withProvidedVariables(
   }
 }
 
-withProvidedVariables.tests_only_resetDebugCache = ((__DEV__
-  ? () => {
-      debugCache = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
-    }
-  : undefined): void | (() => void));
+withProvidedVariables.tests_only_resetDebugCache = (
+  __DEV__
+    ? () => {
+        debugCache = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
+      }
+    : undefined
+) as void | (() => void);
 
 module.exports = withProvidedVariables;