react-doctor 0.1.3 → 0.1.5

package/dist/index.d.ts

@@ -18,6 +18,7 @@ interface Diagnostic {
   severity: "error" | "warning";
   message: string;
   help: string;
+  url?: string;
   line: number;
   column: number;
   category: string;
@@ -75,6 +76,24 @@ interface ReactDoctorConfig {
   failOn?: FailOnLevel;
   customRulesOnly?: boolean;
   share?: boolean;
+  /**
+   * Redirect react-doctor at a different project directory than the one
+   * it was invoked against. Resolved relative to the location of the
+   * config file that declared this field (NOT relative to the CWD), so
+   * the redirect is stable no matter where the CLI / `diagnose()` is
+   * run from. Absolute paths are used as-is.
+   *
+   * Typical use: a monorepo root holds the only `react-doctor.config.json`
+   * (so editor tooling and child commands all find it), but the React
+   * app lives in `apps/web`. Setting `"rootDir": "apps/web"` makes
+   * every invocation that loads this config scan that subproject
+   * without anyone needing to `cd` first or pass an explicit path.
+   *
+   * Ignored if the resolved path does not exist or is not a directory
+   * (a warning is emitted and react-doctor falls back to the originally
+   * requested directory).
+   */
+  rootDir?: string;
   textComponents?: string[];
   /**
    * Names of components that safely route string-only children through a
@@ -211,6 +230,34 @@ declare const filterSourceFiles: (filePaths: string[]) => string[];
 //#region src/utils/summarize-diagnostics.d.ts
 declare const summarizeDiagnostics: (diagnostics: Diagnostic[], worstScore?: number | null, worstScoreLabel?: string | null) => JsonReportSummary;
 //#endregion
+//#region src/errors.d.ts
+declare class ReactDoctorError extends Error {
+  readonly name: string;
+  constructor(message: string, options?: ErrorOptions);
+}
+declare class ProjectNotFoundError extends ReactDoctorError {
+  readonly name = "ProjectNotFoundError";
+  readonly directory: string;
+  constructor(directory: string, options?: ErrorOptions);
+}
+declare class NoReactDependencyError extends ReactDoctorError {
+  readonly name = "NoReactDependencyError";
+  readonly directory: string;
+  constructor(directory: string, options?: ErrorOptions);
+}
+declare class PackageJsonNotFoundError extends ReactDoctorError {
+  readonly name = "PackageJsonNotFoundError";
+  readonly directory: string;
+  constructor(directory: string, options?: ErrorOptions);
+}
+declare class AmbiguousProjectError extends ReactDoctorError {
+  readonly name = "AmbiguousProjectError";
+  readonly directory: string;
+  readonly candidates: readonly string[];
+  constructor(directory: string, candidates: readonly string[], options?: ErrorOptions);
+}
+declare const isReactDoctorError: (value: unknown) => value is ReactDoctorError;
+//#endregion
 //#region src/index.d.ts
 declare const clearCaches: () => void;
 interface ToJsonReportOptions {
@@ -221,5 +268,5 @@ interface ToJsonReportOptions {
 declare const toJsonReport: (result: DiagnoseResult, options: ToJsonReportOptions) => JsonReport;
 declare const diagnose: (directory: string, options?: DiagnoseOptions) => Promise<DiagnoseResult>;
 //#endregion
-export { type DiagnoseOptions, type DiagnoseResult, type Diagnostic, type DiffInfo, type JsonReport, type JsonReportDiffInfo, type JsonReportError, type JsonReportMode, type JsonReportProjectEntry, type JsonReportSummary, type ProjectInfo, type ReactDoctorConfig, type ScoreResult, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, summarizeDiagnostics, toJsonReport };
+export { AmbiguousProjectError, type DiagnoseOptions, type DiagnoseResult, type Diagnostic, type DiffInfo, type JsonReport, type JsonReportDiffInfo, type JsonReportError, type JsonReportMode, type JsonReportProjectEntry, type JsonReportSummary, NoReactDependencyError, PackageJsonNotFoundError, type ProjectInfo, ProjectNotFoundError, type ReactDoctorConfig, ReactDoctorError, type ScoreResult, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 //# sourceMappingURL=index.d.ts.map

package/dist/index.js

@@ -36,6 +36,50 @@ const IGNORED_DIRECTORIES = new Set([
 const PROXY_OUTPUT_MAX_BYTES = 50 * 1024 * 1024;
 const buildNoReactDependencyError = (directory) => `No React dependency found in ${directory}/package.json. Add "react" to dependencies (or peerDependencies) and re-run.`;
 //#endregion
+//#region src/errors.ts
+var ReactDoctorError = class extends Error {
+	name = "ReactDoctorError";
+	constructor(message, options) {
+		super(message, options);
+		Object.setPrototypeOf(this, new.target.prototype);
+	}
+};
+var ProjectNotFoundError = class extends ReactDoctorError {
+	name = "ProjectNotFoundError";
+	directory;
+	constructor(directory, options) {
+		super(`No React project found in ${directory}. Expected a package.json at the directory root or a nested package.json with a React dependency.`, options);
+		this.directory = directory;
+	}
+};
+var NoReactDependencyError = class extends ReactDoctorError {
+	name = "NoReactDependencyError";
+	directory;
+	constructor(directory, options) {
+		super(buildNoReactDependencyError(directory), options);
+		this.directory = directory;
+	}
+};
+var PackageJsonNotFoundError = class extends ReactDoctorError {
+	name = "PackageJsonNotFoundError";
+	directory;
+	constructor(directory, options) {
+		super(`No package.json found in ${directory}`, options);
+		this.directory = directory;
+	}
+};
+var AmbiguousProjectError = class extends ReactDoctorError {
+	name = "AmbiguousProjectError";
+	directory;
+	candidates;
+	constructor(directory, candidates, options) {
+		super(`Multiple React projects found under ${directory} (${candidates.length} candidates): ${candidates.join(", ")}. Re-run diagnose() with one of those subdirectories, or iterate them yourself.`, options);
+		this.directory = directory;
+		this.candidates = candidates;
+	}
+};
+const isReactDoctorError = (value) => value instanceof ReactDoctorError;
+//#endregion
 //#region src/utils/summarize-diagnostics.ts
 const summarizeDiagnostics = (diagnostics, worstScore = null, worstScoreLabel = null) => {
 	let errorCount = 0;
@@ -770,9 +814,9 @@ const resolveCatalogVersionFromCollection = (catalogs, packageName, catalogRefer
 	for (const namedCatalog of Object.values(catalogs.namedCatalogs)) if (namedCatalog[packageName]) return namedCatalog[packageName];
 	return null;
 };
-const resolveCatalogVersion = (packageJson, packageName, rootDirectory) => {
+const resolveCatalogVersion = (packageJson, packageName, rootDirectory, explicitCatalogReference) => {
 	const rawVersion = collectAllDependencies(packageJson)[packageName];
-	const catalogName = rawVersion ? extractCatalogName(rawVersion) : null;
+	const catalogName = explicitCatalogReference ?? (rawVersion ? extractCatalogName(rawVersion) : null);
 	if (isPlainObject(packageJson.catalog)) {
 		const version = resolveVersionFromCatalog(packageJson.catalog, packageName);
 		if (version) return version;
@@ -789,9 +833,22 @@ const resolveCatalogVersion = (packageJson, packageName, rootDirectory) => {
 		}
 	}
 	const workspaces = packageJson.workspaces;
-	if (workspaces && !Array.isArray(workspaces) && isPlainObject(workspaces.catalog)) {
-		const version = resolveVersionFromCatalog(workspaces.catalog, packageName);
-		if (version) return version;
+	if (workspaces && !Array.isArray(workspaces)) {
+		if (isPlainObject(workspaces.catalog)) {
+			const version = resolveVersionFromCatalog(workspaces.catalog, packageName);
+			if (version) return version;
+		}
+		if (isPlainObject(workspaces.catalogs)) {
+			const namedCatalog = catalogName ? workspaces.catalogs[catalogName] : void 0;
+			if (namedCatalog && isPlainObject(namedCatalog)) {
+				const version = resolveVersionFromCatalog(namedCatalog, packageName);
+				if (version) return version;
+			}
+			for (const catalogEntries of Object.values(workspaces.catalogs)) if (isPlainObject(catalogEntries)) {
+				const version = resolveVersionFromCatalog(catalogEntries, packageName);
+				if (version) return version;
+			}
+		}
 	}
 	if (rootDirectory) {
 		const pnpmVersion = resolveCatalogVersionFromCollection(parsePnpmWorkspaceCatalogs(rootDirectory), packageName, catalogName);
@@ -878,7 +935,8 @@ const findDependencyInfoFromMonorepoRoot = (directory) => {
 	};
 	const rootPackageJson = readPackageJson(monorepoPackageJsonPath);
 	const rootInfo = extractDependencyInfo(rootPackageJson);
-	const catalogVersion = resolveCatalogVersion(rootPackageJson, "react", monorepoRoot);
+	const leafPackageJsonPath = path.join(directory, "package.json");
+	const catalogVersion = resolveCatalogVersion(rootPackageJson, "react", monorepoRoot, isFile(leafPackageJsonPath) ? extractCatalogName(collectAllDependencies(readPackageJson(leafPackageJsonPath)).react ?? "") ?? null : null);
 	const workspaceInfo = findReactInWorkspaces(monorepoRoot, rootPackageJson);
 	return {
 		reactVersion: rootInfo.reactVersion ?? catalogVersion ?? workspaceInfo.reactVersion,
@@ -902,6 +960,45 @@ 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) => REACT_DEPENDENCY_NAMES.has(packageName));
+};
+const discoverReactSubprojects = (rootDirectory) => {
+	if (!fs.existsSync(rootDirectory) || !fs.statSync(rootDirectory).isDirectory()) return [];
+	const packages = [];
+	const rootPackageJsonPath = path.join(rootDirectory, "package.json");
+	if (isFile(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);
+		const packageJsonPath = path.join(subdirectory, "package.json");
+		if (!isFile(packageJsonPath)) continue;
+		const packageJson = readPackageJson(packageJsonPath);
+		if (!hasReactDependency(packageJson)) continue;
+		const name = packageJson.name ?? entry.name;
+		packages.push({
+			name,
+			directory: subdirectory
+		});
+	}
+	return packages;
+};
 const hasCompilerPackage = (packageJson) => {
 	const allDependencies = collectAllDependencies(packageJson);
 	return Object.keys(allDependencies).some((packageName) => REACT_COMPILER_PACKAGES.has(packageName));
@@ -942,15 +1039,16 @@ const discoverProject = (directory) => {
 	const cached = cachedProjectInfos.get(directory);
 	if (cached !== void 0) return cached;
 	const packageJsonPath = path.join(directory, "package.json");
-	if (!isFile(packageJsonPath)) throw new Error(`No package.json found in ${directory}`);
+	if (!isFile(packageJsonPath)) throw new PackageJsonNotFoundError(directory);
 	const packageJson = readPackageJson(packageJsonPath);
 	let { reactVersion, framework } = extractDependencyInfo(packageJson);
-	if (!reactVersion) reactVersion = resolveCatalogVersion(packageJson, "react", directory);
+	const leafCatalogReference = extractCatalogName(collectAllDependencies(packageJson).react ?? "") ?? null;
+	if (!reactVersion) reactVersion = resolveCatalogVersion(packageJson, "react", directory, leafCatalogReference);
 	if (!reactVersion) {
 		const monorepoRoot = findMonorepoRoot(directory);
 		if (monorepoRoot) {
 			const monorepoPackageJsonPath = path.join(monorepoRoot, "package.json");
-			if (isFile(monorepoPackageJsonPath)) reactVersion = resolveCatalogVersion(readPackageJson(monorepoPackageJsonPath), "react", monorepoRoot);
+			if (isFile(monorepoPackageJsonPath)) reactVersion = resolveCatalogVersion(readPackageJson(monorepoPackageJsonPath), "react", monorepoRoot, leafCatalogReference);
 		}
 	}
 	if (!reactVersion || framework === "unknown") {
@@ -996,6 +1094,7 @@ const BOOLEAN_FIELD_NAMES = [
 	"respectInlineDisables",
 	"adoptExistingLintConfig"
 ];
+const STRING_FIELD_NAMES = ["rootDir"];
 const warnConfigField$1 = (message) => {
 	process.stderr.write(`[react-doctor] ${message}\n`);
 };
@@ -1011,6 +1110,10 @@ const coerceMaybeBooleanString = (fieldName, value) => {
 	}
 	warnConfigField$1(`config field "${fieldName}" must be a boolean (got ${typeof value}); ignoring this field.`);
 };
+const validateString = (fieldName, value) => {
+	if (typeof value === "string") return value;
+	warnConfigField$1(`config field "${fieldName}" must be a string (got ${typeof value}); ignoring this field.`);
+};
 const validateConfigTypes = (config) => {
 	const validated = { ...config };
 	for (const fieldName of BOOLEAN_FIELD_NAMES) {
@@ -1020,6 +1123,13 @@ const validateConfigTypes = (config) => {
 		if (coerced === void 0) delete validated[fieldName];
 		else validated[fieldName] = coerced;
 	}
+	for (const fieldName of STRING_FIELD_NAMES) {
+		const original = config[fieldName];
+		if (original === void 0) continue;
+		const validatedString = validateString(fieldName, original);
+		if (validatedString === void 0) delete validated[fieldName];
+		else validated[fieldName] = validatedString;
+	}
 	return validated;
 };
 //#endregion
@@ -1031,7 +1141,10 @@ const loadConfigFromDirectory = (directory) => {
 	if (isFile(configFilePath)) try {
 		const fileContent = fs.readFileSync(configFilePath, "utf-8");
 		const parsed = JSON.parse(fileContent);
-		if (isPlainObject(parsed)) return validateConfigTypes(parsed);
+		if (isPlainObject(parsed)) return {
+			config: validateConfigTypes(parsed),
+			sourceDirectory: directory
+		};
 		logger.warn(`${CONFIG_FILENAME} must be a JSON object, ignoring.`);
 	} catch (error) {
 		logger.warn(`Failed to parse ${CONFIG_FILENAME}: ${error instanceof Error ? error.message : String(error)}`);
@@ -1042,7 +1155,10 @@ const loadConfigFromDirectory = (directory) => {
 		const packageJson = JSON.parse(fileContent);
 		if (isPlainObject(packageJson)) {
 			const embeddedConfig = packageJson[PACKAGE_JSON_CONFIG_KEY];
-			if (isPlainObject(embeddedConfig)) return validateConfigTypes(embeddedConfig);
+			if (isPlainObject(embeddedConfig)) return {
+				config: validateConfigTypes(embeddedConfig),
+				sourceDirectory: directory
+			};
 		}
 	} catch {
 		return null;
@@ -1054,7 +1170,7 @@ const cachedConfigs = /* @__PURE__ */ new Map();
 const clearConfigCache = () => {
 	cachedConfigs.clear();
 };
-const loadConfig = (rootDirectory) => {
+const loadConfigWithSource = (rootDirectory) => {
 	const cached = cachedConfigs.get(rootDirectory);
 	if (cached !== void 0) return cached;
 	const localConfig = loadConfigFromDirectory(rootDirectory);
@@ -1261,7 +1377,7 @@ const findEnclosingMultilineJsxOpenerStart = (lines, diagnosticLineIndex) => {
 };
 //#endregion
 //#region src/utils/find-stacked-disable-comments.ts
-const DISABLE_NEXT_LINE_PATTERN = /(?:\/\/|\/\*)\s*react-doctor-disable-next-line\b(?:\s+([\w/\-.,\s]+?))?\s*(?:\*\/)?\s*\}?\s*$/;
+const DISABLE_NEXT_LINE_PATTERN = /(?:\/\/|\/\*)\s*react-doctor-disable-next-line\b(?:\s+([^\r\n]*?))?\s*(?:\*\/)?\s*\}?\s*$/;
 const findStackedDisableCommentsAbove = (lines, anchorIndex) => {
 	const collected = [];
 	let isStillInChain = true;
@@ -1283,13 +1399,21 @@ const findStackedDisableCommentsAbove = (lines, anchorIndex) => {
 };
 //#endregion
 //#region src/utils/is-rule-listed-in-comment.ts
+const stripDescriptionTail = (ruleList) => {
+	const descriptionMatch = ruleList.match(/(?:^|\s)--\s/);
+	if (!descriptionMatch || descriptionMatch.index === void 0) return ruleList;
+	return ruleList.slice(0, descriptionMatch.index);
+};
 const isRuleListedInComment = (ruleList, ruleId) => {
-	if (!ruleList?.trim()) return true;
-	return ruleList.split(/[,\s]+/).some((token) => token.trim() === ruleId);
+	const trimmed = ruleList?.trim();
+	if (!trimmed) return true;
+	const ruleSection = stripDescriptionTail(trimmed).trim();
+	if (!ruleSection) return true;
+	return ruleSection.split(/[,\s]+/).some((token) => token.trim() === ruleId);
 };
 //#endregion
 //#region src/utils/evaluate-suppression.ts
-const DISABLE_LINE_PATTERN = /(?:\/\/|\/\*)\s*react-doctor-disable-line\b(?:\s+([\w/\-.,\s]+?))?\s*(?:\*\/)?\s*\}?\s*$/;
+const DISABLE_LINE_PATTERN = /(?:\/\/|\/\*)\s*react-doctor-disable-line\b(?:\s+([^\r\n]*?))?\s*(?:\*\/)?\s*\}?\s*$/;
 const formatLineGap = (gapLineCount) => `${gapLineCount} line${gapLineCount === 1 ? "" : "s"}`;
 const hasChainSuppressor = (comments, ruleId) => comments.some((comment) => comment.isInChain && isRuleListedInComment(comment.ruleList, ruleId));
 const findAdjacentRuleListMismatch = (comments, ruleId) => comments.find((comment) => comment.isInChain && Boolean(comment.ruleList?.trim()) && !isRuleListedInComment(comment.ruleList, ruleId));
@@ -1514,6 +1638,31 @@ const createNodeReadFileLinesSync = (rootDirectory) => {
 	};
 };
 //#endregion
