react-doctor 0.2.14-dev.6e59f10 → 0.2.14-dev.8b313ba

package/dist/cli.js

@@ -7726,6 +7726,28 @@ const collectIgnorePatterns = (rootDirectory) => {
 	cachedPatternsByRoot.set(rootDirectory, patterns);
 	return patterns;
 };
+/**
+* Resolves a path to its canonical, symlink-free form, falling back to
+* the input when it cannot be realpath'd (broken symlink, permission
+* error) so a best-effort normalization never throws.
+*
+* deslop's dead-code module graph is collected with `fast-glob` (which
+* keeps the scan root's symlinks intact) while imports are resolved
+* through `oxc-resolver` (which returns realpath'd targets). When the
+* project root sits behind a symlink — e.g. macOS iCloud-synced
+* `~/Documents` / `~/Desktop`, or a symlinked checkout — those two path
+* spaces diverge: every resolved import misses the graph and the files
+* they point at (commonly every `@/…` alias target) are mis-reported as
+* unreachable. Canonicalizing the root before the scan keeps both path
+* spaces in agreement.
+*/
+const toCanonicalPath = (filePath) => {
+	try {
+		return fs.realpathSync(filePath);
+	} catch {
+		return filePath;
+	}
+};
 const DEAD_CODE_PLUGIN = "deslop";
 const DEAD_CODE_CATEGORY = "Maintainability";
 const TSCONFIG_FILENAMES$1 = ["tsconfig.json", "tsconfig.base.json"];
