react-doctor 0.0.19 → 0.0.20

package/dist/cli.js

@@ -765,6 +765,46 @@ const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler }) => ({
 	}
 });
 
+//#endregion
+//#region src/utils/neutralize-disable-directives.ts
+const findFilesWithDisableDirectives = (rootDirectory) => {
+	const result = spawnSync("git", [
+		"grep",
+		"-l",
+		"--untracked",
+		"-E",
+		"(eslint|oxlint)-disable"
+	], {
+		cwd: rootDirectory,
+		encoding: "utf-8",
+		maxBuffer: GIT_LS_FILES_MAX_BUFFER_BYTES
+	});
+	if (result.error || result.status === null) return [];
+	return result.stdout.split("\n").filter((filePath) => filePath.length > 0 && SOURCE_FILE_PATTERN.test(filePath));
+};
+const neutralizeContent = (content) => content.replaceAll("eslint-disable", "eslint_disable").replaceAll("oxlint-disable", "oxlint_disable");
+const neutralizeDisableDirectives = (rootDirectory) => {
+	const filePaths = findFilesWithDisableDirectives(rootDirectory);
+	const originalContents = /* @__PURE__ */ new Map();
+	for (const relativePath of filePaths) {
+		const absolutePath = path.join(rootDirectory, relativePath);
+		let originalContent;
+		try {
+			originalContent = fs.readFileSync(absolutePath, "utf-8");
+		} catch {
+			continue;
+		}
+		const neutralizedContent = neutralizeContent(originalContent);
+		if (neutralizedContent !== originalContent) {
+			originalContents.set(absolutePath, originalContent);
+			fs.writeFileSync(absolutePath, neutralizedContent);
+		}
+	}
+	return () => {
+		for (const [absolutePath, originalContent] of originalContents) fs.writeFileSync(absolutePath, originalContent);
+	};
+};
+
 //#endregion
 //#region src/utils/run-oxlint.ts
 const esmRequire = createRequire(import.meta.url);
@@ -832,7 +872,7 @@ const RULE_CATEGORY_MAP = {
 	"react-doctor/async-parallel": "Performance"
 };
 const RULE_HELP_MAP = {
-	"no-derived-state-effect": "Compute during render: `const derived = computeFrom(dep1, dep2)` — no useEffect needed",
+	"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-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",
@@ -934,6 +974,7 @@ const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompil
 		framework,
 		hasReactCompiler
 	});
+	const restoreDisableDirectives = neutralizeDisableDirectives(rootDirectory);
 	try {
 		fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
 		const args = [
@@ -989,6 +1030,7 @@ const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompil
 			};
 		});
 	} finally {
+		restoreDisableDirectives();
 		if (fs.existsSync(configPath)) fs.unlinkSync(configPath);
 	}
 };
@@ -1266,7 +1308,10 @@ const scan = async (directory, inputOptions = {}) => {
 			return lintDiagnostics;
 		} catch (error) {
 			lintSpinner?.fail("Lint checks failed (non-fatal, skipping).");
-			logger.error(String(error));
+			if (error instanceof Error) {
+				logger.error(error.message);
+				if (error.stack) logger.dim(error.stack);
+			} else logger.error(String(error));
 			return [];
 		}
 	})() : Promise.resolve([]);
@@ -1614,7 +1659,7 @@ const maybePromptSkillInstall = async (shouldSkipPrompts) => {
 
 //#endregion
 //#region src/cli.ts
-const VERSION = "0.0.19";
+const VERSION = "0.0.20";
 process.on("SIGINT", () => process.exit(0));
 process.on("SIGTERM", () => process.exit(0));
 const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isScoreOnly) => {