@flint.fyi/jsx 0.3.3 → 0.5.0

package/lib/rules/svgTitles.js

@@ -1,59 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { SyntaxKind } from "typescript";
-import { getTSNodeRange, typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/svgTitles.ts
-var svgTitles_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Reports <svg> elements without a <title> child element.",
-		id: "svgTitles",
-		presets: ["logical", "logicalStrict"]
-	},
-	messages: { missingTitle: {
-		primary: "This <svg> element is missing a <title> child element.",
-		secondary: [
-			"SVG elements without a title are not accessible to screen readers.",
-			"The <title> element provides a text description of the SVG's content.",
-			"This is required for WCAG 1.1.1 compliance."
-		],
-		suggestions: ["Add a <title> child element with descriptive text", "Use aria-label or aria-labelledby as an alternative"]
-	} },
-	setup(context) {
-		function hasValidAriaLabel(attributes) {
-			return attributes.properties.some((property) => {
-				if (property.kind !== SyntaxKind.JsxAttribute || property.name.kind !== SyntaxKind.Identifier || property.name.text !== "aria-label" && property.name.text !== "aria-labelledby") return false;
-				if (!property.initializer) return false;
-				if (property.initializer.kind === SyntaxKind.JsxExpression) {
-					const { expression } = property.initializer;
-					if (!expression) return false;
-					if (expression.kind === SyntaxKind.StringLiteral || expression.kind === SyntaxKind.NoSubstitutionTemplateLiteral) return expression.text !== "";
-					if (expression.kind === SyntaxKind.Identifier) return expression.text !== "undefined";
-				}
-				if (property.initializer.kind === SyntaxKind.StringLiteral) return property.initializer.text !== "";
-				return false;
-			});
-		}
-		function checkElement(node, { sourceFile }) {
-			const tagName = node.kind === SyntaxKind.JsxElement ? node.openingElement.tagName : node.tagName;
-			if (tagName.kind !== SyntaxKind.Identifier || tagName.text.toLowerCase() !== "svg") return;
-			if (hasValidAriaLabel(node.kind === SyntaxKind.JsxElement ? node.openingElement.attributes : node.attributes)) return;
-			if (node.kind === SyntaxKind.JsxElement && node.children.some(isTitleElement)) return;
-			context.report({
-				message: "missingTitle",
-				range: getTSNodeRange(tagName, sourceFile)
-			});
-		}
-		return { visitors: {
-			JsxElement: checkElement,
-			JsxSelfClosingElement: checkElement
-		} };
-	}
-});
-function isTitleElement(node) {
-	if (node.kind !== SyntaxKind.JsxElement && node.kind !== SyntaxKind.JsxSelfClosingElement) return false;
-	const childTagName = node.kind === SyntaxKind.JsxElement ? node.openingElement.tagName : node.tagName;
-	return childTagName.kind === SyntaxKind.Identifier && childTagName.text === "title";
-}
-//#endregion
-export { svgTitles_default as default };
-
-//# sourceMappingURL=svgTitles.js.map

package/lib/rules/tabIndexPositiveValues.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Reports positive tabIndex values.";
-    readonly id: "tabIndexPositiveValues";
-    readonly presets: readonly ["logical", "logicalStrict"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "noPositiveTabIndex", undefined>;
-export default _default;
-//# sourceMappingURL=tabIndexPositiveValues.d.ts.map

package/lib/rules/tabIndexPositiveValues.js

@@ -1,45 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { SyntaxKind } from "typescript";
-import { getTSNodeRange, typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/tabIndexPositiveValues.ts
-var tabIndexPositiveValues_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Reports positive tabIndex values.",
-		id: "tabIndexPositiveValues",
-		presets: ["logical", "logicalStrict"]
-	},
-	messages: { noPositiveTabIndex: {
-		primary: "Positive `tabIndex` values disrupt tab order and make keyboard navigation unpredictable.",
-		secondary: [
-			"Injecting elements on top of the default tab order with positive `tabIndex` values changes from the intuitive order for users.",
-			"Use tabIndex=\"0\" to include elements in the natural tab order, or tabIndex=\"-1\" to make them programmatically focusable.",
-			"This is required for WCAG 2.4.3 compliance."
-		],
-		suggestions: [
-			"Use tabIndex=\"0\" for elements that should be in the tab order",
-			"Use tabIndex=\"-1\" for elements that should be programmatically focusable",
-			"Remove tabIndex to use the default tab order"
-		]
-	} },
-	setup(context) {
-		return { visitors: { JsxAttribute(node, { sourceFile }) {
-			if (node.name.kind !== SyntaxKind.Identifier || node.name.text.toLowerCase() !== "tabindex" || !node.initializer) return;
-			const value = getInitializerValue(node.initializer);
-			if (value !== void 0 && value > 0) context.report({
-				message: "noPositiveTabIndex",
-				range: getTSNodeRange(node, sourceFile)
-			});
-		} } };
-	}
-});
-function getInitializerValue(initializer) {
-	if (initializer.kind === SyntaxKind.StringLiteral) {
-		const parsed = Number(initializer.text);
-		return Number.isNaN(parsed) ? void 0 : parsed;
-	}
-	if (initializer.kind === SyntaxKind.JsxExpression) return initializer.expression?.kind === SyntaxKind.NumericLiteral ? Number(initializer.expression.text) : void 0;
-}
-//#endregion
-export { tabIndexPositiveValues_default as default };
-
-//# sourceMappingURL=tabIndexPositiveValues.js.map

