@wistia/eslint-config 2.6.0 → 2.7.0

package/.yarnrc.yml

@@ -1,7 +1,12 @@
+approvedGitRepositories:
+  - "**"
+
 enableGlobalCache: false
 
+enableScripts: true
+
 nodeLinker: node-modules
 
-yarnPath: .yarn/releases/yarn-4.13.0.cjs
+npmMinimalAgeGate: 7d
 
-npmMinimalAgeGate: '7d'
+yarnPath: .yarn/releases/yarn-4.14.1.cjs

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @wistia/eslint-config
 
+## 2.7.0
+
+### Minor Changes
+
+- [#468](https://github.com/wistia/eslint-config/pull/468) [`9d23cc3`](https://github.com/wistia/eslint-config/commit/9d23cc3dbcdeea1454e5a13d851313f1106dc11c) Thanks [@okize](https://github.com/okize)! - feat: remove `react-ssr-compatibility` config & rules
+
 ## 2.6.0
 
 ### Minor Changes

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "description": "Wistia's ESLint configurations",
-  "packageManager": "yarn@4.13.0",
+  "packageManager": "yarn@4.14.1",
   "type": "module",
   "main": "index.mjs",
   "engines": {
@@ -16,7 +16,6 @@
     "./node": "./src/configs/node.mjs",
     "./playwright": "./src/configs/playwright.mjs",
     "./react": "./src/configs/react.mjs",
-    "./react-ssr-compatibility": "./src/configs/react-ssr-compatibility.mjs",
     "./storybook": "./src/configs/storybook.mjs",
     "./styled-components": "./src/configs/styled-components.mjs",
     "./testing-library": "./src/configs/testing-library.mjs",
@@ -43,9 +42,9 @@
     "@eslint/compat": "^2.0.5",
     "@eslint/js": "^10.0.1",
     "@stylistic/eslint-plugin": "^5.10.0",
-    "@typescript-eslint/eslint-plugin": "^8.58.2",
-    "@typescript-eslint/parser": "^8.58.2",
-    "@vitest/eslint-plugin": "^1.6.15",
+    "@typescript-eslint/eslint-plugin": "^8.59.0",
+    "@typescript-eslint/parser": "^8.59.0",
+    "@vitest/eslint-plugin": "^1.6.16",
     "confusing-browser-globals": "^1.0.11",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-barrel-files": "^3.0.1",
@@ -57,31 +56,28 @@
     "eslint-plugin-jsx-a11y": "^6.10.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.10.1",
+    "eslint-plugin-playwright": "^2.10.2",
     "eslint-plugin-prettier": "^5.5.5",
     "eslint-plugin-promise": "^7.2.1",
-    "eslint-plugin-react-hooks-ssr": "^0.1.5",
-    "eslint-plugin-ssr-friendly": "^1.3.0",
     "eslint-plugin-storybook": "^10.3.5",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
     "eslint-plugin-testing-library": "^7.16.2",
     "globals": "^17.5.0",
-    "globals-vitest": "^4.1.4"
+    "globals-vitest": "^4.1.5"
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.6.0",
-    "@changesets/cli": "^2.30.0",
+    "@changesets/cli": "^2.31.0",
     "@commitlint/cli": "^20.5.0",
     "@commitlint/config-conventional": "^20.5.0",
     "check-export-map": "^1.3.1",
-    "eslint": "^10.2.0",
+    "eslint": "^10.2.1",
     "glob": "^13.0.6",
     "husky": "^9.1.7",
-    "prettier": "^3.8.2",
+    "prettier": "^3.8.3",
     "storybook": "^10.3.5",
-    "typescript": "^6.0.2",
-    "vitest": "^4.1.4"
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.5"
   },
   "peerDependencies": {
     "eslint": ">= 10",

package/test/__snapshots__/javascript.mjs.snap

@@ -64,7 +64,7 @@
       "undefined": false,
       "unescape": false,
     },
-    "parser": "typescript-eslint/parser@8.58.2",
+    "parser": "typescript-eslint/parser@8.59.0",
     "parserOptions": {
       "requireConfigFile": false,
     },

package/test/__snapshots__/typescript.mjs.snap

@@ -64,7 +64,7 @@
       "undefined": false,
       "unescape": false,
     },
