@nmarks/graphql-codegen-per-operation-file-preset 1.0.7 → 1.0.8

package/cjs/index.js

@@ -337,21 +337,6 @@ exports.preset = {
                     ...updatedFragmentImports,
                     ...localFragmentImports,
                 ];
-                /**
-                 * ADD EXTERNAL FRAGMENTS TO DOCUMENT
-                 *
-                 * The plugin needs external fragment definitions in the document to properly resolve types.
-                 * Without these, fragment spreads can't be expanded correctly.
-                 */
-                const singleDefDocumentWithFragments = {
-                    ...singleDefDocument,
-                    definitions: [
-                        ...singleDefDocument.definitions,
-                        ...allExternalFragments.map(fragment => fragment.node),
-                    ],
-                };
-                // Update the source to use the document with external fragments
-                singleDefSource.document = singleDefDocumentWithFragments;
                 /**
                  * CHECK IF TYPES IMPORT IS NEEDED
                  *
@@ -362,7 +347,17 @@ exports.preset = {
                  * - Operations (all operations generate Variables type using Types.Exact)
                  *
                  * We DON'T import for just object type references (e.g., fragment on User)
+                 *
+                 * Note: We check against a document WITH external fragments for type detection,
+                 * but we DON'T pass this to the plugin (that would create duplicate definitions).
                  */
+                const singleDefDocumentWithFragments = {
+                    ...singleDefDocument,
+                    definitions: [
+                        ...singleDefDocument.definitions,
+                        ...allExternalFragments.map(fragment => fragment.node),
+                    ],
+                };
                 const needsTypesImport = needsSchemaTypesImport(singleDefDocumentWithFragments, schemaObject, options.config);
                 // Generate the schema types import statement if needed
                 const importStatements = [];
@@ -401,17 +396,6 @@ exports.preset = {
                     externalFragments: allExternalFragments, // Includes both external + local fragments
                     fragmentImports: allFragmentImports, // Import declarations for all fragments
                 };
-                // DEBUG: Log fragment generation details
-                if (name === 'Items_item') {
-                    console.log('\n=== GENERATING Items_item ===');
-                    console.log('External fragments:', allExternalFragments.map(f => ({ name: f.name, level: f.level })));
-                    console.log('Fragment imports:', allFragmentImports.map(fi => fi.importSource.path));
-                    console.log('Document selections:', definition.selectionSet.selections.map((s) => s.kind === 'Field' ? s.name.value : `...${s.name.value}`));
-                    console.log('Document definitions (should be 1):', singleDefDocument.definitions.length);
-                    console.log('singleDefDocumentWithFragments definitions:', singleDefDocumentWithFragments.definitions.length);
-                    console.log('  - Main definition:', singleDefDocumentWithFragments.definitions[0].kind);
-                    console.log('  - External fragments added:', singleDefDocumentWithFragments.definitions.slice(1).map((d) => d.name.value));
-                }
                 /**
                  * CREATE GENERATION ARTIFACT
                  *

package/esm/index.js

@@ -333,21 +333,6 @@ export const preset = {
                     ...updatedFragmentImports,
                     ...localFragmentImports,
                 ];
-                /**
-                 * ADD EXTERNAL FRAGMENTS TO DOCUMENT
-                 *
-                 * The plugin needs external fragment definitions in the document to properly resolve types.
-                 * Without these, fragment spreads can't be expanded correctly.
-                 */
-                const singleDefDocumentWithFragments = {
-                    ...singleDefDocument,
-                    definitions: [
-                        ...singleDefDocument.definitions,
-                        ...allExternalFragments.map(fragment => fragment.node),
-                    ],
-                };
-                // Update the source to use the document with external fragments
-                singleDefSource.document = singleDefDocumentWithFragments;
                 /**
                  * CHECK IF TYPES IMPORT IS NEEDED
                  *
@@ -358,7 +343,17 @@ export const preset = {
                  * - Operations (all operations generate Variables type using Types.Exact)
                  *
                  * We DON'T import for just object type references (e.g., fragment on User)
+                 *
+                 * Note: We check against a document WITH external fragments for type detection,
+                 * but we DON'T pass this to the plugin (that would create duplicate definitions).
                  */
+                const singleDefDocumentWithFragments = {
+                    ...singleDefDocument,
+                    definitions: [
+                        ...singleDefDocument.definitions,
+                        ...allExternalFragments.map(fragment => fragment.node),
+                    ],
+                };
                 const needsTypesImport = needsSchemaTypesImport(singleDefDocumentWithFragments, schemaObject, options.config);
                 // Generate the schema types import statement if needed
                 const importStatements = [];
@@ -397,17 +392,6 @@ export const preset = {
                     externalFragments: allExternalFragments, // Includes both external + local fragments
                     fragmentImports: allFragmentImports, // Import declarations for all fragments
                 };
-                // DEBUG: Log fragment generation details
-                if (name === 'Items_item') {
-                    console.log('\n=== GENERATING Items_item ===');
-                    console.log('External fragments:', allExternalFragments.map(f => ({ name: f.name, level: f.level })));
-                    console.log('Fragment imports:', allFragmentImports.map(fi => fi.importSource.path));
-                    console.log('Document selections:', definition.selectionSet.selections.map((s) => s.kind === 'Field' ? s.name.value : `...${s.name.value}`));
-                    console.log('Document definitions (should be 1):', singleDefDocument.definitions.length);
-                    console.log('singleDefDocumentWithFragments definitions:', singleDefDocumentWithFragments.definitions.length);
-                    console.log('  - Main definition:', singleDefDocumentWithFragments.definitions[0].kind);
-                    console.log('  - External fragments added:', singleDefDocumentWithFragments.definitions.slice(1).map((d) => d.name.value));
-                }
                 /**
                  * CREATE GENERATION ARTIFACT
                  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nmarks/graphql-codegen-per-operation-file-preset",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "GraphQL Code Generator preset for generating one file per operation/fragment",
   "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"