refactor: remove obsolete JSX utility types and functions

Author: Rel1cx

packages/core/docs/README.md

@@ -31,7 +31,6 @@
 - [EffectKind](type-aliases/EffectKind.md)
 - [HookKind](type-aliases/HookKind.md)
 - [JSXDetectionHint](type-aliases/JSXDetectionHint.md)
-- [TSESTreeJSX](type-aliases/TSESTreeJSX.md)
 
 ## Variables
 
@@ -63,7 +62,6 @@
 - [isForwardRef](variables/isForwardRef.md)
 - [isForwardRefCall](variables/isForwardRefCall.md)
 - [isInversePhase](variables/isInversePhase.md)
-- [isJSX](variables/isJSX.md)
 - [isLazy](variables/isLazy.md)
 - [isLazyCall](variables/isLazyCall.md)
 - [isMemo](variables/isMemo.md)

packages/core/docs/type-aliases/TSESTreeJSX.md

@@ -7,40 +7,6 @@ import * as VAR from "@eslint-react/var";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import { DEFAULT_JSX_DETECTION_HINT, JSXDetectionHint } from "./jsx-detection-hint";
 
-export type TSESTreeJSX =
-  | TSESTree.JSXAttribute
-  | TSESTree.JSXClosingElement
-  | TSESTree.JSXClosingFragment
-  | TSESTree.JSXElement
-  | TSESTree.JSXEmptyExpression
-  | TSESTree.JSXExpressionContainer
-  | TSESTree.JSXFragment
-  | TSESTree.JSXIdentifier
-  | TSESTree.JSXMemberExpression
-  | TSESTree.JSXNamespacedName
-  | TSESTree.JSXOpeningElement
-  | TSESTree.JSXOpeningFragment
-  | TSESTree.JSXSpreadAttribute
-  | TSESTree.JSXSpreadChild
-  | TSESTree.JSXText;
-
-export const isJSX = AST.isOneOf([
-  T.JSXAttribute,
-  T.JSXClosingElement,
-  T.JSXClosingFragment,
-  T.JSXElement,
-  T.JSXEmptyExpression,
-  T.JSXExpressionContainer,
-  T.JSXFragment,
-  T.JSXIdentifier,
-  T.JSXMemberExpression,
-  T.JSXNamespacedName,
-  T.JSXOpeningElement,
-  T.JSXOpeningFragment,
-  T.JSXSpreadAttribute,
-  T.JSXSpreadChild,
-  T.JSXText,
-]);
 /**
  * Check if a node is a `JSXText` or a `Literal` node
  * @param node The AST node to check
@@ -65,7 +31,7 @@ export function isJsxLike(
   hint: JSXDetectionHint = DEFAULT_JSX_DETECTION_HINT,
 ): boolean {
   if (node == null) return false;
-  if (isJSX(node)) return true;
+  if (AST.isJSX(node)) return true;
   switch (node.type) {
     case T.Literal: {
       switch (typeof node.value) {

packages/core/docs/variables/isJSX.md

@@ -65,7 +65,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
 
       for (const lazy of lazyComponentDeclarations) {
         const significantParent = AST.findParentNode(lazy, (n) => {
-          if (ER.isJSX(n)) return true;
+          if (AST.isJSX(n)) return true;
           if (n.type === T.CallExpression) {
             return ER.isReactHookCall(n) || ER.isCreateElementCall(context, n) || ER.isCreateContextCall(context, n);
           }

packages/core/src/jsx/jsx-detection.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unused-vars */
 import type { TSESTree } from "@typescript-eslint/utils";
 import { or } from "@eslint-react/eff";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
@@ -70,9 +69,14 @@ export const isJSXElement = is(T.JSXElement);
 
 export const isJSXFragment = is(T.JSXFragment);
 
+export const isJSXTagNameExpression = isOneOf([
+  T.JSXIdentifier,
+  T.JSXMemberExpression,
+  T.JSXNamespacedName,
+]);
+
 export const isJSX = isOneOf([
   T.JSXAttribute,
-  T.JSXSpreadChild,
   T.JSXClosingElement,
   T.JSXClosingFragment,
   T.JSXElement,
@@ -89,12 +93,6 @@ export const isJSX = isOneOf([
   T.JSXText,
 ]);
 
-export const isJSXTagNameExpression = isOneOf([
-  T.JSXIdentifier,
-  T.JSXMemberExpression,
-  T.JSXNamespacedName,
-]);
-
 export const isDestructuringPattern = isOneOf([
   T.ArrayPattern,
   T.AssignmentPattern,