react-doctor 0.0.40 → 0.0.42

package/dist/index.js

@@ -6,7 +6,6 @@ import { main } from "knip";
 import { createOptions } from "knip/session";
 import os from "node:os";
 import { fileURLToPath } from "node:url";
-
 //#region src/core/build-diagnose-result.ts
 const buildDiagnoseResult = (params) => ({
 	diagnostics: params.diagnostics,
@@ -14,7 +13,6 @@ const buildDiagnoseResult = (params) => ({
 	project: params.project,
 	elapsedMilliseconds: params.elapsedMilliseconds
 });
-
 //#endregion
 //#region src/utils/match-glob-pattern.ts
 const REGEX_SPECIAL_CHARACTERS = /[.+^${}()|[\]\\]/g;
@@ -42,7 +40,6 @@ const compileGlobPattern = (pattern) => {
 	regexSource += "$";
 	return new RegExp(regexSource);
 };
-
 //#endregion
 //#region src/utils/is-ignored-file.ts
 const toRelativePath = (filePath, rootDirectory) => {
@@ -57,7 +54,6 @@ const isFileIgnoredByPatterns = (filePath, rootDirectory, patterns) => {
 	const relativePath = toRelativePath(filePath, rootDirectory);
 	return patterns.some((pattern) => pattern.test(relativePath));
 };
-
 //#endregion
 //#region src/utils/filter-diagnostics.ts
 const resolveCandidateReadPath = (rootDirectory, filePath) => {
@@ -131,13 +127,11 @@ const filterInlineSuppressions = (diagnostics, rootDirectory, readFileLinesSync)
 		return true;
 	});
 };
-
 //#endregion
 //#region src/utils/merge-and-filter-diagnostics.ts
 const mergeAndFilterDiagnostics = (mergedDiagnostics, directory, userConfig, readFileLinesSync) => {
 	return filterInlineSuppressions(userConfig ? filterIgnoredDiagnostics(mergedDiagnostics, userConfig, directory, readFileLinesSync) : mergedDiagnostics, directory, readFileLinesSync);
 };
-
 //#endregion
 //#region src/core/build-result.ts
 const buildDiagnoseTimedResult = async (input) => {
@@ -149,36 +143,35 @@ const buildDiagnoseTimedResult = async (input) => {
 		elapsedMilliseconds
 	};
 };
-
 //#endregion
 //#region src/constants.ts
 const SOURCE_FILE_PATTERN = /\.(tsx?|jsx?)$/;
 const JSX_FILE_PATTERN = /\.(tsx|jsx)$/;
-const ERROR_PREVIEW_LENGTH_CHARS = 200;
-const PERFECT_SCORE = 100;
-const SCORE_GOOD_THRESHOLD = 75;
-const SCORE_OK_THRESHOLD = 50;
 const SCORE_API_URL = "https://www.react.doctor/api/score";
 const FETCH_TIMEOUT_MS = 1e4;
 const GIT_LS_FILES_MAX_BUFFER_BYTES = 50 * 1024 * 1024;
-const SPAWN_ARGS_MAX_LENGTH_CHARS = 24e3;
-const OXLINT_MAX_FILES_PER_BATCH = 500;
 const DEFAULT_BRANCH_CANDIDATES = ["main", "master"];
 const ERROR_RULE_PENALTY = 1.5;
 const WARNING_RULE_PENALTY = .75;
-const MAX_KNIP_RETRIES = 5;
-const GIT_SHOW_MAX_BUFFER_BYTES = 10 * 1024 * 1024;
+const KNIP_CONFIG_LOCATIONS = [
+	"knip.json",
+	"knip.jsonc",
+	".knip.json",
+	".knip.jsonc",
+	"knip.ts",
+	"knip.js",
+	"knip.config.ts",
+	"knip.config.js"
+];
 const IGNORED_DIRECTORIES = new Set([
 	"node_modules",
 	"dist",
 	"build",
 	"coverage"
 ]);
-
 //#endregion
 //#region src/utils/jsx-include-paths.ts
 const computeJsxIncludePaths = (includePaths) => includePaths.length > 0 ? includePaths.filter((filePath) => JSX_FILE_PATTERN.test(filePath)) : void 0;
-
 //#endregion
 //#region src/core/diagnose-core.ts
 const diagnoseCore = async (deps, options = {}) => {
@@ -229,11 +222,9 @@ const diagnoseCore = async (deps, options = {}) => {
 		elapsedMilliseconds: timed.elapsedMilliseconds
 	});
 };
