refactor: rename 'unsafe' to 'loose' in internal API names

Author: Rel1cx

packages/core/docs/README.md

@@ -62,6 +62,7 @@
 - [isCreateElementCall](functions/isCreateElementCall.md)
 - [isCreateRef](functions/isCreateRef.md)
 - [isCreateRefCall](functions/isCreateRefCall.md)
+- [isDeclaredInRenderPropLoose](functions/isDeclaredInRenderPropLoose.md)
 - [isForwardRef](functions/isForwardRef.md)
 - [isForwardRefCall](functions/isForwardRefCall.md)
 - [isFragmentElement](functions/isFragmentElement.md)
@@ -82,6 +83,8 @@
 - [isReactHookCallWithName](functions/isReactHookCallWithName.md)
 - [isReactHookCallWithNameLoose](functions/isReactHookCallWithNameLoose.md)
 - [isReactHookName](functions/isReactHookName.md)
+- [isRenderFunctionLoose](functions/isRenderFunctionLoose.md)
+- [isRenderPropLoose](functions/isRenderPropLoose.md)
 - [isUseCallbackCall](functions/isUseCallbackCall.md)
 - [isUseContextCall](functions/isUseContextCall.md)
 - [isUseDebugValueCall](functions/isUseDebugValueCall.md)
@@ -97,9 +100,6 @@
 - [isUseStateCall](functions/isUseStateCall.md)
 - [isUseSyncExternalStoreCall](functions/isUseSyncExternalStoreCall.md)
 - [isUseTransitionCall](functions/isUseTransitionCall.md)
-- [unsafeIsDeclaredInRenderProp](functions/unsafeIsDeclaredInRenderProp.md)
-- [unsafeIsRenderFunction](functions/unsafeIsRenderFunction.md)
-- [unsafeIsRenderProp](functions/unsafeIsRenderProp.md)
 - [useComponentCollector](functions/useComponentCollector.md)
 - [useComponentCollectorLegacy](functions/useComponentCollectorLegacy.md)
 - [useHookCollector](functions/useHookCollector.md)

packages/core/docs/functions/unsafeIsDeclaredInRenderProp.md renamed to packages/core/docs/functions/isDeclaredInRenderPropLoose.md

@@ -2,11 +2,11 @@
 
 ***
 
-[@eslint-react/core](../README.md) / unsafeIsDeclaredInRenderProp
+[@eslint-react/core](../README.md) / isDeclaredInRenderPropLoose
 
-# Function: unsafeIsDeclaredInRenderProp()
+# Function: isDeclaredInRenderPropLoose()
 
-> **unsafeIsDeclaredInRenderProp**(`node`): `boolean`
+> **isDeclaredInRenderPropLoose**(`node`): `boolean`
 
 Unsafe check whether given node is declared inside a render prop
 ```jsx

packages/core/docs/functions/unsafeIsRenderFunction.md renamed to packages/core/docs/functions/isRenderFunctionLoose.md

@@ -2,11 +2,11 @@
 
 ***
 
-[@eslint-react/core](../README.md) / unsafeIsRenderFunction
+[@eslint-react/core](../README.md) / isRenderFunctionLoose
 
-# Function: unsafeIsRenderFunction()
+# Function: isRenderFunctionLoose()
 
-> **unsafeIsRenderFunction**(`node`, `context`): `boolean`
+> **isRenderFunctionLoose**(`node`, `context`): `boolean`
 
 Unsafe check whether given node is a render function
 ```jsx

packages/core/docs/functions/unsafeIsRenderProp.md renamed to packages/core/docs/functions/isRenderPropLoose.md

@@ -2,11 +2,11 @@
 
 ***
 
-[@eslint-react/core](../README.md) / unsafeIsRenderProp
+[@eslint-react/core](../README.md) / isRenderPropLoose
 
-# Function: unsafeIsRenderProp()
+# Function: isRenderPropLoose()
 
