refactor: rename '_' to unit

Author: Rel1cx

.pkgs/eslint-plugin-local/dist/index.js

@@ -1,6 +1,6 @@
 import * as AST from '@eslint-react/ast';
 import { ESLintUtils, AST_NODE_TYPES } from '@typescript-eslint/utils';
-import { _ } from '@eslint-react/eff';
+import { unit } from '@eslint-react/eff';
 import { findVariable } from '@eslint-react/var';
 import { AST_NODE_TYPES as AST_NODE_TYPES$1 } from '@typescript-eslint/types';
 import { nullThrows, NullThrowsReasons } from '@typescript-eslint/utils/eslint-utils';
@@ -44,7 +44,7 @@ function getRequireExpressionArguments(node) {
       return getRequireExpressionArguments(node.object);
     }
   }
-  return _;
+  return unit;
 }
 
 // src/rules/avoid-multiline-template-expression.ts

.pkgs/eslint-plugin-local/src/utils/is-initialized-from-source.ts

@@ -1,7 +1,7 @@
 import type { Scope } from "@typescript-eslint/scope-manager";
 import type { TSESTree } from "@typescript-eslint/types";
 import * as AST from "@eslint-react/ast";
-import { _ } from "@eslint-react/eff";
+import { unit } from "@eslint-react/eff";
 import { findVariable } from "@eslint-react/var";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 
@@ -46,7 +46,7 @@ export function isInitializedFromSource(
   return parent?.type === T.ImportDeclaration && parent.source.value === source;
 }
 
-function getRequireExpressionArguments(node: TSESTree.Node): TSESTree.CallExpressionArgument[] | _ {
+function getRequireExpressionArguments(node: TSESTree.Node): TSESTree.CallExpressionArgument[] | unit {
   switch (true) {
     // require('source')
     case node.type === T.CallExpression
@@ -59,5 +59,5 @@ function getRequireExpressionArguments(node: TSESTree.Node): TSESTree.CallExpres
       return getRequireExpressionArguments(node.object);
     }
   }
-  return _;
+  return unit;
 }

packages/core/docs/@eslint-react/namespaces/useComponentCollector/type-aliases/ReturnType.md

@@ -38,11 +38,11 @@
 
 #### getCurrentEntry()
 
-> **getCurrentEntry**: () => `FunctionEntry` \| `_`
+> **getCurrentEntry**: () => `FunctionEntry` \| `unit`
 
 ##### Returns
 
-`FunctionEntry` \| `_`
+`FunctionEntry` \| `unit`
 
 ***
 

packages/core/src/component/component-collector-legacy.ts

@@ -1,7 +1,7 @@
 import type { ESLintUtils, TSESTree } from "@typescript-eslint/utils";
 import type { ClassComponent } from "./component-semantic-node";
 import * as AST from "@eslint-react/ast";
-import { _ } from "@eslint-react/eff";
+import { unit } from "@eslint-react/eff";
 
 import { getId } from "@eslint-react/shared";
 import { ComponentFlag } from "./component-flag";
@@ -48,7 +48,7 @@ export function useComponentCollectorLegacy(): useComponentCollectorLegacy.Retur
         name: id?.name,
         node,
         // TODO: Get displayName of class component
-        displayName: _,
+        displayName: unit,
         flag,
         hint: 0n,
         // TODO: Get methods of class component

packages/core/src/component/component-collector.ts

@@ -3,7 +3,7 @@ import type { ESLintUtils } from "@typescript-eslint/utils";
 import type { ComponentDetectionHint } from "./component-detection-hint";
 import type { FunctionComponent } from "./component-semantic-node";
 import * as AST from "@eslint-react/ast";
-import { _ } from "@eslint-react/eff";
+import { unit } from "@eslint-react/eff";
 import { type RuleContext, Selector as SEL } from "@eslint-react/kit";
 import { getId } from "@eslint-react/shared";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
@@ -33,7 +33,7 @@ export declare namespace useComponentCollector {
     ctx: {
       getAllComponents: (node: TSESTree.Program) => Map<string, FunctionComponent>;
       getCurrentEntries: () => FunctionEntry[];
-      getCurrentEntry: () => FunctionEntry | _;
+      getCurrentEntry: () => FunctionEntry | unit;
     };
     listeners: ESLintUtils.RuleListener;
   };