-
 //#endregion
 //#region src/plugin/constants.ts
 const MOTION_LIBRARY_PACKAGES = new Set(["framer-motion", "motion"]);
-
 //#endregion
 //#region src/utils/is-file.ts
 const isFile = (filePath) => {
@@ -243,7 +234,6 @@ const isFile = (filePath) => {
 		return false;
 	}
 };
-
 //#endregion
 //#region src/utils/read-package-json.ts
 const readPackageJson = (packageJsonPath) => {
@@ -258,7 +248,6 @@ const readPackageJson = (packageJsonPath) => {
 		throw error;
 	}
 };
-
 //#endregion
 //#region src/utils/check-reduced-motion.ts
 const REDUCED_MOTION_GREP_PATTERN = "prefers-reduced-motion|useReducedMotion|MotionConfig|reducedMotion";
@@ -300,7 +289,6 @@ const checkReducedMotion = (rootDirectory) => {
 		return [MISSING_REDUCED_MOTION_DIAGNOSTIC];
 	}
 };
-
 //#endregion
 //#region src/utils/find-monorepo-root.ts
 const isMonorepoRoot = (directory) => {
@@ -319,11 +307,9 @@ const findMonorepoRoot = (startDirectory) => {
 	}
 	return null;
 };
-
 //#endregion
 //#region src/utils/is-plain-object.ts
 const isPlainObject = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
-
 //#endregion
 //#region src/utils/discover-project.ts
 const REACT_COMPILER_PACKAGES = new Set([
@@ -665,7 +651,6 @@ const discoverProject = (directory) => {
 		sourceFileCount
 	};
 };
-
 //#endregion
 //#region src/utils/load-config.ts
 const CONFIG_FILENAME = "react-doctor.config.json";
@@ -701,7 +686,6 @@ const loadConfig = (rootDirectory) => {
 	}
 	return null;
 };
-
 //#endregion
 //#region src/utils/read-file-lines-node.ts
 const createNodeReadFileLinesSync = (rootDirectory) => {
@@ -714,7 +698,6 @@ const createNodeReadFileLinesSync = (rootDirectory) => {
 		}
 	};
 };
-
 //#endregion
 //#region src/utils/resolve-lint-include-paths.ts
 const listSourceFilesViaGit = (rootDirectory) => {
@@ -757,12 +740,11 @@ const resolveLintIncludePaths = (rootDirectory, userConfig) => {
 		return !isFileIgnoredByPatterns(filePath, rootDirectory, ignoredPatterns);
 	});
 };
