From d1c3f1a91b7d6516a5dcdae296b8146ecd9d983c Mon Sep 17 00:00:00 2001 From: Rel1cx Date: Tue, 25 Feb 2025 20:59:24 +0800 Subject: [PATCH] refactor: consistent ordering of arguments to context-aware utility functions --- .../getFunctionComponentIdentifier.md | 10 +++++----- .../functions/hasNoneOrValidComponentName.md | 10 +++++----- .../core/docs/functions/hasValidHierarchy.md | 10 +++++----- .../core/docs/functions/isChildrenCount.md | 10 +++++----- .../docs/functions/isChildrenCountCall.md | 10 +++++----- .../core/docs/functions/isChildrenForEach.md | 10 +++++----- .../docs/functions/isChildrenForEachCall.md | 10 +++++----- packages/core/docs/functions/isChildrenMap.md | 10 +++++----- .../core/docs/functions/isChildrenMapCall.md | 10 +++++----- .../functions/isChildrenOfCreateElement.md | 14 ++++++------- .../core/docs/functions/isChildrenOnly.md | 10 +++++----- .../core/docs/functions/isChildrenOnlyCall.md | 10 +++++----- .../core/docs/functions/isChildrenToArray.md | 10 +++++----- .../docs/functions/isChildrenToArrayCall.md | 10 +++++----- .../core/docs/functions/isCloneElement.md | 10 +++++----- .../core/docs/functions/isCloneElementCall.md | 10 +++++----- .../core/docs/functions/isCreateContext.md | 10 +++++----- .../docs/functions/isCreateContextCall.md | 10 +++++----- .../core/docs/functions/isCreateElement.md | 10 +++++----- .../docs/functions/isCreateElementCall.md | 10 +++++----- packages/core/docs/functions/isCreateRef.md | 10 +++++----- .../core/docs/functions/isCreateRefCall.md | 10 +++++----- packages/core/docs/functions/isForwardRef.md | 10 +++++----- .../core/docs/functions/isForwardRefCall.md | 10 +++++----- packages/core/docs/functions/isFromReact.md | 10 +++++----- .../core/docs/functions/isFromReactMember.md | 10 +++++----- .../functions/isInsideCreateElementProps.md | 14 ++++++------- packages/core/docs/functions/isMemo.md | 10 +++++----- packages/core/docs/functions/isMemoCall.md | 10 +++++----- .../core/docs/functions/isReactAPIWithName.md | 20 +++++++++---------- .../docs/functions/isReactHookCallWithName.md | 10 +++++----- .../functions/isReactHookCallWithNameAlias.md | 10 +++++----- .../docs/functions/isRenderFunctionLoose.md | 14 ++++++------- .../core/docs/functions/isRenderPropLoose.md | 14 ++++++------- .../core/docs/functions/isUseCallbackCall.md | 2 +- .../core/docs/functions/isUseContextCall.md | 2 +- .../docs/functions/isUseDebugValueCall.md | 2 +- .../docs/functions/isUseDeferredValueCall.md | 2 +- .../core/docs/functions/isUseEffectCall.md | 2 +- packages/core/docs/functions/isUseIdCall.md | 2 +- .../functions/isUseImperativeHandleCall.md | 2 +- .../functions/isUseInsertionEffectCall.md | 2 +- .../docs/functions/isUseLayoutEffectCall.md | 2 +- packages/core/docs/functions/isUseMemoCall.md | 2 +- .../core/docs/functions/isUseReducerCall.md | 2 +- packages/core/docs/functions/isUseRefCall.md | 2 +- .../core/docs/functions/isUseStateCall.md | 2 +- .../functions/isUseSyncExternalStoreCall.md | 2 +- .../docs/functions/isUseTransitionCall.md | 2 +- .../type-aliases/ReturnType.md | 10 +++++----- .../type-aliases/ReturnType.md | 10 +++++----- .../core/src/component/component-collector.ts | 12 +++++------ packages/core/src/component/component-id.ts | 17 +++++++++------- packages/core/src/component/component-name.ts | 4 ++-- .../src/component/component-render-prop.ts | 10 +++++----- packages/core/src/component/hierarchy.ts | 4 ++-- packages/core/src/element/hierarchy.ts | 10 +++++----- packages/core/src/hook/is.ts | 4 ++-- packages/core/src/utils/is-call-from-react.ts | 12 +++++------ packages/core/src/utils/is-from-react.ts | 4 ++-- .../src/rules/is-from-react.ts | 14 ++++--------- .../src/rules/no-missing-button-type.ts | 2 +- .../src/rules/no-missing-iframe-sandbox.ts | 2 +- .../src/rules/no-unsafe-iframe-sandbox.ts | 2 +- .../src/rules/no-unsafe-target-blank.ts | 2 +- .../utils/get-element-name-on-jsx-and-dom.ts | 2 +- .../no-direct-set-state-in-use-effect.ts | 8 ++++---- ...o-direct-set-state-in-use-layout-effect.ts | 8 ++++---- .../src/rules/no-unnecessary-use-callback.ts | 2 +- .../src/rules/no-unnecessary-use-memo.ts | 2 +- .../src/rules/no-useless-custom-hooks.ts | 4 ++-- .../prefer-use-state-lazy-initialization.ts | 2 +- .../src/utils/is-from-hook-call.ts | 4 ++-- .../src/utils/is-from-use-state-call.ts | 2 +- .../src/rules/use-state.ts | 2 +- .../src/rules/no-leaked-interval.ts | 2 +- .../src/rules/no-leaked-resize-observer.ts | 20 +++++++++---------- .../src/rules/no-leaked-timeout.ts | 2 +- .../src/utils/is-instance-id-equal.ts | 2 +- .../src/rules/ensure-forward-ref-using-ref.ts | 2 +- .../src/rules/no-array-index-key.ts | 10 +++++----- .../src/rules/no-children-count.ts | 2 +- .../src/rules/no-children-for-each.ts | 2 +- .../src/rules/no-children-map.ts | 2 +- .../src/rules/no-children-only.ts | 2 +- .../src/rules/no-children-to-array.ts | 2 +- .../src/rules/no-clone-element.ts | 2 +- .../src/rules/no-create-ref.ts | 2 +- .../src/rules/no-forward-ref.ts | 6 +++--- .../rules/no-missing-context-display-name.ts | 2 +- .../src/rules/no-missing-key.ts | 4 ++-- .../src/rules/no-nested-components.ts | 2 +- .../src/rules/no-use-context.ts | 2 +- .../src/rules/no-useless-fragment.ts | 6 +++--- .../-internal-/interfaces/ReadonlyAction.md | 2 +- .../-internal-/type-aliases/ReadonlyOutput.md | 15 ++++++++++++++ 96 files changed, 327 insertions(+), 315 deletions(-) create mode 100644 packages/shared/docs/-internal-/type-aliases/ReadonlyOutput.md diff --git a/packages/core/docs/functions/getFunctionComponentIdentifier.md b/packages/core/docs/functions/getFunctionComponentIdentifier.md index ccaeffe0dd..8b8282cb7f 100644 --- a/packages/core/docs/functions/getFunctionComponentIdentifier.md +++ b/packages/core/docs/functions/getFunctionComponentIdentifier.md @@ -6,18 +6,18 @@ # Function: getFunctionComponentIdentifier() -> **getFunctionComponentIdentifier**(`node`, `context`): `undefined` \| [`Identifier`](../-internal-/interfaces/Identifier.md) \| [`Identifier`](../-internal-/interfaces/Identifier.md)[] +> **getFunctionComponentIdentifier**(`context`, `node`): `undefined` \| [`Identifier`](../-internal-/interfaces/Identifier.md) \| [`Identifier`](../-internal-/interfaces/Identifier.md)[] ## Parameters -### node - -[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) + ## Returns `undefined` \| [`Identifier`](../-internal-/interfaces/Identifier.md) \| [`Identifier`](../-internal-/interfaces/Identifier.md)[] diff --git a/packages/core/docs/functions/hasNoneOrValidComponentName.md b/packages/core/docs/functions/hasNoneOrValidComponentName.md index 6d35e2bda4..04274244c2 100644 --- a/packages/core/docs/functions/hasNoneOrValidComponentName.md +++ b/packages/core/docs/functions/hasNoneOrValidComponentName.md @@ -6,18 +6,18 @@ # Function: hasNoneOrValidComponentName() -> **hasNoneOrValidComponentName**(`node`, `context`): `boolean` +> **hasNoneOrValidComponentName**(`context`, `node`): `boolean` ## Parameters -### node - -[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/hasValidHierarchy.md b/packages/core/docs/functions/hasValidHierarchy.md index 9a85577ef5..d27c4ef52a 100644 --- a/packages/core/docs/functions/hasValidHierarchy.md +++ b/packages/core/docs/functions/hasValidHierarchy.md @@ -6,20 +6,20 @@ # Function: hasValidHierarchy() -> **hasValidHierarchy**(`node`, `context`, `hint`): `boolean` +> **hasValidHierarchy**(`context`, `node`, `hint`): `boolean` internal ## Parameters -### node - -[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) + ### hint `bigint` diff --git a/packages/core/docs/functions/isChildrenCount.md b/packages/core/docs/functions/isChildrenCount.md index e94cf5445f..99e222a673 100644 --- a/packages/core/docs/functions/isChildrenCount.md +++ b/packages/core/docs/functions/isChildrenCount.md @@ -6,18 +6,18 @@ # Function: isChildrenCount() -> **isChildrenCount**(`node`, `context`): `boolean` +> **isChildrenCount**(`context`, `node`): `boolean` ## Parameters -### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenCountCall.md b/packages/core/docs/functions/isChildrenCountCall.md index 12f733e49b..35a797f2b7 100644 --- a/packages/core/docs/functions/isChildrenCountCall.md +++ b/packages/core/docs/functions/isChildrenCountCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isChildrenCountCall**(`node`, `context`): `node is CallExpression & { callee: MemberExpression }` +> **isChildrenCountCall**(`context`, `node`): `node is CallExpression & { callee: MemberExpression }` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/docs/functions/isChildrenForEach.md b/packages/core/docs/functions/isChildrenForEach.md index fb07d43444..3457e4607c 100644 --- a/packages/core/docs/functions/isChildrenForEach.md +++ b/packages/core/docs/functions/isChildrenForEach.md @@ -6,18 +6,18 @@ # Function: isChildrenForEach() -> **isChildrenForEach**(`node`, `context`): `boolean` +> **isChildrenForEach**(`context`, `node`): `boolean` ## Parameters -### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenForEachCall.md b/packages/core/docs/functions/isChildrenForEachCall.md index 07013a8287..34bf3e157e 100644 --- a/packages/core/docs/functions/isChildrenForEachCall.md +++ b/packages/core/docs/functions/isChildrenForEachCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isChildrenForEachCall**(`node`, `context`): `node is CallExpression & { callee: MemberExpression }` +> **isChildrenForEachCall**(`context`, `node`): `node is CallExpression & { callee: MemberExpression }` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/docs/functions/isChildrenMap.md b/packages/core/docs/functions/isChildrenMap.md index 45628bcb22..252aade479 100644 --- a/packages/core/docs/functions/isChildrenMap.md +++ b/packages/core/docs/functions/isChildrenMap.md @@ -6,18 +6,18 @@ # Function: isChildrenMap() -> **isChildrenMap**(`node`, `context`): `boolean` +> **isChildrenMap**(`context`, `node`): `boolean` ## Parameters -### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenMapCall.md b/packages/core/docs/functions/isChildrenMapCall.md index b8ecbab170..59cf3e1c81 100644 --- a/packages/core/docs/functions/isChildrenMapCall.md +++ b/packages/core/docs/functions/isChildrenMapCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isChildrenMapCall**(`node`, `context`): `node is CallExpression & { callee: MemberExpression }` +> **isChildrenMapCall**(`context`, `node`): `node is CallExpression & { callee: MemberExpression }` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/docs/functions/isChildrenOfCreateElement.md b/packages/core/docs/functions/isChildrenOfCreateElement.md index 39d9fec17f..9619ecf0c3 100644 --- a/packages/core/docs/functions/isChildrenOfCreateElement.md +++ b/packages/core/docs/functions/isChildrenOfCreateElement.md @@ -6,24 +6,24 @@ # Function: isChildrenOfCreateElement() -> **isChildrenOfCreateElement**(`node`, `context`): `boolean` +> **isChildrenOfCreateElement**(`context`, `node`): `boolean` Determines whether inside `createElement`'s children. ## Parameters -### node - -[`Node`](../-internal-/type-aliases/Node.md) - -The AST node to check - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) The rule context +### node + +[`Node`](../-internal-/type-aliases/Node.md) + +The AST node to check + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenOnly.md b/packages/core/docs/functions/isChildrenOnly.md index 49335076e2..9e797868a5 100644 --- a/packages/core/docs/functions/isChildrenOnly.md +++ b/packages/core/docs/functions/isChildrenOnly.md @@ -6,18 +6,18 @@ # Function: isChildrenOnly() -> **isChildrenOnly**(`node`, `context`): `boolean` +> **isChildrenOnly**(`context`, `node`): `boolean` ## Parameters -### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenOnlyCall.md b/packages/core/docs/functions/isChildrenOnlyCall.md index ae41d58309..abfa8a8a84 100644 --- a/packages/core/docs/functions/isChildrenOnlyCall.md +++ b/packages/core/docs/functions/isChildrenOnlyCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isChildrenOnlyCall**(`node`, `context`): `node is CallExpression & { callee: MemberExpression }` +> **isChildrenOnlyCall**(`context`, `node`): `node is CallExpression & { callee: MemberExpression }` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/docs/functions/isChildrenToArray.md b/packages/core/docs/functions/isChildrenToArray.md index 396a97a4b0..d068480c9b 100644 --- a/packages/core/docs/functions/isChildrenToArray.md +++ b/packages/core/docs/functions/isChildrenToArray.md @@ -6,18 +6,18 @@ # Function: isChildrenToArray() -> **isChildrenToArray**(`node`, `context`): `boolean` +> **isChildrenToArray**(`context`, `node`): `boolean` ## Parameters -### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isChildrenToArrayCall.md b/packages/core/docs/functions/isChildrenToArrayCall.md index 44b4b4e847..6d967488f2 100644 --- a/packages/core/docs/functions/isChildrenToArrayCall.md +++ b/packages/core/docs/functions/isChildrenToArrayCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isChildrenToArrayCall**(`node`, `context`): `node is CallExpression & { callee: MemberExpression }` +> **isChildrenToArrayCall**(`context`, `node`): `node is CallExpression & { callee: MemberExpression }` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/docs/functions/isCloneElement.md b/packages/core/docs/functions/isCloneElement.md index c33722c366..6fe1ada8d6 100644 --- a/packages/core/docs/functions/isCloneElement.md +++ b/packages/core/docs/functions/isCloneElement.md @@ -6,18 +6,18 @@ # Function: isCloneElement() -> **isCloneElement**(`node`, `context`): `boolean` +> **isCloneElement**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isCloneElementCall.md b/packages/core/docs/functions/isCloneElementCall.md index fac4b57d21..351e31b6dd 100644 --- a/packages/core/docs/functions/isCloneElementCall.md +++ b/packages/core/docs/functions/isCloneElementCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isCloneElementCall**(`node`, `context`): `node is CallExpression` +> **isCloneElementCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isCreateContext.md b/packages/core/docs/functions/isCreateContext.md index b1bdca1b3d..c72c8c2aa1 100644 --- a/packages/core/docs/functions/isCreateContext.md +++ b/packages/core/docs/functions/isCreateContext.md @@ -6,18 +6,18 @@ # Function: isCreateContext() -> **isCreateContext**(`node`, `context`): `boolean` +> **isCreateContext**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isCreateContextCall.md b/packages/core/docs/functions/isCreateContextCall.md index a6b948afd9..35cffd96c0 100644 --- a/packages/core/docs/functions/isCreateContextCall.md +++ b/packages/core/docs/functions/isCreateContextCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isCreateContextCall**(`node`, `context`): `node is CallExpression` +> **isCreateContextCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isCreateElement.md b/packages/core/docs/functions/isCreateElement.md index 4729b9eb79..fd55c76310 100644 --- a/packages/core/docs/functions/isCreateElement.md +++ b/packages/core/docs/functions/isCreateElement.md @@ -6,18 +6,18 @@ # Function: isCreateElement() -> **isCreateElement**(`node`, `context`): `boolean` +> **isCreateElement**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isCreateElementCall.md b/packages/core/docs/functions/isCreateElementCall.md index ee73fc6123..b078bc4a41 100644 --- a/packages/core/docs/functions/isCreateElementCall.md +++ b/packages/core/docs/functions/isCreateElementCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isCreateElementCall**(`node`, `context`): `node is CallExpression` +> **isCreateElementCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isCreateRef.md b/packages/core/docs/functions/isCreateRef.md index 5163d63b88..f7676300b1 100644 --- a/packages/core/docs/functions/isCreateRef.md +++ b/packages/core/docs/functions/isCreateRef.md @@ -6,18 +6,18 @@ # Function: isCreateRef() -> **isCreateRef**(`node`, `context`): `boolean` +> **isCreateRef**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isCreateRefCall.md b/packages/core/docs/functions/isCreateRefCall.md index 4e34d4e423..b79feda830 100644 --- a/packages/core/docs/functions/isCreateRefCall.md +++ b/packages/core/docs/functions/isCreateRefCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isCreateRefCall**(`node`, `context`): `node is CallExpression` +> **isCreateRefCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isForwardRef.md b/packages/core/docs/functions/isForwardRef.md index 596eb2f64f..3006b53934 100644 --- a/packages/core/docs/functions/isForwardRef.md +++ b/packages/core/docs/functions/isForwardRef.md @@ -6,18 +6,18 @@ # Function: isForwardRef() -> **isForwardRef**(`node`, `context`): `boolean` +> **isForwardRef**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isForwardRefCall.md b/packages/core/docs/functions/isForwardRefCall.md index 6c9aff06b3..2e3392272f 100644 --- a/packages/core/docs/functions/isForwardRefCall.md +++ b/packages/core/docs/functions/isForwardRefCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isForwardRefCall**(`node`, `context`): `node is CallExpression` +> **isForwardRefCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isFromReact.md b/packages/core/docs/functions/isFromReact.md index d1e7d56232..b8ded017d5 100644 --- a/packages/core/docs/functions/isFromReact.md +++ b/packages/core/docs/functions/isFromReact.md @@ -6,7 +6,7 @@ # Function: isFromReact() -> **isFromReact**(`name`): (`node`, `context`) => `boolean` +> **isFromReact**(`name`): (`context`, `node`) => `boolean` ## Parameters @@ -20,14 +20,14 @@ ### Parameters -#### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ### Returns `boolean` diff --git a/packages/core/docs/functions/isFromReactMember.md b/packages/core/docs/functions/isFromReactMember.md index f6459fb234..f1d9b12a70 100644 --- a/packages/core/docs/functions/isFromReactMember.md +++ b/packages/core/docs/functions/isFromReactMember.md @@ -6,7 +6,7 @@ # Function: isFromReactMember() -> **isFromReactMember**(`memberName`, `name`): (`node`, `context`) => `boolean` +> **isFromReactMember**(`memberName`, `name`): (`context`, `node`) => `boolean` ## Parameters @@ -24,14 +24,14 @@ ### Parameters -#### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ### Returns `boolean` diff --git a/packages/core/docs/functions/isInsideCreateElementProps.md b/packages/core/docs/functions/isInsideCreateElementProps.md index 00d8fad404..f9422c6f9f 100644 --- a/packages/core/docs/functions/isInsideCreateElementProps.md +++ b/packages/core/docs/functions/isInsideCreateElementProps.md @@ -6,24 +6,24 @@ # Function: isInsideCreateElementProps() -> **isInsideCreateElementProps**(`node`, `context`): `boolean` +> **isInsideCreateElementProps**(`context`, `node`): `boolean` Determines whether inside `createElement`'s props. ## Parameters -### node - -[`Node`](../-internal-/type-aliases/Node.md) - -The AST node to check - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) The rule context +### node + +[`Node`](../-internal-/type-aliases/Node.md) + +The AST node to check + ## Returns `boolean` diff --git a/packages/core/docs/functions/isMemo.md b/packages/core/docs/functions/isMemo.md index baadf27f7e..96e7ee606f 100644 --- a/packages/core/docs/functions/isMemo.md +++ b/packages/core/docs/functions/isMemo.md @@ -6,18 +6,18 @@ # Function: isMemo() -> **isMemo**(`node`, `context`): `boolean` +> **isMemo**(`context`, `node`): `boolean` ## Parameters -### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + ## Returns `boolean` diff --git a/packages/core/docs/functions/isMemoCall.md b/packages/core/docs/functions/isMemoCall.md index f514384d10..7bba39ff1c 100644 --- a/packages/core/docs/functions/isMemoCall.md +++ b/packages/core/docs/functions/isMemoCall.md @@ -32,18 +32,18 @@ ## Call Signature -> **isMemoCall**(`node`, `context`): `node is CallExpression` +> **isMemoCall**(`context`, `node`): `node is CallExpression` ### Parameters -#### node - -[`Node`](../-internal-/type-aliases/Node.md) - #### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +#### node + +[`Node`](../-internal-/type-aliases/Node.md) + ### Returns `node is CallExpression` diff --git a/packages/core/docs/functions/isReactAPIWithName.md b/packages/core/docs/functions/isReactAPIWithName.md index 31495fac3c..d9b5da292e 100644 --- a/packages/core/docs/functions/isReactAPIWithName.md +++ b/packages/core/docs/functions/isReactAPIWithName.md @@ -8,7 +8,7 @@ ## Call Signature -> **isReactAPIWithName**(`name`): (`node`, `context`) => `boolean` +> **isReactAPIWithName**(`name`): (`context`, `node`) => `boolean` ### Parameters @@ -22,21 +22,21 @@ #### Parameters -##### node - -[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ##### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +##### node + +[`Identifier`](../-internal-/interfaces/Identifier.md) | [`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + #### Returns `boolean` ## Call Signature -> **isReactAPIWithName**(`name`, `member`): (`node`, `context`) => `boolean` +> **isReactAPIWithName**(`name`, `member`): (`context`, `node`) => `boolean` ### Parameters @@ -54,14 +54,14 @@ #### Parameters -##### node - -[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) - ##### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +##### node + +[`MemberExpression`](../-internal-/type-aliases/MemberExpression.md) + #### Returns `boolean` diff --git a/packages/core/docs/functions/isReactHookCallWithName.md b/packages/core/docs/functions/isReactHookCallWithName.md index 54f31b003e..e27f56b5e7 100644 --- a/packages/core/docs/functions/isReactHookCallWithName.md +++ b/packages/core/docs/functions/isReactHookCallWithName.md @@ -6,18 +6,18 @@ # Function: isReactHookCallWithName() -> **isReactHookCallWithName**(`node`, `context`): (`name`) => `boolean` +> **isReactHookCallWithName**(`context`, `node`): (`name`) => `boolean` ## Parameters -### node - -`undefined` | [`CallExpression`](../-internal-/interfaces/CallExpression.md) - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### node + +`undefined` | [`CallExpression`](../-internal-/interfaces/CallExpression.md) + ## Returns `Function` diff --git a/packages/core/docs/functions/isReactHookCallWithNameAlias.md b/packages/core/docs/functions/isReactHookCallWithNameAlias.md index 2235c441d7..af985a6245 100644 --- a/packages/core/docs/functions/isReactHookCallWithNameAlias.md +++ b/packages/core/docs/functions/isReactHookCallWithNameAlias.md @@ -6,18 +6,18 @@ # Function: isReactHookCallWithNameAlias() -> **isReactHookCallWithNameAlias**(`name`, `context`, `alias`): (`node`) => `boolean` +> **isReactHookCallWithNameAlias**(`context`, `name`, `alias`): (`node`) => `boolean` ## Parameters -### name - -`string` - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) +### name + +`string` + ### alias `string`[] diff --git a/packages/core/docs/functions/isRenderFunctionLoose.md b/packages/core/docs/functions/isRenderFunctionLoose.md index aabcbc1811..6cf0b70bdb 100644 --- a/packages/core/docs/functions/isRenderFunctionLoose.md +++ b/packages/core/docs/functions/isRenderFunctionLoose.md @@ -6,7 +6,7 @@ # Function: isRenderFunctionLoose() -> **isRenderFunctionLoose**(`node`, `context`): `boolean` +> **isRenderFunctionLoose**(`context`, `node`): `boolean` Unsafe check whether given node is a render function ```jsx @@ -18,18 +18,18 @@ _ =
} /> ## Parameters -### node - -[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) - -The AST node to check - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) The rule context +### node + +[`TSESTreeFunction`](../-internal-/type-aliases/TSESTreeFunction.md) + +The AST node to check + ## Returns `boolean` diff --git a/packages/core/docs/functions/isRenderPropLoose.md b/packages/core/docs/functions/isRenderPropLoose.md index 5500936042..d4668206a4 100644 --- a/packages/core/docs/functions/isRenderPropLoose.md +++ b/packages/core/docs/functions/isRenderPropLoose.md @@ -6,7 +6,7 @@ # Function: isRenderPropLoose() -> **isRenderPropLoose**(`node`, `context`): `boolean` +> **isRenderPropLoose**(`context`, `node`): `boolean` Unsafe check whether given JSXAttribute is a render prop ```jsx @@ -16,18 +16,18 @@ _ =
} /> ## Parameters -### node - -`JSXAttribute` - -The AST node to check - ### context [`Readonly`](../-internal-/type-aliases/Readonly.md) The rule context +### node + +`JSXAttribute` + +The AST node to check + ## Returns `boolean` diff --git a/packages/core/docs/functions/isUseCallbackCall.md b/packages/core/docs/functions/isUseCallbackCall.md index 0e50bb84e0..cffad8e388 100644 --- a/packages/core/docs/functions/isUseCallbackCall.md +++ b/packages/core/docs/functions/isUseCallbackCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseContextCall.md b/packages/core/docs/functions/isUseContextCall.md index fe7d4d1112..7973c1ea53 100644 --- a/packages/core/docs/functions/isUseContextCall.md +++ b/packages/core/docs/functions/isUseContextCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseDebugValueCall.md b/packages/core/docs/functions/isUseDebugValueCall.md index 19c04a19c3..4931b4fd74 100644 --- a/packages/core/docs/functions/isUseDebugValueCall.md +++ b/packages/core/docs/functions/isUseDebugValueCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseDeferredValueCall.md b/packages/core/docs/functions/isUseDeferredValueCall.md index bd7f587a73..502e9abe1c 100644 --- a/packages/core/docs/functions/isUseDeferredValueCall.md +++ b/packages/core/docs/functions/isUseDeferredValueCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseEffectCall.md b/packages/core/docs/functions/isUseEffectCall.md index 299f2396db..a1d6544fba 100644 --- a/packages/core/docs/functions/isUseEffectCall.md +++ b/packages/core/docs/functions/isUseEffectCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseIdCall.md b/packages/core/docs/functions/isUseIdCall.md index 032c7bf335..2f7434be37 100644 --- a/packages/core/docs/functions/isUseIdCall.md +++ b/packages/core/docs/functions/isUseIdCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseImperativeHandleCall.md b/packages/core/docs/functions/isUseImperativeHandleCall.md index c61855fd32..c304044a8e 100644 --- a/packages/core/docs/functions/isUseImperativeHandleCall.md +++ b/packages/core/docs/functions/isUseImperativeHandleCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseInsertionEffectCall.md b/packages/core/docs/functions/isUseInsertionEffectCall.md index faedc1f647..6acf282052 100644 --- a/packages/core/docs/functions/isUseInsertionEffectCall.md +++ b/packages/core/docs/functions/isUseInsertionEffectCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseLayoutEffectCall.md b/packages/core/docs/functions/isUseLayoutEffectCall.md index b4b50e8e0c..1f4d6ec516 100644 --- a/packages/core/docs/functions/isUseLayoutEffectCall.md +++ b/packages/core/docs/functions/isUseLayoutEffectCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseMemoCall.md b/packages/core/docs/functions/isUseMemoCall.md index d431ecaa61..66745d1949 100644 --- a/packages/core/docs/functions/isUseMemoCall.md +++ b/packages/core/docs/functions/isUseMemoCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseReducerCall.md b/packages/core/docs/functions/isUseReducerCall.md index 7c87caa720..2e63ad06c3 100644 --- a/packages/core/docs/functions/isUseReducerCall.md +++ b/packages/core/docs/functions/isUseReducerCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseRefCall.md b/packages/core/docs/functions/isUseRefCall.md index 9e72304aa5..cf7183096b 100644 --- a/packages/core/docs/functions/isUseRefCall.md +++ b/packages/core/docs/functions/isUseRefCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseStateCall.md b/packages/core/docs/functions/isUseStateCall.md index 3aa12f56ab..59d4486b42 100644 --- a/packages/core/docs/functions/isUseStateCall.md +++ b/packages/core/docs/functions/isUseStateCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseSyncExternalStoreCall.md b/packages/core/docs/functions/isUseSyncExternalStoreCall.md index 8c156d0f62..f4c62eb9bb 100644 --- a/packages/core/docs/functions/isUseSyncExternalStoreCall.md +++ b/packages/core/docs/functions/isUseSyncExternalStoreCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/functions/isUseTransitionCall.md b/packages/core/docs/functions/isUseTransitionCall.md index ff083d52c3..27b848b439 100644 --- a/packages/core/docs/functions/isUseTransitionCall.md +++ b/packages/core/docs/functions/isUseTransitionCall.md @@ -12,7 +12,7 @@ ### a -...\[[`CallExpression`](../-internal-/interfaces/CallExpression.md), [`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>\] +...\[[`Readonly`](../-internal-/type-aliases/Readonly.md)\<[`RuleContext`](../-internal-/interfaces/RuleContext.md)\<`string`, readonly `unknown`[]\>\>, [`CallExpression`](../-internal-/interfaces/CallExpression.md)\] ## Returns diff --git a/packages/core/docs/namespaces/isCallFromReact/type-aliases/ReturnType.md b/packages/core/docs/namespaces/isCallFromReact/type-aliases/ReturnType.md index 343c1f4d28..b921281f81 100644 --- a/packages/core/docs/namespaces/isCallFromReact/type-aliases/ReturnType.md +++ b/packages/core/docs/namespaces/isCallFromReact/type-aliases/ReturnType.md @@ -6,7 +6,7 @@ # Type Alias: ReturnType() -> **ReturnType**: (`context`) => (`node`) => `node is CallExpression`(`node`, `context`) => `node is CallExpression` +> **ReturnType**: (`context`) => (`node`) => `node is CallExpression`(`context`, `node`) => `node is CallExpression` ## Parameters @@ -30,14 +30,14 @@ ## Parameters -### node - -[`Node`](../../../-internal-/type-aliases/Node.md) - ### context [`Readonly`](../../../-internal-/type-aliases/Readonly.md) +### node + +[`Node`](../../../-internal-/type-aliases/Node.md) + ## Returns `node is CallExpression` diff --git a/packages/core/docs/namespaces/isCallFromReactMember/type-aliases/ReturnType.md b/packages/core/docs/namespaces/isCallFromReactMember/type-aliases/ReturnType.md index 0e9da6e593..c9edab5159 100644 --- a/packages/core/docs/namespaces/isCallFromReactMember/type-aliases/ReturnType.md +++ b/packages/core/docs/namespaces/isCallFromReactMember/type-aliases/ReturnType.md @@ -6,7 +6,7 @@ # Type Alias: ReturnType() -> **ReturnType**: (`context`) => (`node`) => `node is CallExpression & { callee: MemberExpression }`(`node`, `context`) => `node is CallExpression & { callee: MemberExpression }` +> **ReturnType**: (`context`) => (`node`) => `node is CallExpression & { callee: MemberExpression }`(`context`, `node`) => `node is CallExpression & { callee: MemberExpression }` ## Parameters @@ -30,14 +30,14 @@ ## Parameters -### node - -[`Node`](../../../-internal-/type-aliases/Node.md) - ### context [`Readonly`](../../../-internal-/type-aliases/Readonly.md) +### node + +[`Node`](../../../-internal-/type-aliases/Node.md) + ## Returns `node is CallExpression & { callee: MemberExpression }` diff --git a/packages/core/src/component/component-collector.ts b/packages/core/src/component/component-collector.ts index 31fae8931a..74fd05df38 100644 --- a/packages/core/src/component/component-collector.ts +++ b/packages/core/src/component/component-collector.ts @@ -97,12 +97,12 @@ export function useComponentCollector( const entry = getCurrentEntry(); if (entry == null) return; const { body } = entry.node; - const isComponent = hasNoneOrValidComponentName(entry.node, context) + const isComponent = hasNoneOrValidComponentName(context, entry.node) && JSX.isJSXValue(body, jsxCtx, hint) - && hasValidHierarchy(entry.node, context, hint); + && hasValidHierarchy(context, entry.node, hint); if (!isComponent) return; const initPath = AST.getFunctionInitPath(entry.node); - const id = getFunctionComponentIdentifier(entry.node, context); + const id = getFunctionComponentIdentifier(context, entry.node); const name = getComponentNameFromIdentifier(id); const key = getId(); components.set(key, { @@ -146,13 +146,13 @@ export function useComponentCollector( "ReturnStatement[type]"(node: TSESTree.ReturnStatement) { const entry = getCurrentEntry(); if (entry == null) return; - const isComponent = hasNoneOrValidComponentName(entry.node, context) + const isComponent = hasNoneOrValidComponentName(context, entry.node) && JSX.isJSXValue(node.argument, jsxCtx, hint) - && hasValidHierarchy(entry.node, context, hint); + && hasValidHierarchy(context, entry.node, hint); if (!isComponent) return; entry.isComponent = true; const initPath = AST.getFunctionInitPath(entry.node); - const id = getFunctionComponentIdentifier(entry.node, context); + const id = getFunctionComponentIdentifier(context, entry.node); const name = getComponentNameFromIdentifier(id); components.set(entry.key, { id, diff --git a/packages/core/src/component/component-id.ts b/packages/core/src/component/component-id.ts index f7cf4d6174..8252e3f99d 100644 --- a/packages/core/src/component/component-id.ts +++ b/packages/core/src/component/component-id.ts @@ -7,18 +7,21 @@ import { AST_NODE_TYPES as T } from "@typescript-eslint/types"; import { isReactHookCallWithNameLoose } from "../hook"; import { isForwardRefCall, isMemoCall } from "../utils"; -function isComponentWrapperCall(node: TSESTree.Node, context: RuleContext) { +function isComponentWrapperCall( + context: RuleContext, + node: TSESTree.Node, +) { if (node.type !== T.CallExpression) { return false; } - return isMemoCall(node, context) - || isForwardRefCall(node, context) + return isMemoCall(context, node) + || isForwardRefCall(context, node) || isReactHookCallWithNameLoose(node)("useCallback"); } export function getFunctionComponentIdentifier( - node: AST.TSESTreeFunction, context: RuleContext, + node: AST.TSESTreeFunction, ): TSESTree.Identifier | TSESTree.Identifier[] | _ { const functionId = AST.getFunctionIdentifier(node); if (functionId != null) { @@ -28,7 +31,7 @@ export function getFunctionComponentIdentifier( // Get function component identifier from `const Component = memo(() => {});` if ( parent.type === T.CallExpression - && isComponentWrapperCall(parent, context) + && isComponentWrapperCall(context, parent) && parent.parent.type === T.VariableDeclarator && parent.parent.id.type === T.Identifier ) { @@ -37,9 +40,9 @@ export function getFunctionComponentIdentifier( // Get function component identifier from `const Component = memo(forwardRef(() => {}));` if ( parent.type === T.CallExpression - && isComponentWrapperCall(parent, context) + && isComponentWrapperCall(context, parent) && parent.parent.type === T.CallExpression - && isComponentWrapperCall(parent.parent, context) + && isComponentWrapperCall(context, parent.parent) && parent.parent.parent.type === T.VariableDeclarator && parent.parent.parent.id.type === T.Identifier ) { diff --git a/packages/core/src/component/component-name.ts b/packages/core/src/component/component-name.ts index 8d4547439a..865eab2e72 100644 --- a/packages/core/src/component/component-name.ts +++ b/packages/core/src/component/component-name.ts @@ -18,8 +18,8 @@ export function isComponentName(name: string) { return RE_COMPONENT_NAME.test(name); } -export function hasNoneOrValidComponentName(node: AST.TSESTreeFunction, context: RuleContext) { - const id = getFunctionComponentIdentifier(node, context); +export function hasNoneOrValidComponentName(context: RuleContext, node: AST.TSESTreeFunction) { + const id = getFunctionComponentIdentifier(context, node); if (id == null) return true; const name = Array.isArray(id) ? id.at(-1)?.name diff --git a/packages/core/src/component/component-render-prop.ts b/packages/core/src/component/component-render-prop.ts index a96c039d2b..afe37489be 100644 --- a/packages/core/src/component/component-render-prop.ts +++ b/packages/core/src/component/component-render-prop.ts @@ -12,11 +12,11 @@ import { AST_NODE_TYPES as T } from "@typescript-eslint/types"; * _ =
} /> * ` ^^^^^^^^^^^^^ ` * ``` - * @param node The AST node to check * @param context The rule context + * @param node The AST node to check * @returns `true` if node is a render function, `false` if not */ -export function isRenderFunctionLoose(node: AST.TSESTreeFunction, context: RuleContext) { +export function isRenderFunctionLoose(context: RuleContext, node: AST.TSESTreeFunction) { const { body, parent } = node; // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions if (AST.getFunctionIdentifier(node)?.name.startsWith("render")) { @@ -43,18 +43,18 @@ export function isRenderFunctionLoose(node: AST.TSESTreeFunction, context: RuleC * _ =
} /> * ` ^^^^^^^^^^^^^^^^^^^^^^^^^ ` * ``` - * @param node The AST node to check * @param context The rule context + * @param node The AST node to check * @returns `true` if node is a render prop, `false` if not */ -export function isRenderPropLoose(node: TSESTree.JSXAttribute, context: RuleContext) { +export function isRenderPropLoose(context: RuleContext, node: TSESTree.JSXAttribute) { if (node.name.type !== T.JSXIdentifier) { return false; } return node.name.name.startsWith("render") && node.value?.type === T.JSXExpressionContainer && AST.isFunction(node.value.expression) - && isRenderFunctionLoose(node.value.expression, context); + && isRenderFunctionLoose(context, node.value.expression); } /** diff --git a/packages/core/src/component/hierarchy.ts b/packages/core/src/component/hierarchy.ts index 14db55bb83..892e46411c 100644 --- a/packages/core/src/component/hierarchy.ts +++ b/packages/core/src/component/hierarchy.ts @@ -9,8 +9,8 @@ import { ERComponentHint } from "./component-collector-hint"; import { isFunctionOfRenderMethod } from "./component-lifecycle"; /** internal */ -export function hasValidHierarchy(node: AST.TSESTreeFunction, context: RuleContext, hint: bigint) { - if (isChildrenOfCreateElement(node, context) || isFunctionOfRenderMethod(node)) { +export function hasValidHierarchy(context: RuleContext, node: AST.TSESTreeFunction, hint: bigint) { + if (isChildrenOfCreateElement(context, node) || isFunctionOfRenderMethod(node)) { return false; } if (hint & ERComponentHint.SkipMapCallback && AST.isMapCallLoose(node.parent)) { diff --git a/packages/core/src/element/hierarchy.ts b/packages/core/src/element/hierarchy.ts index 2f1246e586..12221487ed 100644 --- a/packages/core/src/element/hierarchy.ts +++ b/packages/core/src/element/hierarchy.ts @@ -7,13 +7,13 @@ import { isCreateElementCall } from "../utils"; /** * Determines whether inside `createElement`'s props. - * @param node The AST node to check * @param context The rule context + * @param node The AST node to check * @returns `true` if the node is inside createElement's props */ export function isInsideCreateElementProps( - node: TSESTree.Node, context: RuleContext, + node: TSESTree.Node, ) { const call = AST.findParentNode(node, isCreateElementCall(context)); if (call == null) return false; @@ -24,17 +24,17 @@ export function isInsideCreateElementProps( /** * Determines whether inside `createElement`'s children. - * @param node The AST node to check * @param context The rule context + * @param node The AST node to check * @returns `true` if the node is inside createElement's children */ export function isChildrenOfCreateElement( - node: TSESTree.Node, context: RuleContext, + node: TSESTree.Node, ) { const parent = node.parent; if (parent == null || parent.type !== T.CallExpression) return false; - if (!isCreateElementCall(parent, context)) return false; + if (!isCreateElementCall(context, parent)) return false; return parent.arguments .slice(2) .some((arg) => arg === node); diff --git a/packages/core/src/hook/is.ts b/packages/core/src/hook/is.ts index 39e3343382..7185f1b68b 100644 --- a/packages/core/src/hook/is.ts +++ b/packages/core/src/hook/is.ts @@ -33,7 +33,7 @@ export function isReactHookCall(node: TSESTree.Node | _) { return false; } -export function isReactHookCallWithName(node: TSESTree.CallExpression | _, context: RuleContext) { +export function isReactHookCallWithName(context: RuleContext, node: TSESTree.CallExpression | _) { if (node == null) return constFalse; const settings = unsafeDecodeSettings(context.settings); const importSource = settings.importSource ?? "react"; @@ -72,7 +72,7 @@ export function isReactHookCallWithNameLoose(node: TSESTree.CallExpression | _) }; } -export function isReactHookCallWithNameAlias(name: string, context: RuleContext, alias: string[]) { +export function isReactHookCallWithNameAlias(context: RuleContext, name: string, alias: string[]) { const settings = unsafeDecodeSettings(context); const importSource = settings.importSource ?? "react"; return (node: TSESTree.CallExpression) => { diff --git a/packages/core/src/utils/is-call-from-react.ts b/packages/core/src/utils/is-call-from-react.ts index 1399e35c5b..10d34bdd70 100644 --- a/packages/core/src/utils/is-call-from-react.ts +++ b/packages/core/src/utils/is-call-from-react.ts @@ -9,15 +9,15 @@ import { isFromReact, isFromReactMember } from "./is-from-react"; export declare namespace isCallFromReact { type ReturnType = { (context: RuleContext): (node: TSESTree.Node) => node is TSESTree.CallExpression; - (node: TSESTree.Node, context: RuleContext): node is TSESTree.CallExpression; + (context: RuleContext, node: TSESTree.Node): node is TSESTree.CallExpression; }; } export function isCallFromReact(name: string): isCallFromReact.ReturnType { - return dual(2, (node: TSESTree.Node, context: RuleContext): node is TSESTree.CallExpression => { + return dual(2, (context: RuleContext, node: TSESTree.Node): node is TSESTree.CallExpression => { if (node.type !== T.CallExpression) return false; if (!AST.isOneOf([T.Identifier, T.MemberExpression])(node.callee)) return false; - return isFromReact(name)(node.callee, context); + return isFromReact(name)(context, node.callee); }); } @@ -26,7 +26,7 @@ export declare namespace isCallFromReactMember { (context: RuleContext): (node: TSESTree.Node) => node is & TSESTree.CallExpression & { callee: TSESTree.MemberExpression }; - (node: TSESTree.Node, context: RuleContext): node is + (context: RuleContext, node: TSESTree.Node): node is & TSESTree.CallExpression & { callee: TSESTree.MemberExpression }; }; @@ -36,12 +36,12 @@ export function isCallFromReactMember( pragmaMemberName: string, name: string, ): isCallFromReactMember.ReturnType { - return dual(2, (node: TSESTree.Node, context: RuleContext): node is + return dual(2, (context: RuleContext, node: TSESTree.Node): node is & TSESTree.CallExpression & { callee: TSESTree.MemberExpression } => { if (node.type !== T.CallExpression) return false; if (!AST.is(T.MemberExpression)(node.callee)) return false; - return isFromReactMember(pragmaMemberName, name)(node.callee, context); + return isFromReactMember(pragmaMemberName, name)(context, node.callee); }); } diff --git a/packages/core/src/utils/is-from-react.ts b/packages/core/src/utils/is-from-react.ts index dcc66ae80a..4671a31ec0 100644 --- a/packages/core/src/utils/is-from-react.ts +++ b/packages/core/src/utils/is-from-react.ts @@ -38,7 +38,7 @@ export function isFromReactStrict( } export function isFromReact(name: string) { - return (node: TSESTree.Identifier | TSESTree.MemberExpression, context: RuleContext) => { + return (context: RuleContext, node: TSESTree.Identifier | TSESTree.MemberExpression) => { const { importSource = defaultImportSource, strictImportCheck = false } = unsafeDecodeSettings(context.settings); if (!strictImportCheck) return isFromReactLoose(node, name); return isFromReactStrict(node, name, importSource, context.sourceCode.getScope(node)); @@ -87,7 +87,7 @@ export function isFromReactMemberStrict( } export function isFromReactMember(memberName: string, name: string) { - return (node: TSESTree.MemberExpression, context: RuleContext) => { + return (context: RuleContext, node: TSESTree.MemberExpression) => { const { importSource = defaultImportSource, strictImportCheck = false } = unsafeDecodeSettings(context.settings); if (!strictImportCheck) return isFromReactMemberLoose(node, memberName, name); return isFromReactMemberStrict(node, memberName, name, importSource, context.sourceCode.getScope(node)); diff --git a/packages/plugins/eslint-plugin-react-debug/src/rules/is-from-react.ts b/packages/plugins/eslint-plugin-react-debug/src/rules/is-from-react.ts index 9507b58dcd..70bc6fc9c2 100644 --- a/packages/plugins/eslint-plugin-react-debug/src/rules/is-from-react.ts +++ b/packages/plugins/eslint-plugin-react-debug/src/rules/is-from-react.ts @@ -32,10 +32,7 @@ export default createRule<[], MessageID>({ create(context) { const { importSource = "react" } = getSettingsFromContext(context); - function isFromReact( - node: TSESTree.Identifier | TSESTree.JSXIdentifier, - initialScope: Scope, - ) { + function isFromReact(node: TSESTree.Identifier | TSESTree.JSXIdentifier, initialScope: Scope) { const name = node.name; switch (true) { case node.parent.type === T.MemberExpression @@ -50,18 +47,15 @@ export default createRule<[], MessageID>({ return isInitializedFromReact(name, importSource, initialScope); } } + function visitorFunction(node: TSESTree.Identifier | TSESTree.JSXIdentifier) { const shouldSkipDuplicate = node.parent.type === T.ImportSpecifier && node.parent.imported === node && node.parent.imported.name === node.parent.local.name; - if (shouldSkipDuplicate) { - return; - } + if (shouldSkipDuplicate) return; const name = node.name; const initialScope = context.sourceCode.getScope(node); - if (!isFromReact(node, initialScope)) { - return; - } + if (!isFromReact(node, initialScope)) return; context.report({ messageId: "isFromReact", node, diff --git a/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-button-type.ts b/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-button-type.ts index c606326b2e..452b452709 100644 --- a/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-button-type.ts +++ b/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-button-type.ts @@ -34,8 +34,8 @@ export default createRule<[], MessageID>({ return { JSXElement(node) { const [elementNameOnJsx, elementNameOnDom] = getElementNameOnJsxAndDom( - node, context, + node, polymorphicPropName, additionalComponents, ); diff --git a/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-iframe-sandbox.ts b/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-iframe-sandbox.ts index 2d730f8329..3a818bc8e0 100644 --- a/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-iframe-sandbox.ts +++ b/packages/plugins/eslint-plugin-react-dom/src/rules/no-missing-iframe-sandbox.ts @@ -59,8 +59,8 @@ export default createRule<[], MessageID>({ return { JSXElement(node) { const [elementNameOnJsx, elementNameOnDom] = getElementNameOnJsxAndDom( - node, context, + node, polymorphicPropName, additionalComponents, ); diff --git a/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-iframe-sandbox.ts b/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-iframe-sandbox.ts index 25426a8c56..993eeaa484 100644 --- a/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-iframe-sandbox.ts +++ b/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-iframe-sandbox.ts @@ -44,8 +44,8 @@ export default createRule<[], MessageID>({ return { JSXElement(node) { const [elementNameOnJsx, elementNameOnDom] = getElementNameOnJsxAndDom( - node, context, + node, polymorphicPropName, additionalComponents, ); diff --git a/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-target-blank.ts b/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-target-blank.ts index 994ae8a41c..07fad236f1 100644 --- a/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-target-blank.ts +++ b/packages/plugins/eslint-plugin-react-dom/src/rules/no-unsafe-target-blank.ts @@ -49,8 +49,8 @@ export default createRule<[], MessageID>({ return { JSXElement(node: TSESTree.JSXElement) { const [elementNameOnJsx, elementNameOnDom] = getElementNameOnJsxAndDom( - node, context, + node, polymorphicPropName, additionalComponents, ); diff --git a/packages/plugins/eslint-plugin-react-dom/src/utils/get-element-name-on-jsx-and-dom.ts b/packages/plugins/eslint-plugin-react-dom/src/utils/get-element-name-on-jsx-and-dom.ts index 765bdf117b..ad10b8db5e 100644 --- a/packages/plugins/eslint-plugin-react-dom/src/utils/get-element-name-on-jsx-and-dom.ts +++ b/packages/plugins/eslint-plugin-react-dom/src/utils/get-element-name-on-jsx-and-dom.ts @@ -3,8 +3,8 @@ import type { CustomComponentNormalized, RuleContext } from "@eslint-react/share import type { TSESTree } from "@typescript-eslint/types"; export function getElementNameOnJsxAndDom( - node: TSESTree.JSXElement, context: RuleContext, + node: TSESTree.JSXElement, polymorphicPropName?: string, additionalComponents: CustomComponentNormalized[] = [], ): [string, string] { diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-effect.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-effect.ts index 6ba3d8f88b..cc0f532968 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-effect.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-effect.ts @@ -49,10 +49,10 @@ export default createRule<[], MessageID>({ const settings = getSettingsFromContext(context); const additionalHooks = settings.additionalHooks; - const isUseEffectLikeCall = isReactHookCallWithNameAlias("useEffect", context, additionalHooks.useEffect ?? []); - const isUseStateCall = isReactHookCallWithNameAlias("useState", context, additionalHooks.useState ?? []); - const isUseMemoCall = isReactHookCallWithNameAlias("useMemo", context, additionalHooks.useMemo ?? []); - const isUseCallbackCall = isReactHookCallWithNameAlias("useCallback", context, additionalHooks.useCallback ?? []); + const isUseEffectLikeCall = isReactHookCallWithNameAlias(context, "useEffect", additionalHooks.useEffect ?? []); + const isUseStateCall = isReactHookCallWithNameAlias(context, "useState", additionalHooks.useState ?? []); + const isUseMemoCall = isReactHookCallWithNameAlias(context, "useMemo", additionalHooks.useMemo ?? []); + const isUseCallbackCall = isReactHookCallWithNameAlias(context, "useCallback", additionalHooks.useCallback ?? []); const isSetStateCall = isSetFunctionCall(context, settings); const isIdFromUseStateCall = isFromUseStateCall(context, settings); diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-layout-effect.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-layout-effect.ts index 2705089c4e..171f9daca2 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-layout-effect.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-direct-set-state-in-use-layout-effect.ts @@ -51,13 +51,13 @@ export default createRule<[], MessageID>({ const additionalHooks = settings.additionalHooks; const isUseLayoutEffectLikeCall = isReactHookCallWithNameAlias( - "useLayoutEffect", context, + "useLayoutEffect", additionalHooks.useLayoutEffect ?? [], ); - const isUseStateCall = isReactHookCallWithNameAlias("useState", context, additionalHooks.useState ?? []); - const isUseMemoCall = isReactHookCallWithNameAlias("useMemo", context, additionalHooks.useMemo ?? []); - const isUseCallbackCall = isReactHookCallWithNameAlias("useCallback", context, additionalHooks.useCallback ?? []); + const isUseStateCall = isReactHookCallWithNameAlias(context, "useState", additionalHooks.useState ?? []); + const isUseMemoCall = isReactHookCallWithNameAlias(context, "useMemo", additionalHooks.useMemo ?? []); + const isUseCallbackCall = isReactHookCallWithNameAlias(context, "useCallback", additionalHooks.useCallback ?? []); const isSetStateCall = isSetFunctionCall(context, settings); const isIdFromUseStateCall = isFromUseStateCall(context, settings); diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-callback.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-callback.ts index 0d9a91667c..7f3d7f8199 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-callback.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-callback.ts @@ -43,7 +43,7 @@ export default createRule<[], MessageID>({ return; } const initialScope = context.sourceCode.getScope(node); - if (!isUseCallbackCall(node, context) && !alias.some(isReactHookCallWithNameLoose(node))) { + if (!isUseCallbackCall(context, node) && !alias.some(isReactHookCallWithNameLoose(node))) { return; } const scope = context.sourceCode.getScope(node); diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-memo.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-memo.ts index 11f552dbe7..9273d1c1ac 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-memo.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-unnecessary-use-memo.ts @@ -42,7 +42,7 @@ export default createRule<[], MessageID>({ return; } const initialScope = context.sourceCode.getScope(node); - if (!isUseMemoCall(node, context) && !alias.some(isReactHookCallWithNameLoose(node))) { + if (!isUseMemoCall(context, node) && !alias.some(isReactHookCallWithNameLoose(node))) { return; } const scope = context.sourceCode.getScope(node); diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-useless-custom-hooks.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-useless-custom-hooks.ts index 9cca25218f..eb7cbea9ce 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-useless-custom-hooks.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/no-useless-custom-hooks.ts @@ -15,8 +15,8 @@ export const RULE_FEATURES = [ export type MessageID = CamelCase; function isNodeContainsUseCallComments( - node: TSESTree.Node, context: RuleContext, + node: TSESTree.Node, ) { return context.sourceCode .getCommentsInside(node) @@ -52,7 +52,7 @@ export default createRule<[], MessageID>({ continue; } // Skip hooks with comments that contain calls to other hooks - if (isNodeContainsUseCallComments(node, context)) { + if (isNodeContainsUseCallComments(context, node)) { continue; } context.report({ diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/prefer-use-state-lazy-initialization.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/prefer-use-state-lazy-initialization.ts index fb6267b41d..70aeebba23 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/prefer-use-state-lazy-initialization.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/rules/prefer-use-state-lazy-initialization.ts @@ -43,7 +43,7 @@ export default createRule<[], MessageID>({ if (!isReactHookCall(node)) { return; } - if (!isUseStateCall(node, context) && !alias.some(isReactHookCallWithNameLoose(node))) { + if (!isUseStateCall(context, node) && !alias.some(isReactHookCallWithNameLoose(node))) { return; } const [useStateInput] = node.arguments; diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-hook-call.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-hook-call.ts index e8d0967195..b3d93dd2a1 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-hook-call.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-hook-call.ts @@ -6,8 +6,8 @@ import type { TSESTree } from "@typescript-eslint/types"; import { AST_NODE_TYPES as T } from "@typescript-eslint/types"; export function isFromHookCall( - name: (typeof REACT_BUILD_IN_HOOKS)[number], context: RuleContext, + name: (typeof REACT_BUILD_IN_HOOKS)[number], settings: ESLintReactSettingsNormalized, predicate: (topLevelId: TSESTree.Identifier, call: TSESTree.CallExpression) => boolean = constTrue, ) { @@ -17,7 +17,7 @@ export function isFromHookCall( const variableNode = VAR.getVariableNode(variable, 0); if (variableNode == null) return false; if (variableNode.type !== T.CallExpression) return false; - if (!isReactHookCallWithNameAlias(name, context, hookAlias)(variableNode)) return false; + if (!isReactHookCallWithNameAlias(context, name, hookAlias)(variableNode)) return false; return predicate(topLevelId, variableNode); }; } diff --git a/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-use-state-call.ts b/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-use-state-call.ts index 926b5610a4..d1bc4f2733 100644 --- a/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-use-state-call.ts +++ b/packages/plugins/eslint-plugin-react-hooks-extra/src/utils/is-from-use-state-call.ts @@ -12,5 +12,5 @@ export function isFromUseStateCall(context: RuleContext, settings: ESLintReactSe } return parent.id.elements.findIndex((e) => e?.type === T.Identifier && e.name === topLevelId.name) === 1; }; - return isFromHookCall("useState", context, settings, predicate); + return isFromHookCall(context, "useState", settings, predicate); } diff --git a/packages/plugins/eslint-plugin-react-naming-convention/src/rules/use-state.ts b/packages/plugins/eslint-plugin-react-naming-convention/src/rules/use-state.ts index 4480a45fe1..ee2ef0a33e 100644 --- a/packages/plugins/eslint-plugin-react-naming-convention/src/rules/use-state.ts +++ b/packages/plugins/eslint-plugin-react-naming-convention/src/rules/use-state.ts @@ -63,7 +63,7 @@ export default createRule<[], MessageID>({ continue; } for (const hookCall of hookCalls) { - if (!isUseStateCall(hookCall, context) && !alias.some(isReactHookCallWithNameLoose(hookCall))) { + if (!isUseStateCall(context, hookCall) && !alias.some(isReactHookCallWithNameLoose(hookCall))) { continue; } if (hookCall.parent.type !== T.VariableDeclarator) { diff --git a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-interval.ts b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-interval.ts index 628eee0f18..4b59257dac 100644 --- a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-interval.ts +++ b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-interval.ts @@ -81,7 +81,7 @@ export default createRule<[], MessageID>({ const sEntries: TimerEntry[] = []; const cEntries: TimerEntry[] = []; function isInverseEntry(a: TimerEntry, b: TimerEntry) { - return isInstanceIdEqual(a.timerId, b.timerId, context); + return isInstanceIdEqual(context, a.timerId, b.timerId); } return { [":function"](node: AST.TSESTreeFunction) { diff --git a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-resize-observer.ts b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-resize-observer.ts index 408419fa03..dd6d31462c 100644 --- a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-resize-observer.ts +++ b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-resize-observer.ts @@ -46,7 +46,7 @@ function isNewResizeObserver(node: TSESTree.Node | _) { && node.callee.name === "ResizeObserver"; } -function isFromObserver(node: TSESTree.Expression, context: RuleContext): boolean { +function isFromObserver(context: RuleContext, node: TSESTree.Expression): boolean { switch (true) { case node.type === T.Identifier: { const initialScope = context.sourceCode.getScope(node); @@ -54,22 +54,22 @@ function isFromObserver(node: TSESTree.Expression, context: RuleContext): boolea return isNewResizeObserver(object); } case node.type === T.MemberExpression: - return isFromObserver(node.object, context); + return isFromObserver(context, node.object); default: return false; } } -function getCallKind(node: TSESTree.CallExpression, context: RuleContext): CallKind { +function getCallKind(context: RuleContext, node: TSESTree.CallExpression): CallKind { switch (true) { case node.callee.type === T.Identifier && isMatching(P.union("observe", "unobserve", "disconnect"))(node.callee.name) - && isFromObserver(node.callee, context): + && isFromObserver(context, node.callee): return node.callee.name; case node.callee.type === T.MemberExpression && node.callee.property.type === T.Identifier && isMatching(P.union("observe", "unobserve", "disconnect"))(node.callee.property.name) - && isFromObserver(node.callee, context): + && isFromObserver(context, node.callee): return node.callee.property.name; default: return "other"; @@ -133,7 +133,7 @@ export default createRule<[], MessageID>({ return; } const { object } = node.callee; - match(getCallKind(node, context)) + match(getCallKind(context, node)) .with("disconnect", () => { dEntries.push({ kind: "disconnect", @@ -202,11 +202,11 @@ export default createRule<[], MessageID>({ }, ["Program:exit"]() { for (const { id, node, phaseNode } of observers) { - if (dEntries.some((e) => isInstanceIdEqual(e.observer, id, context))) { + if (dEntries.some((e) => isInstanceIdEqual(context, e.observer, id))) { continue; } - const oentries = oEntries.filter((e) => isInstanceIdEqual(e.observer, id, context)); - const uentries = uEntries.filter((e) => isInstanceIdEqual(e.observer, id, context)); + const oentries = oEntries.filter((e) => isInstanceIdEqual(context, e.observer, id)); + const uentries = uEntries.filter((e) => isInstanceIdEqual(context, e.observer, id)); const isDynamic = (node: TSESTree.Node | _) => node?.type === T.CallExpression || AST.isConditional(node); const isPhaseNode = (node: TSESTree.Node | _) => node === phaseNode; const hasDynamicallyAdded = oentries @@ -216,7 +216,7 @@ export default createRule<[], MessageID>({ continue; } for (const oEntry of oentries) { - if (uentries.some((uEntry) => isInstanceIdEqual(uEntry.element, oEntry.element, context))) { + if (uentries.some((uEntry) => isInstanceIdEqual(context, uEntry.element, oEntry.element))) { continue; } context.report({ messageId: "expectedDisconnectOrUnobserveInCleanup", node: oEntry.node }); diff --git a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-timeout.ts b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-timeout.ts index f2b9943fb8..5bca5a232f 100644 --- a/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-timeout.ts +++ b/packages/plugins/eslint-plugin-react-web-api/src/rules/no-leaked-timeout.ts @@ -80,7 +80,7 @@ export default createRule<[], MessageID>({ const sEntries: TimerEntry[] = []; const rEntries: TimerEntry[] = []; function isInverseEntry(a: TimerEntry, b: TimerEntry) { - return isInstanceIdEqual(a.timerId, b.timerId, context); + return isInstanceIdEqual(context, a.timerId, b.timerId); } return { [":function"](node: AST.TSESTreeFunction) { diff --git a/packages/plugins/eslint-plugin-react-web-api/src/utils/is-instance-id-equal.ts b/packages/plugins/eslint-plugin-react-web-api/src/utils/is-instance-id-equal.ts index e981ab12a0..3689c3d101 100644 --- a/packages/plugins/eslint-plugin-react-web-api/src/utils/is-instance-id-equal.ts +++ b/packages/plugins/eslint-plugin-react-web-api/src/utils/is-instance-id-equal.ts @@ -2,7 +2,7 @@ import type { RuleContext } from "@eslint-react/shared"; import * as VAR from "@eslint-react/var"; import type { TSESTree } from "@typescript-eslint/types"; -export function isInstanceIdEqual(a: TSESTree.Node, b: TSESTree.Node, context: RuleContext) { +export function isInstanceIdEqual(context: RuleContext, a: TSESTree.Node, b: TSESTree.Node) { return VAR.isVariableIdEqual(a, b, [ context.sourceCode.getScope(a), context.sourceCode.getScope(b), diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/ensure-forward-ref-using-ref.ts b/packages/plugins/eslint-plugin-react-x/src/rules/ensure-forward-ref-using-ref.ts index 8fe29899f5..beceeb7a77 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/ensure-forward-ref-using-ref.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/ensure-forward-ref-using-ref.ts @@ -30,7 +30,7 @@ export default createRule<[], MessageID>({ create(context) { return { CallExpression(node) { - if (!isForwardRefCall(node, context)) { + if (!isForwardRefCall(context, node)) { return; } const [component] = node.arguments; diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-array-index-key.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-array-index-key.ts index fdeab3d377..f793acfa67 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-array-index-key.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-array-index-key.ts @@ -50,7 +50,7 @@ function isReactChildrenMethod(name: string): name is typeof reactChildrenMethod return reactChildrenMethod.some((method) => method === name); } -function isUsingReactChildren(node: TSESTree.CallExpression, context: RuleContext) { +function isUsingReactChildren(context: RuleContext, node: TSESTree.CallExpression) { const { importSource = "react" } = unsafeDecodeSettings(context.settings); const { callee } = node; if (!("property" in callee) || !("object" in callee) || !("name" in callee.property)) { @@ -69,7 +69,7 @@ function isUsingReactChildren(node: TSESTree.CallExpression, context: RuleContex return false; } -function getMapIndexParamName(node: TSESTree.CallExpression, context: RuleContext): string | _ { +function getMapIndexParamName(context: RuleContext, node: TSESTree.CallExpression): string | _ { const { callee } = node; if (callee.type !== T.MemberExpression) { return _; @@ -81,7 +81,7 @@ function getMapIndexParamName(node: TSESTree.CallExpression, context: RuleContex if (!iteratorFunctionIndexParamPosition.has(name)) { return _; } - const callbackArg = node.arguments[isUsingReactChildren(node, context) ? 1 : 0]; + const callbackArg = node.arguments[isUsingReactChildren(context, node) ? 1 : 0]; if (callbackArg == null) { return _; } @@ -129,7 +129,7 @@ export default createRule<[], MessageID>({ } function isCreateOrCloneElementCall(node: TSESTree.Node): node is TSESTree.CallExpression { - return isCreateElementCall(node, context) || isCloneElementCall(node, context); + return isCreateElementCall(context, node) || isCloneElementCall(context, node); } function getReportDescriptors(node: TSESTree.Node): ReportDescriptor[] { @@ -192,7 +192,7 @@ export default createRule<[], MessageID>({ return { CallExpression(node) { - indexParamNames.push(getMapIndexParamName(node, context)); + indexParamNames.push(getMapIndexParamName(context, node)); if (node.arguments.length === 0) { return; } diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-count.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-count.ts index 09eefa9eda..4f167aca57 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-count.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-count.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { MemberExpression(node) { - if (isChildrenCount(node, context)) { + if (isChildrenCount(context, node)) { context.report({ messageId: "noChildrenCount", node: node.property, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-for-each.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-for-each.ts index f6aa915245..4d89b021f3 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-for-each.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-for-each.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { MemberExpression(node) { - if (isChildrenForEach(node, context)) { + if (isChildrenForEach(context, node)) { context.report({ messageId: "noChildrenForEach", node: node.property, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-map.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-map.ts index 2914b60827..7f79bd10de 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-map.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-map.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { MemberExpression(node) { - if (isChildrenMap(node, context)) { + if (isChildrenMap(context, node)) { context.report({ messageId: "noChildrenMap", node: node.property, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-only.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-only.ts index 47ba01bf40..069a6a0015 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-only.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-only.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { MemberExpression(node) { - if (isChildrenOnly(node, context)) { + if (isChildrenOnly(context, node)) { context.report({ messageId: "noChildrenOnly", node: node.property, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-to-array.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-to-array.ts index ed95e8360f..0e68a27ffe 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-children-to-array.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-children-to-array.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { MemberExpression(node) { - if (isChildrenToArray(node, context)) { + if (isChildrenToArray(context, node)) { context.report({ messageId: "noChildrenToArray", node: node.property, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-clone-element.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-clone-element.ts index 1edd0af0f5..cecebbf99f 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-clone-element.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-clone-element.ts @@ -28,7 +28,7 @@ export default createRule<[], MessageID>({ create(context) { return { CallExpression(node) { - if (!isCloneElementCall(node, context)) { + if (!isCloneElementCall(context, node)) { return; } context.report({ diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-create-ref.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-create-ref.ts index 4f0f5aaabd..fe41db6d5e 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-create-ref.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-create-ref.ts @@ -29,7 +29,7 @@ export default createRule<[], MessageID>({ create(context) { return { CallExpression(node) { - if (isCreateRefCall(node, context) && AST.findParentNode(node, isClassComponent) == null) { + if (isCreateRefCall(context, node) && AST.findParentNode(node, isClassComponent) == null) { context.report({ messageId: "noCreateRef", node }); } }, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-forward-ref.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-forward-ref.ts index 051a771216..9157118707 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-forward-ref.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-forward-ref.ts @@ -45,13 +45,13 @@ export default createRule<[], MessageID>({ } return { CallExpression(node) { - if (!isForwardRefCall(node, context)) { + if (!isForwardRefCall(context, node)) { return; } context.report({ messageId: "noForwardRef", node, - fix: getFix(node, context), + fix: getFix(context, node), }); }, }; @@ -59,7 +59,7 @@ export default createRule<[], MessageID>({ defaultOptions: [], }); -function getFix(node: TSESTree.CallExpression, context: RuleContext): (fixer: RuleFixer) => RuleFix[] { +function getFix(context: RuleContext, node: TSESTree.CallExpression): (fixer: RuleFixer) => RuleFix[] { return (fixer) => { const [componentNode] = node.arguments; if (componentNode == null || !AST.isFunction(componentNode)) { diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-context-display-name.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-context-display-name.ts index 021e6cdac7..66478d8a37 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-context-display-name.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-context-display-name.ts @@ -36,7 +36,7 @@ export default createRule<[], MessageID>({ const displayNameAssignments: TSESTree.AssignmentExpression[] = []; return { CallExpression(node) { - if (!isCreateContextCall(node, context)) return; + if (!isCreateContextCall(context, node)) return; createCalls.push(node); }, [DISPLAY_NAME_ASSIGNMENT_SELECTOR](node) { diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-key.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-key.ts index e3d0e442d4..d44e8c3ebe 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-key.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-missing-key.ts @@ -111,7 +111,7 @@ export default createRule<[], MessageID>({ } }, CallExpression(node) { - state.isWithinChildrenToArray ||= isChildrenToArrayCall(node, context); + state.isWithinChildrenToArray ||= isChildrenToArrayCall(context, node); if (state.isWithinChildrenToArray) { return; } @@ -135,7 +135,7 @@ export default createRule<[], MessageID>({ } }, "CallExpression:exit"(node) { - if (!isChildrenToArrayCall(node, context)) { + if (!isChildrenToArrayCall(context, node)) { return; } state.isWithinChildrenToArray = false; diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-nested-components.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-nested-components.ts index 597c3b7e40..89c228a018 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-nested-components.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-nested-components.ts @@ -102,7 +102,7 @@ export default createRule<[], MessageID>({ continue; } - if (isInsideCreateElementProps(component, context)) { + if (isInsideCreateElementProps(context, component)) { context.report({ messageId: "nestedComponentInProps", node: component, diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-use-context.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-use-context.ts index 3b977baa50..018e7ced3c 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-use-context.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-use-context.ts @@ -43,7 +43,7 @@ export default createRule<[], MessageID>({ if (!isReactHookCall(node)) { return; } - if (!isReactHookCallWithNameAlias("useContext", context, [...useContextAlias])(node)) { + if (!isReactHookCallWithNameAlias(context, "useContext", [...useContextAlias])(node)) { return; } context.report({ diff --git a/packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.ts b/packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.ts index 2c2555bdc0..1907ec3f25 100644 --- a/packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.ts +++ b/packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.ts @@ -38,8 +38,8 @@ function trimLikeReact(text: string) { } function checkAndReport( - node: TSESTree.JSXElement | TSESTree.JSXFragment, context: RuleContext, + node: TSESTree.JSXElement | TSESTree.JSXFragment, allowExpressions: boolean, ) { function fix(fixer: RuleFixer) { @@ -191,10 +191,10 @@ export default createRule({ return { JSXElement(node) { if (!JSX.isFragmentElement(node)) return; - checkAndReport(node, context, allowExpressions); + checkAndReport(context, node, allowExpressions); }, JSXFragment(node) { - checkAndReport(node, context, allowExpressions); + checkAndReport(context, node, allowExpressions); }, }; }, diff --git a/packages/shared/docs/-internal-/interfaces/ReadonlyAction.md b/packages/shared/docs/-internal-/interfaces/ReadonlyAction.md index 01bc559128..43f9a3a0dc 100644 --- a/packages/shared/docs/-internal-/interfaces/ReadonlyAction.md +++ b/packages/shared/docs/-internal-/interfaces/ReadonlyAction.md @@ -10,7 +10,7 @@ Readonly action interface. ## Extends -- [`BaseTransformation`](BaseTransformation.md)\<`TInput`, [`Readonly`](../type-aliases/Readonly.md)\<`TInput`\>, `never`\> +- [`BaseTransformation`](BaseTransformation.md)\<`TInput`, [`ReadonlyOutput`](../type-aliases/ReadonlyOutput.md)\<`TInput`\>, `never`\> ## Type Parameters diff --git a/packages/shared/docs/-internal-/type-aliases/ReadonlyOutput.md b/packages/shared/docs/-internal-/type-aliases/ReadonlyOutput.md new file mode 100644 index 0000000000..1e8baff677 --- /dev/null +++ b/packages/shared/docs/-internal-/type-aliases/ReadonlyOutput.md @@ -0,0 +1,15 @@ +[**@eslint-react/shared**](../../README.md) + +*** + +[@eslint-react/shared](../../README.md) / [\](../README.md) / ReadonlyOutput + +# Type Alias: ReadonlyOutput\ + +> **ReadonlyOutput**\<`TInput`\>: `TInput` *extends* [`Map`](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Map)\ ? `ReadonlyMap`\<`TKey`, `TValue`\> : `TInput` *extends* [`Set`](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Set)\ ? `ReadonlySet`\<`TValue`\> : [`Readonly`](Readonly.md)\<`TInput`\> + +Readonly output type. + +## Type Parameters + +• **TInput**