@aiou/eslint-config 2.1.0 → 3.0.0

package/dist/index.cjs

@@ -12,7 +12,6 @@ var pluginN = require('eslint-plugin-n');
 var pluginPromise = require('eslint-plugin-promise');
 var globals = require('globals');
 var pluginJsonc = require('eslint-plugin-jsonc');
-var jsoncParser = require('jsonc-eslint-parser');
 var pluginMarkdown = require('@eslint/markdown');
 var pluginNext = require('@next/eslint-plugin-next');
 var pluginProgress = require('@aiou/eslint-plugin-progress');
@@ -22,12 +21,12 @@ var pluginReactRefresh = require('eslint-plugin-react-refresh');
 var eslintPluginRegexp = require('eslint-plugin-regexp');
 var pluginStylistic = require('@stylistic/eslint-plugin');
 var node_module = require('node:module');
+var pluginToml = require('eslint-plugin-toml');
 var pluginTypeScript = require('@typescript-eslint/eslint-plugin');
 var tsParser = require('@typescript-eslint/parser');
 var pluginETC = require('eslint-plugin-etc');
 var pluginUnicorn = require('eslint-plugin-unicorn');
-var pluginYaml = require('eslint-plugin-yml');
-var yamlParser = require('yaml-eslint-parser');
+var pluginYml = require('eslint-plugin-yml');
 
 function _interopDefault (e) { return e && e.__esModule ? e : { default: e }; }
 
@@ -41,7 +40,6 @@ var pluginN__default = /*#__PURE__*/_interopDefault(pluginN);
 var pluginPromise__default = /*#__PURE__*/_interopDefault(pluginPromise);
 var globals__default = /*#__PURE__*/_interopDefault(globals);
 var pluginJsonc__default = /*#__PURE__*/_interopDefault(pluginJsonc);
-var jsoncParser__default = /*#__PURE__*/_interopDefault(jsoncParser);
 var pluginMarkdown__default = /*#__PURE__*/_interopDefault(pluginMarkdown);
 var pluginNext__default = /*#__PURE__*/_interopDefault(pluginNext);
 var pluginProgress__default = /*#__PURE__*/_interopDefault(pluginProgress);
@@ -49,12 +47,12 @@ var pluginReact__default = /*#__PURE__*/_interopDefault(pluginReact);
 var pluginReactHooks__default = /*#__PURE__*/_interopDefault(pluginReactHooks);
 var pluginReactRefresh__default = /*#__PURE__*/_interopDefault(pluginReactRefresh);
 var pluginStylistic__default = /*#__PURE__*/_interopDefault(pluginStylistic);
+var pluginToml__default = /*#__PURE__*/_interopDefault(pluginToml);
 var pluginTypeScript__default = /*#__PURE__*/_interopDefault(pluginTypeScript);
 var tsParser__default = /*#__PURE__*/_interopDefault(tsParser);
 var pluginETC__default = /*#__PURE__*/_interopDefault(pluginETC);
 var pluginUnicorn__default = /*#__PURE__*/_interopDefault(pluginUnicorn);
