@apollo/client 4.0.10 → 4.0.11

package/__cjs/core/ObservableQuery.cjs

@@ -31,6 +31,7 @@ const empty = {
 class ObservableQuery {
     options;
     queryName;
+    variablesUnknown = false;
     /**
     * @internal will be read and written from `QueryInfo`
     * 
@@ -79,6 +80,10 @@ class ObservableQuery {
         const { fetchPolicy = defaultFetchPolicy, 
         // Make sure we don't store "standby" as the initialFetchPolicy.
         initialFetchPolicy = fetchPolicy === "standby" ? defaultFetchPolicy : (fetchPolicy), } = options;
+        if (options[internal_1.variablesUnknownSymbol]) {
+            (0, invariant_1.invariant)(fetchPolicy === "standby", 77);
+            this.variablesUnknown = true;
+        }
         this.lastQuery = transformedQuery;
         this.options = {
             ...options,
@@ -380,7 +385,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(77, variables, queryDef.name?.value || queryDef);
+                __DEV__ && invariant_1.invariant.warn(78, variables, queryDef.name?.value || queryDef);
             }
         }
         if (variables && !(0, equality_1.equal)(this.variables, variables)) {
@@ -396,7 +401,7 @@ class ObservableQuery {
     fetchMore({ query, variables, context, errorPolicy, updateQuery, }) {
         (0, invariant_1.invariant)(
             this.options.fetchPolicy !== "cache-only",
-            78,
+            79,
             (0, internal_1.getOperationName)(this.query, "(anonymous)")
         );
         const combinedOptions = {
@@ -430,7 +435,7 @@ class ObservableQuery {
         let wasUpdated = false;
         const isCached = this.options.fetchPolicy !== "no-cache";
         if (!isCached) {
-            (0, invariant_1.invariant)(updateQuery, 79);
+            (0, invariant_1.invariant)(updateQuery, 80);
         }
         const { finalize, pushNotification } = this.pushOperation(networkStatus_js_1.NetworkStatus.fetchMore);
         pushNotification({
@@ -566,7 +571,7 @@ class ObservableQuery {
                         onError(error);
                     }
                     else {
-                        invariant_1.invariant.error(80, error);
+                        invariant_1.invariant.error(81, error);
                     }
                     return;
                 }
@@ -810,7 +815,7 @@ class ObservableQuery {
                 if (!this.didWarnCacheOnlyPolling &&
                     pollInterval &&
                     fetchPolicy === "cache-only") {
-                    __DEV__ && invariant_1.invariant.warn(81, (0, internal_1.getOperationName)(this.query, "(anonymous)"));
+                    __DEV__ && invariant_1.invariant.warn(82, (0, internal_1.getOperationName)(this.query, "(anonymous)"));
                     this.didWarnCacheOnlyPolling = true;
                 }
             }
@@ -885,6 +890,10 @@ class ObservableQuery {
         const oldVariables = this.variables;
         const oldFetchPolicy = this.options.fetchPolicy;
         const mergedOptions = (0, internal_1.compact)(this.options, newOptions || {});
+        // This request will hit the network, so even if there are no variables,
+        // we now know that's intentional. (see #12996)
+        // Even if that happens only once, we want `variablesUnknown` to stay false permanently.
+        this.variablesUnknown &&= mergedOptions.fetchPolicy === "standby";
         const options = useDisposableObservable ?
             // Disposable Observable fetches receive a shallow copy of this.options
             // (merged with newOptions), leaving this.options unmodified.
@@ -1162,7 +1171,9 @@ class ObservableQuery {
         // "fetchMore" having priority over "polling" or "loading" network statuses
         // as for now we just take the "latest" operation that is still active,
         // as that lines up best with previous behavior[]
-        const operation = Array.from(this.activeOperations.values()).findLast((operation) => isEqualQuery(operation, this) && operation.override !== undefined);
+        const operation = Array.from(this.activeOperations.values())
+            .reverse()
+            .find((operation) => isEqualQuery(operation, this) && operation.override !== undefined);
         return operation?.override ?? baseNetworkStatus;
     }
     abortActiveOperations() {
@@ -1307,7 +1318,7 @@ class ObservableQuery {
 exports.ObservableQuery = ObservableQuery;
 function logMissingFieldErrors(missing) {
     if (environment_1.__DEV__ && missing) {
-        __DEV__ && invariant_1.invariant.debug(82, missing);
+        __DEV__ && invariant_1.invariant.debug(83, missing);
     }
 }
 function isEqualQuery(a, b) {