@viclafouch/eslint-config-viclafouch 3.9.2 → 3.9.3-beta.1

package/rules/variables.js

@@ -0,0 +1,74 @@
+/**
+ * @type {import("eslint").Linter.Config}
+ */
+module.exports = {
+  rules: {
+    // enforce or disallow variable initializations at definition
+    'init-declarations': 'off',
+
+    // disallow the catch clause parameter name being the same as a variable in the outer scope
+    'no-catch-shadow': 'off',
+
+    // disallow deletion of variables
+    '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
+    'no-shadow': 'error',
+
+    // For code readability, prevent creating unclear naming
+    'id-length': [
+      'error',
+      { min: 2, max: Infinity, exceptions: ['t', '_'], properties: 'never' }
+    ],
+
+    // Require the second parameter when using `parseInt`.
+    // Ref: https://eslint.org/docs/rules/radix
+    radix: 2,
+
+    // Prefer object shorthands for properties.
+    // Ref: https://eslint.org/docs/rules/object-shorthand
+    'object-shorthand': [2, 'properties'],
+
+    // disallow shadowing of names such as arguments
+    'no-shadow-restricted-names': 'error',
+
+    // disallow use of undeclared variables unless mentioned in a /*global */ block
+    'no-undef': 'error',
+
+    // disallow use of undefined when initializing variables
+    'no-undef-init': 'error',
+
+    // Most common naming that is not always understandable
+    'id-denylist': [
+      'error',
+      'err',
+      'cb',
+      'arr',
+      'acc',
+      'idx',
+      'ctx',
+      'res',
+      'val',
+      'obj',
+      'el',
+      'elem',
+      'req'
+    ],
+
+    // disallow declaration of variables that are not used in the code
+    'no-unused-vars': [
+      'error',
+      { vars: 'all', args: 'after-used', ignoreRestSiblings: true }
+    ],
+
+    // disallow use of variables before they are defined
+    'no-use-before-define': [
+      'error',
+      { functions: true, classes: true, variables: true }
+    ]
+  }
+}

package/tsconfig.json

@@ -0,0 +1,21 @@
+{
+  "compilerOptions": {
+    "target": "esnext",
+    "lib": ["dom", "dom.iterable", "esnext"],
+    "allowJs": true,
+    "skipLibCheck": true,
+    "strict": true,
+    "forceConsistentCasingInFileNames": true,
+    "noEmit": true,
+    "esModuleInterop": true,
+    "module": "esnext",
+    "noUncheckedIndexedAccess": true,
+    "noPropertyAccessFromIndexSignature": true,
+    "moduleResolution": "node",
+    "resolveJsonModule": true,
+    "isolatedModules": true,
+    "noImplicitAny": true,
+    "jsx": "preserve",
+    "incremental": true
+  }
+}

package/typescript.js

@@ -1,17 +1,7 @@
+/**
+ * @type {import("eslint").Linter.Config}
+ */
 module.exports = {
-  extends: [
-    'airbnb',
-    'airbnb-typescript',
-    'airbnb/hooks',
-    'plugin:@typescript-eslint/recommended',
-    'plugin:@typescript-eslint/recommended-requiring-type-checking',
-    'plugin:prettier/recommended',
-    './.eslintrc-base.js'
-  ],
-  parser: '@typescript-eslint/parser',
-  plugins: ['react', '@typescript-eslint', 'simple-import-sort'],
-  rules: {
-    '@typescript-eslint/ban-ts-comment': 'off',
-    'react/jsx-filename-extension': [1, { extensions: ['.ts', '.tsx'] }]
-  }
+  extends: ['./rules/typescript.js'].map(require.resolve),
+  rules: {}
 }

package/.eslintrc-base.js

