@ngrx/eslint-plugin 14.0.0-beta.0

package/src/rules/effects/prefer-effect-callback-in-block-statement.js

@@ -0,0 +1,99 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __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;
+exports.__esModule = true;
+exports.messageId = void 0;
+var experimental_utils_1 = require("@typescript-eslint/experimental-utils");
+var path = __importStar(require("path"));
+var rule_creator_1 = require("../../rule-creator");
+var utils_1 = require("../../utils");
+exports.messageId = 'preferEffectCallbackInBlockStatement';
+exports["default"] = (0, rule_creator_1.createRule)({
+    name: path.parse(__filename).name,
+    meta: {
+        type: 'suggestion',
+        ngrxModule: 'effects',
+        docs: {
+            description: 'A block statement is easier to troubleshoot.',
+            recommended: 'warn'
+        },
+        schema: [],
+        messages: (_a = {},
+            _a[exports.messageId] = 'The callback of `Effect` should be wrapped in a block statement.',
+            _a),
+        fixable: 'code'
+    },
+    defaultOptions: [],
+    create: function (context) {
+        var _a;
+        var sourceCode = context.getSourceCode();
+        var nonParametrizedEffect = "".concat(utils_1.createEffectExpression, " > ArrowFunctionExpression > .body[type!=/^(ArrowFunctionExpression|BlockStatement)$/]");
+        var parametrizedEffect = "".concat(utils_1.createEffectExpression, " > ArrowFunctionExpression > ArrowFunctionExpression > .body[type!='BlockStatement']");
+        var parametrizedEffectWithinBlockStatement = "".concat(utils_1.createEffectExpression, " > ArrowFunctionExpression > BlockStatement > ReturnStatement > ArrowFunctionExpression > .body[type!='BlockStatement']");
+        return _a = {},
+            _a["".concat(nonParametrizedEffect, ", ").concat(parametrizedEffect, ", ").concat(parametrizedEffectWithinBlockStatement)] = function (node) {
+                context.report({
+                    node: node,
+                    messageId: exports.messageId,
+                    fix: function (fixer) {
+                        var _a = __read(getSafeNodesToApplyFix(sourceCode, node), 2), previousNode = _a[0], nextNode = _a[1];
+                        return [
+                            fixer.insertTextBefore(previousNode, "{ return "),
+                            fixer.insertTextAfter(nextNode, " }"),
+                        ];
+                    }
+                });
+            },
+            _a;
+    }
+});
+function getSafeNodesToApplyFix(sourceCode, node) {
+    var previousToken = sourceCode.getTokenBefore(node);
+    var nextToken = sourceCode.getTokenAfter(node);
+    if (previousToken &&
+        experimental_utils_1.ASTUtils.isOpeningParenToken(previousToken) &&
+        nextToken &&
+        experimental_utils_1.ASTUtils.isClosingParenToken(nextToken)) {
+        return [previousToken, nextToken];
+    }
+    return [node, node];
+}
+//# sourceMappingURL=prefer-effect-callback-in-block-statement.js.map

