@aarongoldenthal/eslint-config-standard 25.0.0 → 26.0.0

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2019 - 2023 Aaron Goldenthal
+Copyright (c) 2019 - 2024 Aaron Goldenthal
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.md

@@ -8,11 +8,11 @@ for `eslint`, `@eslint-community/eslint-plugin-eslint-comments` (formerly
 `eslint-plugin-n` (formerly `eslint-plugin-node`), `eslint-plugin-playwright`,
 `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
+`peerDependencies` so they're installed and don't 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:
+so they're only applicable to a subset of files:
 
 - `jest-config`, `vitest-config`: applicable to files matching
   `['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)']`,
@@ -39,7 +39,7 @@ configure .eslintrc.json with this configuration:
 }
 ```
 
-There is also a `recommended-vitest` configuration that is the same as the
+There is also a `recommended-vitest` configuration that's the same as the
 `recommended` config, but includes the `vitest-config` instead of the
 `jest-config`, and can be configured with:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aarongoldenthal/eslint-config-standard",
-  "version": "25.0.0",
+  "version": "26.0.0",
   "description": "Standard ESLint configuration settings",
   "main": "index.js",
   "scripts": {
@@ -38,20 +38,20 @@
     "eslint": "8.56.0",
     "jest": "29.7.0",
     "jest-junit": "16.0.0",
-    "markdownlint-cli2": "0.11.0",
-    "prettier": "3.1.1"
+    "markdownlint-cli2": "0.12.1",
+    "prettier": "3.2.5"
   },
   "peerDependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "4.1.0",
     "eslint": "^8.27.0",
     "eslint-config-prettier": "9.1.0",
-    "eslint-plugin-jest": "27.6.0",
-    "eslint-plugin-jsdoc": "46.9.1",
-    "eslint-plugin-n": "16.5.0",
-    "eslint-plugin-playwright": "0.20.0",
+    "eslint-plugin-jest": "27.9.0",
+    "eslint-plugin-jsdoc": "48.1.0",
+    "eslint-plugin-n": "16.6.2",
+    "eslint-plugin-playwright": "1.3.0",
     "eslint-plugin-promise": "6.1.1",
-    "eslint-plugin-sonarjs": "0.23.0",
-    "eslint-plugin-unicorn": "50.0.1",
-    "eslint-plugin-vitest": "0.3.18"
+    "eslint-plugin-sonarjs": "0.24.0",
+    "eslint-plugin-unicorn": "51.0.1",
+    "eslint-plugin-vitest": "0.3.22"
   }
 }

package/playwright-config.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const maxExpects = 5;
+
 module.exports = {
     overrides: [
         {
@@ -10,22 +12,36 @@ module.exports = {
             plugins: ['playwright'],
             rules: {
                 'playwright/expect-expect': 'error',
+                'playwright/max-expects': ['error', { max: maxExpects }],
                 'playwright/max-nested-describe': 'error',
                 'playwright/missing-playwright-await': 'error',
+                'playwright/no-commented-out-tests': 'error',
+                'playwright/no-conditional-expect': 'error',
                 'playwright/no-conditional-in-test': 'error',
+                'playwright/no-duplicate-hooks': 'error',
                 'playwright/no-element-handle': 'error',
                 'playwright/no-eval': 'error',
                 'playwright/no-focused-test': 'error',
                 'playwright/no-force-option': 'error',
+                'playwright/no-get-by-title': 'error',
+                'playwright/no-hooks': 'off',
                 'playwright/no-nested-step': 'error',
                 'playwright/no-networkidle': 'error',
                 'playwright/no-nth-methods': 'error',
                 'playwright/no-page-pause': 'error',
                 'playwright/no-raw-locators': 'error',
+                'playwright/no-restricted-matchers': 'off',
                 'playwright/no-skipped-test': 'error',
+                'playwright/no-standalone-expect': 'error',
+                'playwright/no-unsafe-references': 'error',
                 'playwright/no-useless-await': 'error',
                 'playwright/no-useless-not': 'error',
+                'playwright/no-wait-for-selector': 'error',
                 'playwright/no-wait-for-timeout': 'error',
+                'playwright/prefer-comparison-matcher': 'error',
+                'playwright/prefer-equality-matcher': 'error',
+                'playwright/prefer-hooks-in-order': 'error',
+                'playwright/prefer-hooks-on-top': 'error',
                 'playwright/prefer-lowercase-title': 'error',
                 'playwright/prefer-strict-equal': 'error',
                 'playwright/prefer-to-be': 'error',
@@ -33,9 +49,11 @@ module.exports = {
                 'playwright/prefer-to-have-count': 'error',
                 'playwright/prefer-to-have-length': 'error',
                 'playwright/prefer-web-first-assertions': 'error',
+                'playwright/require-hook': 'error',
                 'playwright/require-soft-assertions': 'off',
                 'playwright/require-top-level-describe': 'error',
-                'playwright/valid-expect': 'error'
+                'playwright/valid-expect': 'error',
+                'playwright/valid-title': 'error'
             }
         }
     ]

package/unicorn-config.js

@@ -15,7 +15,7 @@ module.exports = {
     rules: {
         'unicorn/better-regex': 'error', // Recommended
         'unicorn/catch-error-name': 'error', // Recommended
-        'unicorn/consistent-destructuring': 'error', // Recommended
+        'unicorn/consistent-destructuring': 'error',
         'unicorn/consistent-function-scoping': 'error', // Recommended
         'unicorn/custom-error-definition': 'error',
         'unicorn/empty-brace-spaces': 'off', // Recommended, Prettier