update bob (#1329)

* update bob

* use ts everywhere

* add legacy configs

* legacy configs

* adjust tests

* everything works

* eslint fixes

* fix ci

* assert { type: "json" }

* fix prebuild

* fix alpha release

* try new version

Author: dimaMachina

.eslintrc.cjs

@@ -58,5 +58,11 @@ module.exports = {
         '@typescript-eslint/no-unused-vars': 'off',
       },
     },
+    {
+      files: ['scripts/**'],
+      rules: {
+        'no-console': 'off'
+      }
+    }
   ],
 };

.github/workflows/tests.yml

@@ -36,7 +36,7 @@ jobs:
           packageManager: pnpm
 
       - name: Use GraphQL v${{matrix.graphql_version}}
-        run: node ./scripts/match-graphql.js ${{matrix.graphql_version}}
+        run: node scripts/match-graphql.js ${{matrix.graphql_version}}
 
       - name: Install Dependencies
         run: pnpm i --no-frozen-lockfile

.npmrc

@@ -1,2 +1,3 @@
 strict-peer-dependencies=false
 shell-emulator=true
+enable-pre-post-scripts=true

package.json

@@ -5,20 +5,19 @@
   "license": "MIT",
   "private": true,
   "scripts": {
-    "build": "pnpm transpile-ts && bob build",
+    "build": "tsc && bob build",
     "ci:lint": "eslint --ignore-path .gitignore --ext ts,js . --output-file eslint_report.json --format json",
     "create-rule": "tsx scripts/create-rule.ts",
     "generate:configs": "tsx scripts/generate-configs.ts",
     "generate:docs": "tsx scripts/generate-docs.ts",
     "lint": "eslint --ignore-path .gitignore --ext ts,js .",
-    "postbuild": "cp -r README.md docs ./packages/plugin/dist/",
-    "postinstall": "node scripts/patch-graphql.js",
-    "prebuild": "rimraf packages/*/dist ./tsconfig.tsbuildinfo",
+    "postbuild": "cp -r README.md docs ./packages/plugin/dist/ && tsx scripts/postbuild.ts",
+    "postinstall": "tsx scripts/patch-graphql.ts",
+    "prebuild": "rimraf ./tsconfig.tsbuildinfo",
     "prerelease": "pnpm build",
     "prettier": "prettier --cache --write --list-different .",
     "release": "changeset publish",
-    "test": "vitest .",
-    "transpile-ts": "tsc --project tsconfig.json"
+    "test": "vitest ."
   },
   "devDependencies": {
     "@changesets/changelog-github": "0.4.8",
@@ -29,7 +28,7 @@
     "@types/node": "18.11.17",
     "@typescript-eslint/eslint-plugin": "5.47.0",
     "@typescript-eslint/parser": "5.47.0",
-    "bob-the-bundler": "1.7.3",
+    "bob-the-bundler": "4.2.0-alpha-20221222123652-a454f30",
     "chalk": "^4.1.2",
     "dedent": "0.7.0",
     "enquirer": "2.3.6",
@@ -49,7 +48,7 @@
     "rimraf": "3.0.2",
     "tsx": "^3.12.1",
     "typescript": "4.9.4",
-    "vitest": "0.26.1"
+    "vitest": "0.26.2"
   },
   "resolutions": {
     "graphql": "16.6.0"
@@ -65,7 +64,7 @@
       "[email protected]": "patches/[email protected]",
       "[email protected]": "patches/[email protected]",
       "[email protected]": "patches/[email protected]",
-      "[email protected].1": "patches/[email protected].1.patch"
+      "[email protected].2": "patches/[email protected].2.patch"
     }
   }
 }

packages/plugin/package.json

@@ -5,30 +5,32 @@
   "repository": "https://github.com/B2o5T/graphql-eslint",
   "author": "Dotan Simha <[email protected]>",
   "license": "MIT",
-  "main": "dist/index.js",
-  "module": "dist/index.mjs",
+  "main": "dist/cjs/index.js",
+  "module": "dist/esm/index.js",
   "exports": {
     "./package.json": "./package.json",
     ".": {
-      "require": "./dist/index.js",
-      "import": "./dist/index.mjs"
-    },
-    "./*": {
-      "require": "./dist/*.js",
-      "import": "./dist/*.mjs"
+      "require": {
+        "types": "./dist/typings/index.d.cts",
+        "default": "./dist/cjs/index.js"
+      },
+      "import": {
+        "types": "./dist/typings/index.d.ts",
+        "default": "./dist/esm/index.js"
+      },
+      "default": {
+        "types": "./dist/typings/index.d.ts",
+        "default": "./dist/esm/index.js"
+      }
     }
   },