package/src/rules/effects/prefer-effect-callback-in-block-statement.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-effect-callback-in-block-statement.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/effects/prefer-effect-callback-in-block-statement.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,4EAAiE;AACjE,yCAA6B;AAC7B,mDAAgD;AAChD,qCAAqD;AAExC,QAAA,SAAS,GAAG,sCAAsC,CAAC;AAKhE,qBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,SAAS;QACrB,IAAI,EAAE;YACJ,WAAW,EAAE,8CAA8C;YAC3D,WAAW,EAAE,MAAM;SACpB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IACR,kEAAkE;eACrE;QACD,OAAO,EAAE,MAAM;KAChB;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACd,IAAM,UAAU,GAAG,OAAO,CAAC,aAAa,EAAE,CAAC;QAC3C,IAAM,qBAAqB,GACzB,UAAG,8BAAsB,2FAAiG,CAAC;QAC7H,IAAM,kBAAkB,GACtB,UAAG,8BAAsB,yFAA+F,CAAC;QAC3H,IAAM,sCAAsC,GAC1C,UAAG,8BAAsB,4HAAkI,CAAC;QAE9J;YACE,GAAC,UAAG,qBAAqB,eAAK,kBAAkB,eAAK,sCAAsC,CAAE,IAA7F,UACE,IAA8C;gBAE9C,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI,MAAA;oBACJ,SAAS,mBAAA;oBACT,GAAG,EAAE,UAAC,KAAK;wBACH,IAAA,KAAA,OAA2B,sBAAsB,CACrD,UAAU,EACV,IAAI,CACL,IAAA,EAHM,YAAY,QAAA,EAAE,QAAQ,QAG5B,CAAC;wBACF,OAAO;4BACL,KAAK,CAAC,gBAAgB,CAAC,YAAY,EAAE,WAAW,CAAC;4BACjD,KAAK,CAAC,eAAe,CAAC,QAAQ,EAAE,IAAI,CAAC;yBACtC,CAAC;oBACJ,CAAC;iBACF,CAAC,CAAC;YACL,CAAC;eACD;IACJ,CAAC;CACF,CAAC,CAAC;AAEH,SAAS,sBAAsB,CAC7B,UAAyC,EACzC,IAAmB;IAEnB,IAAM,aAAa,GAAG,UAAU,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC;IACtD,IAAM,SAAS,GAAG,UAAU,CAAC,aAAa,CAAC,IAAI,CAAC,CAAC;IAEjD,IACE,aAAa;QACb,6BAAQ,CAAC,mBAAmB,CAAC,aAAa,CAAC;QAC3C,SAAS;QACT,6BAAQ,CAAC,mBAAmB,CAAC,SAAS,CAAC,EACvC;QACA,OAAO,CAAC,aAAa,EAAE,SAAS,CAAU,CAAC;KAC5C;IAED,OAAO,CAAC,IAAI,EAAE,IAAI,CAAU,CAAC;AAC/B,CAAC","sourcesContent":["import type { TSESLint, TSESTree } from '@typescript-eslint/experimental-utils';\nimport { ASTUtils } from '@typescript-eslint/experimental-utils';\nimport * as path from 'path';\nimport { createRule } from '../../rule-creator';\nimport { createEffectExpression } from '../../utils';\n\nexport const messageId = 'preferEffectCallbackInBlockStatement';\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: 'effects',\n    docs: {\n      description: 'A block statement is easier to troubleshoot.',\n      recommended: 'warn',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        'The callback of `Effect` should be wrapped in a block statement.',\n    },\n    fixable: 'code',\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const sourceCode = context.getSourceCode();\n    const nonParametrizedEffect =\n      `${createEffectExpression} > ArrowFunctionExpression > .body[type!=/^(ArrowFunctionExpression|BlockStatement)$/]` as const;\n    const parametrizedEffect =\n      `${createEffectExpression} > ArrowFunctionExpression > ArrowFunctionExpression > .body[type!='BlockStatement']` as const;\n    const parametrizedEffectWithinBlockStatement =\n      `${createEffectExpression} > ArrowFunctionExpression > BlockStatement > ReturnStatement > ArrowFunctionExpression > .body[type!='BlockStatement']` as const;\n\n    return {\n      [`${nonParametrizedEffect}, ${parametrizedEffect}, ${parametrizedEffectWithinBlockStatement}`](\n        node: TSESTree.ArrowFunctionExpression['body']\n      ) {\n        context.report({\n          node,\n          messageId,\n          fix: (fixer) => {\n            const [previousNode, nextNode] = getSafeNodesToApplyFix(\n              sourceCode,\n              node\n            );\n            return [\n              fixer.insertTextBefore(previousNode, `{ return `),\n              fixer.insertTextAfter(nextNode, ` }`),\n            ];\n          },\n        });\n      },\n    };\n  },\n});\n\nfunction getSafeNodesToApplyFix(\n  sourceCode: Readonly<TSESLint.SourceCode>,\n  node: TSESTree.Node\n) {\n  const previousToken = sourceCode.getTokenBefore(node);\n  const nextToken = sourceCode.getTokenAfter(node);\n\n  if (\n    previousToken &&\n    ASTUtils.isOpeningParenToken(previousToken) &&\n    nextToken &&\n    ASTUtils.isClosingParenToken(nextToken)\n  ) {\n    return [previousToken, nextToken] as const;\n  }\n\n  return [node, node] as const;\n}\n"]}

package/src/rules/effects/use-effects-lifecycle-interface.js

