@allthings/eslint-config 3.1.0-beta.1 → 3.2.0-beta.1

package/README.md

@@ -59,6 +59,17 @@ to restore the initial state of dependencies
 
 Or you could release a dev npm version with `yarn deploy:dev`. Remember to update the version in package.json
 
+### Testing the config
+
+The config is protected by a two-layer pipeline that runs in CI on every push:
+
+- **`yarn test:audit`** — static rule audit. Loads each config and verifies every enabled rule still resolves in its plugin (catches rules removed or renamed by a dependency bump). Reports any rules whose `meta.deprecated` is set so we can migrate before they disappear.
+- **`yarn test:behavioral`** — fixture-based check. Lints `test/fixtures/full/` against `index.js` and `test/fixtures/node/` against `node.js`. Files under `src/clean/` must lint with zero errors; files under `src/violations/` must trigger the rule IDs declared in `manifest.json`.
+
+`yarn test` runs both. The CI workflow exposes them as separate steps so a Renovate PR makes it clear which layer regressed.
+
+When adding or modifying a rule that consumers rely on, add a fixture line that exercises it — either a clean usage in `clean/` or a violation file with a `manifest.json` entry.
+
 ### Production release
 
 !! DO NOT `npm version` !!

package/index.js

@@ -75,6 +75,14 @@ export default [
           },
         },
       ],
+      '@stylistic/padding-line-between-statements': [
+        'error',
+        {
+          blankLine: 'always',
+          prev: '*',
+          next: 'return',
+        },
+      ],
       '@stylistic/quotes': [
         'error',
         'single',
@@ -83,6 +91,13 @@ export default [
         },
       ],
       '@stylistic/semi': ['error', 'never'],
