@egs33/eslint-config 4.1.3 → 4.2.1

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,7 @@ 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';
@@ -13,6 +14,7 @@ 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 defineConfig([
   {
@@ -51,6 +53,7 @@ export default defineConfig([
   importPlugin.flatConfigs.recommended,
   array.configs.all,
   regexpConfig['flat/recommended'],
+  eslintPluginUnicorn.configs.recommended,
   {
     rules: {
       ...bestPractices,
@@ -69,6 +72,7 @@ export default defineConfig([
       'array-func/prefer-array-from': 'off',
       ...PromiseRules,
       ...StyleRules,
+      ...UnicornRules,
     },
   },
 ]);

package/package.json

@@ -1,27 +1,28 @@
 {
   "name": "@egs33/eslint-config",
   "description": "eslint config for me",
-  "version": "4.1.3",
+  "version": "4.2.1",
   "author": "egs33",
   "type": "module",
   "dependencies": {
-    "@eslint/js": "^9.39.2",
+    "@eslint/js": "^10.0.1",
     "@stylistic/eslint-plugin": "^5.6.1",
-    "@typescript-eslint/eslint-plugin": "^8.51.0",
-    "@typescript-eslint/parser": "^8.51.0",
+    "@typescript-eslint/eslint-plugin": "^8.57.1",
+    "@typescript-eslint/parser": "^8.57.1",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-array-func": "^5.1.0",
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-n": "^17.23.1",
     "eslint-plugin-promise": "^7.1.0",
-    "eslint-plugin-regexp": "^2.6.0",
-    "globals": "^16.5.0"
+    "eslint-plugin-regexp": "^3.1.0",
+    "eslint-plugin-unicorn": "^63.0.0",
+    "globals": "^17.4.0"
   },
   "devDependencies": {
-    "eslint": "^9.39.2",
+    "eslint": "^10.1.0",
     "svelte-eslint-parser": "^1.4.1",
     "typescript": "^5.9.3",
-    "typescript-eslint": "^8.51.0"
+    "typescript-eslint": "^8.57.1"
   },
   "engines": {
     "node": ">=20"
@@ -46,7 +47,7 @@
   "license": "MIT",
   "main": "nodejs.js",
   "peerDependencies": {
-    "eslint": "^9.39.2",
+    "eslint": "^9.39.2 || ^10.0.0",
     "eslint-plugin-svelte": "^3.13.1",
     "svelte-eslint-parser": "^1.4.1",
     "typescript": "^5.6.3",

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/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

@@ -63,7 +63,6 @@ const config = defineConfig([
       '@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'],