react-doctor 0.0.28 → 0.0.30

package/dist/index.d.ts

@@ -1,5 +1,6 @@
 //#region src/types.d.ts
-type Framework = "nextjs" | "vite" | "cra" | "remix" | "gatsby" | "unknown";
+type FailOnLevel = "error" | "warning" | "none";
+type Framework = "nextjs" | "vite" | "cra" | "remix" | "gatsby" | "expo" | "react-native" | "unknown";
 interface ProjectInfo {
   rootDirectory: string;
   projectName: string;
@@ -41,6 +42,7 @@ interface ReactDoctorConfig {
   deadCode?: boolean;
   verbose?: boolean;
   diff?: boolean | string;
+  failOn?: FailOnLevel;
 }
 //#endregion
 //#region src/utils/get-diff-files.d.ts

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","names":[],"sources":["../src/types.ts","../src/utils/get-diff-files.ts","../src/index.ts"],"mappings":";KAAY,SAAA;AAAA,UAEK,WAAA;EACf,aAAA;EACA,WAAA;EACA,YAAA;EACA,SAAA,EAAW,SAAA;EACX,aAAA;EACA,gBAAA;EACA,eAAA;AAAA;AAAA,UAiCe,UAAA;EACf,QAAA;EACA,MAAA;EACA,IAAA;EACA,QAAA;EACA,OAAA;EACA,IAAA;EACA,IAAA;EACA,MAAA;EACA,QAAA;EACA,MAAA;AAAA;AAAA,UA4Be,WAAA;EACf,KAAA;EACA,KAAA;AAAA;AAAA,UAyBe,QAAA;EACf,aAAA;EACA,UAAA;EACA,YAAA;EACA,gBAAA;AAAA;AAAA,UA2Ce,uBAAA;EACf,KAAA;EACA,KAAA;AAAA;AAAA,UAGe,iBAAA;EACf,MAAA,GAAS,uBAAA;EACT,IAAA;EACA,QAAA;EACA,OAAA;EACA,IAAA;AAAA;;;cCrFW,WAAA,GAAe,SAAA,UAAmB,kBAAA,cAA8B,QAAA;AAAA,cAiBhE,iBAAA,GAAqB,SAAA;;;UCnFjB,eAAA;EACf,IAAA;EACA,QAAA;EACA,YAAA;AAAA;AAAA,UAGe,cAAA;EACf,WAAA,EAAa,UAAA;EACb,KAAA,EAAO,WAAA;EACP,OAAA,EAAS,WAAA;EACT,mBAAA;AAAA;AAAA,cAGW,QAAA,GACX,SAAA,UACA,OAAA,GAAS,eAAA,KACR,OAAA,CAAQ,cAAA"}
+{"version":3,"file":"index.d.ts","names":[],"sources":["../src/types.ts","../src/utils/get-diff-files.ts","../src/index.ts"],"mappings":";KAAY,WAAA;AAAA,KAEA,SAAA;AAAA,UAUK,WAAA;EACf,aAAA;EACA,WAAA;EACA,YAAA;EACA,SAAA,EAAW,SAAA;EACX,aAAA;EACA,gBAAA;EACA,eAAA;AAAA;AAAA,UAiCe,UAAA;EACf,QAAA;EACA,MAAA;EACA,IAAA;EACA,QAAA;EACA,OAAA;EACA,IAAA;EACA,IAAA;EACA,MAAA;EACA,QAAA;EACA,MAAA;AAAA;AAAA,UA4Be,WAAA;EACf,KAAA;EACA,KAAA;AAAA;AAAA,UAyBe,QAAA;EACf,aAAA;EACA,UAAA;EACA,YAAA;EACA,gBAAA;AAAA;AAAA,UA2Ce,uBAAA;EACf,KAAA;EACA,KAAA;AAAA;AAAA,UAGe,iBAAA;EACf,MAAA,GAAS,uBAAA;EACT,IAAA;EACA,QAAA;EACA,OAAA;EACA,IAAA;EACA,MAAA,GAAS,WAAA;AAAA;;;cChGE,WAAA,GAAe,SAAA,UAAmB,kBAAA,cAA8B,QAAA;AAAA,cAiBhE,iBAAA,GAAqB,SAAA;;;UCnFjB,eAAA;EACf,IAAA;EACA,QAAA;EACA,YAAA;AAAA;AAAA,UAGe,cAAA;EACf,WAAA,EAAa,UAAA;EACb,KAAA,EAAO,WAAA;EACP,OAAA,EAAS,WAAA;EACT,mBAAA;AAAA;AAAA,cAGW,QAAA,GACX,SAAA,UACA,OAAA,GAAS,eAAA,KACR,OAAA,CAAQ,cAAA"}

package/dist/index.js

@@ -12,11 +12,18 @@ import { fileURLToPath } from "node:url";
 const SOURCE_FILE_PATTERN = /\.(tsx?|jsx?)$/;
 const JSX_FILE_PATTERN = /\.(tsx|jsx)$/;
 const ERROR_PREVIEW_LENGTH_CHARS = 200;
+const PERFECT_SCORE = 100;
+const SCORE_GOOD_THRESHOLD = 75;
+const SCORE_OK_THRESHOLD = 50;
 const SCORE_API_URL = "https://www.react.doctor/api/score";
 const FETCH_TIMEOUT_MS = 1e4;
 const GIT_LS_FILES_MAX_BUFFER_BYTES = 50 * 1024 * 1024;
 const SPAWN_ARGS_MAX_LENGTH_CHARS = 24e3;
 const DEFAULT_BRANCH_CANDIDATES = ["main", "master"];
+const ERROR_RULE_PENALTY = 1.5;
+const WARNING_RULE_PENALTY = .75;
+const ERROR_ESTIMATED_FIX_RATE = .85;
+const WARNING_ESTIMATED_FIX_RATE = .8;
 const MAX_KNIP_RETRIES = 5;
 const AMI_WEBSITE_URL = "https://ami.dev";
 const AMI_INSTALL_URL = `${AMI_WEBSITE_URL}/install.sh`;
@@ -71,6 +78,46 @@ const proxyFetch = async (url, init) => {
 
 //#endregion
 //#region src/utils/calculate-score.ts
+const getScoreLabel = (score) => {
+	if (score >= SCORE_GOOD_THRESHOLD) return "Great";
+	if (score >= SCORE_OK_THRESHOLD) return "Needs work";
+	return "Critical";
+};
+const countUniqueRules = (diagnostics) => {
+	const errorRules = /* @__PURE__ */ new Set();
+	const warningRules = /* @__PURE__ */ new Set();
+	for (const diagnostic of diagnostics) {
+		const ruleKey = `${diagnostic.plugin}/${diagnostic.rule}`;
+		if (diagnostic.severity === "error") errorRules.add(ruleKey);
+		else warningRules.add(ruleKey);
+	}
+	return {
+		errorRuleCount: errorRules.size,
+		warningRuleCount: warningRules.size
+	};
+};
+const scoreFromRuleCounts = (errorRuleCount, warningRuleCount) => {
+	const penalty = errorRuleCount * ERROR_RULE_PENALTY + warningRuleCount * WARNING_RULE_PENALTY;
+	return Math.max(0, Math.round(PERFECT_SCORE - penalty));
+};
+const estimateScoreLocally = (diagnostics) => {
+	const { errorRuleCount, warningRuleCount } = countUniqueRules(diagnostics);
+	const currentScore = scoreFromRuleCounts(errorRuleCount, warningRuleCount);
+	const estimatedScore = scoreFromRuleCounts(Math.round(errorRuleCount * (1 - ERROR_ESTIMATED_FIX_RATE)), Math.round(warningRuleCount * (1 - WARNING_ESTIMATED_FIX_RATE)));
+	return {
+		currentScore,
+		currentLabel: getScoreLabel(currentScore),
+		estimatedScore,
+		estimatedLabel: getScoreLabel(estimatedScore)
+	};
+};
+const calculateScoreLocally = (diagnostics) => {
+	const { currentScore, currentLabel } = estimateScoreLocally(diagnostics);
+	return {
+		score: currentScore,
+		label: currentLabel
+	};
+};
 const calculateScore = async (diagnostics) => {
 	try {
 		const response = await proxyFetch(SCORE_API_URL, {
@@ -78,10 +125,10 @@ const calculateScore = async (diagnostics) => {
 			headers: { "Content-Type": "application/json" },
 			body: JSON.stringify({ diagnostics })
 		});
-		if (!response.ok) return null;
+		if (!response.ok) return calculateScoreLocally(diagnostics);
 		return await response.json();
 	} catch {
-		return null;
+		return calculateScoreLocally(diagnostics);
 	}
 };
 
@@ -89,13 +136,33 @@ const calculateScore = async (diagnostics) => {
 //#region src/plugin/constants.ts
 const MOTION_LIBRARY_PACKAGES = new Set(["framer-motion", "motion"]);
 
+//#endregion
+//#region src/utils/is-file.ts
+const isFile = (filePath) => {
+	try {
+		return fs.statSync(filePath).isFile();
+	} catch {
+		return false;
+	}
+};
+
 //#endregion
 //#region src/utils/read-package-json.ts
-const readPackageJson = (packageJsonPath) => JSON.parse(fs.readFileSync(packageJsonPath, "utf-8"));
+const readPackageJson = (packageJsonPath) => {
+	try {
+		return JSON.parse(fs.readFileSync(packageJsonPath, "utf-8"));
+	} catch (error) {
+		if (error instanceof Error && "code" in error) {
+			const { code } = error;
+			if (code === "EISDIR" || code === "EACCES") return {};
+		}
+		throw error;
+	}
+};
 
 //#endregion
 //#region src/utils/check-reduced-motion.ts
-const REDUCED_MOTION_GREP_PATTERN = "prefers-reduced-motion|useReducedMotion";
+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",
@@ -111,7 +178,7 @@ const MISSING_REDUCED_MOTION_DIAGNOSTIC = {
 };
 const checkReducedMotion = (rootDirectory) => {
 	const packageJsonPath = path.join(rootDirectory, "package.json");
-	if (!fs.existsSync(packageJsonPath)) return [];
+	if (!isFile(packageJsonPath)) return [];
 	let hasMotionLibrary = false;
 	try {
 		const packageJson = readPackageJson(packageJsonPath);
@@ -139,7 +206,7 @@ const checkReducedMotion = (rootDirectory) => {
 //#region src/utils/match-glob-pattern.ts
 const REGEX_SPECIAL_CHARACTERS = /[.+^${}()|[\]\\]/g;
 const compileGlobPattern = (pattern) => {
-	const normalizedPattern = pattern.replace(/\\/g, "/");
+	const normalizedPattern = pattern.replace(/\\/g, "/").replace(/^\//, "");
 	let regexSource = "^";
 	let characterIndex = 0;
 	while (characterIndex < normalizedPattern.length) if (normalizedPattern[characterIndex] === "*" && normalizedPattern[characterIndex + 1] === "*") if (normalizedPattern[characterIndex + 2] === "/") {
@@ -177,25 +244,67 @@ const filterIgnoredDiagnostics = (diagnostics, config) => {
 		return true;
 	});
 };
+const DISABLE_NEXT_LINE_PATTERN = /\/\/\s*react-doctor-disable-next-line\b(?:\s+(.+))?/;
+const DISABLE_LINE_PATTERN = /\/\/\s*react-doctor-disable-line\b(?:\s+(.+))?/;
+const isRuleSuppressed = (commentRules, ruleId) => {
+	if (!commentRules?.trim()) return true;
+	return commentRules.split(/[,\s]+/).some((rule) => rule.trim() === ruleId);
+};
+const filterInlineSuppressions = (diagnostics, rootDirectory) => {
+	const fileLineCache = /* @__PURE__ */ new Map();
+	const getFileLines = (filePath) => {
+		const cached = fileLineCache.get(filePath);
+		if (cached !== void 0) return cached;
+		const absolutePath = path.isAbsolute(filePath) ? filePath : path.join(rootDirectory, filePath);
+		try {
+			const lines = fs.readFileSync(absolutePath, "utf-8").split("\n");
+			fileLineCache.set(filePath, lines);
+			return lines;
+		} catch {
+			fileLineCache.set(filePath, null);
+			return null;
+		}
+	};
+	return diagnostics.filter((diagnostic) => {
+		if (diagnostic.line <= 0) return true;
+		const lines = getFileLines(diagnostic.filePath);
+		if (!lines) return true;
+		const ruleId = `${diagnostic.plugin}/${diagnostic.rule}`;
+		const currentLine = lines[diagnostic.line - 1];
+		if (currentLine) {
+			const lineMatch = currentLine.match(DISABLE_LINE_PATTERN);
+			if (lineMatch && isRuleSuppressed(lineMatch[1], ruleId)) return false;
+		}
+		if (diagnostic.line >= 2) {
+			const prevLine = lines[diagnostic.line - 2];
+			if (prevLine) {
+				const nextLineMatch = prevLine.match(DISABLE_NEXT_LINE_PATTERN);
+				if (nextLineMatch && isRuleSuppressed(nextLineMatch[1], ruleId)) return false;
+			}
+		}
+		return true;
+	});
+};
 
 //#endregion
 //#region src/utils/combine-diagnostics.ts
 const computeJsxIncludePaths = (includePaths) => includePaths.length > 0 ? includePaths.filter((filePath) => JSX_FILE_PATTERN.test(filePath)) : void 0;
 const combineDiagnostics = (lintDiagnostics, deadCodeDiagnostics, directory, isDiffMode, userConfig) => {
-	const allDiagnostics = [
+	const merged = [
 		...lintDiagnostics,
 		...deadCodeDiagnostics,
 		...isDiffMode ? [] : checkReducedMotion(directory)
 	];
-	return userConfig ? filterIgnoredDiagnostics(allDiagnostics, userConfig) : allDiagnostics;
+	return filterInlineSuppressions(userConfig ? filterIgnoredDiagnostics(merged, userConfig) : merged, directory);
 };
 
 //#endregion
 //#region src/utils/find-monorepo-root.ts
 const isMonorepoRoot = (directory) => {
-	if (fs.existsSync(path.join(directory, "pnpm-workspace.yaml"))) return true;
+	if (isFile(path.join(directory, "pnpm-workspace.yaml"))) return true;
+	if (isFile(path.join(directory, "nx.json"))) return true;
 	const packageJsonPath = path.join(directory, "package.json");
-	if (!fs.existsSync(packageJsonPath)) return false;
+	if (!isFile(packageJsonPath)) return false;
 	const packageJson = readPackageJson(packageJsonPath);
 	return Array.isArray(packageJson.workspaces) || Boolean(packageJson.workspaces?.packages);
 };
@@ -208,6 +317,10 @@ const findMonorepoRoot = (startDirectory) => {
 	return null;
 };
 
+//#endregion
+//#region src/utils/is-plain-object.ts
+const isPlainObject = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
+
 //#endregion
 //#region src/utils/discover-project.ts
 const REACT_COMPILER_PACKAGES = new Set([
@@ -246,9 +359,33 @@ const FRAMEWORK_PACKAGES = {
 	vite: "vite",
 	"react-scripts": "cra",
 	"@remix-run/react": "remix",
-	gatsby: "gatsby"
+	gatsby: "gatsby",
+	expo: "expo",
+	"react-native": "react-native"
+};
+const IGNORED_DIRECTORIES = new Set([
+	"node_modules",
+	"dist",
+	"build",
+	"coverage"
+]);
+const countSourceFilesViaFilesystem = (rootDirectory) => {
+	let count = 0;
+	const stack = [rootDirectory];
+	while (stack.length > 0) {
+		const currentDirectory = stack.pop();
+		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true });
+		for (const entry of entries) {
+			if (entry.isDirectory()) {
+				if (!entry.name.startsWith(".") && !IGNORED_DIRECTORIES.has(entry.name)) stack.push(path.join(currentDirectory, entry.name));
+				continue;
+			}
+			if (entry.isFile() && SOURCE_FILE_PATTERN.test(entry.name)) count++;
+		}
+	}
+	return count;
 };
-const countSourceFiles = (rootDirectory) => {
+const countSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
 		"--cached",
@@ -259,9 +396,10 @@ const countSourceFiles = (rootDirectory) => {
 		encoding: "utf-8",
 		maxBuffer: GIT_LS_FILES_MAX_BUFFER_BYTES
 	});
-	if (result.error || result.status !== 0) return 0;
+	if (result.error || result.status !== 0) return null;
 	return result.stdout.split("\n").filter((filePath) => filePath.length > 0 && SOURCE_FILE_PATTERN.test(filePath)).length;
 };
+const countSourceFiles = (rootDirectory) => countSourceFilesViaGit(rootDirectory) ?? countSourceFilesViaFilesystem(rootDirectory);
 const collectAllDependencies = (packageJson) => ({
 	...packageJson.peerDependencies,
 	...packageJson.dependencies,
@@ -271,16 +409,37 @@ const detectFramework = (dependencies) => {
 	for (const [packageName, frameworkName] of Object.entries(FRAMEWORK_PACKAGES)) if (dependencies[packageName]) return frameworkName;
 	return "unknown";
 };
+const isCatalogReference = (version) => version.startsWith("catalog:");
+const resolveVersionFromCatalog = (catalog, packageName) => {
+	const version = catalog[packageName];
+	if (typeof version === "string" && !isCatalogReference(version)) return version;
+	return null;
+};
+const resolveCatalogVersion = (packageJson, packageName) => {
+	const raw = packageJson;
+	if (isPlainObject(raw.catalog)) {
+		const version = resolveVersionFromCatalog(raw.catalog, packageName);
+		if (version) return version;
+	}
+	if (isPlainObject(raw.catalogs)) {
+		for (const catalogEntries of Object.values(raw.catalogs)) if (isPlainObject(catalogEntries)) {
+			const version = resolveVersionFromCatalog(catalogEntries, packageName);
+			if (version) return version;
+		}
+	}
+	return null;
+};
 const extractDependencyInfo = (packageJson) => {
 	const allDependencies = collectAllDependencies(packageJson);
+	const rawVersion = allDependencies.react ?? null;
 	return {
-		reactVersion: allDependencies.react ?? null,
+		reactVersion: rawVersion && !isCatalogReference(rawVersion) ? rawVersion : null,
 		framework: detectFramework(allDependencies)
 	};
 };
 const parsePnpmWorkspacePatterns = (rootDirectory) => {
 	const workspacePath = path.join(rootDirectory, "pnpm-workspace.yaml");
-	if (!fs.existsSync(workspacePath)) return [];
+	if (!isFile(workspacePath)) return [];
 	const content = fs.readFileSync(workspacePath, "utf-8");
 	const patterns = [];
 	let isInsidePackagesBlock = false;
@@ -306,14 +465,14 @@ const resolveWorkspaceDirectories = (rootDirectory, pattern) => {
 	const cleanPattern = pattern.replace(/["']/g, "").replace(/\/\*\*$/, "/*");
 	if (!cleanPattern.includes("*")) {
 		const directoryPath = path.join(rootDirectory, cleanPattern);
-		if (fs.existsSync(directoryPath) && fs.existsSync(path.join(directoryPath, "package.json"))) return [directoryPath];
+		if (fs.existsSync(directoryPath) && isFile(path.join(directoryPath, "package.json"))) return [directoryPath];
 		return [];
 	}
 	const wildcardIndex = cleanPattern.indexOf("*");
 	const baseDirectory = path.join(rootDirectory, cleanPattern.slice(0, wildcardIndex));
 	const suffixAfterWildcard = cleanPattern.slice(wildcardIndex + 1);
 	if (!fs.existsSync(baseDirectory) || !fs.statSync(baseDirectory).isDirectory()) return [];
-	return fs.readdirSync(baseDirectory).map((entry) => path.join(baseDirectory, entry, suffixAfterWildcard)).filter((entryPath) => fs.existsSync(entryPath) && fs.statSync(entryPath).isDirectory() && fs.existsSync(path.join(entryPath, "package.json")));
+	return fs.readdirSync(baseDirectory).map((entry) => path.join(baseDirectory, entry, suffixAfterWildcard)).filter((entryPath) => fs.existsSync(entryPath) && fs.statSync(entryPath).isDirectory() && isFile(path.join(entryPath, "package.json")));
 };
 const findDependencyInfoFromMonorepoRoot = (directory) => {
 	const monorepoRoot = findMonorepoRoot(directory);
@@ -321,11 +480,17 @@ const findDependencyInfoFromMonorepoRoot = (directory) => {
 		reactVersion: null,
 		framework: "unknown"
 	};
-	const rootPackageJson = readPackageJson(path.join(monorepoRoot, "package.json"));
+	const monorepoPackageJsonPath = path.join(monorepoRoot, "package.json");
+	if (!isFile(monorepoPackageJsonPath)) return {
+		reactVersion: null,
+		framework: "unknown"
+	};
+	const rootPackageJson = readPackageJson(monorepoPackageJsonPath);
 	const rootInfo = extractDependencyInfo(rootPackageJson);
+	const catalogVersion = resolveCatalogVersion(rootPackageJson, "react");
 	const workspaceInfo = findReactInWorkspaces(monorepoRoot, rootPackageJson);
 	return {
-		reactVersion: rootInfo.reactVersion ?? workspaceInfo.reactVersion,
+		reactVersion: rootInfo.reactVersion ?? catalogVersion ?? workspaceInfo.reactVersion,
 		framework: rootInfo.framework !== "unknown" ? rootInfo.framework : workspaceInfo.framework
 	};
 };
@@ -351,7 +516,7 @@ const hasCompilerPackage = (packageJson) => {
 	return Object.keys(allDependencies).some((packageName) => REACT_COMPILER_PACKAGES.has(packageName));
 };
 const fileContainsPattern = (filePath, pattern) => {
-	if (!fs.existsSync(filePath)) return false;
+	if (!isFile(filePath)) return false;
 	const content = fs.readFileSync(filePath, "utf-8");
 	return pattern.test(content);
 };
@@ -365,7 +530,7 @@ const detectReactCompiler = (directory, packageJson) => {
 	let ancestorDirectory = path.dirname(directory);
 	while (ancestorDirectory !== path.dirname(ancestorDirectory)) {
 		const ancestorPackagePath = path.join(ancestorDirectory, "package.json");
-		if (fs.existsSync(ancestorPackagePath)) {
+		if (isFile(ancestorPackagePath)) {
 			if (hasCompilerPackage(readPackageJson(ancestorPackagePath))) return true;
 		}
 		ancestorDirectory = path.dirname(ancestorDirectory);
@@ -374,9 +539,10 @@ const detectReactCompiler = (directory, packageJson) => {
 };
 const discoverProject = (directory) => {
 	const packageJsonPath = path.join(directory, "package.json");
-	if (!fs.existsSync(packageJsonPath)) throw new Error(`No package.json found in ${directory}`);
+	if (!isFile(packageJsonPath)) throw new Error(`No package.json found in ${directory}`);
 	const packageJson = readPackageJson(packageJsonPath);
 	let { reactVersion, framework } = extractDependencyInfo(packageJson);
+	if (!reactVersion) reactVersion = resolveCatalogVersion(packageJson, "react");
 	if (!reactVersion || framework === "unknown") {
 		const workspaceInfo = findReactInWorkspaces(directory, packageJson);
 		if (!reactVersion && workspaceInfo.reactVersion) reactVersion = workspaceInfo.reactVersion;
@@ -406,23 +572,18 @@ const discoverProject = (directory) => {
 //#region src/utils/load-config.ts
 const CONFIG_FILENAME = "react-doctor.config.json";
 const PACKAGE_JSON_CONFIG_KEY = "reactDoctor";
-const isPlainObject = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 const loadConfig = (rootDirectory) => {
 	const configFilePath = path.join(rootDirectory, CONFIG_FILENAME);
-	if (fs.existsSync(configFilePath)) try {
+	if (isFile(configFilePath)) try {
 		const fileContent = fs.readFileSync(configFilePath, "utf-8");
 		const parsed = JSON.parse(fileContent);
-		if (!isPlainObject(parsed)) {
-			console.warn(`Warning: ${CONFIG_FILENAME} must be a JSON object, ignoring.`);
-			return null;
-		}
-		return parsed;
+		if (isPlainObject(parsed)) return parsed;
+		console.warn(`Warning: ${CONFIG_FILENAME} must be a JSON object, ignoring.`);
 	} catch (error) {
 		console.warn(`Warning: Failed to parse ${CONFIG_FILENAME}: ${error instanceof Error ? error.message : String(error)}`);
-		return null;
 	}
 	const packageJsonPath = path.join(rootDirectory, "package.json");
-	if (fs.existsSync(packageJsonPath)) try {
+	if (isFile(packageJsonPath)) try {
 		const fileContent = fs.readFileSync(packageJsonPath, "utf-8");
 		const embeddedConfig = JSON.parse(fileContent)[PACKAGE_JSON_CONFIG_KEY];
 		if (isPlainObject(embeddedConfig)) return embeddedConfig;
@@ -490,11 +651,15 @@ const CONFIG_LOADING_ERROR_PATTERN = /Error loading .*\/([a-z-]+)\.config\./;
 const extractFailedPluginName = (error) => {
 	return String(error).match(CONFIG_LOADING_ERROR_PATTERN)?.[1] ?? null;
 };
+const TSCONFIG_FILENAMES = ["tsconfig.base.json", "tsconfig.json"];
+const resolveTsConfigFile = (directory) => TSCONFIG_FILENAMES.find((filename) => fs.existsSync(path.join(directory, filename)));
 const runKnipWithOptions = async (knipCwd, workspaceName) => {
+	const tsConfigFile = resolveTsConfigFile(knipCwd);
 	const options = await silenced(() => createOptions({
 		cwd: knipCwd,
 		isShowProgress: false,
-		...workspaceName ? { workspace: workspaceName } : {}
+		...workspaceName ? { workspace: workspaceName } : {},
+		...tsConfigFile ? { tsConfigFile } : {}
 	}));
 	const parsedConfig = options.parsedConfig;
 	for (let attempt = 0; attempt <= MAX_KNIP_RETRIES; attempt++) try {
@@ -516,7 +681,7 @@ const runKnip = async (rootDirectory) => {
 	let knipResult;
 	if (monorepoRoot) {
 		const packageJsonPath = path.join(rootDirectory, "package.json");
-		const workspaceName = (fs.existsSync(packageJsonPath) ? JSON.parse(fs.readFileSync(packageJsonPath, "utf-8")) : {}).name ?? path.basename(rootDirectory);
+		const workspaceName = (isFile(packageJsonPath) ? JSON.parse(fs.readFileSync(packageJsonPath, "utf-8")) : {}).name ?? path.basename(rootDirectory);
 		try {
 			knipResult = await runKnipWithOptions(monorepoRoot, workspaceName);
 		} catch {
@@ -566,6 +731,16 @@ const NEXTJS_RULES = {
 	"react-doctor/nextjs-no-head-import": "error",
 	"react-doctor/nextjs-no-side-effect-in-get-handler": "error"
 };
+const REACT_NATIVE_RULES = {
+	"react-doctor/rn-no-raw-text": "error",
+	"react-doctor/rn-no-deprecated-modules": "error",
+	"react-doctor/rn-no-legacy-expo-packages": "warn",
+	"react-doctor/rn-no-dimensions-get": "warn",
+	"react-doctor/rn-no-inline-flatlist-renderitem": "warn",
+	"react-doctor/rn-no-legacy-shadow-styles": "warn",
+	"react-doctor/rn-prefer-reanimated": "warn",
+	"react-doctor/rn-no-single-element-style-array": "warn"
+};
 const REACT_COMPILER_RULES = {
 	"react-hooks-js/set-state-in-render": "error",
 	"react-hooks-js/immutability": "error",
@@ -669,7 +844,8 @@ const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler }) => ({
 		"react-doctor/server-after-nonblocking": "warn",
 		"react-doctor/client-passive-event-listeners": "warn",
 		"react-doctor/async-parallel": "warn",
-		...framework === "nextjs" ? NEXTJS_RULES : {}
+		...framework === "nextjs" ? NEXTJS_RULES : {},
+		...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {}
 	}
 });
 
@@ -777,10 +953,18 @@ const RULE_CATEGORY_MAP = {
 	"react-doctor/server-auth-actions": "Server",
 	"react-doctor/server-after-nonblocking": "Server",
 	"react-doctor/client-passive-event-listeners": "Performance",
-	"react-doctor/async-parallel": "Performance"
+	"react-doctor/async-parallel": "Performance",
+	"react-doctor/rn-no-raw-text": "React Native",
+	"react-doctor/rn-no-deprecated-modules": "React Native",
+	"react-doctor/rn-no-legacy-expo-packages": "React Native",
+	"react-doctor/rn-no-dimensions-get": "React Native",
+	"react-doctor/rn-no-inline-flatlist-renderitem": "React Native",
+	"react-doctor/rn-no-legacy-shadow-styles": "React Native",
+	"react-doctor/rn-prefer-reanimated": "React Native",
+	"react-doctor/rn-no-single-element-style-array": "React Native"
 };
 const RULE_HELP_MAP = {
-	"no-derived-state-effect": "For derived state, compute inline: `const x = fn(dep)`. For state resets on prop change, use a key prop: `<Component key={prop} />`",
+	"no-derived-state-effect": "For derived state, compute inline: `const x = fn(dep)`. For state resets on prop change, use a key prop: `<Component key={prop} />`. See https://react.dev/learn/you-might-not-need-an-effect",
 	"no-fetch-in-effect": "Use `useQuery()` from @tanstack/react-query, `useSWR()`, or fetch in a Server Component instead",
 	"no-cascading-set-state": "Combine into useReducer: `const [state, dispatch] = useReducer(reducer, initialState)`",
 	"no-effect-event-handler": "Move the conditional logic into onClick, onChange, or onSubmit handlers directly",
@@ -820,7 +1004,7 @@ const RULE_HELP_MAP = {
 	"nextjs-no-use-search-params-without-suspense": "Wrap the component using useSearchParams: `<Suspense fallback={<Skeleton />}><SearchComponent /></Suspense>`",
 	"nextjs-no-client-fetch-for-server-data": "Remove 'use client' and fetch directly in the Server Component — no API round-trip, secrets stay on server",
 	"nextjs-missing-metadata": "Add `export const metadata = { title: '...', description: '...' }` or `export async function generateMetadata()`",
-	"nextjs-no-client-side-redirect": "Use `redirect('/path')` from 'next/navigation' in a Server Component, or handle in middleware",
+	"nextjs-no-client-side-redirect": "Use `redirect('/path')` from 'next/navigation' directly (works in both server and client components), or handle in middleware",
 	"nextjs-no-redirect-in-try-catch": "Move the redirect/notFound call outside the try block, or add `unstable_rethrow(error)` in the catch",
 	"nextjs-image-missing-sizes": "Add sizes for responsive behavior: `sizes=\"(max-width: 768px) 100vw, 50vw\"` matching your layout breakpoints",
 	"nextjs-no-native-script": "`import Script from \"next/script\"` — use `strategy=\"afterInteractive\"` for analytics or `\"lazyOnload\"` for widgets",
@@ -833,7 +1017,15 @@ const RULE_HELP_MAP = {
 	"server-auth-actions": "Add `const session = await auth()` at the top and throw/redirect if unauthorized before any data access",
 	"server-after-nonblocking": "`import { after } from 'next/server'` then wrap: `after(() => analytics.track(...))` — response isn't blocked",
 	"client-passive-event-listeners": "Add `{ passive: true }` as the third argument: `addEventListener('scroll', handler, { passive: true })`",
-	"async-parallel": "Use `const [a, b] = await Promise.all([fetchA(), fetchB()])` to run independent operations concurrently"
+	"async-parallel": "Use `const [a, b] = await Promise.all([fetchA(), fetchB()])` to run independent operations concurrently",
+	"rn-no-raw-text": "Wrap text in a `<Text>` component: `<Text>{value}</Text>` — raw strings outside `<Text>` crash on React Native",
+	"rn-no-deprecated-modules": "Import from the community package instead — deprecated modules were removed from the react-native core",
+	"rn-no-legacy-expo-packages": "Migrate to the recommended replacement package — legacy Expo packages are no longer maintained",
+	"rn-no-dimensions-get": "Use `const { width, height } = useWindowDimensions()` — it updates reactively on rotation and resize",
+	"rn-no-inline-flatlist-renderitem": "Extract renderItem to a named function or wrap in useCallback to avoid re-creating on every render",
+	"rn-no-legacy-shadow-styles": "Use `boxShadow` for cross-platform shadows on the new architecture instead of platform-specific shadow properties",
+	"rn-prefer-reanimated": "Use `import Animated from 'react-native-reanimated'` — animations run on the UI thread instead of the JS thread",
+	"rn-no-single-element-style-array": "Use `style={value}` instead of `style={[value]}` — single-element arrays add unnecessary allocation"
 };
 const FILEPATH_WITH_LOCATION_PATTERN = /\S+\.\w+:\d+:\d+[\s\S]*$/;
 const REACT_COMPILER_MESSAGE = "React Compiler can't optimize this code";
@@ -900,7 +1092,14 @@ const spawnOxlint = (args, rootDirectory, nodeBinaryPath) => new Promise((resolv
 	child.stdout.on("data", (buffer) => stdoutBuffers.push(buffer));
 	child.stderr.on("data", (buffer) => stderrBuffers.push(buffer));
 	child.on("error", (error) => reject(/* @__PURE__ */ new Error(`Failed to run oxlint: ${error.message}`)));
-	child.on("close", () => {
+	child.on("close", (code, signal) => {
+		if (signal) {
+			const stderrOutput = Buffer.concat(stderrBuffers).toString("utf-8").trim();
+			const hint = signal === "SIGABRT" ? " (out of memory — try scanning fewer files with --diff)" : "";
+			const detail = stderrOutput ? `: ${stderrOutput}` : "";
+			reject(/* @__PURE__ */ new Error(`oxlint was killed by ${signal}${hint}${detail}`));
+			return;
+		}
 		const output = Buffer.concat(stdoutBuffers).toString("utf-8").trim();
 		if (!output) {
 			const stderrOutput = Buffer.concat(stderrBuffers).toString("utf-8").trim();