@@ -0,0 +1,88 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var _a;
+exports.__esModule = true;
+exports.messageId = void 0;
+var path = __importStar(require("path"));
+var rule_creator_1 = require("../../rule-creator");
+var utils_1 = require("../../utils");
+exports.messageId = 'useEffectsLifecycleInterface';
+exports["default"] = (0, rule_creator_1.createRule)({
+    name: path.parse(__filename).name,
+    meta: {
+        type: 'suggestion',
+        ngrxModule: 'effects',
+        docs: {
+            description: 'Ensures classes implement lifecycle interfaces corresponding to the declared lifecycle methods.',
+            recommended: 'warn'
+        },
+        fixable: 'code',
+        schema: [],
+        messages: (_a = {},
+            _a[exports.messageId] = 'Lifecycle interface `{{ interfaceName }}` should be implemented for method `{{ methodName }}`.',
+            _a)
+    },
+    defaultOptions: [],
+    create: function (context) {
+        var _a;
+        var lifecycleMapper = {
+            ngrxOnIdentifyEffects: 'OnIdentifyEffects',
+            ngrxOnInitEffects: 'OnInitEffects',
+            ngrxOnRunEffects: 'OnRunEffects'
+        };
+        var lifecyclesPattern = Object.keys(lifecycleMapper).join('|');
+        return _a = {},
+            _a["ClassDeclaration > ClassBody > MethodDefinition > Identifier[name=/".concat(lifecyclesPattern, "/]")] = function (node) {
+                var classDeclaration = node.parent.parent.parent;
+                var methodName = node.name;
+                var interfaceName = lifecycleMapper[methodName];
+                if ((0, utils_1.getInterface)(classDeclaration, interfaceName)) {
+                    return;
+                }
+                context.report({
+                    fix: function (fixer) {
+                        var _a = (0, utils_1.getImplementsSchemaFixer)(classDeclaration, interfaceName), implementsNodeReplace = _a.implementsNodeReplace, implementsTextReplace = _a.implementsTextReplace;
+                        return [
+                            fixer.insertTextAfter(implementsNodeReplace, implementsTextReplace),
+                        ].concat((0, utils_1.getImportAddFix)({
+                            compatibleWithTypeOnlyImport: true,
+                            fixer: fixer,
+                            importName: interfaceName,
+                            moduleName: utils_1.NGRX_MODULE_PATHS.effects,
+                            node: classDeclaration
+                        }));
+                    },
+                    node: node,
+                    messageId: exports.messageId,
+                    data: {
+                        interfaceName: interfaceName,
+                        methodName: methodName
+                    }
+                });
+            },
+            _a;
+    }
+});
+//# sourceMappingURL=use-effects-lifecycle-interface.js.map

package/src/rules/effects/use-effects-lifecycle-interface.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"use-effects-lifecycle-interface.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/effects/use-effects-lifecycle-interface.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAKqB;AAER,QAAA,SAAS,GAAG,8BAA8B,CAAC;AAKxD,qBAAe,IAAA,yBAAU,EAAsB;IAC7C,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,IAAI;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,UAAU,EAAE,SAAS;QACrB,IAAI,EAAE;YACJ,WAAW,EACT,iGAAiG;YACnG,WAAW,EAAE,MAAM;SACpB;QACD,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IACR,gGAAgG;eACnG;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACd,IAAM,eAAe,GAAG;YACtB,qBAAqB,EAAE,mBAAmB;YAC1C,iBAAiB,EAAE,eAAe;YAClC,gBAAgB,EAAE,cAAc;SACxB,CAAC;QACX,IAAM,iBAAiB,GAAG,MAAM,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAEjE;YACE,GAAC,6EAAsE,iBAAiB,OAAI,IAA5F,UACE,IAKC;gBAED,IAAM,gBAAgB,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC;gBACnD,IAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC;gBAC7B,IAAM,aAAa,GAAG,eAAe,CAAC,UAAU,CAAC,CAAC;gBAElD,IAAI,IAAA,oBAAY,EAAC,gBAAgB,EAAE,aAAa,CAAC,EAAE;oBACjD,OAAO;iBACR;gBAED,OAAO,CAAC,MAAM,CAAC;oBACb,GAAG,EAAE,UAAC,KAAK;wBACH,IAAA,KACJ,IAAA,gCAAwB,EAAC,gBAAgB,EAAE,aAAa,CAAC,EADnD,qBAAqB,2BAAA,EAAE,qBAAqB,2BACO,CAAC;wBAC5D,OAAO;4BACL,KAAK,CAAC,eAAe,CACnB,qBAAqB,EACrB,qBAAqB,CACtB;yBACF,CAAC,MAAM,CACN,IAAA,uBAAe,EAAC;4BACd,4BAA4B,EAAE,IAAI;4BAClC,KAAK,OAAA;4BACL,UAAU,EAAE,aAAa;4BACzB,UAAU,EAAE,yBAAiB,CAAC,OAAO;4BACrC,IAAI,EAAE,gBAAgB;yBACvB,CAAC,CACH,CAAC;oBACJ,CAAC;oBACD,IAAI,MAAA;oBACJ,SAAS,mBAAA;oBACT,IAAI,EAAE;wBACJ,aAAa,eAAA;wBACb,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 {\n  getImplementsSchemaFixer,\n  getImportAddFix,\n  getInterface,\n  NGRX_MODULE_PATHS,\n} from '../../utils';\n\nexport const messageId = 'useEffectsLifecycleInterface';\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: 'effects',\n    docs: {\n      description:\n        'Ensures classes implement lifecycle interfaces corresponding to the declared lifecycle methods.',\n      recommended: 'warn',\n    },\n    fixable: 'code',\n    schema: [],\n    messages: {\n      [messageId]:\n        'Lifecycle interface `{{ interfaceName }}` should be implemented for method `{{ methodName }}`.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const lifecycleMapper = {\n      ngrxOnIdentifyEffects: 'OnIdentifyEffects',\n      ngrxOnInitEffects: 'OnInitEffects',\n      ngrxOnRunEffects: 'OnRunEffects',\n    } as const;\n    const lifecyclesPattern = Object.keys(lifecycleMapper).join('|');\n\n    return {\n      [`ClassDeclaration > ClassBody > MethodDefinition > Identifier[name=/${lifecyclesPattern}/]`](\n        node: TSESTree.Identifier & {\n          name: keyof typeof lifecycleMapper;\n          parent: TSESTree.MethodDefinition & {\n            parent: TSESTree.ClassBody & { parent: TSESTree.ClassDeclaration };\n          };\n        }\n      ) {\n        const classDeclaration = node.parent.parent.parent;\n        const methodName = node.name;\n        const interfaceName = lifecycleMapper[methodName];\n\n        if (getInterface(classDeclaration, interfaceName)) {\n          return;\n        }\n\n        context.report({\n          fix: (fixer) => {\n            const { implementsNodeReplace, implementsTextReplace } =\n              getImplementsSchemaFixer(classDeclaration, interfaceName);\n            return [\n              fixer.insertTextAfter(\n                implementsNodeReplace,\n                implementsTextReplace\n              ),\n            ].concat(\n              getImportAddFix({\n                compatibleWithTypeOnlyImport: true,\n                fixer,\n                importName: interfaceName,\n                moduleName: NGRX_MODULE_PATHS.effects,\n                node: classDeclaration,\n              })\n            );\n          },\n          node,\n          messageId,\n          data: {\n            interfaceName,\n            methodName,\n          },\n        });\n      },\n    };\n  },\n});\n"]}

