eslint-plugin-effector 0.3.0 → 0.4.2

package/README.md

@@ -41,10 +41,24 @@ To configure individual rules:
 
 ## Supported Rules
 
-- [effector/enforce-store-naming-convention](/rules/enforce-store-naming-convention/enforce-store-naming-convention.md)
-- [effector/enforce-effect-naming-convention](/rules/enforce-effect-naming-convention/enforce-effect-naming-convention.md)
-- [effector/no-getState](/rules/no-getState/no-getState.md)
-- [effector/no-unnecessary-duplication](/rules/no-unnecessary-duplication/no-unnecessary-duplication.md)
-- [effector/no-useless-methods](/rules/no-useless-methods/no-useless-methods.md)
-- [effector/no-ambiguity-target](/rules/no-ambiguity-target/no-ambiguity-target.md)
-- [effector/prefer-sample-over-forward-with-mapping](/rules/prefer-sample-over-forward-with-mapping/prefer-sample-over-forward-with-mapping.md)
+- [effector/enforce-store-naming-convention](rules/enforce-store-naming-convention/enforce-store-naming-convention.md)
+- [effector/enforce-effect-naming-convention](rules/enforce-effect-naming-convention/enforce-effect-naming-convention.md)
+- [effector/no-getState](rules/no-getState/no-getState.md)
+- [effector/no-unnecessary-duplication](rules/no-unnecessary-duplication/no-unnecessary-duplication.md)
+- [effector/no-useless-methods](rules/no-useless-methods/no-useless-methods.md)
+- [effector/no-ambiguity-target](rules/no-ambiguity-target/no-ambiguity-target.md)
+- [effector/prefer-sample-over-forward-with-mapping](rules/prefer-sample-over-forward-with-mapping/prefer-sample-over-forward-with-mapping.md)
+- [effector/no-watch](rules/no-watch/no-watch.md)
+- [effector/no-unnecessary-combination](rules/no-unnecessary-combination/no-unnecessary-combination.md)
+
+## Maintenance
+
+### Release flow
+
+1. Bump `version` in [package.json](package.json)
+2. Fill [CHANGELOG.md](CHANGELOG.md)
+3. Commit changes by `git commin -m "Release X.X.X"`
+4. Create git tag for release by `git tag -a vX.X.X -m "vX.X.X"`
+5. Push changes to remote by `git push --follow-tags`
+6. Release package to registry by `yarn clean-publish`
+7. Fill release page with changelog on GitHub

package/config/recommended.js

@@ -7,5 +7,7 @@ module.exports = {
     "effector/no-unnecessary-duplication": "warn",
     "effector/prefer-sample-over-forward-with-mapping": "warn",
     "effector/no-ambiguity-target": "warn",
+    "effector/no-watch": "warn",
+    "effector/no-unnecessary-combination": "warn",
   },
 };

package/index.js

@@ -7,8 +7,10 @@ module.exports = {
     "prefer-sample-over-forward-with-mapping": require("./rules/prefer-sample-over-forward-with-mapping/prefer-sample-over-forward-with-mapping"),
     "no-useless-methods": require("./rules/no-useless-methods/no-useless-methods"),
     "no-ambiguity-target": require("./rules/no-ambiguity-target/no-ambiguity-target"),
+    "no-watch": require("./rules/no-watch/no-watch"),
+    "no-unnecessary-combination": require("./rules/no-unnecessary-combination/no-unnecessary-combination"),
   },
   configs: {
     recommended: require("./config/recommended"),
-  }
+  },
 };

package/package.json

