@graphql-eslint/eslint-plugin 4.3.0 → 4.3.1-alpha-20241209185034-de2d7397da8c26620a8930dd12b6dff42e43f537

package/cjs/rules/require-description/index.js

@@ -8,57 +8,60 @@ var _utils = require('@graphql-tools/utils');
 
 
 var _utilsjs = require('../../utils.js');
-const RULE_ID = "require-description", ALLOWED_KINDS = [
+const RULE_ID = "require-description";
+const ALLOWED_KINDS = [
   ..._utilsjs.TYPES_KINDS,
   _graphql.Kind.DIRECTIVE_DEFINITION,
   _graphql.Kind.FIELD_DEFINITION,
   _graphql.Kind.INPUT_VALUE_DEFINITION,
   _graphql.Kind.ENUM_VALUE_DEFINITION,
   _graphql.Kind.OPERATION_DEFINITION
-], schema = {
+];
+const schema = {
   type: "array",
   minItems: 1,
   maxItems: 1,
   items: {
     type: "object",
-    additionalProperties: !1,
+    additionalProperties: false,
     minProperties: 1,
     properties: {
       types: {
         type: "boolean",
-        enum: [!0],
+        enum: [true],
         description: `Includes:
-${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
-`)}`
+${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join("\n")}`
       },
       rootField: {
         type: "boolean",
-        enum: [!0],
+        enum: [true],
         description: "Definitions within `Query`, `Mutation`, and `Subscription` root types."
       },
       ignoredSelectors: {
         ..._utilsjs.ARRAY_DEFAULT_OPTIONS,
-        description: ["Ignore specific selectors", _utilsjs.eslintSelectorsTip].join(`
-`)
+        description: ["Ignore specific selectors", _utilsjs.eslintSelectorsTip].join("\n")
       },
       ...Object.fromEntries(
         [...ALLOWED_KINDS].sort().map((kind) => {
           let description = `> [!NOTE]
 >
 > Read more about this kind on [spec.graphql.org](https://spec.graphql.org/October2021/#${kind}).`;
-          return kind === _graphql.Kind.OPERATION_DEFINITION && (description += [
-            "",
-            "",
-            "> [!WARNING]",
-            ">",
-            '> You must use only comment syntax `#` and not description syntax `"""` or `"`.'
-          ].join(`
-`)), [kind, { type: "boolean", description }];
+          if (kind === _graphql.Kind.OPERATION_DEFINITION) {
+            description += [
+              "",
+              "",
+              "> [!WARNING]",
+              ">",
+              '> You must use only comment syntax `#` and not description syntax `"""` or `"`.'
+            ].join("\n");
+          }
+          return [kind, { type: "boolean", description }];
         })
       )
     }
   }
