react-doctor 0.2.1 → 0.2.3

package/dist/index.js

@@ -2,7 +2,8 @@ import { createRequire } from "node:module";
 import path from "node:path";
 import fs from "node:fs";
 import { spawn, spawnSync } from "node:child_process";
-import reactDoctorPlugin, { ALL_REACT_DOCTOR_RULE_KEYS, BUILTIN_A11Y_RULES, BUILTIN_REACT_RULES, FRAMEWORK_SPECIFIC_RULE_KEYS, MOTION_LIBRARY_PACKAGES, REACT_COMPILER_RULES, YOU_MIGHT_NOT_NEED_EFFECT_RULES } from "oxlint-plugin-react-doctor";
+import reactDoctorPlugin, { ALL_REACT_DOCTOR_RULE_KEYS, FRAMEWORK_SPECIFIC_RULE_KEYS, MOTION_LIBRARY_PACKAGES, REACT_COMPILER_RULES, REACT_DOCTOR_RULES } from "oxlint-plugin-react-doctor";
+import { gzipSync } from "node:zlib";
 import os from "node:os";
 import * as ts from "typescript";
 //#region \0rolldown/runtime.js
@@ -680,19 +681,6 @@ const findReactInWorkspaces = (rootDirectory, packageJson) => {
 	}
 	return result;
 };
-const REACT_DEPENDENCY_NAMES = new Set([
-	"react",
-	"react-native",
-	"next"
-]);
-const hasReactDependency = (packageJson) => {
-	const allDependencies = {
-		...packageJson.peerDependencies,
-		...packageJson.dependencies,
-		...packageJson.devDependencies
-	};
-	return Object.keys(allDependencies).some((packageName) => REACT_DEPENDENCY_NAMES.has(packageName));
-};
 const findDependencyInfoFromMonorepoRoot = (directory) => {
 	const monorepoRoot = findMonorepoRoot(directory);
 	if (!monorepoRoot) return EMPTY_DEPENDENCY_INFO;
@@ -720,13 +708,13 @@ const findDependencyInfoFromMonorepoRoot = (directory) => {
 			"peerDependencies"
 		]
 	}) : null;
-	const shouldUseReactFallback = leafPackageJson ? hasReactDependency(leafPackageJson) : true;
+	const shouldUseReactFallback = !leafReactDeclaration?.hasDeclaration;
 	const shouldUseTailwindFallback = leafTailwindDeclaration?.hasDeclaration ?? true;
 	const reactCatalogVersion = shouldUseReactFallback ? resolveCatalogVersion(rootPackageJson, "react", monorepoRoot, leafReactDeclaration?.catalogReference) : null;
 	const tailwindCatalogVersion = shouldUseTailwindFallback ? resolveCatalogVersion(rootPackageJson, "tailwindcss", monorepoRoot, leafTailwindDeclaration?.catalogReference) : null;
 	const workspaceInfo = findReactInWorkspaces(monorepoRoot, rootPackageJson);
 	return {
-		reactVersion: shouldUseReactFallback ? reactCatalogVersion ?? rootInfo.reactVersion ?? workspaceInfo.reactVersion : null,
+		reactVersion: shouldUseReactFallback ? reactCatalogVersion ?? rootInfo.reactVersion ?? workspaceInfo.reactVersion : rootInfo.reactVersion ?? workspaceInfo.reactVersion,
 		tailwindVersion: shouldUseTailwindFallback ? tailwindCatalogVersion ?? rootInfo.tailwindVersion ?? workspaceInfo.tailwindVersion : null,
 		framework: rootInfo.framework !== "unknown" ? rootInfo.framework : workspaceInfo.framework
 	};
@@ -793,6 +781,19 @@ const resolveEffectiveReactMajor = (reactVersion, packageJson) => {
 	if (peerFloor === null) return hasUpperBoundOnlyPeerRange(peerReactRange) ? null : installedReactMajor;
 	return installedReactMajor !== null ? Math.min(installedReactMajor, peerFloor) : peerFloor;
 };
