refactor: clean up redundant code

Author: Rel1cx

plugins/eslint-plugin-react-debug/src/rules/jsx/jsx.ts

@@ -4,11 +4,9 @@ import type { TSESTreeJSXElementLike } from "@eslint-react/ast";
 import * as core from "@eslint-react/core";
 import { type RuleContext, type RuleFeature, type RuleListener } from "@eslint-react/eslint";
 import { getElementFullType, isFragmentElement } from "@eslint-react/jsx";
-import { flow } from "@local/eff";
 import { AST_NODE_TYPES as AST } from "@typescript-eslint/types";
 import { P, match } from "ts-pattern";
 import ts from "typescript";
-import { report } from "./lib";
 
 export const RULE_NAME = "jsx";
 
@@ -36,37 +34,35 @@ export default createRule<[], MessageID>({
 
 export function create(context: RuleContext<MessageID, []>): RuleListener {
   const jsxConfig = core.getJsxConfig(context);
-
-  function visit(node: TSESTreeJSXElementLike) {
-    return {
-      data: {
-        json: stringify({
-          kind: match(node)
-            .with({ type: AST.JSXElement }, (n) => isFragmentElement(n, jsxConfig.jsxFragmentFactory) ? "fragment" : "element")
-            .with({ type: AST.JSXFragment }, () => "fragment")
-            .exhaustive(),
-          type: getElementFullType(node),
-          jsx: match(jsxConfig.jsx)
-            .with(ts.JsxEmit.None, () => "none")
-            .with(ts.JsxEmit.ReactJSX, () => "react-jsx")
-            .with(ts.JsxEmit.ReactJSXDev, () => "react-jsx-dev")
-            .with(ts.JsxEmit.React, () => "react")
-            .with(ts.JsxEmit.ReactNative, () => "react-native")
-            .with(ts.JsxEmit.Preserve, () => "preserve")
-            .otherwise(() => "unknown"),
-          jsxFactory: jsxConfig.jsxFactory,
-          jsxFragmentFactory: jsxConfig.jsxFragmentFactory,
-          jsxImportSource: jsxConfig.jsxImportSource,
-          jsxRuntime: match(jsxConfig.jsx)
-            .with(P.union(ts.JsxEmit.None, ts.JsxEmit.ReactJSX, ts.JsxEmit.ReactJSXDev), () => "automatic")
-            .otherwise(() => "classic"),
-        }),
-      },
-      messageId: "default",
-      node,
-    } as const;
-  }
   return {
-    "JSXElement, JSXFragment": flow(visit, report(context)),
+    "JSXElement, JSXFragment": (node: TSESTreeJSXElementLike) => {
+      context.report({
+        data: {
+          json: stringify({
+            kind: match(node)
+              .with({ type: AST.JSXElement }, (n) => isFragmentElement(n, jsxConfig.jsxFragmentFactory) ? "fragment" : "element")
+              .with({ type: AST.JSXFragment }, () => "fragment")
+              .exhaustive(),
+            type: getElementFullType(node),
+            jsx: match(jsxConfig.jsx)
+              .with(ts.JsxEmit.None, () => "none")
+              .with(ts.JsxEmit.ReactJSX, () => "react-jsx")
+              .with(ts.JsxEmit.ReactJSXDev, () => "react-jsx-dev")
+              .with(ts.JsxEmit.React, () => "react")
+              .with(ts.JsxEmit.ReactNative, () => "react-native")
+              .with(ts.JsxEmit.Preserve, () => "preserve")
+              .otherwise(() => "unknown"),
+            jsxFactory: jsxConfig.jsxFactory,
+            jsxFragmentFactory: jsxConfig.jsxFragmentFactory,
+            jsxImportSource: jsxConfig.jsxImportSource,
+            jsxRuntime: match(jsxConfig.jsx)
+              .with(P.union(ts.JsxEmit.None, ts.JsxEmit.ReactJSX, ts.JsxEmit.ReactJSXDev), () => "automatic")
+              .otherwise(() => "classic"),
+          }),
+        },
+        messageId: "default",
+        node,
+      });
+    },
   };
 }