@innovixx/eslint-config 3.0.0 → 3.0.2

package/config/configs/base/rules/style.mjs

@@ -351,7 +351,7 @@ const styleRules = {
     'no-spaced-func': 'error',
 
     // disallow tab characters entirely
-    'no-tabs': 'error',
+    'no-tabs': 'off',
 
     // disallow the use of ternary operators
     'no-ternary': 'off',

package/config/configs/typescript/index.mjs

@@ -1,7 +1,7 @@
 import tsPlugin from '@typescript-eslint/eslint-plugin';
 import tsParser from '@typescript-eslint/parser';
 import typeScriptSettings from './settings/typescript.mjs';
-import variableRules from './rules/variables.mjs';
+import allRules from './rules/all.mjs';
 import { deepMerge } from '../../deepMerge.mjs';
 
 /** @type {import('eslint').Linter.Config} */
@@ -11,7 +11,7 @@ export const index = deepMerge(
     settings: typeScriptSettings.settings,
   },
   {
-    rules: variableRules.rules,
+    rules: allRules.rules,
   },
   {
     plugins: {
@@ -21,6 +21,14 @@ export const index = deepMerge(
   {
     languageOptions: {
       parser: tsParser,
+      parserOptions: {
+        ecmaVersion: 2020,
+        project: './tsconfig.json',
+        sourceType: 'module',
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
     },
   },
 );

package/config/configs/typescript/rules/all.mjs

@@ -0,0 +1,158 @@
+/** @type {import('eslint').Linter.Config} */
+
+const variableRules = {
+  rules: {
+    '@typescript-eslint/adjacent-overload-signatures': 'error',
+    '@typescript-eslint/array-type': 'error',
+    '@typescript-eslint/await-thenable': 'error',
+    '@typescript-eslint/ban-ts-comment': 'error',
+    '@typescript-eslint/ban-tslint-comment': 'error',
+    '@typescript-eslint/class-literal-property-style': 'error',
+    'class-methods-use-this': 'off',
+    '@typescript-eslint/class-methods-use-this': 'error',
+    '@typescript-eslint/consistent-generic-constructors': 'error',
+    '@typescript-eslint/consistent-indexed-object-style': 'error',
+    'consistent-return': 'off',
+    '@typescript-eslint/consistent-return': 'error',
+    '@typescript-eslint/consistent-type-assertions': 'error',
+    '@typescript-eslint/consistent-type-definitions': 'error',
+    '@typescript-eslint/consistent-type-exports': 'error',
+    '@typescript-eslint/consistent-type-imports': 'error',
+    'default-param-last': 'off',
+    '@typescript-eslint/default-param-last': 'error',
+    'dot-notation': 'off',
+    '@typescript-eslint/dot-notation': 'error',
+    '@typescript-eslint/explicit-function-return-type': 'error',
+    '@typescript-eslint/explicit-member-accessibility': 'error',
+    '@typescript-eslint/explicit-module-boundary-types': 'error',
+    'init-declarations': 'off',
+    '@typescript-eslint/init-declarations': 'error',
+    'max-params': 'off',
+    '@typescript-eslint/max-params': 'error',
+    '@typescript-eslint/member-ordering': 'error',
+    '@typescript-eslint/method-signature-style': 'error',
+    '@typescript-eslint/naming-convention': 'off',
+    'no-array-constructor': 'off',
+    '@typescript-eslint/no-array-constructor': 'error',
+    '@typescript-eslint/no-array-delete': 'error',
+    '@typescript-eslint/no-base-to-string': 'error',
+    '@typescript-eslint/no-confusing-non-null-assertion': 'error',
+    '@typescript-eslint/no-confusing-void-expression': 'error',
+    '@typescript-eslint/no-deprecated': 'error',
+    'no-dupe-class-members': 'off',
+    '@typescript-eslint/no-dupe-class-members': 'error',
+    '@typescript-eslint/no-duplicate-enum-values': 'error',
+    '@typescript-eslint/no-duplicate-type-constituents': 'error',
+    '@typescript-eslint/no-dynamic-delete': 'error',
+    'no-empty-function': 'off',
+    '@typescript-eslint/no-empty-function': 'error',
+    '@typescript-eslint/no-empty-object-type': 'error',
+    '@typescript-eslint/no-explicit-any': 'error',
+    '@typescript-eslint/no-extra-non-null-assertion': 'error',
+    '@typescript-eslint/no-extraneous-class': 'error',
+    '@typescript-eslint/no-floating-promises': 'error',
+    '@typescript-eslint/no-for-in-array': 'error',
+    'no-implied-eval': 'off',
+    '@typescript-eslint/no-implied-eval': 'error',
+    '@typescript-eslint/no-import-type-side-effects': 'error',
+    '@typescript-eslint/no-inferrable-types': 'error',
+    'no-invalid-this': 'off',
+    '@typescript-eslint/no-invalid-this': 'error',
+    '@typescript-eslint/no-invalid-void-type': 'error',
+    'no-loop-func': 'off',
+    '@typescript-eslint/no-loop-func': 'error',
+    'no-magic-numbers': 'off',
+    '@typescript-eslint/no-magic-numbers': 'off',
+    '@typescript-eslint/no-meaningless-void-operator': 'error',
+    '@typescript-eslint/no-misused-new': 'error',
+    '@typescript-eslint/no-misused-promises': 'error',
+    '@typescript-eslint/no-misused-spread': 'error',
+    '@typescript-eslint/no-mixed-enums': 'error',
+    '@typescript-eslint/no-namespace': 'error',
+    '@typescript-eslint/no-non-null-asserted-nullish-coalescing': 'error',
+    '@typescript-eslint/no-non-null-asserted-optional-chain': 'error',
+    '@typescript-eslint/no-non-null-assertion': 'error',
+    'no-redeclare': 'off',
+    '@typescript-eslint/no-redeclare': 'error',
+    '@typescript-eslint/no-redundant-type-constituents': 'error',
+    '@typescript-eslint/no-require-imports': 'error',
+    'no-restricted-imports': 'off',
+    '@typescript-eslint/no-restricted-imports': 'error',
+    '@typescript-eslint/no-restricted-types': 'error',
+    'no-shadow': 'off',
+    '@typescript-eslint/no-shadow': 'error',
+    '@typescript-eslint/no-this-alias': 'error',
+    '@typescript-eslint/no-unnecessary-boolean-literal-compare': 'error',
+    '@typescript-eslint/no-unnecessary-condition': 'error',
+    '@typescript-eslint/no-unnecessary-parameter-property-assignment':
+      'error',
+    '@typescript-eslint/no-unnecessary-qualifier': 'error',
+    '@typescript-eslint/no-unnecessary-template-expression': 'error',
+    '@typescript-eslint/no-unnecessary-type-arguments': 'error',
+    '@typescript-eslint/no-unnecessary-type-assertion': 'error',
+    '@typescript-eslint/no-unnecessary-type-constraint': 'error',
+    '@typescript-eslint/no-unnecessary-type-parameters': 'error',
+    '@typescript-eslint/no-unsafe-argument': 'error',
+    '@typescript-eslint/no-unsafe-assignment': 'error',
+    '@typescript-eslint/no-unsafe-call': 'error',
+    '@typescript-eslint/no-unsafe-declaration-merging': 'error',
+    '@typescript-eslint/no-unsafe-enum-comparison': 'error',
+    '@typescript-eslint/no-unsafe-function-type': 'error',
+    '@typescript-eslint/no-unsafe-member-access': 'error',
+    '@typescript-eslint/no-unsafe-return': 'error',
+    '@typescript-eslint/no-unsafe-type-assertion': 'error',
+    '@typescript-eslint/no-unsafe-unary-minus': 'error',
+    'no-unused-expressions': 'off',
+    '@typescript-eslint/no-unused-expressions': 'error',
+    'no-unused-vars': 'off',
+    '@typescript-eslint/no-unused-vars': 'error',
+    'no-use-before-define': 'off',
+    '@typescript-eslint/no-use-before-define': 'error',
+    'no-useless-constructor': 'off',
+    '@typescript-eslint/no-useless-constructor': 'error',
+    '@typescript-eslint/no-useless-empty-export': 'error',
+    '@typescript-eslint/no-wrapper-object-types': 'error',
+    '@typescript-eslint/non-nullable-type-assertion-style': 'error',
+    'no-throw-literal': 'off',
+    '@typescript-eslint/only-throw-error': 'error',
+    '@typescript-eslint/parameter-properties': 'error',
+    '@typescript-eslint/prefer-as-const': 'error',
+    'prefer-destructuring': 'off',
+    '@typescript-eslint/prefer-destructuring': 'error',
+    '@typescript-eslint/prefer-enum-initializers': 'error',
+    '@typescript-eslint/prefer-find': 'error',
+    '@typescript-eslint/prefer-for-of': 'error',
+    '@typescript-eslint/prefer-function-type': 'error',
+    '@typescript-eslint/prefer-includes': 'error',
+    '@typescript-eslint/prefer-literal-enum-member': 'error',
+    '@typescript-eslint/prefer-namespace-keyword': 'error',
+    '@typescript-eslint/prefer-nullish-coalescing': 'error',
+    '@typescript-eslint/prefer-optional-chain': 'error',
+    'prefer-promise-reject-errors': 'off',
+    '@typescript-eslint/prefer-promise-reject-errors': 'error',
+    '@typescript-eslint/prefer-readonly': 'error',
+    '@typescript-eslint/prefer-readonly-parameter-types': 'off',
+    '@typescript-eslint/prefer-reduce-type-parameter': 'error',
+    '@typescript-eslint/prefer-regexp-exec': 'error',
+    '@typescript-eslint/prefer-return-this-type': 'error',
+    '@typescript-eslint/prefer-string-starts-ends-with': 'error',
+    '@typescript-eslint/promise-function-async': 'error',
+    '@typescript-eslint/related-getter-setter-pairs': 'error',
+    '@typescript-eslint/require-array-sort-compare': 'error',
+    'require-await': 'off',
+    '@typescript-eslint/require-await': 'error',
+    '@typescript-eslint/restrict-plus-operands': 'error',
+    '@typescript-eslint/restrict-template-expressions': 'error',
+    'no-return-await': 'off',
+    '@typescript-eslint/return-await': 'error',
+    '@typescript-eslint/strict-boolean-expressions': 'error',
+    '@typescript-eslint/switch-exhaustiveness-check': 'error',
+    '@typescript-eslint/triple-slash-reference': 'error',
+    '@typescript-eslint/typedef': 'error',
+    '@typescript-eslint/unbound-method': 'error',
+    '@typescript-eslint/unified-signatures': 'error',
+    '@typescript-eslint/use-unknown-in-catch-callback-variable': 'error',
+  },
+};
+
+export default variableRules;

package/config/index.mjs

@@ -1,10 +1,10 @@
-import tseslint from 'typescript-eslint'
-import { configs as regexpPluginConfigs } from 'eslint-plugin-regexp'
-import reactExtends from './configs/react/index.mjs'
-import jestExtends from './configs/jest/index.mjs'
-import globals from 'globals'
-import typescriptParser from '@typescript-eslint/parser'
-import { deepMerge } from './deepMerge.mjs'
+import tseslint from 'typescript-eslint';
+import { configs as regexpPluginConfigs } from 'eslint-plugin-regexp';
+import globals from 'globals';
+import typescriptParser from '@typescript-eslint/parser';
+import reactExtends from './configs/react/index.mjs';
+import jestExtends from './configs/jest/index.mjs';
+import { deepMerge } from './deepMerge.mjs';
 
 const baseRules = {
   'class-methods-use-this': 'off',
@@ -19,14 +19,14 @@ const baseRules = {
   'no-use-before-define': 'off',
   'object-shorthand': 'warn',
   'no-useless-escape': 'warn',
-}
+};
 
 const reactA11yRules = {
   'jsx-a11y/anchor-is-valid': 'warn',
   'jsx-a11y/control-has-associated-label': 'warn',
   'jsx-a11y/no-static-element-interactions': 'warn',
   'jsx-a11y/label-has-associated-control': 'warn',
-}
+};
 
 const typeScriptRules = {
   '@typescript-eslint/no-use-before-define': 'off',
@@ -73,14 +73,14 @@ const typeScriptRules = {
     },
   ],
   '@typescript-eslint/no-empty-object-type': 'warn',
-}
+};
 
 /** @typedef {import('eslint').Linter.Config} Config */
 
 /** @type {FlatConfig} */
 const baseExtends = deepMerge(
   regexpPluginConfigs['flat/recommended'],
-)
+);
 
 /** @type {Config[]} */
 export const rootEslintConfig = [
@@ -90,12 +90,12 @@ export const rootEslintConfig = [
       parserOptions: {
         ecmaFeatures: {
           jsx: true,
-        }
+        },
       },
       ecmaVersion: 'latest',
       sourceType: 'module',
       globals: {
-        ...globals.node
+        ...globals.node,
       },
       parser: typescriptParser,
     },