+const REACT_DEPENDENCY_NAMES = new Set([
+	"react",
+	"react-native",
+	"next"
+]);
+const hasReactDependency = (packageJson) => {
+	const allDependencies = {
+		...packageJson.peerDependencies,
+		...packageJson.dependencies,
+		...packageJson.devDependencies
+	};
+	return Object.keys(allDependencies).some((packageName) => REACT_DEPENDENCY_NAMES.has(packageName));
+};
 const listWorkspacePackages = (rootDirectory) => {
 	const packageJsonPath = path.join(rootDirectory, "package.json");
 	if (!isFile(packageJsonPath)) return [];
@@ -3012,6 +3013,136 @@ const getDiagnosticRuleIdentity = (diagnostic) => ({
 	category: diagnostic.category,
 	tags: diagnostic.plugin === "react-doctor" ? reactDoctorPlugin.rules[diagnostic.rule]?.tags ?? [] : []
 });
+const LEGACY_RULE_KEY_TO_NATIVE_RULE_KEY = {
+	"effect/no-adjust-state-on-prop-change": "react-doctor/no-adjust-state-on-prop-change",
+	"effect/no-chain-state-updates": "react-doctor/no-chain-state-updates",
+	"effect/no-derived-state": "react-doctor/no-derived-state",
+	"effect/no-event-handler": "react-doctor/no-event-handler",
+	"effect/no-initialize-state": "react-doctor/no-initialize-state",
+	"effect/no-pass-data-to-parent": "react-doctor/no-pass-data-to-parent",
+	"effect/no-pass-live-state-to-parent": "react-doctor/no-pass-live-state-to-parent",
+	"effect/no-reset-all-state-on-prop-change": "react-doctor/no-reset-all-state-on-prop-change",
+	"jsx-a11y/alt-text": "react-doctor/alt-text",
+	"jsx-a11y/anchor-ambiguous-text": "react-doctor/anchor-ambiguous-text",
+	"jsx-a11y/anchor-has-content": "react-doctor/anchor-has-content",
+	"jsx-a11y/anchor-is-valid": "react-doctor/anchor-is-valid",
+	"jsx-a11y/aria-activedescendant-has-tabindex": "react-doctor/aria-activedescendant-has-tabindex",
+	"jsx-a11y/aria-props": "react-doctor/aria-props",
+	"jsx-a11y/aria-proptypes": "react-doctor/aria-proptypes",
+	"jsx-a11y/aria-role": "react-doctor/aria-role",
+	"jsx-a11y/aria-unsupported-elements": "react-doctor/aria-unsupported-elements",
+	"jsx-a11y/autocomplete-valid": "react-doctor/autocomplete-valid",
+	"jsx-a11y/click-events-have-key-events": "react-doctor/click-events-have-key-events",
+	"jsx-a11y/control-has-associated-label": "react-doctor/control-has-associated-label",
+	"jsx-a11y/heading-has-content": "react-doctor/heading-has-content",
+	"jsx-a11y/html-has-lang": "react-doctor/html-has-lang",
+	"jsx-a11y/iframe-has-title": "react-doctor/iframe-has-title",
+	"jsx-a11y/img-redundant-alt": "react-doctor/img-redundant-alt",
+	"jsx-a11y/interactive-supports-focus": "react-doctor/interactive-supports-focus",
+	"jsx-a11y/label-has-associated-control": "react-doctor/label-has-associated-control",
+	"jsx-a11y/lang": "react-doctor/lang",
+	"jsx-a11y/media-has-caption": "react-doctor/media-has-caption",
+	"jsx-a11y/mouse-events-have-key-events": "react-doctor/mouse-events-have-key-events",
+	"jsx-a11y/no-access-key": "react-doctor/no-access-key",
+	"jsx-a11y/no-aria-hidden-on-focusable": "react-doctor/no-aria-hidden-on-focusable",
+	"jsx-a11y/no-autofocus": "react-doctor/no-autofocus",
+	"jsx-a11y/no-distracting-elements": "react-doctor/no-distracting-elements",
+	"jsx-a11y/no-interactive-element-to-noninteractive-role": "react-doctor/no-interactive-element-to-noninteractive-role",
+	"jsx-a11y/no-noninteractive-element-interactions": "react-doctor/no-noninteractive-element-interactions",
+	"jsx-a11y/no-noninteractive-element-to-interactive-role": "react-doctor/no-noninteractive-element-to-interactive-role",
+	"jsx-a11y/no-noninteractive-tabindex": "react-doctor/no-noninteractive-tabindex",
+	"jsx-a11y/no-redundant-roles": "react-doctor/no-redundant-roles",
+	"jsx-a11y/no-static-element-interactions": "react-doctor/no-static-element-interactions",
+	"jsx-a11y/prefer-tag-over-role": "react-doctor/prefer-tag-over-role",
+	"jsx-a11y/role-has-required-aria-props": "react-doctor/role-has-required-aria-props",
+	"jsx-a11y/role-supports-aria-props": "react-doctor/role-supports-aria-props",
+	"jsx-a11y/scope": "react-doctor/scope",
+	"jsx-a11y/tabindex-no-positive": "react-doctor/tabindex-no-positive",
+	"react-hooks/exhaustive-deps": "react-doctor/exhaustive-deps",
+	"react-hooks/rules-of-hooks": "react-doctor/rules-of-hooks",
+	"react-perf/jsx-no-jsx-as-prop": "react-doctor/jsx-no-jsx-as-prop",
+	"react-perf/jsx-no-new-array-as-prop": "react-doctor/jsx-no-new-array-as-prop",
+	"react-perf/jsx-no-new-function-as-prop": "react-doctor/jsx-no-new-function-as-prop",
+	"react-perf/jsx-no-new-object-as-prop": "react-doctor/jsx-no-new-object-as-prop",
+	"react-refresh/only-export-components": "react-doctor/only-export-components",
+	"react/button-has-type": "react-doctor/button-has-type",
+	"react/checked-requires-onchange-or-readonly": "react-doctor/checked-requires-onchange-or-readonly",
+	"react/display-name": "react-doctor/display-name",
+	"react/exhaustive-deps": "react-doctor/exhaustive-deps",
+	"react/forbid-component-props": "react-doctor/forbid-component-props",
+	"react/forbid-dom-props": "react-doctor/forbid-dom-props",
+	"react/forbid-elements": "react-doctor/forbid-elements",
+	"react/forward-ref-uses-ref": "react-doctor/forward-ref-uses-ref",
+	"react/hook-use-state": "react-doctor/hook-use-state",
+	"react/iframe-missing-sandbox": "react-doctor/iframe-missing-sandbox",
+	"react/jsx-boolean-value": "react-doctor/jsx-boolean-value",
+	"react/jsx-curly-brace-presence": "react-doctor/jsx-curly-brace-presence",
+	"react/jsx-filename-extension": "react-doctor/jsx-filename-extension",
+	"react/jsx-fragments": "react-doctor/jsx-fragments",
+	"react/jsx-handler-names": "react-doctor/jsx-handler-names",
+	"react/jsx-key": "react-doctor/jsx-key",
+	"react/jsx-max-depth": "react-doctor/jsx-max-depth",
+	"react/jsx-no-comment-textnodes": "react-doctor/jsx-no-comment-textnodes",
+	"react/jsx-no-constructed-context-values": "react-doctor/jsx-no-constructed-context-values",
+	"react/jsx-no-duplicate-props": "react-doctor/jsx-no-duplicate-props",
+	"react/jsx-no-jsx-as-prop": "react-doctor/jsx-no-jsx-as-prop",
+	"react/jsx-no-new-array-as-prop": "react-doctor/jsx-no-new-array-as-prop",
+	"react/jsx-no-new-function-as-prop": "react-doctor/jsx-no-new-function-as-prop",
+	"react/jsx-no-new-object-as-prop": "react-doctor/jsx-no-new-object-as-prop",
+	"react/jsx-no-script-url": "react-doctor/jsx-no-script-url",
+	"react/jsx-no-target-blank": "react-doctor/jsx-no-target-blank",
+	"react/jsx-no-undef": "react-doctor/jsx-no-undef",
+	"react/jsx-no-useless-fragment": "react-doctor/jsx-no-useless-fragment",
+	"react/jsx-pascal-case": "react-doctor/jsx-pascal-case",
+	"react/jsx-props-no-spread-multi": "react-doctor/jsx-props-no-spread-multi",
+	"react/jsx-props-no-spreading": "react-doctor/jsx-props-no-spreading",
+	"react/no-array-index-key": "react-doctor/no-array-index-key",
+	"react/no-children-prop": "react-doctor/no-children-prop",
+	"react/no-clone-element": "react-doctor/no-clone-element",
+	"react/no-danger": "react-doctor/no-danger",
+	"react/no-danger-with-children": "react-doctor/no-danger-with-children",
+	"react/no-did-mount-set-state": "react-doctor/no-did-mount-set-state",
+	"react/no-did-update-set-state": "react-doctor/no-did-update-set-state",
+	"react/no-direct-mutation-state": "react-doctor/no-direct-mutation-state",
+	"react/no-find-dom-node": "react-doctor/no-find-dom-node",
+	"react/no-is-mounted": "react-doctor/no-is-mounted",
+	"react/no-multi-comp": "react-doctor/no-multi-comp",
+	"react/no-namespace": "react-doctor/no-namespace",
+	"react/no-react-children": "react-doctor/no-react-children",
+	"react/no-redundant-should-component-update": "react-doctor/no-redundant-should-component-update",
+	"react/no-render-return-value": "react-doctor/no-render-return-value",
+	"react/no-set-state": "react-doctor/no-set-state",
+	"react/no-string-refs": "react-doctor/no-string-refs",
+	"react/no-this-in-sfc": "react-doctor/no-this-in-sfc",
+	"react/no-unescaped-entities": "react-doctor/no-unescaped-entities",
+	"react/no-unknown-property": "react-doctor/no-unknown-property",
+	"react/no-unsafe": "react-doctor/no-unsafe",
+	"react/no-unstable-nested-components": "react-doctor/no-unstable-nested-components",
+	"react/no-will-update-set-state": "react-doctor/no-will-update-set-state",
+	"react/only-export-components": "react-doctor/only-export-components",
+	"react/prefer-es6-class": "react-doctor/prefer-es6-class",
+	"react/prefer-function-component": "react-doctor/prefer-function-component",
+	"react/react-in-jsx-scope": "react-doctor/react-in-jsx-scope",
+	"react/require-render-return": "react-doctor/require-render-return",
+	"react/rules-of-hooks": "react-doctor/rules-of-hooks",
+	"react/self-closing-comp": "react-doctor/self-closing-comp",
+	"react/state-in-constructor": "react-doctor/state-in-constructor",
+	"react/style-prop-object": "react-doctor/style-prop-object",
+	"react/void-dom-elements-no-children": "react-doctor/void-dom-elements-no-children"
+};
+const NATIVE_RULE_KEY_TO_LEGACY_RULE_KEYS = /* @__PURE__ */ new Map();
+for (const [legacyRuleKey, nativeRuleKey] of Object.entries(LEGACY_RULE_KEY_TO_NATIVE_RULE_KEY)) {
+	const aliases = NATIVE_RULE_KEY_TO_LEGACY_RULE_KEYS.get(nativeRuleKey) ?? [];
+	aliases.push(legacyRuleKey);
+	NATIVE_RULE_KEY_TO_LEGACY_RULE_KEYS.set(nativeRuleKey, aliases);
+}
+const getLegacyRuleKeysForNative = (ruleKey) => NATIVE_RULE_KEY_TO_LEGACY_RULE_KEYS.get(ruleKey) ?? [];
+const canonicalizeRuleKey = (ruleKey) => LEGACY_RULE_KEY_TO_NATIVE_RULE_KEY[ruleKey] ?? ruleKey;
+const isSameRuleKey = (candidateRuleKey, targetRuleKey) => canonicalizeRuleKey(candidateRuleKey) === canonicalizeRuleKey(targetRuleKey);
+const getEquivalentRuleKeys = (ruleKey) => {
+	const nativeRuleKey = canonicalizeRuleKey(ruleKey);
+	return [nativeRuleKey, ...getLegacyRuleKeysForNative(nativeRuleKey)];
+};
 /**
 * Resolves the user-configured severity override for a rule.
 * Per-rule overrides win over per-category overrides. Returns
@@ -3020,7 +3151,14 @@ const getDiagnosticRuleIdentity = (diagnostic) => ({
 */
 const resolveRuleSeverityOverride = (input, controls) => {
 	if (!controls) return void 0;
-	return controls.rules?.[input.ruleKey] ?? (input.category !== void 0 ? controls.categories?.[input.category] : void 0);
+	const exactRuleOverride = controls.rules?.[input.ruleKey];
+	if (exactRuleOverride !== void 0) return exactRuleOverride;
+	for (const equivalentRuleKey of getEquivalentRuleKeys(input.ruleKey)) {
+		if (equivalentRuleKey === input.ruleKey) continue;
+		const equivalentRuleOverride = controls.rules?.[equivalentRuleKey];
+		if (equivalentRuleOverride !== void 0) return equivalentRuleOverride;
+	}
+	return input.category !== void 0 ? controls.categories?.[input.category] : void 0;
 };
 const SEVERITY_FOR_OVERRIDE = {
 	error: "error",
@@ -3229,14 +3367,19 @@ const describeFailure = (error) => {
 	if (error instanceof Error && error.message) return error.message;
 	return String(error);
 };
-const calculateScore = async (diagnostics) => {
+const calculateScore = async (diagnostics, options = {}) => {
 	const controller = new AbortController();
 	const timeoutId = setTimeout(() => controller.abort(), FETCH_TIMEOUT_MS);
+	const requestUrl = options.isCi ? `${SCORE_API_URL}?ci=1` : SCORE_API_URL;
 	try {
-		const response = await fetch(SCORE_API_URL, {
+		const compressedBody = gzipSync(JSON.stringify({ diagnostics: stripFilePaths(diagnostics) }));
+		const response = await fetch(requestUrl, {
 			method: "POST",
-			headers: { "Content-Type": "application/json" },
-			body: JSON.stringify({ diagnostics: stripFilePaths(diagnostics) }),
+			headers: {
+				"Content-Type": "application/json",
+				"Content-Encoding": "gzip"
+			},
+			body: compressedBody,
 			signal: controller.signal
 		});
 		if (!response.ok) {
@@ -3323,60 +3466,6 @@ const canOxlintExtendConfig = (configPath) => {
 	if (extendsEntries.length === 0) return true;
 	return extendsEntries.some((entry) => typeof entry === "string" && isLocalPathExtend(entry));
 };
-const REDUCED_MOTION_GREP_PATTERN = "prefers-reduced-motion|useReducedMotion|MotionConfig|reducedMotion";
-const REDUCED_MOTION_FILE_GLOBS = [
-	"*.ts",
-	"*.tsx",
-	"*.js",
-	"*.jsx",
-	"*.css",
-	"*.scss"
-];
-const MISSING_REDUCED_MOTION_DIAGNOSTIC = {
-	filePath: "package.json",
-	plugin: "react-doctor",
-	rule: "require-reduced-motion",
-	severity: "error",
-	message: "Project uses a motion library but has no prefers-reduced-motion handling — required for accessibility (WCAG 2.3.3)",
-	help: "Add `useReducedMotion()` from your animation library, or a `@media (prefers-reduced-motion: reduce)` CSS query",
-	line: 0,
-	column: 0,
-	category: "Accessibility"
-};
-const checkReducedMotion = (rootDirectory) => {
-	const packageJsonPath = path.join(rootDirectory, "package.json");
-	if (!isFile(packageJsonPath)) return [];
-	let hasMotionLibrary = false;
-	try {
-		const packageJson = readPackageJson(packageJsonPath);
-		const allDependencies = {
-			...packageJson.dependencies,
-			...packageJson.devDependencies
-		};
-		hasMotionLibrary = Object.keys(allDependencies).some((packageName) => MOTION_LIBRARY_PACKAGES.has(packageName));
-	} catch {
-		return [];
-	}
-	if (!hasMotionLibrary) return [];
-	const result = spawnSync("git", [
-		"grep",
-		"-ql",
-		"-E",
-		REDUCED_MOTION_GREP_PATTERN,
-		"--",
-		...REDUCED_MOTION_FILE_GLOBS
-	], {
-		cwd: rootDirectory,
-		stdio: [
-			"ignore",
-			"pipe",
-			"pipe"
-		]
-	});
-	if (result.error) return [MISSING_REDUCED_MOTION_DIAGNOSTIC];
-	if (result.status === 0) return [];
-	return [MISSING_REDUCED_MOTION_DIAGNOSTIC];
-};
 const LINGUIST_ATTRIBUTE_PATTERN = /^linguist-(?:vendored|generated)(?:=([a-zA-Z0-9]+))?$/i;
 const FALSY_VALUES = new Set([
 	"false",
@@ -3556,6 +3645,148 @@ const collectIgnorePatterns = (rootDirectory) => {
 	cachedPatternsByRoot.set(rootDirectory, patterns);
 	return patterns;
 };
+const TSCONFIG_FILENAMES$1 = ["tsconfig.json", "tsconfig.base.json"];
+const resolveTsConfigPath = (rootDirectory) => {
+	for (const filename of TSCONFIG_FILENAMES$1) {
+		const candidate = path.join(rootDirectory, filename);
+		if (fs.existsSync(candidate)) return candidate;
+	}
+};
+const collectDeadCodeIgnorePatterns = (rootDirectory, userConfig) => {
+	const seen = /* @__PURE__ */ new Set();
+	const sources = [
+		readIgnoreFile(path.join(rootDirectory, ".gitignore")),
+		collectIgnorePatterns(rootDirectory),
+		userConfig?.ignore?.files ?? []
+	];
+	for (const source of sources) for (const pattern of source) seen.add(pattern);
+	return [...seen].filter((pattern) => pattern.length > 0);
+};
+const toRelativeFilePath = (rootDirectory, filePath) => {
+	const relative = toRelativePath(filePath, rootDirectory);
+	return relative.length > 0 ? relative : filePath.replace(/\\/g, "/");
+};
+const checkDeadCode = async (options) => {
+	const { rootDirectory, userConfig } = options;
+	if (!fs.existsSync(path.join(rootDirectory, "package.json"))) return [];
+	const { analyze, defineConfig } = await import("deslop-js");
+	const ignorePatterns = collectDeadCodeIgnorePatterns(rootDirectory, userConfig);
+	const result = await analyze(defineConfig({
+		rootDir: rootDirectory,
+		tsConfigPath: resolveTsConfigPath(rootDirectory),
+		...ignorePatterns.length > 0 ? { ignorePatterns } : {}
+	}));
+	const toRelative = (filePath) => toRelativeFilePath(rootDirectory, filePath);
+	const diagnostics = [];
+	for (const unusedFile of result.unusedFiles) diagnostics.push({
+		filePath: toRelative(unusedFile.path),
+		plugin: "deslop",
+		rule: "unused-file",
+		severity: "warning",
+		message: "Unused file — not reachable from any entry point",
+		help: "Delete the file if it is truly unreachable, or import it from an entry point.",
+		line: 0,
+		column: 0,
+		category: "Dead Code"
+	});
+	for (const unusedExport of result.unusedExports) {
+		const label = unusedExport.isTypeOnly ? "type export" : "export";
+		diagnostics.push({
+			filePath: toRelative(unusedExport.path),
+			plugin: "deslop",
+			rule: unusedExport.isTypeOnly ? "unused-type" : "unused-export",
+			severity: "warning",
+			message: `Unused ${label}: \`${unusedExport.name}\``,
+			help: "Drop the `export` keyword (or remove the declaration) if no other module uses this symbol.",
+			line: unusedExport.line,
+			column: unusedExport.column,
+			category: "Dead Code"
+		});
+	}
+	for (const unusedDependency of result.unusedDependencies) {
+		const label = unusedDependency.isDevDependency ? "devDependency" : "dependency";
+		diagnostics.push({
+			filePath: "package.json",
+			plugin: "deslop",
+			rule: unusedDependency.isDevDependency ? "unused-dev-dependency" : "unused-dependency",
+			severity: "warning",
+			message: `Unused ${label}: \`${unusedDependency.name}\``,
+			help: "Remove the dependency from package.json if it is genuinely unused.",
+			line: 0,
+			column: 0,
+			category: "Dead Code"
+		});
+	}
+	for (const cycle of result.circularDependencies) {
+		if (cycle.files.length === 0) continue;
+		diagnostics.push({
+			filePath: toRelative(cycle.files[0]),
+			plugin: "deslop",
+			rule: "circular-dependency",
+			severity: "warning",
+			message: `Circular import cycle: ${cycle.files.map(toRelative).join(" → ")}`,
+			help: "Break the cycle by extracting the shared code into a third module that both files import.",
+			line: 0,
+			column: 0,
+			category: "Dead Code"
+		});
+	}
+	return diagnostics;
+};
+const REDUCED_MOTION_GREP_PATTERN = "prefers-reduced-motion|useReducedMotion|MotionConfig|reducedMotion";
+const REDUCED_MOTION_FILE_GLOBS = [
+	"*.ts",
+	"*.tsx",
+	"*.js",
+	"*.jsx",
+	"*.css",
+	"*.scss"
+];
+const MISSING_REDUCED_MOTION_DIAGNOSTIC = {
+	filePath: "package.json",
+	plugin: "react-doctor",
+	rule: "require-reduced-motion",
+	severity: "error",
+	message: "Project uses a motion library but has no prefers-reduced-motion handling — required for accessibility (WCAG 2.3.3)",
+	help: "Add `useReducedMotion()` from your animation library, or a `@media (prefers-reduced-motion: reduce)` CSS query",
+	line: 0,
+	column: 0,
+	category: "Accessibility"
+};
+const checkReducedMotion = (rootDirectory) => {
+	const packageJsonPath = path.join(rootDirectory, "package.json");
+	if (!isFile(packageJsonPath)) return [];
+	let hasMotionLibrary = false;
+	try {
+		const packageJson = readPackageJson(packageJsonPath);
+		const allDependencies = {
+			...packageJson.dependencies,
+			...packageJson.devDependencies
+		};
+		hasMotionLibrary = Object.keys(allDependencies).some((packageName) => MOTION_LIBRARY_PACKAGES.has(packageName));
+	} catch {
+		return [];
+	}
+	if (!hasMotionLibrary) return [];
+	const result = spawnSync("git", [
+		"grep",
+		"-ql",
+		"-E",
+		REDUCED_MOTION_GREP_PATTERN,
+		"--",
+		...REDUCED_MOTION_FILE_GLOBS
+	], {
+		cwd: rootDirectory,
+		stdio: [
+			"ignore",
+			"pipe",
+			"pipe"
+		]
+	});
+	if (result.error) return [MISSING_REDUCED_MOTION_DIAGNOSTIC];
+	if (result.status === 0) return [];
+	return [MISSING_REDUCED_MOTION_DIAGNOSTIC];
+};
 const createNodeReadFileLinesSync = (rootDirectory) => {
 	return (filePath) => {
 		const absolutePath = path.isAbsolute(filePath) ? filePath : path.join(rootDirectory, filePath);
@@ -3685,7 +3916,7 @@ const isRuleListedInComment = (ruleList, ruleId) => {
 	if (!trimmed) return true;
 	const ruleSection = stripDescriptionTail(trimmed).trim();
 	if (!ruleSection) return true;
-	return ruleSection.split(/[,\s]+/).some((token) => token.trim() === ruleId);
+	return ruleSection.split(/[,\s]+/).some((token) => isSameRuleKey(token.trim(), ruleId));
 };
 const DISABLE_LINE_PATTERN = /(?:\/\/|\/\*)\s*react-doctor-disable-line\b(?:\s+([^\r\n]*?))?\s*(?:\*\/)?\s*\}?\s*$/;
 const formatLineGap = (gapLineCount) => `${gapLineCount} line${gapLineCount === 1 ? "" : "s"}`;
@@ -3837,6 +4068,10 @@ const isInsideStringOnlyWrapper = (lines, diagnosticLine, diagnosticColumn, wrap
 	}
 	return false;
 };
+const isIgnoredRule = (ignoredRules, ruleIdentifier) => {
+	for (const ignoredRule of ignoredRules) if (isSameRuleKey(ignoredRule, ruleIdentifier)) return true;
+	return false;
+};
 const filterIgnoredDiagnostics = (diagnostics, config, rootDirectory, readFileLinesSync) => {
 	const ignoredRules = new Set(Array.isArray(config.ignore?.rules) ? config.ignore.rules.filter((rule) => typeof rule === "string") : []);
 	const ignoredFilePatterns = compileIgnoredFilePatterns(config);
@@ -3847,8 +4082,7 @@ const filterIgnoredDiagnostics = (diagnostics, config, rootDirectory, readFileLi
 	const hasRawTextWrappers = rawTextWrapperComponentNames.size > 0;
 	const getFileLines = createFileLinesCache(rootDirectory, readFileLinesSync);
 	return diagnostics.filter((diagnostic) => {
-		const ruleIdentifier = `${diagnostic.plugin}/${diagnostic.rule}`;
-		if (ignoredRules.has(ruleIdentifier)) return false;
+		if (isIgnoredRule(ignoredRules, `${diagnostic.plugin}/${diagnostic.rule}`)) return false;
 		if (isFileIgnoredByPatterns(diagnostic.filePath, rootDirectory, ignoredFilePatterns)) return false;
 		if (isDiagnosticIgnoredByOverrides(diagnostic, rootDirectory, compiledOverrides)) return false;
 		if ((hasTextComponents || hasRawTextWrappers) && diagnostic.rule === "rn-no-raw-text" && diagnostic.line > 0) {
@@ -3876,11 +4110,24 @@ const filterInlineSuppressions = (diagnostics, rootDirectory, readFileLinesSync)
 		}] : [diagnostic];
 	});
 };
-const TEST_FILE_PATH_PATTERN = /(?:^|\/)(?:__tests__|__test__|tests|test|__mocks__|cypress|e2e|playwright)\/|\.(?:test|spec|stories|story|fixture|fixtures)\.(?:[cm]?[jt]sx?)$/;
+const TEST_FILE_DIRECTORY_PATTERN = /(?:^|\/)(?:__tests__|__test__|tests|test|__mocks__|cypress|e2e|playwright)\//;
+const TEST_FILE_SUFFIX_PATTERN = /\.(?:test|spec|stories|story|fixture|fixtures)\.(?:[cm]?[jt]sx?)$/;
+const FIXTURE_PROJECT_PATTERN = /\/(?:fixtures|__fixtures__)\//;
+const SOURCE_ROOT_PATTERN = /\/(?:src|app|lib|components|pages|features|modules|packages|apps|frontend|client)\//g;
+const stripAboveSourceRoot = (relativePath) => {
+	const fixtureMatch = FIXTURE_PROJECT_PATTERN.exec(relativePath);
+	if (fixtureMatch === null) return relativePath;
+	let lastIdx = -1;
+	for (const match of relativePath.matchAll(SOURCE_ROOT_PATTERN)) if (match.index !== void 0 && match.index > lastIdx) lastIdx = match.index;
+	if (lastIdx >= 0) return relativePath.slice(lastIdx);
+	return relativePath.slice(fixtureMatch.index + fixtureMatch[0].length - 1);
+};
 const isTestFilePath = (relativePath) => {
 	if (relativePath.length === 0) return false;
 	const forwardSlashed = relativePath.replaceAll("\\", "/");
-	return TEST_FILE_PATH_PATTERN.test(forwardSlashed);
+	if (TEST_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return true;
+	const scoped = stripAboveSourceRoot(forwardSlashed);
+	return TEST_FILE_DIRECTORY_PATTERN.test(scoped);
 };
 const testFileResultCache = /* @__PURE__ */ new Map();
 const clearAutoSuppressionCaches = () => {
@@ -3888,7 +4135,9 @@ const clearAutoSuppressionCaches = () => {
 };
 const shouldAutoSuppress = (diagnostic) => {
 	const filePath = diagnostic.filePath;
-	if ((diagnostic.plugin === "react-doctor" ? reactDoctorPlugin.rules[diagnostic.rule] : null)?.tags?.includes("test-noise")) {
+	const rule = diagnostic.plugin === "react-doctor" ? reactDoctorPlugin.rules[diagnostic.rule] : null;
+	if (rule?.tags?.includes("test-noise")) {
+		if (rule.tags.includes("migration-hint")) return false;
 		let isTest = testFileResultCache.get(filePath);
 		if (isTest === void 0) {
 			isTest = isTestFilePath(filePath);
@@ -3905,9 +4154,13 @@ const mergeAndFilterDiagnostics = (mergedDiagnostics, directory, userConfig, rea
 	return filterInlineSuppressions(filtered, directory, readFileLinesSync);
 };
 const combineDiagnostics = (input) => {
-	const { lintDiagnostics, directory, isDiffMode, userConfig, readFileLinesSync = createNodeReadFileLinesSync(directory), includeEnvironmentChecks = true, respectInlineDisables } = input;
-	const extraDiagnostics = isDiffMode || !includeEnvironmentChecks ? [] : checkReducedMotion(directory);
-	return mergeAndFilterDiagnostics([...lintDiagnostics, ...extraDiagnostics], directory, userConfig, readFileLinesSync, { respectInlineDisables });
+	const { lintDiagnostics, directory, isDiffMode, userConfig, readFileLinesSync = createNodeReadFileLinesSync(directory), includeEnvironmentChecks = true, respectInlineDisables, extraDiagnostics = [] } = input;
+	const environmentDiagnostics = isDiffMode || !includeEnvironmentChecks ? [] : checkReducedMotion(directory);
+	return mergeAndFilterDiagnostics([
+		...lintDiagnostics,
+		...environmentDiagnostics,
+		...extraDiagnostics
+	], directory, userConfig, readFileLinesSync, { respectInlineDisables });
 };
 const findFirstLintConfigInDirectory = (directory) => {
 	for (const filename of ADOPTABLE_LINT_CONFIG_FILENAMES) {
@@ -4030,11 +4283,11 @@ const getUncommittedChangedFiles = (directory) => {
 const getDiffInfo = (directory, explicitBaseBranch) => {
 	if (explicitBaseBranch !== void 0 && explicitBaseBranch.trim().length === 0) throw new Error("Diff base branch cannot be empty.");
 	const currentBranch = getCurrentBranch(directory);
-	if (!currentBranch) return null;
+	if (!currentBranch && !explicitBaseBranch) return null;
 	const baseBranch = explicitBaseBranch ?? detectDefaultBranch(directory);
 	if (!baseBranch) return null;
 	if (explicitBaseBranch && !branchExists(directory, explicitBaseBranch)) throw new Error(`Diff base branch "${explicitBaseBranch}" does not exist (run \`git fetch\` to update remote refs).`);
-	if (currentBranch === baseBranch) {
+	if (currentBranch !== null && currentBranch === baseBranch) {
 		const uncommittedFiles = getUncommittedChangedFiles(directory);
 		if (uncommittedFiles.length === 0) return null;
 		return {
@@ -4061,6 +4314,7 @@ const VALID_RULE_SEVERITIES = [
 ];
 const BOOLEAN_FIELD_NAMES = [
 	"lint",
+	"deadCode",
 	"verbose",
 	"customRulesOnly",
 	"share",
@@ -4416,10 +4670,13 @@ const buildCapabilities = (project) => {
 	if (project.hasTypeScript) capabilities.add("typescript");
 	return capabilities;
 };
-const shouldEnableRule = (requires, tags, capabilities, ignoredTags) => {
+const shouldEnableRule = (requires, tags, capabilities, ignoredTags, disabledBy) => {
 	if (requires) {
 		for (const capability of requires) if (!capabilities.has(capability)) return false;
 	}
+	if (disabledBy) {
+		for (const capability of disabledBy) if (capabilities.has(capability)) return false;
+	}
 	if (tags) {
 		for (const tag of tags) if (ignoredTags.has(tag)) return false;
 	}
@@ -4452,23 +4709,6 @@ const resolveReactHooksJsPlugin = (hasReactCompiler, customRulesOnly) => {
 		availableRuleNames: readPluginRuleNames(pluginSpecifier)
 	};
 };
-const YOU_MIGHT_NOT_NEED_EFFECT_NAMESPACE = "effect";
-const resolveYouMightNotNeedEffectPlugin = (customRulesOnly) => {
-	if (customRulesOnly) return null;
-	let pluginSpecifier;
-	try {
-		pluginSpecifier = esmRequire$1.resolve("eslint-plugin-react-you-might-not-need-an-effect");
-	} catch {
-		return null;
-	}
-	return {
-		entry: {
-			name: YOU_MIGHT_NOT_NEED_EFFECT_NAMESPACE,
-			specifier: pluginSpecifier
-		},
-		availableRuleNames: readPluginRuleNames(pluginSpecifier)
-	};
-};
 const filterRulesToAvailable = (rules, pluginNamespace, availableRuleNames) => {
 	if (availableRuleNames.size === 0) return rules;
 	const ruleKeyPrefix = `${pluginNamespace}/`;
@@ -4487,26 +4727,36 @@ const resolveSettingsRootDirectory = (rootDirectory) => {
 	if (!fs.existsSync(rootDirectory)) return rootDirectory;
 	return fs.realpathSync(rootDirectory);
 };
+const applyRuleSeverityControls = (rules, severityControls) => {
+	const enabledRules = {};
+	for (const [ruleKey, defaultSeverity] of Object.entries(rules)) {
+		const severity = resolveRuleSeverityOverride({ ruleKey }, severityControls) ?? defaultSeverity;
+		if (severity === "off") continue;
+		enabledRules[ruleKey] = severity;
+	}
+	return enabledRules;
+};
 const createOxlintConfig = ({ pluginPath, project, customRulesOnly = false, extendsPaths = [], ignoredTags = /* @__PURE__ */ new Set(), serverAuthFunctionNames, severityControls }) => {
 	const reactHooksJsPlugin = resolveReactHooksJsPlugin(project.hasReactCompiler, customRulesOnly);
-	const reactCompilerRules = reactHooksJsPlugin ? filterRulesToAvailable(REACT_COMPILER_RULES, "react-hooks-js", reactHooksJsPlugin.availableRuleNames) : {};
-	const youMightNotNeedEffectPlugin = resolveYouMightNotNeedEffectPlugin(customRulesOnly);
-	const youMightNotNeedEffectRules = youMightNotNeedEffectPlugin ? filterRulesToAvailable(YOU_MIGHT_NOT_NEED_EFFECT_RULES, YOU_MIGHT_NOT_NEED_EFFECT_NAMESPACE, youMightNotNeedEffectPlugin.availableRuleNames) : {};
+	const reactCompilerRules = reactHooksJsPlugin ? applyRuleSeverityControls(filterRulesToAvailable(REACT_COMPILER_RULES, "react-hooks-js", reactHooksJsPlugin.availableRuleNames), severityControls) : {};
 	const jsPlugins = [];
 	if (reactHooksJsPlugin) jsPlugins.push(reactHooksJsPlugin.entry);
-	if (youMightNotNeedEffectPlugin) jsPlugins.push(youMightNotNeedEffectPlugin.entry);
 	const capabilities = buildCapabilities(project);
 	const enabledReactDoctorRules = {};
-	for (const [ruleId, rule] of Object.entries(reactDoctorPlugin.rules)) {
-		const fullKey = `react-doctor/${ruleId}`;
+	for (const registryEntry of REACT_DOCTOR_RULES) {
+		const rule = reactDoctorPlugin.rules[registryEntry.id];
+		if (!rule) continue;
+		if (customRulesOnly && registryEntry.originallyExternal) continue;
 		if (rule.framework !== "global" && !rule.requires) continue;
-		if (!shouldEnableRule(rule.requires, rule.tags, capabilities, ignoredTags)) continue;
-		const severity = resolveRuleSeverityOverride({
-			ruleKey: fullKey,
+		if (!shouldEnableRule(rule.requires, rule.tags, capabilities, ignoredTags, rule.disabledBy)) continue;
+		const explicitSeverity = resolveRuleSeverityOverride({
+			ruleKey: registryEntry.key,
 			category: rule.category
-		}, severityControls) ?? rule.severity;
+		}, severityControls);
+		if (rule.defaultEnabled === false && explicitSeverity === void 0) continue;
+		const severity = explicitSeverity ?? rule.severity;
 		if (severity === "off") continue;
-		enabledReactDoctorRules[fullKey] = severity;
+		enabledReactDoctorRules[registryEntry.key] = severity;
 	}
 	return {
 		...extendsPaths.length > 0 ? { extends: extendsPaths } : {},
@@ -4519,7 +4769,7 @@ const createOxlintConfig = ({ pluginPath, project, customRulesOnly = false, exte
 			style: "off",
 			nursery: "off"
 		},
-		plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
+		plugins: [],
 		jsPlugins: [...jsPlugins, pluginPath],
 		settings: { "react-doctor": {
 			framework: project.framework,
@@ -4527,10 +4777,7 @@ const createOxlintConfig = ({ pluginPath, project, customRulesOnly = false, exte
 			...serverAuthFunctionNames && serverAuthFunctionNames.length > 0 ? { serverAuthFunctionNames: [...serverAuthFunctionNames] } : {}
 		} },
 		rules: {
-			...customRulesOnly ? {} : BUILTIN_REACT_RULES,
-			...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
 			...reactCompilerRules,
-			...youMightNotNeedEffectRules,
 			...enabledReactDoctorRules
 		}
 	};
@@ -4908,7 +5155,13 @@ const SANITIZED_ENV = (() => {
 	}
 	return sanitized;
 })();
-const OXLINT_SPAWN_TIMEOUT_MS = 6e4;
+const OXLINT_SPAWN_TIMEOUT_MS = (() => {
+	const raw = process.env["REACT_DOCTOR_OXLINT_SPAWN_TIMEOUT_MS"];
+	if (raw === void 0) return 6e4;
+	const parsed = Number(raw);
+	if (!Number.isFinite(parsed) || parsed <= 0) return 6e4;
+	return parsed;
+})();
 const spawnOxlint = (args, rootDirectory, nodeBinaryPath) => new Promise((resolve, reject) => {
 	const child = spawn(nodeBinaryPath, args, {
 		cwd: rootDirectory,
@@ -5088,6 +5341,7 @@ const runOxlint = async (options) => {
 		const spawnLintBatches = async () => {
 			const allDiagnostics = [];
 			const droppedFiles = [];
+			let firstDropReason = null;
 			const spawnLintBatch = async (batch) => {
 				const batchArgs = [...baseArgs, ...batch];
 				try {
@@ -5096,6 +5350,7 @@ const runOxlint = async (options) => {
 					if (!isSplittableOxlintBatchError(error)) throw error;
 					if (batch.length <= 1) {
 						droppedFiles.push(...batch);
+						if (firstDropReason === null && error instanceof Error) firstDropReason = error.message;
 						return [];
 					}
 					const splitIndex = Math.ceil(batch.length / 2);
@@ -5107,7 +5362,8 @@ const runOxlint = async (options) => {
 				const previewCount = 3;
 				const previewFiles = droppedFiles.slice(0, previewCount).join(", ");
 				const remainderHint = droppedFiles.length > previewCount ? `, +${droppedFiles.length - previewCount} more` : "";
-				onPartialFailure(`${droppedFiles.length} file(s) exceeded the ${OXLINT_SPAWN_TIMEOUT_MS / 1e3}s per-batch oxlint budget and were skipped (${previewFiles}${remainderHint})`);
+				const reasonHint = firstDropReason ? ` — first failure: ${firstDropReason}` : "";
+				onPartialFailure(`${droppedFiles.length} file(s) failed to lint and were skipped (${previewFiles}${remainderHint})${reasonHint}`);
 			}
 			return dedupeDiagnostics(allDiagnostics);
 		};
@@ -5154,7 +5410,8 @@ const toJsonReport = (result, options) => buildJsonReport({
 		result: {
 			diagnostics: result.diagnostics,
 			score: result.score,
-			skippedChecks: [],
+			skippedChecks: result.skippedChecks,
+			...result.skippedCheckReasons ? { skippedCheckReasons: result.skippedCheckReasons } : {},
 			project: result.project,
 			elapsedMilliseconds: result.elapsedMilliseconds
 		}
@@ -5177,32 +5434,54 @@ const diagnose = async (directory, options = {}) => {
 	const lintIncludePaths = computeJsxIncludePaths(includePaths) ?? resolveLintIncludePaths(resolvedDirectory, userConfig);
 	const readFileLinesSync = createNodeReadFileLinesSync(resolvedDirectory);
 	const effectiveLint = options.lint ?? userConfig?.lint ?? true;
+	const effectiveDeadCode = options.deadCode ?? userConfig?.deadCode ?? true;
 	const effectiveRespectInlineDisables = options.respectInlineDisables ?? userConfig?.respectInlineDisables ?? true;
 	const ignoredTags = new Set(userConfig?.ignore?.tags ?? []);
+	const lintPromise = effectiveLint ? runOxlint({
+		rootDirectory: resolvedDirectory,
+		project: projectInfo,
+		includePaths: lintIncludePaths,
+		customRulesOnly: userConfig?.customRulesOnly ?? false,
+		respectInlineDisables: effectiveRespectInlineDisables,
+		adoptExistingLintConfig: userConfig?.adoptExistingLintConfig ?? true,
+		ignoredTags,
+		userConfig
+	}).catch((error) => {
+		console.error("Lint failed:", error);
+		return EMPTY_DIAGNOSTICS;
+	}) : Promise.resolve(EMPTY_DIAGNOSTICS);
+	const shouldRunDeadCode = effectiveDeadCode && !isDiffMode;
+	let deadCodeFailureReason = null;
+	const deadCodePromise = shouldRunDeadCode ? checkDeadCode({
+		rootDirectory: resolvedDirectory,
+		userConfig
+	}).catch((error) => {
+		deadCodeFailureReason = error instanceof Error ? error.message : String(error);
+		return EMPTY_DIAGNOSTICS;
+	}) : Promise.resolve(EMPTY_DIAGNOSTICS);
+	const [lintDiagnostics, deadCodeDiagnostics] = await Promise.all([lintPromise, deadCodePromise]);
 	const diagnostics = combineDiagnostics({
-		lintDiagnostics: effectiveLint ? await runOxlint({
-			rootDirectory: resolvedDirectory,
-			project: projectInfo,
-			includePaths: lintIncludePaths,
-			customRulesOnly: userConfig?.customRulesOnly ?? false,
-			respectInlineDisables: effectiveRespectInlineDisables,
-			adoptExistingLintConfig: userConfig?.adoptExistingLintConfig ?? true,
-			ignoredTags,
-			userConfig
-		}).catch((error) => {
-			console.error("Lint failed:", error);
-			return EMPTY_DIAGNOSTICS;
-		}) : EMPTY_DIAGNOSTICS,
+		lintDiagnostics,
 		directory: resolvedDirectory,
 		isDiffMode,
 		userConfig,
 		readFileLinesSync,
-		respectInlineDisables: effectiveRespectInlineDisables
+		respectInlineDisables: effectiveRespectInlineDisables,
+		extraDiagnostics: deadCodeDiagnostics
 	});
 	const elapsedMilliseconds = globalThis.performance.now() - startTime;
+	const score = await calculateScore(diagnostics);
+	const skippedChecks = [];
+	const skippedCheckReasons = {};
+	if (deadCodeFailureReason !== null) {
+		skippedChecks.push("dead-code");
+		skippedCheckReasons["dead-code"] = deadCodeFailureReason;
+	}
 	return {
 		diagnostics,
-		score: await calculateScore(diagnostics),
+		score,
+		skippedChecks,
+		...Object.keys(skippedCheckReasons).length > 0 ? { skippedCheckReasons } : {},
 		project: projectInfo,
 		elapsedMilliseconds
 	};