@w5s/eslint-config 1.0.0-alpha.2 → 1.0.0-alpha.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@w5s/eslint-config",
-  "version": "1.0.0-alpha.2",
+  "version": "1.0.0-alpha.6",
   "description": "ESLint configuration presets",
   "keywords": [
     "eslint",
@@ -45,6 +45,7 @@
     "@babel/plugin-syntax-jsx": "^7.0.0",
     "@typescript-eslint/eslint-plugin": "^5.0.0",
     "@typescript-eslint/parser": "^5.0.0",
+    "eslint-config-airbnb-base": "^15.0.0",
     "eslint-config-prettier": "^8.0.0",
     "eslint-plugin-functional": "^4.2.0",
     "eslint-plugin-import": "^2.25.0",
@@ -71,10 +72,16 @@
   },
   "peerDependencies": {
     "eslint": "5.x || 6.x || 7.x || 8.x",
-    "prettier": "2.x"
+    "prettier": "2.x",
+    "typescript": "4.x"
+  },
+  "peerDependenciesMeta": {
+    "typescript": {
+      "optional": true
+    }
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "1fd5f03e6f071785cc5be4f9a57430485cf9d6fc"
+  "gitHead": "79cf1b971006e8a0255763db12dd46938d228c78"
 }

package/rules/_rule.js

@@ -7,8 +7,71 @@ const warn = 'warn';
 /** @type {'off'} */
 const off = 'off';
 