@@ -112,8 +112,8 @@ export const rootEslintConfig = [
         rules: {
           ...baseRules,
           ...typeScriptRules,
-        }
-      }
+        },
+      },
     ),
     files: ['**/*.ts'],
   },
@@ -130,8 +130,8 @@ export const rootEslintConfig = [
           ...baseRules,
           ...typeScriptRules,
           ...reactA11yRules,
-        }
-      }
+        },
+      },
     ),
     files: ['**/*.tsx'],
   },
@@ -141,10 +141,10 @@ export const rootEslintConfig = [
       rules: {
         ...baseRules,
         ...typeScriptRules,
-      }
+      },
     }),
     files: ['**/*.test.ts', '**/*.test.tsx'],
-  }
-]
+  },
+];
 
-export default rootEslintConfig
+export default rootEslintConfig;

package/demo/components/AnotherComponentTest/index.tsx

@@ -1,3 +1,7 @@
 import React from 'react';
 
-export const AnotherComponentTest: React.FC = () => <div>AnotherComponentTest</div>;
+interface Props {
+  title: string;
+}
+
+export const AnotherComponentTest: React.FC<Props> = ({ title }) => <div>{title}</div>;

package/eslint.config.mjs

@@ -35,10 +35,10 @@ export const rootEslintConfig = [
     ],
   },
   {
-    files: [`**/*.{js,jsx,ts,tsx}`],
+    files: ['**/*.{js,jsx,ts,tsx}'],
   },
 ];
 
 export default [
   ...rootEslintConfig,
-];
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@innovixx/eslint-config",
-  "version": "3.0.0",
+  "version": "3.0.2",
   "main": "configs/index.mjs",
   "repository": "git@github.com:innovixx/eslint-config.git",
   "description": "Opinionated ESLint config for JavaScript developers",

package/tsconfig.json

@@ -0,0 +1,18 @@
+{
+  "compilerOptions": {
+    "target": "ES2020",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "esModuleInterop": true,
+    "forceConsistentCasingInFileNames": true,
+    "strict": true,
+    "skipLibCheck": true,
+    "sourceMap": true,
+    "inlineSources": true,
+    "sourceRoot": "/",
+    "jsx": "react",
+  },
+  "exclude": [
+    "node_modules",
+  ]
+}

package/config/configs/typescript/rules/variables.mjs

@@ -1,10 +0,0 @@
-/** @type {import('eslint').Linter.Config} */
-
-const variableRules = {
-  rules: {
-    // Disallow unused variables.
-    '@typescript-eslint/no-unused-vars': ['error', { argsIgnorePattern: '^_' }],
-  },
-};
-
-export default variableRules;