package/src/rules/index.js

@@ -0,0 +1,59 @@
+"use strict";
+var __assign = (this && this.__assign) || function () {
+    __assign = Object.assign || function(t) {
+        for (var s, i = 1, n = arguments.length; i < n; i++) {
+            s = arguments[i];
+            for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p))
+                t[p] = s[p];
+        }
+        return t;
+    };
+    return __assign.apply(this, arguments);
+};
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+exports.__esModule = true;
+exports.rules = void 0;
+var path = __importStar(require("path"));
+var utils_1 = require("../utils");
+// Copied from https://github.com/jest-community/eslint-plugin-jest/blob/main/src/index.ts
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+var interopRequireDefault = function (obj) {
+    return (obj === null || obj === void 0 ? void 0 : obj.__esModule) ? obj : { "default": obj };
+};
+var importDefault = function (moduleName) {
+    // eslint-disable-next-line @typescript-eslint/no-var-requires
+    return interopRequireDefault(require(moduleName))["default"];
+};
+var rulesDir = __dirname;
+var excludedFiles = ['index'];
+exports.rules = Array.from((0, utils_1.traverseFolder)(rulesDir))
+    .filter(function (rule) { return !excludedFiles.includes(rule.file); })
+    .reduce(function (allRules, rule) {
+    var _a;
+    var ruleModule = importDefault(rule.path);
+    ruleModule.meta.ngrxModule = path.basename(path.dirname(rule.path));
+    return __assign(__assign({}, allRules), (_a = {}, _a[rule.file] = ruleModule, _a));
+}, {});
+//# sourceMappingURL=index.js.map

