react-doctor 0.2.8 → 0.2.9

package/dist/{cli-logger-C35LXalM.js → cli-logger-BliQX9s8.js}

@@ -2045,6 +2045,14 @@ var PackageJsonNotFoundError = class extends Error {
 		this.directory = directory;
 	}
 };
+var NotADirectoryError = class extends Error {
+	name = "NotADirectoryError";
+	resolvedPath;
+	constructor(resolvedPath, options) {
+		super(`Resolved scan target "${resolvedPath}" is not a directory. Ensure the path exists and points to a project directory, not a file.`, options);
+		this.resolvedPath = resolvedPath;
+	}
+};
 var AmbiguousProjectError = class extends Error {
 	name = "AmbiguousProjectError";
 	directory;
@@ -4132,8 +4140,10 @@ const resolveScanTarget = (requestedDirectory) => {
 	const configSourceDirectory = loadedConfig?.sourceDirectory ?? null;
 	const redirectedDirectory = resolveConfigRootDir(userConfig, configSourceDirectory);
 	const directoryAfterRedirect = redirectedDirectory ?? absoluteRequested;
+	const resolvedDirectory = resolveDiagnoseTarget(directoryAfterRedirect) ?? directoryAfterRedirect;
+	if (!isDirectory(resolvedDirectory)) throw existsSync(resolvedDirectory) ? new NotADirectoryError(resolvedDirectory) : new ProjectNotFoundError(resolvedDirectory);
 	return {
-		resolvedDirectory: resolveDiagnoseTarget(directoryAfterRedirect) ?? directoryAfterRedirect,
+		resolvedDirectory,
 		requestedDirectory: absoluteRequested,
 		userConfig,
 		configSourceDirectory,
@@ -4614,7 +4624,7 @@ var Files = class Files extends Context.Service()("react-doctor/Files") {
 	* pattern in react-doctor-evals' test layers.
 	*/
 	static layerInMemory = (tree) => {
-		const resolveAbsolute = (filePath, rootDirectory) => Path.isAbsolute(filePath) ? filePath : Path.join(rootDirectory, filePath);
+		const resolveAbsolute = (filePath, rootDirectory) => Path.isAbsolute(filePath) ? filePath : `${rootDirectory}/${filePath}`;
 		return Layer.succeed(Files, Files.of({
 			readLines: (input) => Effect.sync(() => {
 				const absolute = resolveAbsolute(input.filePath, input.rootDirectory);
@@ -4622,17 +4632,17 @@ var Files = class Files extends Context.Service()("react-doctor/Files") {
 				return content === void 0 ? null : content.split("\n");
 			}),
 			listSourceFiles: (rootDirectory) => Effect.sync(() => {
-				const prefix = rootDirectory.endsWith(Path.sep) ? rootDirectory : `${rootDirectory}${Path.sep}`;
+				const prefix = rootDirectory.endsWith("/") ? rootDirectory : `${rootDirectory}/`;
 				const files = [];
 				for (const absolute of tree.keys()) {
 					if (!absolute.startsWith(prefix)) continue;
-					files.push(absolute.slice(prefix.length).split(Path.sep).join("/"));
+					files.push(absolute.slice(prefix.length));
 				}
 				return files;
 			}),
 			isFile: (filePath) => Effect.sync(() => tree.has(filePath)),
 			isDirectory: (filePath) => Effect.sync(() => {
-				const prefix = filePath.endsWith(Path.sep) ? filePath : `${filePath}${Path.sep}`;
+				const prefix = filePath.endsWith("/") ? filePath : `${filePath}/`;
 				for (const absolute of tree.keys()) if (absolute.startsWith(prefix)) return true;
 				return false;
 			})
@@ -5769,7 +5779,7 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 		const primaryLabel = diagnostic.labels[0];
 		const cleaned = cleanDiagnosticMessage(diagnostic.message, diagnostic.help, plugin, rule, project);
 		return {
-			filePath: diagnostic.filename,
+			filePath: diagnostic.filename.replaceAll("\\", "/"),
 			plugin,
 			rule,
 			severity: diagnostic.severity,
@@ -7046,4 +7056,4 @@ const cliLogger = {
 //#endregion
 export { isReactDoctorError as A, filterSourceFiles as C, groupBy as D, getDiffInfo as E, runInspect as F, toRelativePath as I, listWorkspacePackages as M, resolveScanTarget as N, highlighter as O, restoreLegacyThrow as P, filterDiagnosticsForSurface as S, formatReactDoctorError as T, Score as _, DeadCode as a, buildJsonReportError as b, LintPartialFailures as c, OXLINT_NODE_REQUIREMENT as d, Progress as f, SKILL_NAME as g, SHARE_BASE_URL as h, Config as i, layerOtlp as j, isMonorepoRoot as k, Linter as l, Reporter as m, cli_logger_exports as n, Files as o, Project as p, CANONICAL_GITHUB_URL as r, Git as s, cliLogger as t, NodeResolver as u, StagedFiles as v, formatErrorChain as w, discoverReactSubprojects as x, buildJsonReport as y };
 
-//# sourceMappingURL=cli-logger-C35LXalM.js.map
+//# sourceMappingURL=cli-logger-BliQX9s8.js.map

package/dist/cli.js

@@ -1,5 +1,5 @@
 import { i as __toESM, n as __exportAll, r as __require, t as __commonJSMin } from "./rolldown-runtime-uZX_iqCz.js";
-import { A as isReactDoctorError, C as filterSourceFiles, D as groupBy, E as getDiffInfo, F as runInspect, I as toRelativePath, M as listWorkspacePackages, N as resolveScanTarget, O as highlighter, P as restoreLegacyThrow, S as filterDiagnosticsForSurface, T as formatReactDoctorError, _ as Score, a as DeadCode, b as buildJsonReportError, c as LintPartialFailures, d as OXLINT_NODE_REQUIREMENT, f as Progress, g as SKILL_NAME, h as SHARE_BASE_URL, i as Config, j as layerOtlp, k as isMonorepoRoot, l as Linter, m as Reporter, o as Files, p as Project, r as CANONICAL_GITHUB_URL, s as Git, t as cliLogger, u as NodeResolver, v as StagedFiles, w as formatErrorChain, x as discoverReactSubprojects, y as buildJsonReport } from "./cli-logger-C35LXalM.js";
+import { A as isReactDoctorError, C as filterSourceFiles, D as groupBy, E as getDiffInfo, F as runInspect, I as toRelativePath, M as listWorkspacePackages, N as resolveScanTarget, O as highlighter, P as restoreLegacyThrow, S as filterDiagnosticsForSurface, T as formatReactDoctorError, _ as Score, a as DeadCode, b as buildJsonReportError, c as LintPartialFailures, d as OXLINT_NODE_REQUIREMENT, f as Progress, g as SKILL_NAME, h as SHARE_BASE_URL, i as Config, j as layerOtlp, k as isMonorepoRoot, l as Linter, m as Reporter, o as Files, p as Project, r as CANONICAL_GITHUB_URL, s as Git, t as cliLogger, u as NodeResolver, v as StagedFiles, w as formatErrorChain, x as discoverReactSubprojects, y as buildJsonReport } from "./cli-logger-BliQX9s8.js";
 import { createRequire } from "node:module";
 import { execFileSync, execSync } from "node:child_process";
 import path, { join } from "node:path";
@@ -6323,6 +6323,12 @@ const colorizeByScore = (text, score) => {
 //#region src/cli/utils/render-score-header.ts
 const SCORE_BAR_ANIMATION_FRAME_COUNT = 40;
 const SCORE_BAR_ANIMATION_FRAME_DELAY_MS = 50;
+const PERFECT_SCORE_RAINBOW_FRAME_COUNT = 16;
+const PERFECT_SCORE_RAINBOW_FRAME_DELAY_MS = 50;
+const RAINBOW_HUE_SHIFT_PER_FRAME = 9;
+const RAINBOW_GRADIENT_WIDTH = 80;
+const RAINBOW_OKLCH_LIGHTNESS = .638;
+const RAINBOW_OKLCH_CHROMA = .129;
 const easeOutCubic = (progress) => 1 - (1 - progress) ** 3;
 const sleep = (milliseconds) => Effect.promise(() => new Promise((resolve) => setTimeout(resolve, milliseconds)));
 const buildScoreBarSegments = (filledCount) => {
@@ -6333,6 +6339,36 @@ const buildScoreBarSegments = (filledCount) => {
 	};
 };
 const getFilledCount = (score) => Math.round(score / 100 * 50);
+const joinScoreHeaderFrame = (lines) => `${lines[0]}\n\r${lines[1]}\n\r${lines[2]}\n\r${lines[3]}\n`;
+const buildRawScoreBar = (displayScore) => {
+	const { filledSegment, emptySegment } = buildScoreBarSegments(getFilledCount(displayScore));
+	return filledSegment + emptySegment;
+};
+const buildScoreHeaderLine = (faceLine, rightColumnContent) => {
+	return `  ${faceLine}${rightColumnContent.length > 0 ? "  " : ""}${rightColumnContent}`;
+};
+const getRightColumnOffset = (faceLine) => `  ${faceLine}  `.length;
+const clampColorChannel = (value) => Math.max(0, Math.min(255, Math.round(value)));
+const encodeSrgb = (value) => value <= .0031308 ? value * 12.92 : 1.055 * value ** (1 / 2.4) - .055;
+const oklchToRgb = (lightness, chroma, hue) => {
+	const hueRadians = hue * Math.PI / 180;
+	const labA = chroma * Math.cos(hueRadians);
+	const labB = chroma * Math.sin(hueRadians);
+	const longCone = (lightness + .3963377774 * labA + .2158037573 * labB) ** 3;
+	const mediumCone = (lightness - .1055613458 * labA - .0638541728 * labB) ** 3;
+	const shortCone = (lightness - .0894841775 * labA - 1.291485548 * labB) ** 3;
+	return {
+		red: clampColorChannel(encodeSrgb(4.0767416621 * longCone - 3.3077115913 * mediumCone + .2309699292 * shortCone) * 255),
+		green: clampColorChannel(encodeSrgb(-1.2684380046 * longCone + 2.6097574011 * mediumCone - .3413193965 * shortCone) * 255),
+		blue: clampColorChannel(encodeSrgb(-.0041960863 * longCone - .7034186147 * mediumCone + 1.707614701 * shortCone) * 255)
+	};
+};
+const colorizeTrueColor = (text, { red, green, blue }) => `\x1b[38;2;${red};${green};${blue}m${text}\x1b[39m`;
+const colorizeRainbowText = (text, frame, offset = 0) => [...text].map((character, index) => {
+	if (character === " ") return character;
+	return colorizeTrueColor(character, oklchToRgb(RAINBOW_OKLCH_LIGHTNESS, RAINBOW_OKLCH_CHROMA, ((index + offset) / RAINBOW_GRADIENT_WIDTH * 360 + frame * RAINBOW_HUE_SHIFT_PER_FRAME) % 360));
+}).join("");
+const buildRainbowHeaderLine = (faceLine, rightColumnContent, frame) => colorizeRainbowText(buildScoreHeaderLine(faceLine, rightColumnContent), frame);
 const buildScoreBar = (displayScore, colorScore = displayScore) => {
 	const { filledSegment, emptySegment } = buildScoreBarSegments(getFilledCount(displayScore));
 	return colorizeByScore(filledSegment, colorScore) + highlighter.dim(emptySegment);
@@ -6343,18 +6379,19 @@ const getDoctorFace = (score) => {
 	return ["x x", " ▽ "];
 };
 const BRANDING_LINE = `React Doctor ${highlighter.dim("(https://react.doctor)")}`;
-const buildFaceRenderedLines = (score) => {
+const RAW_BRANDING_LINE = "React Doctor (https://react.doctor)";
+const buildRawFaceLines = (score) => {
 	const [eyes, mouth] = getDoctorFace(score);
-	const colorize = (text) => colorizeByScore(text, score);
 	return [
 		"┌─────┐",
 		`│ ${eyes} │`,
 		`│ ${mouth} │`,
 		"└─────┘"
-	].map(colorize);
+	];
 };
-const buildScoreHeaderLine = (faceLine, rightColumnContent) => {
-	return `  ${faceLine}${rightColumnContent.length > 0 ? "  " : ""}${rightColumnContent}`;
+const buildFaceRenderedLines = (score) => {
+	const colorize = (text) => colorizeByScore(text, score);
+	return buildRawFaceLines(score).map(colorize);
 };
 const writeScoreHeaderLine = (line) => Effect.sync(() => {
 	process.stdout.write(line);
@@ -6365,29 +6402,97 @@ const buildScoreLine = (displayScore, finalScore, label, projectName) => {
 	const projectSuffix = projectName ? ` ${highlighter.dim("·")} ${highlighter.dim(projectName)}` : "";
 	return `${scoreNumber} ${highlighter.dim(`/ 100`)} ${scoreLabel}${projectSuffix}`;
 };
+const buildRawScoreLine = (displayScore, label, projectName) => {
+	return `${displayScore} / 100 ${label}${projectName ? ` · ${projectName}` : ""}`;
+};
+const buildRainbowScoreHeaderFrame = ({ score, displayScore, label, frame, projectName }) => {
+	const rawFaceLines = buildRawFaceLines(score);
+	return joinScoreHeaderFrame([
+		buildRainbowHeaderLine(rawFaceLines[0] ?? "", buildRawScoreLine(displayScore, label, projectName), frame),
+		buildRainbowHeaderLine(rawFaceLines[1] ?? "", buildRawScoreBar(displayScore), frame),
+		buildRainbowHeaderLine(rawFaceLines[2] ?? "", RAW_BRANDING_LINE, frame),
+		buildRainbowHeaderLine(rawFaceLines[3] ?? "", "", frame)
+	]);
+};
+const buildFinalPerfectScoreHeaderFrame = (score, label, frame, projectName) => {
+	const rawFaceLines = buildRawFaceLines(score);
+	const renderedFaceLines = buildFaceRenderedLines(score);
+	const rainbowBarLine = colorizeRainbowText(buildRawScoreBar(score), frame, getRightColumnOffset(rawFaceLines[1] ?? ""));
+	return joinScoreHeaderFrame([
+		buildScoreHeaderLine(renderedFaceLines[0] ?? "", buildScoreLine(score, score, label, projectName)),
+		buildScoreHeaderLine(renderedFaceLines[1] ?? "", rainbowBarLine),
+		buildScoreHeaderLine(renderedFaceLines[2] ?? "", BRANDING_LINE),
+		buildScoreHeaderLine(renderedFaceLines[3] ?? "", "")
+	]);
+};
+const buildInitialScoreHeaderLine = ({ isPerfectScore, shouldAnimate, lineIndex, renderedFaceLine, rawFaceLine, rightColumnContent, rawRightColumnContent, score }) => {
+	if (!isPerfectScore) return buildScoreHeaderLine(renderedFaceLine, rightColumnContent);
+	if (shouldAnimate) return buildRainbowHeaderLine(rawFaceLine, rawRightColumnContent, 0);
+	if (lineIndex !== 1) return buildScoreHeaderLine(renderedFaceLine, rightColumnContent);
+	return buildScoreHeaderLine(renderedFaceLine, colorizeRainbowText(buildRawScoreBar(score), 0, getRightColumnOffset(rawFaceLine)));
+};
 const printAnimatedScore = (scoreFaceLine, barFaceLine, score, label, projectName) => Effect.gen(function* () {
+	const isPerfectScore = score === 100;
 	for (let frame = 0; frame <= SCORE_BAR_ANIMATION_FRAME_COUNT; frame += 1) {
 		const progress = easeOutCubic(frame / SCORE_BAR_ANIMATION_FRAME_COUNT);
 		const animatedScore = Math.round(score * progress);
+		if (isPerfectScore) {
+			yield* writeScoreHeaderLine(`${frame === 0 ? "" : "\x1B[4A"}\r${buildRainbowScoreHeaderFrame({
+				score,
+				displayScore: animatedScore,
+				label,
+				frame,
+				projectName
+			})}`);
+			if (frame < SCORE_BAR_ANIMATION_FRAME_COUNT) yield* sleep(SCORE_BAR_ANIMATION_FRAME_DELAY_MS);
+			continue;
+		}
 		const animatedScoreLine = buildScoreLine(animatedScore, score, label, projectName);
 		const animatedBarLine = buildScoreBar(animatedScore, score);
 		yield* writeScoreHeaderLine(`${frame === 0 ? "" : "\x1B[2A"}\r${buildScoreHeaderLine(scoreFaceLine, animatedScoreLine)}\n\r${buildScoreHeaderLine(barFaceLine, animatedBarLine)}\n`);
 		if (frame < SCORE_BAR_ANIMATION_FRAME_COUNT) yield* sleep(SCORE_BAR_ANIMATION_FRAME_DELAY_MS);
 	}
+	if (!isPerfectScore) return;
+	for (let frame = 0; frame < PERFECT_SCORE_RAINBOW_FRAME_COUNT; frame += 1) {
+		yield* writeScoreHeaderLine(`\x1b[4A\r${buildRainbowScoreHeaderFrame({
+			score,
+			displayScore: score,
+			label,
+			frame,
+			projectName
+		})}`);
+		yield* sleep(PERFECT_SCORE_RAINBOW_FRAME_DELAY_MS);
+	}
+	yield* writeScoreHeaderLine(`\x1b[4A\r${buildFinalPerfectScoreHeaderFrame(score, label, PERFECT_SCORE_RAINBOW_FRAME_COUNT, projectName)}\x1b[2A`);
 });
 const printScoreHeader = (scoreResult, projectName) => Effect.gen(function* () {
+	const isPerfectScore = scoreResult.score === 100;
 	const renderedFaceLines = buildFaceRenderedLines(scoreResult.score);
+	const rawFaceLines = buildRawFaceLines(scoreResult.score);
 	const shouldAnimate = !isSpinnerSilent() && isSpinnerInteractive(process.stdout);
+	const displayScore = shouldAnimate ? 0 : scoreResult.score;
 	const rightColumnLines = [
-		buildScoreLine(shouldAnimate ? 0 : scoreResult.score, scoreResult.score, scoreResult.label, projectName),
+		buildScoreLine(displayScore, scoreResult.score, scoreResult.label, projectName),
 		shouldAnimate ? buildScoreBar(0, scoreResult.score) : buildScoreBar(scoreResult.score),
 		BRANDING_LINE,
 		""
 	];
-	for (let lineIndex = 0; lineIndex < renderedFaceLines.length; lineIndex += 1) {
-		const rightColumnContent = rightColumnLines[lineIndex] ?? "";
-		yield* Console.log(buildScoreHeaderLine(renderedFaceLines[lineIndex], rightColumnContent));
-	}
+	const rawRightColumnLines = [
+		buildRawScoreLine(displayScore, scoreResult.label, projectName),
+		buildRawScoreBar(displayScore),
+		RAW_BRANDING_LINE,
+		""
+	];
+	for (let lineIndex = 0; lineIndex < renderedFaceLines.length; lineIndex += 1) yield* Console.log(buildInitialScoreHeaderLine({
+		isPerfectScore,
+		shouldAnimate,
+		lineIndex,
+		renderedFaceLine: renderedFaceLines[lineIndex] ?? "",
+		rawFaceLine: rawFaceLines[lineIndex] ?? "",
+		rightColumnContent: rightColumnLines[lineIndex] ?? "",
+		rawRightColumnContent: rawRightColumnLines[lineIndex] ?? "",
+		score: scoreResult.score
+	}));
 	yield* Console.log("");
 	if (shouldAnimate) {
 		yield* writeScoreHeaderLine("\x1B[5A");
@@ -6561,7 +6666,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.8";
+const VERSION = "0.2.9";
 //#endregion
 //#region src/inspect.ts
 const silentConsole = makeNoopConsole();
@@ -7362,7 +7467,7 @@ const warnSetupPromptFailure = async (options, error) => {
 		return;
 	}
 	try {
-		const { cliLogger } = await import("./cli-logger-C35LXalM.js").then((n) => n.n);
+		const { cliLogger } = await import("./cli-logger-BliQX9s8.js").then((n) => n.n);
 		cliLogger.warn(message);
 	} catch {}
 };
@@ -8646,7 +8751,8 @@ const defaultInstallDependencyRunner = (input) => {
 		env: {
 			...process.env,
 			REACT_DOCTOR_INSTALL: "1"
-		}
+		},
+		shell: process.platform === "win32"
 	});
 };
 const installReactDoctorDependency = async (options) => {

package/dist/index.d.ts

@@ -355,8 +355,14 @@ interface DiagnoseResult {
   skippedCheckReasons?: Record<string, string>;
   project: ProjectInfo;
   elapsedMilliseconds: number;
-} //#endregion
-//#region src/types/handle-error.d.ts
+}
+/**
+ * A single project to scan as part of a `diagnoseProjects()` batch.
+ * Scan options (`deadCode`, `lint`, etc.) are flat on the entry and
+ * layer on top of the global defaults — omitted fields fall through.
+ * `config` is a full `ReactDoctorConfig` override that replaces the
+ * on-disk `react-doctor.config.json` for this project's scan.
+ */
 //#endregion
 //#region src/types/inspect.d.ts
 interface InspectResult {
@@ -492,13 +498,18 @@ declare class PackageJsonNotFoundError extends Error {
   readonly directory: string;
   constructor(directory: string, options?: ErrorOptions);
 }
+declare class NotADirectoryError extends Error {
+  readonly name = "NotADirectoryError";
+  readonly resolvedPath: string;
+  constructor(resolvedPath: string, options?: ErrorOptions);
+}
 declare class AmbiguousProjectError extends Error {
   readonly name = "AmbiguousProjectError";
   readonly directory: string;
   readonly candidates: readonly string[];
   constructor(directory: string, candidates: readonly string[], options?: ErrorOptions);
 }
-declare const isProjectDiscoveryError: (value: unknown) => value is ProjectNotFoundError | NoReactDependencyError | PackageJsonNotFoundError | AmbiguousProjectError; //#endregion
+declare const isProjectDiscoveryError: (value: unknown) => value is ProjectNotFoundError | NoReactDependencyError | PackageJsonNotFoundError | NotADirectoryError | AmbiguousProjectError; //#endregion
 //#region src/project-info/utils/is-directory.d.ts
 //#endregion
 //#region src/errors.d.ts
@@ -651,7 +662,41 @@ declare const summarizeDiagnostics: (diagnostics: Diagnostic[], worstScore?: num
 //#endregion
 //#region ../api/dist/index.d.ts
 //#region src/diagnose.d.ts
-declare const diagnose: (directory: string, options?: DiagnoseOptions) => Promise<DiagnoseResult>; //#endregion
+declare const diagnose: (directory: string, options?: DiagnoseOptions) => Promise<DiagnoseResult>;
+/**
+ * Scan multiple projects in parallel and return per-project scores,
+ * diagnostics, and an aggregate score (worst-of across all projects).
+ *
+ * Each project runs its own independent `runInspect` pipeline — the
+ * same pipeline `diagnose()` uses — so per-project config overrides,
+ * dead-code analysis, and scoring all work identically to a single
+ * `diagnose()` call.
+ *
+ * Projects that fail (e.g. missing `package.json`, no React dependency)
+ * are included in the result with `ok: false` rather than aborting the
+ * entire batch, so callers always receive partial results.
+ *
+ * ```ts
+ * const result = await diagnoseProjects({
+ *   projects: [
+ *     { directory: "packages/app" },
+ *     { directory: "packages/shared", deadCode: false },
+ *     { directory: "packages/admin", config: {
+ *       rules: { "react-doctor/no-array-index-as-key": "off" },
+ *     }},
+ *   ],
+ *   concurrency: 4,
+ * });
+ *
+ * for (const project of result.projects) {
+ *   if (project.ok) {
+ *     console.log(project.directory, project.score);
+ *   } else {
+ *     console.error(project.directory, project.error);
+ *   }
+ * }
+ * ```
+ */
 //#endregion
 //#region src/index.d.ts
 declare const clearCaches: () => void;
@@ -662,5 +707,5 @@ interface ToJsonReportOptions {
 }
 declare const toJsonReport: (result: DiagnoseResult, options: ToJsonReportOptions) => JsonReport;
 //#endregion
-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, isProjectDiscoveryError, isReactDoctorError, 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, NotADirectoryError, PackageJsonNotFoundError, type ProjectInfo, ProjectNotFoundError, type ReactDoctorConfig, ReactDoctorError, type ScoreResult, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 //# sourceMappingURL=index.d.ts.map

package/dist/index.js

@@ -2067,6 +2067,14 @@ var PackageJsonNotFoundError = class extends Error {
 		this.directory = directory;
 	}
 };
+var NotADirectoryError = class extends Error {
+	name = "NotADirectoryError";
+	resolvedPath;
+	constructor(resolvedPath, options) {
+		super(`Resolved scan target "${resolvedPath}" is not a directory. Ensure the path exists and points to a project directory, not a file.`, options);
+		this.resolvedPath = resolvedPath;
+	}
+};
 var AmbiguousProjectError = class extends Error {
 	name = "AmbiguousProjectError";
 	directory;
@@ -2077,7 +2085,7 @@ var AmbiguousProjectError = class extends Error {
 		this.candidates = candidates;
 	}
 };
-const isProjectDiscoveryError = (value) => value instanceof ProjectNotFoundError || value instanceof NoReactDependencyError || value instanceof PackageJsonNotFoundError || value instanceof AmbiguousProjectError;
+const isProjectDiscoveryError = (value) => value instanceof ProjectNotFoundError || value instanceof NoReactDependencyError || value instanceof PackageJsonNotFoundError || value instanceof NotADirectoryError || value instanceof AmbiguousProjectError;
 const isFile = (filePath) => {
 	try {
 		return fs.statSync(filePath).isFile();
@@ -4160,8 +4168,10 @@ const resolveScanTarget = (requestedDirectory) => {
 	const configSourceDirectory = loadedConfig?.sourceDirectory ?? null;
 	const redirectedDirectory = resolveConfigRootDir(userConfig, configSourceDirectory);
 	const directoryAfterRedirect = redirectedDirectory ?? absoluteRequested;
+	const resolvedDirectory = resolveDiagnoseTarget(directoryAfterRedirect) ?? directoryAfterRedirect;
+	if (!isDirectory(resolvedDirectory)) throw existsSync(resolvedDirectory) ? new NotADirectoryError(resolvedDirectory) : new ProjectNotFoundError(resolvedDirectory);
 	return {
-		resolvedDirectory: resolveDiagnoseTarget(directoryAfterRedirect) ?? directoryAfterRedirect,
+		resolvedDirectory,
 		requestedDirectory: absoluteRequested,
 		userConfig,
 		configSourceDirectory,
@@ -4645,7 +4655,7 @@ var Files = class Files extends Context.Service()("react-doctor/Files") {
 	* pattern in react-doctor-evals' test layers.
 	*/
 	static layerInMemory = (tree) => {
-		const resolveAbsolute = (filePath, rootDirectory) => Path.isAbsolute(filePath) ? filePath : Path.join(rootDirectory, filePath);
+		const resolveAbsolute = (filePath, rootDirectory) => Path.isAbsolute(filePath) ? filePath : `${rootDirectory}/${filePath}`;
 		return Layer.succeed(Files, Files.of({
 			readLines: (input) => Effect.sync(() => {
 				const absolute = resolveAbsolute(input.filePath, input.rootDirectory);
@@ -4653,17 +4663,17 @@ var Files = class Files extends Context.Service()("react-doctor/Files") {
 				return content === void 0 ? null : content.split("\n");
 			}),
 			listSourceFiles: (rootDirectory) => Effect.sync(() => {
-				const prefix = rootDirectory.endsWith(Path.sep) ? rootDirectory : `${rootDirectory}${Path.sep}`;
+				const prefix = rootDirectory.endsWith("/") ? rootDirectory : `${rootDirectory}/`;
 				const files = [];
 				for (const absolute of tree.keys()) {
 					if (!absolute.startsWith(prefix)) continue;
-					files.push(absolute.slice(prefix.length).split(Path.sep).join("/"));
+					files.push(absolute.slice(prefix.length));
 				}
 				return files;
 			}),
 			isFile: (filePath) => Effect.sync(() => tree.has(filePath)),
 			isDirectory: (filePath) => Effect.sync(() => {
-				const prefix = filePath.endsWith(Path.sep) ? filePath : `${filePath}${Path.sep}`;
+				const prefix = filePath.endsWith("/") ? filePath : `${filePath}/`;
 				for (const absolute of tree.keys()) if (absolute.startsWith(prefix)) return true;
 				return false;
 			})
@@ -5800,7 +5810,7 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 		const primaryLabel = diagnostic.labels[0];
 		const cleaned = cleanDiagnosticMessage(diagnostic.message, diagnostic.help, plugin, rule, project);
 		return {
-			filePath: diagnostic.filename,
+			filePath: diagnostic.filename.replaceAll("\\", "/"),
 			plugin,
 			rule,
 			severity: diagnostic.severity,
@@ -6602,22 +6612,7 @@ const runInspect = (input, hooks = {}) => Effect.gen(function* () {
 	"inspect.isCi": input.isCi,
 	"inspect.scoreSurface": input.scoreSurface ?? "score"
 } }));
-/**
-* Default layer stack for the production CLI / programmatic API:
-* real Node-side services for Project / Config / Files / Git / Linter /
-* DeadCode; HTTP for Score; noop Progress (the CLI overrides with
-* `Progress.layerOra(...)` for terminal feedback); the silent Reporter
-* (the orchestrator already returns the diagnostic array via
-* `Stream.runCollect`).
-*
-* Callers tweak by replacing individual layers: `--no-score` swaps
-* `Score.layerHttp` for `Score.layerOf(null)`; `--no-lint` swaps
-* `Linter.layerOxlint` for `Linter.layerOf([])`; `--no-dead-code`
-* swaps `DeadCode.layerNode` for `DeadCode.layerOf([])`; a caller
-* with a pre-loaded config swaps `Config.layerNode` for
-* `Config.layerOf(resolved)`.
-*/
-const layerInspectLive = Layer.mergeAll(Project.layerNode, Config.layerNode, DeadCode.layerNode, Files.layerNode, Git.layerNode, Linter.layerOxlint, LintPartialFailures.layerLive, Progress.layerNoop, Reporter.layerNoop, Score.layerHttp);
+Layer.mergeAll(Project.layerNode, Config.layerNode, DeadCode.layerNode, Files.layerNode, Git.layerNode, Linter.layerOxlint, LintPartialFailures.layerLive, Progress.layerNoop, Reporter.layerNoop, Score.layerHttp);
 const parseNodeVersion = (versionString) => {
 	const [major = 0, minor = 0, patch = 0] = versionString.replace(/^v/, "").trim().split(".").map(Number);
 	return {
@@ -7021,22 +7016,23 @@ import_picocolors.default.red, import_picocolors.default.yellow, import_picocolo
 const clearAutoSuppressionCaches = () => {};
 //#endregion
 //#region ../api/dist/index.js
-const diagnose = async (directory, options = {}) => {
-	const startTime = globalThis.performance.now();
-	const scanTarget = resolveScanTarget(directory);
+const DEFAULT_LAYER = Layer.mergeAll(Project.layerNode, Config.layerNode, DeadCode.layerNode, Files.layerNode, Git.layerNode, Linter.layerOxlint, LintPartialFailures.layerLive, Progress.layerNoop, Reporter.layerNoop, Score.layerHttp);
+const buildInspectProgram = (scanTarget, options, configOverride) => {
+	const effectiveConfig = configOverride ?? scanTarget.userConfig;
 	const includePaths = options.includePaths ?? [];
-	const program = runInspect({
+	return runInspect({
 		directory: scanTarget.resolvedDirectory,
 		includePaths,
-		customRulesOnly: scanTarget.userConfig?.customRulesOnly ?? false,
-		respectInlineDisables: options.respectInlineDisables ?? scanTarget.userConfig?.respectInlineDisables ?? true,
-		adoptExistingLintConfig: scanTarget.userConfig?.adoptExistingLintConfig ?? true,
-		ignoredTags: new Set(scanTarget.userConfig?.ignore?.tags ?? []),
-		runDeadCode: options.deadCode ?? scanTarget.userConfig?.deadCode ?? true,
+		customRulesOnly: effectiveConfig?.customRulesOnly ?? false,
+		respectInlineDisables: options.respectInlineDisables ?? effectiveConfig?.respectInlineDisables ?? true,
+		adoptExistingLintConfig: effectiveConfig?.adoptExistingLintConfig ?? true,
+		ignoredTags: new Set(effectiveConfig?.ignore?.tags ?? []),
+		runDeadCode: options.deadCode ?? effectiveConfig?.deadCode ?? true,
 		isCi: false,
 		resolveLocalGithubViewerPermission: true
 	});
-	const output = await Effect.runPromise(restoreLegacyThrow(program.pipe(Effect.provide(layerInspectLive), Effect.provide(layerOtlp))));
+};
+const outputToDiagnoseResult = (output, elapsedMilliseconds) => {
 	if (output.didLintFail && output.lintFailureReason !== null) console.error("Lint failed:", output.lintFailureReason);
 	const skippedChecks = [];
 	const skippedCheckReasons = {};
@@ -7050,9 +7046,14 @@ const diagnose = async (directory, options = {}) => {
 		skippedChecks,
 		...Object.keys(skippedCheckReasons).length > 0 ? { skippedCheckReasons } : {},
 		project: output.project,
-		elapsedMilliseconds: globalThis.performance.now() - startTime
+		elapsedMilliseconds
 	};
 };
+const diagnose = async (directory, options = {}) => {
+	const startTime = globalThis.performance.now();
+	const program = buildInspectProgram(resolveScanTarget(directory), options);
+	return outputToDiagnoseResult(await Effect.runPromise(restoreLegacyThrow(program.pipe(Effect.provide(DEFAULT_LAYER), Effect.provide(layerOtlp)))), globalThis.performance.now() - startTime);
+};
 //#endregion
 //#region src/index.ts
 const clearCaches = () => {
@@ -7081,6 +7082,6 @@ const toJsonReport = (result, options) => buildJsonReport({
 	totalElapsedMilliseconds: result.elapsedMilliseconds
 });
 //#endregion
-export { AmbiguousProjectError, NoReactDependencyError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
+export { AmbiguousProjectError, NoReactDependencyError, NotADirectoryError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, diagnose, filterSourceFiles, getDiffInfo, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.2.8",
+  "version": "0.2.9",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -58,21 +58,21 @@
     "oxlint": "^1.66.0",
     "prompts": "^2.4.2",
     "typescript": ">=5.0.4 <7",
-    "oxlint-plugin-react-doctor": "0.2.8"
+    "oxlint-plugin-react-doctor": "0.2.9"
   },
   "devDependencies": {
     "@types/prompts": "^2.4.9",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@react-doctor/api": "0.2.8",
-    "@react-doctor/core": "0.2.8"
+    "@react-doctor/api": "0.2.9",
+    "@react-doctor/core": "0.2.9"
   },
   "engines": {
     "node": "^20.19.0 || >=22.12.0"
   },
   "scripts": {
     "dev": "vp pack --watch",
-    "build": "node -e \"require('node:fs').rmSync('dist', { recursive: true, force: true })\" && NODE_ENV=production vp pack",
+    "build": "node -e \"require('node:fs').rmSync('dist', { recursive: true, force: true })\" && cross-env NODE_ENV=production vp pack",
     "typecheck": "tsc --noEmit",
     "test": "vp test run"
   }