houdini-svelte 1.2.1 → 1.2.2

package/build/test-cjs/index.js

@@ -149450,18 +149450,14 @@ function flattenSelections({
   filepath,
   selections,
   fragmentDefinitions,
-  ignoreMaskDisable,
-  keepFragmentSpreadNodes,
-  hoistFragments
+  applyFragments
 }) {
   const fields = new FieldCollection({
     config: config22,
     filepath,
     selections,
     fragmentDefinitions,
-    ignoreMaskDisable: !!ignoreMaskDisable,
-    keepFragmentSpreadNodes: !!keepFragmentSpreadNodes,
-    hoistFragments
+    applyFragments: !!applyFragments
   });
   return fields.toSelectionSet();
 }
@@ -149472,18 +149468,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 });
@@ -149493,21 +149485,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;
@@ -149523,7 +149512,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
@@ -149537,16 +149526,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 HoudiniError2({
@@ -149554,23 +149540,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 = {}) {
@@ -149627,8 +149615,7 @@ var FieldCollection = class {
   }
   walkInlineFragment({
     selection,
-    external,
-    hoistFragments
+    external
   }) {
     const key = selection.typeCondition.name.value;
     if (!this.inlineFragments[key]) {
@@ -149638,7 +149625,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
@@ -149647,11 +149634,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 });
       }
     }
   }
@@ -149661,9 +149648,7 @@ var FieldCollection = class {
       fragmentDefinitions: this.fragmentDefinitions,
       selections: [],
       filepath: this.filepath,
-      ignoreMaskDisable: this.ignoreMaskDisable,
-      keepFragmentSpreadNodes: this.keepFragmentSpreadNodes,
-      hoistFragments: this.hoistFragments
+      applyFragments: this.applyFragments
     });
   }
 };
@@ -151539,7 +151524,8 @@ function prepareSelection({
   inConnection,
   typeMap,
   abstractTypes,
-  globalLoading
+  globalLoading,
+  includeFragments
 }) {
   let object = {};
   const loadingTypes = [];
@@ -151559,7 +151545,8 @@ function prepareSelection({
             document: document2,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields || {}
         );
       } else {
@@ -151607,7 +151594,8 @@ function prepareSelection({
             document: document2,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields
         };
         if (field.directives?.find((d) => d.name.value === config22.loadingDirective)) {
@@ -151725,7 +151713,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;
@@ -152056,14 +152045,13 @@ function artifactGenerator(stats) {
               document: doc,
               rootType,
               globalLoading,
+              includeFragments: doc.kind !== ArtifactKind2.Fragment,
               selections: flattenSelections({
                 config: config22,
                 filepath: doc.filename,
                 selections: selectionSet.selections,
                 fragmentDefinitions,
-                ignoreMaskDisable: docKind !== "HoudiniFragment",
-                keepFragmentSpreadNodes: true,
-                hoistFragments: true
+                applyFragments: doc.kind !== ArtifactKind2.Fragment
               }),
               operations: operationsByPath(
                 config22,
@@ -153000,8 +152988,7 @@ async function generateDocumentTypes(config22, docs) {
         config: config22,
         filepath: filename,
         selections: definition.selectionSet.selections,
-        fragmentDefinitions,
-        keepFragmentSpreadNodes: true
+        fragmentDefinitions
       });
       if (definition?.kind === "OperationDefinition") {
         await generateOperationTypeDefs(

package/build/test-esm/index.js

@@ -149437,18 +149437,14 @@ function flattenSelections({
   filepath,
   selections,
   fragmentDefinitions,
-  ignoreMaskDisable,
-  keepFragmentSpreadNodes,
-  hoistFragments
+  applyFragments
 }) {
   const fields = new FieldCollection({
     config: config22,
     filepath,
     selections,
     fragmentDefinitions,
-    ignoreMaskDisable: !!ignoreMaskDisable,
-    keepFragmentSpreadNodes: !!keepFragmentSpreadNodes,
-    hoistFragments
+    applyFragments: !!applyFragments
   });
   return fields.toSelectionSet();
 }
@@ -149459,18 +149455,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 });
@@ -149480,21 +149472,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;
@@ -149510,7 +149499,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
@@ -149524,16 +149513,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 HoudiniError2({
@@ -149541,23 +149527,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 = {}) {
@@ -149614,8 +149602,7 @@ var FieldCollection = class {
   }
   walkInlineFragment({
     selection,
-    external,
-    hoistFragments
+    external
   }) {
     const key = selection.typeCondition.name.value;
     if (!this.inlineFragments[key]) {
@@ -149625,7 +149612,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
@@ -149634,11 +149621,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 });
       }
     }
   }
@@ -149648,9 +149635,7 @@ var FieldCollection = class {
       fragmentDefinitions: this.fragmentDefinitions,
       selections: [],
       filepath: this.filepath,
-      ignoreMaskDisable: this.ignoreMaskDisable,
-      keepFragmentSpreadNodes: this.keepFragmentSpreadNodes,
-      hoistFragments: this.hoistFragments
+      applyFragments: this.applyFragments
     });
   }
 };
@@ -151526,7 +151511,8 @@ function prepareSelection({
   inConnection,
   typeMap,
   abstractTypes,
-  globalLoading
+  globalLoading,
+  includeFragments
 }) {
   let object = {};
   const loadingTypes = [];
@@ -151546,7 +151532,8 @@ function prepareSelection({
             document: document2,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields || {}
         );
       } else {
@@ -151594,7 +151581,8 @@ function prepareSelection({
             document: document2,
             typeMap,
             abstractTypes,
-            globalLoading
+            globalLoading,
+            includeFragments
           }).fields
         };
         if (field.directives?.find((d) => d.name.value === config22.loadingDirective)) {
@@ -151712,7 +151700,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;
@@ -152043,14 +152032,13 @@ function artifactGenerator(stats) {
               document: doc,
               rootType,
               globalLoading,
+              includeFragments: doc.kind !== ArtifactKind2.Fragment,
               selections: flattenSelections({
                 config: config22,
                 filepath: doc.filename,
                 selections: selectionSet.selections,
                 fragmentDefinitions,
-                ignoreMaskDisable: docKind !== "HoudiniFragment",
-                keepFragmentSpreadNodes: true,
-                hoistFragments: true
+                applyFragments: doc.kind !== ArtifactKind2.Fragment
               }),
               operations: operationsByPath(
                 config22,
@@ -152987,8 +152975,7 @@ async function generateDocumentTypes(config22, docs) {
         config: config22,
         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-svelte",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "The svelte plugin for houdini",
   "keywords": [
     "typescript",
@@ -32,7 +32,7 @@
     "rollup": "^3.7.4",
     "svelte": "^3.57.0",
     "vite": "^4.1.1",
-    "houdini": "^1.2.1"
+    "houdini": "^1.2.2"
   },
   "files": [
     "build"