@wistia/eslint-config 0.11.0 → 0.12.0

package/configs/eslint/default.js

@@ -14,16 +14,12 @@ module.exports = {
 
   env: {
     es2022: true,
-    node: true,
   },
 
   extends: [
     '../../rules/eslint/best-practices',
     '../../rules/eslint/errors',
-    '../../rules/eslint/es6',
     '../../rules/eslint/imports',
-    '../../rules/eslint/node',
     '../../rules/eslint/style',
-    '../../rules/eslint/variables',
   ].map(require.resolve),
 };

package/configs/eslint/node.js

@@ -0,0 +1,9 @@
+require('@rushstack/eslint-patch/modern-module-resolution');
+
+module.exports = {
+  env: {
+    node: true,
+  },
+
+  extends: ['../../rules/eslint/node'].map(require.resolve),
+};

package/configs/eslint/typescript.js

@@ -10,17 +10,13 @@ module.exports = {
 
   env: {
     es2022: true,
-    node: true,
   },
 
   extends: [
     '../../rules/eslint/best-practices',
     '../../rules/eslint/errors',
-    '../../rules/eslint/es6',
-    '../../rules/eslint/node',
     '../../rules/eslint/imports',
     '../../rules/eslint/style',
-    '../../rules/eslint/variables',
     '../../rules/eslint/typescript',
     '../../rules/eslint/typescript-imports',
   ].map(require.resolve),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "Wistia's ESLint configurations",
   "main": "react.js",
   "exports": {
@@ -8,6 +8,7 @@
     ".": "./configs/eslint/default.js",
     "./cypress": "./configs/eslint/cypress.js",
     "./jest": "./configs/eslint/jest.js",
+    "./node": "./configs/eslint/node.js",
     "./prettier": "./configs/eslint/prettier.js",
     "./react": "./configs/eslint/react.js",
     "./strict": "./configs/eslint/strict.js",
@@ -43,6 +44,7 @@
     "eslint-plugin-jest-dom": "^4.0.2",
     "eslint-plugin-jest-formatting": "^3.1.0",
     "eslint-plugin-jsx-a11y": "^6.6.0",
+    "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^4.2.1",
     "eslint-plugin-promise": "^6.0.0",
     "eslint-plugin-react": "^7.30.1",

package/rules/eslint/best-practices.js

@@ -31,6 +31,10 @@ module.exports = {
     // https://eslint.org/docs/rules/consistent-return
     'consistent-return': 'error',
 
+    // verify super() callings in constructors
+    // https://eslint.org/docs/rules/constructor-super
+    'constructor-super': 'error',
+
     // specify curly brace conventions for all control statements
     // https://eslint.org/docs/rules/curly
     curly: ['error', 'multi-line'], // multiline
@@ -66,6 +70,10 @@ module.exports = {
     // https://eslint.org/docs/rules/guard-for-in
     'guard-for-in': 'error',
 
+    // enforce or disallow variable initializations at definition
+    // https://eslint.org/docs/rules/init-declarations
+    'init-declarations': 'off',
+
     // enforce a maximum number of classes per file
     // https://eslint.org/docs/rules/max-classes-per-file
     'max-classes-per-file': ['error', 1],
@@ -82,6 +90,23 @@ module.exports = {
     // https://eslint.org/docs/rules/no-case-declarations
     'no-case-declarations': 'error',
 
+    // disallow modifying variables of class declarations
+    // https://eslint.org/docs/rules/no-class-assign
+    'no-class-assign': 'error',
+
+    // disallow arrow functions where they could be confused with comparisons
+    // https://eslint.org/docs/rules/no-confusing-arrow
+    'no-confusing-arrow': [
+      'error',
+      {
+        allowParens: true,
+      },
+    ],
+
+    // disallow modifying variables that are declared using const
+    // https://eslint.org/docs/rules/no-const-assign
+    'no-const-assign': 'error',
+
     // Disallow returning value in constructor
     // https://eslint.org/docs/rules/no-constructor-return
     'no-constructor-return': 'error',
@@ -90,6 +115,15 @@ module.exports = {
     // https://eslint.org/docs/rules/no-div-regex
     'no-div-regex': 'off',
 
+    // disallow duplicate class members
+    // https://eslint.org/docs/rules/no-dupe-class-members
+    'no-dupe-class-members': 'error',
+
+    // disallow importing from the same path more than once
+    // https://eslint.org/docs/rules/no-duplicate-imports
+    // replaced by https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
+    'no-duplicate-imports': 'off',
+
     // disallow else after a return in an if
     // https://eslint.org/docs/rules/no-else-return
     'no-else-return': ['error', { allowElseIf: false }],
@@ -143,6 +177,10 @@ module.exports = {
     // https://eslint.org/docs/rules/no-native-reassign
     'no-native-reassign': 'off',
 
+    // disallow symbol constructor
+    // https://eslint.org/docs/rules/no-new-symbol
+    'no-new-symbol': 'error',
+
     // disallow implicit type conversions
     // https://eslint.org/docs/rules/no-implicit-coercion
     'no-implicit-coercion': [
@@ -264,6 +302,18 @@ module.exports = {
     // https://eslint.org/docs/rules/no-redeclare
     'no-redeclare': 'error',
 
+    // Disallow specified names in exports
+    // https://eslint.org/docs/rules/no-restricted-exports
+    'no-restricted-exports': [
+      'error',
+      {
+        restrictedNamedExports: [
+          'default', // use `export default` to provide a default export
+          'then', // this will cause tons of confusion when your module is dynamically `import()`ed, and will break in most node ESM versions
+        ],
+      },
+    ],
+
     // browser globals that commonly cause confusion and are not recommended
     // to use without an explicit `window`.` qualifier
     // https://eslint.org/docs/rules/no-restricted-globals
@@ -281,6 +331,16 @@ module.exports = {
       },
     ].concat(confusingBrowserGlobals),
 
+    // disallow specific imports
+    // https://eslint.org/docs/rules/no-restricted-imports
+    'no-restricted-imports': [
+      'off',
+      {
+        paths: [],
+        patterns: [],
+      },
+    ],
+
     // disallow certain object properties
     // https://eslint.org/docs/rules/no-restricted-properties
     'no-restricted-properties': [
@@ -364,6 +424,10 @@ module.exports = {
     // https://eslint.org/docs/rules/no-sequences
     'no-sequences': 'error',
 
+    // disallow to use this/super before super() calling in constructors.
+    // https://eslint.org/docs/rules/no-this-before-super
+    'no-this-before-super': 'error',
+
     // restrict what can be thrown as an exception
     // https://eslint.org/docs/rules/no-throw-literal
     'no-throw-literal': 'error',
@@ -399,6 +463,14 @@ module.exports = {
     // https://eslint.org/docs/rules/no-useless-concat
     'no-useless-concat': 'error',
 
+    // disallow useless computed property keys
+    // https://eslint.org/docs/rules/no-useless-computed-key
+    'no-useless-computed-key': 'error',
+
+    // disallow unnecessary constructor
+    // https://eslint.org/docs/rules/no-useless-constructor
+    'no-useless-constructor': 'error',
+
     // disallow unnecessary string escaping
     // https://eslint.org/docs/rules/no-useless-escape
     'no-useless-escape': 'error',
@@ -407,6 +479,21 @@ module.exports = {
     // https://eslint.org/docs/rules/no-useless-return
     'no-useless-return': 'error',
 
+    // disallow renaming import, export, and destructured assignments to the same name
+    // https://eslint.org/docs/rules/no-useless-rename
+    'no-useless-rename': [
+      'error',
+      {
+        ignoreDestructuring: false,
+        ignoreImport: false,
+        ignoreExport: false,
+      },
+    ],
+
+    // require let or const instead of var
+    // https://eslint.org/docs/rules/no-var
+    'no-var': 'error',
+
     // disallow use of void operator
     // https://eslint.org/docs/rules/no-void
     'no-void': 'error',
@@ -419,18 +506,112 @@ module.exports = {
     // https://eslint.org/docs/rules/no-with
     'no-with': 'error',
 
-    // require using Error objects as Promise rejection reasons
-    // https://eslint.org/docs/rules/prefer-promise-reject-errors
-    'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
+    // disallow the catch clause parameter name being the same as a variable in the outer scope
+    // https://eslint.org/docs/rules/no-catch-shadow
+    'no-catch-shadow': 'off',
+
+    // disallow deletion of variables
+    // https://eslint.org/docs/rules/no-delete-var
+    'no-delete-var': 'error',
+
+    // disallow labels that share a name with a variable
+    // https://eslint.org/docs/rules/no-label-var
+    'no-label-var': 'error',
+
+    // disallow declaration of variables already declared in the outer scope
+    // https://eslint.org/docs/rules/no-shadow
+    'no-shadow': 'error',
+
+    // disallow shadowing of names such as arguments
+    // https://eslint.org/docs/rules/no-shadow-restricted-names
+    'no-shadow-restricted-names': 'error',
+
+    // disallow use of undeclared variables unless mentioned in a /*global */ block
+    // https://eslint.org/docs/rules/no-undef
+    'no-undef': 'error',
+
+    // disallow use of undefined when initializing variables
+    // https://eslint.org/docs/rules/no-undef-init
+    'no-undef-init': 'error',
+
+    // disallow use of undefined variable
+    // https://eslint.org/docs/rules/no-undefined
+    'no-undefined': 'off',
+
+    // disallow declaration of variables that are not used in the code
+    // https://eslint.org/docs/rules/no-unused-vars
+    'no-unused-vars': ['error', { vars: 'all', args: 'after-used', ignoreRestSiblings: true }],
+
+    // disallow use of variables before they are defined
+    // https://eslint.org/docs/rules/no-use-before-define
+    'no-use-before-define': ['error', { functions: true, classes: true, variables: true }],
+
+    // require method and property shorthand syntax for object literals
+    // https://eslint.org/docs/rules/object-shorthand
+    'object-shorthand': [
+      'error',
+      'always',
+      {
+        ignoreConstructors: false,
+        avoidQuotes: true,
+      },
+    ],
+
+    // suggest using arrow functions as callbacks
+    // https://eslint.org/docs/rules/prefer-arrow-callback
+    'prefer-arrow-callback': [
+      'error',
+      {
+        allowNamedFunctions: false,
+        allowUnboundThis: true,
+      },
+    ],
+
+    // suggest using of const declaration for variables that are never modified after declared
+    // https://eslint.org/docs/rules/prefer-const
+    'prefer-const': [
+      'error',
+      {
+        destructuring: 'any',
+        ignoreReadBeforeAssign: true,
+      },
+    ],
+
+    // Prefer destructuring from arrays and objects
+    // https://eslint.org/docs/rules/prefer-destructuring
+    'prefer-destructuring': [
+      'error',
+      {
+        VariableDeclarator: {
+          array: false,
+          object: true,
+        },
+        AssignmentExpression: {
+          array: true,
+          object: false,
+        },
+      },
+      {
+        enforceForRenamedProperties: false,
+      },
+    ],
 
     // Suggest using named capture group in regular expression
     // https://eslint.org/docs/rules/prefer-named-capture-group
     'prefer-named-capture-group': 'off',
 
+    // disallow parseInt() in favor of binary, octal, and hexadecimal literals
+    // https://eslint.org/docs/rules/prefer-numeric-literals
+    'prefer-numeric-literals': 'error',
+
     // Prefer Object.hasOwn() over Object.prototype.hasOwnProperty.call()
     // https://eslint.org/docs/rules/prefer-object-has-own
     'prefer-object-has-own': 'error',
 
+    // require using Error objects as Promise rejection reasons
+    // https://eslint.org/docs/rules/prefer-promise-reject-errors
+    'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
+
     // https://eslint.org/docs/rules/prefer-regex-literals
     'prefer-regex-literals': [
       'error',
@@ -439,6 +620,22 @@ module.exports = {
       },
     ],
 
+    // suggest using Reflect methods where applicable
+    // https://eslint.org/docs/rules/prefer-reflect
+    'prefer-reflect': 'off',
+
+    // use rest parameters instead of arguments
+    // https://eslint.org/docs/rules/prefer-rest-params
+    'prefer-rest-params': 'error',
+
+    // suggest using the spread syntax instead of .apply()
+    // https://eslint.org/docs/rules/prefer-spread
+    'prefer-spread': 'error',
+
+    // suggest using template literals instead of string concatenation
+    // https://eslint.org/docs/rules/prefer-template
+    'prefer-template': 'error',
+
     // require use of the second argument for parseInt()
     // https://eslint.org/docs/rules/radix
     radix: 'error',
@@ -451,9 +648,29 @@ module.exports = {
     // https://eslint.org/docs/rules/require-unicode-regexp
     'require-unicode-regexp': 'off',
 
+    // disallow generator functions that do not have yield
+    // https://eslint.org/docs/rules/require-yield
+    'require-yield': 'error',
+
+    // import sorting
+    // https://eslint.org/docs/rules/sort-imports
+    'sort-imports': [
+      'off',
+      {
+        ignoreCase: false,
+        ignoreDeclarationSort: false,
+        ignoreMemberSort: false,
+        memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single'],
+      },
+    ],
+
     // compiler/transpiler inserts `'use strict';` for us
     strict: ['error', 'never'],
 
+    // require a Symbol description
+    // https://eslint.org/docs/rules/symbol-description
+    'symbol-description': 'error',
+
     // requires to declare all vars on top of their containing scope
     // https://eslint.org/docs/rules/vars-on-top
     'vars-on-top': 'error',

package/rules/eslint/node.js

@@ -1,4 +1,9 @@
+// only add node rules
+// see: https://github.com/mysticatea/eslint-plugin-node#-rules
+
 module.exports = {
+  plugins: ['eslint-plugin-node'],
+
   rules: {
     // enforce return after a callback
     // https://eslint.org/docs/rules/callback-return
@@ -38,10 +43,161 @@ module.exports = {
 
     // restrict usage of specified node modules
     // https://eslint.org/docs/rules/no-restricted-modules
+    // deprecated in eslint 7
     'no-restricted-modules': 'off',
 
     // disallow use of synchronous methods
     // https://eslint.org/docs/rules/no-sync
     'no-sync': 'off',
+
+    // require error handling in callbacks
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/handle-callback-err
+    'node/handle-callback-err': 'error',
+
+    // ensure Node.js-style error-first callback pattern is followed
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-callback-literal
+    'node/no-callback-literal': 'error',
+
+    // disallow the assignment to exports
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-exports-assign
+    'node/no-exports-assign': 'error',
+
+    // disallow import declarations which import extraneous modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-extraneous-import
+    'node/no-extraneous-import': 'error',
+
+    // disallow require() expressions which import extraneous modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-extraneous-require
+    'node/no-extraneous-require': 'error',
+
+    // disallow import declarations which import non-existence modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-missing-import
+    'node/no-missing-import': 'error',
+
+    // disallow require() expressions which import non-existence modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-missing-require
+    'node/no-missing-require': 'error',
+
+    // disallow new operators with calls to require
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-new-require
+    'node/no-new-require': 'error',
+
+    // disallow string concatenation with __dirname and __filename
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-path-concat
+    'node/no-path-concat': 'error',
+
+    // disallow the use of process.exit()
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-process-exit
+    'node/no-process-exit': 'error',
+
+    // disallow bin files that npm ignores
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-bin
+    'node/no-unpublished-bin': 'error',
+
+    // disallow import declarations which import private modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-import
+    'node/no-unpublished-import': 'error',
+
+    // disallow require() expressions which import private modules
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-require
+    'node/no-unpublished-require': 'error',
+
+    // disallow unsupported ECMAScript built-ins on the specified version
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unsupported-features/es-builtins
+    'node/no-unsupported-features/es-builtins': 'error',
+
+    // disallow unsupported ECMAScript syntax on the specified version
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unsupported-features/es-syntax
+    'node/no-unsupported-features/es-syntax': ['error', { ignores: ['modules'] }],
+
+    // disallow unsupported Node.js built-in APIs on the specified version
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unsupported-features/node-builtins
+    'node/no-unsupported-features/node-builtins': 'error',
+
+    // make process.exit() expressions the same code path as throw
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/process-exit-as-throw
+    'node/process-exit-as-throw': 'error',
+
+    // suggest correct usage of shebang
+    // decision: turned this off because it expects files to be listed under `bin` in package.json
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/shebang
+    'node/shebang': 'off',
+
+    // disallow deprecated APIs
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-deprecated-api
+    'node/no-deprecated-api': 'error',
+
+    // require return statements after callbacks
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/callback-return
+    'node/callback-return': 'error',
+
+    // enforce either module.exports or exports
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/exports-style
+    'node/exports-style': 'error',
+
+    // enforce the style of file extensions in import declarations
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/file-extension-in-import
+    'node/file-extension-in-import': 'error',
+
+    // require require() calls to be placed at top-level module scope
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/global-require
+    'node/global-require': 'error',
+
+    // disallow require calls to be mixed with regular variable declarations
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-mixed-requires
+    'node/no-mixed-requires': 'error',
+
+    // disallow the use of process.env
+    // decision: process.env is used by dotenv
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-process-env
+    'node/no-process-env': 'off',
+
+    // disallow specified modules when loaded by import declarations
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-restricted-import
+    'node/no-restricted-import': 'error',
+
+    // disallow specified modules when loaded by require
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-restricted-require
+    'node/no-restricted-require': 'error',
+
+    // disallow synchronous methods
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-sync
+    'node/no-sync': 'error',
+
+    // enforce either Buffer or require("buffer").Buffer
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/buffer
+    'node/prefer-global/buffer': 'error',
+
+    // enforce either console or require("console")
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/console
+    'node/prefer-global/console': 'error',
+
+    // enforce either process or require("process")
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/process
+    'node/prefer-global/process': 'error',
+
+    // enforce either TextDecoder or require("util").TextDecoder
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/text-decoder
+    'node/prefer-global/text-decoder': 'error',
+
+    // enforce either TextEncoder or require("util").TextEncoder
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/text-encoder
+    'node/prefer-global/text-encoder': 'error',
+
+    // enforce either URLSearchParams or require("url").URLSearchParams
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/url-search-params
+    'node/prefer-global/url-search-params': 'error',
+
+    // enforce either URL or require("url").URL
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-global/url
+    'node/prefer-global/url': 'error',
+
+    // enforce require("dns").promises
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-promises/dns
+    'node/prefer-promises/dns': 'error',
+
+    // enforce require("fs").promises
+    // https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/prefer-promises/fs
+    'node/prefer-promises/fs': 'error',
   },
 };

package/rules/eslint/style.js

@@ -14,6 +14,24 @@ module.exports = {
     // https://eslint.org/docs/rules/array-bracket-spacing
     'array-bracket-spacing': ['error', 'never'],
 
+    // enforces no braces where they can be omitted
+    // https://eslint.org/docs/rules/arrow-body-style
+    'arrow-body-style': [
+      'error',
+      'as-needed',
+      {
+        requireReturnForObjectLiteral: false,
+      },
+    ],
+
+    // require parens in arrow function arguments
+    // https://eslint.org/docs/rules/arrow-parens
+    'arrow-parens': ['error', 'always'],
+
+    // require space before/after arrow function's arrow
+    // https://eslint.org/docs/rules/arrow-spacing
+    'arrow-spacing': ['error', { before: true, after: true }],
+
     // enforce spacing inside single-line blocks
     // https://eslint.org/docs/rules/block-spacing
     'block-spacing': ['error', 'always'],
@@ -128,6 +146,10 @@ module.exports = {
     // https://eslint.org/docs/rules/function-paren-newline
     'function-paren-newline': ['error', 'multiline-arguments'],
 
+    // enforce the spacing around the * in generator functions
+    // https://eslint.org/docs/rules/generator-star-spacing
+    'generator-star-spacing': ['error', { before: false, after: true }],
+
     // disallow specified identifiers
     // https://eslint.org/docs/rules/id-denylist
     'id-denylist': 'off',
@@ -563,6 +585,10 @@ module.exports = {
     // https://eslint.org/docs/rules/quotes
     quotes: ['error', 'single', { avoidEscape: true }],
 
+    // enforce spacing between object rest-spread
+    // https://eslint.org/docs/rules/rest-spread-spacing
+    'rest-spread-spacing': ['error', 'never'],
+
     // do not require jsdoc
     // https://eslint.org/docs/rules/require-jsdoc
     'require-jsdoc': 'off',
@@ -643,6 +669,10 @@ module.exports = {
     // https://eslint.org/docs/rules/switch-colon-spacing
     'switch-colon-spacing': ['error', { after: true, before: false }],
 
+    // enforce usage of spacing in template strings
+    // https://eslint.org/docs/rules/template-curly-spacing
+    'template-curly-spacing': 'error',
+
     // Require or disallow spacing between template tags and their literals
     // https://eslint.org/docs/rules/template-tag-spacing
     'template-tag-spacing': ['error', 'never'],
@@ -654,5 +684,9 @@ module.exports = {
     // require regex literals to be wrapped in parentheses
     // https://eslint.org/docs/rules/wrap-regex
     'wrap-regex': 'off',
+
+    // enforce spacing around the * in yield* expressions
+    // https://eslint.org/docs/rules/yield-star-spacing
+    'yield-star-spacing': ['error', 'after'],
   },
 };

package/rules/eslint/es6.js

@@ -1,210 +0,0 @@
-module.exports = {
-  rules: {
-    // enforces no braces where they can be omitted
-    // https://eslint.org/docs/rules/arrow-body-style
-    'arrow-body-style': [
-      'error',
-      'as-needed',
-      {
-        requireReturnForObjectLiteral: false,
-      },
-    ],
-
-    // require parens in arrow function arguments
-    // https://eslint.org/docs/rules/arrow-parens
-    'arrow-parens': ['error', 'always'],
-
-    // require space before/after arrow function's arrow
-    // https://eslint.org/docs/rules/arrow-spacing
-    'arrow-spacing': ['error', { before: true, after: true }],
-
-    // verify super() callings in constructors
-    // https://eslint.org/docs/rules/constructor-super
-    'constructor-super': 'error',
-
-    // enforce the spacing around the * in generator functions
-    // https://eslint.org/docs/rules/generator-star-spacing
-    'generator-star-spacing': ['error', { before: false, after: true }],
-
-    // disallow modifying variables of class declarations
-    // https://eslint.org/docs/rules/no-class-assign
-    'no-class-assign': 'error',
-
-    // disallow arrow functions where they could be confused with comparisons
-    // https://eslint.org/docs/rules/no-confusing-arrow
-    'no-confusing-arrow': [
-      'error',
-      {
-        allowParens: true,
-      },
-    ],
-
-    // disallow modifying variables that are declared using const
-    // https://eslint.org/docs/rules/no-const-assign
-    'no-const-assign': 'error',
-
-    // disallow duplicate class members
-    // https://eslint.org/docs/rules/no-dupe-class-members
-    'no-dupe-class-members': 'error',
-
-    // disallow importing from the same path more than once
-    // https://eslint.org/docs/rules/no-duplicate-imports
-    // replaced by https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
-    'no-duplicate-imports': 'off',
-
-    // disallow symbol constructor
-    // https://eslint.org/docs/rules/no-new-symbol
-    'no-new-symbol': 'error',
-
-    // Disallow specified names in exports
-    // https://eslint.org/docs/rules/no-restricted-exports
-    'no-restricted-exports': [
-      'error',
-      {
-        restrictedNamedExports: [
-          'default', // use `export default` to provide a default export
-          'then', // this will cause tons of confusion when your module is dynamically `import()`ed, and will break in most node ESM versions
-        ],
-      },
-    ],
-
-    // disallow specific imports
-    // https://eslint.org/docs/rules/no-restricted-imports
-    'no-restricted-imports': [
-      'off',
-      {
-        paths: [],
-        patterns: [],
-      },
-    ],
-
-    // disallow to use this/super before super() calling in constructors.
-    // https://eslint.org/docs/rules/no-this-before-super
-    'no-this-before-super': 'error',
-
-    // disallow useless computed property keys
-    // https://eslint.org/docs/rules/no-useless-computed-key
-    'no-useless-computed-key': 'error',
-
-    // disallow unnecessary constructor
-    // https://eslint.org/docs/rules/no-useless-constructor
-    'no-useless-constructor': 'error',
-
-    // disallow renaming import, export, and destructured assignments to the same name
-    // https://eslint.org/docs/rules/no-useless-rename
-    'no-useless-rename': [
-      'error',
-      {
-        ignoreDestructuring: false,
-        ignoreImport: false,
-        ignoreExport: false,
-      },
-    ],
-
-    // require let or const instead of var
-    // https://eslint.org/docs/rules/no-var
-    'no-var': 'error',
-
-    // require method and property shorthand syntax for object literals
-    // https://eslint.org/docs/rules/object-shorthand
-    'object-shorthand': [
-      'error',
-      'always',
-      {
-        ignoreConstructors: false,
-        avoidQuotes: true,
-      },
-    ],
-
-    // suggest using arrow functions as callbacks
-    // https://eslint.org/docs/rules/prefer-arrow-callback
-    'prefer-arrow-callback': [
-      'error',
-      {
-        allowNamedFunctions: false,
-        allowUnboundThis: true,
-      },
-    ],
-
-    // suggest using of const declaration for variables that are never modified after declared
-    // https://eslint.org/docs/rules/prefer-const
-    'prefer-const': [
-      'error',
-      {
-        destructuring: 'any',
-        ignoreReadBeforeAssign: true,
-      },
-    ],
-
-    // Prefer destructuring from arrays and objects
-    // https://eslint.org/docs/rules/prefer-destructuring
-    'prefer-destructuring': [
-      'error',
-      {
-        VariableDeclarator: {
-          array: false,
-          object: true,
-        },
-        AssignmentExpression: {
-          array: true,
-          object: false,
-        },
-      },
-      {
-        enforceForRenamedProperties: false,
-      },
-    ],
-
-    // disallow parseInt() in favor of binary, octal, and hexadecimal literals
-    // https://eslint.org/docs/rules/prefer-numeric-literals
-    'prefer-numeric-literals': 'error',
-
-    // suggest using Reflect methods where applicable
-    // https://eslint.org/docs/rules/prefer-reflect
-    'prefer-reflect': 'off',
-
-    // use rest parameters instead of arguments
-    // https://eslint.org/docs/rules/prefer-rest-params
-    'prefer-rest-params': 'error',
-
-    // suggest using the spread syntax instead of .apply()
-    // https://eslint.org/docs/rules/prefer-spread
-    'prefer-spread': 'error',
-
-    // suggest using template literals instead of string concatenation
-    // https://eslint.org/docs/rules/prefer-template
-    'prefer-template': 'error',
-
-    // disallow generator functions that do not have yield
-    // https://eslint.org/docs/rules/require-yield
-    'require-yield': 'error',
-
-    // enforce spacing between object rest-spread
-    // https://eslint.org/docs/rules/rest-spread-spacing
-    'rest-spread-spacing': ['error', 'never'],
-
-    // import sorting
-    // https://eslint.org/docs/rules/sort-imports
-    'sort-imports': [
-      'off',
-      {
-        ignoreCase: false,
-        ignoreDeclarationSort: false,
-        ignoreMemberSort: false,
-        memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single'],
-      },
-    ],
-
-    // require a Symbol description
-    // https://eslint.org/docs/rules/symbol-description
-    'symbol-description': 'error',
-
-    // enforce usage of spacing in template strings
-    // https://eslint.org/docs/rules/template-curly-spacing
-    'template-curly-spacing': 'error',
-
-    // enforce spacing around the * in yield* expressions
-    // https://eslint.org/docs/rules/yield-star-spacing
-    'yield-star-spacing': ['error', 'after'],
-  },
-};

package/rules/eslint/variables.js

@@ -1,47 +0,0 @@
-module.exports = {
-  rules: {
-    // enforce or disallow variable initializations at definition
-    // https://eslint.org/docs/rules/init-declarations
-    'init-declarations': 'off',
-
-    // disallow the catch clause parameter name being the same as a variable in the outer scope
-    // https://eslint.org/docs/rules/no-catch-shadow
-    'no-catch-shadow': 'off',
-
-    // disallow deletion of variables
-    // https://eslint.org/docs/rules/no-delete-var
-    'no-delete-var': 'error',
-
-    // disallow labels that share a name with a variable
-    // https://eslint.org/docs/rules/no-label-var
-    'no-label-var': 'error',
-
-    // disallow declaration of variables already declared in the outer scope
-    // https://eslint.org/docs/rules/no-shadow
-    'no-shadow': 'error',
-
-    // disallow shadowing of names such as arguments
-    // https://eslint.org/docs/rules/no-shadow-restricted-names
-    'no-shadow-restricted-names': 'error',
-
-    // disallow use of undeclared variables unless mentioned in a /*global */ block
-    // https://eslint.org/docs/rules/no-undef
-    'no-undef': 'error',
-
-    // disallow use of undefined when initializing variables
-    // https://eslint.org/docs/rules/no-undef-init
-    'no-undef-init': 'error',
-
-    // disallow use of undefined variable
-    // https://eslint.org/docs/rules/no-undefined
-    'no-undefined': 'off',
-
-    // disallow declaration of variables that are not used in the code
-    // https://eslint.org/docs/rules/no-unused-vars
-    'no-unused-vars': ['error', { vars: 'all', args: 'after-used', ignoreRestSiblings: true }],
-
-    // disallow use of variables before they are defined
-    // https://eslint.org/docs/rules/no-use-before-define
-    'no-use-before-define': ['error', { functions: true, classes: true, variables: true }],
-  },
-};