react-doctor 0.0.28 → 0.0.29

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

@@ -194,6 +194,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);
@@ -246,9 +247,33 @@ const FRAMEWORK_PACKAGES = {
 	vite: "vite",
 	"react-scripts": "cra",
 	"@remix-run/react": "remix",
-	gatsby: "gatsby"
+	gatsby: "gatsby",
+	expo: "expo",
+	"react-native": "react-native"
 };
-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",
@@ -259,9 +284,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,
@@ -412,14 +438,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 {
@@ -490,11 +512,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 {
@@ -566,6 +592,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 +705,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,7 +814,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} />`",
@@ -833,7 +878,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";