@ngrx/eslint-plugin 18.0.0-beta.1 → 18.0.0-rc.0

package/src/rules/store/good-action-hygiene.js

@@ -22,28 +22,11 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var __read = (this && this.__read) || function (o, n) {
-    var m = typeof Symbol === "function" && o[Symbol.iterator];
-    if (!m) return o;
-    var i = m.call(o), r, ar = [], e;
-    try {
-        while ((n === void 0 || n-- > 0) && !(r = i.next()).done) ar.push(r.value);
-    }
-    catch (error) { e = { error: error }; }
-    finally {
-        try {
-            if (r && !r.done && (m = i["return"])) m.call(i);
-        }
-        finally { if (e) throw e.error; }
-    }
-    return ar;
-};
-var _a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.messageId = void 0;
-var path = __importStar(require("path"));
-var rule_creator_1 = require("../../rule-creator");
-var utils_1 = require("../../utils");
+const path = __importStar(require("path"));
+const rule_creator_1 = require("../../rule-creator");
+const utils_1 = require("../../utils");
 exports.messageId = 'goodActionHygiene';
 exports.default = (0, rule_creator_1.createRule)({
     name: path.parse(__filename).name,
@@ -52,33 +35,30 @@ exports.default = (0, rule_creator_1.createRule)({
         ngrxModule: 'store',
         docs: {
             description: 'Ensures the use of good action hygiene.',
-            recommended: 'warn',
         },
         schema: [],
-        messages: (_a = {},
-            _a[exports.messageId] = 'Action type `{{ actionType }}` does not follow the good action hygiene practice, use "[Source] {{ actionType }}" to define action types.',
-            _a),
+        messages: {
+            [exports.messageId]: 'Action type `{{ actionType }}` does not follow the good action hygiene practice, use "[Source] {{ actionType }}" to define action types.',
+        },
     },
     defaultOptions: [],
-    create: function (context) {
-        var _a;
-        var sourceEventPattern = /[[].*[\]]\s.*/;
-        return _a = {},
-            _a[utils_1.actionCreatorWithLiteral] = function (_a) {
-                var _b = __read(_a.arguments, 1), node = _b[0];
-                var actionType = node.value;
+    create: (context) => {
+        const sourceEventPattern = /[[].*[\]]\s.*/;
+        return {
+            [utils_1.actionCreatorWithLiteral]({ arguments: [node], }) {
+                const { value: actionType } = node;
                 if (sourceEventPattern.test(actionType)) {
                     return;
                 }
                 context.report({
-                    node: node,
+                    node,
                     messageId: exports.messageId,
                     data: {
-                        actionType: actionType,
+                        actionType,
                     },
                 });
             },
-            _a;
+        };
     },
 });
 //# sourceMappingURL=good-action-hygiene.js.map

package/src/rules/store/good-action-hygiene.js.map

@@ -1 +1 @@
-{"version":3,"file":"good-action-hygiene.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/good-action-hygiene.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAAuD;AAE1C,QAAA,SAAS,GAAG,mBAAmB,CAAC;AAK7C,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,yCAAyC;YACtD,WAAW,EAAE,MAAM;SACpB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IACR,0IAA0I;eAC7I;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACd,IAAM,kBAAkB,GAAG,eAAe,CAAC;QAE3C;YACE,GAAC,gCAAwB,IAAzB,UAA2B,EAI1B;oBAHC,KAAA,uBAAiB,EAAL,IAAI,QAAA;gBAIR,IAAO,UAAU,GAAK,IAAI,MAAT,CAAU;gBAEnC,IAAI,kBAAkB,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;oBACxC,OAAO;gBACT,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI,MAAA;oBACJ,SAAS,mBAAA;oBACT,IAAI,EAAE;wBACJ,UAAU,YAAA;qBACX;iBACF,CAAC,CAAC;YACL,CAAC;eACD;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/experimental-utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport { actionCreatorWithLiteral } from '../../utils';\n\nexport const messageId = 'goodActionHygiene';\n\ntype MessageIds = typeof messageId;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    ngrxModule: 'store',\n    docs: {\n      description: 'Ensures the use of good action hygiene.',\n      recommended: 'warn',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        'Action type `{{ actionType }}` does not follow the good action hygiene practice, use \"[Source] {{ actionType }}\" to define action types.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const sourceEventPattern = /[[].*[\\]]\\s.*/;\n\n    return {\n      [actionCreatorWithLiteral]({\n        arguments: [node],\n      }: Omit<TSESTree.CallExpression, 'arguments'> & {\n        arguments: TSESTree.StringLiteral[];\n      }) {\n        const { value: actionType } = node;\n\n        if (sourceEventPattern.test(actionType)) {\n          return;\n        }\n\n        context.report({\n          node,\n          messageId,\n          data: {\n            actionType,\n          },\n        });\n      },\n    };\n  },\n});\n"]}
+{"version":3,"file":"good-action-hygiene.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/good-action-hygiene.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;AACA,2CAA6B;AAC7B,qDAAgD;AAChD,uCAAuD;AAE1C,QAAA,SAAS,GAAG,mBAAmB,CAAC;AAK7C,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,yCAAyC;SACvD;QACD,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE;YACR,CAAC,iBAAS,CAAC,EACT,0IAA0I;SAC7I;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE;QAClB,MAAM,kBAAkB,GAAG,eAAe,CAAC;QAE3C,OAAO;YACL,CAAC,gCAAwB,CAAC,CAAC,EACzB,SAAS,EAAE,CAAC,IAAI,CAAC,GAGlB;gBACC,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,GAAG,IAAI,CAAC;gBAEnC,IAAI,kBAAkB,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;oBACxC,OAAO;gBACT,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAT,iBAAS;oBACT,IAAI,EAAE;wBACJ,UAAU;qBACX;iBACF,CAAC,CAAC;YACL,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport { actionCreatorWithLiteral } from '../../utils';\n\nexport const messageId = 'goodActionHygiene';\n\ntype MessageIds = typeof messageId;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    ngrxModule: 'store',\n    docs: {\n      description: 'Ensures the use of good action hygiene.',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        'Action type `{{ actionType }}` does not follow the good action hygiene practice, use \"[Source] {{ actionType }}\" to define action types.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const sourceEventPattern = /[[].*[\\]]\\s.*/;\n\n    return {\n      [actionCreatorWithLiteral]({\n        arguments: [node],\n      }: Omit<TSESTree.CallExpression, 'arguments'> & {\n        arguments: TSESTree.StringLiteral[];\n      }) {\n        const { value: actionType } = node;\n\n        if (sourceEventPattern.test(actionType)) {\n          return;\n        }\n\n        context.report({\n          node,\n          messageId,\n          data: {\n            actionType,\n          },\n        });\n      },\n    };\n  },\n});\n"]}

package/src/rules/store/no-multiple-global-stores.d.ts

@@ -0,0 +1,6 @@
+import type { TSESLint } from '@typescript-eslint/utils';
+export declare const noMultipleGlobalStores = "noMultipleGlobalStores";
+export declare const noMultipleGlobalStoresSuggest = "noMultipleGlobalStoresSuggest";
+type MessageIds = typeof noMultipleGlobalStores | typeof noMultipleGlobalStoresSuggest;
+declare const _default: TSESLint.RuleModule<MessageIds, readonly [], TSESLint.RuleListener>;
+export default _default;

package/src/rules/store/no-multiple-global-stores.js

@@ -22,48 +22,11 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var __values = (this && this.__values) || function(o) {
-    var s = typeof Symbol === "function" && Symbol.iterator, m = s && o[s], i = 0;
-    if (m) return m.call(o);
-    if (o && typeof o.length === "number") return {
-        next: function () {
-            if (o && i >= o.length) o = void 0;
-            return { value: o && o[i++], done: !o };
-        }
-    };
-    throw new TypeError(s ? "Object is not iterable." : "Symbol.iterator is not defined.");
-};
-var __read = (this && this.__read) || function (o, n) {
-    var m = typeof Symbol === "function" && o[Symbol.iterator];
-    if (!m) return o;
-    var i = m.call(o), r, ar = [], e;
-    try {
-        while ((n === void 0 || n-- > 0) && !(r = i.next()).done) ar.push(r.value);
-    }
-    catch (error) { e = { error: error }; }
-    finally {
-        try {
-            if (r && !r.done && (m = i["return"])) m.call(i);
-        }
-        finally { if (e) throw e.error; }
-    }
-    return ar;
-};
-var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
-    if (pack || arguments.length === 2) for (var i = 0, l = from.length, ar; i < l; i++) {
-        if (ar || !(i in from)) {
-            if (!ar) ar = Array.prototype.slice.call(from, 0, i);
-            ar[i] = from[i];
-        }
-    }
-    return to.concat(ar || Array.prototype.slice.call(from));
-};
-var _a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noMultipleGlobalStoresSuggest = exports.noMultipleGlobalStores = void 0;
-var path = __importStar(require("path"));
-var rule_creator_1 = require("../../rule-creator");
-var utils_1 = require("../../utils");
+const path = __importStar(require("path"));
+const rule_creator_1 = require("../../rule-creator");
+const utils_1 = require("../../utils");
 exports.noMultipleGlobalStores = 'noMultipleGlobalStores';
 exports.noMultipleGlobalStoresSuggest = 'noMultipleGlobalStoresSuggest';
 exports.default = (0, rule_creator_1.createRule)({
@@ -74,62 +37,36 @@ exports.default = (0, rule_creator_1.createRule)({
         ngrxModule: 'store',
         docs: {
             description: 'There should only be one global store injected.',
-            recommended: 'warn',
-            suggestion: true,
         },
         schema: [],
-        messages: (_a = {},
-            _a[exports.noMultipleGlobalStores] = 'Global store should be injected only once.',
-            _a[exports.noMultipleGlobalStoresSuggest] = 'Remove this reference.',
-            _a),
+        messages: {
+            [exports.noMultipleGlobalStores]: 'Global store should be injected only once.',
+            [exports.noMultipleGlobalStoresSuggest]: 'Remove this reference.',
+        },
     },
     defaultOptions: [],
-    create: function (context) {
+    create: (context) => {
         return {
-            Program: function () {
-                var e_1, _a, e_2, _b;
-                var _c = (0, utils_1.getNgRxStores)(context), _d = _c.identifiers, identifiers = _d === void 0 ? [] : _d, sourceCode = _c.sourceCode;
-                var flattenedIdentifiers = groupBy(identifiers).values();
-                try {
-                    for (var flattenedIdentifiers_1 = __values(flattenedIdentifiers), flattenedIdentifiers_1_1 = flattenedIdentifiers_1.next(); !flattenedIdentifiers_1_1.done; flattenedIdentifiers_1_1 = flattenedIdentifiers_1.next()) {
-                        var identifiers_2 = flattenedIdentifiers_1_1.value;
-                        if (identifiers_2.length <= 1) {
-                            continue;
-                        }
-                        var _loop_1 = function (node) {
-                            var nodeToReport = getNodeToReport(node);
-                            context.report({
-                                node: nodeToReport,
-                                messageId: exports.noMultipleGlobalStores,
-                                suggest: [
-                                    {
-                                        messageId: exports.noMultipleGlobalStoresSuggest,
-                                        fix: function (fixer) { return getFixes(sourceCode, fixer, nodeToReport); },
-                                    },
-                                ],
-                            });
-                        };
-                        try {
-                            for (var identifiers_1 = (e_2 = void 0, __values(identifiers_2)), identifiers_1_1 = identifiers_1.next(); !identifiers_1_1.done; identifiers_1_1 = identifiers_1.next()) {
-                                var node = identifiers_1_1.value;
-                                _loop_1(node);
-                            }
-                        }
-                        catch (e_2_1) { e_2 = { error: e_2_1 }; }
-                        finally {
-                            try {
-                                if (identifiers_1_1 && !identifiers_1_1.done && (_b = identifiers_1.return)) _b.call(identifiers_1);
-                            }
-                            finally { if (e_2) throw e_2.error; }
-                        }
+            Program() {
+                const { identifiers = [], sourceCode } = (0, utils_1.getNgRxStores)(context);
+                const flattenedIdentifiers = groupBy(identifiers).values();
+                for (const identifiers of flattenedIdentifiers) {
+                    if (identifiers.length <= 1) {
+                        continue;
                     }
-                }
-                catch (e_1_1) { e_1 = { error: e_1_1 }; }
-                finally {
-                    try {
-                        if (flattenedIdentifiers_1_1 && !flattenedIdentifiers_1_1.done && (_a = flattenedIdentifiers_1.return)) _a.call(flattenedIdentifiers_1);
+                    for (const node of identifiers) {
+                        const nodeToReport = getNodeToReport(node);
+                        context.report({
+                            node: nodeToReport,
+                            messageId: exports.noMultipleGlobalStores,
+                            suggest: [
+                                {
+                                    messageId: exports.noMultipleGlobalStoresSuggest,
+                                    fix: (fixer) => getFixes(sourceCode, fixer, nodeToReport),
+                                },
+                            ],
+                        });
                     }
-                    finally { if (e_1) throw e_1.error; }
                 }
             },
         };
@@ -139,19 +76,20 @@ function getNodeToReport(node) {
     return node.parent && (0, utils_1.isTSParameterProperty)(node.parent) ? node.parent : node;
 }
 function getFixes(sourceCode, fixer, node) {
-    var parent = node.parent;
-    var nodeToRemove = parent && (0, utils_1.isTSParameterProperty)(parent) ? parent : node;
+    const { parent } = node;
+    const nodeToRemove = parent && (0, utils_1.isTSParameterProperty)(parent) ? parent : node;
     return (0, utils_1.getNodeToCommaRemoveFix)(sourceCode, fixer, nodeToRemove);
 }
 function groupBy(identifiers) {
-    return identifiers.reduce(function (accumulator, identifier) {
-        var parent = (0, utils_1.isTSParameterProperty)(identifier.parent)
+    return identifiers.reduce((accumulator, identifier) => {
+        const parent = (0, utils_1.isTSParameterProperty)(identifier.parent)
             ? identifier.parent.parent
             : identifier.parent;
-        var collectedIdentifiers = accumulator.get(parent);
-        return accumulator.set(parent, __spreadArray(__spreadArray([], __read((collectedIdentifiers !== null && collectedIdentifiers !== void 0 ? collectedIdentifiers : [])), false), [
+        const collectedIdentifiers = accumulator.get(parent);
+        return accumulator.set(parent, [
+            ...(collectedIdentifiers ?? []),
             identifier,
-        ], false));
+        ]);
     }, new Map());
 }
 //# sourceMappingURL=no-multiple-global-stores.js.map

package/src/rules/store/no-multiple-global-stores.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-multiple-global-stores.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-multiple-global-stores.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAIqB;AAER,QAAA,sBAAsB,GAAG,wBAAwB,CAAC;AAClD,QAAA,6BAA6B,GAAG,+BAA+B,CAAC;AAO7E,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,cAAc,EAAE,IAAI;QACpB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,iDAAiD;YAC9D,WAAW,EAAE,MAAM;YACnB,UAAU,EAAE,IAAI;SACjB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,8BAAsB,IAAG,4CAA4C;YACtE,GAAC,qCAA6B,IAAG,wBAAwB;eAC1D;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;QACd,OAAO;YACL,OAAO;;gBACC,IAAA,KAAmC,IAAA,qBAAa,EAAC,OAAO,CAAC,EAAvD,mBAAgB,EAAhB,WAAW,mBAAG,EAAE,KAAA,EAAE,UAAU,gBAA2B,CAAC;gBAChE,IAAM,oBAAoB,GAAG,OAAO,CAAC,WAAW,CAAC,CAAC,MAAM,EAAE,CAAC;;oBAE3D,KAA0B,IAAA,yBAAA,SAAA,oBAAoB,CAAA,0DAAA,4FAAE,CAAC;wBAA5C,IAAM,aAAW,iCAAA;wBACpB,IAAI,aAAW,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;4BAC5B,SAAS;wBACX,CAAC;gDAEU,IAAI;4BACb,IAAM,YAAY,GAAG,eAAe,CAAC,IAAI,CAAC,CAAC;4BAC3C,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,EAAE,YAAY;gCAClB,SAAS,EAAE,8BAAsB;gCACjC,OAAO,EAAE;oCACP;wCACE,SAAS,EAAE,qCAA6B;wCACxC,GAAG,EAAE,UAAC,KAAK,IAAK,OAAA,QAAQ,CAAC,UAAU,EAAE,KAAK,EAAE,YAAY,CAAC,EAAzC,CAAyC;qCAC1D;iCACF;6BACF,CAAC,CAAC;;;4BAXL,KAAmB,IAAA,+BAAA,SAAA,aAAW,CAAA,CAAA,wCAAA;gCAAzB,IAAM,IAAI,wBAAA;wCAAJ,IAAI;6BAYd;;;;;;;;;oBACH,CAAC;;;;;;;;;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAEH,SAAS,eAAe,CAAC,IAAmB;IAC1C,OAAO,IAAI,CAAC,MAAM,IAAI,IAAA,6BAAqB,EAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC;AAChF,CAAC;AAED,SAAS,QAAQ,CACf,UAAyC,EACzC,KAAyB,EACzB,IAAmB;IAEX,IAAA,MAAM,GAAK,IAAI,OAAT,CAAU;IACxB,IAAM,YAAY,GAAG,MAAM,IAAI,IAAA,6BAAqB,EAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC;IAC7E,OAAO,IAAA,+BAAuB,EAAC,UAAU,EAAE,KAAK,EAAE,YAAY,CAAC,CAAC;AAClE,CAAC;AAID,SAAS,OAAO,CAAC,WAAwB;IACvC,OAAO,WAAW,CAAC,MAAM,CACvB,UAAC,WAAW,EAAE,UAAU;QACtB,IAAM,MAAM,GAAG,IAAA,6BAAqB,EAAC,UAAU,CAAC,MAAM,CAAC;YACrD,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,MAAM;YAC1B,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC;QACtB,IAAM,oBAAoB,GAAG,WAAW,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;QACrD,OAAO,WAAW,CAAC,GAAG,CAAC,MAAM,yCACxB,CAAC,oBAAoB,aAApB,oBAAoB,cAApB,oBAAoB,GAAI,EAAE,CAAC;YAC/B,UAAU;kBACV,CAAC;IACL,CAAC,EACD,IAAI,GAAG,EAAE,CACV,CAAC;AACJ,CAAC","sourcesContent":["import type { TSESLint, TSESTree } from '@typescript-eslint/experimental-utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport {\n  getNgRxStores,\n  getNodeToCommaRemoveFix,\n  isTSParameterProperty,\n} from '../../utils';\n\nexport const noMultipleGlobalStores = 'noMultipleGlobalStores';\nexport const noMultipleGlobalStoresSuggest = 'noMultipleGlobalStoresSuggest';\n\ntype MessageIds =\n  | typeof noMultipleGlobalStores\n  | typeof noMultipleGlobalStoresSuggest;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    hasSuggestions: true,\n    ngrxModule: 'store',\n    docs: {\n      description: 'There should only be one global store injected.',\n      recommended: 'warn',\n      suggestion: true,\n    },\n    schema: [],\n    messages: {\n      [noMultipleGlobalStores]: 'Global store should be injected only once.',\n      [noMultipleGlobalStoresSuggest]: 'Remove this reference.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    return {\n      Program() {\n        const { identifiers = [], sourceCode } = getNgRxStores(context);\n        const flattenedIdentifiers = groupBy(identifiers).values();\n\n        for (const identifiers of flattenedIdentifiers) {\n          if (identifiers.length <= 1) {\n            continue;\n          }\n\n          for (const node of identifiers) {\n            const nodeToReport = getNodeToReport(node);\n            context.report({\n              node: nodeToReport,\n              messageId: noMultipleGlobalStores,\n              suggest: [\n                {\n                  messageId: noMultipleGlobalStoresSuggest,\n                  fix: (fixer) => getFixes(sourceCode, fixer, nodeToReport),\n                },\n              ],\n            });\n          }\n        }\n      },\n    };\n  },\n});\n\nfunction getNodeToReport(node: TSESTree.Node) {\n  return node.parent && isTSParameterProperty(node.parent) ? node.parent : node;\n}\n\nfunction getFixes(\n  sourceCode: Readonly<TSESLint.SourceCode>,\n  fixer: TSESLint.RuleFixer,\n  node: TSESTree.Node\n) {\n  const { parent } = node;\n  const nodeToRemove = parent && isTSParameterProperty(parent) ? parent : node;\n  return getNodeToCommaRemoveFix(sourceCode, fixer, nodeToRemove);\n}\n\ntype Identifiers = NonNullable<ReturnType<typeof getNgRxStores>['identifiers']>;\n\nfunction groupBy(identifiers: Identifiers): Map<TSESTree.Node, Identifiers> {\n  return identifiers.reduce<Map<TSESTree.Node, Identifiers>>(\n    (accumulator, identifier) => {\n      const parent = isTSParameterProperty(identifier.parent)\n        ? identifier.parent.parent\n        : identifier.parent;\n      const collectedIdentifiers = accumulator.get(parent);\n      return accumulator.set(parent, [\n        ...(collectedIdentifiers ?? []),\n        identifier,\n      ]);\n    },\n    new Map()\n  );\n}\n"]}
+{"version":3,"file":"no-multiple-global-stores.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-multiple-global-stores.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;AACA,2CAA6B;AAC7B,qDAAgD;AAChD,uCAIqB;AAER,QAAA,sBAAsB,GAAG,wBAAwB,CAAC;AAClD,QAAA,6BAA6B,GAAG,+BAA+B,CAAC;AAO7E,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,cAAc,EAAE,IAAI;QACpB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,iDAAiD;SAC/D;QACD,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE;YACR,CAAC,8BAAsB,CAAC,EAAE,4CAA4C;YACtE,CAAC,qCAA6B,CAAC,EAAE,wBAAwB;SAC1D;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE;QAClB,OAAO;YACL,OAAO;gBACL,MAAM,EAAE,WAAW,GAAG,EAAE,EAAE,UAAU,EAAE,GAAG,IAAA,qBAAa,EAAC,OAAO,CAAC,CAAC;gBAChE,MAAM,oBAAoB,GAAG,OAAO,CAAC,WAAW,CAAC,CAAC,MAAM,EAAE,CAAC;gBAE3D,KAAK,MAAM,WAAW,IAAI,oBAAoB,EAAE,CAAC;oBAC/C,IAAI,WAAW,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;wBAC5B,SAAS;oBACX,CAAC;oBAED,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;wBAC/B,MAAM,YAAY,GAAG,eAAe,CAAC,IAAI,CAAC,CAAC;wBAC3C,OAAO,CAAC,MAAM,CAAC;4BACb,IAAI,EAAE,YAAY;4BAClB,SAAS,EAAE,8BAAsB;4BACjC,OAAO,EAAE;gCACP;oCACE,SAAS,EAAE,qCAA6B;oCACxC,GAAG,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,QAAQ,CAAC,UAAU,EAAE,KAAK,EAAE,YAAY,CAAC;iCAC1D;6BACF;yBACF,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAEH,SAAS,eAAe,CAAC,IAAmB;IAC1C,OAAO,IAAI,CAAC,MAAM,IAAI,IAAA,6BAAqB,EAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC;AAChF,CAAC;AAED,SAAS,QAAQ,CACf,UAAyC,EACzC,KAAyB,EACzB,IAAmB;IAEnB,MAAM,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC;IACxB,MAAM,YAAY,GAAG,MAAM,IAAI,IAAA,6BAAqB,EAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC;IAC7E,OAAO,IAAA,+BAAuB,EAAC,UAAU,EAAE,KAAK,EAAE,YAAY,CAAC,CAAC;AAClE,CAAC;AAID,SAAS,OAAO,CAAC,WAAwB;IACvC,OAAO,WAAW,CAAC,MAAM,CACvB,CAAC,WAAW,EAAE,UAAU,EAAE,EAAE;QAC1B,MAAM,MAAM,GAAG,IAAA,6BAAqB,EAAC,UAAU,CAAC,MAAM,CAAC;YACrD,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,MAAM;YAC1B,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC;QACtB,MAAM,oBAAoB,GAAG,WAAW,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;QACrD,OAAO,WAAW,CAAC,GAAG,CAAC,MAAM,EAAE;YAC7B,GAAG,CAAC,oBAAoB,IAAI,EAAE,CAAC;YAC/B,UAAU;SACX,CAAC,CAAC;IACL,CAAC,EACD,IAAI,GAAG,EAAE,CACV,CAAC;AACJ,CAAC","sourcesContent":["import type { TSESLint, TSESTree } from '@typescript-eslint/utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport {\n  getNgRxStores,\n  getNodeToCommaRemoveFix,\n  isTSParameterProperty,\n} from '../../utils';\n\nexport const noMultipleGlobalStores = 'noMultipleGlobalStores';\nexport const noMultipleGlobalStoresSuggest = 'noMultipleGlobalStoresSuggest';\n\ntype MessageIds =\n  | typeof noMultipleGlobalStores\n  | typeof noMultipleGlobalStoresSuggest;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    hasSuggestions: true,\n    ngrxModule: 'store',\n    docs: {\n      description: 'There should only be one global store injected.',\n    },\n    schema: [],\n    messages: {\n      [noMultipleGlobalStores]: 'Global store should be injected only once.',\n      [noMultipleGlobalStoresSuggest]: 'Remove this reference.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    return {\n      Program() {\n        const { identifiers = [], sourceCode } = getNgRxStores(context);\n        const flattenedIdentifiers = groupBy(identifiers).values();\n\n        for (const identifiers of flattenedIdentifiers) {\n          if (identifiers.length <= 1) {\n            continue;\n          }\n\n          for (const node of identifiers) {\n            const nodeToReport = getNodeToReport(node);\n            context.report({\n              node: nodeToReport,\n              messageId: noMultipleGlobalStores,\n              suggest: [\n                {\n                  messageId: noMultipleGlobalStoresSuggest,\n                  fix: (fixer) => getFixes(sourceCode, fixer, nodeToReport),\n                },\n              ],\n            });\n          }\n        }\n      },\n    };\n  },\n});\n\nfunction getNodeToReport(node: TSESTree.Node) {\n  return node.parent && isTSParameterProperty(node.parent) ? node.parent : node;\n}\n\nfunction getFixes(\n  sourceCode: Readonly<TSESLint.SourceCode>,\n  fixer: TSESLint.RuleFixer,\n  node: TSESTree.Node\n) {\n  const { parent } = node;\n  const nodeToRemove = parent && isTSParameterProperty(parent) ? parent : node;\n  return getNodeToCommaRemoveFix(sourceCode, fixer, nodeToRemove);\n}\n\ntype Identifiers = NonNullable<ReturnType<typeof getNgRxStores>['identifiers']>;\n\nfunction groupBy(identifiers: Identifiers): Map<TSESTree.Node, Identifiers> {\n  return identifiers.reduce<Map<TSESTree.Node, Identifiers>>(\n    (accumulator, identifier) => {\n      const parent = isTSParameterProperty(identifier.parent)\n        ? identifier.parent.parent\n        : identifier.parent;\n      const collectedIdentifiers = accumulator.get(parent);\n      return accumulator.set(parent, [\n        ...(collectedIdentifiers ?? []),\n        identifier,\n      ]);\n    },\n    new Map()\n  );\n}\n"]}

package/src/rules/store/no-reducer-in-key-names.d.ts

@@ -0,0 +1,5 @@
+export declare const noReducerInKeyNames = "noReducerInKeyNames";
+export declare const noReducerInKeyNamesSuggest = "noReducerInKeyNamesSuggest";
+type MessageIds = typeof noReducerInKeyNames | typeof noReducerInKeyNamesSuggest;
+declare const _default: import("@typescript-eslint/utils/ts-eslint").RuleModule<MessageIds, readonly [], import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+export default _default;

package/src/rules/store/no-reducer-in-key-names.js

@@ -22,15 +22,14 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var _a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noReducerInKeyNamesSuggest = exports.noReducerInKeyNames = void 0;
-var path = __importStar(require("path"));
-var rule_creator_1 = require("../../rule-creator");
-var utils_1 = require("../../utils");
+const path = __importStar(require("path"));
+const rule_creator_1 = require("../../rule-creator");
+const utils_1 = require("../../utils");
 exports.noReducerInKeyNames = 'noReducerInKeyNames';
 exports.noReducerInKeyNamesSuggest = 'noReducerInKeyNamesSuggest';
-var reducerKeyword = 'reducer';
+const reducerKeyword = 'reducer';
 exports.default = (0, rule_creator_1.createRule)({
     name: path.parse(__filename).name,
     meta: {
@@ -38,29 +37,26 @@ exports.default = (0, rule_creator_1.createRule)({
         hasSuggestions: true,
         ngrxModule: 'store',
         docs: {
-            description: "Avoid the word \"".concat(reducerKeyword, "\" in the key names."),
-            recommended: 'warn',
-            suggestion: true,
+            description: `Avoid the word "${reducerKeyword}" in the key names.`,
         },
         schema: [],
-        messages: (_a = {},
-            _a[exports.noReducerInKeyNames] = "Avoid the word \"".concat(reducerKeyword, "\" in the key names to better represent the state."),
-            _a[exports.noReducerInKeyNamesSuggest] = "Remove the word \"".concat(reducerKeyword, "\"."),
-            _a),
+        messages: {
+            [exports.noReducerInKeyNames]: `Avoid the word "${reducerKeyword}" in the key names to better represent the state.`,
+            [exports.noReducerInKeyNamesSuggest]: `Remove the word "${reducerKeyword}".`,
+        },
     },
     defaultOptions: [],
-    create: function (context) {
-        var _a;
-        return _a = {},
-            _a[":matches(".concat(utils_1.storeActionReducerMap, ", ").concat(utils_1.actionReducerMap, ") > ").concat((0, utils_1.metadataProperty)(/reducer/i), " > .key")] = function (node) {
+    create: (context) => {
+        return {
+            [`:matches(${utils_1.storeActionReducerMap}, ${utils_1.actionReducerMap}) > ${(0, utils_1.metadataProperty)(/reducer/i)} > .key`](node) {
                 context.report({
-                    node: node,
+                    node,
                     messageId: exports.noReducerInKeyNames,
                     suggest: [
                         {
                             messageId: exports.noReducerInKeyNamesSuggest,
-                            fix: function (fixer) {
-                                var keyName = (0, utils_1.getRawText)(node);
+                            fix: (fixer) => {
+                                const keyName = (0, utils_1.getRawText)(node);
                                 if (!keyName) {
                                     return null;
                                 }
@@ -70,7 +66,7 @@ exports.default = (0, rule_creator_1.createRule)({
                     ],
                 });
             },
-            _a;
+        };
     },
 });
 //# sourceMappingURL=no-reducer-in-key-names.js.map

package/src/rules/store/no-reducer-in-key-names.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-reducer-in-key-names.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-reducer-in-key-names.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAKqB;AAER,QAAA,mBAAmB,GAAG,qBAAqB,CAAC;AAC5C,QAAA,0BAA0B,GAAG,4BAA4B,CAAC;AAOvE,IAAM,cAAc,GAAG,SAAS,CAAC;AAEjC,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,cAAc,EAAE,IAAI;QACpB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,2BAAmB,cAAc,yBAAqB;YACnE,WAAW,EAAE,MAAM;YACnB,UAAU,EAAE,IAAI;SACjB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,2BAAmB,IAAG,2BAAmB,cAAc,uDAAmD;YAC3G,GAAC,kCAA0B,IAAG,4BAAoB,cAAc,QAAI;eACrE;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACd;YACE,GAAC,mBAAY,6BAAqB,eAAK,wBAAgB,iBAAO,IAAA,wBAAgB,EAC5E,UAAU,CACX,YAAS,IAFV,UAEY,IAA8B;gBACxC,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI,MAAA;oBACJ,SAAS,EAAE,2BAAmB;oBAC9B,OAAO,EAAE;wBACP;4BACE,SAAS,EAAE,kCAA0B;4BACrC,GAAG,EAAE,UAAC,KAAK;gCACT,IAAM,OAAO,GAAG,IAAA,kBAAU,EAAC,IAAI,CAAC,CAAC;gCAEjC,IAAI,CAAC,OAAO,EAAE,CAAC;oCACb,OAAO,IAAI,CAAC;gCACd,CAAC;gCAED,OAAO,KAAK,CAAC,WAAW,CACtB,IAAI,EACJ,OAAO,CAAC,OAAO,CAAC,IAAI,MAAM,CAAC,cAAc,EAAE,GAAG,CAAC,EAAE,EAAE,CAAC,CACrD,CAAC;4BACJ,CAAC;yBACF;qBACF;iBACF,CAAC,CAAC;YACL,CAAC;eACD;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/experimental-utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport {\n  actionReducerMap,\n  getRawText,\n  metadataProperty,\n  storeActionReducerMap,\n} from '../../utils';\n\nexport const noReducerInKeyNames = 'noReducerInKeyNames';\nexport const noReducerInKeyNamesSuggest = 'noReducerInKeyNamesSuggest';\n\ntype MessageIds =\n  | typeof noReducerInKeyNames\n  | typeof noReducerInKeyNamesSuggest;\ntype Options = readonly [];\n\nconst reducerKeyword = 'reducer';\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    hasSuggestions: true,\n    ngrxModule: 'store',\n    docs: {\n      description: `Avoid the word \"${reducerKeyword}\" in the key names.`,\n      recommended: 'warn',\n      suggestion: true,\n    },\n    schema: [],\n    messages: {\n      [noReducerInKeyNames]: `Avoid the word \"${reducerKeyword}\" in the key names to better represent the state.`,\n      [noReducerInKeyNamesSuggest]: `Remove the word \"${reducerKeyword}\".`,\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    return {\n      [`:matches(${storeActionReducerMap}, ${actionReducerMap}) > ${metadataProperty(\n        /reducer/i\n      )} > .key`](node: TSESTree.Property['key']) {\n        context.report({\n          node,\n          messageId: noReducerInKeyNames,\n          suggest: [\n            {\n              messageId: noReducerInKeyNamesSuggest,\n              fix: (fixer) => {\n                const keyName = getRawText(node);\n\n                if (!keyName) {\n                  return null;\n                }\n\n                return fixer.replaceText(\n                  node,\n                  keyName.replace(new RegExp(reducerKeyword, 'i'), '')\n                );\n              },\n            },\n          ],\n        });\n      },\n    };\n  },\n});\n"]}
+{"version":3,"file":"no-reducer-in-key-names.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-reducer-in-key-names.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;AACA,2CAA6B;AAC7B,qDAAgD;AAChD,uCAKqB;AAER,QAAA,mBAAmB,GAAG,qBAAqB,CAAC;AAC5C,QAAA,0BAA0B,GAAG,4BAA4B,CAAC;AAOvE,MAAM,cAAc,GAAG,SAAS,CAAC;AAEjC,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,cAAc,EAAE,IAAI;QACpB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EAAE,mBAAmB,cAAc,qBAAqB;SACpE;QACD,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE;YACR,CAAC,2BAAmB,CAAC,EAAE,mBAAmB,cAAc,mDAAmD;YAC3G,CAAC,kCAA0B,CAAC,EAAE,oBAAoB,cAAc,IAAI;SACrE;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE;QAClB,OAAO;YACL,CAAC,YAAY,6BAAqB,KAAK,wBAAgB,OAAO,IAAA,wBAAgB,EAC5E,UAAU,CACX,SAAS,CAAC,CAAC,IAA8B;gBACxC,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,2BAAmB;oBAC9B,OAAO,EAAE;wBACP;4BACE,SAAS,EAAE,kCAA0B;4BACrC,GAAG,EAAE,CAAC,KAAK,EAAE,EAAE;gCACb,MAAM,OAAO,GAAG,IAAA,kBAAU,EAAC,IAAI,CAAC,CAAC;gCAEjC,IAAI,CAAC,OAAO,EAAE,CAAC;oCACb,OAAO,IAAI,CAAC;gCACd,CAAC;gCAED,OAAO,KAAK,CAAC,WAAW,CACtB,IAAI,EACJ,OAAO,CAAC,OAAO,CAAC,IAAI,MAAM,CAAC,cAAc,EAAE,GAAG,CAAC,EAAE,EAAE,CAAC,CACrD,CAAC;4BACJ,CAAC;yBACF;qBACF;iBACF,CAAC,CAAC;YACL,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport {\n  actionReducerMap,\n  getRawText,\n  metadataProperty,\n  storeActionReducerMap,\n} from '../../utils';\n\nexport const noReducerInKeyNames = 'noReducerInKeyNames';\nexport const noReducerInKeyNamesSuggest = 'noReducerInKeyNamesSuggest';\n\ntype MessageIds =\n  | typeof noReducerInKeyNames\n  | typeof noReducerInKeyNamesSuggest;\ntype Options = readonly [];\n\nconst reducerKeyword = 'reducer';\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    hasSuggestions: true,\n    ngrxModule: 'store',\n    docs: {\n      description: `Avoid the word \"${reducerKeyword}\" in the key names.`,\n    },\n    schema: [],\n    messages: {\n      [noReducerInKeyNames]: `Avoid the word \"${reducerKeyword}\" in the key names to better represent the state.`,\n      [noReducerInKeyNamesSuggest]: `Remove the word \"${reducerKeyword}\".`,\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    return {\n      [`:matches(${storeActionReducerMap}, ${actionReducerMap}) > ${metadataProperty(\n        /reducer/i\n      )} > .key`](node: TSESTree.Property['key']) {\n        context.report({\n          node,\n          messageId: noReducerInKeyNames,\n          suggest: [\n            {\n              messageId: noReducerInKeyNamesSuggest,\n              fix: (fixer) => {\n                const keyName = getRawText(node);\n\n                if (!keyName) {\n                  return null;\n                }\n\n                return fixer.replaceText(\n                  node,\n                  keyName.replace(new RegExp(reducerKeyword, 'i'), '')\n                );\n              },\n            },\n          ],\n        });\n      },\n    };\n  },\n});\n"]}

package/src/rules/store/no-store-subscription.d.ts

@@ -0,0 +1,3 @@
+export declare const messageId = "noStoreSubscription";
+declare const _default: import("@typescript-eslint/utils/ts-eslint").RuleModule<"noStoreSubscription", readonly [], import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+export default _default;

package/src/rules/store/no-store-subscription.js

@@ -22,12 +22,11 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var _a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.messageId = void 0;
-var path = __importStar(require("path"));
-var rule_creator_1 = require("../../rule-creator");
-var utils_1 = require("../../utils");
+const path = __importStar(require("path"));
+const rule_creator_1 = require("../../rule-creator");
+const utils_1 = require("../../utils");
 exports.messageId = 'noStoreSubscription';
 exports.default = (0, rule_creator_1.createRule)({
     name: path.parse(__filename).name,
@@ -36,29 +35,27 @@ exports.default = (0, rule_creator_1.createRule)({
         ngrxModule: 'store',
         docs: {
             description: 'Using the `async` pipe is preferred over `store` subscription.',
-            recommended: 'warn',
         },
         schema: [],
-        messages: (_a = {},
-            _a[exports.messageId] = '`Store` subscription is forbidden. Use the `async` pipe instead.',
-            _a),
+        messages: {
+            [exports.messageId]: '`Store` subscription is forbidden. Use the `async` pipe instead.',
+        },
     },
     defaultOptions: [],
-    create: function (context) {
-        var _a;
-        var _b = (0, utils_1.getNgRxStores)(context).identifiers, identifiers = _b === void 0 ? [] : _b;
-        var storeNames = identifiers.length > 0 ? (0, utils_1.asPattern)(identifiers) : null;
+    create: (context) => {
+        const { identifiers = [] } = (0, utils_1.getNgRxStores)(context);
+        const storeNames = identifiers.length > 0 ? (0, utils_1.asPattern)(identifiers) : null;
         if (!storeNames) {
             return {};
         }
-        return _a = {},
-            _a["".concat((0, utils_1.namedCallableExpression)(storeNames), " > MemberExpression > Identifier[name='subscribe']")] = function (node) {
+        return {
+            [`${(0, utils_1.namedCallableExpression)(storeNames)} > MemberExpression > Identifier[name='subscribe']`](node) {
                 context.report({
-                    node: node,
+                    node,
                     messageId: exports.messageId,
                 });
             },
-            _a;
+        };
     },
 });
 //# sourceMappingURL=no-store-subscription.js.map

package/src/rules/store/no-store-subscription.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-store-subscription.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-store-subscription.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAAgF;AAEnE,QAAA,SAAS,GAAG,qBAAqB,CAAC;AAK/C,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EACT,gEAAgE;YAClE,WAAW,EAAE,MAAM;SACpB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IACR,kEAAkE;eACrE;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACN,IAAA,KAAqB,IAAA,qBAAa,EAAC,OAAO,CAAC,YAA3B,EAAhB,WAAW,mBAAG,EAAE,KAAA,CAA4B;QACpD,IAAM,UAAU,GAAG,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,IAAA,iBAAS,EAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QAE1E,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAO,EAAE,CAAC;QACZ,CAAC;QAED;YACE,GAAC,UAAG,IAAA,+BAAuB,EACzB,UAAU,CACX,uDAAoD,IAFrD,UAGE,IAAyB;gBAEzB,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI,MAAA;oBACJ,SAAS,mBAAA;iBACV,CAAC,CAAC;YACL,CAAC;eACD;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/experimental-utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport { asPattern, getNgRxStores, namedCallableExpression } from '../../utils';\n\nexport const messageId = 'noStoreSubscription';\n\ntype MessageIds = typeof messageId;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    ngrxModule: 'store',\n    docs: {\n      description:\n        'Using the `async` pipe is preferred over `store` subscription.',\n      recommended: 'warn',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        '`Store` subscription is forbidden. Use the `async` pipe instead.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const { identifiers = [] } = getNgRxStores(context);\n    const storeNames = identifiers.length > 0 ? asPattern(identifiers) : null;\n\n    if (!storeNames) {\n      return {};\n    }\n\n    return {\n      [`${namedCallableExpression(\n        storeNames\n      )} > MemberExpression > Identifier[name='subscribe']`](\n        node: TSESTree.Identifier\n      ) {\n        context.report({\n          node,\n          messageId,\n        });\n      },\n    };\n  },\n});\n"]}
+{"version":3,"file":"no-store-subscription.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/no-store-subscription.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;AACA,2CAA6B;AAC7B,qDAAgD;AAChD,uCAAgF;AAEnE,QAAA,SAAS,GAAG,qBAAqB,CAAC;AAK/C,kBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,OAAO;QACnB,IAAI,EAAE;YACJ,WAAW,EACT,gEAAgE;SACnE;QACD,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE;YACR,CAAC,iBAAS,CAAC,EACT,kEAAkE;SACrE;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE;QAClB,MAAM,EAAE,WAAW,GAAG,EAAE,EAAE,GAAG,IAAA,qBAAa,EAAC,OAAO,CAAC,CAAC;QACpD,MAAM,UAAU,GAAG,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,IAAA,iBAAS,EAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QAE1E,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,OAAO;YACL,CAAC,GAAG,IAAA,+BAAuB,EACzB,UAAU,CACX,oDAAoD,CAAC,CACpD,IAAyB;gBAEzB,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAT,iBAAS;iBACV,CAAC,CAAC;YACL,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC","sourcesContent":["import type { TSESTree } from '@typescript-eslint/utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport { asPattern, getNgRxStores, namedCallableExpression } from '../../utils';\n\nexport const messageId = 'noStoreSubscription';\n\ntype MessageIds = typeof messageId;\ntype Options = readonly [];\n\nexport default createRule<Options, MessageIds>({\n  name: path.parse(__filename).name,\n  meta: {\n    type: 'suggestion',\n    ngrxModule: 'store',\n    docs: {\n      description:\n        'Using the `async` pipe is preferred over `store` subscription.',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        '`Store` subscription is forbidden. Use the `async` pipe instead.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const { identifiers = [] } = getNgRxStores(context);\n    const storeNames = identifiers.length > 0 ? asPattern(identifiers) : null;\n\n    if (!storeNames) {\n      return {};\n    }\n\n    return {\n      [`${namedCallableExpression(\n        storeNames\n      )} > MemberExpression > Identifier[name='subscribe']`](\n        node: TSESTree.Identifier\n      ) {\n        context.report({\n          node,\n          messageId,\n        });\n      },\n    };\n  },\n});\n"]}

package/src/rules/store/no-typed-global-store.d.ts

@@ -0,0 +1,5 @@
+export declare const noTypedStore = "noTypedStore";
+export declare const noTypedStoreSuggest = "noTypedStoreSuggest";
+type MessageIds = typeof noTypedStore | typeof noTypedStoreSuggest;
+declare const _default: import("@typescript-eslint/utils/ts-eslint").RuleModule<MessageIds, readonly [], import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+export default _default;

package/src/rules/store/no-typed-global-store.js

@@ -22,39 +22,11 @@ var __importStar = (this && this.__importStar) || function (mod) {
     __setModuleDefault(result, mod);
     return result;
 };
-var __values = (this && this.__values) || function(o) {
-    var s = typeof Symbol === "function" && Symbol.iterator, m = s && o[s], i = 0;
-    if (m) return m.call(o);
-    if (o && typeof o.length === "number") return {
-        next: function () {
-            if (o && i >= o.length) o = void 0;
-            return { value: o && o[i++], done: !o };
-        }
-    };
-    throw new TypeError(s ? "Object is not iterable." : "Symbol.iterator is not defined.");
-};
-var __read = (this && this.__read) || function (o, n) {
-    var m = typeof Symbol === "function" && o[Symbol.iterator];
-    if (!m) return o;
-    var i = m.call(o), r, ar = [], e;
-    try {
-        while ((n === void 0 || n-- > 0) && !(r = i.next()).done) ar.push(r.value);
-    }
-    catch (error) { e = { error: error }; }
-    finally {
-        try {
-            if (r && !r.done && (m = i["return"])) m.call(i);
-        }
-        finally { if (e) throw e.error; }
-    }
-    return ar;
-};
-var _a;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noTypedStoreSuggest = exports.noTypedStore = void 0;
-var path = __importStar(require("path"));
-var rule_creator_1 = require("../../rule-creator");
-var utils_1 = require("../../utils");
+const path = __importStar(require("path"));
+const rule_creator_1 = require("../../rule-creator");
+const utils_1 = require("../../utils");
 exports.noTypedStore = 'noTypedStore';
 exports.noTypedStoreSuggest = 'noTypedStoreSuggest';
 exports.default = (0, rule_creator_1.createRule)({
@@ -65,62 +37,49 @@ exports.default = (0, rule_creator_1.createRule)({
         ngrxModule: 'store',
         docs: {
             description: 'The global store should not be typed.',
-            recommended: 'warn',
-            suggestion: true,
         },
         schema: [],
-        messages: (_a = {},
-            _a[exports.noTypedStore] = '`Store` should not be typed, use `Store` (without generic) instead.',
-            _a[exports.noTypedStoreSuggest] = 'Remove generic from `Store`.',
-            _a),
+        messages: {
+            [exports.noTypedStore]: '`Store` should not be typed, use `Store` (without generic) instead.',
+            [exports.noTypedStoreSuggest]: 'Remove generic from `Store`.',
+        },
     },
     defaultOptions: [],
-    create: function (context) {
+    create: (context) => {
         return {
-            Program: function () {
-                var e_1, _a;
-                var _b = (0, utils_1.getNgRxStores)(context).identifiers, identifiers = _b === void 0 ? [] : _b;
-                try {
-                    for (var identifiers_1 = __values(identifiers), identifiers_1_1 = identifiers_1.next(); !identifiers_1_1.done; identifiers_1_1 = identifiers_1.next()) {
-                        var identifier = identifiers_1_1.value;
-                        // using inject()
-                        if (!identifier.typeAnnotation) {
-                            var parent_1 = identifier.parent;
-                            if ((0, utils_1.isPropertyDefinition)(parent_1) &&
-                                parent_1.value &&
-                                (0, utils_1.isCallExpression)(parent_1.value) &&
-                                parent_1.value.arguments.length) {
-                                var _c = __read(parent_1.value.arguments, 1), storeArgument = _c[0];
-                                if ((0, utils_1.isTSInstantiationExpression)(storeArgument)) {
-                                    report(storeArgument.typeParameters);
-                                }
+            Program() {
+                const { identifiers = [] } = (0, utils_1.getNgRxStores)(context);
+                for (const identifier of identifiers) {
+                    // using inject()
+                    if (!identifier.typeAnnotation) {
+                        const { parent } = identifier;
+                        if ((0, utils_1.isPropertyDefinition)(parent) &&
+                            parent.value &&
+                            (0, utils_1.isCallExpression)(parent.value) &&
+                            parent.value.arguments.length) {
+                            const [storeArgument] = parent.value.arguments;
+                            if ((0, utils_1.isTSInstantiationExpression)(storeArgument)) {
+                                report(storeArgument.typeArguments);
                             }
-                            continue;
                         }
-                        if (!(0, utils_1.isTSTypeReference)(identifier.typeAnnotation.typeAnnotation) ||
-                            !identifier.typeAnnotation.typeAnnotation.typeParameters) {
-                            continue;
-                        }
-                        report(identifier.typeAnnotation.typeAnnotation.typeParameters);
+                        continue;
                     }
-                }
-                catch (e_1_1) { e_1 = { error: e_1_1 }; }
-                finally {
-                    try {
-                        if (identifiers_1_1 && !identifiers_1_1.done && (_a = identifiers_1.return)) _a.call(identifiers_1);
+                    if (!(0, utils_1.isTSTypeReference)(identifier.typeAnnotation.typeAnnotation) ||
+                        !identifier.typeAnnotation.typeAnnotation.typeArguments) {
+                        continue;
                     }
-                    finally { if (e_1) throw e_1.error; }
+                    report(identifier.typeAnnotation.typeAnnotation.typeArguments);
                 }
             },
         };
-        function report(typeParameters) {
+        function report(typeArguments) {
             context.report({
-                node: typeParameters,
+                node: typeArguments,
                 messageId: exports.noTypedStore,
                 suggest: [
                     {
                         messageId: exports.noTypedStoreSuggest,
-                        fix: function (fixer) { return fixer.remove(typeParameters); },
+                        fix: (fixer) => fixer.remove(typeArguments),
                     },
                 ],
             });