@rebeccastevens/eslint-config 1.5.4 → 1.5.6

package/README.md

@@ -28,6 +28,8 @@ pnpm add -D  \
 pnpm dlx install-peerdeps @rebeccastevens/eslint-config --dev -o -Y
 ```
 
+Note: This project doesn't strictly follow semantic versioning so be sure to pin the version you are using.
+
 ## Usage
 
 ### Language

package/dist/common-overrides.cjs

@@ -1,5 +1,8 @@
 'use strict';
 
+const jsxFiles = ["**/*.{jsx,cjsx,mjsx,tsx,ctsx,mtsx}"];
+const testFiles = ["{test,tests}/**/*", "**/*.{spec,test}.*"];
+
 var commonOverrides = {
     overrides: [
         {
@@ -16,16 +19,19 @@ var commonOverrides = {
             extends: ["@rebeccastevens/eslint-config/script"],
         },
         {
-            files: ["{test,tests}/**/*", "**/*.test.*"],
+            files: testFiles,
             extends: ["@rebeccastevens/eslint-config/test"],
         },
         {
-            files: ["**/*.{cjs,cts}"],
-            rules: {
-                "@typescript-eslint/no-var-requires": "off",
-                "import/no-dynamic-require": "off",
-                "node/no-missing-require": "off",
-                "unicorn/prefer-module": "off",
+            files: jsxFiles,
+            parserOptions: {
+                ecmaVersion: "latest",
+                sourceType: "module",
+                ecmaFeatures: {
+                    globalReturn: false,
+                    impliedStrict: true,
+                    jsx: true,
+                },
             },
         },
     ],

package/dist/common-overrides.mjs

@@ -1,3 +1,6 @@
+const jsxFiles = ["**/*.{jsx,cjsx,mjsx,tsx,ctsx,mtsx}"];
+const testFiles = ["{test,tests}/**/*", "**/*.{spec,test}.*"];
+
 var commonOverrides = {
     overrides: [
         {
@@ -14,16 +17,19 @@ var commonOverrides = {
             extends: ["@rebeccastevens/eslint-config/script"],
         },
         {
-            files: ["{test,tests}/**/*", "**/*.test.*"],
+            files: testFiles,
             extends: ["@rebeccastevens/eslint-config/test"],
         },
         {
-            files: ["**/*.{cjs,cts}"],
-            rules: {
-                "@typescript-eslint/no-var-requires": "off",
-                "import/no-dynamic-require": "off",
-                "node/no-missing-require": "off",
-                "unicorn/prefer-module": "off",
+            files: jsxFiles,
+            parserOptions: {
+                ecmaVersion: "latest",
+                sourceType: "module",
+                ecmaFeatures: {
+                    globalReturn: false,
+                    impliedStrict: true,
+                    jsx: true,
+                },
             },
         },
     ],

package/dist/modern.cjs

@@ -27,6 +27,7 @@ const rules$4 = {
     "require-jsdoc": "off",
     "valid-jsdoc": "off",
 };
+const overrides$4 = [];
 
 const rules$3 = {
     "array-bracket-spacing": ["error", "never"],
@@ -314,6 +315,7 @@ const rules$3 = {
     ],
     "yield-star-spacing": ["error", "after"],
 };
+const overrides$3 = [];
 
 const rules$2 = {
     "constructor-super": "error",
@@ -397,6 +399,11 @@ const rules$2 = {
         },
     ],
 };
+const overrides$2 = [];
+
+const typescriptFiles = ["**/*.{ts,tsx,mts,mtsx,cts,ctsx}"];
+const typescriptDeclarationFiles = ["**/*.d.{ts,mts,cts}"];
+const commonJsFiles = ["**/*.{cjs,cts}"];
 
 const useNumberIsFinite = "Please use Number.isFinite instead";
 const useNumberIsNan = "Please use Number.isNaN instead";
@@ -579,6 +586,14 @@ const rules$1 = {
     "symbol-description": "error",
     "yoda": ["error", "never"],
 };
+const overrides$1 = [
+    {
+        files: typescriptDeclarationFiles,
+        rules: {
+            "init-declarations": "off",
+        },
+    },
+];
 
 const rules = {
     ...rules$2,
@@ -586,6 +601,12 @@ const rules = {
     ...rules$3,
     ...rules$4,
 };
+const overrides = [
+    ...overrides$4,
+    ...overrides$3,
+    ...overrides$2,
+    ...overrides$1,
+];
 
 const settings$9 = {
     plugins: ["eslint-comments"],
@@ -804,9 +825,19 @@ const settings$7 = {
     },
     overrides: [
         {
-            files: ["**/*.cjs"],
+            files: commonJsFiles,
             rules: {
                 "import/no-commonjs": "off",
+                "import/no-dynamic-require": "off",
+            },
+        },
+        {
+            files: typescriptFiles,
+            rules: {
+                "import/no-unresolved": "off",
+                "import/named": "off",
+                "import/default": "off",
+                "import/namespace": "off",
             },
         },
     ],
@@ -814,28 +845,14 @@ const settings$7 = {
 
 const settings$6 = {
     plugins: ["jsdoc"],
+    extends: ["plugin:jsdoc/recommended"],
     rules: {
-        "jsdoc/check-access": "warn",
-        "jsdoc/check-alignment": "warn",
         // waiting on https://github.com/eslint/eslint/issues/14745
         // "jsdoc/check-examples": "warn",
         "jsdoc/check-indentation": "warn",
         "jsdoc/check-line-alignment": "warn",
-        "jsdoc/check-param-names": "warn",
-        "jsdoc/check-property-names": "warn",
-        "jsdoc/check-tag-names": "warn",
-        "jsdoc/check-types": "warn",
-        "jsdoc/check-values": "warn",
-        "jsdoc/empty-tags": "warn",
-        "jsdoc/implements-on-classes": "warn",
-        "jsdoc/multiline-blocks": "warn",
-        "jsdoc/newline-after-description": "warn",
         "jsdoc/no-bad-blocks": "warn",
         "jsdoc/no-defaults": "warn",
-        "jsdoc/no-multi-asterisks": "warn",
-        // For TypeScript
-        "jsdoc/no-types": "warn",
-        "jsdoc/no-undefined-types": "warn",
         "jsdoc/require-asterisk-prefix": "warn",
         "jsdoc/require-description": "warn",
         // Rule is too strict.
@@ -858,21 +875,27 @@ const settings$6 = {
                 },
             },
         ],
-        "jsdoc/require-param-description": "warn",
-        "jsdoc/require-param-name": "warn",
-        "jsdoc/require-property-description": "warn",
-        "jsdoc/require-property-name": "warn",
-        "jsdoc/require-returns-check": "warn",
-        "jsdoc/require-returns-description": "warn",
-        "jsdoc/require-yields": "warn",
-        "jsdoc/require-yields-check": "warn",
+        "jsdoc/require-throws": "warn",
         "jsdoc/tag-lines": [
             "warn",
             "never",
-            { tags: { example: { lines: "always" } } },
+            { noEndLines: true, tags: { example: { lines: "always" } } },
         ],
-        "jsdoc/valid-types": "warn",
     },
+    overrides: [
+        {
+            files: typescriptFiles,
+            rules: {
+                "jsdoc/no-types": "warn",
+                "jsdoc/require-param-type": "off",
+                "jsdoc/require-param": "off",
+                "jsdoc/require-property-type": "off",
+                "jsdoc/require-property": "off",
+                "jsdoc/require-returns-type": "off",
+                "jsdoc/require-returns": "off",
+            },
+        },
+    ],
 };
 
 const settings$5 = {
@@ -983,6 +1006,20 @@ const settings$4 = {
         "node/prefer-promises/dns": "error",
         "node/prefer-promises/fs": "error",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "node/no-missing-require": "off",
+            },
+        },
+        {
+            files: typescriptFiles,
+            rules: {
+                "node/no-unsupported-features/es-syntax": "off",
+            },
+        },
+    ],
 };
 
 const settings$3 = {
@@ -1045,25 +1082,31 @@ const settings = {
         "unicorn/prefer-top-level-await": "error",
         "unicorn/prevent-abbreviations": "off",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "unicorn/prefer-module": "off",
+            },
+        },
+    ],
 };
 
 const baseConfig = {
     parser: "babel-eslint",
     parserOptions: {
-        ecmaVersion: 2021,
+        ecmaVersion: "latest",
         ecmaFeatures: {
             globalReturn: false,
             impliedStrict: true,
         },
         sourceType: "module",
-        extraFileExtensions: [".cjs", ".mjs"],
     },
     extends: ["eslint:recommended"],
-    env: {
-        es6: true,
-    },
     rules: rules,
+    overrides: overrides,
     ignorePatterns: ["dist/"],
+    reportUnusedDisableDirectives: true,
 };
 var modern = deepmergeTs.deepmerge(baseConfig, settings$9, settings$8, settings$7, settings$6, settings$5, settings$4, settings$3, settings$2, settings$1, settings);
 

package/dist/modern.mjs

@@ -25,6 +25,7 @@ const rules$4 = {
     "require-jsdoc": "off",
     "valid-jsdoc": "off",
 };
+const overrides$4 = [];
 
 const rules$3 = {
     "array-bracket-spacing": ["error", "never"],
@@ -312,6 +313,7 @@ const rules$3 = {
     ],
     "yield-star-spacing": ["error", "after"],
 };
+const overrides$3 = [];
 
 const rules$2 = {
     "constructor-super": "error",
@@ -395,6 +397,11 @@ const rules$2 = {
         },
     ],
 };
+const overrides$2 = [];
+
+const typescriptFiles = ["**/*.{ts,tsx,mts,mtsx,cts,ctsx}"];
+const typescriptDeclarationFiles = ["**/*.d.{ts,mts,cts}"];
+const commonJsFiles = ["**/*.{cjs,cts}"];
 
 const useNumberIsFinite = "Please use Number.isFinite instead";
 const useNumberIsNan = "Please use Number.isNaN instead";
@@ -577,6 +584,14 @@ const rules$1 = {
     "symbol-description": "error",
     "yoda": ["error", "never"],
 };
+const overrides$1 = [
+    {
+        files: typescriptDeclarationFiles,
+        rules: {
+            "init-declarations": "off",
+        },
+    },
+];
 
 const rules = {
     ...rules$2,
@@ -584,6 +599,12 @@ const rules = {
     ...rules$3,
     ...rules$4,
 };
+const overrides = [
+    ...overrides$4,
+    ...overrides$3,
+    ...overrides$2,
+    ...overrides$1,
+];
 
 const settings$9 = {
     plugins: ["eslint-comments"],
@@ -802,9 +823,19 @@ const settings$7 = {
     },
     overrides: [
         {
-            files: ["**/*.cjs"],
+            files: commonJsFiles,
             rules: {
                 "import/no-commonjs": "off",
+                "import/no-dynamic-require": "off",
+            },
+        },
+        {
+            files: typescriptFiles,
+            rules: {
+                "import/no-unresolved": "off",
+                "import/named": "off",
+                "import/default": "off",
+                "import/namespace": "off",
             },
         },
     ],
@@ -812,28 +843,14 @@ const settings$7 = {
 
 const settings$6 = {
     plugins: ["jsdoc"],
+    extends: ["plugin:jsdoc/recommended"],
     rules: {
-        "jsdoc/check-access": "warn",
-        "jsdoc/check-alignment": "warn",
         // waiting on https://github.com/eslint/eslint/issues/14745
         // "jsdoc/check-examples": "warn",
         "jsdoc/check-indentation": "warn",
         "jsdoc/check-line-alignment": "warn",
-        "jsdoc/check-param-names": "warn",
-        "jsdoc/check-property-names": "warn",
-        "jsdoc/check-tag-names": "warn",
-        "jsdoc/check-types": "warn",
-        "jsdoc/check-values": "warn",
-        "jsdoc/empty-tags": "warn",
-        "jsdoc/implements-on-classes": "warn",
-        "jsdoc/multiline-blocks": "warn",
-        "jsdoc/newline-after-description": "warn",
         "jsdoc/no-bad-blocks": "warn",
         "jsdoc/no-defaults": "warn",
-        "jsdoc/no-multi-asterisks": "warn",
-        // For TypeScript
-        "jsdoc/no-types": "warn",
-        "jsdoc/no-undefined-types": "warn",
         "jsdoc/require-asterisk-prefix": "warn",
         "jsdoc/require-description": "warn",
         // Rule is too strict.
@@ -856,21 +873,27 @@ const settings$6 = {
                 },
             },
         ],
-        "jsdoc/require-param-description": "warn",
-        "jsdoc/require-param-name": "warn",
-        "jsdoc/require-property-description": "warn",
-        "jsdoc/require-property-name": "warn",
-        "jsdoc/require-returns-check": "warn",
-        "jsdoc/require-returns-description": "warn",
-        "jsdoc/require-yields": "warn",
-        "jsdoc/require-yields-check": "warn",
+        "jsdoc/require-throws": "warn",
         "jsdoc/tag-lines": [
             "warn",
             "never",
-            { tags: { example: { lines: "always" } } },
+            { noEndLines: true, tags: { example: { lines: "always" } } },
         ],
-        "jsdoc/valid-types": "warn",
     },
+    overrides: [
+        {
+            files: typescriptFiles,
+            rules: {
+                "jsdoc/no-types": "warn",
+                "jsdoc/require-param-type": "off",
+                "jsdoc/require-param": "off",
+                "jsdoc/require-property-type": "off",
+                "jsdoc/require-property": "off",
+                "jsdoc/require-returns-type": "off",
+                "jsdoc/require-returns": "off",
+            },
+        },
+    ],
 };
 
 const settings$5 = {
@@ -981,6 +1004,20 @@ const settings$4 = {
         "node/prefer-promises/dns": "error",
         "node/prefer-promises/fs": "error",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "node/no-missing-require": "off",
+            },
+        },
+        {
+            files: typescriptFiles,
+            rules: {
+                "node/no-unsupported-features/es-syntax": "off",
+            },
+        },
+    ],
 };
 
 const settings$3 = {
@@ -1043,25 +1080,31 @@ const settings = {
         "unicorn/prefer-top-level-await": "error",
         "unicorn/prevent-abbreviations": "off",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "unicorn/prefer-module": "off",
+            },
+        },
+    ],
 };
 
 const baseConfig = {
     parser: "babel-eslint",
     parserOptions: {
-        ecmaVersion: 2021,
+        ecmaVersion: "latest",
         ecmaFeatures: {
             globalReturn: false,
             impliedStrict: true,
         },
         sourceType: "module",
-        extraFileExtensions: [".cjs", ".mjs"],
     },
     extends: ["eslint:recommended"],
-    env: {
-        es6: true,
-    },
     rules: rules,
+    overrides: overrides,
     ignorePatterns: ["dist/"],
+    reportUnusedDisableDirectives: true,
 };
 var modern = deepmerge(baseConfig, settings$9, settings$8, settings$7, settings$6, settings$5, settings$4, settings$3, settings$2, settings$1, settings);
 

package/dist/typescript.cjs

@@ -2,6 +2,9 @@
 
 var deepmergeTs = require('deepmerge-ts');
 
+const typescriptDeclarationFiles = ["**/*.d.{ts,mts,cts}"];
+const commonJsFiles = ["**/*.{cjs,cts}"];
+
 const settings = {
     plugins: ["@typescript-eslint"],
     extends: [
@@ -266,6 +269,21 @@ const settings = {
         "@typescript-eslint/unified-signatures": "warn",
         "@typescript-eslint/no-redeclare": "error",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "@typescript-eslint/no-var-requires": "off",
+            },
+        },
+        {
+            files: typescriptDeclarationFiles,
+            rules: {
+                "@typescript-eslint/consistent-type-definitions": "off",
+                "@typescript-eslint/triple-slash-reference": "off",
+            },
+        },
+    ],
 };
 
 const baseConfig = {
@@ -296,33 +314,22 @@ const baseConfig = {
             ".tsx",
             ".mjs",
             ".mts",
+            ".mtsx",
             ".cjs",
             ".cts",
+            ".ctsx",
         ],
         "import/parsers": {
-            "@typescript-eslint/parser": [".ts", ".tsx", ".mts", ".cts"],
+            "@typescript-eslint/parser": [
+                ".ts",
+                ".tsx",
+                ".mts",
+                ".mtsx",
+                ".cts",
+                ".ctsx",
+            ],
         },
     },
-    overrides: [
-        {
-            files: ["**/*.{ts,tsx,mts,cts}"],
-            rules: {
-                "import/no-unresolved": "off",
-                "import/named": "off",
-                "import/default": "off",
-                "import/namespace": "off",
-                "node/no-unsupported-features/es-syntax": "off",
-            },
-        },
-        {
-            files: "**/*.d.{ts,mts,cts}",
-            rules: {
-                "@typescript-eslint/consistent-type-definitions": "off",
-                "@typescript-eslint/triple-slash-reference": "off",
-                "init-declarations": "off",
-            },
-        },
-    ],
 };
 var typescript = deepmergeTs.deepmerge(baseConfig, settings);
 

package/dist/typescript.mjs

@@ -1,5 +1,8 @@
 import { deepmerge } from 'deepmerge-ts';
 
+const typescriptDeclarationFiles = ["**/*.d.{ts,mts,cts}"];
+const commonJsFiles = ["**/*.{cjs,cts}"];
+
 const settings = {
     plugins: ["@typescript-eslint"],
     extends: [
@@ -264,6 +267,21 @@ const settings = {
         "@typescript-eslint/unified-signatures": "warn",
         "@typescript-eslint/no-redeclare": "error",
     },
+    overrides: [
+        {
+            files: commonJsFiles,
+            rules: {
+                "@typescript-eslint/no-var-requires": "off",
+            },
+        },
+        {
+            files: typescriptDeclarationFiles,
+            rules: {
+                "@typescript-eslint/consistent-type-definitions": "off",
+                "@typescript-eslint/triple-slash-reference": "off",
+            },
+        },
+    ],
 };
 
 const baseConfig = {
@@ -294,33 +312,22 @@ const baseConfig = {
             ".tsx",
             ".mjs",
             ".mts",
+            ".mtsx",
             ".cjs",
             ".cts",
+            ".ctsx",
         ],
         "import/parsers": {
-            "@typescript-eslint/parser": [".ts", ".tsx", ".mts", ".cts"],
+            "@typescript-eslint/parser": [
+                ".ts",
+                ".tsx",
+                ".mts",
+                ".mtsx",
+                ".cts",
+                ".ctsx",
+            ],
         },
     },
-    overrides: [
-        {
-            files: ["**/*.{ts,tsx,mts,cts}"],
-            rules: {
-                "import/no-unresolved": "off",
-                "import/named": "off",
-                "import/default": "off",
-                "import/namespace": "off",
-                "node/no-unsupported-features/es-syntax": "off",
-            },
-        },
-        {
-            files: "**/*.d.{ts,mts,cts}",
-            rules: {
-                "@typescript-eslint/consistent-type-definitions": "off",
-                "@typescript-eslint/triple-slash-reference": "off",
-                "init-declarations": "off",
-            },
-        },
-    ],
 };
 var typescript = deepmerge(baseConfig, settings);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rebeccastevens/eslint-config",
-  "version": "1.5.4",
+  "version": "1.5.6",
   "description": "My ESLint shareable config.",
   "keywords": [
     "eslint config"
@@ -82,7 +82,7 @@
     "@typescript-eslint/eslint-plugin": "5.54.1",
     "@typescript-eslint/parser": "5.54.1",
     "commitizen": "4.3.0",
-    "cspell": "6.28.0",
+    "cspell": "6.29.0",
     "cz-conventional-changelog": "3.3.0",
     "eslint": "8.36.0",
     "eslint-config-prettier": "8.7.0",