@apollo/client 4.0.0-alpha.21 → 4.0.0-alpha.23

package/__cjs/core/ObservableQuery.cjs

@@ -191,13 +191,6 @@ class ObservableQuery {
         const fetchPolicy = this.queryManager.prioritizeCacheValues ?
             "cache-first"
             : initialFetchPolicy || this.options.fetchPolicy;
-        const defaultResult = {
-            data: undefined,
-            dataState: "empty",
-            loading: true,
-            networkStatus: networkStatus_js_1.NetworkStatus.loading,
-            partial: true,
-        };
         const cacheResult = () => {
             const diff = this.getCacheDiff();
             // TODO: queryInfo.getDiff should handle this since cache.diff returns a
@@ -216,7 +209,13 @@ class ObservableQuery {
             });
         };
         switch (fetchPolicy) {
-            case "cache-only":
+            case "cache-only": {
+                return {
+                    ...cacheResult(),
+                    loading: false,
+                    networkStatus: networkStatus_js_1.NetworkStatus.ready,
+                };
+            }
             case "cache-first":
                 return cacheResult();
             case "cache-and-network":
@@ -226,13 +225,9 @@ class ObservableQuery {
                     networkStatus: networkStatus_js_1.NetworkStatus.loading,
                 };
             case "standby":
-                return {
-                    ...defaultResult,
-                    loading: false,
-                    networkStatus: networkStatus_js_1.NetworkStatus.ready,
-                };
+                return empty;
             default:
-                return defaultResult;
+                return uninitialized;
         }
     }
     resubscribeCache() {
@@ -335,6 +330,7 @@ class ObservableQuery {
      * the previous values of those variables will be used.
      */
     refetch(variables) {
+        const { fetchPolicy } = this.options;
         const reobserveOptions = {
             // Always disable polling for refetches.
             pollInterval: 0,
@@ -342,7 +338,6 @@ class ObservableQuery {
         // Unless the provided fetchPolicy always consults the network
         // (no-cache, network-only, or cache-and-network), override it with
         // network-only to force the refetch for this fetchQuery call.
-        const { fetchPolicy } = this.options;
         if (fetchPolicy === "no-cache") {
             reobserveOptions.fetchPolicy = "no-cache";
         }
@@ -353,7 +348,7 @@ class ObservableQuery {
             const queryDef = (0, internal_1.getQueryDefinition)(this.query);
             const vars = queryDef.variableDefinitions;
             if (!vars || !vars.some((v) => v.variable.name.value === "variables")) {
-                __DEV__ && invariant_1.invariant.warn(76, variables, queryDef.name?.value || queryDef);
+                __DEV__ && invariant_1.invariant.warn(79, variables, queryDef.name?.value || queryDef);
             }
         }
         if (variables && !(0, equality_1.equal)(this.variables, variables)) {
@@ -369,16 +364,21 @@ class ObservableQuery {
     /**
      * A function that helps you fetch the next set of results for a [paginated list field](https://www.apollographql.com/docs/react/pagination/core-api/).
      */
-    fetchMore(fetchMoreOptions) {
+    fetchMore({ query, variables, context, errorPolicy, updateQuery, }) {
+        (0, invariant_1.invariant)(
+            this.options.fetchPolicy !== "cache-only",
+            80,
+            (0, internal_1.getOperationName)(this.query, "(anonymous)")
+        );
         const combinedOptions = {
-            ...(fetchMoreOptions.query ? fetchMoreOptions : ({
-                ...this.options,
-                query: this.options.query,
-                ...fetchMoreOptions,
-                variables: {
-                    ...this.variables,
-                    ...fetchMoreOptions.variables,
-                },
+            ...(0, internal_1.compact)(this.options, { errorPolicy: "none" }, {
+                query,
+                context,
+                errorPolicy,
+            }),
+            variables: (query ? variables : ({
+                ...this.variables,
+                ...variables,
             })),
             // The fetchMore request goes immediately to the network and does
             // not automatically write its result to the cache (hence no-cache
@@ -395,14 +395,13 @@ class ObservableQuery {
         // as well as the document passed in `fetchMoreOptions` to ensure the cache
         // uses the most up-to-date document which may rely on runtime conditionals.
         this.lastQuery =
-            fetchMoreOptions.query ?
+            query ?
                 this.transformDocument(this.options.query)
                 : combinedOptions.query;
         let wasUpdated = false;
-        const updateQuery = fetchMoreOptions?.updateQuery;
         const isCached = this.options.fetchPolicy !== "no-cache";
         if (!isCached) {
-            (0, invariant_1.invariant)(updateQuery, 77);
+            (0, invariant_1.invariant)(updateQuery, 81);
         }
         const { finalize, pushNotification } = this.pushOperation(networkStatus_js_1.NetworkStatus.fetchMore);
         pushNotification({
@@ -426,7 +425,6 @@ class ObservableQuery {
                 // fetchMore cache results back to this ObservableQuery.
                 this.queryManager.cache.batch({
                     update: (cache) => {
-                        const { updateQuery } = fetchMoreOptions;
                         if (updateQuery) {
                             cache.updateQuery({
                                 query: this.query,
@@ -535,7 +533,7 @@ class ObservableQuery {
                         onError(error);
                     }
                     else {
-                        __DEV__ && invariant_1.invariant.error(78, error);
+                        __DEV__ && invariant_1.invariant.error(82, error);
                     }
                     return;
                 }
@@ -559,9 +557,10 @@ class ObservableQuery {
     * 
     * @deprecated This is an internal API and should not be used directly. This can be removed or changed at any time.
     */
-    silentSetOptions(newOptions) {
+    applyOptions(newOptions) {
         const mergedOptions = (0, internal_1.compact)(this.options, newOptions || {});
         assign(this.options, mergedOptions);
+        this.updatePolling();
     }
     /**
      * Update the variables of this observable query, and fetch the new results
@@ -768,20 +767,28 @@ class ObservableQuery {
         return { fromLink, subscription, observable };
     }
     // Turns polling on or off based on this.options.pollInterval.
+    didWarnCacheOnlyPolling = false;
     updatePolling() {
         // Avoid polling in SSR mode
         if (this.queryManager.ssrMode) {
             return;
         }
-        const { pollingInfo, options: { pollInterval }, } = this;
-        if (!pollInterval || !this.hasObservers()) {
+        const { pollingInfo, options: { fetchPolicy, pollInterval }, } = this;
+        if (!pollInterval || !this.hasObservers() || fetchPolicy === "cache-only") {
+            if (environment_1.__DEV__) {
+                if (!this.didWarnCacheOnlyPolling &&
+                    pollInterval &&
+                    fetchPolicy === "cache-only") {
+                    __DEV__ && invariant_1.invariant.warn(83, (0, internal_1.getOperationName)(this.query, "(anonymous)"));
+                    this.didWarnCacheOnlyPolling = true;
+                }
+            }
             this.cancelPolling();
             return;
         }
-        if (pollingInfo && pollingInfo.interval === pollInterval) {
+        if (pollingInfo?.interval === pollInterval) {
             return;
         }
-        (0, invariant_1.invariant)(pollInterval, 79);
         const info = pollingInfo || (this.pollingInfo = {});
         info.interval = pollInterval;
         const maybeFetch = () => {
@@ -981,17 +988,14 @@ class ObservableQuery {
         return this.queryManager.transform(document);
     }
     maskResult(result) {
-        return result && "data" in result ?
-            {
-                ...result,
-                data: this.queryManager.maskOperation({
-                    document: this.query,
-                    data: result.data,
-                    fetchPolicy: this.options.fetchPolicy,
-                    cause: this,
-                }),
-            }
-            : result;
+        const masked = this.queryManager.maskOperation({
+            document: this.query,
+            data: result.data,
+            fetchPolicy: this.options.fetchPolicy,
+            cause: this,
+        });
+        // Maintain object identity as much as possible
+        return masked === result.data ? result : { ...result, data: masked };
     }
     dirty = false;
     notifyTimeout;
@@ -1198,7 +1202,7 @@ class ObservableQuery {
                         loading: false,
                     }
                     : notification.value;
-            if (result.error && result.dataState === "streaming") {
+            if (notification.kind === "E" && result.dataState === "streaming") {
                 result.dataState = "complete";
             }
             if (result.error) {
@@ -1268,7 +1272,7 @@ class ObservableQuery {
 exports.ObservableQuery = ObservableQuery;
 function logMissingFieldErrors(missing) {
     if (environment_1.__DEV__ && missing) {
-        __DEV__ && invariant_1.invariant.debug(80, missing);
+        __DEV__ && invariant_1.invariant.debug(84, missing);
     }
 }
 function isEqualQuery(a, b) {