@graphql-codegen/visitor-plugin-common 6.0.0-alpha-20250816104654-3cefeabda1bcfa915d43b4515cb86dd2dfb9df5b → 6.0.0-alpha-20250827103001-1f42d2b3d7d1c009e2fc2ed9b513fdda20e78b50

package/cjs/base-resolvers-visitor.js

@@ -60,7 +60,7 @@ class BaseResolversVisitor extends base_visitor_js_1.BaseVisitor {
             directiveContextTypes: (0, utils_js_1.getConfigValue)(rawConfig.directiveContextTypes, []),
             resolverTypeSuffix: (0, utils_js_1.getConfigValue)(rawConfig.resolverTypeSuffix, 'Resolvers'),
             allResolversTypeName: (0, utils_js_1.getConfigValue)(rawConfig.allResolversTypeName, 'Resolvers'),
-            rootValueType: (0, mappers_js_1.parseMapper)(rawConfig.rootValueType || '{}', 'RootValueType'),
+            rootValueType: (0, mappers_js_1.parseMapper)(rawConfig.rootValueType || 'Record<PropertyKey, never>', 'RootValueType'),
             namespacedImportName: (0, utils_js_1.getConfigValue)(rawConfig.namespacedImportName, ''),
             avoidOptionals: (0, avoid_optionals_js_1.normalizeAvoidOptionals)(rawConfig.avoidOptionals),
             defaultMapper: rawConfig.defaultMapper

package/cjs/base-types-visitor.js

@@ -129,7 +129,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.input)
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withBlock(node.fields.join('\n'));
     }
     getInputObjectOneOfDeclarationBlock(node) {
@@ -140,14 +140,13 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind(declarationKind)
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withContent(`\n` + node.fields.join('\n  |'));
     }
     InputObjectTypeDefinition(node) {
         if (this.config.onlyEnums)
             return '';
-        // Why the heck is node.name a string and not { value: string } at runtime ?!
-        if ((0, utils_js_1.isOneOfInputObjectType)(this._schema.getType(node.name))) {
+        if ((0, utils_js_1.isOneOfInputObjectType)(this._schema.getType(node.name.value))) {
             return this.getInputObjectOneOfDeclarationBlock(node).string;
         }
         return this.getInputObjectDeclarationBlock(node).string;
@@ -155,16 +154,13 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
     InputValueDefinition(node) {
         if (this.config.onlyEnums)
             return '';
-        const comment = (0, utils_js_1.transformComment)(node.description, 1);
+        const comment = (0, utils_js_1.transformComment)(node.description.value, 1);
         const { input } = this._parsedConfig.declarationKind;
         let type = node.type;
         if (node.directives && this.config.directiveArgumentAndInputFieldMappings) {
             type = this._getDirectiveOverrideType(node.directives) || type;
         }
-        return comment + (0, utils_js_1.indent)(`${node.name}: ${type}${this.getPunctuation(input)}`);
-    }
-    Name(node) {
-        return node.value;
+        return comment + (0, utils_js_1.indent)(`${node.name.value}: ${type}${this.getPunctuation(input)}`);
     }
     FieldDefinition(node) {
         if (this.config.onlyEnums)
@@ -172,7 +168,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
         const typeString = node.type;
         const { type } = this._parsedConfig.declarationKind;
         const comment = this.getNodeComment(node);
-        return comment + (0, utils_js_1.indent)(`${node.name}: ${typeString}${this.getPunctuation(type)}`);
+        return comment + (0, utils_js_1.indent)(`${node.name.value}: ${typeString}${this.getPunctuation(type)}`);
     }
     UnionTypeDefinition(node, key, parent) {
         if (this.config.onlyOperationTypes || this.config.onlyEnums)
@@ -185,7 +181,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind('type')
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description.value)
             .withContent(possibleTypes).string;
     }
     mergeInterfaces(interfaces, hasOtherFields) {
@@ -201,7 +197,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
         const allFields = [
             ...(this.config.addTypename
                 ? [
-                    (0, utils_js_1.indent)(`${this.config.immutableTypes ? 'readonly ' : ''}${optionalTypename}: '${node.name}'${this.getPunctuation(type)}`),
+                    (0, utils_js_1.indent)(`${this.config.immutableTypes ? 'readonly ' : ''}${optionalTypename}: '${node.name.value}'${this.getPunctuation(type)}`),
                 ]
                 : []),
             ...node.fields,
@@ -211,7 +207,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind(type)
             .withName(this.convertName(node))
-            .withComment(node.description);
+            .withComment(node.description?.value);
         if (type === 'interface' || type === 'class') {
             if (interfacesNames.length > 0) {
                 const keyword = interfacesType === 'interface' && type === 'class' ? 'implements' : 'extends';
@@ -240,7 +236,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.interface)
             .withName(this.convertName(node))
-            .withComment(node.description);
+            .withComment(node.description?.value);
         return declarationBlock.withBlock(node.fields.join('\n'));
     }
     InterfaceTypeDefinition(node, key, parent) {
@@ -294,7 +290,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .filter(Boolean);
     }
     EnumTypeDefinition(node) {
-        const enumName = node.name;
+        const enumName = node.name.value;
         // In case of mapped external enum string
         if (this.config.enumValues[enumName]?.sourceFile) {
             return null;
@@ -306,13 +302,9 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             useTypesPrefix: this.config.enumPrefix,
             useTypesSuffix: this.config.enumSuffix,
         }))
-            .withComment(node.description)
+            .withComment(node.description.value)
             .withBlock(this.buildEnumValuesBlock(enumName, node.values)).string;
     }
-    // We are using it in order to transform "description" field
-    StringValue(node) {
-        return node.value;
-    }
     makeValidEnumIdentifier(identifier) {
         if (/^[0-9]/.exec(identifier)) {
             return (0, utils_js_1.wrapWithSingleQuotes)(identifier, true);
@@ -331,9 +323,9 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             }));
             const comment = this.getNodeComment(enumOption);
             const schemaEnumValue = schemaEnumType && !this.config.ignoreEnumValuesFromSchema
