@wistia/eslint-config 1.6.2 → 2.0.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @wistia/eslint-config
 
+## 2.0.0
+
+### Major Changes
+
+- [#435](https://github.com/wistia/eslint-config/pull/435) [`40f6fb3`](https://github.com/wistia/eslint-config/commit/40f6fb3473a990c7f8ce7ab153d5a33c1fa092c3) Thanks [@okize](https://github.com/okize)! - feat!: upgrade to eslint v10
+
+### Minor Changes
+
+- [#433](https://github.com/wistia/eslint-config/pull/433) [`2d98694`](https://github.com/wistia/eslint-config/commit/2d9869488b18d007f386e12754db7a0e71a7d5d8) Thanks [@okize](https://github.com/okize)! - feat: switch to the official vitest plugin
+
 ## 1.6.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "1.6.2",
+  "version": "2.0.0",
   "description": "Wistia's ESLint configurations",
   "packageManager": "yarn@4.12.0",
   "type": "module",
@@ -39,24 +39,25 @@
     "test:export": "check-export-map"
   },
   "dependencies": {
-    "@eslint/compat": "^2.0.1",
-    "@eslint/js": "^9.39.2",
-    "@stylistic/eslint-plugin": "^5.7.0",
-    "@typescript-eslint/eslint-plugin": "^8.53.1",
-    "@typescript-eslint/parser": "^8.53.1",
+    "@eslint/compat": "^2.0.3",
+    "@eslint/js": "^10.0.1",
+    "@stylistic/eslint-plugin": "^5.10.0",
+    "@typescript-eslint/eslint-plugin": "^8.57.1",
+    "@typescript-eslint/parser": "^8.57.1",
+    "@vitest/eslint-plugin": "^1.6.13",
     "confusing-browser-globals": "^1.0.11",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-barrel-files": "^3.0.1",
     "eslint-plugin-filenames": "^1.3.2",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jest": "^29.12.1",
+    "eslint-plugin-jest": "^29.15.0",
     "eslint-plugin-jest-dom": "^5.5.0",
     "eslint-plugin-jest-formatting": "^3.1.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-n": "^17.23.2",
+    "eslint-plugin-n": "^17.24.0",
     "eslint-plugin-no-only-tests": "^3.3.0",
     "eslint-plugin-no-typeof-window-undefined": "^0.0.2",
-    "eslint-plugin-playwright": "^2.5.0",
+    "eslint-plugin-playwright": "^2.10.1",
     "eslint-plugin-prettier": "^5.5.5",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-react": "^7.37.5",
@@ -64,30 +65,29 @@
     "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-react-hooks-ssr": "^0.1.5",
     "eslint-plugin-ssr-friendly": "^1.3.0",
-    "eslint-plugin-storybook": "^10.2.0",
+    "eslint-plugin-storybook": "^10.3.2",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
-    "eslint-plugin-testing-library": "^7.15.4",
-    "eslint-plugin-vitest": "^0.5.4",
-    "globals": "^17.0.0",
-    "globals-vitest": "^4.0.15"
+    "eslint-plugin-testing-library": "^7.16.1",
+    "globals": "^17.4.0",
+    "globals-vitest": "^4.1.0"
   },
   "devDependencies": {
-    "@changesets/changelog-github": "^0.5.2",
-    "@changesets/cli": "^2.29.8",
-    "@commitlint/cli": "^20.3.1",
-    "@commitlint/config-conventional": "^20.3.1",
+    "@changesets/changelog-github": "^0.6.0",
+    "@changesets/cli": "^2.30.0",
+    "@commitlint/cli": "^20.5.0",
+    "@commitlint/config-conventional": "^20.5.0",
     "check-export-map": "^1.3.1",
-    "eslint": "^9.39.2",
-    "glob": "^13.0.0",
+    "eslint": "^10.1.0",
+    "glob": "^13.0.6",
     "husky": "^9.1.7",
-    "prettier": "^3.8.0",
-    "storybook": "^10.2.0",
+    "prettier": "^3.8.1",
+    "storybook": "^10.3.2",
     "typescript": "^5.9.3",
-    "vitest": "^4.0.17"
+    "vitest": "^4.1.0"
   },
   "peerDependencies": {
-    "eslint": ">= 9",
-    "storybook": ">= 8",
+    "eslint": ">= 10",
+    "storybook": ">= 9",
     "typescript": ">= 5"
   },
   "peerDependenciesMeta": {

package/src/configs/vitest.mjs

@@ -1,5 +1,5 @@
 import globalsVitest from 'globals-vitest';
-import vitestPlugin from 'eslint-plugin-vitest';
+import vitestPlugin from '@vitest/eslint-plugin';
 import jestFormattingPlugin from 'eslint-plugin-jest-formatting';
 import noOnlyTestsPlugin from 'eslint-plugin-no-only-tests';
 import vitestRules from '../rules/vitest.mjs';

package/src/rules/vitest.mjs

@@ -1,5 +1,5 @@
 // only add vitest rules
-// see: https://github.com/veritem/eslint-plugin-vitest?tab=readme-ov-file#rules
+// see: https://github.com/vitest-dev/eslint-plugin-vitest?tab=readme-ov-file#rules
 
 export default {
   // Prevent focused tests (.only)
@@ -7,19 +7,19 @@ export default {
   'no-only-tests/no-only-tests': 'error',
 
   // Require .test test file pattern
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-filename.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-filename.md
   'vitest/consistent-test-filename': 'error',
 
   // Prefer test or it but not both
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-it.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-it.md
   'vitest/consistent-test-it': 'error',
 
   // Enforce having expectation in test body
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/expect-expect.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/expect-expect.md
   'vitest/expect-expect': 'error',
 
   // Enforce a maximum number of expect per test
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/max-expects.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/max-expects.md
   'vitest/max-expects': [
     'error',
     {
@@ -28,198 +28,198 @@ export default {
   ],
 
   // Nested describe block should be less than set max value or default value
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/max-nested-describe.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/max-nested-describe.md
   'vitest/max-nested-describe': 'error',
 
   // Disallow alias methods
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-alias-methods.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-alias-methods.md
   'vitest/no-alias-methods': 'error',
 
   // Disallow commented out tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-commented-out-tests.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-commented-out-tests.md
   'vitest/no-commented-out-tests': 'error',
 
   // Disallow conditional expects
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-expect.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-expect.md
   'vitest/no-conditional-expect': 'error',
 
   // Disallow conditional tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-in-test.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-in-test.md
   'vitest/no-conditional-in-test': 'error',
 
   // Disallow conditional tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-tests.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-tests.md
   'vitest/no-conditional-tests': 'error',
 
   // Disallow disabled tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-disabled-tests.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-disabled-tests.md
   // decision: it is often useful to be allowed to add a .skip
   'vitest/no-disabled-tests': 'off',
 
   // Disallow using a callback in asynchronous tests and hooks
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-done-callback.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-done-callback.md
   // decision: this rule is deprecated.
   // 'vitest/no-done-callback': 'error',
 
   // Disallow duplicate hooks and teardown hooks
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-duplicate-hooks.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-duplicate-hooks.md
   'vitest/no-duplicate-hooks': 'error',
 
   // Disallow focused tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-focused-tests.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-focused-tests.md
   'vitest/no-focused-tests': 'error',
 
   // Disallow setup and teardown hooks
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-hooks.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-hooks.md
   'vitest/no-hooks': 'off',
 
   // Disallow identical titles
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-identical-title.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-identical-title.md
   'vitest/no-identical-title': 'error',
 
   // Disallow importing node:test
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-import-node-test.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-import-node-test.md
   'vitest/no-import-node-test': 'error',
 
   // Disallow string interpolation in snapshots
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-interpolation-in-snapshots.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-interpolation-in-snapshots.md
   'vitest/no-interpolation-in-snapshots': 'error',
 
   // Disallow large snapshots
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-large-snapshots.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-large-snapshots.md
   'vitest/no-large-snapshots': ['error', { maxSize: 500 }],
 
   // Disallow importing from mocks directory
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-mocks-import.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-mocks-import.md
   'vitest/no-mocks-import': 'error',
 
   // Disallow the use of certain matchers
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-matchers.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-matchers.md
   'vitest/no-restricted-matchers': 'error',
 
   // Disallow specific vi. methods
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-vi-methods.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-vi-methods.md
   'vitest/no-restricted-vi-methods': 'error',
 
   // Disallow using expect outside of it or test blocks
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-standalone-expect.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-standalone-expect.md
   'vitest/no-standalone-expect': 'error',
 
   // Disallow using test as a prefix
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-test-prefixes.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-test-prefixes.md
   'vitest/no-test-prefixes': 'error',
 
   // Disallow return statements in tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-test-return-statement.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-test-return-statement.md
   'vitest/no-test-return-statement': 'error',
 
   // Suggest using toBeCalledWith() or toHaveBeenCalledWith()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-called-with.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-called-with.md
   'vitest/prefer-called-with': 'error',
 
   // Suggest using the built-in comparison matchers
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-comparison-matcher.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-comparison-matcher.md
   'vitest/prefer-comparison-matcher': 'error',
 
   // Prefer each rather than manual loops
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-each.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-each.md
   'vitest/prefer-each': 'error',
 
   // Suggest using the built-in quality matchers
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-equality-matcher.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-equality-matcher.md
   'vitest/prefer-equality-matcher': 'error',
 
   // Suggest using expect assertions instead of callbacks
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-assertions.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-assertions.md
   'vitest/prefer-expect-assertions': 'off',
 
   // Suggest using expect().resolves over expect(await ...) syntax
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-resolves.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-resolves.md
   'vitest/prefer-expect-resolves': 'error',
 
   // Prefer having hooks in consistent order
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-in-order.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-in-order.md
   'vitest/prefer-hooks-in-order': 'error',
 
   // Suggest having hooks before any test cases
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-on-top.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-on-top.md
   'vitest/prefer-hooks-on-top': 'error',
 
   // Enforce lowercase titles
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-lowercase-title.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-lowercase-title.md
   'vitest/prefer-lowercase-title': 'off',
 
   // Prefer mock resolved/rejected shorthands for promises
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-mock-promise-shorthand.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-mock-promise-shorthand.md
   'vitest/prefer-mock-promise-shorthand': 'error',
 
   // Prefer including a hint with external snapshots
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-snapshot-hint.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-snapshot-hint.md
   'vitest/prefer-snapshot-hint': 'error',
 
   // Suggest using vi.spyOn
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-spy-on.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-spy-on.md
   'vitest/prefer-spy-on': 'error',
 
   // Prefer strict equal over equal
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-strict-equal.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-strict-equal.md
   'vitest/prefer-strict-equal': 'error',
 
   // Suggest using toBe()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be.md
   'vitest/prefer-to-be': 'error',
 
   // Suggest using toBeFalsy()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-falsy.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-falsy.md
   // decision: not necessary to br prescriptive here
   'vitest/prefer-to-be-falsy': 'off',
 
   // Prefer toBeObject()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-object.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-object.md
   'vitest/prefer-to-be-object': 'error',
 
   // Suggest using toBeTruthy
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-truthy.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-truthy.md
   // decision: not necessary to br prescriptive here
   'vitest/prefer-to-be-truthy': 'off',
 
   // Prefer using toContain()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-contain.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-contain.md
   'vitest/prefer-to-contain': 'error',
 
   // Suggest using toHaveLength()
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-have-length.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-have-length.md
   'vitest/prefer-to-have-length': 'error',
 
   // Suggest using test.todo
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-todo.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/prefer-todo.md
   'vitest/prefer-todo': 'error',
 
   // Require setup and teardown to be within a hook
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-hook.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/require-hook.md
   'vitest/require-hook': 'error',
 
   // Require local Test Context for concurrent snapshot tests
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-local-test-context-for-concurrent-snapshots.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/require-local-test-context-for-concurrent-snapshots.md
   'vitest/require-local-test-context-for-concurrent-snapshots': 'error',
 
   // Require toThrow() to be called with an error message
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-to-throw-message.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/require-to-throw-message.md
   'vitest/require-to-throw-message': 'error',
 
   // Enforce that all tests are in a top-level describe
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-top-level-describe.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/require-top-level-describe.md
   'vitest/require-top-level-describe': 'error',
 
   // Enforce valid describe callback
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-describe-callback.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/valid-describe-callback.md
   'vitest/valid-describe-callback': 'error',
 
   // Enforce valid expect() usage
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-expect.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/valid-expect.md
   'vitest/valid-expect': 'error',
 
   // Enforce valid titles
-  // https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-title.md
+  // https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/valid-title.md
   'vitest/valid-title': 'error',
 };