@mkaradeniz/eslint-config 4.0.16 → 4.0.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mkaradeniz/eslint-config",
-  "version": "4.0.16",
+  "version": "4.0.18",
   "private": false,
   "files": [
     "base.mjs",
@@ -38,18 +38,22 @@
     "eslint-plugin-react-compiler": "19.1.0-rc.2",
     "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-tsdoc": "^0.5.0",
-    "eslint-plugin-turbo": "2.7.2",
+    "eslint-plugin-turbo": "2.7.3",
     "eslint-plugin-unicorn": "^62.0.0",
     "eslint-plugin-unused-imports": "^4.3.0",
     "globals": "^17.0.0",
-    "typescript-eslint": "^8.51.0"
+    "typescript-eslint": "^8.52.0"
   },
   "devDependencies": {
-    "@mkaradeniz/prettier-config": "^4.0.3",
+    "@mkaradeniz/prettier-config": "^4.0.6",
+    "eslint": "^9.39.2",
     "husky": "^9.1.7",
     "lint-staged": "^16.2.7",
     "prettier": "^3.7.4",
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3"
+  },
+  "peerDependencies": {
+    "eslint": "^9.39.2"
   }
 }

package/rules/baseRules.mjs

@@ -1,10 +1,10 @@
 export const baseRules = {
   rules: {
-    'array-callback-return': ['warn', { allowImplicit: true, checkForEach: false, allowVoid: false }],
+    'array-callback-return': ['warn', { allowImplicit: true, allowVoid: false, checkForEach: false }],
     'block-scoped-var': ['warn'],
     camelcase: [
       'warn',
-      { allow: ['^UNSAFE_'], ignoreDestructuring: false, properties: 'never', ignoreImports: false, ignoreGlobals: false },
+      { allow: ['^UNSAFE_'], ignoreDestructuring: false, ignoreGlobals: false, ignoreImports: false, properties: 'never' },
     ],
     curly: ['warn', 'all'],
     'default-case-last': ['warn'],
@@ -44,7 +44,7 @@ export const baseRules = {
     'no-extra-label': ['warn'],
     'no-fallthrough': ['off'],
     'no-global-assign': ['warn'],
-    'no-implicit-coercion': ['warn', { boolean: true, number: true, string: true, disallowTemplateShorthand: false }],
+    'no-implicit-coercion': ['warn', { boolean: true, disallowTemplateShorthand: false, number: true, string: true }],
     'no-inner-declarations': ['warn'],
     'no-invalid-regexp': ['warn'],
     'no-irregular-whitespace': ['warn'],
@@ -69,14 +69,17 @@ export const baseRules = {
     'no-regex-spaces': ['warn'],
     'no-restricted-syntax': [
       'warn',
-      { selector: 'SwitchCase > *.consequent[type!="BlockStatement"]', message: 'Switch cases without blocks are disallowed.' },
       {
-        selector: 'CallExpression[callee.property.name="sort"]',
+        message: 'Switch cases without blocks are disallowed.',
+        selector: 'SwitchCase > *.consequent[type!="BlockStatement"]',
+      },
+      {
         message: 'Use .toSorted() instead of .sort() to avoid mutating the original array.',
+        selector: 'CallExpression[callee.property.name="sort"]',
       },
       {
-        selector: 'CallExpression[callee.property.name="reverse"]',
         message: 'Use .toReversed() instead of .reverse() to avoid mutating the original array.',
+        selector: 'CallExpression[callee.property.name="reverse"]',
       },
     ],
     'no-return-assign': ['warn'],
@@ -120,6 +123,7 @@ export const baseRules = {
     'prefer-spread': ['warn'],
     'prefer-template': ['warn'],
     'require-yield': ['warn'],
+    'sort-keys': ['warn', 'asc', { allowLineSeparatedGroups: true, caseSensitive: true, minKeys: 2, natural: true }],
     'symbol-description': ['warn'],
     'use-isnan': ['warn'],
     'valid-typeof': ['warn'],
@@ -161,6 +165,5 @@ export const baseRules = {
     ],
     'unicorn/number-literal-case': ['warn'],
     'unicorn/prefer-node-protocol': ['warn'],
-    'sort-keys': ['warn', 'asc', { allowLineSeparatedGroups: true, caseSensitive: true, minKeys: 2, natural: true }],
   },
 };

package/rules/reactRules.mjs

@@ -5,7 +5,6 @@ export const reactRules = {
       {
         paths: [
           {
-            name: 'react',
             importNames: [
               'createContext',
               'fordwardRef',
@@ -29,6 +28,7 @@ export const reactRules = {
             ],
             message:
               "React built-in hooks & functions should not be important directly. `Import React from 'react'` and then use `React.useBuiltInHook()` || `React.function()`.",
+            name: 'react',
           },
         ],
       },
@@ -41,10 +41,11 @@ export const reactRules = {
     'react/jsx-curly-brace-presence': ['warn', { children: 'ignore', props: 'never' }],
     'react/jsx-fragments': ['warn'],
     'react/jsx-key': ['warn'],
+    'react/jsx-newline': ['warn', { prevent: false }],
     'react/jsx-no-comment-textnodes': ['warn'],
     'react/jsx-no-duplicate-props': ['warn'],
     'react/jsx-no-leaked-render': ['warn'],
-    'react/jsx-no-target-blank': ['warn', { allowReferrer: true, links: true, forms: false }],
+    'react/jsx-no-target-blank': ['warn', { allowReferrer: true, forms: false, links: true }],
     'react/jsx-no-undef': ['warn'],
     'react/jsx-no-useless-fragment': ['warn', { allowExpressions: true }],
     'react/jsx-pascal-case': ['warn'],
@@ -64,14 +65,17 @@ export const reactRules = {
     'react/no-unstable-nested-components': ['warn'],
     'react/require-render-return': ['warn'],
     'react/self-closing-comp': ['warn'],
+
     // react-hooks
-    'react-hooks/rules-of-hooks': ['warn'],
     'react-hooks/exhaustive-deps': ['warn'],
+    'react-hooks/rules-of-hooks': ['warn'],
+
     // @stylistic/jsx
     /**
      * https://eslint.style/rules/default/jsx-newline
      */
-    '@stylistic/jsx-pascal-case': ['warn', { allowAllCaps: false, allowNamespace: true, allowLeadingUnderscore: false }],
+    '@stylistic/jsx-pascal-case': ['warn', { allowAllCaps: false, allowLeadingUnderscore: false, allowNamespace: true }],
+
     // jsx-a11y
     'jsx-a11y/alt-text': ['warn'],
     'jsx-a11y/anchor-has-content': ['warn'],
@@ -105,14 +109,14 @@ export const reactRules = {
     'jsx-a11y/no-access-key': ['warn'],
     'jsx-a11y/no-autofocus': ['warn'],
     'jsx-a11y/no-distracting-elements': ['warn'],
-    'jsx-a11y/no-interactive-element-to-noninteractive-role': ['warn', { tr: ['none', 'presentation'], canvas: ['img'] }],
+    'jsx-a11y/no-interactive-element-to-noninteractive-role': ['warn', { canvas: ['img'], tr: ['none', 'presentation'] }],
     'jsx-a11y/no-noninteractive-element-interactions': [
       'warn',
       {
-        handlers: ['onClick', 'onError', 'onLoad', 'onMouseDown', 'onMouseUp', 'onKeyPress', 'onKeyDown', 'onKeyUp'],
         alert: ['onKeyUp', 'onKeyDown', 'onKeyPress'],
         body: ['onError', 'onLoad'],
         dialog: ['onKeyUp', 'onKeyDown', 'onKeyPress'],
+        handlers: ['onClick', 'onError', 'onLoad', 'onMouseDown', 'onMouseUp', 'onKeyPress', 'onKeyDown', 'onKeyUp'],
         iframe: ['onError', 'onLoad'],
         img: ['onError', 'onLoad'],
       },
@@ -120,15 +124,15 @@ export const reactRules = {
     'jsx-a11y/no-noninteractive-element-to-interactive-role': [
       'warn',
       {
-        ul: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
-        ol: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
+        fieldset: ['radiogroup', 'presentation'],
         li: ['menuitem', 'option', 'row', 'tab', 'treeitem'],
+        ol: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
         table: ['grid'],
         td: ['gridcell'],
-        fieldset: ['radiogroup', 'presentation'],
+        ul: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
       },
     ],
-    'jsx-a11y/no-noninteractive-tabindex': ['warn', { tags: [], roles: ['tabpanel'], allowExpressionValues: true }],
+    'jsx-a11y/no-noninteractive-tabindex': ['warn', { allowExpressionValues: true, roles: ['tabpanel'], tags: [] }],
     'jsx-a11y/no-redundant-roles': ['warn'],
     'jsx-a11y/no-static-element-interactions': [
       'warn',

package/rules/typescriptRules.mjs

@@ -23,6 +23,7 @@ export const typescriptRules = {
     'no-unused-expressions': ['off'],
     'no-unused-vars': ['off'],
     'no-useless-constructor': ['off'],
+
     // Typescript-specific rules.
     '@typescript-eslint/adjacent-overload-signatures': ['warn'],
     '@typescript-eslint/array-type': ['warn', { default: 'generic' }],
@@ -33,20 +34,15 @@ export const typescriptRules = {
     '@typescript-eslint/consistent-indexed-object-style': ['warn'],
     '@typescript-eslint/consistent-type-assertions': ['warn'],
     '@typescript-eslint/consistent-type-definitions': ['warn', 'type'],
-    '@typescript-eslint/consistent-type-imports': ['warn', { prefer: 'type-imports', fixStyle: 'inline-type-imports' }],
+    '@typescript-eslint/consistent-type-imports': ['warn', { fixStyle: 'inline-type-imports', prefer: 'type-imports' }],
     '@typescript-eslint/naming-convention': [
       'warn',
       {
-        selector: 'variable',
+        filter: { match: false, regex: '^(UNSAFE_|__)' },
         format: ['camelCase', 'UPPER_CASE', 'PascalCase'],
-        filter: {
-          regex: '^(UNSAFE_|__)',
-          match: false,
-        },
+        selector: 'variable',
       },
       {
-        selector: 'variable',
-        types: ['boolean'],
         format: ['PascalCase', 'camelCase'],
         prefix: [
           'are',
@@ -68,6 +64,8 @@ export const typescriptRules = {
           'were',
           'will',
         ],
+        selector: 'variable',
+        types: ['boolean'],
       },
     ],
     '@typescript-eslint/no-array-constructor': ['warn'],
@@ -114,7 +112,6 @@ export const typescriptRules = {
       'warn',
       {
         blankLine: 'always',
-        prev: '*',
         next: [
           'block',
           'break',
@@ -134,16 +131,13 @@ export const typescriptRules = {
           'while',
           'with',
         ],
+        prev: '*',
       },
-      {
-        blankLine: 'always',
-        prev: ['case', 'default', 'directive'],
-        next: '*',
-      },
-      { blankLine: 'always', prev: 'directive', next: '*' },
-      { blankLine: 'any', prev: 'directive', next: 'directive' },
-      { blankLine: 'always', prev: '*', next: ['enum', 'interface', 'type'] },
-      { blankLine: 'never', prev: 'function-overload', next: 'function' },
+      { blankLine: 'always', next: '*', prev: ['case', 'default', 'directive'] },
+      { blankLine: 'always', next: '*', prev: 'directive' },
+      { blankLine: 'any', next: 'directive', prev: 'directive' },
+      { blankLine: 'always', next: ['enum', 'interface', 'type'], prev: '*' },
+      { blankLine: 'never', next: 'function', prev: 'function-overload' },
     ],
   },
 };