houdini 1.2.1 → 1.2.3

package/build/vite-esm/index.js

@@ -67153,10 +67153,10 @@ var ListManager = class {
     this.lists.get(list.name).get(parentID).lists.push(handler);
     this.listsByField.get(parentID).get(list.key).push(handler);
   }
-  removeIDFromAllLists(id) {
+  removeIDFromAllLists(id, layer) {
     for (const fieldMap of this.lists.values()) {
       for (const list of fieldMap.values()) {
-        list.removeID(id);
+        list.removeID(id, void 0, layer);
       }
     }
   }
@@ -67313,7 +67313,7 @@ var List = class {
       layer: layer?.id
     });
   }
-  removeID(id, variables = {}) {
+  removeID(id, variables = {}, layer) {
     if (!this.validateWhen()) {
       return;
     }
@@ -67360,7 +67360,7 @@ var List = class {
       subscribers.map((sub) => sub[0]),
       variables
     );
-    this.cache._internal_unstable.storage.remove(parentID, targetKey, targetID);
+    this.cache._internal_unstable.storage.remove(parentID, targetKey, targetID, layer);
     for (const [spec] of subscribers) {
       spec.set(
         this.cache._internal_unstable.getSelection({
@@ -67373,12 +67373,12 @@ var List = class {
     }
     return true;
   }
-  remove(data, variables = {}) {
+  remove(data, variables = {}, layer) {
     const targetID = this.cache._internal_unstable.id(this.listType(data), data);
     if (!targetID) {
       return;
     }
-    return this.removeID(targetID, variables);
+    return this.removeID(targetID, variables, layer);
   }
   listType(data) {
     return data.__typename || this.type;
@@ -67410,7 +67410,7 @@ var List = class {
     layer,
     where
   }) {
-    if (!this.remove(data, variables)) {
+    if (!this.remove(data, variables, layer)) {
       this.addToList(selection, data, variables, where, layer);
     }
   }
@@ -67547,7 +67547,7 @@ var StaleManager = class {
 // src/runtime/cache/storage.ts
 var InMemoryStorage = class {
   data;
-  idCount = 0;
+  idCount = 1;
   rank = 0;
   constructor() {
     this.data = [];
@@ -67567,11 +67567,11 @@ var InMemoryStorage = class {
   insert(id, field, location, target) {
     return this.topLayer.insert(id, field, location, target);
   }
-  remove(id, field, target) {
-    return this.topLayer.remove(id, field, target);
+  remove(id, field, target, layerToUser = this.topLayer) {
+    return layerToUser.remove(id, field, target);
   }
-  delete(id) {
-    return this.topLayer.delete(id);
+  delete(id, layerToUser = this.topLayer) {
+    return layerToUser.delete(id);
   }
   deleteField(id, field) {
     return this.topLayer.deleteField(id, field);
@@ -67943,6 +67943,9 @@ var InMemorySubscriptions = class {
   subscribers = {};
   referenceCounts = {};
   keyVersions = {};
+  activeFields(parent2) {
+    return Object.keys(this.subscribers[parent2] || {});
+  }
   add({
     parent: parent2,
     spec,
@@ -68211,20 +68214,7 @@ var Cache = class {
   }) {
     const layer = layerID ? this._internal_unstable.storage.getLayer(layerID) : this._internal_unstable.storage.topLayer;
     const subscribers = this._internal_unstable.writeSelection({ ...args, layer }).map((sub) => sub[0]);
-    const notified = [];
-    for (const spec of subscribers.concat(notifySubscribers)) {
-      if (!notified.includes(spec.set)) {
-        notified.push(spec.set);
-        spec.set(
-          this._internal_unstable.getSelection({
-            parent: spec.parentID || rootID,
-            selection: spec.selection,
-            variables: spec.variables?.() || {},
-            ignoreMasking: false
-          }).data
-        );
-      }
-    }
+    this.#notifySubscribers(subscribers.concat(notifySubscribers));
     return subscribers;
   }
   read(...args) {
@@ -68263,10 +68253,10 @@ var Cache = class {
     }
     return handler;
   }
-  delete(id) {
+  delete(id, layer) {
     this._internal_unstable.subscriptions.removeAllSubscribers(id);
-    this._internal_unstable.lists.removeIDFromAllLists(id);
-    this._internal_unstable.storage.delete(id);
+    this._internal_unstable.lists.removeIDFromAllLists(id, layer);
+    this._internal_unstable.storage.delete(id, layer);
   }
   setConfig(config4) {
     this._internal_unstable.setConfig(config4);
@@ -68298,6 +68288,70 @@ var Cache = class {
   config() {
     return this._internal_unstable.config;
   }
+  clearLayer(layerID) {
+    const layer = this._internal_unstable.storage.getLayer(layerID);
+    if (!layer) {
+      throw new Error("Cannot find layer with id: " + layerID);
+    }
+    const toNotify = [];
+    const allFields = [];
+    for (const target of [layer.fields, layer.links]) {
+      for (const [id, fields] of Object.entries(target)) {
+        allFields.push(
+          ...Object.entries(fields).map(([field, value]) => ({ id, field, value }))
+        );
+      }
+    }
+    const displayFields = [];
+    for (const pair of allFields) {
+      const { displayLayers } = this._internal_unstable.storage.get(pair.id, pair.field);
+      if (!displayLayers.includes(layerID)) {
+        continue;
+      }
+      displayFields.push(pair);
+    }
+    for (const [id, operation] of Object.entries(layer.operations)) {
+      if (operation.deleted) {
+        displayFields.push(
+          ...this._internal_unstable.subscriptions.activeFields(id).map((field) => ({ id, field }))
+        );
+      }
+      const fields = Object.keys(operation.fields ?? {});
+      if (fields.length > 0) {
+        displayFields.push(...fields.map((field) => ({ id, field })));
+      }
+    }
+    layer.clear();
+    for (const display of displayFields) {
+      const { field, id } = display;
+      const notify = !("value" in display) || this._internal_unstable.storage.get(id, field).value !== display.value;
+      if (notify) {
+        toNotify.push(
+          ...this._internal_unstable.subscriptions.get(id, field).map((sub) => sub[0])
+        );
+      }
+    }
+    this.#notifySubscribers(toNotify);
+  }
+  #notifySubscribers(subs) {
+    if (subs.length === 0) {
+      return;
+    }
+    const notified = [];
+    for (const spec of subs) {
+      if (!notified.includes(spec.set)) {
+        notified.push(spec.set);
+        spec.set(
+          this._internal_unstable.getSelection({
+            parent: spec.parentID || rootID,
+            selection: spec.selection,
+            variables: spec.variables?.() || {},
+            ignoreMasking: false
+          }).data
+        );
+      }
+    }
+  }
 };
 var CacheInternal = class {
   _disabled = false;
@@ -68585,8 +68639,16 @@ var CacheInternal = class {
               operation.position || "last",
               layer
             );
+          } else if (operation.action === "toggle" && target instanceof Object && fieldSelection && operation.list) {
+            this.cache.list(operation.list, parentID, operation.target === "all").when(operation.when).toggleElement({
+              selection: fieldSelection,
+              data: target,
+              variables,
+              where: operation.position || "last",
+              layer
+            });
           } else if (operation.action === "remove" && target instanceof Object && fieldSelection && operation.list) {
-            this.cache.list(operation.list, parentID, operation.target === "all").when(operation.when).remove(target, variables);
+            this.cache.list(operation.list, parentID, operation.target === "all").when(operation.when).remove(target, variables, layer);
           } else if (operation.action === "delete" && operation.type) {
             if (typeof target !== "string") {
               throw new Error("Cannot delete a record with a non-string ID");
@@ -68595,15 +68657,7 @@ var CacheInternal = class {
             if (!targetID) {
               continue;
             }
-            this.cache.delete(targetID);
-          } else if (operation.action === "toggle" && target instanceof Object && fieldSelection && operation.list) {
-            this.cache.list(operation.list, parentID, operation.target === "all").when(operation.when).toggleElement({
-              selection: fieldSelection,
-              data: target,
-              variables,
-              where: operation.position || "last",
-              layer
-            });
+            this.cache.delete(targetID, layer);
           }
         }
       }
@@ -69138,7 +69192,7 @@ var fragment = documentPlugin(ArtifactKind.Fragment, function() {
 var mutation = documentPlugin(ArtifactKind.Mutation, () => {
   return {
     async start(ctx, { next, marshalVariables }) {
-      const layer = cache_default._internal_unstable.storage.createLayer(true);
+      const layerOptimistic = cache_default._internal_unstable.storage.createLayer(true);
       const optimisticResponse = ctx.stuff.optimisticResponse;
       let toNotify = [];
       if (optimisticResponse) {
@@ -69149,25 +69203,29 @@ var mutation = documentPlugin(ArtifactKind.Mutation, () => {
             data: optimisticResponse
           }),
           variables: marshalVariables(ctx),
-          layer: layer.id
+          layer: layerOptimistic.id
         });
       }
       ctx.cacheParams = {
         ...ctx.cacheParams,
-        layer,
+        layer: layerOptimistic,
         notifySubscribers: toNotify,
         forceNotify: true
       };
       next(ctx);
     },
     afterNetwork(ctx, { resolve: resolve2 }) {
-      ctx.cacheParams?.layer?.clear();
+      if (ctx.cacheParams?.layer) {
+        cache_default.clearLayer(ctx.cacheParams.layer.id);
+      }
       resolve2(ctx);
     },
     end(ctx, { resolve: resolve2, value }) {
       const hasErrors = value.errors && value.errors.length > 0;
       if (hasErrors) {
-        ctx.cacheParams?.layer?.clear();
+        if (ctx.cacheParams?.layer) {
+          cache_default.clearLayer(ctx.cacheParams.layer.id);
+        }
       }
       if (ctx.cacheParams?.layer) {
         cache_default._internal_unstable.storage.resolveLayer(ctx.cacheParams.layer.id);
@@ -69177,7 +69235,7 @@ var mutation = documentPlugin(ArtifactKind.Mutation, () => {
     catch(ctx, { error }) {
       if (ctx.cacheParams?.layer) {
         const { layer } = ctx.cacheParams;
-        layer.clear();
+        cache_default.clearLayer(layer.id);
         cache_default._internal_unstable.storage.resolveLayer(layer.id);
       }
       throw error;
@@ -70473,18 +70531,14 @@ function flattenSelections({
   filepath,
   selections,
   fragmentDefinitions,
-  ignoreMaskDisable,
-  keepFragmentSpreadNodes,
-  hoistFragments
+  applyFragments
 }) {
   const fields = new FieldCollection({
     config: config4,
     filepath,
     selections,
     fragmentDefinitions,
-    ignoreMaskDisable: !!ignoreMaskDisable,
-    keepFragmentSpreadNodes: !!keepFragmentSpreadNodes,
-    hoistFragments
+    applyFragments: !!applyFragments
   });
   return fields.toSelectionSet();
 }
@@ -70495,18 +70549,14 @@ var FieldCollection = class {
   fields;
   inlineFragments;
   fragmentSpreads;
-  ignoreMaskDisable;
-  keepFragmentSpreadNodes;
-  hoistFragments;
+  applyFragments;
   constructor(args) {
     this.config = args.config;
     this.fragmentDefinitions = args.fragmentDefinitions;
-    this.ignoreMaskDisable = args.ignoreMaskDisable;
-    this.keepFragmentSpreadNodes = args.keepFragmentSpreadNodes;
+    this.applyFragments = args.applyFragments;
     this.fields = {};
     this.inlineFragments = {};
     this.fragmentSpreads = {};
-    this.hoistFragments = !!args.hoistFragments;
     this.filepath = args.filepath;
     for (const selection of args.selections) {
       this.add({ selection });
@@ -70516,21 +70566,18 @@ var FieldCollection = class {
     return Object.keys(this.fields).length + Object.keys(this.inlineFragments).length + Object.keys(this.fragmentSpreads).length;
   }
   add({ selection, external }) {
-    let includeFragments = this.config.defaultFragmentMasking === "disable";
+    let include = this.config.defaultFragmentMasking === "disable";
     const maskEnableDirective = selection.directives?.find(
       ({ name }) => name.value === this.config.maskEnableDirective
     );
     if (maskEnableDirective) {
-      includeFragments = false;
+      include = false;
     }
     const maskDisableDirective = selection.directives?.find(
       ({ name }) => name.value === this.config.maskDisableDirective
     );
     if (maskDisableDirective) {
-      includeFragments = true;
-    }
-    if (this.ignoreMaskDisable) {
-      includeFragments = true;
+      include = true;
     }
     if (selection.kind === "Field") {
       const key = selection.alias?.value || selection.name.value;
@@ -70546,7 +70593,7 @@ var FieldCollection = class {
           external
         });
       }
-      if (!external && includeFragments) {
+      if (this.applyFragments && !external) {
         this.fields[key].selection.fragmentSpreads = {
           ...this.collectFragmentSpreads(selection.selectionSet?.selections ?? []),
           ...this.fields[key].selection.fragmentSpreads
@@ -70560,16 +70607,13 @@ var FieldCollection = class {
       }
     }
     if (selection.kind === "InlineFragment" && selection.typeCondition) {
-      this.walkInlineFragment({ selection, external, hoistFragments: this.hoistFragments });
+      this.walkInlineFragment({ selection, external });
       return;
     }
     if (selection.kind === "FragmentSpread") {
-      if (this.keepFragmentSpreadNodes && !external) {
+      if (!external || include) {
         this.fragmentSpreads[selection.name.value] = selection;
       }
-      if (!includeFragments) {
-        return;
-      }
       const definition = this.fragmentDefinitions[selection.name.value];
       if (!definition) {
         throw new HoudiniError({
@@ -70577,23 +70621,25 @@ var FieldCollection = class {
           message: "Could not find referenced fragment definition: " + selection.name.value + "\n" + JSON.stringify(Object.keys(this.fragmentDefinitions), null, 4)
         });
       }
-      this.add({
-        selection: {
-          kind: "InlineFragment",
-          typeCondition: {
-            kind: "NamedType",
-            name: {
-              kind: "Name",
-              value: definition.typeCondition.name.value
+      if (this.applyFragments || include) {
+        this.add({
+          selection: {
+            kind: "InlineFragment",
+            typeCondition: {
+              kind: "NamedType",
+              name: {
+                kind: "Name",
+                value: definition.typeCondition.name.value
+              }
+            },
+            selectionSet: {
+              kind: "SelectionSet",
+              selections: [...definition.selectionSet.selections]
             }
           },
-          selectionSet: {
-            kind: "SelectionSet",
-            selections: [...definition.selectionSet.selections]
-          }
-        },
-        external
-      });
+          external: !include
+        });
+      }
     }
   }
   collectFragmentSpreads(selections, result = {}) {
@@ -70650,8 +70696,7 @@ var FieldCollection = class {
   }
   walkInlineFragment({
     selection,
-    external,
-    hoistFragments
+    external
   }) {
     const key = selection.typeCondition.name.value;
     if (!this.inlineFragments[key]) {
@@ -70661,7 +70706,7 @@ var FieldCollection = class {
       };
     }
     for (const subselect of selection.selectionSet.selections || []) {
-      if (subselect.kind === "FragmentSpread" && !hoistFragments || subselect.kind === "Field" || subselect.kind === "InlineFragment" && !subselect.typeCondition) {
+      if (subselect.kind === "Field" || subselect.kind === "InlineFragment" && !subselect.typeCondition) {
         this.inlineFragments[key].selection.add({
           selection: subselect,
           external
@@ -70670,11 +70715,11 @@ var FieldCollection = class {
       } else if (subselect.kind === "FragmentSpread") {
         this.add({
           selection: subselect,
-          external: true
+          external
         });
         continue;
       } else {
-        this.walkInlineFragment({ selection: subselect, external, hoistFragments });
+        this.walkInlineFragment({ selection: subselect, external });
       }
     }
   }
@@ -70684,9 +70729,7 @@ var FieldCollection = class {
       fragmentDefinitions: this.fragmentDefinitions,
       selections: [],
       filepath: this.filepath,
-      ignoreMaskDisable: this.ignoreMaskDisable,
-      keepFragmentSpreadNodes: this.keepFragmentSpreadNodes,
-      hoistFragments: this.hoistFragments
+      applyFragments: this.applyFragments
     });
   }
 };
@@ -71615,7 +71658,7 @@ function operationObject({
   if (operationKind === "insert" || operationKind === "toggle") {
     operation.position = position;
   }
-  if (operationKind === "insert" && allLists) {
+  if (allLists && operationKind !== "delete") {
     operation.target = "all";
   }
   if (parentID) {
@@ -72612,7 +72655,8 @@ function prepareSelection({
   inConnection,
   typeMap,
   abstractTypes,
-  globalLoading
+  globalLoading,
+  includeFragments
 }) {
   let object = {};
   const loadingTypes = [];
@@ -72632,7 +72676,8 @@ function prepareSelection({
             document,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields || {}
         );
       } else {
@@ -72680,7 +72725,8 @@ function prepareSelection({
             document,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields
         };
         if (field.directives?.find((d) => d.name.value === config4.loadingDirective)) {
@@ -72798,7 +72844,8 @@ function prepareSelection({
           inConnection: connectionState,
           typeMap,
           abstractTypes,
-          globalLoading: forceLoading
+          globalLoading: forceLoading,
+          includeFragments
         });
         if (Object.values(fieldObj.selection.fields ?? {}).some((field2) => field2.required)) {
           fieldObj.nullable = true;
@@ -73131,14 +73178,13 @@ function artifactGenerator(stats) {
               document: doc,
               rootType,
               globalLoading,
+              includeFragments: doc.kind !== ArtifactKind.Fragment,
               selections: flattenSelections({
                 config: config4,
                 filepath: doc.filename,
                 selections: selectionSet.selections,
                 fragmentDefinitions,
-                ignoreMaskDisable: docKind !== "HoudiniFragment",
-                keepFragmentSpreadNodes: true,
-                hoistFragments: true
+                applyFragments: doc.kind !== ArtifactKind.Fragment
               }),
               operations: operationsByPath(
                 config4,
@@ -74103,8 +74149,7 @@ async function generateDocumentTypes(config4, docs) {
         config: config4,
         filepath: filename,
         selections: definition.selectionSet.selections,
-        fragmentDefinitions,
-        keepFragmentSpreadNodes: true
+        fragmentDefinitions
       });
       if (definition?.kind === "OperationDefinition") {
         await generateOperationTypeDefs(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "houdini",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "The disappearing GraphQL clients",
   "keywords": [
     "typescript",