package/src/rules/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../modules/eslint-plugin/src/rules/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,yCAA6B;AAG7B,kCAA0C;AAE1C,0FAA0F;AAE1F,8DAA8D;AAC9D,IAAM,qBAAqB,GAAG,UAAC,GAAQ;IACrC,OAAA,CAAA,GAAG,aAAH,GAAG,uBAAH,GAAG,CAAE,UAAU,EAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,SAAO,EAAE,GAAG,EAAE;AAAxC,CAAwC,CAAC;AAE3C,IAAM,aAAa,GAAG,UAAC,UAAkB;IACvC,8DAA8D;IAC9D,OAAA,qBAAqB,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC,SAAO,CAAA;AAAlD,CAAkD,CAAC;AAErD,IAAM,QAAQ,GAAG,SAAS,CAAC;AAC3B,IAAM,aAAa,GAAG,CAAC,OAAO,CAAC,CAAC;AAEnB,QAAA,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,IAAA,sBAAc,EAAC,QAAQ,CAAC,CAAC;KACtD,MAAM,CAAC,UAAC,IAAI,IAAK,OAAA,CAAC,aAAa,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,EAAlC,CAAkC,CAAC;KACpD,MAAM,CAA6C,UAAC,QAAQ,EAAE,IAAI;;IACjE,IAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,IAAI,CAA+B,CAAC;IAC1E,UAAU,CAAC,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC,QAAQ,CACxC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CACT,CAAC;IACjB,6BACK,QAAQ,gBACV,IAAI,CAAC,IAAI,IAAG,UAAU,OACvB;AACJ,CAAC,EAAE,EAAE,CAAC,CAAC","sourcesContent":["import * as path from 'path';\nimport type { NgRxRuleModule } from '../rule-creator';\nimport type { NGRX_MODULE } from '../utils';\nimport { traverseFolder } from '../utils';\n\n// Copied from https://github.com/jest-community/eslint-plugin-jest/blob/main/src/index.ts\n\n// eslint-disable-next-line @typescript-eslint/no-explicit-any\nconst interopRequireDefault = (obj: any): { default: unknown } =>\n  obj?.__esModule ? obj : { default: obj };\n\nconst importDefault = (moduleName: string) =>\n  // eslint-disable-next-line @typescript-eslint/no-var-requires\n  interopRequireDefault(require(moduleName)).default;\n\nconst rulesDir = __dirname;\nconst excludedFiles = ['index'];\n\nexport const rules = Array.from(traverseFolder(rulesDir))\n  .filter((rule) => !excludedFiles.includes(rule.file))\n  .reduce<Record<string, NgRxRuleModule<[], string>>>((allRules, rule) => {\n    const ruleModule = importDefault(rule.path) as NgRxRuleModule<[], string>;\n    ruleModule.meta.ngrxModule = path.basename(\n      path.dirname(rule.path)\n    ) as NGRX_MODULE;\n    return {\n      ...allRules,\n      [rule.file]: ruleModule,\n    };\n  }, {});\n"]}

package/src/rules/store/avoid-combining-selectors.js

@@ -0,0 +1,65 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var _a;
+exports.__esModule = true;
+exports.messageId = void 0;
+var path = __importStar(require("path"));
+var rule_creator_1 = require("../../rule-creator");
+var utils_1 = require("../../utils");
+exports.messageId = 'avoidCombiningSelectors';
+exports["default"] = (0, rule_creator_1.createRule)({
+    name: path.parse(__filename).name,
+    meta: {
+        type: 'suggestion',
+        ngrxModule: 'store',
+        docs: {
+            description: 'Prefer combining selectors at the selector level.',
+            recommended: 'warn'
+        },
+        schema: [],
+        messages: (_a = {},
+            _a[exports.messageId] = 'Combine selectors at the selector level.',
+            _a)
+    },
+    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;
+        if (!storeNames) {
+            return {};
+        }
+        var pipeableOrStoreSelect = ":matches(".concat((0, utils_1.namedExpression)(storeNames), "[callee.property.name='pipe']:has(CallExpression[callee.name='select']), ").concat((0, utils_1.selectExpression)(storeNames), ")");
+        return _a = {},
+            _a["CallExpression[callee.name='combineLatest'][arguments.length>1] ".concat(pipeableOrStoreSelect, " ~ ").concat(pipeableOrStoreSelect)] = function (node) {
+                context.report({
+                    node: node,
+                    messageId: exports.messageId
+                });
+            },
+            _a;
+    }
+});
+//# sourceMappingURL=avoid-combining-selectors.js.map

package/src/rules/store/avoid-combining-selectors.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"avoid-combining-selectors.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/avoid-combining-selectors.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAKqB;AAER,QAAA,SAAS,GAAG,yBAAyB,CAAC;AAKnD,qBAAe,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,mDAAmD;YAChE,WAAW,EAAE,MAAM;SACpB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IAAG,0CAA0C;eACxD;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;YACf,OAAO,EAAE,CAAC;SACX;QAED,IAAM,qBAAqB,GAAG,mBAAY,IAAA,uBAAe,EACvD,UAAU,CACX,sFAA4E,IAAA,wBAAgB,EAC3F,UAAU,CACX,MAAY,CAAC;QAEd;YACE,GAAC,0EAAmE,qBAAqB,gBAAM,qBAAqB,CAAE,IAAtH,UACE,IAA6B;gBAE7B,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 {\n  asPattern,\n  getNgRxStores,\n  namedExpression,\n  selectExpression,\n} from '../../utils';\n\nexport const messageId = 'avoidCombiningSelectors';\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: 'Prefer combining selectors at the selector level.',\n      recommended: 'warn',\n    },\n    schema: [],\n    messages: {\n      [messageId]: 'Combine selectors at the selector level.',\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    const pipeableOrStoreSelect = `:matches(${namedExpression(\n      storeNames\n    )}[callee.property.name='pipe']:has(CallExpression[callee.name='select']), ${selectExpression(\n      storeNames\n    )})` as const;\n\n    return {\n      [`CallExpression[callee.name='combineLatest'][arguments.length>1] ${pipeableOrStoreSelect} ~ ${pipeableOrStoreSelect}`](\n        node: TSESTree.CallExpression\n      ) {\n        context.report({\n          node,\n          messageId,\n        });\n      },\n    };\n  },\n});\n"]}

