eslint-config-seek 11.1.1 → 11.1.3

package/base.js

@@ -4,6 +4,9 @@ const root = require('find-root')(process.cwd());
 const OFF = 0;
 const ERROR = 2;
 
+const rulesDirPlugin = require('eslint-plugin-rulesdir');
+rulesDirPlugin.RULES_DIR = path.join(__dirname, 'rules');
+
 const baseRules = {
   // Possible Errors
   'no-console': ERROR,
@@ -210,11 +213,17 @@ const baseConfig = {
     {
       // Cypress config
       files: [`**/cypress/**/*.{${allExtensions}}`],
+      // eslint-plugin-cypress doesn't support ESLint v8.
+      // Use fork by `@finsit` until this is solved.
+      // https://github.com/cypress-io/eslint-plugin-cypress/issues/89
+      extends: ['plugin:@finsit/cypress/recommended'],
       env: {
-        'cypress/globals': true,
+        '@finsit/cypress/globals': true,
+      },
+      plugins: ['@finsit/cypress', 'rulesdir'],
+      rules: {
+        'rulesdir/unsafe-to-chain-command': ERROR,
       },
-      extends: ['plugin:cypress/recommended'],
-      plugins: ['cypress'],
     },
   ],
 };

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "eslint-config-seek",
-  "version": "11.1.1",
+  "version": "11.1.3",
   "description": "ESLint configuration used by SEEK",
   "main": "index.js",
   "files": [
     "index.js",
     "base.js",
-    "extensions.js"
+    "extensions.js",
+    "rules/*"
   ],
   "repository": {
     "type": "git",
@@ -22,15 +23,16 @@
     "@babel/core": "^7.21.0",
     "@babel/eslint-parser": "^7.19.1",
     "@babel/preset-react": "^7.18.6",
+    "@finsit/eslint-plugin-cypress": "^3.1.1",
     "@typescript-eslint/eslint-plugin": "^5.53.0",
     "@typescript-eslint/parser": "^5.53.0",
     "eslint-config-prettier": "^8.6.0",
     "eslint-import-resolver-typescript": "3.5.3",
-    "eslint-plugin-cypress": "^2.12.1",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.2.1",
     "eslint-plugin-react": "^7.32.2",
     "eslint-plugin-react-hooks": "^4.6.0",
+    "eslint-plugin-rulesdir": "^0.2.2",
     "find-root": "^1.1.0"
   },
   "devDependencies": {
@@ -44,7 +46,7 @@
     "eslint": ">=6",
     "typescript": ">=4.5"
   },
-  "packageManager": "pnpm@7.27.1",
+  "packageManager": "pnpm@8.5.1",
   "volta": {
     "node": "16.19.1"
   },

package/rules/unsafe-to-chain-command.js

@@ -0,0 +1,88 @@
+/** This rule is copied from the original `eslint-plugin-cypress` so we can use the fork (which
+ * supports eslint 8) while having the same recommended rules as the upstream
+ * https://github.com/foretagsplatsen/eslint-plugin-cypress
+ * https://github.com/cypress-io/eslint-plugin-cypress/blob/c626ad543f65babf1def5caabd1bc9bb9900d2c7/lib/rules/unsafe-to-chain-command.js
+ */
+// eslint-disable-next-line strict
+'use strict';
+
+module.exports = {
+  meta: {
+    docs: {
+      description: 'Actions should be in the end of chains, not in the middle',
+      category: 'Possible Errors',
+      recommended: true,
+      url: 'https://docs.cypress.io/guides/core-concepts/retry-ability#Actions-should-be-at-the-end-of-chains-not-the-middle',
+    },
+    schema: [],
+    messages: {
+      unexpected:
+        'It is unsafe to chain further commands that rely on the subject after this command. It is best to split the chain, chaining again from `cy.` in a next command line.',
+    },
+  },
+  create(context) {
+    return {
+      CallExpression(node) {
+        if (
+          isRootCypress(node) &&
+          isActionUnsafeToChain(node) &&
+          node.parent.type === 'MemberExpression'
+        ) {
+          context.report({ node, messageId: 'unexpected' });
+        }
+      },
+    };
+  },
+};
+
+function isRootCypress(node) {
+  while (node.type === 'CallExpression') {
+    if (node.callee.type !== 'MemberExpression') {
+      return false;
+    }
+
+    if (
+      node.callee.object.type === 'Identifier' &&
+      node.callee.object.name === 'cy'
+    ) {
+      return true;
+    }
+
+    // eslint-disable-next-line no-param-reassign
+    node = node.callee.object;
+  }
+
+  return false;
+}
+
+function isActionUnsafeToChain(node) {
+  // commands listed in the documentation with text: 'It is unsafe to chain further commands that rely on the subject after xxx'
+  const unsafeToChainActions = [
+    'blur',
+    'clear',
+    'click',
+    'check',
+    'dblclick',
+    'each',
+    'focus',
+    'rightclick',
+    'screenshot',
+    'scrollIntoView',
+    'scrollTo',
+    'select',
+    'selectFile',
+    'spread',
+    'submit',
+    'type',
+    'trigger',
+    'uncheck',
+    'within',
+  ];
+
+  return (
+    node.callee &&
+    node.callee.property &&
+    node.callee.property.type === 'Identifier' &&
+    unsafeToChainActions.includes(node.callee.property.name)
+  );
+}