@egs33/eslint-config 2.2.1 → 3.0.1

package/package.json

@@ -1,43 +1,41 @@
 {
   "name": "@egs33/eslint-config",
   "description": "eslint config for me",
-  "version": "2.2.1",
+  "version": "3.0.1",
   "author": "egs33",
   "type": "module",
   "dependencies": {
-    "@eslint/eslintrc": "^2.0.3",
-    "eslint-config-airbnb-base": "^15.0.0",
-    "eslint-config-airbnb-typescript": "^17.0.0",
-    "eslint-plugin-array-func": "^3.1.7",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-n": "^16.0.1",
-    "eslint-plugin-promise": "^6.0.1",
-    "eslint-plugin-regexp": "^1.9.0",
-    "globals": "^13.20.0"
+    "@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"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^6.1.0",
-    "@typescript-eslint/parser": "^6.1.0",
-    "eslint": "^8.44.0",
-    "eslint-plugin-vue": "^9.5.0",
-    "svelte-eslint-parser": "^0.32.2",
-    "typescript": "^5.0.2",
-    "vue-eslint-parser": "^9.1.0"
+    "eslint": "^8.57.0 || ^9.0.0",
+    "svelte-eslint-parser": "^0.33.1",
+    "typescript": "^5.3.3",
+    "typescript-eslint": "^7.1.0"
   },
   "engines": {
-    "node": ">=16"
+    "node": ">=20"
   },
   "files": [
     "util.js",
     "base.js",
     "browser.js",
     "nodejs.js",
-    "vue3.js",
     "typescript-base.js",
     "typescript-node.js",
     "typescript-browser.js",
     "typescript-svelte.js",
-    "plugin-rules"
+    "plugin-rules",
+    "airbnb"
   ],
   "homepage": "https://github.com/egs33/eslint-config",
   "keywords": [
@@ -47,30 +45,18 @@
   "license": "MIT",
   "main": "nodejs.js",
   "peerDependencies": {
-    "@typescript-eslint/eslint-plugin": "^6.1.0",
-    "@typescript-eslint/parser": "^6.1.0",
-    "eslint": "^8.44.0",
-    "eslint-plugin-vue": "^9.5.0",
-    "svelte-eslint-parser": ">=0.32.2",
-    "typescript": ">=4.4.4",
-    "vue-eslint-parser": "^9.1.0"
+    "eslint": "^8.57.0",
+    "svelte-eslint-parser": ">=0.33.1",
+    "typescript": "^5.3.3",
+    "typescript-eslint": "^7.1.0"
   },
   "peerDependenciesMeta": {
-    "@typescript-eslint/eslint-plugin": {
-      "optional": true
-    },
-    "@typescript-eslint/parser": {
-      "optional": true
-    },
-    "eslint-plugin-vue": {
+    "typescript-eslint": {
       "optional": true
     },
     "svelte-eslint-parser": {
       "optional": true
     },
-    "vue-eslint-parser": {
-      "optional": true
-    },
     "typescript": {
       "optional": true
     }

package/plugin-rules/imports.js

@@ -0,0 +1,265 @@
+export const imports = {
+
+  // ensure imports point to files/modules that can be resolved
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md
+  'import/no-unresolved': ['error', { commonjs: true, caseSensitive: true }],
+
+  // ensure named imports coupled with named exports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/named.md#when-not-to-use-it
+  'import/named': 'error',
+
+  // ensure default import coupled with default export
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/default.md#when-not-to-use-it
+  'import/default': 'off',
+
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/namespace.md
+  'import/namespace': 'off',
+
+  // Helpful warnings:
+
+  // disallow invalid exports, e.g. multiple defaults
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/export.md
+  'import/export': 'error',
+
+  // do not allow a default import name to match a named export
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-named-as-default.md
+  'import/no-named-as-default': 'error',
+
+  // warn on accessing default export property names that are also named exports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-named-as-default-member.md
+  'import/no-named-as-default-member': 'error',
+
+  // disallow use of jsdoc-marked-deprecated imports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-deprecated.md
+  'import/no-deprecated': 'off',
+
+  // Forbid the use of extraneous packages
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-extraneous-dependencies.md
+  // paths are treated both as absolute paths, and relative to process.cwd()
+  'import/no-extraneous-dependencies': [
+    'error', {
+      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}', // repos with a single test file
+        'test-*.{js,jsx}', // repos with multiple top-level test files
+        '**/*{.,_}{test,spec}.{js,jsx}', // tests where the extension or filename suffix denotes that it is a test
+        '**/jest.config.js', // jest config
+        '**/jest.setup.js', // jest setup
+        '**/vue.config.js', // 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
+      ],
+      optionalDependencies: false,
+    },
+  ],
+
+  // Forbid mutable exports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-mutable-exports.md
+  'import/no-mutable-exports': 'error',
+
+  // Module systems:
+
+  // disallow require()
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-commonjs.md
+  'import/no-commonjs': 'off',
+
+  // disallow AMD require/define
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-amd.md
+  'import/no-amd': 'error',
+
+  // No Node.js builtin modules
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-nodejs-modules.md
+  // TODO: enable?
+  'import/no-nodejs-modules': 'off',
+
+  // Style guide:
+
+  // disallow non-import statements appearing before import statements
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/first.md
+  'import/first': 'error',
+
+  // disallow non-import statements appearing before import statements
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/imports-first.md
+  // deprecated: use `import/first`
+  'import/imports-first': 'off',
+
+  // disallow duplicate imports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
+  'import/no-duplicates': 'error',
+
+  // disallow namespace imports
+  // TODO: enable?
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-namespace.md
+  'import/no-namespace': 'off',
+
+  // Ensure consistent use of file extension within the import path
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/extensions.md
+  'import/extensions': [
+    'error', 'ignorePackages', {
+      js: 'never',
+      mjs: 'never',
+      jsx: 'never',
+    },
+  ],
+
+  // ensure absolute imports are above relative imports and that unassigned imports are ignored
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/order.md
+  // TODO: enforce a stricter convention in module import order?
+  'import/order': ['error', { groups: [['builtin', 'external', 'internal']] }],
+
+  // Require a newline after the last import/require in a group
+  // 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',
+
+  // Forbid modules to have too many dependencies
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/max-dependencies.md
+  'import/max-dependencies': ['off', { max: 10 }],
+
+  // Forbid import of modules using absolute paths
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-absolute-path.md
+  'import/no-absolute-path': 'error',
+
+  // Forbid require() calls with expressions
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-dynamic-require.md
+  'import/no-dynamic-require': 'error',
+
+  // prevent importing the submodules of other modules
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-internal-modules.md
+  'import/no-internal-modules': [
+    'off', {
+      allow: [],
+    },
+  ],
+
+  // Warn if a module could be mistakenly parsed as a script by a consumer
+  // leveraging Unambiguous JavaScript Grammar
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/unambiguous.md
+  // this should not be enabled until this proposal has at least been *presented* to TC39.
+  // At the moment, it's not a thing.
+  'import/unambiguous': 'off',
+
+  // Forbid Webpack loader syntax in imports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-webpack-loader-syntax.md
+  'import/no-webpack-loader-syntax': 'error',
+
+  // Prevent unassigned imports
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-unassigned-import.md
+  // importing for side effects is perfectly acceptable, if you need side effects.
+  'import/no-unassigned-import': 'off',
+
+  // Prevent importing the default as if it were named
+  // https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-named-default.md
+  'import/no-named-default': 'error',
+
+  // Reports if a module's default export is unnamed
+  // https://github.com/import-js/eslint-plugin-import/blob/d9b712ac7fd1fddc391f7b234827925c160d956f/docs/rules/no-anonymous-default-export.md
+  'import/no-anonymous-default-export': [
+    'off', {
+      allowArray: false,
+      allowArrowFunction: false,
+      allowAnonymousClass: false,
+      allowAnonymousFunction: false,
+      allowLiteral: false,
+      allowObject: false,
+    },
+  ],
+
+  // This rule enforces that all exports are declared at the bottom of the file.
+  // https://github.com/import-js/eslint-plugin-import/blob/98acd6afd04dcb6920b81330114e146dc8532ea4/docs/rules/exports-last.md
+  // TODO: enable?
+  'import/exports-last': 'off',
+
+  // Reports when named exports are not grouped together in a single export declaration
+  // or when multiple assignments to CommonJS module.exports or exports object are present
+  // in a single file.
+  // 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',
+
+  // Forbid a module from importing itself
+  // https://github.com/import-js/eslint-plugin-import/blob/44a038c06487964394b1e15b64f3bd34e5d40cde/docs/rules/no-self-import.md
+  'import/no-self-import': 'error',
+
+  // 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: '∞' }],
+
+  // 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
+  'import/no-useless-path-segments': ['error', { commonjs: true }],
+
+  // dynamic imports require a leading comment with a webpackChunkName
+  // https://github.com/import-js/eslint-plugin-import/blob/ebafcbf59ec9f653b2ac2a0156ca3bcba0a7cf57/docs/rules/dynamic-import-chunkname.md
+  'import/dynamic-import-chunkname': [
+    'off', {
+      importFunctions: [],
+      webpackChunknameFormat: '[0-9a-zA-Z-_/.]+',
+    },
+  ],
+
+  // Use this rule to prevent imports to folders in relative parent paths.
+  // https://github.com/import-js/eslint-plugin-import/blob/c34f14f67f077acd5a61b3da9c0b0de298d20059/docs/rules/no-relative-parent-imports.md
+  'import/no-relative-parent-imports': 'off',
+
+  // Reports modules without any exports, or with unused exports
+  // https://github.com/import-js/eslint-plugin-import/blob/f63dd261809de6883b13b6b5b960e6d7f42a7813/docs/rules/no-unused-modules.md
+  // TODO: enable once it supports CJS
+  'import/no-unused-modules': [
+    'off', {
+      ignoreExports: [],
+      missingExports: true,
+      unusedExports: true,
+    },
+  ],
+
+  // Reports the use of import declarations
+  // with CommonJS exports in any module except for the main module.
+  // https://github.com/import-js/eslint-plugin-import/blob/1012eb951767279ce3b540a4ec4f29236104bb5b/docs/rules/no-import-module-exports.md
+  'import/no-import-module-exports': [
+    'error', {
+      exceptions: [],
+    },
+  ],
+
+  // Use this rule to prevent importing packages through relative paths.
+  // https://github.com/import-js/eslint-plugin-import/blob/1012eb951767279ce3b540a4ec4f29236104bb5b/docs/rules/no-relative-packages.md
+  'import/no-relative-packages': 'error',
+
+  // enforce a consistent style for type specifiers (inline or top-level)
+  // https://github.com/import-js/eslint-plugin-import/blob/d5fc8b670dc8e6903dbb7b0894452f60c03089f5/docs/rules/consistent-type-specifier-style.md
+  // TODO, semver-major: enable (just in case)
+  'import/consistent-type-specifier-style': ['off', 'prefer-inline'],
+
+  // Reports the use of empty named import blocks.
+  // https://github.com/import-js/eslint-plugin-import/blob/d5fc8b670dc8e6903dbb7b0894452f60c03089f5/docs/rules/no-empty-named-blocks.md
+  // TODO, semver-minor: enable
+  'import/no-empty-named-blocks': 'off',
+};

package/typescript-base.js

@@ -1,12 +1,15 @@
 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,
-  ...compat.extends('plugin:@typescript-eslint/strict-type-checked'),
-  ...compat.extends('plugin:@typescript-eslint/stylistic-type-checked'),
+  { '@typescript-eslint': tseslint.plugin },
+  ...tseslint.configs.strictTypeChecked.map(c => ({ ...c, plugins: {} })),
+  ...tseslint.configs.stylisticTypeChecked.map(c => ({ ...c, plugins: {} })),
   ...compat.extends('airbnb-typescript/base'),
   {
     languageOptions: {

package/typescript-svelte.js

@@ -1,6 +1,6 @@
 import svelteParser from 'svelte-eslint-parser';
 // eslint-disable-next-line import/no-unresolved
-import tsParser from '@typescript-eslint/parser';
+import tseslint from 'typescript-eslint';
 import typescriptBrowser from './typescript-browser.js';
 
 export default [
@@ -9,7 +9,7 @@ export default [
     languageOptions: {
       parser: svelteParser,
       parserOptions: {
-        parser: tsParser,
+        parser: tseslint.parser,
         project: './tsconfig.eslint.json',
         extraFileExtensions: ['.svelte'],
       },
@@ -46,10 +46,12 @@ export default [
       'svelte/no-at-html-tags': 'error',
       'svelte/no-target-blank': 'off', // use noreferrer only is ok
 
-      'svelte/block-lang': ['error', {
-        script: ['ts', null],
-        style: ['scss', null],
-      }],
+      'svelte/block-lang': [
+        'error', {
+          script: ['ts', null],
+          style: ['scss', null],
+        },
+      ],
       'svelte/button-has-type': 'error',
       'svelte/no-at-debug-tags': 'warn',
       'svelte/no-immutable-reactive-statements': 'error',

package/vue3.js

@@ -1,116 +0,0 @@
-import { FlatCompat } from '@eslint/eslintrc';
-import parser from 'vue-eslint-parser';
-import browser from './browser.js';
-
-const compat = new FlatCompat();
-
-const config = [
-  ...browser,
-  ...compat.extends('plugin:vue/vue3-recommended'),
-  {
-    languageOptions: {
-      parser,
-    },
-    rules: {
-      'import/no-default-export': 'off',
-      'vue/max-attributes-per-line': ['error', {
-        singleline: 4,
-      }],
-      'vue/html-indent': ['error', 2],
-      'vue/html-closing-bracket-newline': ['error', {
-        singleline: 'never',
-        multiline: 'never',
-      }],
-      // use airbnb rules in vue templates
-      'vue/array-bracket-spacing': 'error',
-      'vue/arrow-spacing': 'error',
-      'vue/block-spacing': 'error',
-      'vue/brace-style': [
-        'error',
-        '1tbs',
-        {
-          allowSingleLine: true,
-        },
-      ],
-      'vue/camelcase': [
-        'error',
-        {
-          properties: 'never',
-        },
-      ],
-      'vue/comma-dangle': [
-        'error',
-        {
-          arrays: 'always-multiline',
-          objects: 'always-multiline',
-          imports: 'always-multiline',
-          exports: 'always-multiline',
-          functions: 'always-multiline',
-        },
-      ],
-      'vue/comma-spacing': 'error',
-      'vue/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,
-          },
-        },
-      ],
-      'vue/dot-location': [
-        'error',
-        'property',
-      ],
-      'vue/dot-notation': 'error',
-      'vue/eqeqeq': 'error',
-      'vue/key-spacing': 'error',
-      'vue/keyword-spacing': 'error',
-      'vue/max-len': ['error', {
-        code: 100,
-        template: 100,
-        comments: 100,
-        ignoreUrls: true,
-        ignoreRegExpLiterals: true,
-        ignoreStrings: true,
-        ignoreTemplateLiterals: true,
-      }],
-      'vue/no-empty-pattern': 'error',
-      'vue/no-irregular-whitespace': 'error',
-      'vue/no-restricted-syntax': [
-        'error',
-        'ForInStatement',
-        'LabeledStatement',
-        'WithStatement',
-      ],
-      'vue/no-useless-concat': 'error',
-      'vue/object-curly-spacing': [
-        'error',
-        'always',
-      ],
-      'vue/prefer-template': 'error',
-      'vue/space-in-parens': 'error',
-      'vue/space-infix-ops': 'error',
-      'vue/space-unary-ops': 'error',
-      'vue/template-curly-spacing': 'error',
-    },
-  },
-];
-
-// default name '.vue' is invalid processer name in flat config
-const vueConfig = config.find((c) => c?.plugins?.vue);
-if (vueConfig) {
-  vueConfig.plugins.vue.processors.vue = vueConfig.plugins.vue.processors['.vue'];
-}
-
-export default [...config, { processor: 'vue/vue' }];