@@ -1,33 +1,28 @@
 {
   "name": "eslint-plugin-effector",
-  "version": "0.3.0",
+  "version": "0.4.2",
   "description": "Enforcing best practices for Effector",
+  "keywords": [
+    "eslint",
+    "eslint-plugin",
+    "eslintplugin",
+    "effector"
+  ],
+  "repository": "effector/eslint-plugin",
   "main": "index.js",
   "author": "Igor Kamyshev <igor@kamyshev.me>",
   "license": "MIT",
-  "scripts": {
-    "test": "jest"
-  },
   "publishConfig": {
     "access": "public"
   },
   "engines": {
     "node": "^14 || ^16"
   },
-  "devDependencies": {
-    "@typescript-eslint/experimental-utils": "^4.29.3",
-    "@typescript-eslint/parser": "^4.29.3",
-    "effector": "^22.0.0",
-    "eslint": "^7.32.0",
-    "glob": "^7.1.7",
-    "jest": "^27.1.0",
-    "typescript": "^4.4.2"
-  },
   "peerDependencies": {
     "effector": "*",
-    "eslint": "*"
+    "eslint": "7 || 8"
   },
   "dependencies": {
     "prettier": "^2.3.2"
   }
-}
+}

package/rules/enforce-effect-naming-convention/enforce-effect-naming-convention.js

