oxlint-plugin-react-doctor 0.5.0 → 0.5.1-dev.b819e5f

package/dist/index.js

@@ -432,6 +432,7 @@ const SETTER_PATTERN = /^set[A-Z]/;
 const RENDER_FUNCTION_PATTERN = /^render[A-Z]/;
 const UPPERCASE_PATTERN = /^[A-Z]/;
 const REACT_HANDLER_PROP_PATTERN = /^on[A-Z]/;
+const HANDLER_FUNCTION_NAME_PATTERN = /^(?:on|handle)[A-Z]/;
 const EFFECT_HOOK_NAMES$1 = new Set(["useEffect", "useLayoutEffect"]);
 const HOOKS_WITH_DEPS = new Set([
 	"useEffect",
@@ -24499,6 +24500,54 @@ const UTILITY_FILE_BASENAMES = new Set([
 	"defaults.tsx",
 	"defaults.jsx"
 ]);
+const ROUTE_FACTORY_CALLEE_NAMES = new Set([
+	...TANSTACK_ROUTE_CREATION_FUNCTIONS,
+	"createLazyFileRoute",
+	"createLazyRoute",
+	"createAPIFileRoute",
+	"createServerFileRoute",
+	"createServerRootRoute",
+	"createServerRoute",
+	"createBrowserRouter",
+	"createHashRouter",
+	"createMemoryRouter",
+	"createStaticRouter",
+	"createRouter"
+]);
+const ROUTE_MODULE_ALLOWED_EXPORT_NAMES = new Set([
+	"loader",
+	"clientLoader",
+	"action",
+	"clientAction",
+	"headers",
+	"meta",
+	"links",
+	"handle",
+	"shouldRevalidate",
+	"middleware",
+	"unstable_middleware",
+	"getServerSideProps",
+	"getStaticProps",
+	"getStaticPaths",
+	"getInitialProps",
+	"reportWebVitals",
+	"metadata",
+	"generateMetadata",
+	"generateStaticParams",
+	"generateImageMetadata",
+	"generateSitemaps",
+	"viewport",
+	"generateViewport",
+	"revalidate",
+	"dynamic",
+	"dynamicParams",
+	"fetchCache",
+	"runtime",
+	"preferredRegion",
+	"maxDuration",
+	"experimental_ppr",
+	"unstable_settings"
+]);
 //#endregion
 //#region src/plugin/rules/react-builtins/only-export-components.ts
 const NAMED_EXPORT_MESSAGE = "This file exports non-components, so Fast Refresh can't safely preserve component state.";
@@ -24542,6 +24591,18 @@ const isReactCreateContext = (initializer) => {
 	if (isNodeOfType(callee, "MemberExpression") && isNodeOfType(callee.property, "Identifier") && callee.property.name === "createContext") return true;
 	return false;
 };
+const isRouteFactoryName = (name) => ROUTE_FACTORY_CALLEE_NAMES.has(name);
+const isRouteFactoryCall = (expression) => {
+	let currentCall = expression;
+	while (isNodeOfType(currentCall, "CallExpression")) {
+		const callee = currentCall.callee;
+		if (isNodeOfType(callee, "Identifier") && isRouteFactoryName(callee.name)) return true;
+		if (isNodeOfType(callee, "MemberExpression") && isNodeOfType(callee.property, "Identifier") && isRouteFactoryName(callee.property.name)) return true;
+		if (!isNodeOfType(callee, "CallExpression")) return false;
+		currentCall = callee;
+	}
+	return false;
+};
 const isReactHocName = (name, state) => state.customHocs.has(name);
 const isHocCallee = (callee, state) => {
 	if (isNodeOfType(callee, "Identifier")) return isReactHocName(callee.name, state);
@@ -24575,10 +24636,12 @@ const isReactComponentInitializer = (expression, state) => {
 const classifyExport = (name, reportNode, isFunction, initializer, state) => {
 	if (initializer) {
 		const expression = skipTsExpression(initializer);
+		if (isRouteFactoryCall(expression)) return { kind: "react-component" };
 		if (isNodeOfType(expression, "CallExpression") && isHocCallee(expression.callee, state) && expression.arguments.length > 0 && isReactComponentName(name)) return { kind: "react-component" };
 		if (isNodeOfType(expression, "ConditionalExpression") && isReactComponentName(name) && isReactComponentInitializer(expression.consequent, state) && isReactComponentInitializer(expression.alternate, state)) return { kind: "react-component" };
 	}
 	if (state.allowExportNames.has(name)) return { kind: "allowed" };
+	if (ROUTE_MODULE_ALLOWED_EXPORT_NAMES.has(name)) return { kind: "allowed" };
 	if (/^use[A-Z]/.test(name)) return { kind: "allowed" };
 	if (state.allowConstantExport && initializer) {
 		const expression = skipTsExpression(initializer);
@@ -24724,6 +24787,10 @@ const onlyExportComponents = defineRule({
 						continue;
 					}
 					if (isNodeOfType(stripped, "CallExpression")) {
+						if (isRouteFactoryCall(stripped)) {
+							hasReactExport = true;
+							continue;
+						}
 						const isHoc = isHocCallee(stripped.callee, state);
 						const firstArg = stripped.arguments[0];
 						const firstArgIsValid = Boolean(firstArg) && (() => {
@@ -34290,6 +34357,9 @@ const tanstackStartNoNavigateInRender = defineRule({
 		let eventHandlerDepth = 0;
 		const isDeferredHookCall = (node) => isHookCall$1(node, EFFECT_HOOK_NAMES$1) || isHookCall$1(node, "useCallback") || isHookCall$1(node, "useMemo");
 		const isEventHandlerAttribute = (node) => isNodeOfType(node, "JSXAttribute") && isNodeOfType(node.name, "JSXIdentifier") && typeof node.name.name === "string" && node.name.name.startsWith("on") && UPPERCASE_PATTERN.test(node.name.name.charAt(2));
+		const isEventHandlerProperty = (node) => isNodeOfType(node, "Property") && isFunctionLike$2(node.value) && (isNodeOfType(node.key, "Identifier") && typeof node.key.name === "string" && REACT_HANDLER_PROP_PATTERN.test(node.key.name) || isNodeOfType(node.key, "Literal") && typeof node.key.value === "string" && REACT_HANDLER_PROP_PATTERN.test(node.key.value));
+		const isHandlerNamedVariableDeclarator = (node) => isNodeOfType(node, "VariableDeclarator") && isNodeOfType(node.id, "Identifier") && typeof node.id.name === "string" && HANDLER_FUNCTION_NAME_PATTERN.test(node.id.name) && isFunctionLike$2(node.init);
+		const isHandlerNamedFunctionDeclaration = (node) => isNodeOfType(node, "FunctionDeclaration") && isNodeOfType(node.id, "Identifier") && typeof node.id.name === "string" && HANDLER_FUNCTION_NAME_PATTERN.test(node.id.name);
 		return {
 			CallExpression(node) {
 				const filename = normalizeFilename$1(context.filename ?? "");
@@ -34315,6 +34385,36 @@ const tanstackStartNoNavigateInRender = defineRule({
 				const filename = normalizeFilename$1(context.filename ?? "");
 				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
 				if (isEventHandlerAttribute(node)) eventHandlerDepth = Math.max(0, eventHandlerDepth - 1);
+			},
+			Property(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isEventHandlerProperty(node)) eventHandlerDepth++;
+			},
+			"Property:exit"(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isEventHandlerProperty(node)) eventHandlerDepth = Math.max(0, eventHandlerDepth - 1);
+			},
+			VariableDeclarator(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isHandlerNamedVariableDeclarator(node)) eventHandlerDepth++;
+			},
+			"VariableDeclarator:exit"(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isHandlerNamedVariableDeclarator(node)) eventHandlerDepth = Math.max(0, eventHandlerDepth - 1);
+			},
+			FunctionDeclaration(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isHandlerNamedFunctionDeclaration(node)) eventHandlerDepth++;
+			},
+			"FunctionDeclaration:exit"(node) {
+				const filename = normalizeFilename$1(context.filename ?? "");
+				if (!TANSTACK_ROUTE_FILE_PATTERN.test(filename)) return;
+				if (isHandlerNamedFunctionDeclaration(node)) eventHandlerDepth = Math.max(0, eventHandlerDepth - 1);
 			}
 		};
 	}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-react-doctor",
-  "version": "0.5.0",
+  "version": "0.5.1-dev.b819e5f",
   "description": "oxlint plugin for React Doctor: diagnose React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",