refactor: getFunctionIdentifier add function in class support

Author: Rel1cx

CHANGELOG.md

@@ -2,8 +2,21 @@
 
 ### Release Notes
 
+#### Reduce false positives in rule `react/no-unstable-nested-components`
+
+#### Reduce false positives in rule `debug/function-component`
+
 ---
 
+#### 🏠 Internal
+
+- `@eslint-react/ast`
+  - Refactor `getFunctionIdentifier` function to follow spec convention for `IsAnonymousFunctionDefinition()` usage in ECMAScript spec.
+  - Improve module structure.
+
+- `@eslint-react/core`
+  - Move construction detection from `@eslint-react/ast` to `@eslint-react/core`.
+
 #### Authors: 1
 
 - Eva1ent ([@Rel1cx](https://github.com/Rel1cx))

packages/ast/src/function/function-identifier.ts

@@ -10,7 +10,7 @@
 import { O } from "@eslint-react/tools";
 import type { TSESTree } from "@typescript-eslint/types";
 
-import { NodeType, type TSESTreeFunction } from "../node";
+import { isOneOf, NodeType, type TSESTreeFunction } from "../node";
 
 export function getFunctionIdentifier(node: TSESTreeFunction): O.Option<TSESTree.Identifier> {
   if (node.id) {
@@ -47,13 +47,16 @@ export function getFunctionIdentifier(node: TSESTreeFunction): O.Option<TSESTree
     // {useHook: () => {}}
     // {useHook() {}}
     return O.some(node.parent.key);
+  }
 
-    // NOTE: We could also support `ClassProperty` and `MethodDefinition`
-    // here to be pedantic. However, hooks in a class are an anti-pattern. So
-    // we don't allow it to error early.
-    //
+  if (
+    isOneOf([NodeType.MethodDefinition, NodeType.PropertyDefinition])(node.parent)
+    && node.parent.value === node
+    && node.parent.key.type === NodeType.Identifier
+  ) {
     // class {useHook = () => {}}
     // class {useHook() {}}
+    return O.some(node.parent.key);
   }
 
   if (