package/src/rules/store/avoid-dispatching-multiple-actions-sequentially.js

@@ -0,0 +1,95 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __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 _a;
+exports.__esModule = true;
+exports.messageId = void 0;
+var path = __importStar(require("path"));
+var rule_creator_1 = require("../../rule-creator");
+var utils_1 = require("../../utils");
+exports.messageId = 'avoidDispatchingMultipleActionsSequentially';
+exports["default"] = (0, rule_creator_1.createRule)({
+    name: path.parse(__filename).name,
+    meta: {
+        type: 'suggestion',
+        ngrxModule: 'store',
+        docs: {
+            description: 'It is recommended to only dispatch one `Action` at a time.',
+            recommended: 'warn'
+        },
+        schema: [],
+        messages: (_a = {},
+            _a[exports.messageId] = 'Avoid dispatching many actions in a row to accomplish a larger conceptual "transaction".',
+            _a)
+    },
+    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;
+        if (!storeNames) {
+            return {};
+        }
+        var collectedDispatches = new Set();
+        return _a = {},
+            _a["BlockStatement > ExpressionStatement > ".concat((0, utils_1.dispatchExpression)(storeNames))] = function (node) {
+                collectedDispatches.add(node);
+            },
+            _a['BlockStatement:exit'] = function () {
+                var e_1, _a;
+                if (collectedDispatches.size > 1) {
+                    try {
+                        for (var collectedDispatches_1 = __values(collectedDispatches), collectedDispatches_1_1 = collectedDispatches_1.next(); !collectedDispatches_1_1.done; collectedDispatches_1_1 = collectedDispatches_1.next()) {
+                            var node = collectedDispatches_1_1.value;
+                            context.report({
+                                node: node,
+                                messageId: exports.messageId
+                            });
+                        }
+                    }
+                    catch (e_1_1) { e_1 = { error: e_1_1 }; }
+                    finally {
+                        try {
+                            if (collectedDispatches_1_1 && !collectedDispatches_1_1.done && (_a = collectedDispatches_1["return"])) _a.call(collectedDispatches_1);
+                        }
+                        finally { if (e_1) throw e_1.error; }
+                    }
+                }
+                collectedDispatches.clear();
+            },
+            _a;
+    }
+});
+//# sourceMappingURL=avoid-dispatching-multiple-actions-sequentially.js.map

package/src/rules/store/avoid-dispatching-multiple-actions-sequentially.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"avoid-dispatching-multiple-actions-sequentially.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/avoid-dispatching-multiple-actions-sequentially.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAA2E;AAE9D,QAAA,SAAS,GAAG,6CAA6C,CAAC;AAKvE,qBAAe,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,4DAA4D;YACzE,WAAW,EAAE,MAAM;SACpB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,iBAAS,IACR,0FAA0F;eAC7F;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;YACf,OAAO,EAAE,CAAC;SACX;QAED,IAAM,mBAAmB,GAAG,IAAI,GAAG,EAA2B,CAAC;QAE/D;YACE,GAAC,iDAA0C,IAAA,0BAAkB,EAC3D,UAAU,CACX,CAAE,IAFH,UAEK,IAA6B;gBAChC,mBAAmB,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAChC,CAAC;YACD,yBAAqB,GAArB;;gBACE,IAAI,mBAAmB,CAAC,IAAI,GAAG,CAAC,EAAE;;wBAChC,KAAmB,IAAA,wBAAA,SAAA,mBAAmB,CAAA,wDAAA,yFAAE;4BAAnC,IAAM,IAAI,gCAAA;4BACb,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,MAAA;gCACJ,SAAS,mBAAA;6BACV,CAAC,CAAC;yBACJ;;;;;;;;;iBACF;gBAED,mBAAmB,CAAC,KAAK,EAAE,CAAC;YAC9B,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, dispatchExpression, getNgRxStores } from '../../utils';\n\nexport const messageId = 'avoidDispatchingMultipleActionsSequentially';\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: 'It is recommended to only dispatch one `Action` at a time.',\n      recommended: 'warn',\n    },\n    schema: [],\n    messages: {\n      [messageId]:\n        'Avoid dispatching many actions in a row to accomplish a larger conceptual \"transaction\".',\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    const collectedDispatches = new Set<TSESTree.CallExpression>();\n\n    return {\n      [`BlockStatement > ExpressionStatement > ${dispatchExpression(\n        storeNames\n      )}`](node: TSESTree.CallExpression) {\n        collectedDispatches.add(node);\n      },\n      'BlockStatement:exit'() {\n        if (collectedDispatches.size > 1) {\n          for (const node of collectedDispatches) {\n            context.report({\n              node,\n              messageId,\n            });\n          }\n        }\n\n        collectedDispatches.clear();\n      },\n    };\n  },\n});\n"]}