+//#region src/utils/resolve-config-root-dir.ts
+const resolveConfigRootDir = (config, configSourceDirectory) => {
+	if (!config || !configSourceDirectory) return null;
+	const rawRootDir = config.rootDir;
+	if (typeof rawRootDir !== "string") return null;
+	const trimmedRootDir = rawRootDir.trim();
+	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()) {
+		logger.warn(`react-doctor config "rootDir" points to "${rawRootDir}" (resolved to ${resolvedRootDir}), which is not a directory. Ignoring.`);
+		return null;
+	}
+	return resolvedRootDir;
+};
+//#endregion
+//#region src/utils/resolve-diagnose-target.ts
+const resolveDiagnoseTarget = (directory) => {
+	if (isFile(path.join(directory, "package.json"))) return directory;
+	const reactSubprojects = discoverReactSubprojects(directory);
+	if (reactSubprojects.length === 0) return null;
+	if (reactSubprojects.length === 1) return reactSubprojects[0].directory;
+	throw new AmbiguousProjectError(directory, reactSubprojects.map((subproject) => path.relative(directory, subproject.directory)));
+};
+//#endregion
 //#region src/utils/resolve-lint-include-paths.ts
 const listSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
@@ -1923,6 +2072,16 @@ const TANSTACK_START_RULES = {
 	"react-doctor/tanstack-start-redirect-in-try-catch": "warn",
 	"react-doctor/tanstack-start-loader-parallel-fetch": "warn"
 };