-}, rule = exports.rule = {
+};
+const rule = {
   meta: {
     docs: {
       category: "Schema",
@@ -67,7 +70,7 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
       examples: [
         {
           title: "Incorrect",
-          usage: [{ types: !0, FieldDefinition: !0 }],
+          usage: [{ types: true, FieldDefinition: true }],
           code: (
             /* GraphQL */
             `
@@ -79,7 +82,7 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
         },
         {
           title: "Correct",
-          usage: [{ types: !0, FieldDefinition: !0 }],
+          usage: [{ types: true, FieldDefinition: true }],
           code: (
             /* GraphQL */
             `
@@ -97,7 +100,7 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
         },
         {
           title: "Correct",
-          usage: [{ OperationDefinition: !0 }],
+          usage: [{ OperationDefinition: true }],
           code: (
             /* GraphQL */
             `
@@ -110,7 +113,7 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
         },
         {
           title: "Correct",
-          usage: [{ rootField: !0 }],
+          usage: [{ rootField: true }],
           code: (
             /* GraphQL */
             `
@@ -158,12 +161,12 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
       ],
       configOptions: [
         {
-          types: !0,
-          [_graphql.Kind.DIRECTIVE_DEFINITION]: !0,
-          rootField: !0
+          types: true,
+          [_graphql.Kind.DIRECTIVE_DEFINITION]: true,
+          rootField: true
         }
       ],
-      recommended: !0
+      recommended: true
     },
     type: "suggestion",
     messages: {
@@ -172,11 +175,18 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
     schema
   },
   create(context) {
-    const { types, rootField, ignoredSelectors = [], ...restOptions } = context.options[0] || {}, kinds = new Set(types ? _utilsjs.TYPES_KINDS : []);
-    for (const [kind, isEnabled] of Object.entries(restOptions))
-      isEnabled ? kinds.add(kind) : kinds.delete(kind);
+    const { types, rootField, ignoredSelectors = [], ...restOptions } = context.options[0] || {};
+    const kinds = new Set(types ? _utilsjs.TYPES_KINDS : []);
+    for (const [kind, isEnabled] of Object.entries(restOptions)) {
+      if (isEnabled) {
+        kinds.add(kind);
+      } else {
+        kinds.delete(kind);
+      }
+    }
     if (rootField) {
-      const schema2 = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context), rootTypeNames = _utils.getRootTypeNames.call(void 0, schema2);
+      const schema2 = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context);
+      const rootTypeNames = _utils.getRootTypeNames.call(void 0, schema2);
       kinds.add(
         `:matches(ObjectTypeDefinition, ObjectTypeExtension)[name.value=/^(${[
           ...rootTypeNames
@@ -184,27 +194,35 @@ ${_utilsjs.TYPES_KINDS.map((kind) => `- \`${kind}\``).join(`
       );
     }
     let selector = `:matches(${[...kinds]})`;
-    for (const str of ignoredSelectors)
+    for (const str of ignoredSelectors) {
       selector += `:not(${str})`;
+    }
     return {
       [selector](node) {
         let description = "";
         const isOperation = node.kind === _graphql.Kind.OPERATION_DEFINITION;
         if (isOperation) {
-          const rawNode = node.rawNode(), { prev, line } = rawNode.loc.startToken;
+          const rawNode = node.rawNode();
+          const { prev, line } = rawNode.loc.startToken;
           if (_optionalChain([prev, 'optionalAccess', _ => _.kind]) === _graphql.TokenKind.COMMENT) {
-            const value = prev.value.trim(), linesBefore = line - prev.line;
-            !value.startsWith("eslint") && linesBefore === 1 && (description = value);
+            const value = prev.value.trim();
+            const linesBefore = line - prev.line;
+            if (!value.startsWith("eslint") && linesBefore === 1) {
+              description = value;
+            }
           }
-        } else
+        } else {
           description = _optionalChain([node, 'access', _2 => _2.description, 'optionalAccess', _3 => _3.value, 'access', _4 => _4.trim, 'call', _5 => _5()]) || "";
-        description.length === 0 && context.report({
-          loc: isOperation ? _utilsjs.getLocation.call(void 0, node.loc.start, node.operation) : node.name.loc,
-          messageId: RULE_ID,
-          data: {
-            nodeName: _utilsjs.getNodeName.call(void 0, node)
-          }
-        });
+        }
+        if (description.length === 0) {
+          context.report({
+            loc: isOperation ? _utilsjs.getLocation.call(void 0, node.loc.start, node.operation) : node.name.loc,
+            messageId: RULE_ID,
+            data: {
+              nodeName: _utilsjs.getNodeName.call(void 0, node)
+            }
+          });
+        }
       }
     };
   }

package/cjs/rules/require-field-of-type-query-in-mutation-result/index.js

@@ -1,13 +1,14 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _optionalChain(ops) { let lastAccessLHS = undefined; let value = ops[0]; let i = 1; while (i < ops.length) { const op = ops[i]; const fn = ops[i + 1]; i += 2; if ((op === 'optionalAccess' || op === 'optionalCall') && value == null) { return undefined; } if (op === 'access' || op === 'optionalAccess') { lastAccessLHS = value; value = fn(value); } else if (op === 'call' || op === 'optionalCall') { value = fn((...args) => value.call(lastAccessLHS, ...args)); lastAccessLHS = undefined; } } return value; }var _graphql = require('graphql');
 var _utilsjs = require('../../utils.js');
-const RULE_ID = "require-field-of-type-query-in-mutation-result", rule = exports.rule = {
+const RULE_ID = "require-field-of-type-query-in-mutation-result";
+const rule = {
   meta: {
     type: "suggestion",
     docs: {
       category: "Schema",
       description: "Allow the client in one round-trip not only to call mutation but also to get a wagon of data to update their application.\n> Currently, no errors are reported for result type `union`, `interface` and `scalar`.",
       url: `https://the-guild.dev/graphql/eslint/rules/${RULE_ID}`,
-      requiresSchema: !0,
+      requiresSchema: true,
       examples: [
         {
           title: "Incorrect",
@@ -47,16 +48,26 @@ const RULE_ID = "require-field-of-type-query-in-mutation-result", rule = exports
     schema: []
   },
   create(context) {
-    const schema = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context), mutationType = schema.getMutationType(), queryType = schema.getQueryType();
-    return !mutationType || !queryType ? {} : {
-      [`:matches(ObjectTypeDefinition, ObjectTypeExtension)[name.value=${mutationType.name}] > FieldDefinition > .gqlType Name`](node) {
-        const typeName = node.value, graphQLType = schema.getType(typeName);
+    const schema = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context);
+    const mutationType = schema.getMutationType();
+    const queryType = schema.getQueryType();
+    if (!mutationType || !queryType) {
+      return {};
+    }
+    const selector = `:matches(ObjectTypeDefinition, ObjectTypeExtension)[name.value=${mutationType.name}] > FieldDefinition > .gqlType Name`;
+    return {
+      [selector](node) {
+        const typeName = node.value;
+        const graphQLType = schema.getType(typeName);
         if (_graphql.isObjectType.call(void 0, graphQLType)) {
           const { fields } = graphQLType.astNode;
-          _optionalChain([fields, 'optionalAccess', _ => _.some, 'call', _2 => _2((field) => _utilsjs.getTypeName.call(void 0, field) === queryType.name)]) || context.report({
-            node,
-            message: `Mutation result type "${graphQLType.name}" must contain field of type "${queryType.name}"`
-          });
+          const hasQueryType = _optionalChain([fields, 'optionalAccess', _ => _.some, 'call', _2 => _2((field) => _utilsjs.getTypeName.call(void 0, field) === queryType.name)]);
+          if (!hasQueryType) {
+            context.report({
+              node,
+              message: `Mutation result type "${graphQLType.name}" must contain field of type "${queryType.name}"`
+            });
+          }
         }
       }
     };

package/cjs/rules/require-import-fragment/index.js

@@ -1,6 +1,8 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; } function _optionalChain(ops) { let lastAccessLHS = undefined; let value = ops[0]; let i = 1; while (i < ops.length) { const op = ops[i]; const fn = ops[i + 1]; i += 2; if ((op === 'optionalAccess' || op === 'optionalCall') && value == null) { return undefined; } if (op === 'access' || op === 'optionalAccess') { lastAccessLHS = value; value = fn(value); } else if (op === 'call' || op === 'optionalCall') { value = fn((...args) => value.call(lastAccessLHS, ...args)); lastAccessLHS = undefined; } } return value; }var _nodepath = require('node:path'); var _nodepath2 = _interopRequireDefault(_nodepath);
 var _utilsjs = require('../../utils.js');
-const RULE_ID = "require-import-fragment", SUGGESTION_ID = "add-import-expression", rule = exports.rule = {
+const RULE_ID = "require-import-fragment";
+const SUGGESTION_ID = "add-import-expression";
+const rule = {
   meta: {
     type: "suggestion",
     docs: {
@@ -64,9 +66,9 @@ const RULE_ID = "require-import-fragment", SUGGESTION_ID = "add-import-expressio
           )
         }
       ],
-      requiresSiblings: !0
+      requiresSiblings: true
     },
-    hasSuggestions: !0,
+    hasSuggestions: true,
     messages: {
       [RULE_ID]: 'Expected "{{fragmentName}}" fragment to be imported.',
       [SUGGESTION_ID]: 'Add import expression for "{{fragmentName}}".'
@@ -74,24 +76,31 @@ const RULE_ID = "require-import-fragment", SUGGESTION_ID = "add-import-expressio
     schema: []
   },
   create(context) {
-    const comments = context.getSourceCode().getAllComments(), siblings = _utilsjs.requireGraphQLOperations.call(void 0, RULE_ID, context), filePath = context.filename;
+    const comments = context.getSourceCode().getAllComments();
+    const siblings = _utilsjs.requireGraphQLOperations.call(void 0, RULE_ID, context);
+    const filePath = context.filename;
     return {
       "FragmentSpread > .name"(node) {
-        const fragmentName = node.value, fragmentsFromSiblings = siblings.getFragment(fragmentName);
+        const fragmentName = node.value;
+        const fragmentsFromSiblings = siblings.getFragment(fragmentName);
         for (const comment of comments) {
-          if (comment.type !== "Line" || !new RegExp(
+          if (comment.type !== "Line") continue;
+          const isPossibleImported = new RegExp(
             `^\\s*import\\s+(${fragmentName}\\s+from\\s+)?['"]`
-          ).test(comment.value)) continue;
+          ).test(comment.value);
+          if (!isPossibleImported) continue;
           const extractedImportPath = _optionalChain([comment, 'access', _ => _.value, 'access', _2 => _2.match, 'call', _3 => _3(/(["'])((?:\1|.)*?)\1/), 'optionalAccess', _4 => _4[2]]);
           if (!extractedImportPath) continue;
           const importPath = _nodepath2.default.join(filePath, "..", extractedImportPath);
-          if (fragmentsFromSiblings.some(
+          const hasInSiblings = fragmentsFromSiblings.some(
             (source) => source.filePath === importPath
-          )) return;
+          );
+          if (hasInSiblings) return;
         }
-        if (fragmentsFromSiblings.some(
+        const fragmentInSameFile = fragmentsFromSiblings.some(
           (source) => source.filePath === filePath
-        )) return;
+        );
+        if (fragmentInSameFile) return;
         const suggestedFilePaths = fragmentsFromSiblings.length ? fragmentsFromSiblings.map(
           (o) => (
             // Use always forward slash for suggested import path

package/cjs/rules/require-nullable-fields-with-oneof/index.js

@@ -1,6 +1,7 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true});var _graphql = require('graphql');
 var _utilsjs = require('../../utils.js');
-const RULE_ID = "require-nullable-fields-with-oneof", rule = exports.rule = {
+const RULE_ID = "require-nullable-fields-with-oneof";
+const rule = {
   meta: {
     type: "suggestion",
     docs: {
@@ -42,16 +43,22 @@ const RULE_ID = "require-nullable-fields-with-oneof", rule = exports.rule = {
   create(context) {
     return {
       "Directive[name.value=oneOf]"({ parent }) {
-        if ([
+        const isTypeOrInput = [
           _graphql.Kind.OBJECT_TYPE_DEFINITION,
           _graphql.Kind.INPUT_OBJECT_TYPE_DEFINITION
-        ].includes(parent.kind))
-          for (const field of parent.fields || [])
-            field.gqlType.kind === _graphql.Kind.NON_NULL_TYPE && context.report({
+        ].includes(parent.kind);
+        if (!isTypeOrInput) {
+          return;
+        }
+        for (const field of parent.fields || []) {
+          if (field.gqlType.kind === _graphql.Kind.NON_NULL_TYPE) {
+            context.report({
               node: field.name,
               messageId: RULE_ID,
               data: { nodeName: _utilsjs.getNodeName.call(void 0, field) }
             });
+          }
+        }
       }
     };
   }

package/cjs/rules/require-nullable-result-in-root/index.js

@@ -1,14 +1,15 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _optionalChain(ops) { let lastAccessLHS = undefined; let value = ops[0]; let i = 1; while (i < ops.length) { const op = ops[i]; const fn = ops[i + 1]; i += 2; if ((op === 'optionalAccess' || op === 'optionalCall') && value == null) { return undefined; } if (op === 'access' || op === 'optionalAccess') { lastAccessLHS = value; value = fn(value); } else if (op === 'call' || op === 'optionalCall') { value = fn((...args) => value.call(lastAccessLHS, ...args)); lastAccessLHS = undefined; } } return value; }var _graphql = require('graphql');
 var _utilsjs = require('../../utils.js');
-const RULE_ID = "require-nullable-result-in-root", rule = exports.rule = {
+const RULE_ID = "require-nullable-result-in-root";
+const rule = {
   meta: {
     type: "suggestion",
-    hasSuggestions: !0,
+    hasSuggestions: true,
     docs: {
       category: "Schema",
       description: "Require nullable fields in root types.",
       url: `https://the-guild.dev/graphql/eslint/rules/${RULE_ID}`,
-      requiresSchema: !0,
+      requiresSchema: true,
       examples: [
         {
           title: "Incorrect",
@@ -42,34 +43,38 @@ const RULE_ID = "require-nullable-result-in-root", rule = exports.rule = {
     schema: []
   },
   create(context) {
-    const schema = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context), rootTypeNames = new Set(
+    const schema = _utilsjs.requireGraphQLSchema.call(void 0, RULE_ID, context);
+    const rootTypeNames = new Set(
       [schema.getQueryType(), schema.getMutationType()].filter((v) => !!v).map((type) => type.name)
-    ), sourceCode = context.getSourceCode();
+    );
+    const sourceCode = context.getSourceCode();
     return {
       "ObjectTypeDefinition,ObjectTypeExtension"(node) {
-        if (rootTypeNames.has(node.name.value))
-          for (const field of node.fields || []) {
-            if (field.gqlType.type !== _graphql.Kind.NON_NULL_TYPE || field.gqlType.gqlType.type !== _graphql.Kind.NAMED_TYPE)
-              continue;
-            const name = field.gqlType.gqlType.name.value, type = schema.getType(name), resultType = _optionalChain([type, 'optionalAccess', _ => _.astNode]) ? _utilsjs.getNodeName.call(void 0, type.astNode) : _optionalChain([type, 'optionalAccess', _2 => _2.name]);
-            context.report({
-              node: field.gqlType,
-              messageId: RULE_ID,
-              data: {
-                resultType: resultType || "",
-                rootType: _utilsjs.getNodeName.call(void 0, node)
-              },
-              suggest: [
-                {
-                  desc: `Make ${resultType} nullable`,
-                  fix(fixer) {
-                    const text = sourceCode.getText(field.gqlType);
-                    return fixer.replaceText(field.gqlType, text.replace("!", ""));
-                  }
+        if (!rootTypeNames.has(node.name.value)) return;
+        for (const field of node.fields || []) {
+          if (field.gqlType.type !== _graphql.Kind.NON_NULL_TYPE || field.gqlType.gqlType.type !== _graphql.Kind.NAMED_TYPE)
+            continue;
+          const name = field.gqlType.gqlType.name.value;
+          const type = schema.getType(name);
+          const resultType = _optionalChain([type, 'optionalAccess', _ => _.astNode]) ? _utilsjs.getNodeName.call(void 0, type.astNode) : _optionalChain([type, 'optionalAccess', _2 => _2.name]);
+          context.report({
+            node: field.gqlType,
+            messageId: RULE_ID,
+            data: {
+              resultType: resultType || "",
+              rootType: _utilsjs.getNodeName.call(void 0, node)
+            },
+            suggest: [
+              {
+                desc: `Make ${resultType} nullable`,
+                fix(fixer) {
+                  const text = sourceCode.getText(field.gqlType);
+                  return fixer.replaceText(field.gqlType, text.replace("!", ""));
                 }
-              ]
-            });
-          }
+              }
+            ]
+          });
+        }
       }
     };
   }