-  "typings": "dist/index.d.ts",
+  "typings": "dist/typings/index.d.ts",
   "keywords": [
     "eslint",
     "eslintplugin",
     "eslint-plugin",
     "graphql"
   ],
-  "scripts": {
-    "prepack": "bob prepack",
-    "test": "jest"
-  },
   "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"
   },
@@ -59,48 +61,7 @@
     "access": "public"
   },
   "sideEffects": false,
-  "buildOptions": {
-    "input": "./src/index.ts",
-    "copy": "./src/configs",
-    "external": [
-      "eslint",
-      "graphql",
-      "graphql/validation/rules/ExecutableDefinitionsRule",
-      "graphql/validation/rules/FieldsOnCorrectTypeRule",
-      "graphql/validation/rules/FragmentsOnCompositeTypesRule",
-      "graphql/validation/rules/KnownArgumentNamesRule",
-      "graphql/validation/rules/KnownDirectivesRule",
-      "graphql/validation/rules/KnownFragmentNamesRule",
-      "graphql/validation/rules/KnownTypeNamesRule",
-      "graphql/validation/rules/LoneAnonymousOperationRule",
-      "graphql/validation/rules/LoneSchemaDefinition",
-      "graphql/validation/rules/NoFragmentCyclesRule",
-      "graphql/validation/rules/NoUndefinedVariablesRule",
-      "graphql/validation/rules/NoUnusedFragmentsRule",
-      "graphql/validation/rules/NoUnusedVariablesRule",
-      "graphql/validation/rules/OverlappingFieldsCanBeMergedRule",
-      "graphql/validation/rules/PossibleFragmentSpreadsRule",
-      "graphql/validation/rules/PossibleTypeExtensionsRule",
-      "graphql/validation/rules/ProvidedRequiredArgumentsRule",
-      "graphql/validation/rules/ScalarLeafsRule",
-      "graphql/validation/rules/SingleFieldSubscriptionsRule",
-      "graphql/validation/rules/UniqueArgumentNamesRule",
-      "graphql/validation/rules/UniqueDirectiveNamesRule",
-      "graphql/validation/rules/UniqueDirectivesPerLocationRule",
-      "graphql/validation/rules/UniqueEnumValueNamesRule",
-      "graphql/validation/rules/UniqueFieldDefinitionNamesRule",
-      "graphql/validation/rules/UniqueInputFieldNamesRule",
-      "graphql/validation/rules/UniqueOperationTypesRule",
-      "graphql/validation/rules/UniqueTypeNamesRule",
-      "graphql/validation/rules/UniqueVariableNamesRule",
-      "graphql/validation/rules/ValuesOfCorrectTypeRule",
-      "graphql/validation/rules/VariablesAreInputTypesRule",
-      "graphql/validation/rules/VariablesInAllowedPositionRule",
-      "graphql/language",
-      "minimatch"
-    ]
-  },
   "typescript": {
-    "definition": "dist/index.d.ts"
+    "definition": "dist/typings/index.d.ts"
   }
 }

packages/plugin/src/configs/base.json

@@ -0,0 +1,4 @@
+export default {
+  parser: '@graphql-eslint/eslint-plugin',
+  plugins: ['@graphql-eslint'],
+};

packages/plugin/src/configs/base.ts

@@ -0,0 +1,13 @@
+import schemaRecommendedConfig from './schema-recommended';
+import schemaAllConfig from './schema-all';
+import operationsRecommendedConfig from './operations-recommended';
+import operationsAllConfig from './operations-all';
+import relayConfig from './relay';
+
+export const configs = {
+  'schema-recommended': schemaRecommendedConfig,
+  'schema-all': schemaAllConfig,
+  'operations-recommended': operationsRecommendedConfig,
+  'operations-all': operationsAllConfig,
+  relay: relayConfig,
+};

packages/plugin/src/configs/index.ts

@@ -0,0 +1,29 @@
+/*
+ * 🚨 IMPORTANT! Do not manually modify this file. Run: `yarn generate-configs`
+ */
+
+export default {
+  extends: ['./configs/base', './configs/operations-recommended'],
+  rules: {
+    '@graphql-eslint/alphabetize': [
+      'error',
+      {
+        selections: ['OperationDefinition', 'FragmentDefinition'],
+        variables: ['OperationDefinition'],
+        arguments: ['Field', 'Directive'],
+      },
+    ],
+    '@graphql-eslint/lone-executable-definition': 'error',
+    '@graphql-eslint/match-document-filename': [
+      'error',
+      {
+        query: 'kebab-case',
+        mutation: 'kebab-case',
+        subscription: 'kebab-case',
+        fragment: 'kebab-case',
+      },
+    ],
+    '@graphql-eslint/unique-fragment-name': 'error',
+    '@graphql-eslint/unique-operation-name': 'error',
+  },
+};