@graphql-codegen/visitor-plugin-common 5.7.1-alpha-20250221120941-a01a9a425c7b5284592b7e1cfdf2cb663fa9e736 → 5.7.1-alpha-20250221122145-5ef68e775c396313780244f87714d33191a46522

package/cjs/selection-set-to-object.js

@@ -411,9 +411,13 @@ class SelectionSetToObject {
                 }
                 continue;
             }
-            // 2. Handle Fragment selection nodes
-            // 2a. If `inlineFragmentTypes` is 'combine' or 'mask', the fragment is masked i.e. do not generate inline types
-            // In some cases like Apollo `@unmask`, the fragment would be generated inline.
+            // 2. Handle Fragment Spread nodes
+            // A Fragment Spread can be:
+            // - masked: the fields declared in the Fragment do not appear in the operation types
+            // - inline: the fields declared in the Fragment appear in the operation types
+            // 2a. If `inlineFragmentTypes` is 'combine' or 'mask', the Fragment Spread is masked by default
+            // In some cases, a masked node could be unmasked (i.e. treated as inline):
+            // - Fragment spread node is marked with Apollo `@unmask`, e.g. `...User @unmask`
             if (this._config.inlineFragmentTypes === 'combine' || this._config.inlineFragmentTypes === 'mask') {
                 let isMasked = true;
                 if (this._config.customDirectives.apolloUnmask &&
@@ -425,7 +429,7 @@ class SelectionSetToObject {
                     continue;
                 }
             }
-            // 2b. If the Fragment is not masked, handle Fragment Spreads by generating inline types.
+            // 2b. If the Fragment Spread is not masked, generate inline types.
             const fragmentType = this._schema.getType(selectionNode.onType);
             if (fragmentType == null) {
                 throw new TypeError(`Unexpected error: Type ${selectionNode.onType} does not exist within schema.`);

package/esm/selection-set-to-object.js

@@ -407,9 +407,13 @@ export class SelectionSetToObject {
                 }
                 continue;
             }
-            // 2. Handle Fragment selection nodes
-            // 2a. If `inlineFragmentTypes` is 'combine' or 'mask', the fragment is masked i.e. do not generate inline types
-            // In some cases like Apollo `@unmask`, the fragment would be generated inline.
+            // 2. Handle Fragment Spread nodes
+            // A Fragment Spread can be:
+            // - masked: the fields declared in the Fragment do not appear in the operation types
+            // - inline: the fields declared in the Fragment appear in the operation types
+            // 2a. If `inlineFragmentTypes` is 'combine' or 'mask', the Fragment Spread is masked by default
+            // In some cases, a masked node could be unmasked (i.e. treated as inline):
+            // - Fragment spread node is marked with Apollo `@unmask`, e.g. `...User @unmask`
             if (this._config.inlineFragmentTypes === 'combine' || this._config.inlineFragmentTypes === 'mask') {
                 let isMasked = true;
                 if (this._config.customDirectives.apolloUnmask &&
@@ -421,7 +425,7 @@ export class SelectionSetToObject {
                     continue;
                 }
             }
-            // 2b. If the Fragment is not masked, handle Fragment Spreads by generating inline types.
+            // 2b. If the Fragment Spread is not masked, generate inline types.
             const fragmentType = this._schema.getType(selectionNode.onType);
             if (fragmentType == null) {
                 throw new TypeError(`Unexpected error: Type ${selectionNode.onType} does not exist within schema.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-codegen/visitor-plugin-common",
-  "version": "5.7.1-alpha-20250221120941-a01a9a425c7b5284592b7e1cfdf2cb663fa9e736",
+  "version": "5.7.1-alpha-20250221122145-5ef68e775c396313780244f87714d33191a46522",
   "peerDependencies": {
     "graphql": "^0.8.0 || ^0.9.0 || ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0 || ^16.0.0"
   },