-                ? schemaEnumType.getValue(enumOption.name).value
+                ? schemaEnumType.getValue(enumOption.name.value).value
                 : undefined;
-            let enumValue = typeof schemaEnumValue === 'undefined' ? enumOption.name : schemaEnumValue;
+            let enumValue = typeof schemaEnumValue === 'undefined' ? enumOption.name.value : schemaEnumValue;
             if (typeof this.config.enumValues[typeName]?.mappedValues?.[enumValue] !== 'undefined') {
                 enumValue = this.config.enumValues[typeName].mappedValues[enumValue];
             }
@@ -350,7 +342,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.arguments)
             .withName(this.convertName(name))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withBlock(this._argumentsTransformer.transform(field.arguments));
     }
     getArgumentsObjectTypeDefinition(node, name, field) {
@@ -382,7 +374,7 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
     _getDirectiveOverrideType(directives) {
         const type = directives
             .map(directive => {
-            const directiveName = directive.name;
+            const directiveName = directive.name.value;
             if (this.config.directiveArgumentAndInputFieldMappings[directiveName]) {
                 return this._getDirectiveArgumentNadInputFieldMapping(directiveName);
             }
@@ -393,14 +385,14 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
         return type || null;
     }
     _getTypeForNode(node, isVisitingInputType) {
-        const typeAsString = node.name;
+        const typeAsString = node.name.value;
         if (this.scalars[typeAsString]) {
             return this._getScalar(typeAsString, isVisitingInputType ? 'input' : 'output');
         }
         if (this.config.enumValues[typeAsString]) {
             return this.config.enumValues[typeAsString].typeIdentifier;
         }
-        const schemaType = this._schema.getType(node.name);
+        const schemaType = this._schema.getType(typeAsString);
         if (schemaType && (0, graphql_1.isEnumType)(schemaType)) {
             return this.convertName(node, {
                 useTypesPrefix: this.config.enumPrefix,
@@ -426,8 +418,8 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
         return null;
     }
     getNodeComment(node) {
-        let commentText = node.description;
-        const deprecationDirective = node.directives.find((v) => v.name === 'deprecated');
+        let commentText = node.description?.value;
+        const deprecationDirective = node.directives.find(v => v.name.value === 'deprecated');
         if (deprecationDirective) {
             const deprecationReason = this.getDeprecationReason(deprecationDirective);
             commentText = `${commentText ? `${commentText}\n` : ''}@deprecated ${deprecationReason}`;
@@ -436,11 +428,11 @@ class BaseTypesVisitor extends base_visitor_js_1.BaseVisitor {
         return comment;
     }
     getDeprecationReason(directive) {
-        if (directive.name === 'deprecated') {
-            const hasArguments = directive.arguments.length > 0;
+        if (directive.name.value === 'deprecated') {
             let reason = 'Field no longer supported';
-            if (hasArguments) {
-                reason = directive.arguments[0].value;
+            const deprecatedReason = directive.arguments[0];
+            if (deprecatedReason && deprecatedReason.value.kind === graphql_1.Kind.STRING) {
+                reason = deprecatedReason.value.value;
             }
             return reason;
         }

package/cjs/base-visitor.js

@@ -18,7 +18,6 @@ class BaseVisitor {
             addTypename: !rawConfig.skipTypename,
             nonOptionalTypename: !!rawConfig.nonOptionalTypename,
             useTypeImports: !!rawConfig.useTypeImports,
-            dedupeFragments: !!rawConfig.dedupeFragments,
             allowEnumStringTypes: !!rawConfig.allowEnumStringTypes,
             inlineFragmentTypes: rawConfig.inlineFragmentTypes ?? 'inline',
             emitLegacyCommonJSImports: rawConfig.emitLegacyCommonJSImports === undefined ? true : !!rawConfig.emitLegacyCommonJSImports,

package/cjs/client-side-base-visitor.js

@@ -48,12 +48,7 @@ class ClientSideBaseVisitor extends base_visitor_js_1.BaseVisitor {
             documentVariableSuffix: (0, utils_js_1.getConfigValue)(rawConfig.documentVariableSuffix, 'Document'),
             fragmentVariablePrefix: (0, utils_js_1.getConfigValue)(rawConfig.fragmentVariablePrefix, ''),
             fragmentVariableSuffix: (0, utils_js_1.getConfigValue)(rawConfig.fragmentVariableSuffix, 'FragmentDoc'),
-            documentMode: ((rawConfig) => {
-                if (typeof rawConfig.noGraphQLTag === 'boolean') {
-                    return rawConfig.noGraphQLTag ? DocumentMode.documentNode : DocumentMode.graphQLTag;
-                }
-                return (0, utils_js_1.getConfigValue)(rawConfig.documentMode, DocumentMode.graphQLTag);
-            })(rawConfig),
+            documentMode: (0, utils_js_1.getConfigValue)(rawConfig.documentMode, DocumentMode.graphQLTag),
             importDocumentNodeExternallyFrom: (0, utils_js_1.getConfigValue)(rawConfig.importDocumentNodeExternallyFrom, ''),
             pureMagicComment: (0, utils_js_1.getConfigValue)(rawConfig.pureMagicComment, false),
             experimentalFragmentVariables: (0, utils_js_1.getConfigValue)(rawConfig.experimentalFragmentVariables, false),
@@ -95,7 +90,7 @@ class ClientSideBaseVisitor extends base_visitor_js_1.BaseVisitor {
     _transformFragments(fragmentNames) {
         return fragmentNames.map(document => this.getFragmentVariableName(document));
     }
-    _includeFragments(fragments, nodeKind) {
+    _includeFragments(fragments) {
         if (fragments && fragments.length > 0) {
             if (this.config.documentMode === DocumentMode.documentNode || this.config.documentMode === DocumentMode.string) {
                 return Array.from(this._fragments.values())
@@ -106,9 +101,6 @@ class ClientSideBaseVisitor extends base_visitor_js_1.BaseVisitor {
             if (this.config.documentMode === DocumentMode.documentNodeImportFragments) {
                 return '';
             }
-            if (this.config.dedupeFragments && nodeKind !== 'OperationDefinition') {
-                return '';
-            }
             return String(fragments.map(name => '${' + name + '}').join('\n'));
         }
         return '';
@@ -117,14 +109,12 @@ class ClientSideBaseVisitor extends base_visitor_js_1.BaseVisitor {
         return documentStr;
     }
     _gql(node) {
-        const includeNestedFragments = this.config.documentMode === DocumentMode.documentNode ||
-            this.config.documentMode === DocumentMode.string ||
-            (this.config.dedupeFragments && node.kind === 'OperationDefinition');
+        const includeNestedFragments = this.config.documentMode === DocumentMode.documentNode || this.config.documentMode === DocumentMode.string;
         const fragmentNames = this._extractFragments(node, includeNestedFragments);
         const fragments = this._transformFragments(fragmentNames);
         const doc = this._prepareDocument(`
     ${(0, graphql_1.print)(node).split('\\').join('\\\\') /* Re-escape escaped values in GraphQL syntax */}
-    ${this._includeFragments(fragments, node.kind)}`);
+    ${this._includeFragments(fragments)}`);
         if (this.config.documentMode === DocumentMode.documentNode) {
             let gqlObj = (0, graphql_tag_1.default)([doc]);
             if (this.config.optimizeDocumentNode) {

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

@@ -534,7 +534,7 @@ class SelectionSetToObject {
         return 'never';
     }
     getEmptyObjectType() {
-        return `{}`;
+        return 'Record<PropertyKey, never>';
     }
     getEmptyObjectTypeString(mustAddEmptyObject) {
         return mustAddEmptyObject ? this.getEmptyObjectType() : ``;

package/esm/base-resolvers-visitor.js

@@ -56,7 +56,7 @@ export class BaseResolversVisitor extends BaseVisitor {
             directiveContextTypes: getConfigValue(rawConfig.directiveContextTypes, []),
             resolverTypeSuffix: getConfigValue(rawConfig.resolverTypeSuffix, 'Resolvers'),
             allResolversTypeName: getConfigValue(rawConfig.allResolversTypeName, 'Resolvers'),
-            rootValueType: parseMapper(rawConfig.rootValueType || '{}', 'RootValueType'),
+            rootValueType: parseMapper(rawConfig.rootValueType || 'Record<PropertyKey, never>', 'RootValueType'),
             namespacedImportName: getConfigValue(rawConfig.namespacedImportName, ''),
             avoidOptionals: normalizeAvoidOptionals(rawConfig.avoidOptionals),
             defaultMapper: rawConfig.defaultMapper

package/esm/base-types-visitor.js

@@ -126,7 +126,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.input)
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withBlock(node.fields.join('\n'));
     }
     getInputObjectOneOfDeclarationBlock(node) {
@@ -137,14 +137,13 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind(declarationKind)
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withContent(`\n` + node.fields.join('\n  |'));
     }
     InputObjectTypeDefinition(node) {
         if (this.config.onlyEnums)
             return '';
-        // Why the heck is node.name a string and not { value: string } at runtime ?!
-        if (isOneOfInputObjectType(this._schema.getType(node.name))) {
+        if (isOneOfInputObjectType(this._schema.getType(node.name.value))) {
             return this.getInputObjectOneOfDeclarationBlock(node).string;
         }
         return this.getInputObjectDeclarationBlock(node).string;
@@ -152,16 +151,13 @@ export class BaseTypesVisitor extends BaseVisitor {
     InputValueDefinition(node) {
         if (this.config.onlyEnums)
             return '';
-        const comment = transformComment(node.description, 1);
+        const comment = transformComment(node.description.value, 1);
         const { input } = this._parsedConfig.declarationKind;
         let type = node.type;
         if (node.directives && this.config.directiveArgumentAndInputFieldMappings) {
             type = this._getDirectiveOverrideType(node.directives) || type;
         }
-        return comment + indent(`${node.name}: ${type}${this.getPunctuation(input)}`);
-    }
-    Name(node) {
-        return node.value;
+        return comment + indent(`${node.name.value}: ${type}${this.getPunctuation(input)}`);
     }
     FieldDefinition(node) {
         if (this.config.onlyEnums)
@@ -169,7 +165,7 @@ export class BaseTypesVisitor extends BaseVisitor {
         const typeString = node.type;
         const { type } = this._parsedConfig.declarationKind;
         const comment = this.getNodeComment(node);
-        return comment + indent(`${node.name}: ${typeString}${this.getPunctuation(type)}`);
+        return comment + indent(`${node.name.value}: ${typeString}${this.getPunctuation(type)}`);
     }
     UnionTypeDefinition(node, key, parent) {
         if (this.config.onlyOperationTypes || this.config.onlyEnums)
@@ -182,7 +178,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind('type')
             .withName(this.convertName(node))
-            .withComment(node.description)
+            .withComment(node.description.value)
             .withContent(possibleTypes).string;
     }
     mergeInterfaces(interfaces, hasOtherFields) {
@@ -198,7 +194,7 @@ export class BaseTypesVisitor extends BaseVisitor {
         const allFields = [
             ...(this.config.addTypename
                 ? [
-                    indent(`${this.config.immutableTypes ? 'readonly ' : ''}${optionalTypename}: '${node.name}'${this.getPunctuation(type)}`),
+                    indent(`${this.config.immutableTypes ? 'readonly ' : ''}${optionalTypename}: '${node.name.value}'${this.getPunctuation(type)}`),
                 ]
                 : []),
             ...node.fields,
@@ -208,7 +204,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind(type)
             .withName(this.convertName(node))
-            .withComment(node.description);
+            .withComment(node.description?.value);
         if (type === 'interface' || type === 'class') {
             if (interfacesNames.length > 0) {
                 const keyword = interfacesType === 'interface' && type === 'class' ? 'implements' : 'extends';
@@ -237,7 +233,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.interface)
             .withName(this.convertName(node))
-            .withComment(node.description);
+            .withComment(node.description?.value);
         return declarationBlock.withBlock(node.fields.join('\n'));
     }
     InterfaceTypeDefinition(node, key, parent) {
@@ -291,7 +287,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .filter(Boolean);
     }
     EnumTypeDefinition(node) {
-        const enumName = node.name;
+        const enumName = node.name.value;
         // In case of mapped external enum string
         if (this.config.enumValues[enumName]?.sourceFile) {
             return null;
@@ -303,13 +299,9 @@ export class BaseTypesVisitor extends BaseVisitor {
             useTypesPrefix: this.config.enumPrefix,
             useTypesSuffix: this.config.enumSuffix,
         }))
-            .withComment(node.description)
+            .withComment(node.description.value)
             .withBlock(this.buildEnumValuesBlock(enumName, node.values)).string;
     }
-    // We are using it in order to transform "description" field
-    StringValue(node) {
-        return node.value;
-    }
     makeValidEnumIdentifier(identifier) {
         if (/^[0-9]/.exec(identifier)) {
             return wrapWithSingleQuotes(identifier, true);
@@ -328,9 +320,9 @@ export class BaseTypesVisitor extends BaseVisitor {
             }));
             const comment = this.getNodeComment(enumOption);
             const schemaEnumValue = schemaEnumType && !this.config.ignoreEnumValuesFromSchema
-                ? schemaEnumType.getValue(enumOption.name).value
+                ? schemaEnumType.getValue(enumOption.name.value).value
                 : undefined;
-            let enumValue = typeof schemaEnumValue === 'undefined' ? enumOption.name : schemaEnumValue;
+            let enumValue = typeof schemaEnumValue === 'undefined' ? enumOption.name.value : schemaEnumValue;
             if (typeof this.config.enumValues[typeName]?.mappedValues?.[enumValue] !== 'undefined') {
                 enumValue = this.config.enumValues[typeName].mappedValues[enumValue];
             }
@@ -347,7 +339,7 @@ export class BaseTypesVisitor extends BaseVisitor {
             .export()
             .asKind(this._parsedConfig.declarationKind.arguments)
             .withName(this.convertName(name))
-            .withComment(node.description)
+            .withComment(node.description?.value)
             .withBlock(this._argumentsTransformer.transform(field.arguments));
     }
     getArgumentsObjectTypeDefinition(node, name, field) {
@@ -379,7 +371,7 @@ export class BaseTypesVisitor extends BaseVisitor {
     _getDirectiveOverrideType(directives) {
         const type = directives
             .map(directive => {
-            const directiveName = directive.name;
+            const directiveName = directive.name.value;
             if (this.config.directiveArgumentAndInputFieldMappings[directiveName]) {
                 return this._getDirectiveArgumentNadInputFieldMapping(directiveName);
             }
@@ -390,14 +382,14 @@ export class BaseTypesVisitor extends BaseVisitor {
         return type || null;
     }
     _getTypeForNode(node, isVisitingInputType) {
-        const typeAsString = node.name;
+        const typeAsString = node.name.value;
         if (this.scalars[typeAsString]) {
             return this._getScalar(typeAsString, isVisitingInputType ? 'input' : 'output');
         }
         if (this.config.enumValues[typeAsString]) {
             return this.config.enumValues[typeAsString].typeIdentifier;
         }
-        const schemaType = this._schema.getType(node.name);
+        const schemaType = this._schema.getType(typeAsString);
         if (schemaType && isEnumType(schemaType)) {
             return this.convertName(node, {
                 useTypesPrefix: this.config.enumPrefix,
@@ -423,8 +415,8 @@ export class BaseTypesVisitor extends BaseVisitor {
         return null;
     }
     getNodeComment(node) {
-        let commentText = node.description;
-        const deprecationDirective = node.directives.find((v) => v.name === 'deprecated');
+        let commentText = node.description?.value;
+        const deprecationDirective = node.directives.find(v => v.name.value === 'deprecated');
         if (deprecationDirective) {
             const deprecationReason = this.getDeprecationReason(deprecationDirective);
             commentText = `${commentText ? `${commentText}\n` : ''}@deprecated ${deprecationReason}`;
@@ -433,11 +425,11 @@ export class BaseTypesVisitor extends BaseVisitor {
         return comment;
     }
     getDeprecationReason(directive) {
-        if (directive.name === 'deprecated') {
-            const hasArguments = directive.arguments.length > 0;
+        if (directive.name.value === 'deprecated') {
             let reason = 'Field no longer supported';
-            if (hasArguments) {
-                reason = directive.arguments[0].value;
+            const deprecatedReason = directive.arguments[0];
+            if (deprecatedReason && deprecatedReason.value.kind === Kind.STRING) {
+                reason = deprecatedReason.value.value;
             }
             return reason;
         }

package/esm/base-visitor.js

@@ -14,7 +14,6 @@ export class BaseVisitor {
             addTypename: !rawConfig.skipTypename,
             nonOptionalTypename: !!rawConfig.nonOptionalTypename,
             useTypeImports: !!rawConfig.useTypeImports,
-            dedupeFragments: !!rawConfig.dedupeFragments,
             allowEnumStringTypes: !!rawConfig.allowEnumStringTypes,
             inlineFragmentTypes: rawConfig.inlineFragmentTypes ?? 'inline',
             emitLegacyCommonJSImports: rawConfig.emitLegacyCommonJSImports === undefined ? true : !!rawConfig.emitLegacyCommonJSImports,

package/esm/client-side-base-visitor.js

@@ -44,12 +44,7 @@ export class ClientSideBaseVisitor extends BaseVisitor {
             documentVariableSuffix: getConfigValue(rawConfig.documentVariableSuffix, 'Document'),
             fragmentVariablePrefix: getConfigValue(rawConfig.fragmentVariablePrefix, ''),
             fragmentVariableSuffix: getConfigValue(rawConfig.fragmentVariableSuffix, 'FragmentDoc'),
-            documentMode: ((rawConfig) => {
-                if (typeof rawConfig.noGraphQLTag === 'boolean') {
-                    return rawConfig.noGraphQLTag ? DocumentMode.documentNode : DocumentMode.graphQLTag;
-                }
-                return getConfigValue(rawConfig.documentMode, DocumentMode.graphQLTag);
-            })(rawConfig),
+            documentMode: getConfigValue(rawConfig.documentMode, DocumentMode.graphQLTag),
             importDocumentNodeExternallyFrom: getConfigValue(rawConfig.importDocumentNodeExternallyFrom, ''),
             pureMagicComment: getConfigValue(rawConfig.pureMagicComment, false),
             experimentalFragmentVariables: getConfigValue(rawConfig.experimentalFragmentVariables, false),
@@ -91,7 +86,7 @@ export class ClientSideBaseVisitor extends BaseVisitor {
     _transformFragments(fragmentNames) {
         return fragmentNames.map(document => this.getFragmentVariableName(document));
     }
-    _includeFragments(fragments, nodeKind) {
+    _includeFragments(fragments) {
         if (fragments && fragments.length > 0) {
             if (this.config.documentMode === DocumentMode.documentNode || this.config.documentMode === DocumentMode.string) {
                 return Array.from(this._fragments.values())
@@ -102,9 +97,6 @@ export class ClientSideBaseVisitor extends BaseVisitor {
             if (this.config.documentMode === DocumentMode.documentNodeImportFragments) {
                 return '';
             }
-            if (this.config.dedupeFragments && nodeKind !== 'OperationDefinition') {
-                return '';
-            }
             return String(fragments.map(name => '${' + name + '}').join('\n'));
         }
         return '';
@@ -113,14 +105,12 @@ export class ClientSideBaseVisitor extends BaseVisitor {
         return documentStr;
     }
     _gql(node) {
-        const includeNestedFragments = this.config.documentMode === DocumentMode.documentNode ||
-            this.config.documentMode === DocumentMode.string ||
-            (this.config.dedupeFragments && node.kind === 'OperationDefinition');
+        const includeNestedFragments = this.config.documentMode === DocumentMode.documentNode || this.config.documentMode === DocumentMode.string;
         const fragmentNames = this._extractFragments(node, includeNestedFragments);
         const fragments = this._transformFragments(fragmentNames);
         const doc = this._prepareDocument(`
     ${print(node).split('\\').join('\\\\') /* Re-escape escaped values in GraphQL syntax */}
-    ${this._includeFragments(fragments, node.kind)}`);
+    ${this._includeFragments(fragments)}`);
         if (this.config.documentMode === DocumentMode.documentNode) {
             let gqlObj = gqlTag([doc]);
             if (this.config.optimizeDocumentNode) {

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

@@ -530,7 +530,7 @@ export class SelectionSetToObject {
         return 'never';
     }
     getEmptyObjectType() {
-        return `{}`;
+        return 'Record<PropertyKey, never>';
     }
     getEmptyObjectTypeString(mustAddEmptyObject) {
         return mustAddEmptyObject ? this.getEmptyObjectType() : ``;

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "@graphql-codegen/visitor-plugin-common",
-  "version": "6.0.0-alpha-20250816104654-3cefeabda1bcfa915d43b4515cb86dd2dfb9df5b",
+  "version": "6.0.0-alpha-20250827103001-1f42d2b3d7d1c009e2fc2ed9b513fdda20e78b50",
   "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"
   },
   "dependencies": {
     "@graphql-tools/optimize": "^2.0.0",
-    "@graphql-codegen/plugin-helpers": "6.0.0-alpha-20250816104654-3cefeabda1bcfa915d43b4515cb86dd2dfb9df5b",
+    "@graphql-codegen/plugin-helpers": "6.0.0-alpha-20250827103001-1f42d2b3d7d1c009e2fc2ed9b513fdda20e78b50",
     "@graphql-tools/relay-operation-optimizer": "^7.0.0",
     "@graphql-tools/utils": "^10.0.0",
-    "auto-bind": "~4.0.0",
+    "auto-bind": "^5.0.0",
     "dependency-graph": "^1.0.0",
     "graphql-tag": "^2.11.0",
     "parse-filepath": "^1.0.2",

package/typings/base-types-visitor.d.cts

@@ -1,4 +1,4 @@
-import { DirectiveDefinitionNode, DirectiveNode, EnumTypeDefinitionNode, EnumValueDefinitionNode, FieldDefinitionNode, GraphQLSchema, InputObjectTypeDefinitionNode, InputValueDefinitionNode, InterfaceTypeDefinitionNode, ListTypeNode, NamedTypeNode, NameNode, NonNullTypeNode, ObjectTypeDefinitionNode, ScalarTypeDefinitionNode, StringValueNode, UnionTypeDefinitionNode } from 'graphql';
+import { DirectiveDefinitionNode, DirectiveNode, EnumTypeDefinitionNode, EnumValueDefinitionNode, FieldDefinitionNode, GraphQLSchema, InputObjectTypeDefinitionNode, InputValueDefinitionNode, InterfaceTypeDefinitionNode, ListTypeNode, NamedTypeNode, NonNullTypeNode, ObjectTypeDefinitionNode, ScalarTypeDefinitionNode, UnionTypeDefinitionNode } from 'graphql';
 import { BaseVisitor, ParsedConfig, RawConfig } from './base-visitor.cjs';
 import { DeclarationKind, DeclarationKindConfig, DirectiveArgumentAndInputFieldMappings, EnumValuesMap, NormalizedScalarsMap, ParsedDirectiveArgumentAndInputFieldMappings, ParsedEnumValuesMap } from './types.cjs';
 import { DeclarationBlock, DeclarationBlockConfig } from './utils.cjs';
@@ -468,7 +468,6 @@ export declare class BaseTypesVisitor<TRawConfig extends RawTypesConfig = RawTyp
     getInputObjectOneOfDeclarationBlock(node: InputObjectTypeDefinitionNode): DeclarationBlock;
     InputObjectTypeDefinition(node: InputObjectTypeDefinitionNode): string;
     InputValueDefinition(node: InputValueDefinitionNode): string;
-    Name(node: NameNode): string;
     FieldDefinition(node: FieldDefinitionNode): string;
     UnionTypeDefinition(node: UnionTypeDefinitionNode, key: string | number | undefined, parent: any): string;
     protected mergeInterfaces(interfaces: string[], hasOtherFields: boolean): string;
@@ -483,7 +482,6 @@ export declare class BaseTypesVisitor<TRawConfig extends RawTypesConfig = RawTyp
     protected handleEnumValueMapper(typeIdentifier: string, importIdentifier: string | null, sourceIdentifier: string | null, sourceFile: string | null): string[];
     getEnumsImports(): string[];
     EnumTypeDefinition(node: EnumTypeDefinitionNode): string;
-    StringValue(node: StringValueNode): string;
     protected makeValidEnumIdentifier(identifier: string): string;
     protected buildEnumValuesBlock(typeName: string, values: ReadonlyArray<EnumValueDefinitionNode>): string;
     DirectiveDefinition(_node: DirectiveDefinitionNode): string;

package/typings/base-types-visitor.d.ts

@@ -1,4 +1,4 @@
-import { DirectiveDefinitionNode, DirectiveNode, EnumTypeDefinitionNode, EnumValueDefinitionNode, FieldDefinitionNode, GraphQLSchema, InputObjectTypeDefinitionNode, InputValueDefinitionNode, InterfaceTypeDefinitionNode, ListTypeNode, NamedTypeNode, NameNode, NonNullTypeNode, ObjectTypeDefinitionNode, ScalarTypeDefinitionNode, StringValueNode, UnionTypeDefinitionNode } from 'graphql';
+import { DirectiveDefinitionNode, DirectiveNode, EnumTypeDefinitionNode, EnumValueDefinitionNode, FieldDefinitionNode, GraphQLSchema, InputObjectTypeDefinitionNode, InputValueDefinitionNode, InterfaceTypeDefinitionNode, ListTypeNode, NamedTypeNode, NonNullTypeNode, ObjectTypeDefinitionNode, ScalarTypeDefinitionNode, UnionTypeDefinitionNode } from 'graphql';
 import { BaseVisitor, ParsedConfig, RawConfig } from './base-visitor.js';
 import { DeclarationKind, DeclarationKindConfig, DirectiveArgumentAndInputFieldMappings, EnumValuesMap, NormalizedScalarsMap, ParsedDirectiveArgumentAndInputFieldMappings, ParsedEnumValuesMap } from './types.js';
 import { DeclarationBlock, DeclarationBlockConfig } from './utils.js';
@@ -468,7 +468,6 @@ export declare class BaseTypesVisitor<TRawConfig extends RawTypesConfig = RawTyp
     getInputObjectOneOfDeclarationBlock(node: InputObjectTypeDefinitionNode): DeclarationBlock;
     InputObjectTypeDefinition(node: InputObjectTypeDefinitionNode): string;
     InputValueDefinition(node: InputValueDefinitionNode): string;
-    Name(node: NameNode): string;
     FieldDefinition(node: FieldDefinitionNode): string;
     UnionTypeDefinition(node: UnionTypeDefinitionNode, key: string | number | undefined, parent: any): string;
     protected mergeInterfaces(interfaces: string[], hasOtherFields: boolean): string;
@@ -483,7 +482,6 @@ export declare class BaseTypesVisitor<TRawConfig extends RawTypesConfig = RawTyp
     protected handleEnumValueMapper(typeIdentifier: string, importIdentifier: string | null, sourceIdentifier: string | null, sourceFile: string | null): string[];
     getEnumsImports(): string[];
     EnumTypeDefinition(node: EnumTypeDefinitionNode): string;
-    StringValue(node: StringValueNode): string;
     protected makeValidEnumIdentifier(identifier: string): string;
     protected buildEnumValuesBlock(typeName: string, values: ReadonlyArray<EnumValueDefinitionNode>): string;
     DirectiveDefinition(_node: DirectiveDefinitionNode): string;

package/typings/base-visitor.d.cts

@@ -19,7 +19,6 @@ export interface ParsedConfig {
     fragmentImports: ImportDeclaration<FragmentImport>[];
     immutableTypes: boolean;
     useTypeImports: boolean;
-    dedupeFragments: boolean;
     allowEnumStringTypes: boolean;
     inlineFragmentTypes: InlineFragmentTypeOptions;
     emitLegacyCommonJSImports: boolean;
@@ -329,15 +328,6 @@ export interface RawConfig {
      * @ignore
      */
     globalNamespace?: boolean;
-    /**
-     * @description  Removes fragment duplicates for reducing data transfer.
-     * It is done by removing sub-fragments imports from fragment definition
-     * Instead - all of them are imported to the Operation node.
-     * @type boolean
-     * @default false
-     * @deprecated This option is no longer needed. It will be removed in the next major version.
-     */
-    dedupeFragments?: boolean;
     /**
      * @ignore
      */

package/typings/base-visitor.d.ts

@@ -19,7 +19,6 @@ export interface ParsedConfig {
     fragmentImports: ImportDeclaration<FragmentImport>[];
     immutableTypes: boolean;
     useTypeImports: boolean;
-    dedupeFragments: boolean;
     allowEnumStringTypes: boolean;
     inlineFragmentTypes: InlineFragmentTypeOptions;
     emitLegacyCommonJSImports: boolean;
@@ -329,15 +328,6 @@ export interface RawConfig {
      * @ignore
      */
     globalNamespace?: boolean;
-    /**
-     * @description  Removes fragment duplicates for reducing data transfer.
-     * It is done by removing sub-fragments imports from fragment definition
-     * Instead - all of them are imported to the Operation node.
-     * @type boolean
-     * @default false
-     * @deprecated This option is no longer needed. It will be removed in the next major version.
-     */
-    dedupeFragments?: boolean;
     /**
      * @ignore
      */

package/typings/client-side-base-visitor.d.cts

@@ -10,11 +10,6 @@ export declare enum DocumentMode {
     string = "string"
 }
 export interface RawClientSideBasePluginConfig extends RawConfig {
-    /**
-     * @description Deprecated. Changes the documentMode to `documentNode`.
-     * @default false
-     */
-    noGraphQLTag?: boolean;
     /**
      * @default graphql-tag#gql
      * @description Customize from which module will `gql` be imported from.
@@ -224,7 +219,7 @@ export declare class ClientSideBaseVisitor<TRawConfig extends RawClientSideBaseP
     constructor(_schema: GraphQLSchema, fragments: LoadedFragment[], rawConfig: TRawConfig, additionalConfig: Partial<TPluginConfig>, documents?: Types.DocumentFile[]);
     protected _extractFragments(document: FragmentDefinitionNode | OperationDefinitionNode, withNested?: boolean): string[];
     protected _transformFragments(fragmentNames: Array<string>): string[];
-    protected _includeFragments(fragments: string[], nodeKind: 'FragmentDefinition' | 'OperationDefinition'): string;
+    protected _includeFragments(fragments: string[]): string;
     protected _prepareDocument(documentStr: string): string;
     protected _gql(node: FragmentDefinitionNode | OperationDefinitionNode): string;
     protected _getGraphQLCodegenMetadata(node: OperationDefinitionNode, definitions?: ReadonlyArray<DefinitionNode>): Record<string, any> | void | undefined;

package/typings/client-side-base-visitor.d.ts

@@ -10,11 +10,6 @@ export declare enum DocumentMode {
     string = "string"
 }
 export interface RawClientSideBasePluginConfig extends RawConfig {
-    /**
-     * @description Deprecated. Changes the documentMode to `documentNode`.
-     * @default false
-     */
-    noGraphQLTag?: boolean;
     /**
      * @default graphql-tag#gql
      * @description Customize from which module will `gql` be imported from.
@@ -224,7 +219,7 @@ export declare class ClientSideBaseVisitor<TRawConfig extends RawClientSideBaseP
     constructor(_schema: GraphQLSchema, fragments: LoadedFragment[], rawConfig: TRawConfig, additionalConfig: Partial<TPluginConfig>, documents?: Types.DocumentFile[]);
     protected _extractFragments(document: FragmentDefinitionNode | OperationDefinitionNode, withNested?: boolean): string[];
     protected _transformFragments(fragmentNames: Array<string>): string[];
-    protected _includeFragments(fragments: string[], nodeKind: 'FragmentDefinition' | 'OperationDefinition'): string;
+    protected _includeFragments(fragments: string[]): string;
     protected _prepareDocument(documentStr: string): string;
     protected _gql(node: FragmentDefinitionNode | OperationDefinitionNode): string;
     protected _getGraphQLCodegenMetadata(node: OperationDefinitionNode, definitions?: ReadonlyArray<DefinitionNode>): Record<string, any> | void | undefined;