@apollo/client 4.0.0-alpha.1 → 4.0.0-alpha.10

package/cache/inmemory/types.d.ts

@@ -1,4 +1,5 @@
 import type { DocumentNode, FieldNode } from "graphql";
+import type { OperationVariables } from "@apollo/client";
 import type { Reference, StoreObject, StoreValue } from "@apollo/client/utilities";
 import type { Transaction } from "../core/cache.js";
 import type { AllFieldsModifier, CanReadFunction, Modifiers, ToReferenceFunction } from "../core/types/common.js";
@@ -60,7 +61,7 @@ export interface NormalizedCacheObject {
 export type OptimisticStoreItem = {
     id: string;
     data: NormalizedCacheObject;
-    transaction: Transaction<NormalizedCacheObject>;
+    transaction: Transaction;
 };
 export type ReadQueryOptions = {
     /**
@@ -73,14 +74,6 @@ export type ReadQueryOptions = {
     query: DocumentNode;
     variables?: Object;
     previousResult?: any;
-    /**
-     * @deprecated
-     * Using `canonizeResults` can result in memory leaks so we generally do not
-     * recommend using this option anymore.
-     * A future version of Apollo Client will contain a similar feature without
-     * the risk of memory leaks.
-     */
-    canonizeResults?: boolean;
     rootId?: string;
     config?: ApolloReducerConfig;
 };
@@ -99,13 +92,6 @@ export interface InMemoryCacheConfig extends ApolloReducerConfig {
      * Please use `cacheSizes` instead.
      */
     resultCacheMaxSize?: number;
-    /**
-     * @deprecated
-     * Using `canonizeResults` can result in memory leaks so we generally do not
-     * recommend using this option anymore.
-     * A future version of Apollo Client will contain a similar feature.
-     */
-    canonizeResults?: boolean;
     fragments?: FragmentRegistryAPI;
 }
 export interface MergeInfo {
@@ -119,7 +105,7 @@ export interface MergeTree {
 }
 export interface ReadMergeModifyContext {
     store: NormalizedCache;
-    variables?: Record<string, any>;
+    variables?: OperationVariables;
     varString?: string;
 }
 //# sourceMappingURL=types.d.ts.map

package/cache/inmemory/writeToStore.d.ts

@@ -1,5 +1,5 @@
 import type { FieldNode, SelectionSetNode } from "graphql";
-import type { Cache } from "@apollo/client/core";
+import type { Cache, OperationVariables } from "@apollo/client";
 import type { FragmentMap, FragmentMapFunction, Reference, StoreObject } from "@apollo/client/utilities";
 import type { InMemoryCache } from "./inMemoryCache.js";
 import type { StoreReader } from "./readFromStore.js";
@@ -27,7 +27,7 @@ export declare class StoreWriter {
     private reader?;
     private fragments?;
     constructor(cache: InMemoryCache, reader?: StoreReader | undefined, fragments?: InMemoryCacheConfig["fragments"]);
-    writeToStore(store: NormalizedCache, { query, result, dataId, variables, overwrite }: Cache.WriteOptions): Reference | undefined;
+    writeToStore<TData = unknown, TVariables = OperationVariables>(store: NormalizedCache, { query, result, dataId, variables, overwrite, }: Cache.WriteOptions<TData, TVariables>): Reference | undefined;
     private processSelectionSet;
     private processFieldValue;
     private flattenFields;

package/cache/inmemory/writeToStore.js

@@ -35,7 +35,7 @@ export class StoreWriter {
         this.reader = reader;
         this.fragments = fragments;
     }
-    writeToStore(store, { query, result, dataId, variables, overwrite }) {
+    writeToStore(store, { query, result, dataId, variables, overwrite, }) {
         const operationDefinition = getOperationDefinition(query);
         const merger = makeProcessedFieldsMerger();
         variables = {
@@ -48,7 +48,7 @@ export class StoreWriter {
             merge(existing, incoming) {
                 return merger.merge(existing, incoming);
             },
-            variables,
+            variables: variables,
             varString: canonicalStringify(variables),
             ...extractFragmentContext(query, this.fragments),
             overwrite: !!overwrite,
@@ -65,7 +65,7 @@ export class StoreWriter {
             context,
         });
         if (!isReference(ref)) {
-            throw newInvariantError(93, result);
+            throw newInvariantError(92, result);
         }
         // So far, the store has not been modified, so now it's time to process
         // context.incomingById and merge those incoming fields into context.store.
@@ -142,8 +142,8 @@ export class StoreWriter {
         // pending in context.incomingById, which is important whenever keyFields
         // need to be extracted from a child object that processSelectionSet has
         // turned into a Reference.
-        const readField = function () {
-            const options = normalizeReadFieldOptions(arguments, incoming, context.variables);
+        const readField = (...args) => {
+            const options = normalizeReadFieldOptions(args, incoming, context.variables);
             if (isReference(options.from)) {
                 const info = context.incomingById.get(options.from.__ref);
                 if (info) {
@@ -215,7 +215,7 @@ export class StoreWriter {
                 // provide a default value, so its absence from the written data should
                 // not be cause for alarm.
                 !policies.getReadFunction(typename, field.name.value)) {
-                __DEV__ && invariant.error(94, resultKeyNameFromField(field), result);
+                __DEV__ && invariant.error(93, resultKeyNameFromField(field), result);
             }
         });
         // Identify the result object, even if dataId was already provided,
@@ -362,7 +362,7 @@ export class StoreWriter {
                 else {
                     const fragment = getFragmentFromSelection(selection, context.lookupFragment);
                     if (!fragment && selection.kind === Kind.FRAGMENT_SPREAD) {
-                        throw newInvariantError(95, selection.name.value);
+                        throw newInvariantError(94, selection.name.value);
                     }
                     if (fragment &&
                         policies.fragmentMatches(fragment, typename, result, context.variables)) {
@@ -534,7 +534,7 @@ function warnAboutDataLoss(existingRef, incomingObj, storeFieldName, store) {
             }
         });
     }
-    __DEV__ && invariant.warn(96, fieldName, parentType, childTypenames.length ?
+    __DEV__ && invariant.warn(95, fieldName, parentType, childTypenames.length ?
         "either ensure all objects of type " +
             childTypenames.join(" and ") +
             " have an ID or a custom merge function, or "