package/src/rules/store/avoid-duplicate-actions-in-reducer.js

@@ -0,0 +1,150 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __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 __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 __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 _a;
+exports.__esModule = true;
+exports.avoidDuplicateActionsInReducerSuggest = exports.avoidDuplicateActionsInReducer = void 0;
+var path = __importStar(require("path"));
+var rule_creator_1 = require("../../rule-creator");
+var utils_1 = require("../../utils");
+exports.avoidDuplicateActionsInReducer = 'avoidDuplicateActionsInReducer';
+exports.avoidDuplicateActionsInReducerSuggest = 'avoidDuplicateActionsInReducerSuggest';
+exports["default"] = (0, rule_creator_1.createRule)({
+    name: path.parse(__filename).name,
+    meta: {
+        type: 'suggestion',
+        hasSuggestions: true,
+        ngrxModule: 'store',
+        docs: {
+            description: 'A `Reducer` should handle an `Action` once.',
+            recommended: 'warn',
+            suggestion: true
+        },
+        schema: [],
+        messages: (_a = {},
+            _a[exports.avoidDuplicateActionsInReducer] = 'The `Reducer` handles a duplicate `Action` `{{ actionName }}`.',
+            _a[exports.avoidDuplicateActionsInReducerSuggest] = 'Remove this duplication.',
+            _a)
+    },
+    defaultOptions: [],
+    create: function (context) {
+        var _a;
+        var sourceCode = context.getSourceCode();
+        var collectedActions = new Map();
+        return _a = {},
+            _a["".concat(utils_1.createReducer, " > CallExpression[callee.name='on'][arguments.0.type='Identifier']")] = function (_a) {
+                var _b;
+                var _c = __read(_a.arguments, 1), action = _c[0];
+                var actions = (_b = collectedActions.get(action.name)) !== null && _b !== void 0 ? _b : [];
+                collectedActions.set(action.name, __spreadArray(__spreadArray([], __read(actions), false), [action], false));
+            },
+            _a["".concat(utils_1.createReducer, ":exit")] = function () {
+                var e_1, _a, e_2, _b;
+                try {
+                    for (var collectedActions_1 = __values(collectedActions), collectedActions_1_1 = collectedActions_1.next(); !collectedActions_1_1.done; collectedActions_1_1 = collectedActions_1.next()) {
+                        var _c = __read(collectedActions_1_1.value, 2), actionName = _c[0], identifiers = _c[1];
+                        if (identifiers.length <= 1) {
+                            break;
+                        }
+                        var _loop_1 = function (node) {
+                            context.report({
+                                node: node,
+                                messageId: exports.avoidDuplicateActionsInReducer,
+                                data: {
+                                    actionName: actionName
+                                },
+                                suggest: [
+                                    {
+                                        messageId: exports.avoidDuplicateActionsInReducerSuggest,
+                                        fix: function (fixer) {
+                                            return (0, utils_1.getNodeToCommaRemoveFix)(sourceCode, fixer, node.parent);
+                                        }
+                                    },
+                                ]
+                            });
+                        };
+                        try {
+                            for (var identifiers_1 = (e_2 = void 0, __values(identifiers)), 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; }
+                        }
+                    }
+                }
+                catch (e_1_1) { e_1 = { error: e_1_1 }; }
+                finally {
+                    try {
+                        if (collectedActions_1_1 && !collectedActions_1_1.done && (_a = collectedActions_1["return"])) _a.call(collectedActions_1);
+                    }
+                    finally { if (e_1) throw e_1.error; }
+                }
+                collectedActions.clear();
+            },
+            _a;
+    }
+});
+//# sourceMappingURL=avoid-duplicate-actions-in-reducer.js.map