+/**
+ * @typedef {{
+ *  env?: Record<string, boolean>,
+ *  extends?: string[]|string,
+ *  plugins?: string[]|string,
+ *  rules?: Record<string, unknown>,
+ *  settings?: Record<string, unknown>,
+ * }} ESLintConfigInit
+ */
+/**
+ * @typedef {{
+ *  env: Record<string, boolean>,
+ *  extends: string[],
+ *  plugins: string[],
+ *  rules: Record<string, unknown>,
+ *  settings: Record<string, unknown>,
+ * }} ESLintConfig
+ */
+
+/**
+ * @template T
+ * @type {(value: T[]|T|undefined) => T[]} */
+function toArray(value) {
+  if (value == null) {
+    return [];
+  }
+  if (Array.isArray(value)) {
+    return value;
+  }
+  return [value];
+}
+
+/**
+ * @template T
+ * @type {(left: T[]|T|undefined, right:T[]|T|undefined) => T[]}
+ */
+function concatArray(left, right) {
+  return toArray(left).concat(toArray(right));
+}
+
+/** @type {(...configs: ESLintConfigInit[]) => ESLintConfig} */
+function concatESConfig(...configs) {
+  return configs.reduce(
+    (/** @type {ESLintConfig} */ returnValue, /** @type {ESLintConfigInit} */ config) =>
+      Object.assign({}, returnValue, config, {
+        env: Object.assign({}, returnValue.env, config.env),
+        extends: concatArray(returnValue.extends, config.extends),
+        plugins: concatArray(returnValue.plugins, config.plugins),
+        rules: Object.assign({}, returnValue.rules, config.rules),
+        settings: Object.assign({}, returnValue.settings, config.settings),
+      }),
+    {
+      env: {},
+      extends: [],
+      plugins: [],
+      rules: {},
+      settings: {},
+    }
+  );
+}
+
 module.exports = {
+  concatESConfig,
   error,
+  // eslint-disable-next-line no-unused-vars
   fixme: (/** @type {'off'|'warn'|'error'} */ _status) => off,
   off,
   warn,

package/rules/base.js

@@ -1,476 +1,39 @@
-const { off, warn, error } = require('./_rule');
+const { concatESConfig, off, error } = require('./_rule');
 
-module.exports = {
-  env: {
-    es6: true,
-  },
-  extends: [require.resolve('./node')], // TODO: remove that extends and provide a standalone configuration
-  plugins: [],
-  rules: {
-    'accessor-pairs': error,
-    'array-bracket-newline': off,
-    'array-bracket-spacing': [error, 'never'],
-    'array-callback-return': error,
-    'array-element-newline': off,
-    'arrow-body-style': [
-      'error',
-      'as-needed',
-      {
-        requireReturnForObjectLiteral: false,
-      },
-    ],
-    'arrow-parens': [error, 'always'],
-    'arrow-spacing': [error, { after: true, before: true }],
-    'block-scoped-var': error,
-    'block-spacing': [error, 'always'],
-    'brace-style': [error, '1tbs', { allowSingleLine: false }],
-    camelcase: off,
-    'capitalized-comments': off,
-    'class-methods-use-this': off,
-    'comma-dangle': [error, 'always-multiline'],
-    'comma-spacing': [error, { after: true, before: false }],
-    'comma-style': [error, 'last'],
-    complexity: [error, 20],
-    'computed-property-spacing': [error, 'never'],
-    'consistent-return': error,
-    'consistent-this': [error, 'self'],
-    'constructor-super': error,
-    curly: error,
-    'default-case': [error, { commentPattern: '^no default$' }], // require default case in switch statements or ignore by adding `//no default`
-    'dot-location': [error, 'property'],
-    'dot-notation': error,
-    'eol-last': error,
-    eqeqeq: [error, 'smart'],
-    'for-direction': error,
-    'func-call-spacing': [error, 'never'],
-    'func-name-matching': error,
-    'func-names': off,
-    'func-style': off,
-    'function-paren-newline': [error, 'consistent'],
-    'generator-star-spacing': [error, { after: false, before: true }],
-    'getter-return': [error, { allowImplicit: true }],
-    'guard-for-in': error,
-    'id-length': [error, { exceptions: ['t', '$', '_', 'x', 'y', 'z'], max: 50, min: 2, properties: 'never' }],
-    'id-match': off,
-    'implicit-arrow-linebreak': [error, 'beside'],
-    indent: [error, 2],
-    'init-declarations': off,
-    'jsx-quotes': [error, 'prefer-double'],
-    'key-spacing': [
-      error,
-      {
-        afterColon: true,
-        beforeColon: false,
-      },
-    ],
-    'keyword-spacing': [
-      error,
-      {
-        after: true,
-        before: true,
-      },
-    ],
-    'line-comment-position': 0,
-    'linebreak-style': [error, 'unix'],
-    'lines-around-comment': [
-      error,
-      {
-        allowArrayEnd: true,
-        allowArrayStart: true,
-        allowBlockEnd: true,
-        allowBlockStart: true,
-        allowObjectEnd: true,
-        allowObjectStart: true,
-        beforeBlockComment: true,
-        beforeLineComment: true,
-      },
-    ],
-    'lines-around-directive': [error, 'always'],
-    'lines-between-class-members': [error, 'always', { exceptAfterSingleLine: true }],
-    'max-len': [
-      warn,
-      {
-        code: 160,
-      },
-    ],
-    'max-nested-callbacks': [warn, { max: 5 }],
-    'max-statements-per-line': [
-      error,
-      {
-        max: 1,
-      },
-    ],
-    'multiline-comment-style': off,
-    'multiline-ternary': off,
-    'new-cap': [
-      off,
-      {
-        capIsNew: false,
-        newIsCap: true,
-      },
-    ],
-    'new-parens': error,
-    'newline-after-var': off,
-    'newline-before-return': error,
-    'newline-per-chained-call': off,
-    'no-alert': error,
-    'no-array-constructor': error,
-    'no-async-promise-executor': error,
-    'no-await-in-loop': error,
-    'no-caller': error,
-    'no-case-declarations': error,
-    'no-catch-shadow': error,
-    'no-class-assign': error,
-    'no-compare-neg-zero': error,
-    'no-cond-assign': error,
-    'no-confusing-arrow': error,
-    'no-console': error,
-    'no-const-assign': error,
-    'no-constant-condition': warn,
-    'no-continue': off,
-    'no-control-regex': error,
-    'no-debugger': error,
-    'no-delete-var': error,
-    'no-div-regex': error,
-    'no-dupe-args': error,
-    'no-dupe-class-members': error,
-    'no-dupe-else-if': error,
-    'no-dupe-keys': error,
-    'no-duplicate-case': error,
-    'no-duplicate-imports': off,
-    'no-else-return': [error, { allowElseIf: false }],
-    'no-empty': error,
-    'no-empty-character-class': error,
-    'no-empty-function': [error, { allow: ['arrowFunctions', 'functions', 'methods'] }],
-    'no-empty-pattern': error,
-    'no-eq-null': off,
-    'no-eval': error,
-    'no-ex-assign': error,
-    'no-extend-native': error,
-    'no-extra-bind': error,
-    'no-extra-boolean-cast': error,
-    'no-extra-parens': error,
-    'no-extra-semi': error,
-    'no-fallthrough': error,
-    'no-floating-decimal': error,
-    'no-func-assign': error,
-    'no-global-assign': error,
-    'no-implicit-coercion': error,
-    'no-implicit-globals': off,
-    'no-implied-eval': error,
-    'no-import-assign': error,
-    'no-inline-comments': off,
-    'no-inner-declarations': error,
-    'no-invalid-regexp': error,
-    'no-invalid-this': off,
-    'no-irregular-whitespace': error,
-    'no-iterator': error,
-    'no-label-var': error,
-    'no-labels': error,
-    'no-lone-blocks': error,
-    'no-lonely-if': error,
-    'no-loop-func': error,
-    'no-loss-of-precision': error,
-    'no-magic-numbers': off,
-    'no-misleading-character-class': error,
-    'no-mixed-spaces-and-tabs': error,
-    'no-multi-spaces': [
-      error,
-      {
-        ignoreEOLComments: false,
-      },
-    ],
-    'no-multi-str': error,
-    'no-multiple-empty-lines': [
-      error,
-      {
-        max: 1,
-        maxBOF: 0,
-        maxEOF: 1,
-      },
-    ],
-    'no-native-reassign': error,
-    'no-negated-condition': off,
-    'no-negated-in-lhs': error,
-    'no-nested-ternary': off,
-    'no-new': error,
-    'no-new-func': error,
-    'no-new-object': error,
-    'no-new-require': error,
-    'no-new-symbol': error,
-    'no-new-wrappers': error,
-    'no-obj-calls': error,
-    'no-octal': error,
-    'no-octal-escape': error,
-    'no-param-reassign': [
-      error,
-      {
-        props: false,
-      },
-    ],
-    'no-plusplus': error,
-    'no-promise-executor-return': error,
-    'no-proto': error,
-    'no-prototype-builtins': error,
-    'no-redeclare': [
-      error,
-      {
-        builtinGlobals: true,
-      },
-    ],
-    'no-regex-spaces': error,
-    'no-restricted-globals': off,
-    'no-restricted-properties': [
-      error,
-      {
-        message: 'arguments.callee is deprecated',
-        object: 'arguments',
-        property: 'callee',
-      },
-      {
-        message: 'Please use Number.isFinite instead',
-        object: 'global',
-        property: 'isFinite',
-      },
-      {
-        message: 'Please use Number.isFinite instead',
-        object: 'self',
-        property: 'isFinite',
-      },
-      {
-        message: 'Please use Number.isFinite instead',
-        object: 'window',
-        property: 'isFinite',
-      },
-      {
-        message: 'Please use Number.isNaN instead',
-        object: 'global',
-        property: 'isNaN',
-      },
-      {
-        message: 'Please use Number.isNaN instead',
-        object: 'self',
-        property: 'isNaN',
-      },
-      {
-        message: 'Please use Number.isNaN instead',
-        object: 'window',
-        property: 'isNaN',
-      },
-      {
-        message: 'Please use Object.defineProperty instead.',
-        property: '__defineGetter__',
-      },
-      {
-        message: 'Please use Object.defineProperty instead.',
-        property: '__defineSetter__',
-      },
-      {
-        message: 'Use the exponentiation operator (**) instead.',
-        object: 'Math',
-        property: 'pow',
-      },
-    ],
-    'no-restricted-syntax': [
-      error,
-      {
-        message:
-          'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
-        selector: 'ForInStatement',
-      },
-      {
-        message: 'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
-        selector: 'LabeledStatement',
-      },
-      {
-        message: '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
-        selector: 'WithStatement',
-      },
-    ],
-    'no-return-assign': error,
-    'no-return-await': error,
-    'no-script-url': error,
-    'no-self-assign': error,
-    'no-self-compare': error,
-    'no-sequences': error,
-    'no-setter-return': error,
-    'no-shadow': [
-      error,
-      {
-        builtinGlobals: false,
-        hoist: 'all',
-      },
-    ],
-    'no-shadow-restricted-names': error,
-    'no-spaced-func': error,
-    'no-sparse-arrays': error,
-    'no-sync': off,
-    'no-tabs': error,
-    'no-template-curly-in-string': error,
-    'no-ternary': off,
-    'no-this-before-super': error,
-    'no-throw-literal': error,
-    'no-trailing-spaces': error,
-    'no-undef': error,
-    'no-undef-init': error,
-    'no-undefined': off,
-    'no-underscore-dangle': off,
-    'no-unexpected-multiline': error,
-    'no-unmodified-loop-condition': error,
-    'no-unneeded-ternary': off, // error,
-    'no-unreachable': error,
-    'no-unreachable-loop': error,
-    'no-unsafe-finally': error,
-    'no-unsafe-negation': error,
-    'no-unsafe-optional-chaining': [error, { disallowArithmeticOperators: true }],
-    'no-unused-expressions': [
-      error,
-      {
-        allowShortCircuit: false,
-        allowTaggedTemplates: true,
-        allowTernary: false,
-      },
-    ],
-    'no-unused-vars': [
-      error,
-      {
-        args: 'none',
-        ignoreRestSiblings: true,
-        vars: 'all',
-      },
-    ],
-    'no-use-before-define': [
-      error,
-      {
-        classes: false,
-        functions: false,
-        variables: false,
-      },
-    ],
-    'no-useless-backreference': error,
-    'no-useless-call': error,
-    'no-useless-catch': error,
-    'no-useless-computed-key': error,
-    'no-useless-concat': error,
-    'no-useless-constructor': error,
-    'no-useless-escape': error,
-    'no-useless-rename': [
-      error,
-      {
-        ignoreDestructuring: false,
-        ignoreExport: false,
-        ignoreImport: false,
-      },
-    ],
-    'no-useless-return': error,
-    'no-var': error,
-    'no-void': error,
-    'no-warning-comments': [
-      off,
-      {
-        location: 'start',
-        terms: ['todo', '@todo'],
-      },
-    ],
-    'no-whitespace-before-property': error,
-    'no-with': error,
-    'nonblock-statement-body-position': [error, 'below'],
-    'object-curly-spacing': [off, 'never'],
-    'object-property-newline': [
-      error,
-      {
-        allowMultiplePropertiesPerLine: false,
-      },
-    ],
-    'object-shorthand': [error, 'always'],
-    'one-var': [error, 'never'],
-    'one-var-declaration-per-line': error,
-    'operator-assignment': [error, 'always'],
-    'operator-linebreak': [error, 'after'],
-    'padded-blocks': [error, 'never'],
-    'padding-line-between-statements': off,
-    'prefer-arrow-callback': [
-      error,
-      {
-        allowNamedFunctions: false,
-        allowUnboundThis: true,
-      },
-    ],
-    'prefer-const': error,
-    'prefer-destructuring': off,
-    'prefer-named-capture-group': off,
-    'prefer-numeric-literals': error,
-    'prefer-promise-reject-errors': error,
-    'prefer-reflect': off,
-    'prefer-rest-params': error,
-    'prefer-spread': error,
-    'prefer-template': error,
-    'quote-props': [error, 'as-needed'],
-    quotes: [error, 'single'],
-    radix: error,
-    'require-await': off, // https://github.com/airbnb/javascript/issues/2013
-    'require-jsdoc': off,
-    'require-yield': error,
-    semi: [error, 'always'],
-    'semi-spacing': [
-      error,
-      {
-        after: true,
-        before: false,
-      },
-    ],
-    'semi-style': [error, 'last'],
-    'sort-keys': off /* [
-      error,
-      'asc',
-      {
-        caseSensitive: false,
-        natural: true
-      }
-    ], */,
-    'sort-vars': error,
-    'space-before-blocks': [error, 'always'],
-    'space-before-function-paren': [error, 'always'],
-    'space-in-parens': [error, 'never'],
-    'space-infix-ops': error,
-    'space-unary-ops': [
-      error,
-      {
-        nonwords: false,
-        words: true,
-      },
-    ],
-    'spaced-comment': [
-      error,
-      'always',
-      {
-        block: {
-          balanced: true,
-          exceptions: ['*'],
-          markers: ['*package', '!', ',', ':', '::', 'flow-include'],
-        },
-        line: {
-          markers: ['*package', '!', '/', ',', '='],
-        },
-      },
-    ],
-    strict: [error, 'never'],
-    'switch-colon-spacing': [
-      error,
-      {
-        after: true,
-        before: false,
-      },
-    ],
-    'symbol-description': error,
-    'template-tag-spacing': [error, 'never'],
-    'unicode-bom': [error, 'never'],
-    'use-isnan': error,
-    'valid-jsdoc': off,
-    'valid-typeof': [error, { requireStringLiterals: true }],
-    'vars-on-top': error,
-    'wrap-iife': [error, 'inside'],
-    'wrap-regex': off,
-    yoda: off,
-  },
-};
+const baseConfig = concatESConfig(
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/best-practices'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/errors'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/es6'),
+  /** {@link ./import.js} */
+  // require('eslint-config-airbnb-base/rules/imports'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/node'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/strict'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/style'),
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/variables')
+);
+
+module.exports = concatESConfig(
+  baseConfig,
+  // overrides
+  {
+    rules: {
+      // Often useful in jsx
+      'no-nested-ternary': off,
+      // Too strict, for pure code prefer the functional plugin
+      'no-param-reassign': [error, { props: false }],
+      // Allow for-of syntax
+      // @ts-ignore
+      'no-restricted-syntax': baseConfig.rules['no-restricted-syntax'].filter(
+        // @ts-ignore
+        ({ selector }) => selector !== 'ForOfStatement'
+      ),
+    },
+  }
+);

package/rules/import.js

@@ -1,4 +1,4 @@
-const { off, warn, error } = require('./_rule');
+const { off, warn, error, concatESConfig, fixme } = require('./_rule');
 
 /**
  * @see https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
@@ -7,110 +7,17 @@ const { off, warn, error } = require('./_rule');
 // eslint-disable-next-line no-unused-vars
 const performanceIssue = (_status) => off;
 
-module.exports = {
-  plugins: ['import'],
-  rules: {
-    'import/default': error,
-    'import/export': error,
-    'import/exports-last': off,
-    'import/extensions': [
-      error,
-      'ignorePackages',
-      {
-        js: 'never',
-        json: 'always',
-        jsx: 'never',
-        mjs: 'never',
-        ts: 'never',
-        tsx: 'never',
-      },
-    ],
-    'import/first': [error, 'absolute-first'],
-    'import/group-exports': off,
-    'import/max-dependencies': [
-      off,
-      {
-        max: 10,
-      },
-    ],
-    'import/named': error,
-    'import/namespace': error,
-    'import/newline-after-import': error,
-    'import/no-absolute-path': error,
-    'import/no-amd': error,
-    'import/no-anonymous-default-export': off,
-    'import/no-commonjs': off, // Still used widely by nodejs programs
-    'import/no-cycle': [error, { maxDepth: Number.POSITIVE_INFINITY }], // Elm, ReasonML forbids circular dependency
-    'import/no-default-export': off,
-    'import/no-deprecated': performanceIssue(warn),
-    'import/no-duplicates': error,
-    'import/no-dynamic-require': error,
-    'import/no-extraneous-dependencies': [
-      error,
-      {
-        // https://github.com/airbnb/javascript/blob/1eadb93e377da1e56c3f91f26610e5d0a00738a9/packages/eslint-config-airbnb-base/rules/imports.js#L71
-        devDependencies: [
-          'test/**', // tape, common npm pattern
-          'tests/**', // also common npm pattern
-          'spec/**', // mocha, rspec-like pattern
-          '**/__tests__/**', // jest pattern
-          '**/__mocks__/**', // jest pattern
-          'test.{js,jsx,ts,tsx}', // repos with a single test file
-          'test-*.{js,jsx,ts,tsx}', // repos with multiple top-level test files
-          '**/*{.,_}{test,spec}.{js,jsx,ts,tsx}', // tests where the extension or filename suffix denotes that it is a test
-          '**/jest.config.{js,ts}', // jest config
-          '**/jest.setup.{js,ts}', // jest setup
-          '**/vue.config.{js,ts}', // vue-cli config
-          '**/webpack.config.js', // webpack config
-          '**/webpack.config.*.js', // webpack config
-          '**/rollup.config.js', // rollup config
-          '**/rollup.config.*.js', // rollup config
-          '**/gulpfile.js', // gulp config
-          '**/gulpfile.*.js', // gulp config
-          '**/Gruntfile{,.js}', // grunt config
-          '**/protractor.conf.js', // protractor config
-          '**/protractor.conf.*.js', // protractor config
-          '**/karma.conf.js', // karma config
-          '**/.eslintrc.js', // eslint config,
-          '**/markdown.config.js', // markdown magic config,
-        ],
-        optionalDependencies: false,
-      },
-    ],
-    'import/no-internal-modules': off,
-    'import/no-mutable-exports': error,
-    'import/no-named-as-default': performanceIssue(error),
-    'import/no-named-as-default-member': error,
-    'import/no-named-default': error,
-    'import/no-named-export': off,
-    'import/no-namespace': off,
-    'import/no-nodejs-modules': off,
-    'import/no-relative-parent-imports': off,
-    'import/no-restricted-paths': off,
-    'import/no-self-import': error,
-    'import/no-unassigned-import': off,
-    'import/no-unresolved': [error, { caseSensitive: true, commonjs: true }],
-    'import/no-unused-modules': [performanceIssue(error), { unusedExports: true }],
-    'import/no-useless-path-segments': error,
-    'import/no-webpack-loader-syntax': error,
-    'import/order': [
-      error,
-      {
-        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
-        'newlines-between': 'never',
-      },
-    ],
-    'import/prefer-default-export': off,
-    'import/unambiguous': off, // Disable because proposal still in progress
-  },
-  settings: {
-    'import/core-modules': [],
-    'import/extensions': ['.js', '.mjs', '.jsx'],
-    'import/ignore': ['node_modules', '\\.(coffee|scss|css|less|hbs|svg|json)$'],
-    'import/resolver': {
-      node: {
-        extensions: ['.mjs', '.js', '.json'],
-      },
+module.exports = concatESConfig(
+  // @ts-ignore
+  require('eslint-config-airbnb-base/rules/imports'),
+  // Overrides
+  {
+    rules: {
+      'import/no-deprecated': performanceIssue(warn),
+      'import/no-named-as-default': performanceIssue(error),
+      'import/no-unused-modules': performanceIssue(error),
+      'import/prefer-default-export': off, // Not aligned, default export does not bring sufficient semantic
+      'import/unambiguous': fixme(off), // Disable because proposal still in progress
     },
-  },
-};
+  }
+);