@@ -7982,7 +8004,8 @@ const runDeadCodeWorkerWithTimeout = (handle, timeoutMs) => new Promise((resolve
 	});
 });
 const checkDeadCode = async (options) => {
-	const { rootDirectory, userConfig } = options;
+	const { userConfig } = options;
+	const rootDirectory = toCanonicalPath(options.rootDirectory);
 	if (!fs.existsSync(path.join(rootDirectory, "package.json"))) return [];
 	const ignorePatterns = collectDeadCodeIgnorePatterns(rootDirectory, userConfig);
 	const result = parseDeadCodeWorkerResult(await runDeadCodeWorkerWithTimeout((options.createWorker ?? createDeadCodeWorker)({
@@ -10824,6 +10847,20 @@ const groupBy = (items, keyFn) => {
 	}
 	return groups;
 };
+/**
+* Whether a diagnostic's rule has a published per-rule fix recipe at
+* `${PROMPTS_RULES_BASE_URL}/react-doctor/<rule>.md`
+* (see `buildRulePromptUrl`).
+*
+* Recipes are generated from react-doctor's own engine rules, so only
+* those resolve. Dead-code (`deslop`), the synthetic environment and
+* supply-chain checks (`require-reduced-motion`, `require-pnpm-hardening`
+* — `react-doctor`-namespaced but not engine rules), and adopted
+* third-party plugins (`eslint`, `unicorn`, `react-hooks-js`, …) have no
+* recipe, so advertising "fetch the fix recipe" for them sends agents to
+* a 404. Gate the directive on this predicate.
+*/
+const hasPublishedFixRecipe = (diagnostic) => diagnostic.plugin === "react-doctor" && Object.hasOwn(reactDoctorPlugin.rules, diagnostic.rule);
 //#endregion
 //#region ../../node_modules/.pnpm/chalk@5.6.2/node_modules/chalk/source/vendor/ansi-styles/index.js
 const ANSI_BACKGROUND_OFFSET = 10;
@@ -14012,7 +14049,7 @@ const compareByRulePriority = (ruleKeyA, ruleKeyB, rulePriority) => {
 };
 const sortRuleGroupsByImportance = (diagnosticGroups, rulePriority) => diagnosticGroups.toSorted(([ruleKeyA], [ruleKeyB]) => compareByRulePriority(ruleKeyA, ruleKeyB, rulePriority));
 const FETCH_FIX_RECIPE_LABEL = "Fetch & follow the canonical fix recipe before fixing";
-const formatFixRecipeLine = (diagnostic) => `${FETCH_FIX_RECIPE_LABEL}: ${buildRulePromptUrl(diagnostic.plugin, diagnostic.rule)}`;
+const formatFixRecipeLine = (diagnostic) => hasPublishedFixRecipe(diagnostic) ? `${FETCH_FIX_RECIPE_LABEL}: ${buildRulePromptUrl(diagnostic.plugin, diagnostic.rule)}` : null;
 //#endregion
 //#region src/cli/utils/box-text.ts
 const ESCAPE = String.fromCharCode(27);
@@ -14296,7 +14333,8 @@ const formatRuleSummary = (ruleKey, ruleDiagnostics) => {
 	];
 	if (firstDiagnostic.help) sections.push("", `Suggestion: ${firstDiagnostic.help}`);
 	if (firstDiagnostic.url) sections.push("", `Docs: ${firstDiagnostic.url}`);
-	sections.push("", formatFixRecipeLine(firstDiagnostic));
+	const fixRecipeLine = formatFixRecipeLine(firstDiagnostic);
+	if (fixRecipeLine) sections.push("", fixRecipeLine);
 	sections.push("", "Files:");
 	const fileSites = buildVerboseSiteMap(ruleDiagnostics);
 	for (const [filePath, sites] of fileSites) if (sites.length > 0) for (const site of sites) {
@@ -14715,7 +14753,7 @@ const resolveOxlintNodeEffect = (isLintEnabled, isQuiet) => Effect.gen(function*
 const resolveOxlintNode = (isLintEnabled, isQuiet) => Effect.runPromise(resolveOxlintNodeEffect(isLintEnabled, isQuiet).pipe(Effect.provide(NodeResolver.layerNode)));
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.2.14-dev.6e59f10";
+const VERSION = "0.2.14-dev.8b313ba";
 //#endregion
 //#region src/inspect.ts
 const silentConsole = makeNoopConsole();
@@ -15029,7 +15067,9 @@ const buildHandoffPayload = (input) => {
 	topGroups.forEach(([ruleKey, ruleDiagnostics], index) => {
 		const representative = ruleDiagnostics[0];
 		const severityLabel = representative.severity === "error" ? "ERROR" : "WARN";
-		lines.push(`${index + 1}. ${severityLabel} ${representative.category}: ${representative.title ?? ruleKey} (×${ruleDiagnostics.length})`, `   ${representative.message}`, `   ${formatFixRecipeLine(representative)}`);
+		lines.push(`${index + 1}. ${severityLabel} ${representative.category}: ${representative.title ?? ruleKey} (×${ruleDiagnostics.length})`, `   ${representative.message}`);
+		const fixRecipeLine = formatFixRecipeLine(representative);
+		if (fixRecipeLine) lines.push(`   ${fixRecipeLine}`);
 		const uniqueFiles = [...new Set(ruleDiagnostics.map((diagnostic) => diagnostic.filePath))];
 		for (const filePath of uniqueFiles.slice(0, 3)) {
 			const firstSite = ruleDiagnostics.find((diagnostic) => diagnostic.filePath === filePath && diagnostic.line > 0);
@@ -15040,7 +15080,7 @@ const buildHandoffPayload = (input) => {
 	});
 	lines.push("");
 	if (diagnosticsDirectory) lines.push(`Full results for all ${input.diagnostics.length} issues (diagnostics.json + a .txt per rule): ${diagnosticsDirectory}`, "");
-	lines.push("Read each file and fix the root cause — don't suppress. When the top issues are done, run `npx react-doctor@latest --verbose` to verify, then work through the rest from the full results above.");
+	lines.push("Read each file and fix the root cause — don't suppress or silence the rule.", "", "Verify against the real thing, don't assume: confirm each change matches the canonical fix recipe you fetched for that rule, then re-run `npx react-doctor@latest --verbose` and check the issue is actually gone against the real tool before moving on.", "", "Teach me as you go: for every issue you touch, explain it in plain language (no jargon) — what the problem is, why it's a problem, and how serious it is in human terms. Describe the real-world impact and severity concretely (e.g. \"this crashes the page for users on Safari\" vs. \"this is a minor cleanup with no user impact\") so I understand why it matters, not just what changed.", "", "Then work through the rest from the full results above.");
 	return lines.join("\n");
 };
 //#endregion

package/dist/index.js

@@ -4711,6 +4711,28 @@ const collectIgnorePatterns = (rootDirectory) => {
 	cachedPatternsByRoot.set(rootDirectory, patterns);
 	return patterns;
 };
+/**
+* Resolves a path to its canonical, symlink-free form, falling back to
+* the input when it cannot be realpath'd (broken symlink, permission
+* error) so a best-effort normalization never throws.
+*
+* deslop's dead-code module graph is collected with `fast-glob` (which
+* keeps the scan root's symlinks intact) while imports are resolved
+* through `oxc-resolver` (which returns realpath'd targets). When the
+* project root sits behind a symlink — e.g. macOS iCloud-synced
+* `~/Documents` / `~/Desktop`, or a symlinked checkout — those two path
+* spaces diverge: every resolved import misses the graph and the files
+* they point at (commonly every `@/…` alias target) are mis-reported as
+* unreachable. Canonicalizing the root before the scan keeps both path
+* spaces in agreement.
+*/
+const toCanonicalPath = (filePath) => {
+	try {
+		return fs.realpathSync(filePath);
+	} catch {
+		return filePath;
+	}
+};
 const DEAD_CODE_PLUGIN = "deslop";
 const DEAD_CODE_CATEGORY = "Maintainability";
 const TSCONFIG_FILENAMES$1 = ["tsconfig.json", "tsconfig.base.json"];
@@ -4967,7 +4989,8 @@ const runDeadCodeWorkerWithTimeout = (handle, timeoutMs) => new Promise((resolve
 	});
 });
 const checkDeadCode = async (options) => {
-	const { rootDirectory, userConfig } = options;
+	const { userConfig } = options;
+	const rootDirectory = toCanonicalPath(options.rootDirectory);
 	if (!fs.existsSync(path.join(rootDirectory, "package.json"))) return [];
 	const ignorePatterns = collectDeadCodeIgnorePatterns(rootDirectory, userConfig);
 	const result = parseDeadCodeWorkerResult(await runDeadCodeWorkerWithTimeout((options.createWorker ?? createDeadCodeWorker)({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.2.14-dev.6e59f10",
+  "version": "0.2.14-dev.8b313ba",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -59,7 +59,7 @@
     "oxlint": "^1.66.0",
     "prompts": "^2.4.2",
     "typescript": ">=5.0.4 <7",
-    "oxlint-plugin-react-doctor": "0.2.14-dev.6e59f10"
+    "oxlint-plugin-react-doctor": "0.2.14-dev.8b313ba"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",
@@ -67,8 +67,8 @@
     "@xterm/headless": "^6.0.0",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@react-doctor/api": "0.2.14",
-    "@react-doctor/core": "0.2.14"
+    "@react-doctor/core": "0.2.14",
+    "@react-doctor/api": "0.2.14"
   },
   "engines": {
     "node": "^20.19.0 || >=22.12.0"