relay-runtime 20.0.0 → 20.1.1

package/experimental.js

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

package/index.js

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

package/lib/store/RelayResponseNormalizer.js

@@ -139,7 +139,7 @@ var RelayResponseNormalizer = /*#__PURE__*/function () {
                 this._traverseSelections(selection, record, data);
               }
             } else {
-              var implementsInterface = data.hasOwnProperty(abstractKey);
+              var implementsInterface = Object.prototype.hasOwnProperty.call(data, abstractKey);
               var _typeName = RelayModernRecord.getType(record);
               var typeID = generateTypeID(_typeName);
               var typeRecord = this._recordSource.get(typeID);
@@ -157,7 +157,7 @@ var RelayResponseNormalizer = /*#__PURE__*/function () {
         case 'TypeDiscriminator':
           {
             var _abstractKey = selection.abstractKey;
-            var _implementsInterface = data.hasOwnProperty(_abstractKey);
+            var _implementsInterface = Object.prototype.hasOwnProperty.call(data, _abstractKey);
             var _typeName2 = RelayModernRecord.getType(record);
             var _typeID = generateTypeID(_typeName2);
             var _typeRecord = this._recordSource.get(_typeID);

package/mutations/commitMutation.js.flow

@@ -37,7 +37,7 @@ const validateMutation = require('./validateMutation');
 const invariant = require('invariant');
 const warning = require('warning');
 
-export type MutationConfig<TMutation: MutationParameters> = {
+export type MutationConfig<TMutation: MutationParameters> = $ReadOnly<{
   cacheConfig?: CacheConfig,
   configs?: Array<DeclarativeMutationConfig>,
   mutation: GraphQLTaggedNode,
@@ -57,9 +57,9 @@ export type MutationConfig<TMutation: MutationParameters> = {
   updater?: ?SelectorStoreUpdater<TMutation['response']>,
   uploadables?: UploadableMap,
   variables: TMutation['variables'],
-};
+}>;
 
-export type CommitMutationConfig<TVariables, TData, TRawResponse> = {
+export type CommitMutationConfig<TVariables, TData, TRawResponse> = $ReadOnly<{
   cacheConfig?: CacheConfig,
   configs?: Array<DeclarativeMutationConfig>,
   mutation: Mutation<TVariables, TData, TRawResponse>,
@@ -71,8 +71,8 @@ export type CommitMutationConfig<TVariables, TData, TRawResponse> = {
   optimisticUpdater?: ?SelectorStoreUpdater<TData>,
   updater?: ?SelectorStoreUpdater<TData>,
   uploadables?: UploadableMap,
-  variables: TVariables,
-};
+  variables: NoInfer<TVariables>,
+}>;
 
 /**
  * Higher-level helper function to execute a mutation against a specific

package/network/RelayNetworkTypes.js.flow

@@ -26,7 +26,7 @@ export interface INetwork {
 
 export type LogRequestInfoFunction = mixed => void;
 
-export type PayloadData = {[key: string]: mixed};
+export type PayloadData = {+[key: string]: mixed};
 
 export type PayloadError = interface {
   message: string,
@@ -134,4 +134,4 @@ export type SubscribeFunction = (
 ) => RelayObservable<GraphQLResponse>;
 
 export type Uploadable = File | Blob;
-export type UploadableMap = {[key: string]: Uploadable};
+export type UploadableMap = {+[key: string]: Uploadable};

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "relay-runtime",
   "description": "A core runtime for building GraphQL-driven applications.",
-  "version": "20.0.0",
+  "version": "20.1.1",
   "keywords": [
     "graphql",
     "relay"

package/query/fetchQuery.js.flow

@@ -115,7 +115,7 @@ const invariant = require('invariant');
 function fetchQuery<TVariables: Variables, TData, TRawResponse>(
   environment: IEnvironment,
   query: Query<TVariables, TData, TRawResponse>,
-  variables: TVariables,
+  variables: NoInfer<TVariables>,
   options?: $ReadOnly<{
     fetchPolicy?: FetchQueryFetchPolicy,
     networkCacheConfig?: CacheConfig,

package/store/RelayModernRecord.js.flow

@@ -443,7 +443,7 @@ function merge(record1: Record, record2: Record): Record {
   if (ERRORS_KEY in record1 || ERRORS_KEY in record2) {
     const {[ERRORS_KEY]: errors1, ...fields1} = record1;
     const {[ERRORS_KEY]: errors2, ...fields2} = record2;
-    // $FlowIssue[cannot-spread-indexer]
+    // $FlowFixMe[cannot-spread-indexer]
     const updated: Record = {...fields1, ...fields2};
     if (errors1 == null && errors2 == null) {
       return updated;
@@ -465,7 +465,7 @@ function merge(record1: Record, record2: Record): Record {
     }
     return updated;
   } else {
-    // $FlowIssue[cannot-spread-indexer]
+    // $FlowFixMe[cannot-spread-indexer]
     return {...record1, ...record2};
   }
 }

package/store/RelayModernStore.js.flow

@@ -470,6 +470,8 @@ class RelayModernStore implements Store {
           fetchTime: Date.now(),
         };
         this._releaseBuffer.push(id);
+        /* $FlowFixMe[incompatible-call] Natural Inference rollout. See
+         * https://fburl.com/gdoc/y8dn025u */
         this._roots.set(id, temporaryRootEntry);
       }
     }

package/store/RelayReader.js.flow

@@ -252,7 +252,7 @@ class RelayReader {
     const owner = this._fragmentName;
 
     this._fieldErrors.push(
-      this._selector.node.metadata?.throwOnFieldError ?? false
+      (this._selector.node.metadata?.throwOnFieldError ?? false)
         ? {
             kind: 'missing_expected_data.throw',
             owner,

package/store/RelayRecordSource.js.flow

@@ -86,7 +86,7 @@ class RelayRecordSource implements MutableRecordSource {
   }
 
   toJSON(): RecordSourceJSON {
-    const obj: RecordSourceJSON = {};
+    const obj: {...RecordSourceJSON} = {};
     for (const [key, record] of this._records) {
       obj[key] = RelayModernRecord.toJSON<null | void>(record);
     }

package/store/RelayResponseNormalizer.js.flow

@@ -67,7 +67,7 @@ const invariant = require('invariant');
 const warning = require('warning');
 
 export type GetDataID = (
-  fieldValue: {[string]: mixed},
+  fieldValue: {+[string]: mixed},
   typeName: string,
 ) => mixed;
 
@@ -250,7 +250,11 @@ class RelayResponseNormalizer {
               this._traverseSelections(selection, record, data);
             }
           } else {
-            const implementsInterface = data.hasOwnProperty(abstractKey);
+            // $FlowFixMe[method-unbinding] - data could be prototype less
+            const implementsInterface = Object.prototype.hasOwnProperty.call(
+              data,
+              abstractKey,
+            );
             const typeName = RelayModernRecord.getType(record);
             const typeID = generateTypeID(typeName);
             let typeRecord = this._recordSource.get(typeID);
@@ -271,7 +275,11 @@ class RelayResponseNormalizer {
         }
         case 'TypeDiscriminator': {
           const {abstractKey} = selection;
-          const implementsInterface = data.hasOwnProperty(abstractKey);
+          // $FlowFixMe[method-unbinding] - data could be prototype less
+          const implementsInterface = Object.prototype.hasOwnProperty.call(
+            data,
+            abstractKey,
+          );
           const typeName = RelayModernRecord.getType(record);
           const typeID = generateTypeID(typeName);
           let typeRecord = this._recordSource.get(typeID);

package/store/RelayStoreTypes.js.flow

@@ -245,7 +245,7 @@ export interface RecordSource {
 /**
  * A collection of records keyed by id.
  */
-export type RecordSourceJSON = {[DataID]: ?RecordJSON};
+export type RecordSourceJSON = {+[DataID]: ?RecordJSON};
 
 /**
  * A read/write interface for accessing and updating graph data.
@@ -258,10 +258,10 @@ export interface MutableRecordSource extends RecordSource {
 }
 
 export type CheckOptions = {
-  handlers: $ReadOnlyArray<MissingFieldHandler>,
-  defaultActorIdentifier: ActorIdentifier,
-  getTargetForActor: (actorIdentifier: ActorIdentifier) => MutableRecordSource,
-  getSourceForActor: (actorIdentifier: ActorIdentifier) => RecordSource,
+  +handlers: $ReadOnlyArray<MissingFieldHandler>,
+  +defaultActorIdentifier: ActorIdentifier,
+  +getTargetForActor: (actorIdentifier: ActorIdentifier) => MutableRecordSource,
+  +getSourceForActor: (actorIdentifier: ActorIdentifier) => RecordSource,
 };
 
 export type OperationAvailability =

package/store/ResolverCache.js.flow

@@ -32,11 +32,11 @@ export type EvaluationResult<T> = {
   error: ?Error,
 };
 
-export type ResolverFragmentResult = {
+export type ResolverFragmentResult = $ReadOnly<{
   data: mixed,
   isMissingData: boolean,
   fieldErrors: ?FieldErrors,
-};
+}>;
 
 export type GetDataForResolverFragmentFn =
   SingularReaderSelector => ResolverFragmentResult;

package/store/defaultGetDataID.js.flow

@@ -14,7 +14,7 @@
 const {VIEWER_ID, VIEWER_TYPE} = require('./ViewerPattern');
 
 function defaultGetDataID(
-  fieldValue: {[string]: mixed},
+  fieldValue: {+[string]: mixed},
   typeName: string,
 ): mixed {
   if (typeName === VIEWER_TYPE) {

package/subscription/requestSubscription.js.flow

@@ -41,16 +41,17 @@ export type SubscriptionParameters = {
  * Updated Flow type that makes use of typed graphql tagged literals with
  * type information.
  */
-export type GraphQLSubscriptionConfig<TVariables, TData, TRawResponse> = {
-  configs?: Array<DeclarativeMutationConfig>,
-  cacheConfig?: CacheConfig,
-  subscription: GraphQLSubscription<TVariables, TData, TRawResponse>,
-  variables: TVariables,
-  onCompleted?: ?() => void,
-  onError?: ?(error: Error) => void,
-  onNext?: ?(response: ?TData) => void,
-  updater?: ?SelectorStoreUpdater<TData>,
-};
+export type GraphQLSubscriptionConfig<TVariables, TData, TRawResponse> =
+  $ReadOnly<{
+    configs?: Array<DeclarativeMutationConfig>,
+    cacheConfig?: CacheConfig,
+    subscription: GraphQLSubscription<TVariables, TData, TRawResponse>,
+    variables: TVariables,
+    onCompleted?: ?() => void,
+    onError?: ?(error: Error) => void,
+    onNext?: ?(response: ?TData) => void,
+    updater?: ?SelectorStoreUpdater<TData>,
+  }>;
 
 function requestSubscription<TVariables: Variables, TData, TRawResponse>(
   environment: IEnvironment,

package/util/RelayRuntimeTypes.js.flow

@@ -62,14 +62,14 @@ export type VariablesOf<T: OperationType> = T['variables'];
  *   a given instance of executing an operation.
  */
 export type CacheConfig = {
-  force?: ?boolean,
-  poll?: ?number,
-  liveConfigId?: ?string,
-  onSubscribe?: () => void,
-  onResume?: (pauseTimeMs: number) => void,
-  onPause?: (mqttConnectionIsOk: boolean, internetIsOk: boolean) => void,
-  metadata?: {[key: string]: mixed, ...},
-  transactionId?: ?string,
+  +force?: ?boolean,
+  +poll?: ?number,
+  +liveConfigId?: ?string,
+  +onSubscribe?: () => void,
+  +onResume?: (pauseTimeMs: number) => void,
+  +onPause?: (mqttConnectionIsOk: boolean, internetIsOk: boolean) => void,
+  +metadata?: {+[key: string]: mixed, ...},
+  +transactionId?: ?string,
 };
 
 export type FetchQueryFetchPolicy = 'store-or-network' | 'network-only';