@egs33/eslint-config 3.0.2 → 4.0.0

package/package.json

@@ -1,26 +1,26 @@
 {
   "name": "@egs33/eslint-config",
   "description": "eslint config for me",
-  "version": "3.0.2",
+  "version": "4.0.0",
   "author": "egs33",
   "type": "module",
   "dependencies": {
-    "@eslint/eslintrc": "^3.0.2",
-    "@typescript-eslint/eslint-plugin": "^7.1.0",
-    "@typescript-eslint/parser": "^7.1.0",
-    "eslint-config-airbnb-typescript": "^18.0.0",
-    "eslint-plugin-array-func": "^5.0.1",
-    "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-n": "^16.6.2",
-    "eslint-plugin-promise": "^6.1.1",
-    "eslint-plugin-regexp": "^2.2.0",
-    "globals": "^14.0.0"
+    "@eslint/js": "^9.14.0",
+    "@stylistic/eslint-plugin": "^2.10.0",
+    "@typescript-eslint/eslint-plugin": "^8.12.2",
+    "@typescript-eslint/parser": "^8.12.2",
+    "eslint-plugin-array-func": "^5.0.2",
+    "eslint-plugin-import": "^2.31.0",
+    "eslint-plugin-n": "^17.12.0",
+    "eslint-plugin-promise": "^7.1.0",
+    "eslint-plugin-regexp": "^2.6.0",
+    "globals": "^15.11.0"
   },
   "devDependencies": {
-    "eslint": "^8.57.0 || ^9.0.0",
-    "svelte-eslint-parser": "^0.33.1",
-    "typescript": "^5.3.3",
-    "typescript-eslint": "^7.1.0"
+    "eslint": "^9.13.0",
+    "svelte-eslint-parser": "^0.43.0",
+    "typescript": "^5.6.3",
+    "typescript-eslint": "^8.12.2"
   },
   "engines": {
     "node": ">=20"
@@ -45,10 +45,10 @@
   "license": "MIT",
   "main": "nodejs.js",
   "peerDependencies": {
-    "eslint": "^8.57.0",
-    "svelte-eslint-parser": ">=0.33.1",
-    "typescript": "^5.3.3",
-    "typescript-eslint": "^7.1.0"
+    "eslint": "^9.13.0",
+    "svelte-eslint-parser": ">=0.43.0",
+    "typescript": "^5.6.3",
+    "typescript-eslint": "^8.12.2"
   },
   "peerDependenciesMeta": {
     "typescript-eslint": {

package/plugin-rules/imports.js

@@ -124,10 +124,6 @@ export const imports = {
   // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/newline-after-import.md
   'import/newline-after-import': 'error',
 
-  // Require modules with a single export to use a default export
-  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/prefer-default-export.md
-  'import/prefer-default-export': 'error',
-
   // Restrict which files can be imported in a given folder
   // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-restricted-paths.md
   'import/no-restricted-paths': 'off',
@@ -196,10 +192,6 @@ export const imports = {
   // https://github.com/import-js/eslint-plugin-import/blob/44a038c06487964394b1e15b64f3bd34e5d40cde/docs/rules/group-exports.md
   'import/group-exports': 'off',
 
-  // forbid default exports. this is a terrible rule, do not use it.
-  // https://github.com/import-js/eslint-plugin-import/blob/44a038c06487964394b1e15b64f3bd34e5d40cde/docs/rules/no-default-export.md
-  'import/no-default-export': 'off',
-
   // Prohibit named exports. this is a terrible rule, do not use it.
   // https://github.com/import-js/eslint-plugin-import/blob/1ec80fa35fa1819e2d35a70e68fb6a149fb57c5e/docs/rules/no-named-export.md
   'import/no-named-export': 'off',
@@ -210,7 +202,7 @@ export const imports = {
 
   // Forbid cyclical dependencies between modules
   // https://github.com/import-js/eslint-plugin-import/blob/d81f48a2506182738409805f5272eff4d77c9348/docs/rules/no-cycle.md
-  'import/no-cycle': ['error', { maxDepth: '∞' }],
+  'import/no-cycle': ['error', { maxDepth: 20 }],
 
   // Ensures that there are no useless path segments
   // https://github.com/import-js/eslint-plugin-import/blob/ebafcbf59ec9f653b2ac2a0156ca3bcba0a7cf57/docs/rules/no-useless-path-segments.md

package/plugin-rules/stylistic.js

@@ -0,0 +1,186 @@
+export const rules = {
+  '@stylistic/dot-location': ['error', 'property'],
+
+  '@stylistic/no-floating-decimal': 'error',
+  '@stylistic/no-multi-spaces': [
+    'error', {
+      ignoreEOLComments: false,
+    },
+  ],
+  '@stylistic/wrap-iife': ['error', 'outside', { functionPrototypeMethods: false }],
+  '@stylistic/arrow-parens': ['error', 'always'],
+  '@stylistic/array-bracket-spacing': ['error', 'never'],
+  '@stylistic/block-spacing': ['error', 'always'],
+  '@stylistic/brace-style': ['error', '1tbs', { allowSingleLine: true }],
+  '@stylistic/comma-dangle': [
+    'error', {
+      arrays: 'always-multiline',
+      objects: 'always-multiline',
+      imports: 'always-multiline',
+      exports: 'always-multiline',
+      functions: 'always-multiline',
+    },
+  ],
+  '@stylistic/comma-spacing': ['error', { before: false, after: true }],
+  '@stylistic/comma-style': [
+    'error', 'last', {
+      exceptions: {
+        ArrayExpression: false,
+        ArrayPattern: false,
+        ArrowFunctionExpression: false,
+        CallExpression: false,
+        FunctionDeclaration: false,
+        FunctionExpression: false,
+        ImportDeclaration: false,
+        ObjectExpression: false,
+        ObjectPattern: false,
+        VariableDeclaration: false,
+        NewExpression: false,
+      },
+    },
+  ],
+  '@stylistic/computed-property-spacing': ['error', 'never'],
+  '@stylistic/eol-last': ['error', 'always'],
+  '@stylistic/function-call-argument-newline': ['error', 'consistent'],
+  '@stylistic/func-call-spacing': ['error', 'never'],
+  '@stylistic/function-paren-newline': ['error', 'multiline-arguments'],
+  '@stylistic/implicit-arrow-linebreak': ['error', 'beside'],
+  '@stylistic/indent': [
+    'error', 2, {
+      SwitchCase: 1,
+      VariableDeclarator: 1,
+      outerIIFEBody: 1,
+      // MemberExpression: null,
+      FunctionDeclaration: {
+        parameters: 1,
+        body: 1,
+      },
+      FunctionExpression: {
+        parameters: 1,
+        body: 1,
+      },
+      CallExpression: {
+        arguments: 1,
+      },
+      ArrayExpression: 1,
+      ObjectExpression: 1,
+      ImportDeclaration: 1,
+      flatTernaryExpressions: false,
+      // list derived from https://github.com/benjamn/ast-types/blob/HEAD/def/jsx.js
+      ignoredNodes: ['JSXElement', 'JSXElement > *', 'JSXAttribute', 'JSXIdentifier', 'JSXNamespacedName', 'JSXMemberExpression', 'JSXSpreadAttribute', 'JSXExpressionContainer', 'JSXOpeningElement', 'JSXClosingElement', 'JSXFragment', 'JSXOpeningFragment', 'JSXClosingFragment', 'JSXText', 'JSXEmptyExpression', 'JSXSpreadChild'],
+      ignoreComments: false,
+    },
+  ],
+  '@stylistic/jsx-quotes': ['off', 'prefer-double'],
+  '@stylistic/key-spacing': ['error', { beforeColon: false, afterColon: true }],
+  '@stylistic/keyword-spacing': [
+    'error', {
+      before: true,
+      after: true,
+      overrides: {
+        return: { after: true },
+        throw: { after: true },
+        case: { after: true },
+      },
+    },
+  ],
+  '@stylistic/linebreak-style': ['error', 'unix'],
+  '@stylistic/lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: false }],
+  '@stylistic/lines-around-comment': 'off',
+  '@stylistic/max-len': [
+    'error', 120, 2, {
+      ignoreUrls: true,
+      ignoreComments: true,
+      ignoreRegExpLiterals: true,
+      ignoreStrings: true,
+      ignoreTemplateLiterals: true,
+    },
+  ],
+  '@stylistic/max-statements-per-line': ['off', { max: 1 }],
+  '@stylistic/multiline-comment-style': ['off', 'starred-block'],
+  '@stylistic/new-parens': 'error',
+  '@stylistic/newline-per-chained-call': ['error', { ignoreChainWithDepth: 4 }],
+  '@stylistic/no-mixed-operators': [
+    'error', {
+      // the list of arithmetic groups disallows mixing `%` and `**`
+      // with other arithmetic operators.
+      groups: [
+        ['%', '**'],
+        ['%', '+'],
+        ['%', '-'],
+        ['%', '*'],
+        ['%', '/'],
+        ['/', '*'],
+        ['&', '|', '<<', '>>', '>>>'],
+        ['==', '!=', '===', '!=='],
+        ['&&', '||'],
+      ],
+      allowSamePrecedence: false,
+    },
+  ],
+  '@stylistic/no-mixed-spaces-and-tabs': 'error',
+  '@stylistic/no-multiple-empty-lines': ['error', { max: 1, maxBOF: 0, maxEOF: 0 }],
+  '@stylistic/no-tabs': 'error',
+  '@stylistic/no-trailing-spaces': [
+    'error', {
+      skipBlankLines: false,
+      ignoreComments: false,
+    },
+  ],
+  '@stylistic/no-whitespace-before-property': 'error',
+  '@stylistic/nonblock-statement-body-position': ['error', 'beside', { overrides: {} }],
+  '@stylistic/object-curly-spacing': ['error', 'always'],
+  '@stylistic/object-curly-newline': [
+    'error', {
+      ObjectExpression: { minProperties: 4, multiline: true, consistent: true },
+      ObjectPattern: { minProperties: 4, multiline: true, consistent: true },
+      ImportDeclaration: { minProperties: 4, multiline: true, consistent: true },
+      ExportDeclaration: { minProperties: 4, multiline: true, consistent: true },
+    },
+  ],
+  '@stylistic/object-property-newline': [
+    'error', {
+      allowAllPropertiesOnSameLine: true,
+    },
+  ],
+  '@stylistic/quote-props': ['error', 'as-needed', { keywords: false, unnecessary: true, numbers: false }],
+  '@stylistic/quotes': ['error', 'single', { avoidEscape: true }],
+  '@stylistic/semi': ['error', 'always'],
+  '@stylistic/semi-spacing': ['error', { before: false, after: true }],
+  '@stylistic/semi-style': ['error', 'last'],
+  '@stylistic/space-before-blocks': 'error',
+  '@stylistic/space-before-function-paren': [
+    'error', {
+      anonymous: 'always',
+      named: 'never',
+      asyncArrow: 'always',
+    },
+  ],
+  '@stylistic/space-in-parens': ['error', 'never'],
+  '@stylistic/space-infix-ops': 'error',
+  '@stylistic/space-unary-ops': [
+    'error', {
+      words: true,
+      nonwords: false,
+      overrides: {
+      },
+    },
+  ],
+  '@stylistic/spaced-comment': [
+    'error', 'always', {
+      line: {
+        exceptions: ['-', '+'],
+        markers: ['=', '!', '/'], // space here to support sprockets directives, slash for TS /// comments
+      },
+      block: {
+        exceptions: ['-', '+'],
+        markers: ['=', '!', ':', '::'], // space here to support sprockets directives and flow comment types
+        balanced: true,
+      },
+    },
+  ],
+  '@stylistic/switch-colon-spacing': ['error', { after: true, before: false }],
+  '@stylistic/template-tag-spacing': ['error', 'never'],
+  '@stylistic/wrap-regex': 'off',
+  '@stylistic/array-bracket-newline': 'error',
+};

package/typescript-base.js

@@ -1,16 +1,11 @@
-import { FlatCompat } from '@eslint/eslintrc';
 // eslint-disable-next-line import/no-unresolved
 import tseslint from 'typescript-eslint';
 import base from './base.js';
 
-const compat = new FlatCompat();
-
 const config = [
   ...base,
-  { plugins: { '@typescript-eslint': tseslint.plugin } },
-  ...tseslint.configs.strictTypeChecked.map((c) => ({ ...c, plugins: {} })),
-  ...tseslint.configs.stylisticTypeChecked.map((c) => ({ ...c, plugins: {} })),
-  ...compat.extends('airbnb-typescript/base'),
+  ...tseslint.configs.strictTypeChecked,
+  ...tseslint.configs.stylisticTypeChecked,
   {
     languageOptions: {
       parserOptions: {
@@ -18,6 +13,57 @@ const config = [
       },
     },
     rules: {
+      // disable eslint rules for use typescript-eslint rules
+      'default-param-last': 'off',
+      'no-array-constructor': 'off',
+      'no-dupe-class-members': 'off',
+      'no-empty-function': 'off',
+      'no-implied-eval': 'off',
+      'no-new-func': 'off',
+      'no-loss-of-precision': 'off',
+      'no-loop-func': 'off',
+      'no-magic-numbers': 'off',
+      'no-redeclare': 'off',
+      'no-shadow': 'off',
+      'no-throw-literal': 'off',
+      'no-use-before-define': 'off',
+      'no-useless-constructor': 'off',
+
+      camelcase: 'off',
+      '@typescript-eslint/naming-convention': [
+        'error',
+        {
+          selector: 'variable',
+          format: ['camelCase', 'PascalCase', 'UPPER_CASE'],
+        },
+        // Allow camelCase functions (23.2), and PascalCase functions (23.8)
+        {
+          selector: 'function',
+          format: ['camelCase', 'PascalCase'],
+        },
+        {
+          selector: 'typeLike',
+          format: ['PascalCase'],
+        },
+      ],
+
+      'dot-notation': 'off',
+      '@typescript-eslint/dot-notation': ['error', { allowKeywords: true }],
+      'no-unused-expressions': 'off',
+      '@typescript-eslint/no-unused-expressions': [
+        'error', {
+          allowShortCircuit: false,
+          allowTernary: false,
+          allowTaggedTemplates: false,
+        },
+      ],
+      'no-unused-vars': 'off',
+      '@typescript-eslint/no-unused-vars': ['error', { vars: 'all', args: 'after-used', ignoreRestSiblings: true }],
+      'require-await': 'off',
+      '@typescript-eslint/require-await': 'off',
+      'no-return-await': 'off',
+      '@typescript-eslint/return-await': ['error', 'in-try-catch'],
+
       '@typescript-eslint/method-signature-style': ['error', 'property'],
       '@typescript-eslint/restrict-template-expressions': 'off',
       '@typescript-eslint/consistent-type-imports': 'error',
@@ -30,7 +76,7 @@ const config = [
       '@typescript-eslint/prefer-optional-chain': 'error',
       '@typescript-eslint/prefer-nullish-coalescing': 'error',
       '@typescript-eslint/prefer-string-starts-ends-with': 'error',
-      '@typescript-eslint/type-annotation-spacing': 'error',
+      '@stylistic/type-annotation-spacing': 'error',
       '@typescript-eslint/consistent-type-definitions': ['error', 'type'],
     },
   },

package/typescript-svelte.js

@@ -72,8 +72,7 @@ export default [
       'svelte/html-closing-bracket-spacing': 'error',
       'svelte/html-quotes': 'error',
       'svelte/indent': 'error',
-      indent: 'off',
-      '@typescript-eslint/indent': 'off',
+      '@stylistic/indent': 'off',
       'svelte/max-attributes-per-line': ['error', { singleline: 4 }],
       'svelte/mustache-spacing': 'error',
       'svelte/no-extra-reactive-curlies': 'error',