@aarongoldenthal/eslint-config-standard 23.1.0 → 24.0.0

package/README.md

@@ -6,14 +6,16 @@ Custom standard ESLint configuration for all projects. Includes configurations
 for `eslint`, `@eslint-community/eslint-plugin-eslint-comments` (formerly
 `eslint-plugin-eslint-comments`), `eslint-plugin-jest`, `eslint-plugin-jsdoc`,
 `eslint-plugin-n` (formerly `eslint-plugin-node`), `eslint-plugin-playwright`,
-`eslint-plugin-promise`, `eslint-plugin-sonarjs`, and `eslint-plugin-unicorn`.
-This package defines all required configurations as `dependencies` so they are
-installed and do not have to be defined in each project.
+`eslint-plugin-promise`, `eslint-plugin-sonarjs`, `eslint-plugin-unicorn`, and
+`eslint-plugin-vitest`. This package defines all required configurations as
+`peerDependencies` so they are installed and do not have to be defined in each
+project.
 
 Per recommended best practices, the following configurations include `overrides`
 so they are only applicable to a subset of files:
 
-- `jest-config`: applicable to files matching `['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)']`,
+- `jest-config`, `vitest-config`: applicable to files matching
+  `['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)']`,
   which is the [default filter for Jest test files](https://jestjs.io/docs/configuration#testmatch-arraystring).
 - `playwright-config`: applicable to files matching `['**/*.pwtest.[jt]s']`,
   which differentiates them from Jest test files.

package/index.js

@@ -203,9 +203,10 @@ module.exports = {
         'no-nested-ternary': 'off',
         'no-new': 'error',
         'no-new-func': 'error',
-        'no-new-object': 'error',
+        'no-new-object': 'off',
         'no-new-wrappers': 'error',
         'no-nonoctal-decimal-escape': 'error', // Recommended
+        'no-object-constructor': 'error',
         'no-octal': 'error', // Recommended
         'no-octal-escape': 'error',
         'no-param-reassign': ['error', { props: false }],
@@ -219,7 +220,7 @@ module.exports = {
         'no-restricted-properties': 'off',
         'no-restricted-syntax': 'off',
         'no-return-assign': 'error',
-        'no-return-await': 'error',
+        'no-return-await': 'off',
         'no-script-url': 'error',
         'no-sequences': ['error', { allowInParentheses: false }],
         'no-shadow': 'error',

package/package.json

@@ -1,16 +1,16 @@
 {
   "name": "@aarongoldenthal/eslint-config-standard",
-  "version": "23.1.0",
+  "version": "24.0.0",
   "description": "Standard ESLint configuration settings",
   "main": "index.js",
   "scripts": {
-    "hooks-pre-commit": "npm run lint && npm run prettier-check",
-    "hooks-pre-push": "npm audit --audit-level=high && npm test",
-    "lint": "npm run lint-js && npm run lint-md",
-    "lint-js": "eslint .",
-    "lint-md": "markdownlint-cli2 \"**/*.md\" \"#node_modules\"",
-    "prettier-check": "prettier --check .",
-    "prettier-fix": "prettier --write .",
+    "hooks:pre-commit": "npm run lint && npm run prettier:check",
+    "hooks:pre-push": "npm audit --audit-level=high && npm test",
+    "lint": "npm run lint:js && npm run lint:md",
+    "lint:js": "eslint .",
+    "lint:md": "markdownlint-cli2 \"**/*.md\" \"#node_modules\"",
+    "prettier:check": "prettier --check .",
+    "prettier:fix": "prettier --write .",
     "test": "jest --ci"
   },
   "repository": {
@@ -23,7 +23,7 @@
   "author": "Aaron Goldenthal <npm@aarongoldenthal.com>",
   "license": "MIT",
   "engines": {
-    "node": "^16.13.0 || ^18.12.0 || >=20.0.0"
+    "node": "^18.12.0 || >=20.0.0"
   },
   "files": [
     "index.js",
@@ -35,23 +35,23 @@
   },
   "homepage": "https://gitlab.com/gitlab-ci-utils/eslint-config-standard",
   "devDependencies": {
-    "eslint": "8.45.0",
-    "jest": "29.6.1",
+    "eslint": "8.50.0",
+    "jest": "29.7.0",
     "jest-junit": "16.0.0",
-    "markdownlint-cli2": "0.8.1",
-    "prettier": "3.0.0"
+    "markdownlint-cli2": "0.10.0",
+    "prettier": "3.0.3"
   },
   "peerDependencies": {
-    "@eslint-community/eslint-plugin-eslint-comments": "3.2.1",
+    "@eslint-community/eslint-plugin-eslint-comments": "4.1.0",
     "eslint": "^8.27.0",
-    "eslint-config-prettier": "8.8.0",
-    "eslint-plugin-jest": "27.2.3",
-    "eslint-plugin-jsdoc": "46.4.4",
-    "eslint-plugin-n": "16.0.1",
-    "eslint-plugin-playwright": "0.15.3",
+    "eslint-config-prettier": "9.0.0",
+    "eslint-plugin-jest": "27.4.2",
+    "eslint-plugin-jsdoc": "46.8.2",
+    "eslint-plugin-n": "16.1.0",
+    "eslint-plugin-playwright": "0.16.0",
     "eslint-plugin-promise": "6.1.1",
-    "eslint-plugin-sonarjs": "0.19.0",
+    "eslint-plugin-sonarjs": "0.21.0",
     "eslint-plugin-unicorn": "48.0.1",
-    "eslint-plugin-vitest": "0.2.6"
+    "eslint-plugin-vitest": "0.3.1"
   }
 }

package/playwright-config.js

@@ -21,6 +21,7 @@ module.exports = {
                 'playwright/no-networkidle': 'error',
                 'playwright/no-nth-methods': 'error',
                 'playwright/no-page-pause': 'error',
+                'playwright/no-raw-locators': 'error',
                 'playwright/no-skipped-test': 'error',
                 'playwright/no-useless-await': 'error',
                 'playwright/no-useless-not': 'error',

package/sonarjs-config.js

@@ -13,7 +13,10 @@ module.exports = {
         'sonarjs/no-all-duplicated-branches': 'error',
         'sonarjs/no-collapsible-if': 'error',
         'sonarjs/no-collection-size-mischeck': 'error',
-        'sonarjs/no-duplicate-string': ['error', noDuplicateStringThreshold],
+        'sonarjs/no-duplicate-string': [
+            'error',
+            { threshold: noDuplicateStringThreshold }
+        ],
         'sonarjs/no-duplicated-branches': 'error',
         'sonarjs/no-element-overwrite': 'error',
         'sonarjs/no-empty-collection': 'error',

package/unicorn-config.js

@@ -24,12 +24,12 @@ module.exports = {
         'unicorn/expiring-todo-comments': 'off', // Recommended
         'unicorn/explicit-length-check': 'error', // Recommended
         'unicorn/filename-case': ['error', { case: 'kebabCase' }], // Recommended
-        'unicorn/import-style': 'off', // Recommended
+        'unicorn/import-style': 'error', // Recommended
         'unicorn/new-for-builtins': 'error', // Recommended
         'unicorn/no-abusive-eslint-disable': 'error', // Recommended
         'unicorn/no-array-callback-reference': 'error', // Recommended
         'unicorn/no-array-for-each': 'error', // Recommended
-        'unicorn/no-array-method-this-argument': 'off', // Recommended
+        'unicorn/no-array-method-this-argument': 'error', // Recommended
         'unicorn/no-array-push-push': 'error', // Recommended
         'unicorn/no-array-reduce': 'error', // Recommended
         'unicorn/no-await-expression-member': 'error', // Recommended
@@ -45,7 +45,7 @@ module.exports = {
         'unicorn/no-negated-condition': 'error', // Recommended
         'unicorn/no-nested-ternary': 'error', // Recommended, Prettier
         'unicorn/no-new-array': 'error', // Recommended
-        'unicorn/no-new-buffer': 'off', // Recommended
+        'unicorn/no-new-buffer': 'error', // Recommended
         'unicorn/no-null': 'error', // Recommended
         'unicorn/no-object-as-default-parameter': 'error', // Recommended
         'unicorn/no-process-exit': 'error', // Recommended
@@ -82,7 +82,7 @@ module.exports = {
         'unicorn/prefer-dom-node-remove': 'error', // Recommended
         'unicorn/prefer-dom-node-text-content': 'error', // Recommended
         'unicorn/prefer-event-target': 'error', // Recommended
-        'unicorn/prefer-export-from': 'off', // Recommended
+        'unicorn/prefer-export-from': 'error', // Recommended
         'unicorn/prefer-includes': 'error', // Recommended
         'unicorn/prefer-json-parse-buffer': 'off',
         'unicorn/prefer-keyboard-event-key': 'error', // Recommended
@@ -93,13 +93,13 @@ module.exports = {
         'unicorn/prefer-module': 'off', // Recommended
         'unicorn/prefer-native-coercion-functions': 'error', // Recommended
         'unicorn/prefer-negative-index': 'error', // Recommended
-        'unicorn/prefer-node-protocol': 'off', // Recommended
+        'unicorn/prefer-node-protocol': 'error', // Recommended
         'unicorn/prefer-number-properties': 'error', // Recommended
         'unicorn/prefer-object-from-entries': 'off', // Recommended
         'unicorn/prefer-optional-catch-binding': 'error', // Recommended
-        'unicorn/prefer-prototype-methods': 'off', // Recommended
+        'unicorn/prefer-prototype-methods': 'error', // Recommended
         'unicorn/prefer-query-selector': 'error', // Recommended
-        'unicorn/prefer-reflect-apply': 'off', // Recommended
+        'unicorn/prefer-reflect-apply': 'error', // Recommended
         'unicorn/prefer-regexp-test': 'error', // Recommended
         'unicorn/prefer-set-has': 'error', // Recommended
         'unicorn/prefer-set-size': 'error', // Recommended
@@ -129,7 +129,7 @@ module.exports = {
         'unicorn/require-post-message-target-origin': 'off',
         'unicorn/string-content': 'off',
         'unicorn/switch-case-braces': ['error', 'always'], // Recommended
-        'unicorn/template-indent': 'off', // Recommended
+        'unicorn/template-indent': 'error', // Recommended
         'unicorn/text-encoding-identifier-case': 'error', // Recommended
         'unicorn/throw-new-error': 'error' // Recommended
     }