oxlint-plugin-react-doctor 0.6.2-dev.d8628d7 → 0.6.2-dev.da3b19c

package/dist/index.js

@@ -4843,7 +4843,8 @@ const FORM_CONTROL_TAGS = new Set([
 ]);
 const resolveSettings$48 = (settings) => {
 	const reactDoctor = settings?.["react-doctor"];
-	return { inputComponents: (typeof reactDoctor === "object" && reactDoctor !== null ? reactDoctor.autocompleteValid ?? {} : {}).inputComponents ?? [] };
+	const ruleSettings = typeof reactDoctor === "object" && reactDoctor !== null ? reactDoctor.autocompleteValid ?? {} : {};
+	return { inputComponents: new Set(ruleSettings.inputComponents ?? []) };
 };
 const autocompleteValid = defineRule({
 	id: "autocomplete-valid",
@@ -4856,7 +4857,7 @@ const autocompleteValid = defineRule({
 		const settings = resolveSettings$48(context.settings);
 		return { JSXOpeningElement: (node) => {
 			const tag = getElementType(node, context.settings);
-			if (!FORM_CONTROL_TAGS.has(tag) && !settings.inputComponents.includes(tag)) return;
+			if (!FORM_CONTROL_TAGS.has(tag) && !settings.inputComponents.has(tag)) return;
 			const attribute = hasJsxPropIgnoreCase(node.attributes, "autoComplete");
 			if (!attribute) return;
 			const value = getJsxPropStringValue(attribute);
@@ -8636,7 +8637,7 @@ const DEFAULT_HEADING_TAGS = [
 const resolveSettings$40 = (settings) => {
 	const reactDoctor = settings?.["react-doctor"];
 	const ruleSettings = typeof reactDoctor === "object" && reactDoctor !== null ? reactDoctor.headingHasContent ?? {} : {};
-	return { headingTags: [...DEFAULT_HEADING_TAGS, ...ruleSettings.components ?? []] };
+	return { headingTags: new Set([...DEFAULT_HEADING_TAGS, ...ruleSettings.components ?? []]) };
 };
 const headingHasContent = defineRule({
 	id: "heading-has-content",
@@ -8649,7 +8650,7 @@ const headingHasContent = defineRule({
 		const settings = resolveSettings$40(context.settings);
 		return { JSXOpeningElement(node) {
 			const elementType = getElementType(node, context.settings);
-			if (!settings.headingTags.includes(elementType)) return;
+			if (!settings.headingTags.has(elementType)) return;
 			const parent = node.parent;
 			if (parent && isNodeOfType(parent, "JSXElement")) {
 				if (objectHasAccessibleChild(parent, context.settings)) return;
@@ -9555,6 +9556,7 @@ const KEYBOARD_EVENT_HANDLERS = [
 	"onKeyUp"
 ];
 const ALL_EVENT_HANDLERS = [...MOUSE_EVENT_HANDLERS, ...KEYBOARD_EVENT_HANDLERS];
+const ALL_EVENT_HANDLERS_LOWER = new Set(ALL_EVENT_HANDLERS.map((handlerName) => handlerName.toLowerCase()));
 //#endregion
 //#region src/plugin/utils/is-disabled-element.ts
 const isDisabledElement = (openingElement) => {
@@ -9614,7 +9616,16 @@ const interactiveSupportsFocus = defineRule({
 			const roleAttribute = hasJsxPropIgnoreCase(node.attributes, "role");
 			const role = roleAttribute ? getJsxPropStringValue(roleAttribute) : null;
 			if (!role) return;
-			if (!ALL_EVENT_HANDLERS.some((handler) => Boolean(hasJsxPropIgnoreCase(node.attributes, handler)))) return;
+			let hasInteractiveHandler = false;
+			for (const attribute of node.attributes) {
+				if (!isNodeOfType(attribute, "JSXAttribute")) continue;
+				const attributeName = getJsxAttributeName(attribute.name);
+				if (attributeName && ALL_EVENT_HANDLERS_LOWER.has(attributeName.toLowerCase())) {
+					hasInteractiveHandler = true;
+					break;
+				}
+			}
+			if (!hasInteractiveHandler) return;
 			const elementType = getElementType(node, context.settings);
 			if (!HTML_TAGS.has(elementType)) return;
 			if (isDisabledElement(node) || isHiddenFromScreenReader(node, context.settings) || isPresentationRole(node)) return;
@@ -14463,14 +14474,14 @@ const keyLifecycleRisk = defineRule({
 //#region src/plugin/rules/a11y/label-has-associated-control.ts
 const MESSAGE_NO_LABEL = "Blind users can't identify this field because screen readers find no label text, so add visible text, `aria-label`, or `aria-labelledby`.";
 const MESSAGE_NO_CONTROL = "Screen reader users can't tell which input this label names because it's tied to none, so add `htmlFor` or wrap the input inside it.";
-const DEFAULT_CONTROL_COMPONENTS = [
+const DEFAULT_CONTROL_COMPONENTS = new Set([
 	"input",
 	"meter",
 	"output",
 	"progress",
 	"select",
 	"textarea"
-];
+]);
 const DEFAULT_LABEL_ATTRIBUTES = [
 	"alt",
 	"aria-label",
@@ -14482,8 +14493,8 @@ const resolveSettings$24 = (settings) => {
 	const jsxA11y = settings?.["jsx-a11y"];
 	const forAttributes = (typeof jsxA11y === "object" && jsxA11y !== null ? jsxA11y : {}).attributes?.for ?? ["htmlFor"];
 	return {
-		labelComponents: ["label", ...ruleSettings.labelComponents ?? []].sort(),
-		labelAttributes: [...new Set([...DEFAULT_LABEL_ATTRIBUTES, ...ruleSettings.labelAttributes ?? []])].sort(),
+		labelComponents: new Set(["label", ...ruleSettings.labelComponents ?? []]),
+		labelAttributes: new Set([...DEFAULT_LABEL_ATTRIBUTES, ...ruleSettings.labelAttributes ?? []]),
 		controlComponents: ruleSettings.controlComponents ?? [],
 		assert: ruleSettings.assert ?? "either",
 		depth: Math.min(ruleSettings.depth ?? 5, 25),
@@ -14491,7 +14502,7 @@ const resolveSettings$24 = (settings) => {
 	};
 };
 const isControlComponent = (tagName, controlComponents) => {
-	if (DEFAULT_CONTROL_COMPONENTS.includes(tagName)) return true;
+	if (DEFAULT_CONTROL_COMPONENTS.has(tagName)) return true;
 	return controlComponents.some((pattern) => compileGlob(pattern).test(tagName));
 };
 const searchForNestedControl = (child, currentDepth, searchContext) => {
@@ -14517,7 +14528,7 @@ const searchForAccessibleLabel = (child, currentDepth, searchContext) => {
 			const attributeName = attribute.name;
 			if (!isNodeOfType(attributeName, "JSXIdentifier")) continue;
 			const propName = getJsxAttributeName(attributeName);
-			if (!propName || !searchContext.labelAttributes.includes(propName)) continue;
+			if (!propName || !searchContext.labelAttributes.has(propName)) continue;
 			const attributeValue = attribute.value;
 			if (!attributeValue) continue;
 			if (isNodeOfType(attributeValue, "Literal") && typeof attributeValue.value === "string") {
@@ -14539,7 +14550,7 @@ const hasAccessibleLabel = (element, searchContext) => {
 		const attributeName = attribute.name;
 		if (!isNodeOfType(attributeName, "JSXIdentifier")) continue;
 		const propName = getJsxAttributeName(attributeName);
-		if (propName && searchContext.labelAttributes.includes(propName)) return true;
+		if (propName && searchContext.labelAttributes.has(propName)) return true;
 	}
 	for (const child of element.children) if (searchForAccessibleLabel(child, 1, searchContext)) return true;
 	return false;
@@ -14562,7 +14573,7 @@ const labelHasAssociatedControl = defineRule({
 			if (isTestlikeFile) return;
 			const opening = node.openingElement;
 			const tagName = getElementType(opening, context.settings);
-			if (!settings.labelComponents.includes(tagName)) return;
+			if (!settings.labelComponents.has(tagName)) return;
 			const hasHtmlFor = settings.forAttributes.some((attributeName) => Boolean(hasJsxPropIgnoreCase(opening.attributes, attributeName)));
 			const searchContext = {
 				depth: settings.depth,
@@ -15151,9 +15162,9 @@ const resolveSettings$23 = (settings) => {
 	const reactDoctor = settings?.["react-doctor"];
 	const ruleSettings = typeof reactDoctor === "object" && reactDoctor !== null ? reactDoctor.mediaHasCaption ?? {} : {};
 	return {
-		audio: [...DEFAULT_AUDIO, ...ruleSettings.audio ?? []],
-		video: [...DEFAULT_VIDEO, ...ruleSettings.video ?? []],
-		track: [...DEFAULT_TRACK, ...ruleSettings.track ?? []]
+		audio: new Set([...DEFAULT_AUDIO, ...ruleSettings.audio ?? []]),
+		video: new Set([...DEFAULT_VIDEO, ...ruleSettings.video ?? []]),
+		track: new Set([...DEFAULT_TRACK, ...ruleSettings.track ?? []])
 	};
 };
 const evaluateMuted = (attribute) => {
@@ -15182,7 +15193,7 @@ const childMayRenderTrack = (child, trackTags, settings) => {
 	let rendersCaptionTrack = false;
 	walkAst(expression, (inner) => {
 		if (rendersCaptionTrack) return false;
-		if (isNodeOfType(inner, "JSXElement") && trackTags.includes(getElementType(inner.openingElement, settings)) && trackKindMightBeCaptions(inner.openingElement)) {
+		if (isNodeOfType(inner, "JSXElement") && trackTags.has(getElementType(inner.openingElement, settings)) && trackKindMightBeCaptions(inner.openingElement)) {
 			rendersCaptionTrack = true;
 			return false;
 		}
@@ -15200,7 +15211,7 @@ const mediaHasCaption = defineRule({
 		const settings = resolveSettings$23(context.settings);
 		return { JSXOpeningElement(node) {
 			const tag = getElementType(node, context.settings);
-			if (!(settings.audio.includes(tag) || settings.video.includes(tag))) return;
+			if (!(settings.audio.has(tag) || settings.video.has(tag))) return;
 			if (evaluateMuted(hasJsxPropIgnoreCase(node.attributes, "muted")) === true) return;
 			const parent = node.parent;
 			if (!parent || !isNodeOfType(parent, "JSXElement")) {
@@ -15215,7 +15226,7 @@ const mediaHasCaption = defineRule({
 				if (!isNodeOfType(child, "JSXElement")) return false;
 				const opening = child.openingElement;
 				const childTag = getElementType(opening, context.settings);
-				if (!settings.track.includes(childTag)) return false;
+				if (!settings.track.has(childTag)) return false;
 				const kindAttribute = hasJsxPropIgnoreCase(opening.attributes, "kind");
 				if (!kindAttribute) return false;
 				let kindValue = kindAttribute.value;
@@ -24630,14 +24641,14 @@ const collectRoleBranches = (expression, out) => {
 	out.hasOpaqueBranch = true;
 };
 const buildMessage$12 = (tag) => `Keyboard & screen reader users can't trigger this \`<${tag}>\` because it isn't interactive, so use a button or link or add an interactive role.`;
-const INTERACTIVE_HANDLERS = [
+const INTERACTIVE_HANDLERS_LOWER = new Set([
 	"onClick",
 	"onMouseDown",
 	"onMouseUp",
 	"onKeyDown",
 	"onKeyPress",
 	"onKeyUp"
-];
+].map((handlerName) => handlerName.toLowerCase()));
 const noNoninteractiveElementInteractions = defineRule({
 	id: "no-noninteractive-element-interactions",
 	title: "Handler on non-interactive element",
@@ -24652,7 +24663,16 @@ const noNoninteractiveElementInteractions = defineRule({
 			const tag = getElementType(node, context.settings);
 			if (!NON_INTERACTIVE_ELEMENTS.has(tag)) return;
 			if (tag === "label") return;
-			if (!INTERACTIVE_HANDLERS.some((handler) => hasJsxPropIgnoreCase(node.attributes, handler))) return;
+			let hasHandler = false;
+			for (const attribute of node.attributes) {
+				if (!isNodeOfType(attribute, "JSXAttribute")) continue;
+				const attributeName = getJsxAttributeName(attribute.name);
+				if (attributeName && INTERACTIVE_HANDLERS_LOWER.has(attributeName.toLowerCase())) {
+					hasHandler = true;
+					break;
+				}
+			}
+			if (!hasHandler) return;
 			if (isHiddenFromScreenReader(node, context.settings)) return;
 			const roleAttr = hasJsxPropIgnoreCase(node.attributes, "role");
 			if (roleAttr) {
@@ -25965,16 +25985,22 @@ const ELEMENT_ROLE_PAIRS = [
 	["tr", "row"],
 	["ul", "list"]
 ];
-const getElementImplicitRoles = (tag) => {
-	const out = [];
-	for (const [element, role] of ELEMENT_ROLE_PAIRS) if (element === tag && !out.includes(role)) out.push(role);
-	return out;
-};
-const getTagsForRole = (role) => {
-	const out = [];
-	for (const [element, r] of ELEMENT_ROLE_PAIRS) if (r === role && !out.includes(element)) out.push(element);
-	return out;
+const EMPTY_ROLE_LIST = [];
+const buildLookup = (pairs, keyIndex) => {
+	const lookup = /* @__PURE__ */ new Map();
+	for (const pair of pairs) {
+		const key = pair[keyIndex];
+		const value = pair[keyIndex === 0 ? 1 : 0];
+		const values = lookup.get(key);
+		if (!values) lookup.set(key, [value]);
+		else if (!values.includes(value)) values.push(value);
+	}
+	return lookup;
 };
+const IMPLICIT_ROLES_BY_TAG = buildLookup(ELEMENT_ROLE_PAIRS, 0);
+const TAGS_BY_ROLE = buildLookup(ELEMENT_ROLE_PAIRS, 1);
+const getElementImplicitRoles = (tag) => IMPLICIT_ROLES_BY_TAG.get(tag) ?? EMPTY_ROLE_LIST;
+const getTagsForRole = (role) => TAGS_BY_ROLE.get(role) ?? EMPTY_ROLE_LIST;
 //#endregion
 //#region src/plugin/utils/get-implicit-role.ts
 const getImplicitRole = (node, elementType) => {
@@ -26474,6 +26500,7 @@ const TANSTACK_ROUTE_PROPERTY_ORDER = [
 	"headers",
 	"remountDeps"
 ];
+const TANSTACK_ROUTE_PROPERTY_INDEX = new Map(TANSTACK_ROUTE_PROPERTY_ORDER.map((propertyName, orderIndex) => [propertyName, orderIndex]));
 const TANSTACK_ROUTE_CREATION_FUNCTIONS = new Set([
 	"createFileRoute",
 	"createRoute",
@@ -26489,6 +26516,7 @@ const TANSTACK_MIDDLEWARE_METHOD_ORDER = [
 	"server",
 	"handler"
 ];
+const TANSTACK_MIDDLEWARE_METHOD_INDEX = new Map(TANSTACK_MIDDLEWARE_METHOD_ORDER.map((methodName, orderIndex) => [methodName, orderIndex]));
 const TANSTACK_REDIRECT_FUNCTIONS = new Set(["redirect", "notFound"]);
 const TANSTACK_SERVER_FN_FILE_PATTERN = /\.functions(\.[jt]sx?)?$/;
 const TANSTACK_QUERY_HOOKS = new Set([
@@ -27193,14 +27221,21 @@ const noStaticElementInteractions = defineRule({
 	category: "Accessibility",
 	create: (context) => {
 		const settings = resolveSettings$12(context.settings);
+		const handlersLower = new Set(settings.handlers.map((handlerName) => handlerName.toLowerCase()));
 		const isTestlikeFile = isTestlikeFilename(context.filename);
 		return { JSXOpeningElement(node) {
 			if (isTestlikeFile) return;
 			let hasNonBlockerHandler = false;
 			let hasAnyHandler = false;
-			for (const handler of settings.handlers) {
-				const attribute = hasJsxPropIgnoreCase(node.attributes, handler);
-				if (!attribute) continue;
+			let seenHandlerNames = null;
+			for (const attribute of node.attributes) {
+				if (!isNodeOfType(attribute, "JSXAttribute")) continue;
+				const attributeName = getJsxAttributeName(attribute.name);
+				if (!attributeName) continue;
+				const handlerNameLower = attributeName.toLowerCase();
+				if (!handlersLower.has(handlerNameLower)) continue;
+				if (seenHandlerNames?.has(handlerNameLower)) continue;
+				(seenHandlerNames ??= /* @__PURE__ */ new Set()).add(handlerNameLower);
 				if (isNullValue(attribute)) continue;
 				hasAnyHandler = true;
 				if (!isPureEventBlockerHandler(attribute)) {
@@ -28381,13 +28416,13 @@ const DOM_ATTRIBUTES_TO_CAMEL = new Map([
 	["xml:lang", "xmlLang"],
 	["xml:space", "xmlSpace"]
 ]);
-const DOM_PROPERTIES_IGNORE_CASE = [
+const DOM_PROPERTIES_IGNORE_CASE_BY_LOWER = new Map([
 	"charset",
 	"allowFullScreen",
 	"webkitAllowFullScreen",
 	"mozAllowFullScreen",
 	"webkitDirectory"
-];
+].map((name) => [name.toLowerCase(), name]));
 //#endregion
 //#region src/plugin/constants/dom-property-tags.ts
 const DOM_PROPERTY_TO_ALLOWED_TAGS = new Map([
@@ -28591,10 +28626,7 @@ const matchesHtmlTagConventions = (tagName) => {
 	if (!(firstCharacter >= 97 && firstCharacter <= 122)) return false;
 	return !tagName.includes("-");
 };
-const normalizeAttributeCase = (name) => {
-	for (const ignoreCaseName of DOM_PROPERTIES_IGNORE_CASE) if (ignoreCaseName.toLowerCase() === name.toLowerCase()) return ignoreCaseName;
-	return name;
-};
+const normalizeAttributeCase = (name) => DOM_PROPERTIES_IGNORE_CASE_BY_LOWER.get(name.toLowerCase()) ?? name;
 const hasUppercaseChar = (input) => /[A-Z]/.test(input);
 const INVALID_PROP_ON_TAG = (propName, allowedTags) => `React ignores \`${propName}\` here because it only works on these tags: ${allowedTags}.`;
 const DATA_LOWERCASE_REQUIRED = () => `React drops this \`data-*\` prop because of its capital letters.`;
@@ -34791,9 +34823,10 @@ const resolveTextBoundaryName = (openingElement) => {
 	if (isNodeOfType(openingElement.name, "JSXNamespacedName")) return openingElement.name.namespace.name;
 	return resolveJsxElementName(openingElement);
 };
+const TEXT_COMPONENT_KEYWORDS = [...REACT_NATIVE_TEXT_COMPONENT_KEYWORDS];
 const isTextHandlingComponent = (elementName) => {
 	if (REACT_NATIVE_TEXT_COMPONENTS.has(elementName)) return true;
-	return [...REACT_NATIVE_TEXT_COMPONENT_KEYWORDS].some((keyword) => elementName.includes(keyword));
+	return TEXT_COMPONENT_KEYWORDS.some((keyword) => elementName.includes(keyword));
 };
 const isTransparentTextWrapper = (elementName) => elementName !== null && REACT_NATIVE_TEXT_TRANSPARENT_COMPONENTS.has(elementName);
 const isInsideTextHandlingComponent = (node) => {
@@ -41297,10 +41330,10 @@ const tanstackStartRoutePropertyOrder = defineRule({
 			const propertyName = getPropertyKeyName(property);
 			if (propertyName !== null) orderedPropertyNames.push(propertyName);
 		}
-		const sensitiveProperties = orderedPropertyNames.filter((propertyName) => TANSTACK_ROUTE_PROPERTY_ORDER.includes(propertyName));
+		const sensitiveProperties = orderedPropertyNames.filter((propertyName) => TANSTACK_ROUTE_PROPERTY_INDEX.has(propertyName));
 		let lastIndex = -1;
 		for (const propertyName of sensitiveProperties) {
-			const currentIndex = TANSTACK_ROUTE_PROPERTY_ORDER.indexOf(propertyName);
+			const currentIndex = TANSTACK_ROUTE_PROPERTY_INDEX.get(propertyName) ?? -1;
 			if (currentIndex < lastIndex) {
 				const expectedBefore = TANSTACK_ROUTE_PROPERTY_ORDER[lastIndex];
 				context.report({
@@ -41337,10 +41370,10 @@ const tanstackStartServerFnMethodOrder = defineRule({
 		} else return;
 		const ownMethodName = isNodeOfType(node.callee.property, "Identifier") ? node.callee.property.name : null;
 		if (methodNames[methodNames.length - 1] !== ownMethodName) return;
-		const orderSensitiveMethods = methodNames.filter((name) => TANSTACK_MIDDLEWARE_METHOD_ORDER.includes(toMethodOrderToken(name)));
+		const orderSensitiveMethods = methodNames.filter((name) => TANSTACK_MIDDLEWARE_METHOD_INDEX.has(toMethodOrderToken(name)));
 		let lastIndex = -1;
 		for (const methodName of orderSensitiveMethods) {
-			const currentIndex = TANSTACK_MIDDLEWARE_METHOD_ORDER.indexOf(toMethodOrderToken(methodName));
+			const currentIndex = TANSTACK_MIDDLEWARE_METHOD_INDEX.get(toMethodOrderToken(methodName)) ?? -1;
 			if (currentIndex < lastIndex) {
 				const expectedBefore = TANSTACK_MIDDLEWARE_METHOD_ORDER[lastIndex];
 				context.report({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-react-doctor",
-  "version": "0.6.2-dev.d8628d7",
+  "version": "0.6.2-dev.da3b19c",
   "description": "React Doctor rules for oxlint.",
   "keywords": [
     "accessibility",