refactor: minor improvements

Author: Rel1cx

packages/core/docs/functions/stringifyJsx.md

@@ -8,8 +8,7 @@
 
 > **stringifyJsx**(`node`): `string`
 
-Converts a JSX AST node to its string representation
-Handles different JSX node types and returns their textual form
+Incomplete but sufficient stringification of JSX nodes for common use cases
 
 ## Parameters
 

packages/core/src/jsx/index.ts

@@ -2,7 +2,6 @@ export * from "./jsx-attribute";
 export * from "./jsx-attribute-name";
 export * from "./jsx-attribute-value";
 export * from "./jsx-detection";
-export * from "./jsx-detection-hint";
 export * from "./jsx-element-is";
 export * from "./jsx-element-type";
 export * from "./jsx-has";

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

@@ -5,7 +5,56 @@ import * as VAR from "@eslint-react/var";
 import type { Scope } from "@typescript-eslint/scope-manager";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import type { TSESTree } from "@typescript-eslint/utils";
-import { DEFAULT_JSX_DETECTION_HINT, JSXDetectionHint } from "./jsx-detection-hint";
+
+// This is a representation of React Node types for reference
+// type ReactNode =
+//   | ReactElement
+//   | string
+//   | number
+//   | Iterable<ReactNode>
+//   | ReactPortal
+//   | boolean
+//   | null
+//   | undefined
+//   | DO_NOT_USE_OR_YOU_WILL_BE_FIRED_EXPERIMENTAL_REACT_NODES[
+//     keyof DO_NOT_USE_OR_YOU_WILL_BE_FIRED_EXPERIMENTAL_REACT_NODES
+//   ];
+
+/**
+ * BitFlags for configuring JSX detection behavior
+ * Uses BigInt for bit operations to support many flags
+ */
+export type JSXDetectionHint = bigint;
+
+/* eslint-disable perfectionist/sort-objects */
+/**
+ * Flags to control JSX detection behavior:
+ * - Skip* flags: Ignore specific node types when detecting JSX
+ * - Strict* flags: Enforce stricter rules for container types
+ */
+export const JSXDetectionHint = {
+  None: 0n,
+  SkipUndefined: 1n << 0n, // Ignore undefined values
+  SkipNullLiteral: 1n << 1n, // Ignore null literals
+  SkipBooleanLiteral: 1n << 2n, // Ignore boolean literals
+  SkipStringLiteral: 1n << 3n, // Ignore string literals
+  SkipNumberLiteral: 1n << 4n, // Ignore number literals
+  SkipBigIntLiteral: 1n << 5n, // Ignore bigint literals
+  SkipEmptyArray: 1n << 6n, // Ignore empty arrays
+  SkipCreateElement: 1n << 7n, // Ignore React.createElement calls
+  StrictArray: 1n << 8n, // Require all array elements to be JSX
+  StrictLogical: 1n << 9n, // Require both sides of logical expr to be JSX
+  StrictConditional: 1n << 10n, // Require both branches of conditional to be JSX
+} as const;
+/* eslint-enable perfectionist/sort-objects */
+
+/**
+ * Default JSX detection configuration
+ * Skips undefined and boolean literals (common in React)
+ */
+export const DEFAULT_JSX_DETECTION_HINT = 0n
+  | JSXDetectionHint.SkipUndefined
+  | JSXDetectionHint.SkipBooleanLiteral;
 
 /**
  * Checks if a node is a `JSXText` or a `Literal` node

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

@@ -2,17 +2,16 @@ import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import type { TSESTree } from "@typescript-eslint/utils";
 
 /**
- * Converts a JSX AST node to its string representation
- * Handles different JSX node types and returns their textual form
+ * Incomplete but sufficient stringification of JSX nodes for common use cases
  *
  * @param node - JSX node from TypeScript ESTree
  * @returns String representation of the JSX node
  */
 export function stringifyJsx(
   node:
     | TSESTree.JSXIdentifier
-    | TSESTree.JSXMemberExpression
     | TSESTree.JSXNamespacedName
+    | TSESTree.JSXMemberExpression
     | TSESTree.JSXOpeningElement
     | TSESTree.JSXClosingElement
     | TSESTree.JSXOpeningFragment

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

@@ -28,6 +28,8 @@ export function toDelimiterFormat(node: TSESTree.Node, delimiter = " ") {
 }
 
 /**
+ * Incomplete but sufficient stringification of AST nodes for common use cases
+ *
  * @internal
  */
 export function toStringFormat(node: TSESTree.Node, getText: (node: TSESTree.Node) => string): string {