feat!: use unified stylistic rules

Author: coderwyd

package.json

@@ -41,9 +41,7 @@
     "eslint": ">=8.0.0"
   },
   "dependencies": {
-    "@eslint-stylistic/metadata": "0.0.4",
-    "@stylistic/eslint-plugin-js": "0.0.4",
-    "@stylistic/eslint-plugin-ts": "0.0.4",
+    "@stylistic/eslint-plugin": "^0.0.5",
     "@typescript-eslint/eslint-plugin": "^6.7.2",
     "@typescript-eslint/parser": "^6.7.2",
     "astro-eslint-parser": "^0.15.0",
@@ -71,7 +69,6 @@
     "yaml-eslint-parser": "^1.2.2"
   },
   "devDependencies": {
-    "@stylistic/eslint-plugin-migrate": "0.0.4",
     "@types/node": "^20.6.3",
     "@types/react": "^18.2.22",
     "bumpp": "^9.2.0",

pnpm-lock.yaml

@@ -1,28 +1,22 @@
 import type { FlatESLintConfigItem } from 'eslint-define-config'
-import { packages } from '@eslint-stylistic/metadata'
-import { pluginStylisticJs, pluginStylisticTs, pluginTs } from '../plugins'
-import { OFF } from '../flags'
-
-const tsPackage = packages.find(i => i.shortId === 'ts')!
+import { pluginStylistic } from '../plugins'
 
 export const javascriptStylistic: FlatESLintConfigItem[] = [
   {
     plugins: {
-      style: pluginStylisticJs,
+      style: pluginStylistic,
     },
     rules: {
       'antfu/consistent-list-newline': 'error',
       'antfu/if-newline': 'error',
 
-      'comma-dangle': ['error', 'always-multiline'],
       'curly': ['error', 'multi-or-nest', 'consistent'],
-      'quotes': ['error', 'single'],
-      'semi': ['error', 'never'],
 
       'style/array-bracket-spacing': ['error', 'never'],
       'style/arrow-spacing': ['error', { after: true, before: true }],
       'style/block-spacing': ['error', 'always'],
       'style/brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
+      'style/comma-dangle': ['error', 'always-multiline'],
       'style/comma-spacing': ['error', { after: true, before: false }],
       'style/comma-style': ['error', 'last'],
       'style/computed-property-spacing': ['error', 'never', { enforceForClassMembers: true }],
@@ -68,6 +62,7 @@ export const javascriptStylistic: FlatESLintConfigItem[] = [
       'style/key-spacing': ['error', { afterColon: true, beforeColon: false }],
       'style/keyword-spacing': ['error', { after: true, before: true }],
       'style/lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: true }],
+      'style/member-delimiter-style': ['error', { multiline: { delimiter: 'none' } }],
       'style/multiline-ternary': ['error', 'always-multiline'],
       'style/no-mixed-spaces-and-tabs': 'error',
       'style/no-multi-spaces': 'error',
@@ -79,7 +74,9 @@ export const javascriptStylistic: FlatESLintConfigItem[] = [
       'style/object-property-newline': ['error', { allowMultiplePropertiesPerLine: true }],
       'style/operator-linebreak': ['error', 'before'],
       'style/padded-blocks': ['error', { blocks: 'never', classes: 'never', switches: 'never' }],
+      'style/quotes': ['error', 'single'],
       'style/rest-spread-spacing': ['error', 'never'],
+      'style/semi': ['error', 'never'],
       'style/semi-spacing': ['error', { after: true, before: false }],
       'style/space-before-blocks': ['error', 'always'],
       'style/space-before-function-paren': ['error', { anonymous: 'always', asyncArrow: 'always', named: 'never' }],
@@ -99,62 +96,8 @@ export const javascriptStylistic: FlatESLintConfigItem[] = [
       }],
       'style/template-curly-spacing': 'error',
       'style/template-tag-spacing': ['error', 'never'],
+      'style/type-annotation-spacing': ['error', {}],
       'style/yield-star-spacing': ['error', 'both'],
     },
   },
 ]
-
-export const typescriptStylistic: FlatESLintConfigItem[] = [
-  {
-    plugins: {
-      'style-ts': pluginStylisticTs,
-      'ts': pluginTs as any,
-    },
-    rules: {
-      ...stylisticJsToTS(javascriptStylistic[0].rules!),
-
-      'comma-dangle': OFF,
-      'quotes': OFF,
-      'semi': OFF,
-
-      'style-ts/member-delimiter-style': ['error', { multiline: { delimiter: 'none' } }],
-      'style-ts/type-annotation-spacing': ['error', {}],
-
-      'ts/comma-dangle': ['error', 'always-multiline'],
-      'ts/quotes': ['error', 'single'],
-      'ts/semi': ['error', 'never'],
-    },
-  },
-]
-
-// TODO: move to ESLint Stylistic
-function stylisticJsToTS(input: Record<string, any>) {
-  return {
-    // turn off all stylistic rules from style
-    ...Object.fromEntries(
-      Object.entries(input)
-        .map(([key]) => {
-          if (!key.startsWith('style/'))
-            return null!
-          const basename = key.replace('style/', '')
-          if (tsPackage.rules.find(i => i.name === basename))
-            return [key, OFF]
-          return null!
-        })
-        .filter(Boolean),
-    ),
-    // rename all stylistic rules from style to style/ts
-    ...Object.fromEntries(
-      Object.entries(input)
-        .map(([key, value]) => {
-          if (!key.startsWith('style/'))
-            return null!
-          const basename = key.replace('style/', '')
-          return tsPackage.rules.find(i => i.name === basename)
-            ? [`style-ts/${basename}`, value]
-            : null!
-        })
-        .filter(Boolean),
-    ),
-  }
-}

src/configs/stylistic.ts

@@ -8,8 +8,7 @@ export { default as pluginJsdoc } from 'eslint-plugin-jsdoc'
 export { default as pluginJsonc } from 'eslint-plugin-jsonc'
 export { default as pluginMarkdown } from 'eslint-plugin-markdown'
 export { default as pluginNode } from 'eslint-plugin-n'
-export { default as pluginStylisticJs } from '@stylistic/eslint-plugin-js'
-export { default as pluginStylisticTs } from '@stylistic/eslint-plugin-ts'
+export { default as pluginStylistic } from '@stylistic/eslint-plugin'
 export { default as pluginTs } from '@typescript-eslint/eslint-plugin'
 export { default as pluginUnicorn } from 'eslint-plugin-unicorn'
 export { default as pluginUnusedImports } from 'eslint-plugin-unused-imports'