@@ -1,138 +0,0 @@
-module.exports = {
-  plugins: ['react', 'simple-import-sort', 'promise'],
-  env: {
-    browser: true,
-    es6: true,
-    node: true
-  },
-  settings: {
-    react: {
-      version: 'detect'
-    }
-  },
-  rules: {
-    'id-length': [
-      'error',
-      { min: 2, max: Infinity, exceptions: ['t', '_'], properties: 'never' }
-    ],
-    'react/jsx-no-leaked-render': [2, { validStrategies: ['ternary'] }],
-    'id-denylist': [
-      'error',
-      'err',
-      'cb',
-      'arr',
-      'acc',
-      'idx',
-      'ctx',
-      'res',
-      'val',
-      'obj',
-      'src',
-      'el',
-      'elem',
-      'req'
-    ],
-    'react/prop-types': 'off',
-    'simple-import-sort/imports': 'error',
-    'react/jsx-props-no-spreading': 0,
-    'no-return-await': 2,
-    'import/prefer-default-export': 0,
-    'react/jsx-boolean-value': 2,
-    'react/self-closing-comp': 2,
-    'react/hook-use-state': 2,
-    'react/default-props-match-prop-types': 2,
-    'react/no-unused-prop-types': 2,
-    'react/no-unused-prop-types': 2,
-    'promise/prefer-await-to-then': 2,
-    'require-await': 2,
-    'react/sort-prop-types': [
-      2,
-      {
-        requiredFirst: true,
-        callbacksLast: true,
-        sortShapeProp: true
-      }
-    ],
-    curly: 2,
-    'arrow-body-style': ['error', 'always'],
-    'no-restricted-syntax': [
-      'error',
-      {
-        selector:
-          "JSXElement > JSXExpressionContainer > LogicalExpression[operator!='??']",
-        message: 'Please use ternary operator instead'
-      }
-    ],
-    'react/function-component-definition': [
-      2,
-      { namedComponents: 'arrow-function' }
-    ],
-    // Prefer to have a convention for naming states
-    // e.g: [thing, setThing]
-    'react/hook-use-state': 2,
-
-    // Prevent the boolean true as prop value
-    // e.g: <MyComponent isValid /> instead of <MyComponent isValid={true} />
-    'react/jsx-boolean-value': 2,
-
-    // defaultProps is deprecated
-    'react/require-default-props': [
-      'error',
-      {
-        functions: 'defaultArguments'
-      }
-    ],
-    'prettier/prettier': [
-      'error',
-      {
-        semi: false,
-        singleQuote: true,
-        printWidth: 80,
-        tabWidth: 2,
-        jsxSingleQuote: false,
-        trailingComma: 'none',
-        endOfLine: 'auto',
-        bracketSameLine: false,
-        arrowParens: 'always'
-      }
-    ]
-  },
-  overrides: [
-    {
-      files: ['*.jsx', '*.js', '*.ts', '*.tsx'],
-      rules: {
-        'simple-import-sort/imports': [
-          'error',
-          {
-            groups: [
-              [
-                // Anything that starts with react
-                // e.g: import { useState } from 'react'
-                // e.g: import { useFela } from 'react-fela'
-                '^react',
-                // Anything that starts with next
-                // e.g: import { useRouter } from 'next/router'
-                '^next',
-                // Anything that starts with a letter
-                // e.g: import Downshift from 'downshift'
-                '^[a-z]',
-                // Anything that starts with @
-                // e.g: import { yupResolver } from '@hookform/resolvers/yup'
-                '^@',
-                // Anything that starts with a dot
-                // e.g: import { matchIsDate } from './utils/date
-                '^\\.',
-                // Side effect imports from lib
-                // e.g: import 'react-toastify/dist/ReactToastify.css'
-                '^\\u0000',
-                // Side effect import that starts with a dot
-                // e.g: import './setup-config'
-                '^\\u0000\\.'
-              ]
-            ]
-          }
-        ]
-      }
-    }
-  ]
-}

package/.eslintrc.js

@@ -1,17 +0,0 @@
-module.exports = {
-  extends: [
-    'airbnb',
-    'airbnb/hooks',
-    'plugin:prettier/recommended',
-    './.eslintrc-base.js'
-  ],
-  parser: '@babel/eslint-parser',
-  parserOptions: {
-    requireConfigFile: false,
-    ecmaVersion: 'latest',
-    sourceType: 'module',
-    ecmaFeatures: {
-      jsx: true
-    }
-  }
-}