-    "parser": "typescript-eslint/parser@8.58.2",
+    "parser": "typescript-eslint/parser@8.59.0",
     "parserOptions": {
       "ecmaVersion": 2024,
       "project": "./tsconfig.json",
@@ -78,7 +78,7 @@
   "plugins": [
     "@",
     "@stylistic:@stylistic/eslint-plugin@5.10.0",
-    "@typescript-eslint:@typescript-eslint/eslint-plugin@8.58.2",
+    "@typescript-eslint:@typescript-eslint/eslint-plugin@8.59.0",
     "barrel-files:eslint-plugin-barrel-files@3.0.1",
     "filenames",
     "import-x:eslint-plugin-import-x@4.16.2",

package/test/__snapshots__/vitest.mjs.snap

@@ -35,7 +35,7 @@
   },
   "plugins": [
     "@",
-    "vitest:vitest@1.6.15",
+    "vitest:vitest@1.6.16",
     "no-only-tests",
   ],
   "rules": {

package/src/configs/react-ssr-compatibility.mjs

@@ -1,18 +0,0 @@
-import { fixupPluginRules } from '@eslint/compat';
-import ssrFriendlyPlugin from 'eslint-plugin-ssr-friendly';
-import reactHooksSSRPlugin from 'eslint-plugin-react-hooks-ssr';
-import noTypeofWindowUndefinedPlugin from 'eslint-plugin-no-typeof-window-undefined';
-import reactSSRCompatibilityRules from '../rules/react-ssr-compatibility.mjs';
-
-export default [
-  {
-    plugins: {
-      'ssr-friendly': fixupPluginRules(ssrFriendlyPlugin),
-      'react-hooks-ssr': fixupPluginRules(reactHooksSSRPlugin),
-      'no-typeof-window-undefined': fixupPluginRules(noTypeofWindowUndefinedPlugin),
-    },
-    rules: {
-      ...reactSSRCompatibilityRules,
-    },
-  },
-];

package/src/rules/react-ssr-compatibility.mjs

@@ -1,35 +0,0 @@
-// only add rules related to maintaining SSR compatibility
-// see:
-// - https://github.com/kopiro/eslint-plugin-ssr-friendly
-// - https://github.com/correttojs/eslint-plugin-react-hooks-ssr
-// - https://github.com/nirtamir2/eslint-plugin-no-typeof-window-undefined
-
-export default {
-  // Disallow use of DOM globals in module and global scope, as this will break any import/require in a NodeJS environment.
-  // https://github.com/kopiro/eslint-plugin-ssr-friendly#no-dom-globals-in-module-scope
-  'ssr-friendly/no-dom-globals-in-module-scope': 'error',
-
-  // Disallow use of DOM globals in class constructors, as this will break SSR if you're instantiating this class as singleton or you're rendering this component.
-  // https://github.com/kopiro/eslint-plugin-ssr-friendly#no-dom-globals-in-constructor
-  'ssr-friendly/no-dom-globals-in-constructor': 'error',
-
-  // Disallow use of DOM globals in render() method of a React class-component, as this will break SSR if you're rendering this component.
-  // https://github.com/kopiro/eslint-plugin-ssr-friendly#no-dom-globals-in-react-cc-render
-  'ssr-friendly/no-dom-globals-in-react-cc-render': 'error',
-
-  // Disallow use of DOM globals in the render-cycle of a React FC, as this will break SSR if you're rendering this component.
-  // https://github.com/kopiro/eslint-plugin-ssr-friendly#no-dom-globals-in-react-fc
-  'ssr-friendly/no-dom-globals-in-react-fc': 'error',
-
-  // forbid DOM globals within react server side rendering
-  // a global within `useEffect` or custom hook is allowed
-  // a global within a function prefixed by `async` (eg. `asyncMyFunc`) is allowed
-  // a global within a `useState`, `useReducer` and `useMemo` callback is forbidden
-  // a global within a React `Component` is forbidden
-  'react-hooks-ssr/react-hooks-global-ssr': 'error',
-
-  // avoid checking for window since it is supported by Deno
-  // prefer `typeof document === "undefined"` instead
-  // https://github.com/nirtamir2/eslint-plugin-no-typeof-window-undefined#no-typeof-window-undefined
-  'no-typeof-window-undefined/no-typeof-window-undefined': 'error',
-};