@aiou/eslint-config 3.1.1 → 3.2.0

package/README.md

@@ -10,7 +10,7 @@ Flat ESLint config with TypeScript, React, SSR, and more.
 - TypeScript, YAML, JSONC, Markdown, TOML support
 - Built-in React rules via `@eslint-react`, react-refresh, and SSR-safe linting
 - Auto-detected framework support (Next.js, Tailwind CSS)
-- Import sorting, unused import removal, and multi-line import enforcement
+- Import sorting and unused import removal via `eslint-plugin-perfectionist`
 - Sorted keys for `package.json` and `tsconfig.json`
 - Visual linting progress reporter
 
@@ -96,7 +96,7 @@ The following plugins are enabled automatically when their packages are found in
 | Arrow parens | `as-needed` |
 | Curly | `all` |
 | File naming | `kebab-case` (via unicorn) |
-| Import sorting | side-effects → `node:` builtins → packages → relative/alias → virtual → types |
+| Import sorting | type-imports → builtins/externals → internal → relative → side-effects (via `perfectionist`) |
 
 ## Included Plugins
 
@@ -108,8 +108,7 @@ The following plugins are enabled automatically when their packages are found in
 | `eslint-plugin-react-refresh` | React Refresh compatibility |
 | `@stylistic/eslint-plugin` | Formatting rules |
 | `eslint-plugin-import-x` | Import rules |
-| `eslint-plugin-simple-import-sort` | Import sorting |
-| `eslint-plugin-import-newlines` | Multi-line import enforcement |
+| `eslint-plugin-perfectionist` | Import sorting, type sorting (union/intersection/named imports) |
 | `eslint-plugin-unused-imports` | Unused import/variable removal |
 | `eslint-plugin-unicorn` | Best practice rules |
 | `eslint-plugin-n` | Node.js rules |

package/dist/index.mjs

@@ -2,9 +2,7 @@ import { FlatConfigComposer } from 'eslint-flat-config-utils';
 import { isPackageExists } from 'local-pkg';
 import pluginComments from '@eslint-community/eslint-plugin-eslint-comments';
 import ignoreGlobs from '@aiou/eslint-ignore';
-import pluginImportNewlines from 'eslint-plugin-import-newlines';
 import pluginImport from 'eslint-plugin-import-x';
-import pluginSimpleImportSort from 'eslint-plugin-simple-import-sort';
 import pluginUnsedImports from 'eslint-plugin-unused-imports';
 import pluginN from 'eslint-plugin-n';
 import pluginPromise from 'eslint-plugin-promise';
@@ -12,11 +10,13 @@ import globals from 'globals';
 import pluginJsonc from 'eslint-plugin-jsonc';
 import pluginMarkdown from '@eslint/markdown';
 import pluginNext from '@next/eslint-plugin-next';
+import pluginPerfectionist from 'eslint-plugin-perfectionist';
 import pluginProgress from '@aiou/eslint-plugin-progress';
 import pluginReact from '@eslint-react/eslint-plugin';
 import pluginReactHooks from 'eslint-plugin-react-hooks';
 import pluginReactRefresh from 'eslint-plugin-react-refresh';
 import { configs } from 'eslint-plugin-regexp';
+import storybookPlugin from 'eslint-plugin-storybook';
 import pluginStylistic from '@stylistic/eslint-plugin';
 import { createRequire } from 'node:module';
 import pluginToml from 'eslint-plugin-toml';
@@ -76,8 +76,6 @@ const imports = () => {
     {
       plugins: {
         import: pluginImport,
-        "simple-import-sort": pluginSimpleImportSort,
-        "import-newlines": pluginImportNewlines,
         "unused-imports": pluginUnsedImports
       },
       rules: {
@@ -85,29 +83,6 @@ const imports = () => {
         // off: controlled by import/order
         "import/order": "off",
         "sort-imports": "off",
-        "simple-import-sort/imports": [
-          "warn",
-          {
-            groups: [
-              // Side effect imports.
-              [String.raw`^\u0000`],
-              // Node.js builtins prefixed with `node:`.
-              ["^node:"],
-              // Packages.
-              // Things that start with a letter (or digit or underscore), or `@` followed by a letter.
-              [String.raw`^@?\w`],
-              // Relative imports.
-              // Absolute imports and other imports such as `@/foo` or `~/foo`.
-              // Anything not matched in another group.
-              ["^", String.raw`^\.`, String.raw`^@/\w`, String.raw`^~/\w`],
-              // Virtual modules prefixed with `virtual:` or `virtual-`, rollup & vite favor
-              ["^virtual:", "^virtual-"],
-              // Types
-              ["^[^/\\.].*\0$", "^\\..*\0$"]
-            ]
-          }
-        ],
-        "simple-import-sort/exports": "off",
         "import/first": "error",
         "import/newline-after-import": "error",
         "import/no-duplicates": "error",
@@ -133,15 +108,6 @@ const imports = () => {
             args: "after-used",
             argsIgnorePattern: "^_"
           }
-        ],
-        // Enforce newlines inside named import
-        "import-newlines/enforce": [
-          "error",
-          {
-            items: 2,
-            "max-len": 120,
-            semi: false
-          }
         ]
       }
     },
@@ -193,7 +159,8 @@ const imports = () => {
         // ignore require third packages in .eslintrc.* e.g. eslint-define-config
         `**/.eslintrc.${GLOB_SCRIPT_EXT}`,
         `**/**/eslint.config.${GLOB_SCRIPT_EXT}`,
-        "**/{vite,esbuild,rollup,webpack,rspack}.ts"
+        "**/{vite,esbuild,rollup,webpack,rspack}.ts",
+        "**/vitest.setup.*"
       ],
       rules: {
         "import/no-extraneous-dependencies": "off"
@@ -753,7 +720,8 @@ const markdown = () => {
         "import/no-anonymous-default-export": "off",
         "react-refresh/only-export-components": "off",
         "react/jsx-no-undef": "off",
-        "unicorn/filename-case": "off"
+        "unicorn/filename-case": "off",
+        "perfectionist/sort-imports": "off"
       }
     }
   ];
