eslint-plugin-react-debug 3.0.0-rc.5 → 4.0.0-beta.0

package/README.md

@@ -15,7 +15,7 @@ npm install --save-dev eslint-plugin-react-debug
 
 ```ts
 import js from "@eslint/js";
-import debug from "eslint-plugin-react-debug";
+import reactDebug from "eslint-plugin-react-debug";
 import { defineConfig } from "eslint/config";
 import tseslint from "typescript-eslint";
 
@@ -26,7 +26,7 @@ export default defineConfig(
       js.configs.recommended,
       tseslint.configs.recommended,
       // Add configs from eslint-plugin-react-debug
-      debug.configs.all,
+      reactDebug.configs.all,
     ],
     rules: {
       // Put rules you want to override here

package/dist/index.js

@@ -1,7 +1,10 @@
 import { DEFAULT_ESLINT_REACT_SETTINGS, WEBSITE_URL, defineRuleListener, getSettingsFromContext, report } from "@eslint-react/shared";
 import * as core from "@eslint-react/core";
-import { JsxEmit, JsxInspector } from "@eslint-react/core";
+import { isUseRefCall } from "@eslint-react/core";
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
+import { findVariable } from "@typescript-eslint/utils/ast-utils";
+import "@eslint-react/ast";
+import { JsxEmit, getElementFullType, getJsxConfig, isFragmentElement } from "@eslint-react/jsx";
 import { AST_NODE_TYPES as AST_NODE_TYPES$1 } from "@typescript-eslint/types";
 import { P, match } from "ts-pattern";
 
@@ -24,7 +27,7 @@ var __exportAll = (all, no_symbols) => {
 //#endregion
 //#region package.json
 var name$1 = "eslint-plugin-react-debug";
-var version = "3.0.0-rc.5";
+var version = "4.0.0-beta.0";
 
 //#endregion
 //#region src/utils/create-rule.ts
@@ -54,9 +57,9 @@ var class_component_default = createRule({
 	defaultOptions: []
 });
 function create$5(context) {
-	const { ctx, visitor } = core.useComponentCollectorLegacy(context);
+	const { api, visitor } = core.getComponentCollectorLegacy(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { name = "anonymous", node: component } of ctx.getAllComponents(program)) context.report({
+		for (const { name = "anonymous", node: component } of api.getAllComponents(program)) context.report({
 			data: { json: stringify({ name }) },
 			messageId: "default",
 			node: component
@@ -79,12 +82,12 @@ var function_component_default = createRule({
 	defaultOptions: []
 });
 function create$4(context) {
-	const { ctx, visitor } = core.useComponentCollector(context, {
+	const { api, visitor } = core.getComponentCollector(context, {
 		collectDisplayName: true,
 		hint: core.DEFAULT_COMPONENT_DETECTION_HINT
 	});
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { name, displayName, flag, hookCalls, node } of ctx.getAllComponents(program)) context.report({
+		for (const { name, displayName, flag, hookCalls, node } of api.getAllComponents(program)) context.report({
 			data: { json: stringify({
 				name: name ?? "anonymous",
 				displayName: displayName == null ? "none" : context.sourceCode.getText(displayName),
@@ -113,9 +116,9 @@ var hook_default = createRule({
 	defaultOptions: []
 });
 function create$3(context) {
-	const { ctx, visitor } = core.useHookCollector(context);
+	const { api, visitor } = core.getHookCollector(context);
 	return defineRuleListener(visitor, { "Program:exit"(program) {
-		for (const { name, hookCalls, node } of ctx.getAllHooks(program)) context.report({
+		for (const { name, hookCalls, node } of api.getAllHooks(program)) context.report({
 			data: { json: stringify({
 				name,
 				hookCalls: hookCalls.length
@@ -213,11 +216,29 @@ function create$1(context) {
 function getRefInitNode(node, initialScope) {
 	const name = node.name;
 	switch (true) {
-		case node.parent.type === AST_NODE_TYPES.MemberExpression && node.parent.property === node && node.parent.object.type === AST_NODE_TYPES.Identifier: return core.getRefInit(node.parent.object.name, initialScope);
-		case node.parent.type === AST_NODE_TYPES.JSXMemberExpression && node.parent.property === node && node.parent.object.type === AST_NODE_TYPES.JSXIdentifier: return core.getRefInit(node.parent.object.name, initialScope);
-		default: return core.getRefInit(name, initialScope);
+		case node.parent.type === AST_NODE_TYPES.MemberExpression && node.parent.property === node && node.parent.object.type === AST_NODE_TYPES.Identifier: return getRefInit(node.parent.object.name, initialScope);
+		case node.parent.type === AST_NODE_TYPES.JSXMemberExpression && node.parent.property === node && node.parent.object.type === AST_NODE_TYPES.JSXIdentifier: return getRefInit(node.parent.object.name, initialScope);
+		default: return getRefInit(name, initialScope);
 	}
 }
+/**
+* Get the init expression of a ref variable
+* @param name The variable name
+* @param initialScope The initial scope
+* @returns The init expression node if the variable is derived from a ref, or null otherwise
+*/
+function getRefInit(name, initialScope) {
+	for (const { node } of findVariable(initialScope, name)?.defs ?? []) {
+		if (node.type !== AST_NODE_TYPES.VariableDeclarator) continue;
+		const init = node.init;
+		if (init == null) continue;
+		switch (true) {
+			case init.type === AST_NODE_TYPES.MemberExpression && init.object.type === AST_NODE_TYPES.Identifier && (init.object.name === "ref" || init.object.name.endsWith("Ref")): return init;
+			case init.type === AST_NODE_TYPES.CallExpression && isUseRefCall(init): return init;
+		}
+	}
+	return null;
+}
 
 //#endregion
 //#region ../../../.pkgs/eff/dist/index.js
@@ -381,13 +402,12 @@ var jsx_default = createRule({
 	defaultOptions: []
 });
 function create(context) {
-	const jsx = JsxInspector.from(context);
-	const jsxConfig = jsx.jsxConfig;
+	const jsxConfig = getJsxConfig(context);
 	function getReportDescriptor(context) {
 		return (node) => ({
 			data: { json: stringify({
-				kind: match(node).with({ type: AST_NODE_TYPES$1.JSXElement }, (n) => jsx.isFragmentElement(n) ? "fragment" : "element").with({ type: AST_NODE_TYPES$1.JSXFragment }, () => "fragment").exhaustive(),
-				type: jsx.getElementType(node),
+				kind: match(node).with({ type: AST_NODE_TYPES$1.JSXElement }, (n) => isFragmentElement(n, jsxConfig.jsxFragmentFactory) ? "fragment" : "element").with({ type: AST_NODE_TYPES$1.JSXFragment }, () => "fragment").exhaustive(),
+				type: getElementFullType(node),
 				jsx: match(jsxConfig.jsx).with(JsxEmit.None, () => "none").with(JsxEmit.ReactJSX, () => "react-jsx").with(JsxEmit.ReactJSXDev, () => "react-jsx-dev").with(JsxEmit.React, () => "react").with(JsxEmit.ReactNative, () => "react-native").with(JsxEmit.Preserve, () => "preserve").otherwise(() => "unknown"),
 				jsxFactory: jsxConfig.jsxFactory,
 				jsxFragmentFactory: jsxConfig.jsxFragmentFactory,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-debug",
-  "version": "3.0.0-rc.5",
+  "version": "4.0.0-beta.0",
   "description": "ESLint React's ESLint plugin for debugging related rules.",
   "keywords": [
     "react",
@@ -38,21 +38,22 @@
     "./package.json"
   ],
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.57.0",
-    "@typescript-eslint/type-utils": "^8.57.0",
-    "@typescript-eslint/types": "^8.57.0",
-    "@typescript-eslint/utils": "^8.57.0",
+    "@typescript-eslint/scope-manager": "^8.57.2",
+    "@typescript-eslint/type-utils": "^8.57.2",
+    "@typescript-eslint/types": "^8.57.2",
+    "@typescript-eslint/utils": "^8.57.2",
     "ts-pattern": "^5.9.0",
-    "@eslint-react/core": "3.0.0-rc.5",
-    "@eslint-react/ast": "3.0.0-rc.5",
-    "@eslint-react/var": "3.0.0-rc.5",
-    "@eslint-react/shared": "3.0.0-rc.5"
+    "@eslint-react/ast": "4.0.0-beta.0",
+    "@eslint-react/core": "4.0.0-beta.0",
+    "@eslint-react/jsx": "4.0.0-beta.0",
+    "@eslint-react/shared": "4.0.0-beta.0",
+    "@eslint-react/var": "4.0.0-beta.0"
   },
   "devDependencies": {
     "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
-    "eslint": "^10.0.3",
-    "tsdown": "^0.21.2",
+    "eslint": "^10.1.0",
+    "tsdown": "^0.21.4",
     "@local/configs": "0.0.0",
     "@local/eff": "3.0.0-beta.72"
   },