@vertexvis/eslint-config-vertexvis 0.7.2 → 0.8.0

package/README.md

@@ -17,24 +17,12 @@ Add this package and ESLint as `devDependencies` to your `package.json`.
 }
 ```
 
-Add a `.eslintrc.js` file to the root of your project that extends this config.
-
-This package also includes an optional patch file which monkey patches ESLint's
-module resolution for plugins. ESLint resolves plugins relative to the consuming
-package. So any plugins referenced by a sharable config will need to NPM
-installed in the consuming package. There's an [open
-issue](https://github.com/eslint/eslint/issues/3458#issuecomment-516716165) and
-discussion to change this behavior to resolve plugins relative to the imported
-config's `node_modules` folder. This shim mimics this behavior.
+Add an `eslint.config.js` file to the root of your project that imports this
+config.
 
 ```js
-// .eslintrc.js
-
-// Include the optional patch which resolves ESLint plugins relative to the
-// sharable config package.
-require('@vertexvis/eslint-config-vertexvis/patch');
+// eslint.config.js
+import vertexvis from '@vertexvis/eslint-config-vertexvis';
 
-module.exports = {
-  extends: '@vertexvis/vertexvis',
-};
+export default [...vertexvis];
 ```

package/index.js

@@ -1,31 +1,38 @@
-module.exports = {
-  parserOptions: {
-    ecmaVersion: 6,
+import eslintConfigPrettier from 'eslint-config-prettier';
+import simpleImportSort from 'eslint-plugin-simple-import-sort';
+
+export default [
+  {
+    languageOptions: {
+      ecmaVersion: 2019,
+      sourceType: 'module',
+    },
+    plugins: {
+      'simple-import-sort': simpleImportSort,
+    },
+    rules: {
+      eqeqeq: [2, 'smart'],
+      'func-call-spacing': 'off',
+      indent: ['error', 2],
+      'lines-between-class-members': [
+        'error',
+        'always',
+        { exceptAfterSingleLine: true },
+      ],
+      'padding-line-between-statements': [
+        'error',
+        { blankLine: 'always', prev: 'function', next: 'function' },
+        { blankLine: 'always', prev: 'class', next: 'class' },
+        {
+          blankLine: 'always',
+          prev: 'import',
+          next: ['export', 'class', 'function', 'const', 'let', 'var'],
+        },
+      ],
+      'simple-import-sort/imports': 'error',
+      'simple-import-sort/exports': 'error',
+      yoda: 2,
+    },
   },
-  plugins: ['prettier', 'simple-import-sort'],
-  extends: ['plugin:prettier/recommended'],
-  rules: {
-    eqeqeq: [2, 'smart'],
-    'func-call-spacing': 'off',
-    indent: ['error', 2],
-    'lines-between-class-members': [
-      'error',
-      'always',
-      { exceptAfterSingleLine: true },
-    ],
-    'padding-line-between-statements': [
-      'error',
-      { blankLine: 'always', prev: 'function', next: 'function' },
-      { blankLine: 'always', prev: 'class', next: 'class' },
-      {
-        blankLine: 'always',
-        prev: 'import',
-        next: ['export', 'class', 'function', 'const', 'let', 'var'],
-      },
-    ],
-    'prettier/prettier': 'error',
-    'simple-import-sort/imports': 'error',
-    'simple-import-sort/exports': 'error',
-    yoda: 2,
-  },
-};
+  eslintConfigPrettier,
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vertexvis/eslint-config-vertexvis",
-  "version": "0.7.2",
+  "version": "0.8.0",
   "description": "The Vertex sharable ESLint config",
   "license": "MIT",
   "author": "Vertex Developers <support@vertexvis.com> (https://developer.vertexvis.com)",
@@ -16,24 +16,28 @@
     "registry": "https://registry.npmjs.org/",
     "access": "public"
   },
+  "type": "module",
   "main": "./index.js",
+  "exports": {
+    ".": "./index.js",
+    "./package.json": "./package.json"
+  },
   "scripts": {
     "format": "yarn lint --fix",
-    "lint": "eslint --ext .ts,.tsx,.js,.jsx --ignore-path ../../.gitignore .",
+    "lint": "eslint .",
     "build": "exit 0",
     "test": "echo 'No unit tests defined'",
     "test:coverage": "echo 'No unit tests defined'"
   },
   "dependencies": {
-    "eslint-config-prettier": "^9.0.0",
-    "eslint-plugin-prettier": "^5.0.0",
-    "eslint-plugin-simple-import-sort": "^10.0.0",
-    "prettier": "^3.0.3"
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-simple-import-sort": "^13.0.0",
+    "prettier": "^3.8.4"
   },
   "peerDependencies": {
-    "eslint": ">=7.0.0"
+    "eslint": ">=8.0.0"
   },
   "devDependencies": {
-    "eslint": "^8.6.0"
+    "eslint": "^8.57.0"
   }
 }

package/.eslintrc.js

@@ -1,3 +0,0 @@
-module.exports = {
-  extends: './index.js',
-};

package/patch.js

@@ -1,62 +0,0 @@
-// This monkey patches eslint to lookup ESLint plugins relative to a sharable
-// config's NPM package. This allows a consuming package to use a sharable
-// config's plugins without having to add the plugins as NPM dependencies.
-//
-// To use, require this patch from your .eslintrc.js file.
-//
-// ```ts
-// require('@vertexvis/eslint-config-vertexvis');
-//
-// module.exports = {
-//   extends: '@vertexvis/vertexvis'
-// };
-// ```
-
-const path = require('path');
-
-let currentModule = module;
-while (
-  !/[\\/]eslint[\\/]lib[\\/]cli-engine[\\/]config-array-factory\.js/i.test(
-    currentModule.filename
-  )
-) {
-  if (!currentModule.parent) {
-    // This was tested with ESLint 6.1.0; other versions may not work
-    throw new Error(
-      'Failed to patch ESLint because the calling module was not recognized'
-    );
-  }
-  currentModule = currentModule.parent;
-}
-const eslintFolder = path.join(path.dirname(currentModule.filename), '../..');
-
-const configArrayFactoryPath = path.join(
-  eslintFolder,
-  'lib/cli-engine/config-array-factory'
-);
-const configArrayFactoryModule = require(configArrayFactoryPath);
-
-const moduleResolverPath = path.join(
-  eslintFolder,
-  'lib/shared/relative-module-resolver'
-);
-const ModuleResolver = require(moduleResolverPath);
-
-const originalLoadPlugin =
-  configArrayFactoryModule.ConfigArrayFactory.prototype._loadPlugin;
-configArrayFactoryModule.ConfigArrayFactory.prototype._loadPlugin = function (
-  name,
-  importerPath,
-  importerName
-) {
-  const originalResolve = ModuleResolver.resolve;
-  try {
-    ModuleResolver.resolve = function (moduleName, relativeToPath) {
-      // resolve using importerPath instead of relativeToPath
-      return originalResolve.call(this, moduleName, importerPath);
-    };
-    return originalLoadPlugin.apply(this, arguments);
-  } finally {
-    ModuleResolver.resolve = originalResolve;
-  }
-};