@@ -1,6 +1,7 @@
 const {
   extractImportedFromEffector,
 } = require("../../utils/extract-imported-from-effector");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -10,6 +11,7 @@ module.exports = {
         "Enforce Fx as a suffix for any effect created by Effector methods",
       category: "Naming",
       recommended: true,
+      url: createLinkToRule("enforce-effect-naming-convention"),
     },
     messages: {
       invalidName:
@@ -17,6 +19,7 @@ module.exports = {
       renameEffect: 'Rename "{{ effectName }}" to "{{ effectName }}Fx"',
     },
     schema: [],
+    hasSuggestions: true,
   },
   create(context) {
     const parserServices = context.parserServices;

package/rules/enforce-store-naming-convention/enforce-store-naming-convention.js

@@ -2,25 +2,34 @@ const {
   extractImportedFromEffector,
 } = require("../../utils/extract-imported-from-effector");
 const { isStoreNameValid } = require("../../utils/is-store-name-valid");
-const { validateStoreNameConvention } = require("../../utils/validate-store-name-convention");
-const { getStoreNameConvention } = require("../../utils/get-store-name-convention");
-const { getCorrectedStoreName } = require("../../utils/get-corrected-store-name");
+const {
+  validateStoreNameConvention,
+} = require("../../utils/validate-store-name-convention");
+const {
+  getStoreNameConvention,
+} = require("../../utils/get-store-name-convention");
+const {
+  getCorrectedStoreName,
+} = require("../../utils/get-corrected-store-name");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
     type: "problem",
     docs: {
       description:
-          "Enforce $ as a prefix or postfix for any store created by Effector methods",
+        "Enforce $ as a prefix or postfix for any store created by Effector methods",
       category: "Naming",
       recommended: true,
+      url: createLinkToRule("enforce-store-naming-convention"),
     },
     messages: {
       invalidName:
-          'Store "{{ storeName }}" should be named with {{ storeNameConvention }}, rename it to "{{ correctedStoreName }}"',
+        'Store "{{ storeName }}" should be named with {{ storeNameConvention }}, rename it to "{{ correctedStoreName }}"',
       renameStore: 'Rename "{{ storeName }}" to "{{ correctedStoreName }}"',
     },
     schema: [],
+    hasSuggestions: true,
   },
   create(context) {
     const { parserServices } = context;
@@ -36,8 +45,8 @@ module.exports = {
           const type = checker.getTypeAtLocation(originalNode.initializer);
 
           const isEffectorStore =
-              type?.symbol?.escapedName === "Store" &&
-              type?.symbol?.parent?.escapedName?.includes("effector");
+            type?.symbol?.escapedName === "Store" &&
+            type?.symbol?.parent?.escapedName?.includes("effector");
 
           if (!isEffectorStore) {
             return;
@@ -52,7 +61,7 @@ module.exports = {
           reportStoreNameConventionViolation({
             context,
             node,
-            storeName
+            storeName,
           });
         },
       };
@@ -79,7 +88,7 @@ module.exports = {
           }
 
           const resultSavedInVariable =
-              node.parent.type === "VariableDeclarator";
+            node.parent.type === "VariableDeclarator";
           if (!resultSavedInVariable) {
             continue;
           }
@@ -93,7 +102,7 @@ module.exports = {
           reportStoreNameConventionViolation({
             context,
             node: node.parent,
-            storeName
+            storeName,
           });
           return;
         }
@@ -107,7 +116,7 @@ module.exports = {
           }
 
           const resultSavedInVariable =
-              node.parent.type === "VariableDeclarator";
+            node.parent.type === "VariableDeclarator";
           if (!resultSavedInVariable) {
             return;
           }
@@ -118,11 +127,10 @@ module.exports = {
             return;
           }
 
-
           reportStoreNameConventionViolation({
             context,
             node: node.parent,
-            storeName
+            storeName,
           });
           return;
         }
@@ -130,10 +138,10 @@ module.exports = {
         // Store creation in domain
         const STORE_IN_DOMAIN_CREATION_METHODS = ["createStore", "store"];
         if (
-            STORE_IN_DOMAIN_CREATION_METHODS.includes(node.callee?.property?.name)
+          STORE_IN_DOMAIN_CREATION_METHODS.includes(node.callee?.property?.name)
         ) {
           const resultSavedInVariable =
-              node.parent.type === "VariableDeclarator";
+            node.parent.type === "VariableDeclarator";
           if (!resultSavedInVariable) {
             return;
           }
@@ -147,7 +155,7 @@ module.exports = {
           reportStoreNameConventionViolation({
             context,
             node: node.parent,
-            storeName
+            storeName,
           });
           return;
         }
@@ -157,7 +165,6 @@ module.exports = {
 };
 
 function reportStoreNameConventionViolation({ context, node, storeName }) {
-
   const storeNameConvention = getStoreNameConvention(context);
   const correctedStoreName = getCorrectedStoreName(storeName, context);
 
@@ -167,7 +174,7 @@ function reportStoreNameConventionViolation({ context, node, storeName }) {
     data: {
       storeName,
       correctedStoreName,
-      storeNameConvention
+      storeNameConvention,
     },
     suggest: [
       {
@@ -180,4 +187,3 @@ function reportStoreNameConventionViolation({ context, node, storeName }) {
     ],
   });
 }
-

package/rules/enforce-store-naming-convention/enforce-store-naming-convention.md

@@ -17,7 +17,7 @@ Prefix convention will be enforced:
 const $name = createStore(null);
 
 // 👎 bad name
-const name = createStrore(null);
+const name = createStore(null);
 ```
 ## Postfix convention
 

package/rules/no-ambiguity-target/no-ambiguity-target.js

@@ -2,6 +2,7 @@ const {
   extractImportedFromEffector,
 } = require("../../utils/extract-imported-from-effector");
 const { traverseParentByType } = require("../../utils/traverse-parent-by-type");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -10,6 +11,7 @@ module.exports = {
       description: "Forbids ambiguity targets in `sample` and `guard`",
       category: "Quality",
       recommended: true,
+      url: createLinkToRule("no-ambiguity-target"),
     },
     messages: {
       ambiguityTarget:
@@ -46,7 +48,8 @@ module.exports = {
 
           const resultAssignedInVariable = traverseParentByType(
             node,
-            "VariableDeclarator"
+            "VariableDeclarator",
+            { stopOnTypes: ["BlockStatement"] }
           );
           const resultPartOfChain = traverseParentByType(
             node,

package/rules/no-getState/no-getState.js

@@ -2,6 +2,7 @@ const {
   traverseNestedObjectNode,
 } = require("../../utils/traverse-nested-object-node");
 const { isStoreNameValid } = require("../../utils/is-store-name-valid");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -10,6 +11,7 @@ module.exports = {
       description: "Forbids `.getState` calls on any Effector store",
       category: "Quality",
       recommended: true,
+      url: createLinkToRule("no-getState"),
     },
     messages: {
       abusiveCall:

package/rules/no-unnecessary-combination/no-unnecessary-combination.js

@@ -0,0 +1,94 @@
+const {
+  extractImportedFromEffector,
+} = require("../../utils/extract-imported-from-effector");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
+
+module.exports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description:
+        "Forbids unnecessary combinations in `clock`, `source` and `forward`",
+      category: "Quality",
+      recommended: true,
+      url: createLinkToRule("no-unnecessary-combination"),
+    },
+    messages: {
+      unnecessaryCombination:
+        "Method {{ methodName }} is used under the hood, you can omit it.",
+    },
+    schema: [],
+  },
+  create(context) {
+    const importedFromEffector = new Map();
+
+    return {
+      ImportDeclaration(node) {
+        extractImportedFromEffector(importedFromEffector, node);
+      },
+      CallExpression(node) {
+        const METHODS_WITH_POSSIBLE_UNNECESSARY_COMBINATION = [
+          "sample",
+          "guard",
+          "forward",
+        ];
+
+        const CONFIG_ARG_PROPERTIES = ["source", "clock", "from"];
+
+        function toLocalMethod(method) {
+          return importedFromEffector.get(method);
+        }
+
+        const UNNECESSARY_METHODS = {
+          source: ["combine", "merge"].map(toLocalMethod).filter(Boolean),
+          clock: ["merge"].map(toLocalMethod).filter(Boolean),
+          from: ["merge"].map(toLocalMethod).filter(Boolean),
+        };
+
+        for (const method of METHODS_WITH_POSSIBLE_UNNECESSARY_COMBINATION) {
+          const localMethod = importedFromEffector.get(method);
+          if (!localMethod) {
+            continue;
+          }
+
+          const isEffectorMethod = node?.callee?.name === localMethod;
+          if (!isEffectorMethod) {
+            continue;
+          }
+
+          const candidates =
+            node?.arguments?.[0]?.properties?.filter((n) =>
+              CONFIG_ARG_PROPERTIES.includes(n.key.name)
+            ) ?? [];
+
+          if (candidates.length === 0) {
+            continue;
+          }
+
+          for (const candidate of candidates) {
+            const candidateName = candidate?.value?.callee?.name;
+            const argProp = candidate?.key?.name;
+            if (!candidateName || !argProp) {
+              continue;
+            }
+
+            const localUnnecessaryMethods = UNNECESSARY_METHODS[argProp];
+
+            const UnnecessaryMethodIsEffectorMethod =
+              localUnnecessaryMethods.some((m) => m === candidateName);
+
+            if (!UnnecessaryMethodIsEffectorMethod) {
+              continue;
+            }
+
+            context.report({
+              node: candidate?.value,
+              messageId: "unnecessaryCombination",
+              data: { methodName: candidateName },
+            });
+          }
+        }
+      },
+    };
+  },
+};

package/rules/no-unnecessary-combination/no-unnecessary-combination.md

@@ -0,0 +1,14 @@
+# effector/no-unnecessary-combination
+
+Call of `combine`/`merge` in `clock`/`source` is unnecessary. It can be omitted from source code.
+
+```ts
+// 👎 can be simplified
+const badEventOne = guard({ clock: combine($store1, $store2), filter: $filter });
+const badEventOne = guard({ clock: combine($store1, $store2, (store1, store2) => ({x: store1, y: store2})), filter: $filter });
+
+// 👍 better
+const goodEventOne = guard({ clock: [$store1, $store2], filter: $filter });
+const goodEventTwo = guard({ clock: ({x: $store1, x: $store2}), filter: $filter });
+```
+

package/rules/no-unnecessary-duplication/no-unnecessary-duplication.js

@@ -2,6 +2,7 @@ const {
   extractImportedFromEffector,
 } = require("../../utils/extract-imported-from-effector");
 const { areNodesSameInText } = require("../../utils/are-nodes-same-in-text");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -10,6 +11,7 @@ module.exports = {
       description: "Forbids unnecessary duplication in `clock` and `source`",
       category: "Quality",
       recommended: true,
+      url: createLinkToRule("no-unnecessary-duplication"),
     },
     messages: {
       unnecessaryDuplication:
@@ -18,6 +20,7 @@ module.exports = {
       removeSource: "Remove `source`",
     },
     schema: [],
+    hasSuggestions: true,
   },
   create(context) {
     const importedFromEffector = new Map();

package/rules/no-useless-methods/no-useless-methods.js

@@ -2,6 +2,7 @@ const {
   extractImportedFromEffector,
 } = require("../../utils/extract-imported-from-effector");
 const { traverseParentByType } = require("../../utils/traverse-parent-by-type");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -10,6 +11,7 @@ module.exports = {
       description: "Forbids useless calls of `sample` and `guard`",
       category: "Quality",
       recommended: true,
+      url: createLinkToRule("no-useless-methods"),
     },
     messages: {
       uselessMethod:
@@ -68,6 +70,16 @@ module.exports = {
             continue;
           }
 
+          const resultIsWatched = node?.parent?.property?.name === "watch";
+          if (resultIsWatched) {
+            continue;
+          }
+
+          const resultIsArgument = node?.parent?.type === "CallExpression";
+          if (resultIsArgument) {
+            continue;
+          }
+
           context.report({
             node,
             messageId: "uselessMethod",

package/rules/no-watch/no-watch.js

@@ -0,0 +1,59 @@
+const {
+  traverseNestedObjectNode,
+} = require("../../utils/traverse-nested-object-node");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
+
+module.exports = {
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "Avoid `.watch` calls on any Effector unit or operator",
+      category: "Quality",
+      recommended: true,
+      url: createLinkToRule("no-watch"),
+    },
+    messages: {
+      abusiveCall:
+        "Method `.watch` leads to imperative code. Try to replace it with operators (`sample`, `guard`, etc) or use the `target` parameter of the operators.",
+    },
+    schema: [],
+  },
+  create(context) {
+    const { parserServices } = context;
+    if (!parserServices.hasFullTypeInformation) {
+      // JavaScript-way https://github.com/effector/eslint-plugin/issues/48#issuecomment-931107829
+      return {};
+    }
+    const checker = parserServices.program.getTypeChecker();
+
+    return {
+      CallExpression(node) {
+        const methodName = node.callee?.property?.name;
+        if (methodName !== "watch") {
+          return;
+        }
+
+        const object = traverseNestedObjectNode(node.callee?.object);
+        const originalNode = parserServices.esTreeNodeToTSNodeMap.get(object);
+        const type = checker.getTypeAtLocation(originalNode);
+
+        const isEffectorUnit =
+          ["Effect", "Event", "Store"].includes(type?.symbol?.escapedName) &&
+          type?.symbol?.parent?.escapedName?.includes("effector");
+
+        if (!isEffectorUnit) {
+          return;
+        }
+
+        reportWatchCall({ context, node });
+      },
+    };
+  },
+};
+
+function reportWatchCall({ context, node }) {
+  context.report({
+    node,
+    messageId: "abusiveCall",
+  });
+}

package/rules/no-watch/no-watch.md

@@ -0,0 +1,42 @@
+# effector/no-watch
+
+Method `.watch` leads to imperative code. Try replacing it with operators (`forward`, `sample`, etc) or use the `target` parameter of the operators.
+
+> Caution! This rule only works on projects using TypeScript.
+
+```ts
+const myFx = createEffect();
+const myEvent = createEvent();
+const $awesome = createStore();
+
+// 👍 good solutions
+forward({
+  from: myFx.finally,
+  to: myEvent,
+});
+
+guard({
+  clock: myEvent,
+  filter: Boolean,
+  target: myFx,
+});
+
+sample({
+  from: $awesome.updates,
+  fn: identity,
+  to: myEvent,
+});
+
+// 👎 bad solutions
+myFx.finally.watch(myEvent);
+
+myEvent.watch((payload) => {
+  if (Boolean(payload)) {
+    myFx(payload);
+  }
+});
+
+$awesome.updates.watch((data) => {
+  myEvent(identity(data));
+});
+```

package/rules/prefer-sample-over-forward-with-mapping/prefer-sample-over-forward-with-mapping.js

@@ -4,6 +4,7 @@ const {
 const {
   traverseNestedObjectNode,
 } = require("../../utils/traverse-nested-object-node");
+const { createLinkToRule } = require("../../utils/create-link-to-rule");
 
 module.exports = {
   meta: {
@@ -12,6 +13,7 @@ module.exports = {
       description: "Prefer `sample` over `forward` with `.map`/`.prepend`",
       category: "Quality",
       recommended: true,
+      url: createLinkToRule("prefer-sample-over-forward-with-mapping"),
     },
     messages: {
       overMap:

package/rules/tsconfig.json

@@ -5,5 +5,5 @@
     "lib": ["es2017", "es2019"],
     "baseUrl": "./"
   },
-  "include": ["./**/examples/*.ts"]
+  "include": ["./**/examples/**/*.ts"]
 }

package/utils/create-link-to-rule.js

@@ -0,0 +1,5 @@
+function createLinkToRule(name) {
+  return `https://github.com/effector/eslint-plugin/blob/master/rules/${name}/${name}.md`;
+}
+
+module.exports = { createLinkToRule };

package/utils/traverse-parent-by-type.js

@@ -1,5 +1,7 @@
-function traverseParentByType(node, type) {
-  if (!node) {
+function traverseParentByType(node, type, config) {
+  const stopOnTypes = config?.stopOnTypes ?? [];
+
+  if (!node || stopOnTypes.includes(node.type)) {
     return null;
   }
 
@@ -7,7 +9,7 @@ function traverseParentByType(node, type) {
     return node;
   }
 
-  return traverseParentByType(node.parent, type);
+  return traverseParentByType(node.parent, type, config);
 }
 
 module.exports = { traverseParentByType };

package/.github/workflows/ci.yml

@@ -1,43 +0,0 @@
-name: CI
-
-on:
-  push:
-    branches: [master]
-  pull_request:
-    branches: [master]
-
-jobs:
-  checks:
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [14.x, 16.x]
-
-    steps:
-      - uses: actions/checkout@v2
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v2
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: "yarn"
-      - run: yarn install
-      - run: yarn test --coverage
-
-  reports:
-    runs-on: ubuntu-latest
-
-    if: ${{ github.event_name == 'pull_request' }}
-
-    steps:
-      - uses: actions/checkout@v2
-      - name: Use Node.js
-        uses: actions/setup-node@v2
-        with:
-          cache: "yarn"
-      - run: yarn install
-      - run: yarn test --coverage
-      - name: Code Coverage Report
-        uses: romeovs/lcov-reporter-action@v0.2.11
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}

package/CHANGELOG.md

@@ -1,24 +0,0 @@
-# Changelog
-
-## v0.3.0
-
-- Add new rule: `no-useless-methods` ([PR #41](https://github.com/effector/eslint-plugin/pull/41))
-- Add new rule: `no-ambiguity-target` ([PR #42](https://github.com/effector/eslint-plugin/pull/42))
-- Add possibility to configure store's naming convention — suffix of prefix ([PR #37](https://github.com/effector/eslint-plugin/pull/37) by @ilyaryabchinski)
-
-## v0.2.0
-
-- Add tests against Effector 22
-- Specify supported Node.JS versions
-- Add new rule: `prefer-sample-over-forward-with-mapping` ([PR #34](https://github.com/igorkamyshev/eslint-plugin-effector/pull/34))
-
-## v0.1.4
-
-- Exclude test-coverage report from npm-package
-- Fixed SyntaxError in `no-unnecessary-duplication` suggestions ([PR #28](https://github.com/igorkamyshev/eslint-plugin-effector/pull/28))
-
-## v0.1.3
-
-- Fixed false-positive in `no-unnecessary-duplication` with composite `clock`/`source` ([PR #22](https://github.com/igorkamyshev/eslint-plugin-effector/pull/22))
-- Fixed TypeError in `enforce-store-naming-convention` ([PR #25](https://github.com/igorkamyshev/eslint-plugin-effector/pull/25))
-- Fixed false-positive in `enforce-effect-naming-convention` with `combine` ([PR #26](https://github.com/igorkamyshev/eslint-plugin-effector/pull/26))

package/jest.config.js

@@ -1,7 +0,0 @@
-module.exports = {
-  testRegex: "./.+\\.test\\.js$",
-  collectCoverage: false,
-  collectCoverageFrom: ["rules/**/{!(examples),}/*.js"],
-  moduleFileExtensions: ["js"],
-  coverageReporters: ["text-summary", "lcov"],
-};