@egs33/eslint-config 4.1.2 → 4.2.0

package/airbnb/best-practices.js

@@ -37,7 +37,6 @@ export const bestPractices = {
   'no-extra-bind': 'error',
   'no-extra-label': 'error',
   'no-global-assign': ['error', { exceptions: [] }],
-  'no-native-reassign': 'off',
   'no-implicit-coercion': [
     'off', {
       boolean: false,
@@ -128,7 +127,6 @@ export const bestPractices = {
     },
   ],
   'no-return-assign': ['error', 'always'],
-  'no-return-await': 'error',
   'no-script-url': 'error',
   'no-self-assign': [
     'error', {

package/airbnb/errors.js

@@ -6,8 +6,8 @@ export const errors = {
   'no-constant-binary-expression': 'error',
   'no-constant-condition': 'warn',
   'no-empty-character-class': 'error',
-  'no-extra-parens': ['error', 'all', { nestedBinaryExpressions: false }],
-  'no-extra-semi': 'error',
+  '@stylistic/no-extra-parens': ['error', 'all', { nestedBinaryExpressions: false }],
+  '@stylistic/no-extra-semi': 'error',
   'no-inner-declarations': 'error',
   'no-invalid-regexp': 'error',
   'no-new-native-nonconstructor': 'error',

package/airbnb/es6.js

@@ -4,15 +4,14 @@ export const es6 = {
       requireReturnForObjectLiteral: false,
     },
   ],
-  'arrow-spacing': ['error', { before: true, after: true }],
-  'generator-star-spacing': ['error', { before: false, after: true }],
-  'no-confusing-arrow': [
+  '@stylistic/arrow-spacing': ['error', { before: true, after: true }],
+  '@stylistic/generator-star-spacing': ['error', { before: false, after: true }],
+  '@stylistic/no-confusing-arrow': [
     'error', {
       allowParens: true,
     },
   ],
   'no-duplicate-imports': 'off',
-  'no-new-symbol': 'error',
   'no-restricted-exports': [
     'error', {
       restrictedNamedExports: [
@@ -73,8 +72,8 @@ export const es6 = {
   'prefer-rest-params': 'error',
   'prefer-spread': 'error',
   'prefer-template': 'error',
-  'rest-spread-spacing': ['error', 'never'],
+  '@stylistic/rest-spread-spacing': ['error', 'never'],
   'symbol-description': 'error',
-  'template-curly-spacing': 'error',
-  'yield-star-spacing': ['error', 'after'],
+  '@stylistic/template-curly-spacing': 'error',
+  '@stylistic/yield-star-spacing': ['error', 'after'],
 };

package/airbnb/style.js

@@ -1,12 +1,7 @@
 export const style = {
   camelcase: ['error', { properties: 'never', ignoreDestructuring: false }],
   'func-names': 'warn',
-  'lines-around-directive': [
-    'error', {
-      before: 'always',
-      after: 'always',
-    },
-  ],
+  '@stylistic/padding-line-between-statements': ['error', { blankLine: 'always', prev: 'directive', next: '*' }],
   'max-lines-per-function': [
     'off', {
       max: 50,
@@ -28,8 +23,7 @@ export const style = {
   'no-lonely-if': 'error',
   'no-multi-assign': ['error'],
   'no-nested-ternary': 'error',
-  'no-new-object': 'error',
-  'no-spaced-func': 'off',
+  '@stylistic/no-spaced-func': 'off',
   'no-underscore-dangle': [
     'error', {
       allow: [],

package/base.js

@@ -4,6 +4,8 @@ import array from 'eslint-plugin-array-func';
 import stylistic from '@stylistic/eslint-plugin';
 import eslintJs from '@eslint/js';
 import { configs as regexpConfig } from 'eslint-plugin-regexp';
+import eslintPluginUnicorn from 'eslint-plugin-unicorn';
+import { defineConfig } from 'eslint/config';
 import { rules as PromiseRules } from './plugin-rules/promise.js';
 import { rules as StyleRules } from './plugin-rules/stylistic.js';
 import { bestPractices } from './airbnb/best-practices.js';
@@ -12,8 +14,9 @@ import { style } from './airbnb/style.js';
 import { variables } from './airbnb/variables.js';
 import { es6 } from './airbnb/es6.js';
 import { imports } from './plugin-rules/imports.js';
+import { rules as UnicornRules } from './plugin-rules/unicorn.js';
 
-export default [
+export default defineConfig([
   {
     languageOptions: {
       ecmaVersion: 2023,
@@ -22,11 +25,12 @@ export default [
         ecmaVersion: 2023,
       },
     },
-    plugins: { import: importPlugin, promise },
+    plugins: { promise },
     linterOptions: {
       reportUnusedDisableDirectives: true,
     },
     settings: {
+      'import/resolver': 'typescript', // Supports "exports" in package.json
       'import/parsers': {
         espree: ['.js', '.cjs', '.mjs', '.jsx'],
       },
@@ -46,8 +50,10 @@ export default [
     semi: true,
   }),
   eslintJs.configs.recommended,
+  importPlugin.flatConfigs.recommended,
   array.configs.all,
   regexpConfig['flat/recommended'],
+  eslintPluginUnicorn.configs.recommended,
   {
     rules: {
       ...bestPractices,
@@ -66,6 +72,7 @@ export default [
       'array-func/prefer-array-from': 'off',
       ...PromiseRules,
       ...StyleRules,
+      ...UnicornRules,
     },
   },
-];
+]);

package/browser.js

@@ -1,7 +1,8 @@
 import globals from 'globals';
+import { defineConfig } from 'eslint/config';
 import base from './base.js';
 
-export default [
+export default defineConfig([
   ...base,
   {
     languageOptions: {
@@ -10,4 +11,4 @@ export default [
       },
     },
   },
-];
+]);

package/nodejs.js

@@ -1,8 +1,9 @@
 import node from 'eslint-plugin-n/configs/recommended-module.js';
 import globals from 'globals';
+import { defineConfig } from 'eslint/config';
 import base from './base.js';
 
-export default [
+export default defineConfig([
   ...base,
   node,
   {
@@ -20,4 +21,4 @@ export default [
       'import/extensions': ['error', 'always', { ignorePackages: true }],
     },
   },
-];
+]);

package/package.json

@@ -1,26 +1,28 @@
 {
   "name": "@egs33/eslint-config",
   "description": "eslint config for me",
-  "version": "4.1.2",
+  "version": "4.2.0",
   "author": "egs33",
   "type": "module",
   "dependencies": {
-    "@eslint/js": "^9.23.0",
-    "@stylistic/eslint-plugin": "^5.2.3",
-    "@typescript-eslint/eslint-plugin": "^8.28.0",
-    "@typescript-eslint/parser": "^8.28.0",
-    "eslint-plugin-array-func": "^5.0.2",
+    "@eslint/js": "^9.39.2",
+    "@stylistic/eslint-plugin": "^5.6.1",
+    "@typescript-eslint/eslint-plugin": "^8.51.0",
+    "@typescript-eslint/parser": "^8.51.0",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-array-func": "^5.1.0",
     "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-n": "^17.16.2",
+    "eslint-plugin-n": "^17.23.1",
     "eslint-plugin-promise": "^7.1.0",
     "eslint-plugin-regexp": "^2.6.0",
-    "globals": "^16.0.0"
+    "eslint-plugin-unicorn": "^62.0.0",
+    "globals": "^16.5.0"
   },
   "devDependencies": {
-    "eslint": "^9.23.0",
-    "svelte-eslint-parser": "^1.1.0",
-    "typescript": "^5.8.2",
-    "typescript-eslint": "^8.28.0"
+    "eslint": "^9.39.2",
+    "svelte-eslint-parser": "^1.4.1",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.51.0"
   },
   "engines": {
     "node": ">=20"
@@ -45,12 +47,16 @@
   "license": "MIT",
   "main": "nodejs.js",
   "peerDependencies": {
-    "eslint": "^9.13.0",
-    "svelte-eslint-parser": ">=0.43.0",
+    "eslint": "^9.39.2",
+    "eslint-plugin-svelte": "^3.13.1",
+    "svelte-eslint-parser": "^1.4.1",
     "typescript": "^5.6.3",
-    "typescript-eslint": "^8.12.2"
+    "typescript-eslint": "^8.51.0"
   },
   "peerDependenciesMeta": {
+    "eslint-plugin-svelte": {
+      "optional": true
+    },
     "typescript-eslint": {
       "optional": true
     },

package/plugin-rules/imports.js

@@ -91,11 +91,6 @@ export const imports = {
   // 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',

package/plugin-rules/stylistic.js

@@ -132,5 +132,5 @@ export const rules = {
   '@stylistic/switch-colon-spacing': ['error', { after: true, before: false }],
   '@stylistic/wrap-regex': 'off',
   '@stylistic/array-bracket-newline': 'error',
-  '@stylistic/no-extra-parens': ['error', 'all', { nestedBinaryExpressions: false, enforceForArrowConditionals: false }],
+  '@stylistic/no-extra-parens': ['error', 'all', { nestedBinaryExpressions: false, ignoredNodes: ['ArrowFunctionExpression[body.type=ConditionalExpression]'] }],
 };

package/plugin-rules/unicorn.js

@@ -0,0 +1,16 @@
+export const rules = {
+  'unicorn/better-regex': 'error',
+  'unicorn/custom-error-definition': 'error',
+  'unicorn/expiring-todo-comments': 'off',
+  'unicorn/import-style': 'off',
+  'unicorn/no-array-callback-reference': 'off',
+  'unicorn/no-array-reduce': 'off',
+  'unicorn/no-await-expression-member': 'off',
+  'no-nested-ternary': 'off',
+  'unicorn/no-nested-ternary': 'error',
+  'unicorn/no-null': 'off',
+  'unicorn/no-typeof-undefined': 'off',
+  'unicorn/prefer-import-meta-properties': 'error',
+  'unicorn/prefer-ternary': 'off',
+  'unicorn/prevent-abbreviations': 'off',
+};

package/typescript-base.js

@@ -1,11 +1,13 @@
-// eslint-disable-next-line import/no-unresolved
 import tseslint from 'typescript-eslint';
+import importPlugin from 'eslint-plugin-import';
+import { defineConfig } from 'eslint/config';
 import base from './base.js';
 
-const config = [
+const config = defineConfig([
   ...base,
   ...tseslint.configs.strictTypeChecked,
   ...tseslint.configs.stylisticTypeChecked,
+  importPlugin.flatConfigs.typescript,
   {
     languageOptions: {
       parserOptions: {
@@ -61,7 +63,6 @@ const config = [
       '@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'],
@@ -80,6 +81,6 @@ const config = [
       '@typescript-eslint/consistent-type-definitions': ['error', 'type'],
     },
   },
-];
+]);
 
 export default config;

package/typescript-browser.js

@@ -1,7 +1,8 @@
+import { defineConfig } from 'eslint/config';
 import browser from './browser.js';
 import typescriptBase from './typescript-base.js';
 
-export default [
+export default defineConfig([
   ...browser,
   ...typescriptBase,
-];
+]);

package/typescript-node.js

@@ -1,7 +1,8 @@
+import { defineConfig } from 'eslint/config';
 import nodejs from './nodejs.js';
 import typescriptBase from './typescript-base.js';
 
-export default [
+export default defineConfig([
   ...nodejs,
   ...typescriptBase,
   {
@@ -9,4 +10,4 @@ export default [
       'n/no-missing-import': 'off',
     },
   },
-];
+]);

package/typescript-svelte.js

@@ -1,26 +1,22 @@
 import svelteParser from 'svelte-eslint-parser';
-// eslint-disable-next-line import/no-unresolved
+
 import tseslint from 'typescript-eslint';
+import svelte from 'eslint-plugin-svelte';
+import { defineConfig } from 'eslint/config';
 import typescriptBrowser from './typescript-browser.js';
 
-export default [
+export default defineConfig([
   ...typescriptBrowser,
+  ...svelte.configs.recommended,
   {
     languageOptions: {
       parser: svelteParser,
       parserOptions: {
         parser: tseslint.parser,
-        project: './tsconfig.eslint.json',
+        projectService: true,
         extraFileExtensions: ['.svelte'],
       },
     },
-    settings: {
-      'import/resolver': {
-        typescript: {
-          project: './tsconfig.eslint.json',
-        },
-      },
-    },
     rules: {
       'import/no-mutable-exports': 'off',
       'import/no-named-as-default': 'off',
@@ -97,4 +93,4 @@ export default [
       'svelte/system': 'error',
     },
   },
-];
+]);