Added react to lint plugins (#3255)

Author: AndreasArvidsson

.oxlintrc.json

@@ -1,6 +1,14 @@
 {
   "$schema": "./node_modules/oxlint/configuration_schema.json",
-  "plugins": ["eslint", "typescript", "unicorn", "oxc", "import"],
+  "plugins": [
+    "eslint",
+    "typescript",
+    "unicorn",
+    "oxc",
+    "import",
+    "react",
+    "react-perf"
+  ],
   "jsPlugins": [
     {
       "name": "mocha",

packages/app-web-docs/src/docs/components/DynamicTOC.tsx

@@ -25,7 +25,7 @@ export function DynamicTOC({
     } else {
       row.appendChild(toc);
     }
-  }, []);
+  }, [minHeadingLevel, maxHeadingLevel]);
 
   return null;
 }

packages/app-web-docs/src/docs/components/ScrollToHashId.tsx

@@ -33,6 +33,7 @@ export function ScrollToHashId() {
 
       setTimeout(scrollToId, delay);
     }
+    // oxlint-disable-next-line eslint-plugin-react-hooks/exhaustive-deps
   }, []);
 
   return null;

packages/lib-cheatsheet/src/lib/utils/useIsHighlighted.ts

@@ -29,7 +29,7 @@ const useHash = () => {
     return () => {
       window.removeEventListener("hashchange", onHashChange);
     };
-  }, []);
+  }, [onHashChange]);
 
   return hash;
 };

packages/lib-vscode-tutorial-webview/src/App.tsx

@@ -21,7 +21,7 @@ export function App({ vscode }: Props) {
     );
 
     vscode.postMessage({ type: "getInitialState" });
-  }, []);
+  }, [vscode]);
 
   if (state == null) {
     // Just show nothing while we're waiting for initial state

packages/lib-vscode-tutorial-webview/src/TutorialStep.tsx

@@ -55,9 +55,9 @@ export function TutorialStep({ state, vscode }: Props) {
                 case "string":
                   return <span key={j}>{fragment.value}</span>;
                 case "command":
-                  return <Command spokenForm={fragment.value} />;
+                  return <Command key={j} spokenForm={fragment.value} />;
                 case "term":
-                  return <span>"{fragment.value}"</span>;
+                  return <span key={j}>"{fragment.value}"</span>;
                 default: {
                   // Ensure we handle all cases
                   const _unused: never = fragment;