react-doctor 0.2.0 → 0.2.1

package/dist/cli.js

@@ -101,12 +101,31 @@ const IGNORED_DIRECTORIES = new Set([
 	"out",
 	"storybook-static"
 ]);
+const IGNORABLE_READDIR_ERROR_CODES = new Set([
+	"EACCES",
+	"EPERM",
+	"ENOENT",
+	"ENOTDIR"
+]);
+const isIgnorableReaddirError = (error) => {
+	if (typeof error !== "object" || error === null) return false;
+	const errorCode = error.code;
+	return typeof errorCode === "string" && IGNORABLE_READDIR_ERROR_CODES.has(errorCode);
+};
+const readDirectoryEntries = (directoryPath) => {
+	try {
+		return fs.readdirSync(directoryPath, { withFileTypes: true });
+	} catch (error) {
+		if (isIgnorableReaddirError(error)) return [];
+		throw error;
+	}
+};
 const countSourceFilesViaFilesystem = (rootDirectory) => {
 	let count = 0;
 	const stack = [rootDirectory];
 	while (stack.length > 0) {
 		const currentDirectory = stack.pop();
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true });
+		const entries = readDirectoryEntries(currentDirectory);
 		for (const entry of entries) {
 			if (entry.isDirectory()) {
 				if (!entry.name.startsWith(".") && !IGNORED_DIRECTORIES.has(entry.name)) stack.push(path.join(currentDirectory, entry.name));
@@ -141,7 +160,7 @@ const readPackageJsonUncached = (packageJsonPath) => {
 		if (error instanceof SyntaxError) return {};
 		if (error instanceof Error && "code" in error) {
 			const { code } = error;
-			if (code === "EISDIR" || code === "EACCES") return {};
+			if (code === "EISDIR" || code === "EACCES" || code === "EPERM" || code === "ENOENT") return {};
 		}
 		throw error;
 	}
@@ -512,6 +531,13 @@ const getDependencyDeclaration = ({ packageJson, packageName, sections }) => {
 		version: null
 	};
 };
+const isDirectory = (directoryPath) => {
+	try {
+		return fs.statSync(directoryPath).isDirectory();
+	} catch {
+		return false;
+	}
+};
 const NX_PROJECT_DISCOVERY_DIRS = [
 	"apps",
 	"libs",
@@ -522,8 +548,8 @@ const getNxWorkspaceDirectories = (rootDirectory) => {
 	const collected = [];
 	for (const candidate of NX_PROJECT_DISCOVERY_DIRS) {
 		const candidatePath = path.join(rootDirectory, candidate);
-		if (!fs.existsSync(candidatePath) || !fs.statSync(candidatePath).isDirectory()) continue;
-		for (const entry of fs.readdirSync(candidatePath, { withFileTypes: true })) {
+		if (!isDirectory(candidatePath)) continue;
+		for (const entry of readDirectoryEntries(candidatePath)) {
 			if (!entry.isDirectory()) continue;
 			const projectDirectory = path.join(candidatePath, entry.name);
 			if (isFile(path.join(projectDirectory, "project.json")) || isFile(path.join(projectDirectory, "package.json"))) collected.push(`${candidate}/${entry.name}`);
@@ -565,17 +591,17 @@ const resolveWorkspaceDirectories = (rootDirectory, pattern) => {
 	const cleanPattern = pattern.replace(/["']/g, "").replace(/\/\*\*$/, "/*");
 	if (!cleanPattern.includes("*")) {
 		const directoryPath = path.join(rootDirectory, cleanPattern);
-		if (fs.existsSync(directoryPath) && isFile(path.join(directoryPath, "package.json"))) return [directoryPath];
+		if (isDirectory(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 [];
+	if (!isDirectory(baseDirectory)) return [];
 	const resolved = [];
-	for (const entry of fs.readdirSync(baseDirectory)) {
-		const entryPath = path.join(baseDirectory, entry, suffixAfterWildcard);
-		if (fs.existsSync(entryPath) && fs.statSync(entryPath).isDirectory() && isFile(path.join(entryPath, "package.json"))) resolved.push(entryPath);
+	for (const entry of readDirectoryEntries(baseDirectory)) {
+		const entryPath = path.join(baseDirectory, entry.name, suffixAfterWildcard);
+		if (isDirectory(entryPath) && isFile(path.join(entryPath, "package.json"))) resolved.push(entryPath);
 	}
 	return resolved;
 };
@@ -820,7 +846,7 @@ const discoverReactSubprojectsByFilesystem = (rootDirectory) => {
 				});
 			}
 		}
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true }).toSorted((firstEntry, secondEntry) => firstEntry.name.localeCompare(secondEntry.name));
+		const entries = readDirectoryEntries(currentDirectory).toSorted((firstEntry, secondEntry) => firstEntry.name.localeCompare(secondEntry.name));
 		for (const entry of entries) {
 			if (!entry.isDirectory() || entry.name.startsWith(".") || IGNORED_DIRECTORIES.has(entry.name)) continue;
 			pendingDirectories.push(path.join(currentDirectory, entry.name));
@@ -829,7 +855,7 @@ const discoverReactSubprojectsByFilesystem = (rootDirectory) => {
 	return packages;
 };
 const discoverReactSubprojects = (rootDirectory) => {
-	if (!fs.existsSync(rootDirectory) || !fs.statSync(rootDirectory).isDirectory()) return [];
+	if (!isDirectory(rootDirectory)) return [];
 	const manifestPackages = listManifestWorkspacePackages(rootDirectory);
 	if (manifestPackages.length > 0) return manifestPackages;
 	return discoverReactSubprojectsByFilesystem(rootDirectory);
@@ -4289,12 +4315,7 @@ const findFilesWithDisableDirectivesViaFilesystem = (rootDirectory, includePaths
 	while (stack.length > 0) {
 		const current = stack.pop();
 		if (current === void 0) continue;
-		let entries;
-		try {
-			entries = fs.readdirSync(current, { withFileTypes: true });
-		} catch {
-			continue;
-		}
+		const entries = readDirectoryEntries(current);
 		for (const entry of entries) {
 			if (entry.isDirectory()) {
 				if (entry.name.startsWith(".") || IGNORED_DIRECTORIES.has(entry.name)) continue;
@@ -4426,7 +4447,7 @@ const resolveConfigRootDir = (config, configSourceDirectory) => {
 	if (trimmedRootDir.length === 0) return null;
 	const resolvedRootDir = path.isAbsolute(trimmedRootDir) ? trimmedRootDir : path.resolve(configSourceDirectory, trimmedRootDir);
 	if (resolvedRootDir === configSourceDirectory) return null;
-	if (!fs.existsSync(resolvedRootDir) || !fs.statSync(resolvedRootDir).isDirectory()) {
+	if (!isDirectory(resolvedRootDir)) {
 		logger.warn(`react-doctor config "rootDir" points to "${rawRootDir}" (resolved to ${resolvedRootDir}), which is not a directory. Ignoring.`);
 		return null;
 	}
@@ -4452,7 +4473,7 @@ const listSourceFilesViaFilesystem = (rootDirectory) => {
 	const stack = [rootDirectory];
 	while (stack.length > 0) {
 		const currentDirectory = stack.pop();
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true });
+		const entries = readDirectoryEntries(currentDirectory);
 		for (const entry of entries) {
 			const absolutePath = path.join(currentDirectory, entry.name);
 			if (entry.isDirectory()) {
@@ -5977,7 +5998,7 @@ const CI_ENVIRONMENT_VARIABLES = [
 const isCiEnvironment = () => CI_ENVIRONMENT_VARIABLES.some((envVariable) => Boolean(process.env[envVariable])) || process.env.CI === "true";
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.2.0";
+const VERSION = "0.2.1";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;

package/dist/index.d.ts

@@ -1,3 +1,4 @@
+import fs from "node:fs";
 //#region ../types/dist/index.d.ts
 //#region src/config.d.ts
 type FailOnLevel = "error" | "warning" | "none";
@@ -401,7 +402,7 @@ declare class AmbiguousProjectError extends ReactDoctorError {
   constructor(directory: string, candidates: readonly string[], options?: ErrorOptions);
 }
 declare const isReactDoctorError: (value: unknown) => value is ReactDoctorError; //#endregion
-//#region src/utils/is-file.d.ts
+//#region src/utils/is-directory.d.ts
 //#endregion
 //#region ../core/dist/index.d.ts
 //#endregion

package/dist/index.js

@@ -113,12 +113,31 @@ const IGNORED_DIRECTORIES = new Set([
 	"out",
 	"storybook-static"
 ]);
+const IGNORABLE_READDIR_ERROR_CODES = new Set([
+	"EACCES",
+	"EPERM",
+	"ENOENT",
+	"ENOTDIR"
+]);
+const isIgnorableReaddirError = (error) => {
+	if (typeof error !== "object" || error === null) return false;
+	const errorCode = error.code;
+	return typeof errorCode === "string" && IGNORABLE_READDIR_ERROR_CODES.has(errorCode);
+};
+const readDirectoryEntries = (directoryPath) => {
+	try {
+		return fs.readdirSync(directoryPath, { withFileTypes: true });
+	} catch (error) {
+		if (isIgnorableReaddirError(error)) return [];
+		throw error;
+	}
+};
 const countSourceFilesViaFilesystem = (rootDirectory) => {
 	let count = 0;
 	const stack = [rootDirectory];
 	while (stack.length > 0) {
 		const currentDirectory = stack.pop();
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true });
+		const entries = readDirectoryEntries(currentDirectory);
 		for (const entry of entries) {
 			if (entry.isDirectory()) {
 				if (!entry.name.startsWith(".") && !IGNORED_DIRECTORIES.has(entry.name)) stack.push(path.join(currentDirectory, entry.name));
@@ -156,7 +175,7 @@ const readPackageJsonUncached = (packageJsonPath) => {
 		if (error instanceof SyntaxError) return {};
 		if (error instanceof Error && "code" in error) {
 			const { code } = error;
-			if (code === "EISDIR" || code === "EACCES") return {};
+			if (code === "EISDIR" || code === "EACCES" || code === "EPERM" || code === "ENOENT") return {};
 		}
 		throw error;
 	}
@@ -527,6 +546,13 @@ const getDependencyDeclaration = ({ packageJson, packageName, sections }) => {
 		version: null
 	};
 };
+const isDirectory = (directoryPath) => {
+	try {
+		return fs.statSync(directoryPath).isDirectory();
+	} catch {
+		return false;
+	}
+};
 const NX_PROJECT_DISCOVERY_DIRS = [
 	"apps",
 	"libs",
@@ -537,8 +563,8 @@ const getNxWorkspaceDirectories = (rootDirectory) => {
 	const collected = [];
 	for (const candidate of NX_PROJECT_DISCOVERY_DIRS) {
 		const candidatePath = path.join(rootDirectory, candidate);
-		if (!fs.existsSync(candidatePath) || !fs.statSync(candidatePath).isDirectory()) continue;
-		for (const entry of fs.readdirSync(candidatePath, { withFileTypes: true })) {
+		if (!isDirectory(candidatePath)) continue;
+		for (const entry of readDirectoryEntries(candidatePath)) {
 			if (!entry.isDirectory()) continue;
 			const projectDirectory = path.join(candidatePath, entry.name);
 			if (isFile(path.join(projectDirectory, "project.json")) || isFile(path.join(projectDirectory, "package.json"))) collected.push(`${candidate}/${entry.name}`);
@@ -580,17 +606,17 @@ const resolveWorkspaceDirectories = (rootDirectory, pattern) => {
 	const cleanPattern = pattern.replace(/["']/g, "").replace(/\/\*\*$/, "/*");
 	if (!cleanPattern.includes("*")) {
 		const directoryPath = path.join(rootDirectory, cleanPattern);
-		if (fs.existsSync(directoryPath) && isFile(path.join(directoryPath, "package.json"))) return [directoryPath];
+		if (isDirectory(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 [];
+	if (!isDirectory(baseDirectory)) return [];
 	const resolved = [];
-	for (const entry of fs.readdirSync(baseDirectory)) {
-		const entryPath = path.join(baseDirectory, entry, suffixAfterWildcard);
-		if (fs.existsSync(entryPath) && fs.statSync(entryPath).isDirectory() && isFile(path.join(entryPath, "package.json"))) resolved.push(entryPath);
+	for (const entry of readDirectoryEntries(baseDirectory)) {
+		const entryPath = path.join(baseDirectory, entry.name, suffixAfterWildcard);
+		if (isDirectory(entryPath) && isFile(path.join(entryPath, "package.json"))) resolved.push(entryPath);
 	}
 	return resolved;
 };
@@ -835,7 +861,7 @@ const discoverReactSubprojectsByFilesystem = (rootDirectory) => {
 				});
 			}
 		}
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true }).toSorted((firstEntry, secondEntry) => firstEntry.name.localeCompare(secondEntry.name));
+		const entries = readDirectoryEntries(currentDirectory).toSorted((firstEntry, secondEntry) => firstEntry.name.localeCompare(secondEntry.name));
 		for (const entry of entries) {
 			if (!entry.isDirectory() || entry.name.startsWith(".") || IGNORED_DIRECTORIES.has(entry.name)) continue;
 			pendingDirectories.push(path.join(currentDirectory, entry.name));
@@ -844,7 +870,7 @@ const discoverReactSubprojectsByFilesystem = (rootDirectory) => {
 	return packages;
 };
 const discoverReactSubprojects = (rootDirectory) => {
-	if (!fs.existsSync(rootDirectory) || !fs.statSync(rootDirectory).isDirectory()) return [];
+	if (!isDirectory(rootDirectory)) return [];
 	const manifestPackages = listManifestWorkspacePackages(rootDirectory);
 	if (manifestPackages.length > 0) return manifestPackages;
 	return discoverReactSubprojectsByFilesystem(rootDirectory);
@@ -4249,12 +4275,7 @@ const findFilesWithDisableDirectivesViaFilesystem = (rootDirectory, includePaths
 	while (stack.length > 0) {
 		const current = stack.pop();
 		if (current === void 0) continue;
-		let entries;
-		try {
-			entries = fs.readdirSync(current, { withFileTypes: true });
-		} catch {
-			continue;
-		}
+		const entries = readDirectoryEntries(current);
 		for (const entry of entries) {
 			if (entry.isDirectory()) {
 				if (entry.name.startsWith(".") || IGNORED_DIRECTORIES.has(entry.name)) continue;
@@ -4312,7 +4333,7 @@ const resolveConfigRootDir = (config, configSourceDirectory) => {
 	if (trimmedRootDir.length === 0) return null;
 	const resolvedRootDir = path.isAbsolute(trimmedRootDir) ? trimmedRootDir : path.resolve(configSourceDirectory, trimmedRootDir);
 	if (resolvedRootDir === configSourceDirectory) return null;
-	if (!fs.existsSync(resolvedRootDir) || !fs.statSync(resolvedRootDir).isDirectory()) {
+	if (!isDirectory(resolvedRootDir)) {
 		logger.warn(`react-doctor config "rootDir" points to "${rawRootDir}" (resolved to ${resolvedRootDir}), which is not a directory. Ignoring.`);
 		return null;
 	}
@@ -4345,7 +4366,7 @@ const listSourceFilesViaFilesystem = (rootDirectory) => {
 	const stack = [rootDirectory];
 	while (stack.length > 0) {
 		const currentDirectory = stack.pop();
-		const entries = fs.readdirSync(currentDirectory, { withFileTypes: true });
+		const entries = readDirectoryEntries(currentDirectory);
 		for (const entry of entries) {
 			const absolutePath = path.join(currentDirectory, entry.name);
 			if (entry.isDirectory()) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -56,15 +56,15 @@
     "picocolors": "^1.1.1",
     "prompts": "^2.4.2",
     "typescript": ">=5.0.4 <7",
-    "oxlint-plugin-react-doctor": "0.2.0"
+    "oxlint-plugin-react-doctor": "0.2.1"
   },
   "devDependencies": {
     "@types/prompts": "^2.4.9",
     "eslint-plugin-react-hooks": "^7.1.1",
     "eslint-plugin-react-you-might-not-need-an-effect": "^0.10.1",
-    "@react-doctor/core": "0.2.0",
-    "@react-doctor/project-info": "0.2.0",
-    "@react-doctor/types": "0.2.0"
+    "@react-doctor/core": "0.2.1",
+    "@react-doctor/types": "0.2.1",
+    "@react-doctor/project-info": "0.2.1"
   },
   "peerDependencies": {
     "eslint-plugin-react-hooks": "^6 || ^7",