Shouldn't use useId in async Components

Author: eps1lon

packages/eslint-plugin-react-hooks/__tests__/ESLintRulesOfHooks-test.js

@@ -550,21 +550,6 @@ const tests = {
       // TODO: this should error but doesn't.
       // errors: [genericError('useState')],
     },
-    {
-      code: normalizeIndent`
-        async function Page() {
-          useId();
-          React.useId();
-        }
-      `,
-    },
-    {
-      code: normalizeIndent`
-        async function useAsyncHook() {
-          useId();
-        }
-      `,
-    },
   ],
   invalid: [
     {
@@ -1144,6 +1129,26 @@ const tests = {
       `,
       errors: [asyncComponentHookError('useState')],
     },
+    {
+      code: normalizeIndent`
+        async function Page() {
+          useId();
+          React.useId();
+        }
+      `,
+      errors: [
+        asyncComponentHookError('useId'),
+        asyncComponentHookError('React.useId'),
+      ],
+    },
+    {
+      code: normalizeIndent`
+        async function useAsyncHook() {
+          useId();
+        }
+      `,
+      errors: [asyncComponentHookError('useId')],
+    },
     {
       code: normalizeIndent`
         async function notAHook() {

packages/eslint-plugin-react-hooks/src/RulesOfHooks.js

@@ -40,21 +40,6 @@ function isHook(node) {
   }
 }
 
-const serverComponentHooks = new Set(['useId']);
-function isServerComponentHook(node) {
-  if (node.type === 'Identifier') {
-    return serverComponentHooks.has(node.name);
-  } else if (
-    node.type === 'MemberExpression' &&
-    !node.computed &&
-    node.property.type === 'Identifier'
-  ) {
-    return serverComponentHooks.has(node.property.name);
-  } else {
-    return false;
-  }
-}
-
 /**
  * Checks if the node is a React component name. React component names must
  * always start with an uppercase letter.
@@ -519,7 +504,7 @@ export default {
             if (isDirectlyInsideComponentOrHook) {
               // Report an error if the hook is called inside an async function.
               const isAsyncFunction = codePathNode.async;
-              if (isAsyncFunction && !isServerComponentHook(hook)) {
+              if (isAsyncFunction) {
                 context.report({
                   node: hook,
                   message: