@aiou/eslint-config 2.0.3 → 2.1.0

package/dist/index.cjs

@@ -63,7 +63,7 @@ const comments = () => {
         "eslint-comments": pluginComments__default.default
       },
       rules: {
-        "eslint-comments/disable-enable-pair": "error",
+        "eslint-comments/disable-enable-pair": "off",
         "eslint-comments/no-aggregating-enable": "error",
         "eslint-comments/no-duplicate-disable": "error",
         "eslint-comments/no-unlimited-disable": "error",
@@ -154,7 +154,15 @@ const imports = () => {
         "unused-imports/no-unused-imports": "error",
         "unused-imports/no-unused-vars": [
           "warn",
-          { vars: "all", varsIgnorePattern: "^_", args: "after-used", argsIgnorePattern: "^_" }
+          {
+            // https://github.com/sweepline/eslint-plugin-unused-imports/issues/105
+            caughtErrors: "all",
+            caughtErrorsIgnorePattern: "^_",
+            vars: "all",
+            varsIgnorePattern: "^_",
+            args: "after-used",
+            argsIgnorePattern: "^_"
+          }
         ],
         // Enforce newlines inside named import
         "import-newlines/enforce": [
@@ -295,22 +303,7 @@ const javascript = () => {
         // "eqeqeq": ["error", "always", { "null": "ignore" }],
         "func-call-spacing": ["error", "never"],
         "generator-star-spacing": ["error", { before: true, after: true }],
-        indent: ["error", 2, {
-          SwitchCase: 1,
-          VariableDeclarator: 1,
-          outerIIFEBody: 1,
-          MemberExpression: 1,
-          FunctionDeclaration: { parameters: 1, body: 1 },
-          FunctionExpression: { parameters: 1, body: 1 },
-          CallExpression: { arguments: 1 },
-          ArrayExpression: 1,
-          ObjectExpression: 1,
-          ImportDeclaration: 1,
-          flatTernaryExpressions: false,
-          ignoreComments: false,
-          ignoredNodes: ["TemplateLiteral *", "JSXElement", "JSXElement > *", "JSXAttribute", "JSXIdentifier", "JSXNamespacedName", "JSXMemberExpression", "JSXSpreadAttribute", "JSXExpressionContainer", "JSXOpeningElement", "JSXClosingElement", "JSXFragment", "JSXOpeningFragment", "JSXClosingFragment", "JSXText", "JSXEmptyExpression", "JSXSpreadChild"],
-          offsetTernaryExpressions: true
-        }],
+        indent: "off",
         "key-spacing": ["error", { beforeColon: false, afterColon: true }],
         "keyword-spacing": ["error", { before: true, after: true }],
         "lines-between-class-members": ["error", "always", { exceptAfterSingleLine: true }],
@@ -473,7 +466,7 @@ const javascript = () => {
         "promise/param-names": "error",
         "promise/no-new-statics": "error",
         "promise/no-multiple-resolved": "error",
-        "promise/prefer-await-to-then": "warn",
+        "promise/prefer-await-to-then": "off",
         // override
         curly: ["error", "all"],
         "no-unused-vars": "warn",
@@ -545,6 +538,7 @@ const jsonc = () => {
     {
       files: [GLOB_JSON, GLOB_JSON5, GLOB_JSONC],
       ignores: [GLOB_CLAUDE_JSON],
+      name: "jsonc/setup",
       languageOptions: {
         parser: jsoncParser__default.default
       },
@@ -554,6 +548,7 @@ const 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
@@ -897,7 +892,8 @@ const react = () => {
         "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-refresh/only-export-components": "warn",
+        "react-hooks-extra/no-direct-set-state-in-use-effect": "off"
       }
     },
     {
@@ -1061,7 +1057,7 @@ const stylistic = () => {
       rules: {
         ...off,
         ...config.rules,
-        "@stylistic/quotes": ["error", "single"],
+        "@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
@@ -1072,6 +1068,7 @@ const stylistic = () => {
         "comma-dangle": "off",
         "@stylistic/comma-dangle": ["error", "always-multiline"],
         "@stylistic/key-spacing": ["error", { beforeColon: false, afterColon: true }],
+        "@stylistic/multiline-ternary": ["error", "always-multiline"],
         "@stylistic/indent": ["error", 2, {
           SwitchCase: 1,
           VariableDeclarator: 1,
@@ -1083,7 +1080,7 @@ const stylistic = () => {
           ArrayExpression: 1,
           ObjectExpression: 1,
           ImportDeclaration: 1,
-          flatTernaryExpressions: false,
+          flatTernaryExpressions: true,
           ignoreComments: false,
           ignoredNodes: [
             "TemplateLiteral *",
@@ -1218,7 +1215,7 @@ const typescript = () => {
         "@typescript-eslint/ban-ts-comment": ["error", { "ts-ignore": "allow-with-description" }],
         "@typescript-eslint/consistent-type-imports": [
           "error",
-          { prefer: "type-imports", disallowTypeAnnotations: false }
+          { prefer: "type-imports", disallowTypeAnnotations: false, fixStyle: "separate-type-imports" }
         ],
         // Limit `interface` define object types, users could override with *.d.ts declare
         "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
@@ -1232,8 +1229,6 @@ const typescript = () => {
         // use createRequire instead
         "@typescript-eslint/no-require-imports": "error",
         "@typescript-eslint/no-empty-object-type": "off",
-        // https://www.npmjs.com/package/eslint-plugin-etc
-        "etc/no-t": "error",
         /**
          * refs:
          * 1. https://ncjamieson.com/dont-export-const-enums/
@@ -1249,15 +1244,21 @@ const typescript = () => {
             message: "Don't declare enums"
           }
         ],
+        "@typescript-eslint/method-signature-style": ["error", "property"],
+        // https://www.totaltypescript.com/method-shorthand-syntax-considered-harmful,
+        "@typescript-eslint/no-import-type-side-effects": "error",
+        "@typescript-eslint/no-wrapper-object-types": "error",
+        // https://www.npmjs.com/package/eslint-plugin-etc
+        "etc/no-t": "error",
         // required tsconfig.json
-        // 'etc/no-misused-generics': 'error',
+        // 'etc/no-misused-generics'f: 'error',
         // Overrides JS
         // original no-undef not compatiable with typescript
         // refs: https://typescript-eslint.io/linting/troubleshooting/#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
         "no-undef": "off",
         "no-unused-vars": "off",
         "no-redeclare": "off",
-        "@typescript-eslint/no-redeclare": "error",
+        "@typescript-eslint/no-redeclare": ["error", { builtinGlobals: false }],
         "no-dupe-class-members": "off",
         "@typescript-eslint/no-dupe-class-members": "error",
         "no-use-before-define": "off",
@@ -1267,6 +1268,12 @@ const typescript = () => {
         ],
         "no-loss-of-precision": "off",
         "@typescript-eslint/no-loss-of-precision": "error",
+        "no-unused-expressions": "off",
+        "@typescript-eslint/no-unused-expressions": ["error", {
+          allowShortCircuit: true,
+          allowTernary: true,
+          allowTaggedTemplates: true
+        }],
         // off
         "@typescript-eslint/camelcase": "off",
         "@typescript-eslint/explicit-function-return-type": "off",
@@ -1283,7 +1290,11 @@ const typescript = () => {
         "@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 }]
+        "no-void": ["error", { allowAsStatement: true }],
+        "@typescript-eslint/no-dynamic-delete": "off",
+        "@typescript-eslint/no-extraneous-class": "off",
+        "@typescript-eslint/no-useless-constructor": "off",
+        "@typescript-eslint/triple-slash-reference": "off"
       }
     },
     {

package/dist/index.mjs

@@ -34,7 +34,7 @@ const comments = () => {
         "eslint-comments": pluginComments
       },
       rules: {
-        "eslint-comments/disable-enable-pair": "error",
+        "eslint-comments/disable-enable-pair": "off",
         "eslint-comments/no-aggregating-enable": "error",
         "eslint-comments/no-duplicate-disable": "error",
         "eslint-comments/no-unlimited-disable": "error",
@@ -125,7 +125,15 @@ const imports = () => {
         "unused-imports/no-unused-imports": "error",
         "unused-imports/no-unused-vars": [
           "warn",
-          { vars: "all", varsIgnorePattern: "^_", args: "after-used", argsIgnorePattern: "^_" }
+          {
+            // https://github.com/sweepline/eslint-plugin-unused-imports/issues/105
+            caughtErrors: "all",
+            caughtErrorsIgnorePattern: "^_",
+            vars: "all",
+            varsIgnorePattern: "^_",
+            args: "after-used",
+            argsIgnorePattern: "^_"
+          }
         ],
         // Enforce newlines inside named import
         "import-newlines/enforce": [
@@ -266,22 +274,7 @@ const javascript = () => {
         // "eqeqeq": ["error", "always", { "null": "ignore" }],
         "func-call-spacing": ["error", "never"],
         "generator-star-spacing": ["error", { before: true, after: true }],
-        indent: ["error", 2, {
-          SwitchCase: 1,
-          VariableDeclarator: 1,
-          outerIIFEBody: 1,
-          MemberExpression: 1,
-          FunctionDeclaration: { parameters: 1, body: 1 },
-          FunctionExpression: { parameters: 1, body: 1 },
-          CallExpression: { arguments: 1 },
-          ArrayExpression: 1,
-          ObjectExpression: 1,
-          ImportDeclaration: 1,
-          flatTernaryExpressions: false,
-          ignoreComments: false,
-          ignoredNodes: ["TemplateLiteral *", "JSXElement", "JSXElement > *", "JSXAttribute", "JSXIdentifier", "JSXNamespacedName", "JSXMemberExpression", "JSXSpreadAttribute", "JSXExpressionContainer", "JSXOpeningElement", "JSXClosingElement", "JSXFragment", "JSXOpeningFragment", "JSXClosingFragment", "JSXText", "JSXEmptyExpression", "JSXSpreadChild"],
-          offsetTernaryExpressions: true
-        }],
+        indent: "off",
         "key-spacing": ["error", { beforeColon: false, afterColon: true }],
         "keyword-spacing": ["error", { before: true, after: true }],
         "lines-between-class-members": ["error", "always", { exceptAfterSingleLine: true }],
@@ -444,7 +437,7 @@ const javascript = () => {
         "promise/param-names": "error",
         "promise/no-new-statics": "error",
         "promise/no-multiple-resolved": "error",
-        "promise/prefer-await-to-then": "warn",
+        "promise/prefer-await-to-then": "off",
         // override
         curly: ["error", "all"],
         "no-unused-vars": "warn",
@@ -516,6 +509,7 @@ const jsonc = () => {
     {
       files: [GLOB_JSON, GLOB_JSON5, GLOB_JSONC],
       ignores: [GLOB_CLAUDE_JSON],
+      name: "jsonc/setup",
       languageOptions: {
         parser: jsoncParser
       },
@@ -525,6 +519,7 @@ const jsonc = () => {
     },
     {
       files: [GLOB_JSON, GLOB_JSON5, GLOB_JSONC],
+      ignores: [GLOB_CLAUDE_JSON],
       rules: {
         ...pluginJsonc.configs["recommended-with-jsonc"].rules,
         // refs: https://ota-meshi.github.io/eslint-plugin-jsonc/rules/indent.html
@@ -868,7 +863,8 @@ const react = () => {
         "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-refresh/only-export-components": "warn",
+        "react-hooks-extra/no-direct-set-state-in-use-effect": "off"
       }
     },
     {
@@ -1032,7 +1028,7 @@ const stylistic = () => {
       rules: {
         ...off,
         ...config.rules,
-        "@stylistic/quotes": ["error", "single"],
+        "@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
@@ -1043,6 +1039,7 @@ const stylistic = () => {
         "comma-dangle": "off",
         "@stylistic/comma-dangle": ["error", "always-multiline"],
         "@stylistic/key-spacing": ["error", { beforeColon: false, afterColon: true }],
+        "@stylistic/multiline-ternary": ["error", "always-multiline"],
         "@stylistic/indent": ["error", 2, {
           SwitchCase: 1,
           VariableDeclarator: 1,
@@ -1054,7 +1051,7 @@ const stylistic = () => {
           ArrayExpression: 1,
           ObjectExpression: 1,
           ImportDeclaration: 1,
-          flatTernaryExpressions: false,
+          flatTernaryExpressions: true,
           ignoreComments: false,
           ignoredNodes: [
             "TemplateLiteral *",
@@ -1189,7 +1186,7 @@ const typescript = () => {
         "@typescript-eslint/ban-ts-comment": ["error", { "ts-ignore": "allow-with-description" }],
         "@typescript-eslint/consistent-type-imports": [
           "error",
-          { prefer: "type-imports", disallowTypeAnnotations: false }
+          { prefer: "type-imports", disallowTypeAnnotations: false, fixStyle: "separate-type-imports" }
         ],
         // Limit `interface` define object types, users could override with *.d.ts declare
         "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
@@ -1203,8 +1200,6 @@ const typescript = () => {
         // use createRequire instead
         "@typescript-eslint/no-require-imports": "error",
         "@typescript-eslint/no-empty-object-type": "off",
-        // https://www.npmjs.com/package/eslint-plugin-etc
-        "etc/no-t": "error",
         /**
          * refs:
          * 1. https://ncjamieson.com/dont-export-const-enums/
@@ -1220,15 +1215,21 @@ const typescript = () => {
             message: "Don't declare enums"
           }
         ],
+        "@typescript-eslint/method-signature-style": ["error", "property"],
+        // https://www.totaltypescript.com/method-shorthand-syntax-considered-harmful,
+        "@typescript-eslint/no-import-type-side-effects": "error",
+        "@typescript-eslint/no-wrapper-object-types": "error",
+        // https://www.npmjs.com/package/eslint-plugin-etc
+        "etc/no-t": "error",
         // required tsconfig.json
-        // 'etc/no-misused-generics': 'error',
+        // 'etc/no-misused-generics'f: 'error',
         // Overrides JS
         // original no-undef not compatiable with typescript
         // refs: https://typescript-eslint.io/linting/troubleshooting/#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
         "no-undef": "off",
         "no-unused-vars": "off",
         "no-redeclare": "off",
-        "@typescript-eslint/no-redeclare": "error",
+        "@typescript-eslint/no-redeclare": ["error", { builtinGlobals: false }],
         "no-dupe-class-members": "off",
         "@typescript-eslint/no-dupe-class-members": "error",
         "no-use-before-define": "off",
@@ -1238,6 +1239,12 @@ const typescript = () => {
         ],
         "no-loss-of-precision": "off",
         "@typescript-eslint/no-loss-of-precision": "error",
+        "no-unused-expressions": "off",
+        "@typescript-eslint/no-unused-expressions": ["error", {
+          allowShortCircuit: true,
+          allowTernary: true,
+          allowTaggedTemplates: true
+        }],
         // off
         "@typescript-eslint/camelcase": "off",
         "@typescript-eslint/explicit-function-return-type": "off",
@@ -1254,7 +1261,11 @@ const typescript = () => {
         "@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 }]
+        "no-void": ["error", { allowAsStatement: true }],
+        "@typescript-eslint/no-dynamic-delete": "off",
+        "@typescript-eslint/no-extraneous-class": "off",
+        "@typescript-eslint/no-useless-constructor": "off",
+        "@typescript-eslint/triple-slash-reference": "off"
       }
     },
     {

package/dts/typegen.d.ts

@@ -7937,5 +7937,5 @@ type YmlSpacedComment = [] | [("always" | "never")] | [
         markers?: string[];
     }
 ];
-export type ConfigNames = 'markdown/recommended/plugin' | 'markdown/recommended/processor' | 'markdown/recommended/code-blocks' | 'tailwindcss:base' | 'tailwindcss:rules';
+export type ConfigNames = 'markdown/recommended/plugin' | 'markdown/recommended/processor' | 'markdown/recommended/code-blocks' | 'jsonc/setup' | 'tailwindcss:base' | 'tailwindcss:rules';
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiou/eslint-config",
-  "version": "2.0.3",
+  "version": "2.1.0",
   "type": "module",
   "description": "eslint config for JW",
   "keywords": [],