chore: use new graphql linter

Author: optionsome

.eslintignore

@@ -2,6 +2,8 @@ node_modules
 _static
 static
 app/buildInfo.js
+build/schema.graphql
+digitransit-search-util/packages/digitransit-search-util-query-utils/schema/schema.graphql
 digitransit-component/packages/**/lib
 digitransit-store/packages/**/lib
 digitransit-search-util/packages/**/lib

.eslintrc.js

@@ -46,17 +46,6 @@ module.exports = {
     // compat
     'compat/compat': 'warn',
 
-    // graphql
-    'graphql/template-strings': [
-      'error',
-      {
-        env: 'relay',
-        // eslint-disable-next-line global-require
-        schemaJson: require('./build/schema.json').data,
-        tagName: 'graphql',
-      },
-    ],
-
     // prettier
     'prettier/prettier': [
       'error',
@@ -71,8 +60,51 @@ module.exports = {
   env: {
     browser: true,
   },
-  plugins: ['react', 'graphql', 'compat', 'prettier', 'jsx-a11y'],
+  plugins: ['react', 'compat', 'prettier', 'jsx-a11y'],
   settings: {
     polyfills: ['fetch', 'promises'],
   },
+  overrides: [
+    {
+      files: ['*.js'],
+      processor: '@graphql-eslint/graphql',
+    },
+    {
+      files: ['*.graphql'],
+      extends: [
+        'plugin:@graphql-eslint/operations-recommended',
+        'plugin:@graphql-eslint/relay',
+      ],
+      // Available rules can be found at https://the-guild.dev/graphql/eslint/rules/naming-convention
+      rules: {
+        '@graphql-eslint/no-deprecated': 'warn',
+        // Recommended naming conventions have some clashes with relay conventions
+        '@graphql-eslint/naming-convention': [
+          'error',
+          {
+            VariableDefinition: 'camelCase',
+          },
+        ],
+        // Relay directives caused errors and ignoring them didn't work for some reason
+        '@graphql-eslint/known-directives': 'off',
+
+        // These require parserOptions.operations to be defined
+        // but haven't been able to figure out if it is possible
+        // to define them with relay
+        '@graphql-eslint/known-fragment-names': 'off',
+        '@graphql-eslint/no-one-place-fragments': 'off',
+        '@graphql-eslint/no-undefined-variables': 'off',
+        '@graphql-eslint/no-unused-fragments': 'off',
+        '@graphql-eslint/no-unused-variables': 'off',
+        '@graphql-eslint/require-id-when-available': 'off',
+        '@graphql-eslint/require-import-fragment': 'off',
+        '@graphql-eslint/selection-set-depth': 'off',
+        '@graphql-eslint/unique-fragment-name': 'off',
+        '@graphql-eslint/unique-operation-name': 'off',
+      },
+      parserOptions: {
+        schema: './build/schema.graphql',
+      },
+    },
+  ],
 };

.yarnrc.yml

@@ -22,9 +22,6 @@ packageExtensions:
   graphql-config@*:
     peerDependencies:
       typescript: "*"
-  eslint-plugin-graphql@*:
-    peerDependencies:
-      typescript: "*"
   "@lerna/version@*":
     peerDependencies:
       "@octokit/core": "*"

package.json

@@ -238,6 +238,7 @@
     "@babel/runtime": "7.20.6",
     "@commitlint/cli": "9.1.2",
     "@commitlint/config-conventional": "9.1.2",
+    "@graphql-eslint/eslint-plugin": "3.20.1",
     "@lerna/batch-packages": "3.16.0",
     "@lerna/filter-packages": "3.18.0",
     "@lerna/project": "3.21.0",
@@ -268,7 +269,6 @@
     "eslint-config-prettier": "9.1.0",
     "eslint-import-resolver-webpack": "0.13.8",
     "eslint-plugin-compat": "4.2.0",
-    "eslint-plugin-graphql": "4.0.0",
     "eslint-plugin-import": "2.29.1",
     "eslint-plugin-jsx-a11y": "6.8.0",
     "eslint-plugin-prettier": "5.1.2",