package/rules/jest.js

@@ -1,25 +1,13 @@
-const { off, error } = require('./_rule');
+const { off, error, concatESConfig } = require('./_rule');
 
-/**
- * Typescript config is loose because we often have "hack", "mock" in tests
- */
-const tsDisabled = {
-  '@typescript-eslint/no-unsafe-assignment': off,
-  '@typescript-eslint/no-unsafe-call': off,
-  '@typescript-eslint/no-unsafe-member-access': off,
-  '@typescript-eslint/no-unsafe-return': off,
-  '@typescript-eslint/restrict-template-expressions': off,
-  '@typescript-eslint/unbound-method': off,
-};
-
-module.exports = {
-  env: {
-    jest: true,
-  },
-  extends: ['plugin:jest/recommended'],
-  plugins: ['jest'],
-  rules: Object.assign(
-    {
+module.exports = concatESConfig(
+  {
+    env: {
+      jest: true,
+    },
+    extends: ['plugin:jest/recommended'],
+    plugins: ['jest'],
+    rules: {
       'jest/expect-expect': off, // Disabled because it does not handle functions that does the expect
       'jest/no-alias-methods': error,
       'jest/no-commented-out-tests': error,
@@ -41,6 +29,18 @@ module.exports = {
       'jest/valid-expect': error,
       'jest/valid-title': [error, { ignoreTypeOfDescribeName: true }],
     },
-    tsDisabled
-  ),
-};
+  },
+  /**
+   * Typescript config is set to be less strict because we often have "hack", "mock" in tests
+   */
+  {
+    rules: {
+      '@typescript-eslint/no-unsafe-assignment': off,
+      '@typescript-eslint/no-unsafe-call': off,
+      '@typescript-eslint/no-unsafe-member-access': off,
+      '@typescript-eslint/no-unsafe-return': off,
+      '@typescript-eslint/restrict-template-expressions': off,
+      '@typescript-eslint/unbound-method': off,
+    },
+  }
+);

package/rules/typescript.js

@@ -1,53 +1,24 @@
-const { fixme, off, warn, error } = require('./_rule');
-const { rules: baseRules } = require('./base');
+// Inspired by https://github.com/iamturns/eslint-config-airbnb-typescript/blob/master/lib/shared.js
 
-const duplicateTSC = off; // = "off because tsc already checks that"
+const { fixme, off, warn, error, concatESConfig } = require('./_rule');
+const { rules: _baseRules } = require('./base');
+const { rules: _baseImportRules } = require('./import');
+
+// Fix Hack : TS pluging seems to modify the rules
+const deepClone = (/** @type {Record<string, unknown>} */ anyValue) => JSON.parse(JSON.stringify(anyValue));
+const baseRules = deepClone(_baseRules);
+const baseImportRules = deepClone(_baseImportRules);
 
-// https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
-const ruleDisabled = {
-  camelcase: off,
-  'constructor-super': off,
-  'dot-notation': off,
-  'getter-return': off,
-  'import/default': duplicateTSC,
-  'import/export': fixme(error), // https://github.com/benmosher/eslint-plugin-import/issues/1964
-  'import/named': duplicateTSC,
-  'import/namespace': duplicateTSC,
-  'import/no-named-as-default-member': duplicateTSC,
-  'import/no-unresolved': duplicateTSC,
-  'no-array-constructor': off,
-  'no-const-assign': off,
-  'no-dupe-args': off,
-  'no-dupe-class-members': off,
-  'no-dupe-keys': off,
-  'no-empty-function': off,
-  'no-func-assign': off,
-  'no-import-assign': off,
-  'no-inner-declarations': fixme(error), // https://github.com/typescript-eslint/typescript-eslint/issues/239
-  'no-new-symbol': off,
-  'no-obj-calls': off,
-  'no-redeclare': off,
-  'no-setter-return': off,
-  'no-shadow': off, // https://github.com/typescript-eslint/typescript-eslint/issues/2483
-  'no-this-before-super': off,
-  'no-undef': off,
-  'no-unreachable': off,
-  'no-unsafe-negation': off,
-  'no-unused-vars': off,
-  'no-use-before-define': off,
-  'no-useless-constructor': off,
-  'no-var': error,
-  'prefer-const': error,
-  'prefer-rest-params': error,
-  'prefer-spread': error,
-  'valid-typeof': off,
-};
+const duplicateTSC = off; // = "off because tsc already checks that"
 
-module.exports = {
-  extends: ['plugin:@typescript-eslint/recommended-requiring-type-checking'],
-  plugins: ['@typescript-eslint', 'import'],
-  rules: Object.assign(
-    {
+module.exports = concatESConfig(
+  /**
+   * Plugin rules
+   */
+  {
+    extends: ['plugin:@typescript-eslint/recommended-requiring-type-checking'],
+    plugins: ['@typescript-eslint', 'import'],
+    rules: {
       '@typescript-eslint/adjacent-overload-signatures': error,
       '@typescript-eslint/ban-ts-comment': [
         warn,
@@ -60,13 +31,29 @@ module.exports = {
         },
       ],
       '@typescript-eslint/ban-types': error,
+      '@typescript-eslint/brace-style': baseRules['brace-style'],
+      '@typescript-eslint/comma-dangle': [
+        baseRules['comma-dangle'][0],
+        {
+          ...baseRules['comma-dangle'][1],
+          enums: baseRules['comma-dangle'][1].arrays,
+          generics: baseRules['comma-dangle'][1].arrays,
+          tuples: baseRules['comma-dangle'][1].arrays,
+        },
+      ],
+      '@typescript-eslint/comma-spacing': baseRules['comma-spacing'],
       '@typescript-eslint/consistent-type-assertions': [
         error,
         { assertionStyle: 'as', objectLiteralTypeAssertions: 'never' },
       ],
+      '@typescript-eslint/default-param-last': baseRules['default-param-last'],
       '@typescript-eslint/dot-notation': baseRules['dot-notation'],
       '@typescript-eslint/explicit-function-return-type': off,
       '@typescript-eslint/explicit-module-boundary-types': off,
+      '@typescript-eslint/func-call-spacing': baseRules['func-call-spacing'],
+      '@typescript-eslint/indent': baseRules.indent,
+      '@typescript-eslint/keyword-spacing': baseRules['keyword-spacing'],
+      '@typescript-eslint/lines-between-class-members': baseRules['lines-between-class-members'],
       '@typescript-eslint/member-delimiter-style': error,
       '@typescript-eslint/naming-convention': [
         error,
@@ -93,28 +80,43 @@ module.exports = {
           selector: 'typeLike',
         },
       ],
-      '@typescript-eslint/no-array-constructor': error,
+      '@typescript-eslint/no-array-constructor': baseRules['no-array-constructor'],
       '@typescript-eslint/no-base-to-string': error,
+      '@typescript-eslint/no-dupe-class-members': baseRules['no-dupe-class-members'],
       '@typescript-eslint/no-empty-function': baseRules['no-empty-function'],
       '@typescript-eslint/no-empty-interface': [error, { allowSingleExtends: true }],
       '@typescript-eslint/no-explicit-any': off, // if any is explicit then it's wanted
+      '@typescript-eslint/no-extra-parens': baseRules['no-extra-parens'],
+      '@typescript-eslint/no-extra-semi': baseRules['no-extra-semi'],
       '@typescript-eslint/no-implicit-any-catch': error,
       '@typescript-eslint/no-inferrable-types': error,
+      '@typescript-eslint/no-loop-func': baseRules['no-loop-func'],
+      '@typescript-eslint/no-loss-of-precision': baseRules['no-loss-of-precision'],
+      '@typescript-eslint/no-magic-numbers': baseRules['no-magic-numbers'],
       '@typescript-eslint/no-misused-new': error,
       '@typescript-eslint/no-namespace': off, // We don't agree with community, namespaces are great and not deprecated
       '@typescript-eslint/no-non-null-assertion': error,
-      '@typescript-eslint/no-redeclare': fixme(error /* baseRules['no-redeclare'] */), // Error level so it is strongly discouraged
+      '@typescript-eslint/no-redeclare': fixme(baseRules['no-redeclare']),
       '@typescript-eslint/no-require-imports': error,
-      '@typescript-eslint/no-shadow': baseRules['no-shadow'], // Does not allow to declare type and const with same name
+      '@typescript-eslint/no-shadow': baseRules['no-shadow'],
       '@typescript-eslint/no-this-alias': error,
+      '@typescript-eslint/no-throw-literal': baseRules['no-throw-literal'],
       '@typescript-eslint/no-unnecessary-condition': error,
       '@typescript-eslint/no-unsafe-argument': error,
+      '@typescript-eslint/no-unused-expressions': baseRules['no-unused-expressions'],
       '@typescript-eslint/no-unused-vars': baseRules['no-unused-vars'],
       '@typescript-eslint/no-use-before-define': baseRules['no-use-before-define'],
       '@typescript-eslint/no-useless-constructor': baseRules['no-useless-constructor'],
       '@typescript-eslint/no-var-requires': error,
+      '@typescript-eslint/object-curly-spacing': baseRules['object-curly-spacing'],
       '@typescript-eslint/prefer-namespace-keyword': error,
       '@typescript-eslint/prefer-reduce-type-parameter': error,
+      '@typescript-eslint/quotes': baseRules.quotes,
+      '@typescript-eslint/require-await': baseRules['require-await'],
+      '@typescript-eslint/return-await': baseRules['no-return-await'],
+      '@typescript-eslint/semi': baseRules.semi,
+      '@typescript-eslint/space-before-function-paren': baseRules['space-before-function-paren'],
+      '@typescript-eslint/space-infix-ops': baseRules['space-infix-ops'],
       '@typescript-eslint/strict-boolean-expressions': [
         error,
         {
@@ -127,6 +129,106 @@ module.exports = {
       '@typescript-eslint/triple-slash-reference': error,
       '@typescript-eslint/type-annotation-spacing': error,
     },
-    ruleDisabled
-  ),
-};
+  },
+  /**
+   * Import overrides
+   */
+  {
+    rules: {
+      'import/extensions': [
+        baseImportRules['import/extensions'][0],
+        baseImportRules['import/extensions'][1],
+        {
+          ...baseImportRules['import/extensions'][2],
+          ts: 'never',
+          tsx: 'never',
+        },
+      ],
+      'import/no-extraneous-dependencies': [
+        baseImportRules['import/no-extraneous-dependencies'][0],
+        {
+          ...baseImportRules['import/no-extraneous-dependencies'][1],
+          devDependencies: baseImportRules['import/no-extraneous-dependencies'][1].devDependencies.reduce(
+            (/** @type {string[]} */ result, /** @type {string} */ devDep) => {
+              const toAppend = [devDep];
+              const devDepWithTs = devDep.replace(/\bjs(x?)\b/g, 'ts$1');
+              if (devDepWithTs !== devDep) {
+                toAppend.push(devDepWithTs);
+              }
+              return [...result, ...toAppend];
+            },
+            []
+          ),
+        },
+      ],
+    },
+  },
+  /**
+   * Disabled rules
+   */
+  {
+    // https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
+    rules: {
+      'brace-style': off,
+      camelcase: off,
+      'comma-dangle': off,
+      'comma-spacing': off,
+      'constructor-super': off,
+      'default-param-last': off,
+      'dot-notation': off,
+      'func-call-spacing': off,
+      'getter-return': off,
+      'import/default': duplicateTSC,
+      'import/export': fixme(error), // https://github.com/benmosher/eslint-plugin-import/issues/1964
+      'import/named': duplicateTSC,
+      'import/namespace': duplicateTSC,
+      'import/no-named-as-default-member': duplicateTSC,
+      'import/no-unresolved': duplicateTSC,
+      indent: off,
+      'keyword-spacing': off,
+      'lines-between-class-members': off,
+      'no-array-constructor': off,
+      'no-const-assign': off,
+      'no-dupe-args': off,
+      'no-dupe-class-members': off,
+      'no-dupe-keys': off,
+      'no-empty-function': off,
+      'no-extra-parens': off,
+      'no-extra-semi': off,
+      'no-func-assign': off,
+      'no-implied-eval': off,
+      'no-import-assign': off,
+      'no-inner-declarations': fixme(error), // https://github.com/typescript-eslint/typescript-eslint/issues/239
+      'no-loop-func': off,
+      'no-loss-of-precision': off,
+      'no-magic-numbers': off,
+      'no-new-func': off,
+      'no-new-symbol': off,
+      'no-obj-calls': off,
+      'no-redeclare': off,
+      'no-return-await': off,
+      'no-setter-return': off,
+      'no-shadow': off,
+      'no-this-before-super': off,
+      'no-throw-literal': off,
+      'no-undef': off,
+      'no-unreachable': off,
+      'no-unsafe-negation': off,
+      'no-unused-expression': off,
+      'no-unused-vars': off,
+      'no-use-before-define': off,
+      'no-useless-constructor': off,
+      'no-var': error,
+      'object-curly-spacing': off,
+      'prefer-const': error,
+      'prefer-rest-params': error,
+      'prefer-spread': error,
+      quotes: off,
+      'require-await': off,
+      semi: off,
+      'space-before-function-paren': off,
+      'space-infix-ops': off,
+      'valid-typeof': off,
+    },
+  }
+);

package/rules/unicorn.js

@@ -1,18 +1,10 @@
-const { off, warn, error } = require('./_rule');
+const { off, warn, error, concatESConfig } = require('./_rule');
 
-const unsafeRules = {
-  'unicorn/consistent-destructuring': off,
-  'unicorn/no-array-for-each': off, // This rule could change browser compatibility
-  'unicorn/no-object-as-default-parameter': off,
-  'unicorn/prefer-default-parameters': off,
-  'unicorn/prevent-abbreviations': off, // This rule is so dangerous : it potentially break code while fixing in many cases !!
-};
-
-module.exports = {
-  extends: ['plugin:unicorn/recommended'],
-  plugins: ['unicorn'],
-  rules: Object.assign(
-    {
+module.exports = concatESConfig(
+  {
+    extends: ['plugin:unicorn/recommended'],
+    plugins: ['unicorn'],
+    rules: {
       'unicode-bom': [error, 'never'],
       'unicorn/better-regex': error,
       'unicorn/catch-error-name': [error, { name: error }],
@@ -53,6 +45,14 @@ module.exports = {
       'unicorn/prefer-type-error': error,
       'unicorn/throw-new-error': error,
     },
-    unsafeRules
-  ),
-};
+  },
+  {
+    rules: {
+      'unicorn/consistent-destructuring': off,
+      'unicorn/no-array-for-each': off, // This rule could change browser compatibility
+      'unicorn/no-object-as-default-parameter': off,
+      'unicorn/prefer-default-parameters': off,
+      'unicorn/prevent-abbreviations': off, // This rule is so dangerous : it potentially break code while fixing in many cases !!
+    },
+  }
+);

package/ts.js

@@ -14,4 +14,23 @@ module.exports = {
   parserOptions: {
     sourceType: 'module',
   },
+  settings: {
+    // Append 'ts' extensions to Airbnb 'import/extensions' setting
+    'import/extensions': ['.js', '.mjs', '.jsx', '.ts', '.tsx', '.d.ts'],
+
+    // Resolve type definition packages
+    'import/external-module-folders': ['node_modules', 'node_modules/@types'],
+
+    // Apply special parsing for TypeScript files
+    'import/parsers': {
+      '@typescript-eslint/parser': ['.ts', '.tsx', '.d.ts'],
+    },
+
+    // Append 'ts' extensions to Airbnb 'import/resolver' setting
+    'import/resolver': {
+      node: {
+        extensions: ['.mjs', '.js', '.json', '.ts', '.d.ts'],
+      },
+    },
+  },
 };

package/rules/node.js

@@ -1,21 +0,0 @@
-const { off, error } = require('./_rule');
-
-// @see https://eslint.org/blog/2020/02/whats-coming-in-eslint-7.0.0#deprecating-node-js-commonjs-specific-rules
-module.exports = {
-  env: {
-    node: true,
-  },
-  rules: {
-    'callback-return': off,
-    'global-require': error,
-    'handle-callback-err': off,
-    'no-buffer-constructor': error,
-    'no-mixed-requires': [off, false],
-    'no-new-require': error,
-    'no-path-concat': error,
-    'no-process-env': off,
-    'no-process-exit': off,
-    'no-restricted-modules': off,
-    'no-sync': off,
-  },
-};