package/lib/rules/unescapedEntities.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Disallow unescaped HTML entities in JSX text that may cause rendering issues.";
-    readonly id: "unescapedEntities";
-    readonly presets: readonly ["stylistic"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "unescapedEntity", undefined>;
-export default _default;
-//# sourceMappingURL=unescapedEntities.d.ts.map

package/lib/rules/unescapedEntities.js

@@ -1,70 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/unescapedEntities.ts
-const problematicEntities = [
-	{
-		entity: "\"",
-		toBrace: "{\"\\\"\"}",
-		toHTML: """
-	},
-	{
-		entity: "'",
-		toBrace: "{\"'\"}",
-		toHTML: "'"
-	},
-	{
-		entity: ">",
-		toBrace: "{'>'}",
-		toHTML: ">"
-	},
-	{
-		entity: "}",
-		toBrace: "{'}'}",
-		toHTML: "}"
-	}
-];
-var unescapedEntities_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Disallow unescaped HTML entities in JSX text that may cause rendering issues.",
-		id: "unescapedEntities",
-		presets: ["stylistic"]
-	},
-	messages: { unescapedEntity: {
-		primary: "This unescaped entity `{{ entity }}` may not render properly.",
-		secondary: ["Certain characters like >, <, \", ', {, and } can cause issues in JSX when not properly escaped.", "React may not render these characters correctly or they might break JSX syntax."],
-		suggestions: ["Use HTML entities: {{ html }}", "Wrap in JSX expression: {{ brace }}"]
-	} },
-	setup(context) {
-		return { visitors: { JsxText(node, { sourceFile }) {
-			const nodeStart = node.getStart(sourceFile);
-			const reports = [];
-			for (const { entity, toBrace, toHTML } of problematicEntities) {
-				let index = 0;
-				while ((index = node.text.indexOf(entity, index)) !== -1) {
-					reports.push({
-						begin: nodeStart + index,
-						data: {
-							brace: toBrace,
-							entity,
-							html: toHTML
-						},
-						end: nodeStart + index + entity.length
-					});
-					index += entity.length;
-				}
-			}
-			for (const report of reports.toSorted((a, b) => a.begin - b.begin)) context.report({
-				data: report.data,
-				message: "unescapedEntity",
-				range: {
-					begin: report.begin,
-					end: report.end
-				}
-			});
-		} } };
-	}
-});
-//#endregion
-export { unescapedEntities_default as default };
-
-//# sourceMappingURL=unescapedEntities.js.map

package/lib/rules/unnecessaryFragments.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Disallow unnecessary JSX fragments that wrap a single child or have no children.";
-    readonly id: "unnecessaryFragments";
-    readonly presets: readonly ["stylistic"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "unnecessaryFragment", undefined>;
-export default _default;
-//# sourceMappingURL=unnecessaryFragments.d.ts.map

package/lib/rules/unnecessaryFragments.js

@@ -1,51 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { SyntaxKind } from "typescript";
-import { typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/unnecessaryFragments.ts
-var unnecessaryFragments_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Disallow unnecessary JSX fragments that wrap a single child or have no children.",
-		id: "unnecessaryFragments",
-		presets: ["stylistic"]
-	},
-	messages: { unnecessaryFragment: {
-		primary: "Unnecessary fragment wrapping {{ childType }}.",
-		secondary: [
-			"Fragments are only needed when rendering multiple adjacent elements.",
-			"A fragment wrapping a single child or no children adds unnecessary complexity.",
-			"Remove the fragment and use the child directly, or remove empty fragments entirely."
-		],
-		suggestions: ["Remove the fragment wrapper and use the child element directly."]
-	} },
-	setup(context) {
-		function checkNodeChildren(node, range) {
-			const children = node.children.filter((child) => child.kind !== SyntaxKind.JsxText || !!child.text.trim().length);
-			let childType;
-			if (!children.length) childType = "no children";
-			else if (children.length === 1) childType = "a single child";
-			if (childType) context.report({
-				data: { childType },
-				message: "unnecessaryFragment",
-				range
-			});
-		}
-		return { visitors: {
-			JsxElement(node, { sourceFile }) {
-				if (node.openingElement.tagName.kind === SyntaxKind.Identifier && !node.openingElement.attributes.properties.length && node.openingElement.tagName.text === "Fragment") checkNodeChildren(node, {
-					begin: node.openingElement.getStart(sourceFile),
-					end: node.closingElement.getEnd()
-				});
-			},
-			JsxFragment(node, { sourceFile }) {
-				checkNodeChildren(node, {
-					begin: node.openingFragment.getStart(sourceFile),
-					end: node.closingFragment.getEnd()
-				});
-			}
-		} };
-	}
-});
-//#endregion
-export { unnecessaryFragments_default as default };
-
-//# sourceMappingURL=unnecessaryFragments.js.map