+const YOU_MIGHT_NOT_NEED_EFFECT_RULES = {
+	"effect/no-derived-state": "warn",
+	"effect/no-chain-state-updates": "warn",
+	"effect/no-event-handler": "warn",
+	"effect/no-adjust-state-on-prop-change": "warn",
+	"effect/no-reset-all-state-on-prop-change": "warn",
+	"effect/no-pass-live-state-to-parent": "warn",
+	"effect/no-pass-data-to-parent": "warn",
+	"effect/no-initialize-state": "warn"
+};
 const REACT_COMPILER_RULES = {
 	"react-hooks-js/set-state-in-render": "error",
 	"react-hooks-js/immutability": "error",
@@ -1967,6 +2126,23 @@ 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}/`;
@@ -2177,6 +2353,11 @@ const filterRulesByReactMajor = (rules, reactMajorVersion) => {
 const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanStackQuery, customRulesOnly = false, reactMajorVersion = null, extendsPaths = [] }) => {
 	const reactHooksJsPlugin = resolveReactHooksJsPlugin(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 jsPlugins = [];
+	if (reactHooksJsPlugin) jsPlugins.push(reactHooksJsPlugin.entry);
+	if (youMightNotNeedEffectPlugin) jsPlugins.push(youMightNotNeedEffectPlugin.entry);
 	return {
 		...extendsPaths.length > 0 ? { extends: extendsPaths } : {},
 		categories: {
@@ -2189,11 +2370,12 @@ const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, hasTanSta
 			nursery: "off"
 		},
 		plugins: customRulesOnly ? [] : ["react", "jsx-a11y"],
-		jsPlugins: reactHooksJsPlugin ? [reactHooksJsPlugin.entry, pluginPath] : [pluginPath],
+		jsPlugins: [...jsPlugins, pluginPath],
 		rules: {
 			...customRulesOnly ? {} : BUILTIN_REACT_RULES,
 			...customRulesOnly ? {} : BUILTIN_A11Y_RULES,
 			...reactCompilerRules,
+			...youMightNotNeedEffectRules,
 			...filterRulesByReactMajor(GLOBAL_REACT_DOCTOR_RULES, reactMajorVersion),
 			...framework === "nextjs" ? NEXTJS_RULES : {},
 			...framework === "expo" || framework === "react-native" ? REACT_NATIVE_RULES : {},
@@ -2307,6 +2489,7 @@ const PLUGIN_CATEGORY_MAP = {
 	"react-doctor": "Other",
 	"jsx-a11y": "Accessibility",
 	knip: "Dead Code",
+	effect: "State & Effects",
 	eslint: "Correctness",
 	oxc: "Correctness",
 	typescript: "Correctness",
@@ -2821,6 +3004,7 @@ const parseOxlintOutput = (stdout) => {
 			severity: diagnostic.severity,
 			message: cleaned.message,
 			help: cleaned.help,
+			url: diagnostic.url,
 			line: primaryLabel?.span.line ?? 0,
 			column: primaryLabel?.span.column ?? 0,
 			category: resolveDiagnosticCategory(plugin, rule)
@@ -3077,12 +3261,16 @@ const settledOrEmpty = (settled, label) => {
 };
 const diagnose = async (directory, options = {}) => {
 	const startTime = globalThis.performance.now();
-	const resolvedDirectory = path.resolve(directory);
-	const userConfig = loadConfig(resolvedDirectory);
+	const requestedDirectory = path.resolve(directory);
+	const initialLoadedConfig = loadConfigWithSource(requestedDirectory);
+	const directoryAfterRedirect = resolveConfigRootDir(initialLoadedConfig?.config ?? null, initialLoadedConfig?.sourceDirectory ?? null) ?? requestedDirectory;
+	const resolvedDirectory = resolveDiagnoseTarget(directoryAfterRedirect);
+	if (!resolvedDirectory) throw new ProjectNotFoundError(directoryAfterRedirect);
+	const userConfig = initialLoadedConfig?.config ?? loadConfigWithSource(resolvedDirectory)?.config ?? null;
 	const includePaths = options.includePaths ?? [];
 	const isDiffMode = includePaths.length > 0;
 	const projectInfo = discoverProject(resolvedDirectory);
-	if (!projectInfo.reactVersion) throw new Error(buildNoReactDependencyError(resolvedDirectory));
+	if (!projectInfo.reactVersion) throw new NoReactDependencyError(resolvedDirectory);
 	const lintIncludePaths = computeJsxIncludePaths(includePaths) ?? resolveLintIncludePaths(resolvedDirectory, userConfig);
 	const readFileLinesSync = createNodeReadFileLinesSync(resolvedDirectory);
 	const effectiveLint = options.lint ?? userConfig?.lint ?? true;
@@ -3125,6 +3313,6 @@ const diagnose = async (directory, options = {}) => {
 	};
 };
 //#endregion
-export { buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, summarizeDiagnostics, toJsonReport };
+export { AmbiguousProjectError, NoReactDependencyError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -69,14 +69,19 @@
   },
   "devDependencies": {
     "@types/prompts": "^2.4.9",
-    "eslint-plugin-react-hooks": "^7.1.1"
+    "eslint-plugin-react-hooks": "^7.1.1",
+    "eslint-plugin-react-you-might-not-need-an-effect": "^0.10.1"
   },
   "peerDependencies": {
-    "eslint-plugin-react-hooks": "^6 || ^7"
+    "eslint-plugin-react-hooks": "^6 || ^7",
+    "eslint-plugin-react-you-might-not-need-an-effect": "^0.10"
   },
   "peerDependenciesMeta": {
     "eslint-plugin-react-hooks": {
       "optional": true
+    },
+    "eslint-plugin-react-you-might-not-need-an-effect": {
+      "optional": true
     }
   },
   "engines": {