feat(eslint-plugin): further simplify ESLint configs, regroup by pack…

…age.

modules/eslint-plugin/schematics/ng-add/schema.json

@@ -7,72 +7,42 @@
     "config": {
       "description": "The config to be used.",
       "type": "string",
-      "default": "recommended",
+      "default": "all",
       "enum": [
-        "recommended",
         "all",
-        "store-recommended",
-        "store-all",
-        "effects-recommended",
-        "effects-all",
-        "component-store-recommended",
-        "component-store-all",
-        "operators-recommended",
-        "operators-all",
-        "signals-recommended",
-        "signals-all"
+        "component-store",
+        "effects",
+        "operators",
+        "signals",
+        "store"
       ],
       "x-prompt": {
         "message": "Which ESLint configuration would you like to use?",
         "type": "list",
         "items": [
-          {
-            "value": "recommended",
-            "label": "recommended"
-          },
           {
             "value": "all",
             "label": "all"
           },
           {
-            "value": "store-recommended",
-            "label": "store-recommended"
-          },
-          {
-            "value": "store-all",
-            "label": "store-all"
-          },
-          {
-            "value": "effects-recommended",
-            "label": "effects-recommended"
-          },
-          {
-            "value": "effects-all",
-            "label": "effects-all"
-          },
-          {
-            "value": "component-store-recommended",
-            "label": "component-store-recommended"
-          },
-          {
-            "value": "component-store-all",
-            "label": "component-store-all"
+            "value": "component-store",
+            "label": "component-store"
           },
           {
-            "value": "operators-recommended",
-            "label": "operators-recommended"
+            "value": "effects",
+            "label": "effects"
           },
           {
-            "value": "operators-all",
-            "label": "operators-all"
+            "value": "operators",
+            "label": "operators"
           },
           {
-            "value": "signals-recommended",
-            "label": "signals-recommended"
+            "value": "signals",
+            "label": "signals"
           },
           {
-            "value": "signals-all",
-            "label": "signals-all"
+            "value": "store",
+            "label": "store"
           }
         ]
       }

modules/eslint-plugin/schematics/ng-add/schema.ts

@@ -1,10 +1,4 @@
 // eslint-disable-next-line @typescript-eslint/no-empty-interface
 export interface Schema {
-  config:
-    | 'recommended'
-    | 'strict'
-    | 'store'
-    | 'effects'
-    | 'component-store'
-    | 'all';
+  config: 'all' | 'store' | 'effects' | 'component-store' | 'signals';
 }

modules/eslint-plugin/scripts/generate-config.ts

@@ -9,66 +9,24 @@ const prettierConfig = resolveConfig.sync(__dirname);
 const RULE_MODULE = '@ngrx';
 const CONFIG_DIRECTORY = './modules/eslint-plugin/src/configs/';
 
-writeConfig('recommended', (rule) => !rule.meta.docs?.requiresTypeChecking);
 writeConfig('all', (_rule) => true);
-
-writeConfig(
-  'store-recommended',
-  (rule) =>
-    rule.meta.ngrxModule === 'store' && !rule.meta.docs?.requiresTypeChecking
-);
-writeConfig('store-all', (rule) => rule.meta.ngrxModule === 'store');
-
-writeConfig(
-  'effects-recommended',
-  (rule) =>
-    rule.meta.ngrxModule === 'effects' && !rule.meta.docs?.requiresTypeChecking
-);
-writeConfig('effects-all', (rule) => rule.meta.ngrxModule === 'effects');
-
-writeConfig(
-  'component-store-recommended',
-  (rule) =>
-    rule.meta.ngrxModule === 'component-store' &&
-    !rule.meta.docs?.requiresTypeChecking
-);
-
+writeConfig('store', (rule) => rule.meta.ngrxModule === 'store');
+writeConfig('effects', (rule) => rule.meta.ngrxModule === 'effects');
 writeConfig(
-  'component-store-all',
+  'component-store',
   (rule) => rule.meta.ngrxModule === 'component-store'
 );
-
-writeConfig(
-  'operators-recommended',
-  (rule) =>
-    rule.meta.ngrxModule === 'operators' &&
-    !rule.meta.docs?.requiresTypeChecking
-);
-
-writeConfig('operators-all', (rule) => rule.meta.ngrxModule === 'operators');
-
-writeConfig(
-  'signals-recommended',
-  (rule) =>
-    rule.meta.ngrxModule === 'signals' && !rule.meta.docs?.requiresTypeChecking
-);
-
-writeConfig('signals-all', (rule) => rule.meta.ngrxModule === 'signals');
+writeConfig('operators', (rule) => rule.meta.ngrxModule === 'operators');
+writeConfig('signals', (rule) => rule.meta.ngrxModule === 'signals');
 
 function writeConfig(
   configName:
     | 'all'
-    | 'recommended'
-    | 'store-recommended'
-    | 'store-all'
-    | 'effects-recommended'
-    | 'effects-all'
-    | 'component-store-recommended'
-    | 'component-store-all'
-    | 'operators-recommended'
-    | 'operators-all'
-    | 'signals-recommended'
-    | 'signals-all',
+    | 'store'
+    | 'effects'
+    | 'component-store'
+    | 'operators'
+    | 'signals',
   predicate: (rule: NgRxRuleModule<[], string>) => boolean
 ) {
   const rulesForConfig = Object.entries(rulesForGenerate).filter(([_, rule]) =>

modules/eslint-plugin/spec/schematics/ng-add.spec.ts

@@ -10,7 +10,7 @@ const schematicRunner = new SchematicTestRunner(
   path.join(__dirname, '../../schematics/collection.json')
 );
 
-test('registers the plugin with the recommended config', async () => {
+test('registers the plugin with the all config', async () => {
   const appTree = new UnitTestTree(Tree.empty());
 
   const initialConfig = {};
@@ -21,7 +21,7 @@ test('registers the plugin with the recommended config', async () => {
   const eslintContent = appTree.readContent(`.eslintrc.json`);
   const eslintJson = JSON.parse(eslintContent);
   expect(eslintJson).toEqual({
-    overrides: [{ files: ['*.ts'], extends: [`plugin:@ngrx/recommended`] }],
+    overrides: [{ files: ['*.ts'], extends: [`plugin:@ngrx/all`] }],
   });
 });
 
@@ -31,7 +31,7 @@ test('registers the plugin with a different config', async () => {
   const initialConfig = {};
   appTree.create('./.eslintrc.json', JSON.stringify(initialConfig, null, 2));
 
-  const options = { config: 'recommended' };
+  const options = { config: 'store' };
   await schematicRunner.runSchematic('ng-add', options, appTree);
 
   const eslintContent = appTree.readContent(`.eslintrc.json`);
@@ -110,7 +110,7 @@ test('registers the plugin in overrides when it supports TS', async () => {
       },
       {
         files: ['*.ts'],
-        extends: [`plugin:@ngrx/recommended`],
+        extends: [`plugin:@ngrx/all`],
       },
     ],
   });
@@ -120,7 +120,7 @@ test('does not add the plugin if it is already added manually', async () => {
   const appTree = new UnitTestTree(Tree.empty());
 
   const initialConfig = {
-    extends: ['plugin:@ngrx/recommended'],
+    extends: ['plugin:@ngrx/all'],
   };
   appTree.create('.eslintrc.json', JSON.stringify(initialConfig, null, 2));
 
@@ -137,7 +137,7 @@ test('does not add the plugin if it is already added manually as an override', a
   const initialConfig = {
     overrides: [
       {
-        extends: ['plugin:@ngrx/recommended'],
+        extends: ['plugin:@ngrx/all'],
       },
     ],
   };

modules/eslint-plugin/src/configs/component-store-all.ts → modules/eslint-plugin/src/configs/component-store.ts

@@ -20,7 +20,7 @@ export default (
     },
   },
   {
-    name: 'ngrx/component-store-all',
+    name: 'ngrx/component-store',
     languageOptions: {
       parser,
     },

modules/eslint-plugin/src/configs/effects-all.json → modules/eslint-plugin/src/configs/effects.json

@@ -9,10 +9,5 @@
     "@ngrx/prefer-action-creator-in-of-type": "error",
     "@ngrx/prefer-effect-callback-in-block-statement": "error",
     "@ngrx/use-effects-lifecycle-interface": "error"
-  },
-  "parserOptions": {
-    "ecmaVersion": 2020,
-    "sourceType": "module",
-    "project": "./tsconfig.json"
   }
 }

modules/eslint-plugin/src/configs/effects-all.ts → modules/eslint-plugin/src/configs/effects.ts

@@ -20,7 +20,7 @@ export default (
     },
   },
   {
-    name: 'ngrx/effects-all',
+    name: 'ngrx/effects',
     languageOptions: {
       parser,
       parserOptions: {