houdini 0.13.7-alpha.1 → 0.13.7

package/build/runtime/cache/cache.d.ts

@@ -7,7 +7,7 @@ import { InMemorySubscriptions } from './subscription';
 export declare class Cache {
     _internal_unstable: CacheInternal;
     constructor(config: Config);
-    write({ layer: layerID, ...args }: {
+    write({ layer: layerID, notifySubscribers, ...args }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -16,7 +16,9 @@ export declare class Cache {
         parent?: string;
         layer?: LayerID | null;
         applyUpdates?: boolean;
-    }): LayerID;
+        notifySubscribers?: SubscriptionSpec[];
+        forceNotify?: boolean;
+    }): SubscriptionSpec[];
     read(...args: Parameters<CacheInternal['getSelection']>): {
         data: GraphQLObject | null;
         partial: boolean;
@@ -42,7 +44,7 @@ declare class CacheInternal {
         cache: Cache;
         lifetimes: GarbageCollector;
     });
-    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, }: {
+    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, forceNotify, }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -55,6 +57,7 @@ declare class CacheInternal {
         layer: Layer;
         toNotify?: SubscriptionSpec[];
         applyUpdates?: boolean;
+        forceNotify?: boolean;
     }): SubscriptionSpec[];
     getSelection({ selection, parent, variables, }: {
         selection: SubscriptionSelection;
@@ -82,7 +85,7 @@ declare class CacheInternal {
         partial: boolean;
         hasData: boolean;
     };
-    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, }: {
+    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, forceNotify, }: {
         value: GraphQLValue[];
         recordID: string;
         key: string;
@@ -94,6 +97,7 @@ declare class CacheInternal {
         fields: SubscriptionSelection;
         layer: Layer;
         startingWith: number;
+        forceNotify?: boolean;
     }): {
         nestedIDs: LinkedList;
         newIDs: (string | null)[];

package/build/runtime/cache/cache.js

@@ -75,7 +75,7 @@ var Cache = /** @class */ (function () {
     Cache.prototype.write = function (_a) {
         var e_1, _b;
         var _c;
-        var layerID = _a.layer, args = __rest(_a, ["layer"]);
+        var layerID = _a.layer, _d = _a.notifySubscribers, notifySubscribers = _d === void 0 ? [] : _d, args = __rest(_a, ["layer", "notifySubscribers"]);
         // find the correct layer
         var layer = layerID
             ? this._internal_unstable.storage.getLayer(layerID)
@@ -86,8 +86,8 @@ var Cache = /** @class */ (function () {
         // function's identity
         var notified = [];
         try {
-            for (var subscribers_1 = __values(subscribers), subscribers_1_1 = subscribers_1.next(); !subscribers_1_1.done; subscribers_1_1 = subscribers_1.next()) {
-                var spec = subscribers_1_1.value;
+            for (var _e = __values(subscribers.concat(notifySubscribers)), _f = _e.next(); !_f.done; _f = _e.next()) {
+                var spec = _f.value;
                 // if we haven't added the set yet
                 if (!notified.includes(spec.set)) {
                     notified.push(spec.set);
@@ -103,12 +103,12 @@ var Cache = /** @class */ (function () {
         catch (e_1_1) { e_1 = { error: e_1_1 }; }
         finally {
             try {
-                if (subscribers_1_1 && !subscribers_1_1.done && (_b = subscribers_1.return)) _b.call(subscribers_1);
+                if (_f && !_f.done && (_b = _e.return)) _b.call(_e);
             }
             finally { if (e_1) throw e_1.error; }
         }
         // return the id to the caller so they can resolve the layer if it was optimistic
-        return layer.id;
+        return subscribers;
     };
     // reconstruct an object with the fields/relations specified by a selection
     Cache.prototype.read = function () {
@@ -187,7 +187,7 @@ var CacheInternal = /** @class */ (function () {
         var e_2, _b;
         var _this = this;
         var _c;
-        var data = _a.data, selection = _a.selection, _d = _a.variables, variables = _d === void 0 ? {} : _d, _e = _a.root, root = _e === void 0 ? exports.rootID : _e, _f = _a.parent, parent = _f === void 0 ? exports.rootID : _f, _g = _a.applyUpdates, applyUpdates = _g === void 0 ? false : _g, layer = _a.layer, _h = _a.toNotify, toNotify = _h === void 0 ? [] : _h;
+        var data = _a.data, selection = _a.selection, _d = _a.variables, variables = _d === void 0 ? {} : _d, _e = _a.root, root = _e === void 0 ? exports.rootID : _e, _f = _a.parent, parent = _f === void 0 ? exports.rootID : _f, _g = _a.applyUpdates, applyUpdates = _g === void 0 ? false : _g, layer = _a.layer, _h = _a.toNotify, toNotify = _h === void 0 ? [] : _h, forceNotify = _a.forceNotify;
         // if the cache is disabled, dont do anything
         if (this._disabled) {
             return [];
@@ -212,12 +212,12 @@ var CacheInternal = /** @class */ (function () {
             // if the layer we are updating is the top most layer for the field
             // then its value is "live". It is providing the current value and
             // subscribers need to know if the value changed
-            var displayLayer = displayLayers.length === 0 || displayLayers.includes(layer.id);
+            var displayLayer = layer.isDisplayLayer(displayLayers);
             // if we are writing to the display layer we need to refresh the lifetime of the value
             if (displayLayer) {
                 this_1.lifetimes.resetLifetime(parent, key);
             }
-            // any non-scalar is defined as a field with no selection
+            // any scalar is defined as a field with no selection
             if (!fields) {
                 // the value to write to the layer
                 var newValue = value;
@@ -234,7 +234,7 @@ var CacheInternal = /** @class */ (function () {
                 }
                 // if the value changed on a layer that impacts the current latest value
                 var valueChanged = JSON.stringify(newValue) !== JSON.stringify(previousValue);
-                if (valueChanged && displayLayer) {
+                if (displayLayer && (valueChanged || forceNotify)) {
                     // we need to add the fields' subscribers to the set of callbacks
                     // we need to invoke
                     toNotify.push.apply(toNotify, __spread(currentSubcribers));
@@ -290,7 +290,7 @@ var CacheInternal = /** @class */ (function () {
                 // write the link to the layer
                 layer.writeLink(parent, key, linkedID);
                 // if the link target of this field changed and it was responsible for the current subscription
-                if (linkedID && displayLayer && linkChange) {
+                if (linkedID && displayLayer && (linkChange || forceNotify)) {
                     // we need to clear the subscriptions in the previous link
                     // and add them to the new link
                     if (previousValue && typeof previousValue === 'string') {
@@ -317,6 +317,7 @@ var CacheInternal = /** @class */ (function () {
                         toNotify: toNotify,
                         applyUpdates: applyUpdates,
                         layer: layer,
+                        forceNotify: true,
                     });
                 }
             }
@@ -373,6 +374,7 @@ var CacheInternal = /** @class */ (function () {
                     fields: fields,
                     layer: layer,
                     startingWith: applyUpdates && update === 'append' ? oldIDs_2.length : 0,
+                    forceNotify: forceNotify,
                 }), newIDs = _j.newIDs, nestedIDs = _j.nestedIDs;
                 // if we're supposed to apply this write as an update, we need to figure out how
                 if (applyUpdates && update) {
@@ -447,7 +449,7 @@ var CacheInternal = /** @class */ (function () {
                 // is still valid would not be triggered
                 var contentChanged = JSON.stringify(linkedIDs) !== JSON.stringify(oldIDs_2);
                 // we need to look at the last time we saw each subscriber to check if they need to be added to the spec
-                if (contentChanged) {
+                if (contentChanged || forceNotify) {
                     toNotify.push.apply(toNotify, __spread(currentSubcribers));
                 }
                 try {
@@ -777,7 +779,7 @@ var CacheInternal = /** @class */ (function () {
     CacheInternal.prototype.extractNestedListIDs = function (_a) {
         var e_11, _b;
         var _c;
-        var value = _a.value, abstract = _a.abstract, recordID = _a.recordID, key = _a.key, linkedType = _a.linkedType, fields = _a.fields, variables = _a.variables, applyUpdates = _a.applyUpdates, specs = _a.specs, layer = _a.layer, startingWith = _a.startingWith;
+        var value = _a.value, abstract = _a.abstract, recordID = _a.recordID, key = _a.key, linkedType = _a.linkedType, fields = _a.fields, variables = _a.variables, applyUpdates = _a.applyUpdates, specs = _a.specs, layer = _a.layer, startingWith = _a.startingWith, forceNotify = _a.forceNotify;
         // build up the two lists
         var nestedIDs = [];
         var newIDs = [];
@@ -798,6 +800,7 @@ var CacheInternal = /** @class */ (function () {
                     specs: specs,
                     layer: layer,
                     startingWith: startingWith,
+                    forceNotify: forceNotify,
                 });
                 // add the list of new ids to our list
                 newIDs.push.apply(newIDs, __spread(inner.newIDs));
@@ -849,6 +852,7 @@ var CacheInternal = /** @class */ (function () {
                 toNotify: specs,
                 applyUpdates: applyUpdates,
                 layer: layer,
+                forceNotify: forceNotify,
             });
             newIDs.push(linkedID);
             nestedIDs[i] = linkedID;

package/build/runtime/cache/storage.d.ts

@@ -32,6 +32,7 @@ export declare class Layer {
     getOperations(id: string, field: string): Operation[] | undefined;
     writeField(id: string, field: string, value: GraphQLField): LayerID;
     writeLink(id: string, field: string, value: null | string | LinkedList): LayerID;
+    isDisplayLayer(displayLayers: number[]): boolean;
     clear(): void;
     applyDeletes(): void;
     delete(id: string): void;

package/build/runtime/cache/storage.js

@@ -243,6 +243,7 @@ var InMemoryStorage = /** @class */ (function () {
             var layer = this.data[layerIndex++];
             // if the layer is optimistic, we can't go further
             if (layer.optimistic) {
+                layerIndex--;
                 break;
             }
             // apply the layer onto our base
@@ -351,7 +352,14 @@ var Layer = /** @class */ (function () {
         this.links[id] = __assign(__assign({}, this.links[id]), (_b = {}, _b[field] = value, _b));
         return this.id;
     };
+    Layer.prototype.isDisplayLayer = function (displayLayers) {
+        return (displayLayers.length === 0 ||
+            displayLayers.includes(this.id) ||
+            Math.max.apply(Math, __spread(displayLayers)) < this.id);
+    };
     Layer.prototype.clear = function () {
+        // before we clear the data of the layer, look for any subscribers that need to be updated
+        // now that everything has been notified we can reset the data
         this.links = {};
         this.fields = {};
         this.operations = {};

package/build/runtime/mutation.js

@@ -63,15 +63,15 @@ function mutation(document) {
     var sessionStore = adapter_mjs_1.getSession();
     // return an async function that sends the mutation go the server
     return function (variables, mutationConfig) { return __awaiter(_this, void 0, void 0, function () {
-        var optimisticResponse, layer, result, error_1;
+        var layer, optimisticResponse, toNotify, result, error_1;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    optimisticResponse = mutationConfig === null || mutationConfig === void 0 ? void 0 : mutationConfig.optimisticResponse;
                     layer = cache_1.default._internal_unstable.storage.createLayer(true);
-                    // if there is an optimistic response then we need to write the value immediately
+                    optimisticResponse = mutationConfig === null || mutationConfig === void 0 ? void 0 : mutationConfig.optimisticResponse;
+                    toNotify = [];
                     if (optimisticResponse) {
-                        cache_1.default.write({
+                        toNotify = cache_1.default.write({
                             selection: artifact.selection,
                             // make sure that any scalar values get processed into something we can cache
                             data: scalars_1.marshalSelection({
@@ -102,8 +102,14 @@ function mutation(document) {
                         selection: artifact.selection,
                         data: result.data,
                         variables: variables,
-                        // if we had an optimistic response we need to write to the appropriate layer
+                        // write to the mutation's layer
                         layer: layer.id,
+                        // notify any subscribers that we updated with the optimistic response
+                        // in order to address situations where the optimistic update was wrong
+                        notifySubscribers: toNotify,
+                        // make sure that we notify subscribers for any values that we overwrite
+                        // in order to address any race conditions when comparing the previous value
+                        forceNotify: true,
                     });
                     // merge the layer back into the cache
                     cache_1.default._internal_unstable.storage.resolveLayer(layer.id);

package/build/runtime-cjs/cache/cache.d.ts

@@ -7,7 +7,7 @@ import { InMemorySubscriptions } from './subscription';
 export declare class Cache {
     _internal_unstable: CacheInternal;
     constructor(config: Config);
-    write({ layer: layerID, ...args }: {
+    write({ layer: layerID, notifySubscribers, ...args }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -16,7 +16,9 @@ export declare class Cache {
         parent?: string;
         layer?: LayerID | null;
         applyUpdates?: boolean;
-    }): LayerID;
+        notifySubscribers?: SubscriptionSpec[];
+        forceNotify?: boolean;
+    }): SubscriptionSpec[];
     read(...args: Parameters<CacheInternal['getSelection']>): {
         data: GraphQLObject | null;
         partial: boolean;
@@ -42,7 +44,7 @@ declare class CacheInternal {
         cache: Cache;
         lifetimes: GarbageCollector;
     });
-    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, }: {
+    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, forceNotify, }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -55,6 +57,7 @@ declare class CacheInternal {
         layer: Layer;
         toNotify?: SubscriptionSpec[];
         applyUpdates?: boolean;
+        forceNotify?: boolean;
     }): SubscriptionSpec[];
     getSelection({ selection, parent, variables, }: {
         selection: SubscriptionSelection;
@@ -82,7 +85,7 @@ declare class CacheInternal {
         partial: boolean;
         hasData: boolean;
     };
-    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, }: {
+    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, forceNotify, }: {
         value: GraphQLValue[];
         recordID: string;
         key: string;
@@ -94,6 +97,7 @@ declare class CacheInternal {
         fields: SubscriptionSelection;
         layer: Layer;
         startingWith: number;
+        forceNotify?: boolean;
     }): {
         nestedIDs: LinkedList;
         newIDs: (string | null)[];

package/build/runtime-cjs/cache/cache.js

@@ -75,7 +75,7 @@ var Cache = /** @class */ (function () {
     Cache.prototype.write = function (_a) {
         var e_1, _b;
         var _c;
-        var layerID = _a.layer, args = __rest(_a, ["layer"]);
+        var layerID = _a.layer, _d = _a.notifySubscribers, notifySubscribers = _d === void 0 ? [] : _d, args = __rest(_a, ["layer", "notifySubscribers"]);
         // find the correct layer
         var layer = layerID
             ? this._internal_unstable.storage.getLayer(layerID)
@@ -86,8 +86,8 @@ var Cache = /** @class */ (function () {
         // function's identity
         var notified = [];
         try {
-            for (var subscribers_1 = __values(subscribers), subscribers_1_1 = subscribers_1.next(); !subscribers_1_1.done; subscribers_1_1 = subscribers_1.next()) {
-                var spec = subscribers_1_1.value;
+            for (var _e = __values(subscribers.concat(notifySubscribers)), _f = _e.next(); !_f.done; _f = _e.next()) {
+                var spec = _f.value;
                 // if we haven't added the set yet
                 if (!notified.includes(spec.set)) {
                     notified.push(spec.set);
@@ -103,12 +103,12 @@ var Cache = /** @class */ (function () {
         catch (e_1_1) { e_1 = { error: e_1_1 }; }
         finally {
             try {
-                if (subscribers_1_1 && !subscribers_1_1.done && (_b = subscribers_1.return)) _b.call(subscribers_1);
+                if (_f && !_f.done && (_b = _e.return)) _b.call(_e);
             }
             finally { if (e_1) throw e_1.error; }
         }
         // return the id to the caller so they can resolve the layer if it was optimistic
-        return layer.id;
+        return subscribers;
     };
     // reconstruct an object with the fields/relations specified by a selection
     Cache.prototype.read = function () {
@@ -187,7 +187,7 @@ var CacheInternal = /** @class */ (function () {
         var e_2, _b;
         var _this = this;
         var _c;
-        var data = _a.data, selection = _a.selection, _d = _a.variables, variables = _d === void 0 ? {} : _d, _e = _a.root, root = _e === void 0 ? exports.rootID : _e, _f = _a.parent, parent = _f === void 0 ? exports.rootID : _f, _g = _a.applyUpdates, applyUpdates = _g === void 0 ? false : _g, layer = _a.layer, _h = _a.toNotify, toNotify = _h === void 0 ? [] : _h;
+        var data = _a.data, selection = _a.selection, _d = _a.variables, variables = _d === void 0 ? {} : _d, _e = _a.root, root = _e === void 0 ? exports.rootID : _e, _f = _a.parent, parent = _f === void 0 ? exports.rootID : _f, _g = _a.applyUpdates, applyUpdates = _g === void 0 ? false : _g, layer = _a.layer, _h = _a.toNotify, toNotify = _h === void 0 ? [] : _h, forceNotify = _a.forceNotify;
         // if the cache is disabled, dont do anything
         if (this._disabled) {
             return [];
@@ -212,12 +212,12 @@ var CacheInternal = /** @class */ (function () {
             // if the layer we are updating is the top most layer for the field
             // then its value is "live". It is providing the current value and
             // subscribers need to know if the value changed
-            var displayLayer = displayLayers.length === 0 || displayLayers.includes(layer.id);
+            var displayLayer = layer.isDisplayLayer(displayLayers);
             // if we are writing to the display layer we need to refresh the lifetime of the value
             if (displayLayer) {
                 this_1.lifetimes.resetLifetime(parent, key);
             }
-            // any non-scalar is defined as a field with no selection
+            // any scalar is defined as a field with no selection
             if (!fields) {
                 // the value to write to the layer
                 var newValue = value;
@@ -234,7 +234,7 @@ var CacheInternal = /** @class */ (function () {
                 }
                 // if the value changed on a layer that impacts the current latest value
                 var valueChanged = JSON.stringify(newValue) !== JSON.stringify(previousValue);
-                if (valueChanged && displayLayer) {
+                if (displayLayer && (valueChanged || forceNotify)) {
                     // we need to add the fields' subscribers to the set of callbacks
                     // we need to invoke
                     toNotify.push.apply(toNotify, __spread(currentSubcribers));
@@ -290,7 +290,7 @@ var CacheInternal = /** @class */ (function () {
                 // write the link to the layer
                 layer.writeLink(parent, key, linkedID);
                 // if the link target of this field changed and it was responsible for the current subscription
-                if (linkedID && displayLayer && linkChange) {
+                if (linkedID && displayLayer && (linkChange || forceNotify)) {
                     // we need to clear the subscriptions in the previous link
                     // and add them to the new link
                     if (previousValue && typeof previousValue === 'string') {
@@ -317,6 +317,7 @@ var CacheInternal = /** @class */ (function () {
                         toNotify: toNotify,
                         applyUpdates: applyUpdates,
                         layer: layer,
+                        forceNotify: true,
                     });
                 }
             }
@@ -373,6 +374,7 @@ var CacheInternal = /** @class */ (function () {
                     fields: fields,
                     layer: layer,
                     startingWith: applyUpdates && update === 'append' ? oldIDs_2.length : 0,
+                    forceNotify: forceNotify,
                 }), newIDs = _j.newIDs, nestedIDs = _j.nestedIDs;
                 // if we're supposed to apply this write as an update, we need to figure out how
                 if (applyUpdates && update) {
@@ -447,7 +449,7 @@ var CacheInternal = /** @class */ (function () {
                 // is still valid would not be triggered
                 var contentChanged = JSON.stringify(linkedIDs) !== JSON.stringify(oldIDs_2);
                 // we need to look at the last time we saw each subscriber to check if they need to be added to the spec
-                if (contentChanged) {
+                if (contentChanged || forceNotify) {
                     toNotify.push.apply(toNotify, __spread(currentSubcribers));
                 }
                 try {
@@ -777,7 +779,7 @@ var CacheInternal = /** @class */ (function () {
     CacheInternal.prototype.extractNestedListIDs = function (_a) {
         var e_11, _b;
         var _c;
-        var value = _a.value, abstract = _a.abstract, recordID = _a.recordID, key = _a.key, linkedType = _a.linkedType, fields = _a.fields, variables = _a.variables, applyUpdates = _a.applyUpdates, specs = _a.specs, layer = _a.layer, startingWith = _a.startingWith;
+        var value = _a.value, abstract = _a.abstract, recordID = _a.recordID, key = _a.key, linkedType = _a.linkedType, fields = _a.fields, variables = _a.variables, applyUpdates = _a.applyUpdates, specs = _a.specs, layer = _a.layer, startingWith = _a.startingWith, forceNotify = _a.forceNotify;
         // build up the two lists
         var nestedIDs = [];
         var newIDs = [];
@@ -798,6 +800,7 @@ var CacheInternal = /** @class */ (function () {
                     specs: specs,
                     layer: layer,
                     startingWith: startingWith,
+                    forceNotify: forceNotify,
                 });
                 // add the list of new ids to our list
                 newIDs.push.apply(newIDs, __spread(inner.newIDs));
@@ -849,6 +852,7 @@ var CacheInternal = /** @class */ (function () {
                 toNotify: specs,
                 applyUpdates: applyUpdates,
                 layer: layer,
+                forceNotify: forceNotify,
             });
             newIDs.push(linkedID);
             nestedIDs[i] = linkedID;

package/build/runtime-cjs/cache/storage.d.ts

@@ -32,6 +32,7 @@ export declare class Layer {
     getOperations(id: string, field: string): Operation[] | undefined;
     writeField(id: string, field: string, value: GraphQLField): LayerID;
     writeLink(id: string, field: string, value: null | string | LinkedList): LayerID;
+    isDisplayLayer(displayLayers: number[]): boolean;
     clear(): void;
     applyDeletes(): void;
     delete(id: string): void;

package/build/runtime-cjs/cache/storage.js

@@ -243,6 +243,7 @@ var InMemoryStorage = /** @class */ (function () {
             var layer = this.data[layerIndex++];
             // if the layer is optimistic, we can't go further
             if (layer.optimistic) {
+                layerIndex--;
                 break;
             }
             // apply the layer onto our base
@@ -351,7 +352,14 @@ var Layer = /** @class */ (function () {
         this.links[id] = __assign(__assign({}, this.links[id]), (_b = {}, _b[field] = value, _b));
         return this.id;
     };
+    Layer.prototype.isDisplayLayer = function (displayLayers) {
+        return (displayLayers.length === 0 ||
+            displayLayers.includes(this.id) ||
+            Math.max.apply(Math, __spread(displayLayers)) < this.id);
+    };
     Layer.prototype.clear = function () {
+        // before we clear the data of the layer, look for any subscribers that need to be updated
+        // now that everything has been notified we can reset the data
         this.links = {};
         this.fields = {};
         this.operations = {};

package/build/runtime-cjs/mutation.js

@@ -63,15 +63,15 @@ function mutation(document) {
     var sessionStore = adapter_mjs_1.getSession();
     // return an async function that sends the mutation go the server
     return function (variables, mutationConfig) { return __awaiter(_this, void 0, void 0, function () {
-        var optimisticResponse, layer, result, error_1;
+        var layer, optimisticResponse, toNotify, result, error_1;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
-                    optimisticResponse = mutationConfig === null || mutationConfig === void 0 ? void 0 : mutationConfig.optimisticResponse;
                     layer = cache_1.default._internal_unstable.storage.createLayer(true);
-                    // if there is an optimistic response then we need to write the value immediately
+                    optimisticResponse = mutationConfig === null || mutationConfig === void 0 ? void 0 : mutationConfig.optimisticResponse;
+                    toNotify = [];
                     if (optimisticResponse) {
-                        cache_1.default.write({
+                        toNotify = cache_1.default.write({
                             selection: artifact.selection,
                             // make sure that any scalar values get processed into something we can cache
                             data: scalars_1.marshalSelection({
@@ -102,8 +102,14 @@ function mutation(document) {
                         selection: artifact.selection,
                         data: result.data,
                         variables: variables,
-                        // if we had an optimistic response we need to write to the appropriate layer
+                        // write to the mutation's layer
                         layer: layer.id,
+                        // notify any subscribers that we updated with the optimistic response
+                        // in order to address situations where the optimistic update was wrong
+                        notifySubscribers: toNotify,
+                        // make sure that we notify subscribers for any values that we overwrite
+                        // in order to address any race conditions when comparing the previous value
+                        forceNotify: true,
                     });
                     // merge the layer back into the cache
                     cache_1.default._internal_unstable.storage.resolveLayer(layer.id);

package/build/runtime-esm/cache/cache.d.ts

@@ -7,7 +7,7 @@ import { InMemorySubscriptions } from './subscription';
 export declare class Cache {
     _internal_unstable: CacheInternal;
     constructor(config: Config);
-    write({ layer: layerID, ...args }: {
+    write({ layer: layerID, notifySubscribers, ...args }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -16,7 +16,9 @@ export declare class Cache {
         parent?: string;
         layer?: LayerID | null;
         applyUpdates?: boolean;
-    }): LayerID;
+        notifySubscribers?: SubscriptionSpec[];
+        forceNotify?: boolean;
+    }): SubscriptionSpec[];
     read(...args: Parameters<CacheInternal['getSelection']>): {
         data: GraphQLObject;
         partial: boolean;
@@ -42,7 +44,7 @@ declare class CacheInternal {
         cache: Cache;
         lifetimes: GarbageCollector;
     });
-    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, }: {
+    writeSelection({ data, selection, variables, root, parent, applyUpdates, layer, toNotify, forceNotify, }: {
         data: {
             [key: string]: GraphQLValue;
         };
@@ -55,6 +57,7 @@ declare class CacheInternal {
         layer: Layer;
         toNotify?: SubscriptionSpec[];
         applyUpdates?: boolean;
+        forceNotify?: boolean;
     }): SubscriptionSpec[];
     getSelection({ selection, parent, variables, }: {
         selection: SubscriptionSelection;
@@ -82,7 +85,7 @@ declare class CacheInternal {
         partial: boolean;
         hasData: boolean;
     };
-    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, }: {
+    extractNestedListIDs({ value, abstract, recordID, key, linkedType, fields, variables, applyUpdates, specs, layer, startingWith, forceNotify, }: {
         value: GraphQLValue[];
         recordID: string;
         key: string;
@@ -94,6 +97,7 @@ declare class CacheInternal {
         fields: SubscriptionSelection;
         layer: Layer;
         startingWith: number;
+        forceNotify?: boolean;
     }): {
         nestedIDs: LinkedList;
         newIDs: (string | null)[];