@apollo/client 4.3.3 → 4.3.4

package/cache/inmemory/policies.js

@@ -242,7 +242,7 @@ export class Policies {
         const rootId = "ROOT_" + which.toUpperCase();
         const old = this.rootTypenamesById[rootId];
         if (typename !== old) {
-            invariant(!old || old === which, 114, which);
+            invariant(!old || old === which, 115, which);
             // First, delete any old __typename associated with this rootId from
             // rootIdsByTypename.
             if (old)
@@ -385,7 +385,7 @@ export class Policies {
                 if (supertypeSet.has(supertype)) {
                     if (!typenameSupertypeSet.has(supertype)) {
                         if (checkingFuzzySubtypes) {
-                            __DEV__ && invariant.warn(115, typename, supertype);
+                            __DEV__ && invariant.warn(116, typename, supertype);
                         }
                         // Record positive results for faster future lookup.
                         // Unfortunately, we cannot safely cache negative results,
@@ -640,7 +640,7 @@ export function normalizeReadFieldOptions(readFieldArgs, objectOrReference, vari
         }
     }
     if (__DEV__ && options.from === void 0) {
-        __DEV__ && invariant.warn(116, stringifyForDisplay(Array.from(readFieldArgs)));
+        __DEV__ && invariant.warn(117, stringifyForDisplay(Array.from(readFieldArgs)));
     }
     if (void 0 === options.variables) {
         options.variables = variables;
@@ -650,7 +650,7 @@ export function normalizeReadFieldOptions(readFieldArgs, objectOrReference, vari
 function makeMergeObjectsFunction(store) {
     return function mergeObjects(existing, incoming) {
         if (isArray(existing) || isArray(incoming)) {
-            throw newInvariantError(117);
+            throw newInvariantError(118);
         }
         // These dynamic checks are necessary because the parameters of a
         // custom merge function can easily have the any type, so the type
@@ -687,6 +687,6 @@ function makeMergeObjectsFunction(store) {
     };
 }
 function warnAboutScalarConfig(typename, fieldName, scalar, kind) {
-    __DEV__ && invariant.warn(118, `${typename}.${fieldName}`, scalar, kind);
+    __DEV__ && invariant.warn(119, `${typename}.${fieldName}`, scalar, kind);
 }
 //# sourceMappingURL=policies.js.map

package/cache/inmemory/readFromStore.js

@@ -69,7 +69,7 @@ export class StoreReader {
                 20000 /* defaultCacheSizes["inMemoryCache.prunePartialBoundaries"] */,
             makeCacheKey: ({ boundaries, context, selectionSet }) => {
                 if (supportsResultCaching(context.store)) {
-                    return this.keyMaker.lookup(selectionSet, boundaries, context.streamInfo, context.deferInfo);
+                    return this.keyMaker.lookup(selectionSet, boundaries, context.streamInfo, context.isDeferPending);
                 }
             },
         });
@@ -84,7 +84,7 @@ export class StoreReader {
                 20000 /* defaultCacheSizes["inMemoryCache.prunePartialStreamArray"] */,
             makeCacheKey: ({ field, context, boundaries }) => {
                 if (supportsResultCaching(context.store)) {
-                    return this.keyMaker.lookup(field, boundaries, context.streamInfo, context.deferInfo);
+                    return this.keyMaker.lookup(field, boundaries, context.streamInfo, context.isDeferPending);
                 }
             },
         });
@@ -282,7 +282,7 @@ export class StoreReader {
                         if (typename) {
                             const policy = policies["getFieldPolicy"](typename, fieldName);
                             if (policy?.scalar) {
-                                __DEV__ && invariant.warn(119, `${typename}.${fieldName}`, policy.scalar);
+                                __DEV__ && invariant.warn(120, `${typename}.${fieldName}`, policy.scalar);
                             }
                         }
                     }
@@ -319,7 +319,7 @@ export class StoreReader {
             else {
                 const fragment = getFragmentFromSelection(selection, context.lookupFragment);
                 if (!fragment && selection.kind === Kind.FRAGMENT_SPREAD) {
-                    throw newInvariantError(120, selection.name.value);
+                    throw newInvariantError(121, selection.name.value);
                 }
                 if (fragment && policies.fragmentMatches(fragment, typename)) {
                     const isDeferBoundary = isDeferredFragment(selection, context.variables);
@@ -450,7 +450,10 @@ export class StoreReader {
         if (data == null || !boundaries) {
             return { result: data, dataState: "complete" };
         }
-        const merger = new DeepMerger();
+        // `truncate` is only necessary for the Defer20220824 protocol. the
+        // `arrayMerge` option can be removed when we stop supporting the old
+        // protocol. See the comment below on array merging for more info.
+        const merger = new DeepMerger({ arrayMerge: "truncate" });
         let changed = false;
         let dataState = "complete";
         const result = {};
@@ -479,7 +482,22 @@ export class StoreReader {
                         path: path.concat(resultName),
                     });
                     changed ||= pruned.result !== fieldValue;
-                    result[resultName] = pruned.result;
+                    const previous = result[resultName];
+                    let merged = merger.merge(previous, pruned.result);
+                    // The Defer20220824 protocol allows more than one selection of a
+                    // list field with `@stream` when the `@stream` arguments are the
+                    // same. Each selection can prune the list to a different length, so
+                    // we keep the shortest length. The `truncate` option of the merger
+                    // handles a shorter later visit, and this check handles a shorter
+                    // earlier visit. GraphQL 17 rejects any overlapping `@stream` field,
+                    // so all visits give the same length. When we no longer support the
+                    // older protocol, we can remove this check and the `truncate` option,
+                    // and simplify to:
+                    // result[resultName] = merger.merge(result[resultName], pruned.result);
+                    if (previous && previous.length < merged.length) {
+                        merged = merged.slice(0, previous.length);
+                    }
+                    result[resultName] = merged;
                     dataState = mergeDataState(dataState, pruned.dataState);
                 }
                 else if (!selection.selectionSet) {
@@ -513,10 +531,10 @@ export class StoreReader {
             // dropped or not.
             const fragment = getFragmentFromSelection(selection, lookupFragment);
             let prune = false;
-            if (context.deferInfo && isDeferredFragment(selection, variables)) {
+            if (context.isDeferPending && isDeferredFragment(selection, variables)) {
                 const directive = selection.directives?.find((d) => d.name.value === "defer");
                 const label = directive && getDirectiveArgValue(directive, "label", Kind.STRING);
-                prune = !!context.deferInfo.peekArray(path.concat(label || []));
+                prune = context.isDeferPending(path, label);
             }
             if (fragment && policies.fragmentMatches(fragment, data.__typename)) {
                 if (boundaries.has(selection) || prune) {
@@ -616,7 +634,7 @@ function assertSelectionSetForIdValue(store, field, fieldValue) {
             if (isNonNullObject(value)) {
                 invariant(
                     !isReference(value),
-                    121,
+                    122,
                     getTypenameFromStoreObject(store, value),
                     field.name.value
                 );
@@ -680,7 +698,7 @@ function shouldPrune({ dataState }, context) {
         (context.streamInfo ||
             // The network hasn't delivered these @defer boundaries yet, so prune
             // the (possibly complete) cached data sitting at them.
-            context.deferInfo));
+            context.isDeferPending));
     }
     return false;
 }