package/src/rules/store/avoid-duplicate-actions-in-reducer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"avoid-duplicate-actions-in-reducer.js","sourceRoot":"","sources":["../../../../../../modules/eslint-plugin/src/rules/store/avoid-duplicate-actions-in-reducer.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AACA,yCAA6B;AAC7B,mDAAgD;AAChD,qCAAqE;AAExD,QAAA,8BAA8B,GAAG,gCAAgC,CAAC;AAClE,QAAA,qCAAqC,GAChD,uCAAuC,CAAC;AAQ1C,qBAAe,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,6CAA6C;YAC1D,WAAW,EAAE,MAAM;YACnB,UAAU,EAAE,IAAI;SACjB;QACD,MAAM,EAAE,EAAE;QACV,QAAQ;YACN,GAAC,sCAA8B,IAC7B,gEAAgE;YAClE,GAAC,6CAAqC,IAAG,0BAA0B;eACpE;KACF;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,EAAE,UAAC,OAAO;;QACd,IAAM,UAAU,GAAG,OAAO,CAAC,aAAa,EAAE,CAAC;QAC3C,IAAM,gBAAgB,GAAG,IAAI,GAAG,EAAoB,CAAC;QAErD;YACE,GAAC,UAAG,qBAAa,uEAAoE,IAArF,UAAuF,EAItF;;oBAHC,KAAA,uBAAmB,EAAP,MAAM,QAAA;gBAIlB,IAAM,OAAO,GAAG,MAAA,gBAAgB,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,mCAAI,EAAE,CAAC;gBACxD,gBAAgB,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,yCAAM,OAAO,YAAE,MAAM,UAAE,CAAC;YAC1D,CAAC;YACD,GAAC,UAAG,qBAAa,UAAO,IAAxB;;;oBACE,KAAwC,IAAA,qBAAA,SAAA,gBAAgB,CAAA,kDAAA,gFAAE;wBAA/C,IAAA,KAAA,qCAAyB,EAAxB,UAAU,QAAA,EAAE,WAAW,QAAA;wBACjC,IAAI,WAAW,CAAC,MAAM,IAAI,CAAC,EAAE;4BAC3B,MAAM;yBACP;gDAEU,IAAI;4BACb,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,MAAA;gCACJ,SAAS,EAAE,sCAA8B;gCACzC,IAAI,EAAE;oCACJ,UAAU,YAAA;iCACX;gCACD,OAAO,EAAE;oCACP;wCACE,SAAS,EAAE,6CAAqC;wCAChD,GAAG,EAAE,UAAC,KAAK;4CACT,OAAA,IAAA,+BAAuB,EAAC,UAAU,EAAE,KAAK,EAAE,IAAI,CAAC,MAAM,CAAC;wCAAvD,CAAuD;qCAC1D;iCACF;6BACF,CAAC,CAAC;;;4BAdL,KAAmB,IAAA,+BAAA,SAAA,WAAW,CAAA,CAAA,wCAAA;gCAAzB,IAAM,IAAI,wBAAA;wCAAJ,IAAI;6BAed;;;;;;;;;qBACF;;;;;;;;;gBAED,gBAAgB,CAAC,KAAK,EAAE,CAAC;YAC3B,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 { createReducer, getNodeToCommaRemoveFix } from '../../utils';\n\nexport const avoidDuplicateActionsInReducer = 'avoidDuplicateActionsInReducer';\nexport const avoidDuplicateActionsInReducerSuggest =\n  'avoidDuplicateActionsInReducerSuggest';\n\ntype MessageIds =\n  | typeof avoidDuplicateActionsInReducer\n  | typeof avoidDuplicateActionsInReducerSuggest;\ntype Options = readonly [];\ntype Action = TSESTree.Identifier & { parent: TSESTree.CallExpression };\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: 'A `Reducer` should handle an `Action` once.',\n      recommended: 'warn',\n      suggestion: true,\n    },\n    schema: [],\n    messages: {\n      [avoidDuplicateActionsInReducer]:\n        'The `Reducer` handles a duplicate `Action` `{{ actionName }}`.',\n      [avoidDuplicateActionsInReducerSuggest]: 'Remove this duplication.',\n    },\n  },\n  defaultOptions: [],\n  create: (context) => {\n    const sourceCode = context.getSourceCode();\n    const collectedActions = new Map<string, Action[]>();\n\n    return {\n      [`${createReducer} > CallExpression[callee.name='on'][arguments.0.type='Identifier']`]({\n        arguments: [action],\n      }: TSESTree.CallExpression & {\n        arguments: Action[];\n      }) {\n        const actions = collectedActions.get(action.name) ?? [];\n        collectedActions.set(action.name, [...actions, action]);\n      },\n      [`${createReducer}:exit`]() {\n        for (const [actionName, identifiers] of collectedActions) {\n          if (identifiers.length <= 1) {\n            break;\n          }\n\n          for (const node of identifiers) {\n            context.report({\n              node,\n              messageId: avoidDuplicateActionsInReducer,\n              data: {\n                actionName,\n              },\n              suggest: [\n                {\n                  messageId: avoidDuplicateActionsInReducerSuggest,\n                  fix: (fixer) =>\n                    getNodeToCommaRemoveFix(sourceCode, fixer, node.parent),\n                },\n              ],\n            });\n          }\n        }\n\n        collectedActions.clear();\n      },\n    };\n  },\n});\n"]}