-var pluginYaml__default = /*#__PURE__*/_interopDefault(pluginYaml);
-var yamlParser__default = /*#__PURE__*/_interopDefault(yamlParser);
+var pluginYml__default = /*#__PURE__*/_interopDefault(pluginYml);
 
 const comments = () => {
   const config = [
@@ -474,7 +472,7 @@ const javascript = () => {
         camelcase: "off",
         "no-constant-condition": "warn",
         "no-debugger": process.env.NODE_ENV === "production" ? "error" : "off",
-        "no-console": process.env.NODE_ENV === "production" ? "error" : "off",
+        "no-console": process.env.NODE_ENV === "production" ? ["error", { allow: ["debug", "error", "warn"] }] : "off",
         "no-cond-assign": ["error", "always"],
         "no-restricted-syntax": [
           "error",
@@ -535,23 +533,11 @@ const javascript = () => {
 
 const jsonc = () => {
   const config = [
-    {
-      files: [GLOB_JSON, GLOB_JSON5, GLOB_JSONC],
-      ignores: [GLOB_CLAUDE_JSON],
-      name: "jsonc/setup",
-      languageOptions: {
-        parser: jsoncParser__default.default
-      },
-      plugins: {
-        jsonc: pluginJsonc__default.default
-      }
-    },
+    ...pluginJsonc__default.default.configs["recommended-with-jsonc"],
     {
       files: [GLOB_JSON, GLOB_JSON5, GLOB_JSONC],
       ignores: [GLOB_CLAUDE_JSON],
       rules: {
-        ...pluginJsonc__default.default.configs["recommended-with-jsonc"].rules,
-        // refs: https://ota-meshi.github.io/eslint-plugin-jsonc/rules/indent.html
         "jsonc/array-bracket-newline": [
           "error",
           {
@@ -630,8 +616,6 @@ const jsonc = () => {
           },
           {
             pathPattern: "^exports.*$",
-            // According to webpack, default condition should be last item
-            // Typescript bugs: types should be first one
             order: ["types", "import", "require", "node", "browser", "default"]
           },
           {
@@ -645,7 +629,6 @@ const jsonc = () => {
         ]
       }
     },
-    // refs: https://github.com/sxzz/eslint-config/blob/main/src/configs/sort.ts
     {
       files: ["**/tsconfig.json", "**/tsconfig.*.json"],
       rules: {
@@ -772,14 +755,10 @@ const jsonc = () => {
 
 const markdown = () => {
   const config = [
-    ...pluginMarkdown__default.default.configs.processor.map((config2) => ({
-      ...config2
-    })),
+    ...pluginMarkdown__default.default.configs.processor,
     {
-      // Code blocks in markdown file
       files: [`${GLOB_MARKDOWN}/*.${GLOB_SCRIPT_EXT}`],
       rules: {
-        // https://eslint-react.xyz/docs/migration
         "react/no-missing-component-display-name": "off",
         "@typescript-eslint/no-redeclare": "off",
         "@typescript-eslint/no-unused-vars": "off",
@@ -797,7 +776,6 @@ const markdown = () => {
         "no-undef": "off",
         "no-unused-expressions": "off",
         "no-unused-vars": "off",
-        // Off imports
         "import/no-extraneous-dependencies": "off",
         "import/no-default-export": "off",
         "import/no-anonymous-default-export": "off",
@@ -817,7 +795,6 @@ const next = () => {
         "@next/next": pluginNext__default.default
       },
       rules: {
-        // Enable rules for next project
         ...pluginNext__default.default.configs.recommended.rules
       }
     }
@@ -853,7 +830,6 @@ function renameRules(rules, map) {
 }
 
 const react = () => {
-  const plugins = pluginReact__default.default.configs.all.plugins;
   const config = [
     {
       files: [GLOB_JSX, GLOB_TSX],
@@ -866,34 +842,29 @@ const react = () => {
         sourceType: "module"
       },
       settings: {
-        react: {
-          // 'detect' will throw warn on monorepo
+        "react-x": {
           version: "18.0"
         }
       },
       plugins: {
-        react: plugins["@eslint-react"],
-        "react-dom": plugins["@eslint-react/dom"],
+        react: pluginReact__default.default,
+        "react-dom": pluginReact__default.default,
         "react-hooks": pluginReactHooks__default.default,
-        "react-hooks-extra": plugins["@eslint-react/hooks-extra"],
-        "react-naming-convention": plugins["@eslint-react/naming-convention"],
+        "react-hooks-extra": pluginReact__default.default,
+        "react-naming-convention": pluginReact__default.default,
         "react-refresh": pluginReactRefresh__default.default,
-        "react-web-api": plugins["@eslint-react/web-api"]
+        "react-web-api": pluginReact__default.default
       },
       rules: {
-        ...renameRules(plugins["@eslint-react"].configs["recommended-typescript"].rules, { "react-x": "react" }),
-        ...renameRules(plugins["@eslint-react/dom"].configs.recommended.rules, { "@eslint-react": "react-dom" }),
-        ...renameRules(pluginReactHooks__default.default.configs.recommended.rules, { "@eslint-react": "react-hooks" }),
-        ...renameRules(plugins["@eslint-react/hooks-extra"].configs.recommended.rules, { "@eslint-react": "react-hooks-extra" }),
-        ...renameRules(plugins["@eslint-react/naming-convention"].configs.recommended.rules, { "@eslint-react": "react-naming-convention" }),
-        ...renameRules(plugins["@eslint-react/web-api"].configs.recommended.rules, { "@eslint-react": "react-web-api" }),
-        "react/no-prop-types": "error",
+        ...renameRules(pluginReact__default.default.configs["recommended-typescript"].rules, { "@eslint-react": "react" }),
+        ...renameRules(pluginReact__default.default.configs["dom"].rules, { "@eslint-react": "react-dom" }),
+        ...renameRules(pluginReact__default.default.configs["x"].rules, { "@eslint-react": "react-hooks-extra" }),
+        ...renameRules(pluginReact__default.default.configs["x"].rules, { "@eslint-react": "react-naming-convention" }),
+        ...renameRules(pluginReact__default.default.configs["web-api"].rules, { "@eslint-react": "react-web-api" }),
+        ...pluginReactHooks__default.default.configs.flat.recommended.rules,
         "react/no-nested-component-definitions": "warn",
         "react-dom/no-unknown-property": "off",
-        "react/avoid-shorthand-boolean": ["error"],
-        // https://github.com/ArnaudBarre/eslint-plugin-react-refresh
-        "react-refresh/only-export-components": "warn",
-        "react-hooks-extra/no-direct-set-state-in-use-effect": "off"
+        "react-refresh/only-export-components": "warn"
       }
     },
     {
@@ -914,9 +885,6 @@ const react = () => {
         `**/*{-entry,entry.}*.${GLOB_SCRIPT_EXT}`,
         GLOB_TEST_SCRIPT,
         GLOB_TEST_DIRS,
-        // With next.js pages will export getStaticProps funcs
-        // There are some bad cases for not all files are route files
-        // RECOMMEND: pages/routes should only contain page files
         GLOB_PAGES_DIRS
       ],
       ignores: [
@@ -942,8 +910,7 @@ const ssrReact = () => {
         sourceType: "module"
       },
       settings: {
-        react: {
-          // 'detect' will throw warn on monorepo
+        "react-x": {
           version: "18.0"
         }
       },
@@ -965,7 +932,7 @@ const ssrReact = () => {
             selector: "Identifier[name='window']:not(UnaryExpression[operator='typeof'] > Identifier)",
             message: `Direct window access not allowed. Use 'typeof window !== "undefined"' check first`
           },
-          // Document 对象
+          // Document
           {
             selector: "MemberExpression[object.name='document']:not(LogicalExpression[right=MemberExpression] > MemberExpression[object.name='document'])",
             message: `Use 'typeof document !== "undefined" && document.xxx' for safe DOM access`
@@ -974,27 +941,26 @@ const ssrReact = () => {
             selector: "Identifier[name='document']:not(UnaryExpression[operator='typeof'] > Identifier)",
             message: `Direct document access not allowed. Use 'typeof document !== "undefined"' check first`
           },
-          // Navigator 对象
+          // Navigator
           {
             selector: "MemberExpression[object.name='navigator']:not(LogicalExpression[right=MemberExpression] > MemberExpression[object.name='navigator'])",
             message: `Use 'typeof navigator !== "undefined" && navigator.xxx' for safe DOM access`
           },
-          // Location 对象
+          // Location
           {
             selector: "MemberExpression[object.name='location']:not(LogicalExpression[right=MemberExpression] > MemberExpression[object.name='location'])",
             message: `Use 'typeof location !== "undefined" && location.xxx' for safe DOM access`
           },
-          // History 对象
+          // History
           {
             selector: "MemberExpression[object.name='history']:not(LogicalExpression[right=MemberExpression] > MemberExpression[object.name='history'])",
             message: `Use 'typeof history !== "undefined" && history.xxx' for safe DOM access`
           },
-          // Storage 对象们
+          // Storage
           {
             selector: "MemberExpression[object.name='localStorage']:not(LogicalExpression[right=MemberExpression] > MemberExpression[object.name='localStorage'])",
             message: `Use 'typeof localStorage !== "undefined" && localStorage.xxx' for safe DOM access`
           },
-          // Fork from enum
           {
             selector: "TSEnumDeclaration",
             message: "Don't declare enums"
@@ -1005,7 +971,6 @@ const ssrReact = () => {
     {
       files: [
         `**/*config*.${GLOB_SCRIPT_EXT}`,
-        // Client entry files not need to be SSR friendly
         `**/*{client-entry,entry.client}*.${GLOB_SCRIPT_EXT}`,
         GLOB_TEST_SCRIPT,
         GLOB_TEST_DIRS
@@ -1030,7 +995,7 @@ const ssrReact = () => {
 const regexp = () => {
   const config = [
     {
-      ...eslintPluginRegexp.configs["flat/recommended"]
+      ...eslintPluginRegexp.configs.recommended
     }
   ];
   return config;
@@ -1038,16 +1003,19 @@ const regexp = () => {
 
 const stylistic = () => {
   const config = pluginStylistic__default.default.configs.customize({
-    flat: true,
     indent: 2,
     jsx: true,
     quotes: "single",
     semi: false
   });
   const off = {};
-  Object.keys(config.rules ?? {}).forEach((key) => {
-    off[key.replace("@stylistic/", "")] = "off";
-  });
+  for (const key of Object.keys(config.rules ?? {})) {
+    const bare = key.replace("@stylistic/", "");
+    if (bare === "function-call-spacing") {
+      continue;
+    }
+    off[bare] = "off";
+  }
   const configs = [
     {
       files: [GLOB_JS, GLOB_JSX, GLOB_TS, GLOB_TSX],
@@ -1060,7 +1028,6 @@ const stylistic = () => {
         "@stylistic/quotes": ["error", "single", { avoidEscape: true }],
         "@stylistic/quote-props": ["error", "as-needed"],
         "@stylistic/array-bracket-spacing": ["error", "never"],
-        // https://eslint.org/docs/latest/rules/brace-style#1tbs
         "@stylistic/brace-style": ["error", "1tbs"],
         "@stylistic/block-spacing": ["error", "always"],
         "@stylistic/comma-spacing": ["error", { before: false, after: true }],
@@ -1114,8 +1081,7 @@ const stylistic = () => {
         "@stylistic/operator-linebreak": ["error", "before"],
         "@stylistic/member-delimiter-style": ["error", { multiline: { delimiter: "none" } }],
         "@stylistic/type-annotation-spacing": ["error", {}],
-        // no-spaced-func alert -- https://eslint.org/docs/latest/rules/no-spaced-func
-        "@stylistic/func-call-spacing": ["error", "never"],
+        "@stylistic/function-call-spacing": ["error", "never"],
         "@stylistic/semi": ["error", "never"],
         "@stylistic/space-before-blocks": ["error", "always"],
         "@stylistic/space-before-function-paren": [
@@ -1135,8 +1101,6 @@ const stylistic = () => {
           { exceptAfterSingleLine: true }
         ],
         "@stylistic/jsx-quotes": ["error", "prefer-double"],
-        // React
-        // Enforce new line when declare jsx element e.g. const element = (\newline<div
         "@stylistic/jsx-wrap-multilines": [
           "warn",
           {
@@ -1150,15 +1114,10 @@ const stylistic = () => {
           }
         ],
         "@stylistic/jsx-closing-tag-location": "error",
-        // Enforce disallow </ div>
         "@stylistic/jsx-tag-spacing": ["error", { closingSlash: "never", beforeSelfClosing: "always", afterOpening: "never", beforeClosing: "allow" }],
         "@stylistic/jsx-max-props-per-line": ["warn", { maximum: 1, when: "multiline" }],
-        // indentLogicalExpressions will fix && element
-        "@stylistic/jsx-indent": ["warn", 2, { indentLogicalExpressions: true }],
         "@stylistic/jsx-indent-props": ["warn", 2],
-        // < and > should be on the same line(y-axis) if jsx is multiline
         "@stylistic/jsx-closing-bracket-location": ["warn", "tag-aligned"],
-        // Enforce new line when multiline props jsx
         "@stylistic/jsx-first-prop-new-line": ["warn", "multiline-multiprop"],
         "@stylistic/jsx-one-expression-per-line": "off"
       }
@@ -1182,6 +1141,35 @@ const tailwindcss = () => {
   return config;
 };
 
+const toml = () => {
+  const config = [
+    ...pluginToml__default.default.configs.recommended,
+    {
+      rules: {
+        "toml/comma-style": "error",
+        "toml/keys-order": "error",
+        "toml/no-space-dots": "error",
+        "toml/no-unreadable-number-separator": "error",
+        "toml/precision-of-fractional-seconds": "error",
+        "toml/precision-of-integer": "error",
+        "toml/tables-order": "error",
+        "toml/array-bracket-newline": "error",
+        "toml/array-bracket-spacing": "error",
+        "toml/array-element-newline": "error",
+        "toml/indent": ["error", 4],
+        "toml/inline-table-curly-spacing": "error",
+        "toml/key-spacing": "error",
+        "toml/padding-line-between-pairs": "error",
+        "toml/padding-line-between-tables": "error",
+        "toml/quoted-keys": "error",
+        "toml/spaced-comment": "error",
+        "toml/table-bracket-spacing": "error"
+      }
+    }
+  ];
+  return config;
+};
+
 const typescript = () => {
   const config = [
     {
@@ -1315,6 +1303,7 @@ const unicorn = () => {
         unicorn: pluginUnicorn__default.default
       },
       rules: {
+        ...pluginUnicorn__default.default.configs.recommended.rules,
         "unicorn/consistent-empty-array-spread": "error",
         "unicorn/error-message": "error",
         "unicorn/escape-case": "error",
@@ -1338,17 +1327,10 @@ const unicorn = () => {
 
 const yml = () => {
   const config = [
+    ...pluginYml__default.default.configs.standard,
     {
       files: [GLOB_YAML],
-      languageOptions: {
-        parser: yamlParser__default.default
-      },
-      plugins: {
-        yml: pluginYaml__default.default
-      },
       rules: {
-        ...pluginYaml__default.default.configs.standard.rules,
-        // yml
         "spaced-comment": "off",
         "yml/quotes": ["error", { prefer: "double", avoidEscape: false }],
         "yml/no-empty-document": "off"
@@ -1357,12 +1339,6 @@ const yml = () => {
     {
       files: [GLOB_WORKFLOW_YAML],
       rules: {
-        /**
-         * on:
-         *  push:
-         *  pull_request:
-         *   branches: []
-         */
         "yml/no-empty-mapping-value": "off"
       }
     }
@@ -1384,7 +1360,8 @@ const presetTypescript = [
 const presetLangsExtensions = [
   ...yml(),
   ...markdown(),
-  ...jsonc()
+  ...jsonc(),
+  ...toml()
 ];
 const presetDefault = [
   ...presetTypescript,