Support OperationDefinition naming convention (#426)

* Add OperationDefinition to naming-convertion rule

* Add OperationDefinition test to naming convention test

* added missing changeset file

Co-authored-by: Dotan Simha <[email protected]>

Author: Godsenal

.changeset/ninety-colts-switch.md

@@ -0,0 +1,5 @@
+---
+'@graphql-eslint/eslint-plugin': patch
+---
+
+Support OperationDefinition in naming-convention rule

.husky/_/husky.sh

@@ -0,0 +1,30 @@
+#!/bin/sh
+if [ -z "$husky_skip_init" ]; then
+  debug () {
+    [ "$HUSKY_DEBUG" = "1" ] && echo "husky (debug) - $1"
+  }
+
+  readonly hook_name="$(basename "$0")"
+  debug "starting $hook_name..."
+
+  if [ "$HUSKY" = "0" ]; then
+    debug "HUSKY env variable is set to 0, skipping hook"
+    exit 0
+  fi
+
+  if [ -f ~/.huskyrc ]; then
+    debug "sourcing ~/.huskyrc"
+    . ~/.huskyrc
+  fi
+
+  export readonly husky_skip_init=1
+  sh -e "$0" "$@"
+  exitCode="$?"
+
+  if [ $exitCode != 0 ]; then
+    echo "husky - $hook_name hook exited with code $exitCode (error)"
+    exit $exitCode
+  fi
+
+  exit 0
+fi

packages/plugin/src/rules/naming-convention.ts

@@ -327,6 +327,12 @@ const rule: GraphQLESLintRule<NamingConventionRuleConfig> = {
           checkNode(node.name, property, 'Input property');
         }
       },
+      OperationDefinition: node => {
+        if (options.OperationDefinition) {
+          const property = normalisePropertyOption(options.OperationDefinition);
+          checkNode(node.name, property, 'Operation');
+        }
+      },
       FragmentDefinition: node => {
         if (options.FragmentDefinition) {
           const property = normalisePropertyOption(options.FragmentDefinition);

packages/plugin/tests/naming-convention.spec.ts

@@ -24,6 +24,7 @@ ruleTester.runGraphQLTests('naming-convention', rule, {
       }`,
       options: [
         {
+          OperationDefinition: 'PascalCase',
           ObjectTypeDefinition: 'PascalCase',
           FieldDefinition: 'camelCase',
           EnumValueDefinition: 'UPPER_CASE',
@@ -226,5 +227,17 @@ ruleTester.runGraphQLTests('naming-convention', rule, {
         { message: 'Query "getC" should not have one of the following prefix(es): get, query' },
       ],
     },
+    {
+      code: 'query Foo { foo } query getBar { bar }',
+      options: [
+        {
+          OperationDefinition: { style: 'camelCase', forbiddenPrefixes: ['get'] },
+        },
+      ],
+      errors: [
+        { message: 'Operation name "Foo" should be in camelCase format' },
+        { message: 'Operation "getBar" should not have one of the following prefix(es): get' },
+      ],
+    },
   ],
 });