sanity-io
diff --git a/‎.changeset/remove-forward-ref.md‎
Lines changed: 1 addition & 1 deletion b/‎.changeset/remove-forward-ref.md‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎.oxlintrc.json‎
Lines changed: 1 addition & 11 deletions b/‎.oxlintrc.json‎
Lines changed: 1 addition & 11 deletions
diff --git a/‎packages/ui/src/core/utils/errorBoundary.tsx‎
Lines changed: 0 additions & 1 deletion b/‎packages/ui/src/core/utils/errorBoundary.tsx‎
Lines changed: 0 additions & 1 deletion
@@ -2,4 +2,4 @@
 "@sanity/ui": minor
 ---
 
-All components are now plain function components that receive `ref` as a regular prop (React 19 semantics) instead of being wrapped in `forwardRef`. Refs keep working exactly as before, but the components no longer pass `react-is` checks like `isForwardRef`. The legacy React APIs (`forwardRef`, `createRef`, `createElement`, `Component`, `PureComponent`) are now banned via oxlint across the repo.
+All components are now plain function components that receive `ref` as a regular prop (React 19 semantics) instead of being wrapped in `forwardRef`. Refs keep working exactly as before, but the components no longer pass `react-is` checks like `isForwardRef`. The legacy React APIs (`forwardRef`, `createRef`, `createElement`) are banned via oxlint `no-restricted-imports`, and class components are banned via `react/prefer-function-component` (with `allowErrorBoundary` so the `ErrorBoundary` class remains valid).
@@ -57,11 +57,6 @@
             "name": "react",
             "importNames": ["createElement"],
             "message": "Use JSX instead of `createElement`."
-          },
-          {
-            "name": "react",
-            "importNames": ["Component", "PureComponent"],
-            "message": "Use function components with hooks instead of class components."
           }
         ]
       }
@@ -133,7 +128,7 @@
       // `^18 || ^19`), where refs only attach to function components wrapped in
       // `forwardRef` — so `forwardRef` stays allowed there (along with the
       // `createRef`/`createElement` used by the tests that pin that contract).
-      // Class components remain banned.
+      // Class components remain banned via the root `react/prefer-function-component`.
       "files": ["packages/icons/**", "packages/logos/**"],
       "rules": {
         "eslint/no-restricted-imports": [
@@ -144,11 +139,6 @@
                 "name": "styled-components",
                 "importNames": ["default"],
                 "message": "Please use `import {styled} from \"styled-components\"` instead."
-              },
-              {
-                "name": "react",
-                "importNames": ["Component", "PureComponent"],
-                "message": "Use function components with hooks instead of class components."
               }
             ]
           }
 
@@ -1,4 +1,3 @@
-// oxlint-disable-next-line no-restricted-imports -- error boundaries require a class component; React has no function-component equivalent for `getDerivedStateFromError`/`componentDidCatch`
 import {Component, PropsWithChildren} from 'react'
 
 /**
Original file line number	Diff line number	Diff line change
`@@ -57,11 +57,6 @@`
`57`	`57`	`"name": "react",`
`58`	`58`	`"importNames": ["createElement"],`
`59`	`59`	"message": "Use JSX instead of `createElement`."
`60`		`- },`
`61`		`- {`
`62`		`- "name": "react",`
`63`		`- "importNames": ["Component", "PureComponent"],`
`64`		`- "message": "Use function components with hooks instead of class components."`
`65`	`60`	`}`
`66`	`61`	`]`
`67`	`62`	`}`
`@@ -133,7 +128,7 @@`
`133`	`128`	// `^18 \|\| ^19`), where refs only attach to function components wrapped in
`134`	`129`	// `forwardRef` — so `forwardRef` stays allowed there (along with the
`135`	`130`	// `createRef`/`createElement` used by the tests that pin that contract).
`136`		`- // Class components remain banned.`
	`131`	+ // Class components remain banned via the root `react/prefer-function-component`.
`137`	`132`	`"files": ["packages/icons/", "packages/logos/"],`
`138`	`133`	`"rules": {`
`139`	`134`	`"eslint/no-restricted-imports": [`
`@@ -144,11 +139,6 @@`
`144`	`139`	`"name": "styled-components",`
`145`	`140`	`"importNames": ["default"],`
`146`	`141`	"message": "Please use `import {styled} from \"styled-components\"` instead."
`147`		`- },`
`148`		`- {`
`149`		`- "name": "react",`
`150`		`- "importNames": ["Component", "PureComponent"],`
`151`		`- "message": "Use function components with hooks instead of class components."`
`152`	`142`	`}`
`153`	`143`	`]`
`154`	`144`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		-// oxlint-disable-next-line no-restricted-imports -- error boundaries require a class component; React has no function-component equivalent for `getDerivedStateFromError`/`componentDidCatch`
`2`	`1`	`import {Component, PropsWithChildren} from 'react'`
`3`	`2`
`4`	`3`	`/**`