@@ -775,6 +743,46 @@ const next = () => {
   return config;
 };
 
+const perfectionist = () => {
+  return [
+    {
+      plugins: {
+        perfectionist: pluginPerfectionist
+      },
+      rules: {
+        "perfectionist/sort-imports": [
+          "warn",
+          {
+            type: "natural",
+            order: "asc",
+            newlinesBetween: "ignore",
+            groups: [
+              "type-import",
+              ["value-builtin", "value-external"],
+              "type-internal",
+              "value-internal",
+              ["type-parent", "type-sibling", "type-index"],
+              ["value-parent", "value-sibling", "value-index"],
+              "side-effect",
+              "unknown"
+            ],
+            customGroups: [
+              {
+                groupName: "virtual-modules",
+                elementNamePattern: ["^virtual:", "^virtual-"]
+              }
+            ],
+            internalPattern: ["^~/.+", "^@/.+", "^#.+"]
+          }
+        ],
+        "perfectionist/sort-union-types": ["error", { type: "natural", order: "asc" }],
+        "perfectionist/sort-intersection-types": ["error", { type: "natural", order: "asc" }],
+        "perfectionist/sort-named-imports": ["warn", { type: "natural", order: "asc" }]
+      }
+    }
+  ];
+};
+
 const progress = () => {
   const config = [
     {
@@ -823,33 +831,28 @@ const react = () => {
         react: pluginReact,
         "react-dom": pluginReact,
         "react-hooks": pluginReactHooks,
-        "react-hooks-extra": pluginReact,
         "react-naming-convention": pluginReact,
         "react-refresh": pluginReactRefresh,
         "react-web-api": pluginReact
       },
       rules: {
         ...renameRules(pluginReact.configs["recommended-typescript"].rules, { "@eslint-react": "react" }),
-        ...renameRules(pluginReact.configs.dom.rules, { "@eslint-react": "react-dom" }),
-        ...renameRules(pluginReact.configs.x.rules, { "@eslint-react": "react-hooks-extra" }),
-        ...renameRules(pluginReact.configs.x.rules, { "@eslint-react": "react-naming-convention" }),
-        ...renameRules(pluginReact.configs["web-api"].rules, { "@eslint-react": "react-web-api" }),
         ...pluginReactHooks.configs.flat.recommended.rules,
         "react/no-nested-component-definitions": "warn",
-        "react-dom/no-unknown-property": "off",
+        "react-dom/dom-no-unknown-property": "off",
         "react-refresh/only-export-components": "warn"
       }
     },
     {
       files: ["src/components/**/*.{ts,tsx}"],
       rules: {
-        "react-naming-convention/filename": ["warn", { rule: "PascalCase" }]
+        "unicorn/filename-case": ["warn", { case: "pascalCase" }]
       }
     },
     {
       files: ["src/hooks/**/use*.{ts,tsx}"],
       rules: {
-        "react-naming-convention/filename": ["warn", { rule: "kebab-case" }]
+        "unicorn/filename-case": ["warn", { case: "kebabCase" }]
       }
     },
     {
@@ -974,6 +977,13 @@ const regexp = () => {
   return config;
 };
 
+const storybook = () => {
+  const config = [
+    ...storybookPlugin.configs["flat/recommended"]
+  ];
+  return config;
+};
+
 const stylistic = () => {
   const config = pluginStylistic.configs.customize({
     indent: 2,
@@ -1180,10 +1190,7 @@ const typescript = () => {
         ],
         // Limit `interface` define object types, users could override with *.d.ts declare
         "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
-        "@typescript-eslint/prefer-ts-expect-error": "warn",
         "@typescript-eslint/default-param-last": "error",
-        // Sort type S = A | B
-        "@typescript-eslint/sort-type-constituents": "error",
         "@typescript-eslint/unbound-method": "off",
         "@typescript-eslint/prefer-for-of": "error",
         // When .ts files compiled to .mjs, will throw require is not found
@@ -1227,8 +1234,7 @@ const typescript = () => {
           "error",
           { functions: false, classes: false, variables: true }
         ],
-        "no-loss-of-precision": "off",
-        "@typescript-eslint/no-loss-of-precision": "error",
+        "no-loss-of-precision": "error",
         "no-unused-expressions": "off",
         "@typescript-eslint/no-unused-expressions": ["error", {
           allowShortCircuit: true,
@@ -1236,19 +1242,13 @@ const typescript = () => {
           allowTaggedTemplates: true
         }],
         // off
-        "@typescript-eslint/camelcase": "off",
         "@typescript-eslint/explicit-function-return-type": "off",
         "@typescript-eslint/explicit-member-accessibility": "off",
         "@typescript-eslint/no-explicit-any": "off",
-        "@typescript-eslint/no-parameter-properties": "off",
-        "@typescript-eslint/no-empty-interface": "off",
-        "@typescript-eslint/ban-ts-ignore": "off",
         "@typescript-eslint/no-empty-function": "off",
         "@typescript-eslint/no-non-null-assertion": "off",
         "@typescript-eslint/explicit-module-boundary-types": "off",
-        "@typescript-eslint/ban-types": "off",
         "@typescript-eslint/no-namespace": "off",
-        "@typescript-eslint/no-var-requires": "off",
         // https://www.npmjs.com/package/eslint-plugin-unused-imports
         "@typescript-eslint/no-unused-vars": "off",
         "no-void": ["error", { allowAsStatement: true }],
@@ -1292,6 +1292,8 @@ const unicorn = () => {
         "unicorn/prefer-string-starts-ends-with": "error",
         "unicorn/prefer-type-error": "error",
         "unicorn/throw-new-error": "error",
+        "unicorn/prevent-abbreviations": "off",
+        "unicorn/no-null": "off",
         "unicorn/filename-case": ["error", {
           case: "kebabCase",
           ignore: [/^[A-Z]+\.md$/, /^[A-Z]+\.yml$/, /^\.?[A-Z]/, /^Dockerfile$/]
@@ -1328,7 +1330,8 @@ const presetJavascript = [
   ...javascript(),
   ...comments(),
   ...imports(),
-  ...unicorn()
+  ...unicorn(),
+  ...perfectionist()
 ];
 const presetTypescript = [
   ...presetJavascript,
@@ -1345,18 +1348,20 @@ const presetDefault = [
   ...react(),
   ...stylistic(),
   ...presetLangsExtensions,
-  ...progress()
+  ...progress(),
+  ...storybook()
 ];
 const all = [
   ...presetDefault,
   ...tailwindcss(),
   ...next(),
   ...ssrReact(),
-  ...regexp()
+  ...regexp(),
+  ...storybook()
 ];
-const aiou = ({ ssr = true, regexp: regexp$1 = true } = { ssr: true, regexp: true }, ...userConfigs) => {
+const aiou = ({ ssr = true, regexp: regexp$1 = true, tailwindcss: enableTailwindcss = true } = {}, ...userConfigs) => {
   const configs = [...presetDefault];
-  if (isPackageExists("tailwindcss")) {
+  if (enableTailwindcss && isPackageExists("tailwindcss")) {
     configs.push(...tailwindcss());
   }
   if (isPackageExists("next")) {

package/dts/configs/perfectionist.d.ts

@@ -0,0 +1,2 @@
+import type { Config } from '../type';
+export declare const perfectionist: () => Config[];

package/dts/configs/storybook.d.ts

@@ -0,0 +1,2 @@
+import type { Config } from '../type';
+export declare const storybook: () => Config[];

package/dts/globs.d.ts

@@ -17,3 +17,5 @@ export declare const GLOB_MARKDOWN = "**/*.md";
 export declare const GLOB_VUE = "**/*.vue";
 export declare const GLOB_YAML = "**/*.y?(a)ml";
 export declare const GLOB_WORKFLOW_YAML = "**/.github/**/*.y?(a)ml";
+export declare const GLOB_STORYBOOK_STORIES = "**/*.stories.@(ts|tsx|js|jsx|mjs|cjs)";
+export declare const GLOB_STORYBOOK_MAIN = ".storybook/main.@(js|cjs|mjs|ts)";

package/dts/index.d.ts

@@ -1,12 +1,13 @@
-import { FlatConfigComposer } from 'eslint-flat-config-utils';
 import type { Linter } from 'eslint';
 import type { Arrayable, Awaitable } from 'eslint-flat-config-utils';
+import { FlatConfigComposer } from 'eslint-flat-config-utils';
 import type { Config } from './type';
 import type { ConfigNames } from './typegen';
 export declare const all: Config[];
 interface Options {
     ssr?: boolean;
     regexp?: boolean;
+    tailwindcss?: boolean;
 }
-export declare const aiou: ({ ssr, regexp }?: Options, ...userConfigs: Awaitable<Arrayable<Config> | FlatConfigComposer<any, any> | Linter.Config[]>[]) => FlatConfigComposer<Config, ConfigNames>;
+export declare const aiou: ({ ssr, regexp, tailwindcss: enableTailwindcss }?: Options, ...userConfigs: Awaitable<Arrayable<Config> | FlatConfigComposer<any, any> | Linter.Config[]>[]) => FlatConfigComposer<Config, ConfigNames>;
 export {};