refactor: refactor JsxRuntime namespace to JsxRuntimeOptions (#1039)

Author: Rel1cx

packages/plugins/eslint-plugin-react-x/src/rules/jsx-uses-react.spec.ts

@@ -3,7 +3,7 @@ import tsx from "dedent";
 
 import { JsxEmit } from "typescript";
 import { defaultLanguageOptionsWithTypes, getProjectForJsxRuntime } from "../../../../../test";
-import rule, { debug, RULE_NAME } from "./jsx-uses-react";
+import rule, { RULE_NAME } from "./jsx-uses-react";
 
 const ruleTester = new RuleTester({
   languageOptions: {
@@ -16,8 +16,8 @@ const ruleTester = new RuleTester({
   },
 });
 
-// eslint-disable-next-line @typescript-eslint/no-unused-expressions, @typescript-eslint/no-unnecessary-condition
-debug
+// eslint-disable-next-line @typescript-eslint/no-unused-expressions
+process.env["ESLINT_REACT_DEBUG"] === "1"
   ? ruleTester.run(RULE_NAME, rule, {
     invalid: [
       {

packages/plugins/eslint-plugin-react-x/src/rules/jsx-uses-react.ts

@@ -2,7 +2,7 @@ import type { TSESTree } from "@typescript-eslint/types";
 import type { RuleListener } from "@typescript-eslint/utils/ts-eslint";
 
 import type { CamelCase } from "string-ts";
-import { JsxRuntime, type RuleContext, type RuleFeature } from "@eslint-react/kit";
+import { JsxRuntimeOptions, type RuleContext, type RuleFeature } from "@eslint-react/kit";
 import { JsxEmit } from "typescript";
 import { createRule } from "../utils";
 
@@ -12,8 +12,6 @@ export const RULE_FEATURES = [] as const satisfies RuleFeature[];
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-export const debug = false;
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -32,15 +30,16 @@ export default createRule<[], MessageID>({
 });
 
 export function create(context: RuleContext<MessageID, []>): RuleListener {
-  const jsxOptions = JsxRuntime.getJsxRuntimeOptions(context);
-  const jsxAnnotation = JsxRuntime.getJsxRuntimeAnnotation(context);
-  const jsx = jsxAnnotation.jsxRuntime === "classic"
-    ? JsxEmit.React
-    : jsxOptions.jsx;
-  if (jsx === JsxEmit.ReactJSX || jsx === JsxEmit.ReactJSXDev) return {};
+  const jsxRuntimeOptionsFromContext = JsxRuntimeOptions.getFromContext(context);
+  const jsxRuntimeOptionsFromAnnotation = JsxRuntimeOptions.getFromAnnotation(context);
+  const jsxRuntimeOptions = {
+    ...jsxRuntimeOptionsFromContext,
+    ...jsxRuntimeOptionsFromAnnotation,
+  };
 
-  const jsxFactory = jsxAnnotation.jsx ?? jsxOptions.jsxFactory;
-  const jsxFragmentFactory = jsxAnnotation.jsxFrag ?? jsxOptions.jsxFragmentFactory;
+  const { jsx, jsxFactory, jsxFragmentFactory } = jsxRuntimeOptions;
+
+  if (jsx === JsxEmit.ReactJSX || jsx === JsxEmit.ReactJSXDev) return {};
 
   function handleJsxElement(node: TSESTree.Node) {
     context.sourceCode.markVariableAsUsed(jsxFactory, node);
@@ -60,8 +59,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
 }
 
 function debugReport(context: RuleContext, node: TSESTree.Node, name: string) {
-  // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-  if (!debug) return;
+  if (process.env["ESLINT_REACT_DEBUG"] !== "1") return;
   context.report({
     messageId: "jsxUsesReact",
     node,