+      '@stylistic/spaced-comment': [
+        'error',
+        'always',
+        {
+          markers: ['/'],
+        },
+      ],
       '@stylistic/type-annotation-spacing': 'error',
       '@typescript-eslint/adjacent-overload-signatures': 'error',
       '@typescript-eslint/array-type': [
@@ -117,7 +132,7 @@ export default [
       '@typescript-eslint/no-base-to-string': 'warn',
       // need for tests' mockups
       '@typescript-eslint/no-empty-function': 'warn',
-      '@typescript-eslint/no-empty-interface': 'error',
+      '@typescript-eslint/no-empty-object-type': 'error',
       '@typescript-eslint/no-explicit-any': 'warn',
       // turn on later
       '@typescript-eslint/no-floating-promises': 'warn',
@@ -126,7 +141,7 @@ export default [
       '@typescript-eslint/no-misused-new': 'error',
       '@typescript-eslint/no-namespace': 'error',
       '@typescript-eslint/no-redundant-type-constituents': 'warn',
-      '@typescript-eslint/no-require-imports': 'warn',
+      '@typescript-eslint/no-require-imports': 'error',
       '@typescript-eslint/no-shadow': [
         'error',
         {
@@ -147,7 +162,6 @@ export default [
         { varsIgnorePattern: '^jsx$' },
       ],
       '@typescript-eslint/no-use-before-define': 'error',
-      '@typescript-eslint/no-var-requires': 'error',
       '@typescript-eslint/prefer-for-of': 'error',
       '@typescript-eslint/prefer-function-type': 'error',
       '@typescript-eslint/prefer-namespace-keyword': 'error',
@@ -162,7 +176,6 @@ export default [
           lib: 'always',
         },
       ],
-      '@typescript-eslint/typedef': 'error',
       '@typescript-eslint/unified-signatures': 'error',
       'arrow-body-style': ['error', 'as-needed'],
       complexity: [
@@ -226,7 +239,6 @@ export default [
       'no-new-wrappers': 'error',
       'no-param-reassign': 'error',
       'no-redeclare': 'error',
-      'no-return-await': 'error',
       'no-sequences': 'error',
       'no-sparse-arrays': 'error',
       'no-template-curly-in-string': 'error',
@@ -238,14 +250,6 @@ export default [
       'no-var': 'error',
       'object-shorthand': 'error',
       'one-var': ['error', 'never'],
-      'padding-line-between-statements': [
-        'error',
-        {
-          blankLine: 'always',
-          prev: '*',
-          next: 'return',
-        },
-      ],
       'prefer-arrow/prefer-arrow-functions': 'warn',
       'prefer-const': 'error',
       'prefer-object-spread': 'error',
@@ -273,13 +277,6 @@ export default [
       'simple-import-sort/exports': 'error',
       'simple-import-sort/imports': 'error',
       'sort-keys': 'error',
-      'spaced-comment': [
-        'error',
-        'always',
-        {
-          markers: ['/'],
-        },
-      ],
       'unicorn/better-regex': 'off',
       'unicorn/consistent-existence-index-check': 'off',
       'unicorn/expiring-todo-comments': 'off',

package/node.js

@@ -61,6 +61,14 @@ export default [
           },
         },
       ],
+      '@stylistic/padding-line-between-statements': [
+        'error',
+        {
+          blankLine: 'always',
+          prev: '*',
+          next: 'return',
+        },
+      ],
       '@stylistic/quotes': [
         'error',
         'single',
@@ -69,6 +77,13 @@ export default [
         },
       ],
       '@stylistic/semi': ['error', 'never'],
+      '@stylistic/spaced-comment': [
+        'error',
+        'always',
+        {
+          markers: ['/'],
+        },
+      ],
       '@stylistic/type-annotation-spacing': 'error',
       '@typescript-eslint/adjacent-overload-signatures': 'error',
       '@typescript-eslint/array-type': [
@@ -104,7 +119,7 @@ export default [
       '@typescript-eslint/no-base-to-string': 'warn',
       // need for tests' mockups
       '@typescript-eslint/no-empty-function': 'warn',
-      '@typescript-eslint/no-empty-interface': 'error',
+      '@typescript-eslint/no-empty-object-type': 'error',
       '@typescript-eslint/no-explicit-any': 'warn',
       // turn on later
       '@typescript-eslint/no-floating-promises': 'warn',
@@ -113,7 +128,7 @@ export default [
       '@typescript-eslint/no-misused-new': 'error',
       '@typescript-eslint/no-namespace': 'error',
       '@typescript-eslint/no-redundant-type-constituents': 'warn',
-      '@typescript-eslint/no-require-imports': 'warn',
+      '@typescript-eslint/no-require-imports': 'error',
       '@typescript-eslint/no-shadow': [
         'error',
         {
@@ -130,7 +145,6 @@ export default [
       '@typescript-eslint/no-unused-expressions': 'error',
       '@typescript-eslint/no-unused-vars': 'error',
       '@typescript-eslint/no-use-before-define': 'error',
-      '@typescript-eslint/no-var-requires': 'error',
       '@typescript-eslint/prefer-for-of': 'error',
       '@typescript-eslint/prefer-function-type': 'error',
       '@typescript-eslint/prefer-namespace-keyword': 'error',
@@ -145,7 +159,6 @@ export default [
           lib: 'always',
         },
       ],
-      '@typescript-eslint/typedef': 'error',
       '@typescript-eslint/unified-signatures': 'error',
       'arrow-body-style': ['error', 'as-needed'],
       complexity: [
@@ -209,7 +222,6 @@ export default [
       'no-new-wrappers': 'error',
       'no-param-reassign': 'error',
       'no-redeclare': 'error',
-      'no-return-await': 'error',
       'no-sequences': 'error',
       'no-sparse-arrays': 'error',
       'no-template-curly-in-string': 'error',
@@ -221,14 +233,6 @@ export default [
       'no-var': 'error',
       'object-shorthand': 'error',
       'one-var': ['error', 'never'],
-      'padding-line-between-statements': [
-        'error',
-        {
-          blankLine: 'always',
-          prev: '*',
-          next: 'return',
-        },
-      ],
       'prefer-arrow/prefer-arrow-functions': 'warn',
       'prefer-const': 'error',
       'prefer-object-spread': 'error',
@@ -238,13 +242,6 @@ export default [
       'simple-import-sort/exports': 'error',
       'simple-import-sort/imports': 'error',
       'sort-keys': 'error',
-      'spaced-comment': [
-        'error',
-        'always',
-        {
-          markers: ['/'],
-        },
-      ],
       'unicorn/better-regex': 'off',
       'unicorn/consistent-existence-index-check': 'off',
       'unicorn/expiring-todo-comments': 'off',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allthings/eslint-config",
-  "version": "3.1.0-beta.1",
+  "version": "3.2.0-beta.1",
   "type": "module",
   "description": "ESlint shareable config for Allthings style",
   "main": "index.js",
@@ -23,7 +23,9 @@
   "scripts": {
     "lint": "prettier --check .",
     "semantic-release": "semantic-release",
-    "test": "eslint test/",
+    "test": "yarn test:audit && yarn test:behavioral",
+    "test:audit": "node test/audit.js",
+    "test:behavioral": "node test/behavioral.js",
     "deps": "yarn upgrade-interactive --latest"
   },
   "dependencies": {
@@ -52,8 +54,11 @@
     "typescript": ">=5.9.0"
   },
   "devDependencies": {
+    "@types/jest": "30.0.0",
+    "@types/react": "18.3.28",
     "eslint": "9.39.4",
     "prettier": "3.8.1",
+    "react": "18.3.1",
     "semantic-release": "25.0.3",
     "typescript": "5.9.3"
   },