-> **unsafeIsRenderProp**(`node`, `context`): `boolean`
+> **isRenderPropLoose**(`node`, `context`): `boolean`
 
 Unsafe check whether given JSXAttribute is a render prop
 ```jsx

packages/core/src/render-prop/hierarchy.ts

@@ -15,7 +15,7 @@ import { isMatching, P } from "ts-pattern";
  * @param node The AST node to check
  * @returns `true` if component is declared inside a render property, `false` if not
  */
-export function unsafeIsDirectValueOfRenderProperty(node: TSESTree.Node) {
+export function isDirectValueOfRenderPropertyLoose(node: TSESTree.Node) {
   const matching = isMatching({
     key: {
       type: NodeType.Identifier,
@@ -38,8 +38,8 @@ export function unsafeIsDirectValueOfRenderProperty(node: TSESTree.Node) {
  * @param node The AST node to check
  * @returns `true` if component is declared inside a render prop, `false` if not
  */
-export function unsafeIsDeclaredInRenderProp(node: TSESTree.Node) {
-  if (unsafeIsDirectValueOfRenderProperty(node)) {
+export function isDeclaredInRenderPropLoose(node: TSESTree.Node) {
+  if (isDirectValueOfRenderPropertyLoose(node)) {
     return true;
   }
 

packages/core/src/render-prop/is.ts

@@ -18,7 +18,7 @@ import { isMatching, P } from "ts-pattern";
  * @param context The rule context
  * @returns `true` if node is a render function, `false` if not
  */
-export function unsafeIsRenderFunction(node: TSESTreeFunction, context: RuleContext) {
+export function isRenderFunctionLoose(node: TSESTreeFunction, context: RuleContext) {
   const { body, parent } = node;
 
   const maybeId = getFunctionIdentifier(node);
@@ -56,7 +56,7 @@ export function unsafeIsRenderFunction(node: TSESTreeFunction, context: RuleCont
  * @param context The rule context
  * @returns `true` if node is a render prop, `false` if not
  */
-export function unsafeIsRenderProp(node: TSESTree.JSXAttribute, context: RuleContext) {
+export function isRenderPropLoose(node: TSESTree.JSXAttribute, context: RuleContext) {
   return isMatching({
     type: NodeType.JSXAttribute,
     name: {
@@ -67,5 +67,5 @@ export function unsafeIsRenderProp(node: TSESTree.JSXAttribute, context: RuleCon
       type: NodeType.JSXExpressionContainer,
       expression: P.when(isFunction),
     },
-  }, node) && unsafeIsRenderFunction(node.value.expression, context);
+  }, node) && isRenderFunctionLoose(node.value.expression, context);
 }

packages/plugins/eslint-plugin-react-core/src/rules/no-nested-components.ts

@@ -2,10 +2,10 @@ import type { TSESTreeClass, TSESTreeFunction } from "@eslint-react/ast";
 import { isClass, isFunction, NodeType, traverseUp, traverseUpGuard } from "@eslint-react/ast";
 import {
   ERComponentHint,
+  isDeclaredInRenderPropLoose,
+  isDirectValueOfRenderPropertyLoose,
   isInsideCreateElementProps,
   isInsideRenderMethod,
-  unsafeIsDeclaredInRenderProp,
-  unsafeIsDirectValueOfRenderProperty,
   useComponentCollector,
   useComponentCollectorLegacy,
 } from "@eslint-react/core";
@@ -64,14 +64,14 @@ export default createRule<[], MessageID>({
         };
         for (const { name: componentName, node: component } of functionComponents) {
           // Do not mark objects containing render methods
-          if (unsafeIsDirectValueOfRenderProperty(component)) continue;
+          if (isDirectValueOfRenderPropertyLoose(component)) continue;
           // Do not mark anonymous function components to reduce false positives
           if (O.isNone(componentName)) continue;
           const name = componentName.value;
           const isInsideProperty = component.parent.type === NodeType.Property;
           const isInsideJSXPropValue = isInsidePropValue(component);
           if (isInsideJSXPropValue) {
-            if (!unsafeIsDeclaredInRenderProp(component)) {
+            if (!isDeclaredInRenderPropLoose(component)) {
               context.report({
                 data: {
                   name,
@@ -95,7 +95,7 @@ export default createRule<[], MessageID>({
             continue;
           }
           const maybeParentComponent = traverseUpGuard(component, isFunctionComponent);
-          if (O.isSome(maybeParentComponent) && !unsafeIsDirectValueOfRenderProperty(maybeParentComponent.value)) {
+          if (O.isSome(maybeParentComponent) && !isDirectValueOfRenderPropertyLoose(maybeParentComponent.value)) {
             context.report({
               data: {
                 name,