chore(lint): prevent import * from 'graphql' in plugins (#576)

Author: benjie

.eslintrc.js

@@ -49,6 +49,32 @@ module.exports = {
     "no-inner-declarations": "warn",
   },
   overrides: [
+    // Rules for plugins
+    {
+      files: [
+        "packages/graphile-build/src/plugins/**/*.ts",
+        "packages/graphile-build-pg/src/**/*.ts",
+        "packages/graphile-utils/src/**/*.ts",
+        "packages/pg-pubsub/src/**/*.ts",
+        "packages/postgraphile-core/src/**/*.ts",
+        "packages/subscriptions-lds/src/**/*.ts",
+      ],
+      rules: {
+        "no-restricted-imports": [
+          "error",
+          {
+            paths: [
+              {
+                name: "graphql",
+                message:
+                  'Please refer to `build.graphql` instead, or use `import("graphql")` in value positions. (This helps us to avoid multiple `graphql` modules in the `node_modules` tree from causing issues.)',
+              },
+            ],
+          },
+        ],
+      },
+    },
+
     // Rules for Flow only
     {
       files: ["*.js", "*.jsx"],

packages/graphile-utils/src/gql.ts

@@ -1,3 +1,4 @@
+// eslint-disable-next-line no-restricted-imports
 import { parse, visit, ASTNode, DocumentNode, DefinitionNode } from "graphql";
 const $$embed = Symbol("graphile-embed");