@@ -114,7 +114,7 @@ export function useComponentCollector(
         kind: "function",
         name,
         node: entry.node,
-        displayName: _,
+        displayName: unit,
         flag: getComponentFlagFromInitPath(initPath),
         hint,
         hookCalls: entry.hookCalls,
@@ -128,7 +128,7 @@ export function useComponentCollector(
           if (left.type !== T.MemberExpression) return;
           const componentName = left.object.type === T.Identifier
             ? left.object.name
-            : _;
+            : unit;
           const component = [...components.values()]
             .findLast(({ name }) => name != null && name === componentName);
           if (component == null) return;
@@ -163,7 +163,7 @@ export function useComponentCollector(
         kind: "function",
         name,
         node: entry.node,
-        displayName: _,
+        displayName: unit,
         flag: getComponentFlagFromInitPath(initPath),
         hint,
         hookCalls: entry.hookCalls,

packages/core/src/component/component-id.ts

@@ -1,14 +1,14 @@
 import type { RuleContext } from "@eslint-react/kit";
 import type { TSESTree } from "@typescript-eslint/types";
 import * as AST from "@eslint-react/ast";
-import { _ } from "@eslint-react/eff";
+import { unit } from "@eslint-react/eff";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import { isComponentWrapperCallLoose } from "./component-wrapper";
 
 export function getFunctionComponentId(
   context: RuleContext,
   node: AST.TSESTreeFunction,
-): TSESTree.Identifier | TSESTree.Identifier[] | _ {
+): TSESTree.Identifier | TSESTree.Identifier[] | unit {
   const functionId = AST.getFunctionId(node);
   if (functionId != null) {
     return functionId;
@@ -34,5 +34,5 @@ export function getFunctionComponentId(
   ) {
     return parent.parent.parent.id;
   }
-  return _;
+  return unit;
 }

packages/core/src/component/component-name.ts

@@ -1,7 +1,7 @@
 import type * as AST from "@eslint-react/ast";
 import type { RuleContext } from "@eslint-react/kit";
 import type { TSESTree } from "@typescript-eslint/types";
-import { _ } from "@eslint-react/eff";
+import { unit } from "@eslint-react/eff";
 import { RegExp as RE } from "@eslint-react/kit";
 
 import { getFunctionComponentId } from "./component-id";
@@ -14,8 +14,8 @@ export function isComponentNameLoose(name: string) {
   return RE.COMPONENT_NAME_LOOSE.test(name);
 }
 
-export function getComponentNameFromId(id: TSESTree.Identifier | TSESTree.Identifier[] | _) {
-  if (id == null) return _;
+export function getComponentNameFromId(id: TSESTree.Identifier | TSESTree.Identifier[] | unit) {
+  if (id == null) return unit;
   return Array.isArray(id)
     ? id.map((n) => n.name).join(".")
     : id.name;

packages/core/src/component/component-semantic-node.ts

@@ -1,5 +1,5 @@
 import type * as AST from "@eslint-react/ast";
-import type { _ } from "@eslint-react/eff";
+import type { unit } from "@eslint-react/eff";
 import type { TSESTree } from "@typescript-eslint/types";
 
 import type { SemanticNode } from "../semantic";
@@ -9,33 +9,33 @@ import type { ComponentFlag } from "./component-flag";
 /* eslint-disable perfectionist/sort-interfaces */
 export interface FunctionComponent extends SemanticNode {
   id:
-    | _
+    | unit
     | TSESTree.Identifier
     | TSESTree.Identifier[];
   kind: "function";
   node: AST.TSESTreeFunction;
   flag: ComponentFlag;
   hint: ComponentDetectionHint;
   initPath:
-    | _
+    | unit
     | AST.FunctionInitPath;
   hookCalls: TSESTree.CallExpression[];
   displayName:
-    | _
+    | unit
     | TSESTree.Expression;
 }
 
 export interface ClassComponent extends SemanticNode {
   id:
-    | _
+    | unit
     | TSESTree.Identifier;
   kind: "class";
   node: AST.TSESTreeClass;
   flag: ComponentFlag;
   hint: ComponentDetectionHint;
   methods: AST.TSESTreeMethodOrProperty[];
   displayName:
-    | _
+    | unit
     | TSESTree.Expression;
 }
 /* eslint-enable perfectionist/sort-interfaces */

packages/core/src/hook/hook-hierarchy.ts

@@ -1,10 +1,10 @@
-import type { _ } from "@eslint-react/eff";
+import type { unit } from "@eslint-react/eff";
 import type { TSESTree } from "@typescript-eslint/types";
 import * as AST from "@eslint-react/ast";
 import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import { isUseEffectCallLoose } from "./hook-is";
 
-export function isFunctionOfUseEffectSetup(node: TSESTree.Node | _) {
+export function isFunctionOfUseEffectSetup(node: TSESTree.Node | unit) {
   if (node == null) return false;
   return node.parent?.type === T.CallExpression
     && node.parent.callee !== node
@@ -13,7 +13,7 @@ export function isFunctionOfUseEffectSetup(node: TSESTree.Node | _) {
     && isUseEffectCallLoose(node.parent);
 }
 
-export function isFunctionOfUseEffectCleanup(node: TSESTree.Node | _) {
+export function isFunctionOfUseEffectCleanup(node: TSESTree.Node | unit) {
   if (node == null) return false;
   const pReturn = AST.findParentNode(node, AST.is(T.ReturnStatement));
   const pFunction = AST.findParentNode(node, AST.isFunction); // Correctly named variable

packages/core/src/hook/hook-is.ts

@@ -1,4 +1,4 @@
-import type { _ } from "@eslint-react/eff";
+import type { unit } from "@eslint-react/eff";
 import type { RuleContext } from "@eslint-react/kit";
 import type { TSESTree } from "@typescript-eslint/types";
 import * as AST from "@eslint-react/ast";
@@ -9,7 +9,7 @@ import { AST_NODE_TYPES as T } from "@typescript-eslint/types";
 import { isInitializedFromReact } from "../utils";
 import { isReactHookName } from "./hook-name";
 
-export function isReactHook(node: AST.TSESTreeFunction | _) {
+export function isReactHook(node: AST.TSESTreeFunction | unit) {
   if (node == null) return false;
   const id = AST.getFunctionId(node);
   return id?.name != null && isReactHookName(id.name);
@@ -20,7 +20,7 @@ export function isReactHook(node: AST.TSESTreeFunction | _) {
  * @param node The node to check.
  * @returns `true` if the node is a React Hook call, `false` otherwise.
  */
-export function isReactHookCall(node: TSESTree.Node | _) {
+export function isReactHookCall(node: TSESTree.Node | unit) {
   if (node == null) return false;
   if (node.type !== T.CallExpression) {
     return false;
@@ -35,7 +35,7 @@ export function isReactHookCall(node: TSESTree.Node | _) {
 }
 
 /* eslint-disable function/function-return-boolean */
-export function isReactHookCallWithName(context: RuleContext, node: TSESTree.Node | _) {
+export function isReactHookCallWithName(context: RuleContext, node: TSESTree.Node | unit) {
   if (node == null || node.type !== T.CallExpression) return constFalse;
   const {
     importSource = DEFAULT_ESLINT_REACT_SETTINGS.importSource,
@@ -58,7 +58,7 @@ export function isReactHookCallWithName(context: RuleContext, node: TSESTree.Nod
   };
 }
 
-export function isReactHookCallWithNameLoose(node: TSESTree.Node | _) {
+export function isReactHookCallWithNameLoose(node: TSESTree.Node | unit) {
   if (node == null || node.type !== T.CallExpression) return constFalse;
   return (name: string) => {
     switch (node.callee.type) {
@@ -72,7 +72,7 @@ export function isReactHookCallWithNameLoose(node: TSESTree.Node | _) {
   };
 }
 
-export function isReactHookCallWithNameAlias(context: RuleContext, name: string, alias: _ | string[] = []) {
+export function isReactHookCallWithNameAlias(context: RuleContext, name: string, alias: unit | string[] = []) {
   const {
     importSource = DEFAULT_ESLINT_REACT_SETTINGS.importSource,
     skipImportCheck = true,
@@ -95,7 +95,7 @@ export function isReactHookCallWithNameAlias(context: RuleContext, name: string,
 }
 /* eslint-enable function/function-return-boolean */
 
-export function isUseEffectCallLoose(node: TSESTree.Node | _) {
+export function isUseEffectCallLoose(node: TSESTree.Node | unit) {
   if (node == null) return false;
   if (node.type !== T.CallExpression) {
     return false;