react-doctor 0.0.27 → 0.0.29

package/dist/cli.js

@@ -1,18 +1,18 @@
 #!/usr/bin/env node
 import { createRequire } from "node:module";
 import { execSync, spawn, spawnSync } from "node:child_process";
-import fs, { appendFileSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import fs, { appendFileSync, existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from "node:fs";
 import os, { homedir, tmpdir } from "node:os";
 import path, { join } from "node:path";
 import { Command } from "commander";
 import { randomUUID } from "node:crypto";
 import { performance } from "node:perf_hooks";
 import pc from "picocolors";
+import basePrompts from "prompts";
 import { main } from "knip";
 import { createOptions } from "knip/session";
 import { fileURLToPath } from "node:url";
 import ora from "ora";
-import basePrompts from "prompts";
 
 //#region src/constants.ts
 const SOURCE_FILE_PATTERN = /\.(tsx?|jsx?)$/;
@@ -40,6 +40,8 @@ const WARNING_RULE_PENALTY = .75;
 const ERROR_ESTIMATED_FIX_RATE = .85;
 const WARNING_ESTIMATED_FIX_RATE = .8;
 const MAX_KNIP_RETRIES = 5;
+const OXLINT_NODE_REQUIREMENT = "^20.19.0 || >=22.12.0";
+const OXLINT_RECOMMENDED_NODE_MAJOR = 24;
 const AMI_WEBSITE_URL = "https://ami.dev";
 const AMI_INSTALL_URL = `${AMI_WEBSITE_URL}/install.sh`;
 const AMI_RELEASES_URL = "https://github.com/millionco/ami-releases/releases";
@@ -281,6 +283,7 @@ const combineDiagnostics = (lintDiagnostics, deadCodeDiagnostics, directory, isD
 //#region src/utils/find-monorepo-root.ts
 const isMonorepoRoot = (directory) => {
 	if (fs.existsSync(path.join(directory, "pnpm-workspace.yaml"))) return true;
+	if (fs.existsSync(path.join(directory, "nx.json"))) return true;
 	const packageJsonPath = path.join(directory, "package.json");
 	if (!fs.existsSync(packageJsonPath)) return false;
 	const packageJson = readPackageJson(packageJsonPath);
@@ -333,7 +336,9 @@ const FRAMEWORK_PACKAGES = {
 	vite: "vite",
 	"react-scripts": "cra",
 	"@remix-run/react": "remix",
-	gatsby: "gatsby"
+	gatsby: "gatsby",
+	expo: "expo",
+	"react-native": "react-native"
 };
 const FRAMEWORK_DISPLAY_NAMES = {
 	nextjs: "Next.js",
@@ -341,10 +346,34 @@ const FRAMEWORK_DISPLAY_NAMES = {
 	cra: "Create React App",
 	remix: "Remix",
 	gatsby: "Gatsby",
+	expo: "Expo",
+	"react-native": "React Native",
 	unknown: "React"
 };
 const formatFrameworkName = (framework) => FRAMEWORK_DISPLAY_NAMES[framework];
-const countSourceFiles = (rootDirectory) => {
+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 countSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
 		"--cached",
@@ -355,9 +384,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,
@@ -442,14 +472,30 @@ const findReactInWorkspaces = (rootDirectory, packageJson) => {
 	}
 	return result;
 };
+const REACT_DEPENDENCY_NAMES = new Set([
+	"react",
+	"react-native",
+	"next"
+]);
 const hasReactDependency = (packageJson) => {
 	const allDependencies = collectAllDependencies(packageJson);
-	return Object.keys(allDependencies).some((packageName) => packageName === "next" || packageName.includes("react"));
+	return Object.keys(allDependencies).some((packageName) => REACT_DEPENDENCY_NAMES.has(packageName));
 };
 const discoverReactSubprojects = (rootDirectory) => {
 	if (!fs.existsSync(rootDirectory) || !fs.statSync(rootDirectory).isDirectory()) return [];
-	const entries = fs.readdirSync(rootDirectory, { withFileTypes: true });
 	const packages = [];
+	const rootPackageJsonPath = path.join(rootDirectory, "package.json");
+	if (fs.existsSync(rootPackageJsonPath)) {
+		const rootPackageJson = readPackageJson(rootPackageJsonPath);
+		if (hasReactDependency(rootPackageJson)) {
+			const name = rootPackageJson.name ?? path.basename(rootDirectory);
+			packages.push({
+				name,
+				directory: rootDirectory
+			});
+		}
+	}
+	const entries = fs.readdirSync(rootDirectory, { withFileTypes: true });
 	for (const entry of entries) {
 		if (!entry.isDirectory() || entry.name.startsWith(".") || entry.name === "node_modules") continue;
 		const subdirectory = path.join(rootDirectory, entry.name);
@@ -621,14 +667,10 @@ const loadConfig = (rootDirectory) => {
 	if (fs.existsSync(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 {
@@ -641,6 +683,167 @@ const loadConfig = (rootDirectory) => {
 	return null;
 };
 
+//#endregion
+//#region src/utils/should-auto-select-current-choice.ts
+const shouldAutoSelectCurrentChoice = (choiceStates, cursor) => {
+	if (choiceStates.some((choiceState) => choiceState.selected)) return false;
+	const currentChoice = choiceStates[cursor];
+	return Boolean(currentChoice) && !currentChoice.disabled;
+};
+
+//#endregion
+//#region src/utils/should-select-all-choices.ts
+const shouldSelectAllChoices = (choiceStates) => {
+	return choiceStates.filter((choiceState) => !choiceState.disabled).some((choiceState) => choiceState.selected !== true);
+};
+
+//#endregion
+//#region src/utils/prompts.ts
+const require = createRequire(import.meta.url);
+const PROMPTS_MULTISELECT_MODULE_PATH = "prompts/lib/elements/multiselect";
+const PROMPTS_SELECT_MODULE_PATH = "prompts/lib/elements/select";
+let didPatchMultiselectToggleAll = false;
+let didPatchMultiselectSubmit = false;
+let didPatchSelectBanner = false;
+const selectBannerMap = /* @__PURE__ */ new Map();
+const setSelectBanner = (banner, targetIndex) => {
+	selectBannerMap.set(targetIndex, banner);
+};
+const clearSelectBanner = () => {
+	selectBannerMap.clear();
+};
+const onCancel = () => {
+	logger.break();
+	logger.log("Cancelled.");
+	logger.dim("Run `npx react-doctor@latest --fix` to fix issues.");
+	logger.break();
+	process.exit(0);
+};
+const patchMultiselectToggleAll = () => {
+	if (didPatchMultiselectToggleAll) return;
+	didPatchMultiselectToggleAll = true;
+	const multiselectPromptConstructor = require(PROMPTS_MULTISELECT_MODULE_PATH);
+	multiselectPromptConstructor.prototype.toggleAll = function() {
+		const isCurrentChoiceDisabled = Boolean(this.value[this.cursor]?.disabled);
+		if (this.maxChoices !== void 0 || isCurrentChoiceDisabled) {
+			this.bell();
+			return;
+		}
+		const shouldSelectAllEnabledChoices = shouldSelectAllChoices(this.value);
+		for (const choiceState of this.value) {
+			if (choiceState.disabled) continue;
+			choiceState.selected = shouldSelectAllEnabledChoices;
+		}
+		this.render();
+	};
+};
+const patchMultiselectSubmit = () => {
+	if (didPatchMultiselectSubmit) return;
+	didPatchMultiselectSubmit = true;
+	const multiselectPromptConstructor = require(PROMPTS_MULTISELECT_MODULE_PATH);
+	const originalSubmit = multiselectPromptConstructor.prototype.submit;
+	multiselectPromptConstructor.prototype.submit = function() {
+		if (shouldAutoSelectCurrentChoice(this.value, this.cursor)) this.value[this.cursor].selected = true;
+		originalSubmit.call(this);
+	};
+};
+const patchSelectBanner = () => {
+	if (didPatchSelectBanner) return;
+	didPatchSelectBanner = true;
+	const selectConstructor = require(PROMPTS_SELECT_MODULE_PATH);
+	const promptsClear = require("prompts/lib/util/clear");
+	const originalRender = selectConstructor.prototype.render;
+	selectConstructor.prototype.render = function() {
+		originalRender.call(this);
+		const banner = selectBannerMap.get(this.cursor);
+		if (!banner || this.closed || this.done) return;
+		this.out.write(promptsClear(this.outputText, this.out.columns));
+		this.outputText = `${banner}\n\n${this.outputText}`;
+		this.out.write(this.outputText);
+	};
+};
+const prompts = (questions) => {
+	patchMultiselectToggleAll();
+	patchMultiselectSubmit();
+	patchSelectBanner();
+	return basePrompts(questions, { onCancel });
+};
+
+//#endregion
+//#region src/utils/resolve-compatible-node.ts
+const parseNodeVersion = (versionString) => {
+	const [major = 0, minor = 0, patch = 0] = versionString.replace(/^v/, "").trim().split(".").map(Number);
+	return {
+		major,
+		minor,
+		patch
+	};
+};
+const isNodeVersionCompatibleWithOxlint = ({ major, minor }) => {
+	if (major === 20 && minor >= 19) return true;
+	if (major === 22 && minor >= 12) return true;
+	if (major > 22) return true;
+	return false;
+};
+const isCurrentNodeCompatibleWithOxlint = () => isNodeVersionCompatibleWithOxlint(parseNodeVersion(process.version));
+const getNvmDirectory = () => {
+	const envNvmDirectory = process.env.NVM_DIR;
+	if (envNvmDirectory && existsSync(envNvmDirectory)) return envNvmDirectory;
+	const defaultNvmDirectory = path.join(os.homedir(), ".nvm");
+	if (existsSync(defaultNvmDirectory)) return defaultNvmDirectory;
+	return null;
+};
+const isNvmInstalled = () => getNvmDirectory() !== null;
+const findCompatibleNvmBinary = () => {
+	const nvmDirectory = getNvmDirectory();
+	if (!nvmDirectory) return null;
+	const versionsDirectory = path.join(nvmDirectory, "versions", "node");
+	if (!existsSync(versionsDirectory)) return null;
+	const compatibleVersions = readdirSync(versionsDirectory).filter((directoryName) => directoryName.startsWith("v")).map((directoryName) => ({
+		directoryName,
+		...parseNodeVersion(directoryName)
+	})).filter((version) => isNodeVersionCompatibleWithOxlint(version)).sort((versionA, versionB) => versionB.major - versionA.major || versionB.minor - versionA.minor || versionB.patch - versionA.patch);
+	if (compatibleVersions.length === 0) return null;
+	const bestVersion = compatibleVersions[0];
+	const binaryPath = path.join(versionsDirectory, bestVersion.directoryName, "bin", "node");
+	return existsSync(binaryPath) ? binaryPath : null;
+};
+const getNodeVersionFromBinary = (binaryPath) => {
+	try {
+		return execSync(`"${binaryPath}" --version`, { encoding: "utf-8" }).trim();
+	} catch {
+		return null;
+	}
+};
+const installNodeViaNvm = () => {
+	const nvmDirectory = getNvmDirectory();
+	if (!nvmDirectory) return false;
+	const nvmScript = path.join(nvmDirectory, "nvm.sh");
+	if (!existsSync(nvmScript)) return false;
+	try {
+		execSync(`bash -c ". '${nvmScript}' && nvm install ${OXLINT_RECOMMENDED_NODE_MAJOR}"`, { stdio: "inherit" });
+		return findCompatibleNvmBinary() !== null;
+	} catch {
+		return false;
+	}
+};
+const resolveNodeForOxlint = () => {
+	if (isCurrentNodeCompatibleWithOxlint()) return {
+		binaryPath: process.execPath,
+		isCurrentNode: true,
+		version: process.version
+	};
+	const nvmBinaryPath = findCompatibleNvmBinary();
+	if (!nvmBinaryPath) return null;
+	const version = getNodeVersionFromBinary(nvmBinaryPath);
+	if (!version) return null;
+	return {
+		binaryPath: nvmBinaryPath,
+		isCurrentNode: false,
+		version
+	};
+};
+
 //#endregion
 //#region src/utils/run-knip.ts
 const KNIP_CATEGORY_MAP = {
@@ -699,11 +902,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 {
@@ -775,6 +982,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",
@@ -878,7 +1095,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 : {}
 	}
 });
 
@@ -986,7 +1204,15 @@ 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} />`",
@@ -1042,7 +1268,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";
@@ -1102,8 +1336,8 @@ const batchIncludePaths = (baseArgs, includePaths) => {
 	if (currentBatch.length > 0) batches.push(currentBatch);
 	return batches;
 };
-const spawnOxlint = (args, rootDirectory) => new Promise((resolve, reject) => {
-	const child = spawn(process.execPath, args, { cwd: rootDirectory });
+const spawnOxlint = (args, rootDirectory, nodeBinaryPath) => new Promise((resolve, reject) => {
+	const child = spawn(nodeBinaryPath, args, { cwd: rootDirectory });
 	const stdoutBuffers = [];
 	const stderrBuffers = [];
 	child.stdout.on("data", (buffer) => stdoutBuffers.push(buffer));
@@ -1146,7 +1380,7 @@ const parseOxlintOutput = (stdout) => {
 		};
 	});
 };
-const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompiler, includePaths) => {
+const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompiler, includePaths, nodeBinaryPath = process.execPath) => {
 	if (includePaths !== void 0 && includePaths.length === 0) return [];
 	const configPath = path.join(os.tmpdir(), `react-doctor-oxlintrc-${process.pid}.json`);
 	const config = createOxlintConfig({
@@ -1168,7 +1402,7 @@ const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompil
 		const fileBatches = includePaths !== void 0 ? batchIncludePaths(baseArgs, includePaths) : [["."]];
 		const allDiagnostics = [];
 		for (const batch of fileBatches) {
-			const stdout = await spawnOxlint([...baseArgs, ...batch], rootDirectory);
+			const stdout = await spawnOxlint([...baseArgs, ...batch], rootDirectory, nodeBinaryPath);
 			allDiagnostics.push(...parseOxlintOutput(stdout));
 		}
 		return allDiagnostics;
@@ -1393,6 +1627,44 @@ const printSummary = (diagnostics, elapsedMilliseconds, scoreResult, projectName
 	logger.break();
 	logger.dim(`  Share your results: ${highlighter.info(shareUrl)}`);
 };
+const resolveOxlintNode = async (isLintEnabled, isScoreOnly) => {
+	if (!isLintEnabled) return null;
+	const nodeResolution = resolveNodeForOxlint();
+	if (nodeResolution) {
+		if (!nodeResolution.isCurrentNode && !isScoreOnly) {
+			logger.warn(`Node ${process.version} is unsupported by oxlint. Using Node ${nodeResolution.version} from nvm.`);
+			logger.break();
+		}
+		return nodeResolution.binaryPath;
+	}
+	if (isScoreOnly) return null;
+	logger.warn(`Node ${process.version} is not compatible with oxlint (requires ${OXLINT_NODE_REQUIREMENT}). Lint checks will be skipped.`);
+	if (isNvmInstalled() && process.stdin.isTTY) {
+		const { shouldInstallNode } = await prompts({
+			type: "confirm",
+			name: "shouldInstallNode",
+			message: `Install Node ${OXLINT_RECOMMENDED_NODE_MAJOR} via nvm to enable lint checks?`,
+			initial: true
+		});
+		if (shouldInstallNode) {
+			logger.break();
+			const freshResolution = installNodeViaNvm() ? resolveNodeForOxlint() : null;
+			if (freshResolution) {
+				logger.break();
+				logger.success(`Node ${freshResolution.version} installed. Using it for lint checks.`);
+				logger.break();
+				return freshResolution.binaryPath;
+			}
+			logger.break();
+			logger.warn("Failed to install Node via nvm. Skipping lint checks.");
+			logger.break();
+			return null;
+		}
+	} else if (isNvmInstalled()) logger.dim(`  Run: nvm install ${OXLINT_RECOMMENDED_NODE_MAJOR}`);
+	else logger.dim(`  Install nvm (https://github.com/nvm-sh/nvm) and run: nvm install ${OXLINT_RECOMMENDED_NODE_MAJOR}`);
+	logger.break();
+	return null;
+};
 const mergeScanOptions = (inputOptions, userConfig) => ({
 	lint: inputOptions.lint ?? userConfig?.lint ?? true,
 	deadCode: inputOptions.deadCode ?? userConfig?.deadCode ?? true,
@@ -1428,19 +1700,26 @@ const scan = async (directory, inputOptions = {}) => {
 	const jsxIncludePaths = computeJsxIncludePaths(includePaths);
 	let didLintFail = false;
 	let didDeadCodeFail = false;
-	const lintPromise = options.lint ? (async () => {
+	const resolvedNodeBinaryPath = await resolveOxlintNode(options.lint, options.scoreOnly);
+	if (options.lint && !resolvedNodeBinaryPath) didLintFail = true;
+	const lintPromise = resolvedNodeBinaryPath ? (async () => {
 		const lintSpinner = options.scoreOnly ? null : spinner("Running lint checks...").start();
 		try {
-			const lintDiagnostics = await runOxlint(directory, projectInfo.hasTypeScript, projectInfo.framework, projectInfo.hasReactCompiler, jsxIncludePaths);
+			const lintDiagnostics = await runOxlint(directory, projectInfo.hasTypeScript, projectInfo.framework, projectInfo.hasReactCompiler, jsxIncludePaths, resolvedNodeBinaryPath);
 			lintSpinner?.succeed("Running lint checks.");
 			return lintDiagnostics;
 		} catch (error) {
 			didLintFail = true;
-			lintSpinner?.fail("Lint checks failed (non-fatal, skipping).");
-			if (error instanceof Error) {
-				logger.error(error.message);
-				if (error.stack) logger.dim(error.stack);
-			} else logger.error(String(error));
+			if (!options.scoreOnly) {
+				const errorMessage = error instanceof Error ? error.message : String(error);
+				if (errorMessage.includes("native binding")) {
+					lintSpinner?.fail(`Lint checks failed — oxlint native binding not found (Node ${process.version}).`);
+					logger.dim(`  Upgrade to Node ${OXLINT_NODE_REQUIREMENT} or run: npx -p oxlint@latest react-doctor@latest`);
+				} else {
+					lintSpinner?.fail("Lint checks failed (non-fatal, skipping).");
+					logger.error(errorMessage);
+				}
+			}
 			return [];
 		}
 	})() : Promise.resolve([]);
@@ -1452,8 +1731,10 @@ const scan = async (directory, inputOptions = {}) => {
 			return knipDiagnostics;
 		} catch (error) {
 			didDeadCodeFail = true;
-			deadCodeSpinner?.fail("Dead code detection failed (non-fatal, skipping).");
-			logger.error(String(error));
+			if (!options.scoreOnly) {
+				deadCodeSpinner?.fail("Dead code detection failed (non-fatal, skipping).");
+				logger.error(String(error));
+			}
 			return [];
 		}
 	})() : Promise.resolve([]);
@@ -1603,92 +1884,6 @@ const handleError = (error, options = DEFAULT_HANDLE_ERROR_OPTIONS) => {
 	process.exitCode = 1;
 };
 
-//#endregion
-//#region src/utils/should-auto-select-current-choice.ts
-const shouldAutoSelectCurrentChoice = (choiceStates, cursor) => {
-	if (choiceStates.some((choiceState) => choiceState.selected)) return false;
-	const currentChoice = choiceStates[cursor];
-	return Boolean(currentChoice) && !currentChoice.disabled;
-};
-
-//#endregion
-//#region src/utils/should-select-all-choices.ts
-const shouldSelectAllChoices = (choiceStates) => {
-	return choiceStates.filter((choiceState) => !choiceState.disabled).some((choiceState) => choiceState.selected !== true);
-};
-
-//#endregion
-//#region src/utils/prompts.ts
-const require = createRequire(import.meta.url);
-const PROMPTS_MULTISELECT_MODULE_PATH = "prompts/lib/elements/multiselect";
-const PROMPTS_SELECT_MODULE_PATH = "prompts/lib/elements/select";
-let didPatchMultiselectToggleAll = false;
-let didPatchMultiselectSubmit = false;
-let didPatchSelectBanner = false;
-const selectBannerMap = /* @__PURE__ */ new Map();
-const setSelectBanner = (banner, targetIndex) => {
-	selectBannerMap.set(targetIndex, banner);
-};
-const clearSelectBanner = () => {
-	selectBannerMap.clear();
-};
-const onCancel = () => {
-	logger.break();
-	logger.log("Cancelled.");
-	logger.dim("Run `npx react-doctor@latest --fix` to fix issues.");
-	logger.break();
-	process.exit(0);
-};
-const patchMultiselectToggleAll = () => {
-	if (didPatchMultiselectToggleAll) return;
-	didPatchMultiselectToggleAll = true;
-	const multiselectPromptConstructor = require(PROMPTS_MULTISELECT_MODULE_PATH);
-	multiselectPromptConstructor.prototype.toggleAll = function() {
-		const isCurrentChoiceDisabled = Boolean(this.value[this.cursor]?.disabled);
-		if (this.maxChoices !== void 0 || isCurrentChoiceDisabled) {
-			this.bell();
-			return;
-		}
-		const shouldSelectAllEnabledChoices = shouldSelectAllChoices(this.value);
-		for (const choiceState of this.value) {
-			if (choiceState.disabled) continue;
-			choiceState.selected = shouldSelectAllEnabledChoices;
-		}
-		this.render();
-	};
-};
-const patchMultiselectSubmit = () => {
-	if (didPatchMultiselectSubmit) return;
-	didPatchMultiselectSubmit = true;
-	const multiselectPromptConstructor = require(PROMPTS_MULTISELECT_MODULE_PATH);
-	const originalSubmit = multiselectPromptConstructor.prototype.submit;
-	multiselectPromptConstructor.prototype.submit = function() {
-		if (shouldAutoSelectCurrentChoice(this.value, this.cursor)) this.value[this.cursor].selected = true;
-		originalSubmit.call(this);
-	};
-};
-const patchSelectBanner = () => {
-	if (didPatchSelectBanner) return;
-	didPatchSelectBanner = true;
-	const selectConstructor = require(PROMPTS_SELECT_MODULE_PATH);
-	const promptsClear = require("prompts/lib/util/clear");
-	const originalRender = selectConstructor.prototype.render;
-	selectConstructor.prototype.render = function() {
-		originalRender.call(this);
-		const banner = selectBannerMap.get(this.cursor);
-		if (!banner || this.closed || this.done) return;
-		this.out.write(promptsClear(this.outputText, this.out.columns));
-		this.outputText = `${banner}\n\n${this.outputText}`;
-		this.out.write(this.outputText);
-	};
-};
-const prompts = (questions) => {
-	patchMultiselectToggleAll();
-	patchMultiselectSubmit();
-	patchSelectBanner();
-	return basePrompts(questions, { onCancel });
-};
-
 //#endregion
 //#region src/utils/select-projects.ts
 const selectProjects = async (rootDirectory, projectFlag, skipPrompts) => {
@@ -1909,7 +2104,18 @@ const maybePromptSkillInstall = async (shouldSkipPrompts) => {
 
 //#endregion
 //#region src/cli.ts
-const VERSION = "0.0.27";
+const VERSION = "0.0.29";
+const VALID_FAIL_ON_LEVELS = new Set([
+	"error",
+	"warning",
+	"none"
+]);
+const isValidFailOnLevel = (level) => VALID_FAIL_ON_LEVELS.has(level);
+const shouldFailForDiagnostics = (diagnostics, failOnLevel) => {
+	if (failOnLevel === "none") return false;
+	if (failOnLevel === "warning") return diagnostics.length > 0;
+	return diagnostics.some((diagnostic) => diagnostic.severity === "error");
+};
 const exitWithFixHint = () => {
 	logger.break();
 	logger.log("Cancelled.");
@@ -1932,8 +2138,8 @@ const isAutomatedEnvironment = () => AUTOMATED_ENVIRONMENT_VARIABLES.some((envVa
 const resolveCliScanOptions = (flags, userConfig, programInstance) => {
 	const isCliOverride = (optionName) => programInstance.getOptionValueSource(optionName) === "cli";
 	return {
-		lint: isCliOverride("lint") ? flags.lint : userConfig?.lint ?? flags.lint,
-		deadCode: isCliOverride("deadCode") ? flags.deadCode : userConfig?.deadCode ?? flags.deadCode,
+		lint: isCliOverride("lint") ? flags.lint : userConfig?.lint ?? true,
+		deadCode: isCliOverride("deadCode") ? flags.deadCode : userConfig?.deadCode ?? true,
 		verbose: isCliOverride("verbose") ? Boolean(flags.verbose) : userConfig?.verbose ?? false,
 		scoreOnly: flags.score,
 		offline: flags.offline
@@ -1961,7 +2167,7 @@ const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isSco
 	});
 	return Boolean(shouldScanChangedOnly);
 };
-const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--no-lint", "skip linting").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch").option("--offline", "skip telemetry (anonymous, not stored, only used to calculate score)").option("--no-ami", "skip Ami-related prompts").option("--fix", "open Ami to auto-fix all issues").action(async (directory, flags) => {
+const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--lint", "enable linting").option("--no-lint", "skip linting").option("--dead-code", "enable dead code detection").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch").option("--offline", "skip telemetry (anonymous, not stored, only used to calculate score)").option("--no-ami", "skip Ami-related prompts").option("--fail-on <level>", "exit with error code on diagnostics: error, warning, none", "none").option("--fix", "open Ami to auto-fix all issues").action(async (directory, flags) => {
 	const isScoreOnly = flags.score;
 	try {
 		const resolvedDirectory = path.resolve(directory);
@@ -2011,6 +2217,8 @@ const program = new Command().name("react-doctor").description("Diagnose React c
 			allDiagnostics.push(...scanResult.diagnostics);
 			if (!isScoreOnly) logger.break();
 		}
+		const resolvedFailOn = program.getOptionValueSource("failOn") === "cli" ? flags.failOn : userConfig?.failOn ?? flags.failOn;
+		if (shouldFailForDiagnostics(allDiagnostics, isValidFailOnLevel(resolvedFailOn) ? resolvedFailOn : "none")) process.exitCode = 1;
 		if (flags.fix) openAmiToFix(resolvedDirectory);
 		if (!isScoreOnly && !shouldSkipAmiPrompts && !flags.fix) {
 			await maybePromptSkillInstall(shouldSkipAmiPrompts);
@@ -2073,7 +2281,11 @@ const openAmiToFix = (directory) => {
 	if (!isInstalled) {
 		if (process.platform === "darwin") {
 			installAmi();
-			logger.success("Ami installed successfully.");
+			if (isAmiInstalled()) logger.success("Ami installed successfully.");
+			else {
+				logger.error("Installation could not be verified.");
+				logger.dim(`Install manually at ${highlighter.info(AMI_WEBSITE_URL)}`);
+			}
 		} else {
 			logger.error("Ami is not installed.");
 			logger.dim(`Download at ${highlighter.info(AMI_RELEASES_URL)}`);