-
 //#endregion
 //#region src/core/calculate-score-locally.ts
 const getScoreLabel = (score) => {
-	if (score >= SCORE_GOOD_THRESHOLD) return "Great";
-	if (score >= SCORE_OK_THRESHOLD) return "Needs work";
+	if (score >= 75) return "Great";
+	if (score >= 50) return "Needs work";
 	return "Critical";
 };
 const countUniqueRules = (diagnostics) => {
@@ -780,7 +762,7 @@ const countUniqueRules = (diagnostics) => {
 };
 const scoreFromRuleCounts = (errorRuleCount, warningRuleCount) => {
 	const penalty = errorRuleCount * ERROR_RULE_PENALTY + warningRuleCount * WARNING_RULE_PENALTY;
-	return Math.max(0, Math.round(PERFECT_SCORE - penalty));
+	return Math.max(0, Math.round(100 - penalty));
 };
 const calculateScoreLocally = (diagnostics) => {
 	const { errorRuleCount, warningRuleCount } = countUniqueRules(diagnostics);
@@ -790,7 +772,6 @@ const calculateScoreLocally = (diagnostics) => {
 		label: getScoreLabel(score)
 	};
 };
-
 //#endregion
 //#region src/core/try-score-from-api.ts
 const parseScoreResult = (value) => {
@@ -822,7 +803,6 @@ const tryScoreFromApi = async (diagnostics, fetchImplementation) => {
 		clearTimeout(timeoutId);
 	}
 };
-
 //#endregion
 //#region src/utils/proxy-fetch.ts
 const getGlobalProcess = () => {
@@ -865,7 +845,6 @@ const proxyFetch = async (url, init) => {
 		clearTimeout(timeoutId);
 	}
 };
-
 //#endregion
 //#region src/utils/calculate-score-node.ts
 const calculateScore = async (diagnostics) => {
@@ -873,7 +852,6 @@ const calculateScore = async (diagnostics) => {
 	if (apiScore) return apiScore;
 	return calculateScoreLocally(diagnostics);
 };
-
 //#endregion
 //#region src/utils/collect-unused-file-paths.ts
 const collectUnusedFilePaths = (filesIssues) => {
@@ -887,7 +865,34 @@ const collectUnusedFilePaths = (filesIssues) => {
 	}
 	return unusedFilePaths;
 };
-
+//#endregion
+//#region src/utils/format-error-chain.ts
+const collectErrorChain = (rootError) => {
+	const errorChain = [];
+	const visitedErrors = /* @__PURE__ */ new Set();
+	let currentError = rootError;
+	while (currentError !== void 0 && !visitedErrors.has(currentError)) {
+		visitedErrors.add(currentError);
+		errorChain.push(currentError);
+		currentError = currentError instanceof Error ? currentError.cause : void 0;
+	}
+	return errorChain;
+};
+const formatErrorMessage = (error) => error instanceof Error ? error.message || error.name : String(error);
+const getErrorChainMessages = (rootError) => collectErrorChain(rootError).map(formatErrorMessage);
+//#endregion
+//#region src/utils/extract-failed-plugin-name.ts
+const PLUGIN_CONFIG_PATTERN = /(?:^|[/\\\s])([a-z][a-z0-9-]*)\.config\./i;
+const extractFailedPluginName = (error) => {
+	for (const errorMessage of getErrorChainMessages(error)) {
+		const pluginNameMatch = errorMessage.match(PLUGIN_CONFIG_PATTERN);
+		if (pluginNameMatch?.[1]) return pluginNameMatch[1].toLowerCase();
+	}
+	return null;
+};
+//#endregion
+//#region src/utils/has-knip-config.ts
+const hasKnipConfig = (directory) => KNIP_CONFIG_LOCATIONS.some((configFilename) => isFile(path.join(directory, configFilename)));
 //#endregion
 //#region src/utils/run-knip.ts
 const KNIP_CATEGORY_MAP = {
@@ -942,12 +947,15 @@ const silenced = async (fn) => {
 		console.error = originalError;
 	}
 };
-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 tryDisableFailedPlugin = (error, parsedConfig, disabledPlugins) => {
+	const failedPlugin = extractFailedPluginName(error);
+	if (!failedPlugin || !(failedPlugin in parsedConfig) || disabledPlugins.has(failedPlugin)) return false;
+	disabledPlugins.add(failedPlugin);
+	parsedConfig[failedPlugin] = false;
+	return true;
+};
 const runKnipWithOptions = async (knipCwd, workspaceName) => {
 	const tsConfigFile = resolveTsConfigFile(knipCwd);
 	const options = await silenced(() => createOptions({
@@ -957,33 +965,36 @@ const runKnipWithOptions = async (knipCwd, workspaceName) => {
 		...tsConfigFile ? { tsConfigFile } : {}
 	}));
 	const parsedConfig = options.parsedConfig;
-	for (let attempt = 0; attempt <= MAX_KNIP_RETRIES; attempt++) try {
+	const disabledPlugins = /* @__PURE__ */ new Set();
+	let lastKnipError;
+	for (let attempt = 0; attempt <= 5; attempt++) try {
 		return await silenced(() => main(options));
 	} catch (error) {
-		const failedPlugin = extractFailedPluginName(error);
-		if (!failedPlugin || attempt === MAX_KNIP_RETRIES) throw error;
-		parsedConfig[failedPlugin] = false;
+		lastKnipError = error;
+		if (!tryDisableFailedPlugin(error, parsedConfig, disabledPlugins)) throw error;
 	}
-	throw new Error("Unreachable");
+	throw lastKnipError;
 };
 const hasNodeModules = (directory) => {
 	const nodeModulesPath = path.join(directory, "node_modules");
 	return fs.existsSync(nodeModulesPath) && fs.statSync(nodeModulesPath).isDirectory();
 };
+const resolveWorkspaceName = (rootDirectory) => {
+	const packageJsonPath = path.join(rootDirectory, "package.json");
+	return (isFile(packageJsonPath) ? readPackageJson(packageJsonPath) : {}).name ?? path.basename(rootDirectory);
+};
+const runKnipForProject = async (rootDirectory, monorepoRoot) => {
+	if (!monorepoRoot || hasKnipConfig(rootDirectory)) return runKnipWithOptions(rootDirectory);
+	try {
+		return await runKnipWithOptions(monorepoRoot, resolveWorkspaceName(rootDirectory));
+	} catch {
+		return runKnipWithOptions(rootDirectory);
+	}
+};
 const runKnip = async (rootDirectory) => {
 	const monorepoRoot = findMonorepoRoot(rootDirectory);
 	if (!(hasNodeModules(rootDirectory) || monorepoRoot !== null && hasNodeModules(monorepoRoot))) return [];
-	let knipResult;
-	if (monorepoRoot) {
-		const packageJsonPath = path.join(rootDirectory, "package.json");
-		const workspaceName = (isFile(packageJsonPath) ? JSON.parse(fs.readFileSync(packageJsonPath, "utf-8")) : {}).name ?? path.basename(rootDirectory);
-		try {
-			knipResult = await runKnipWithOptions(monorepoRoot, workspaceName);
-		} catch {
-			knipResult = await runKnipWithOptions(rootDirectory);
-		}
-	} else knipResult = await runKnipWithOptions(rootDirectory);
-	const { issues } = knipResult;
+	const { issues } = await runKnipForProject(rootDirectory, monorepoRoot);
 	const diagnostics = [];
 	for (const unusedFilePath of collectUnusedFilePaths(issues.files)) diagnostics.push({
 		filePath: path.relative(rootDirectory, unusedFilePath),
@@ -1004,7 +1015,6 @@ const runKnip = async (rootDirectory) => {
 	]) diagnostics.push(...collectIssueRecords(issues[issueType], issueType, rootDirectory));
 	return diagnostics;
 };
-
 //#endregion
 //#region src/oxlint-config.ts
 const esmRequire$1 = createRequire(import.meta.url);
@@ -1188,7 +1198,6 @@ const createOxlintConfig = ({ pluginPath, framework, hasReactCompiler, customRul
 		...framework === "tanstack-start" ? TANSTACK_START_RULES : {}
 	}
 });
-
 //#endregion
 //#region src/utils/neutralize-disable-directives.ts
 const findFilesWithDisableDirectives = (rootDirectory, includePaths) => {
@@ -1231,7 +1240,6 @@ const neutralizeDisableDirectives = (rootDirectory, includePaths) => {
 		for (const [absolutePath, originalContent] of originalContents) fs.writeFileSync(absolutePath, originalContent);
 	};
 };
-
 //#endregion
 //#region src/utils/run-oxlint.ts
 const esmRequire = createRequire(import.meta.url);
@@ -1491,8 +1499,8 @@ const batchIncludePaths = (baseArgs, includePaths) => {
 	let currentBatchLength = baseArgsLength;
 	for (const filePath of includePaths) {
 		const entryLength = filePath.length + 1;
-		const exceedsArgLength = currentBatch.length > 0 && currentBatchLength + entryLength > SPAWN_ARGS_MAX_LENGTH_CHARS;
-		const exceedsFileCount = currentBatch.length >= OXLINT_MAX_FILES_PER_BATCH;
+		const exceedsArgLength = currentBatch.length > 0 && currentBatchLength + entryLength > 24e3;
+		const exceedsFileCount = currentBatch.length >= 500;
 		if (exceedsArgLength || exceedsFileCount) {
 			batches.push(currentBatch);
 			currentBatch = [];
@@ -1536,7 +1544,7 @@ const parseOxlintOutput = (stdout) => {
 	try {
 		output = JSON.parse(stdout);
 	} catch {
-		throw new Error(`Failed to parse oxlint output: ${stdout.slice(0, ERROR_PREVIEW_LENGTH_CHARS)}`);
+		throw new Error(`Failed to parse oxlint output: ${stdout.slice(0, 200)}`);
 	}
 	return output.diagnostics.filter((diagnostic) => diagnostic.code && JSX_FILE_PATTERN.test(diagnostic.filename)).map((diagnostic) => {
 		const { plugin, rule } = parseRuleCode(diagnostic.code);
@@ -1587,7 +1595,6 @@ const runOxlint = async (rootDirectory, hasTypeScript, framework, hasReactCompil
 		if (fs.existsSync(configPath)) fs.unlinkSync(configPath);
 	}
 };
-
 //#endregion
 //#region src/utils/get-diff-files.ts
 const getCurrentBranch = (directory) => {
@@ -1667,7 +1674,6 @@ const getDiffInfo = (directory, explicitBaseBranch) => {
 	};
 };
 const filterSourceFiles = (filePaths) => filePaths.filter((filePath) => SOURCE_FILE_PATTERN.test(filePath));
-
 //#endregion
 //#region src/index.ts
 const diagnose = async (directory, options = {}) => {
@@ -1692,7 +1698,7 @@ const diagnose = async (directory, options = {}) => {
 		lintIncludePaths
 	});
 };
-
 //#endregion
 export { diagnose, filterSourceFiles, getDiffInfo };
+
 //# sourceMappingURL=index.js.map