react-doctor 0.2.10 → 0.2.11-dev.d917f62

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-BRBUS1pE.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-Df45H6Lw.js";
 import { createRequire } from "node:module";
 import { execFileSync, execSync } from "node:child_process";
 import path, { join } from "node:path";
@@ -6666,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.10";
+const VERSION = "0.2.11-dev.d917f62";
 //#endregion
 //#region src/inspect.ts
 const silentConsole = makeNoopConsole();
@@ -6764,9 +6764,7 @@ const runInspectWithRuntime = async (directory, options, userConfig, hasConfigOv
 	const output = await Effect.runPromise(restoreLegacyThrow(programWithLayers));
 	const didLintFail = lintBindingMissing || output.didLintFail;
 	const lintFailureReason = lintBindingMissing ? `oxlint native binding not found for Node ${process.version}; expected one matching ${OXLINT_NODE_REQUIREMENT}` : output.lintFailureReason;
-	const lintFailureReasonTag = output.lintFailureReasonTag;
-	const isNativeBindingFailure = lintFailureReasonTag === "OxlintUnavailable" || lintFailureReasonTag === "OxlintSpawnFailed";
-	if (!options.scoreOnly && !lintBindingMissing && output.didLintFail && lintFailureReason !== null) if (isNativeBindingFailure && /native binding/.test(lintFailureReason)) runConsole(Console.log(highlighter.gray(`  Upgrade to Node ${OXLINT_NODE_REQUIREMENT} or run: npx -p oxlint@latest react-doctor@latest`)));
+	if (!options.scoreOnly && !lintBindingMissing && output.didLintFail && lintFailureReason !== null) if (output.lintFailureReasonKind === "native-binding-missing") runConsole(Console.log(highlighter.gray(`  Upgrade to Node ${OXLINT_NODE_REQUIREMENT} or run: npx -p oxlint@latest react-doctor@latest`)));
 	else runConsole(Console.error(highlighter.error(lintFailureReason)));
 	const inspectDiagnostics = output.diagnostics;
 	const score = didLintFail ? null : output.score;
@@ -7463,7 +7461,7 @@ const warnSetupPromptFailure = async (options, error) => {
 		return;
 	}
 	try {
-		const { cliLogger } = await import("./cli-logger-BRBUS1pE.js").then((n) => n.n);
+		const { cliLogger } = await import("./cli-logger-Df45H6Lw.js").then((n) => n.n);
 		cliLogger.warn(message);
 	} catch {}
 };
@@ -7566,12 +7564,14 @@ const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isQui
 		message: "Choose what to scan",
 		choices: [{
 			title: "Full codebase",
+			description: "Scan every source file",
 			value: "full"
 		}, {
-			title: `Changed files (${changedSourceFiles.length})`,
+			title: diffInfo.isCurrentChanges ? `Uncommitted changes (${changedSourceFiles.length})` : `Changed files on ${diffInfo.currentBranch ?? "this branch"} (${changedSourceFiles.length})`,
+			description: diffInfo.isCurrentChanges ? "Compare working tree changes against HEAD" : `Compare against ${diffInfo.baseBranch} from the branch merge-base`,
 			value: "branch"
 		}],
-		initial: 0
+		initial: diffInfo.isCurrentChanges ? 0 : 1
 	});
 	return scanScope === "branch";
 };
@@ -7601,13 +7601,13 @@ const VALID_FAIL_ON_LEVELS = new Set([
 	"warning",
 	"none"
 ]);
-const DEFAULT_FAIL_ON_LEVEL = "error";
+const DEFAULT_FAIL_ON_LEVEL = "none";
 const isValidFailOnLevel = (level) => VALID_FAIL_ON_LEVELS.has(level);
 const resolveFailOnLevel = (flags, userConfig) => {
 	const sourceValue = flags.failOn ?? userConfig?.failOn ?? DEFAULT_FAIL_ON_LEVEL;
 	if (isValidFailOnLevel(sourceValue)) return sourceValue;
-	cliLogger.warn(`Invalid failOn level "${sourceValue}". Expected one of: error, warning, none. Falling back to "none".`);
-	return "none";
+	cliLogger.warn(`Invalid failOn level "${sourceValue}". Expected one of: error, warning, none. Falling back to "${DEFAULT_FAIL_ON_LEVEL}".`);
+	return DEFAULT_FAIL_ON_LEVEL;
 };
 //#endregion
 //#region src/cli/utils/resolve-project-diff-include-paths.ts
@@ -7892,7 +7892,14 @@ const inspectAction = async (directory, flags) => {
 				cliLogger.log(`Scanning ${highlighter.info(`${stagedFiles.length}`)} staged files...`);
 				cliLogger.break();
 			}
-			const snapshot = await materializeStagedFiles(resolvedDirectory, stagedFiles, mkdtempSync(path.join(tmpdir(), STAGED_FILES_TEMP_DIR_PREFIX)));
+			const tempDirectory = mkdtempSync(path.join(tmpdir(), STAGED_FILES_TEMP_DIR_PREFIX));
+			const snapshot = await materializeStagedFiles(resolvedDirectory, stagedFiles, tempDirectory).catch((error) => {
+				rmSync(tempDirectory, {
+					recursive: true,
+					force: true
+				});
+				throw error;
+			});
 			try {
 				const scanResult = await inspect(snapshot.tempDirectory, {
 					...scanOptions,
@@ -7901,7 +7908,7 @@ const inspectAction = async (directory, flags) => {
 				});
 				const remappedDiagnostics = scanResult.diagnostics.map((diagnostic) => ({
 					...diagnostic,
-					filePath: path.isAbsolute(diagnostic.filePath) ? diagnostic.filePath.replaceAll(snapshot.tempDirectory, resolvedDirectory) : diagnostic.filePath
+					filePath: path.isAbsolute(diagnostic.filePath) ? diagnostic.filePath.replaceAll(snapshot.tempDirectory, () => resolvedDirectory) : diagnostic.filePath
 				}));
 				finalizeScans({
 					diagnostics: remappedDiagnostics,
@@ -9139,7 +9146,7 @@ const stripUnknownCliFlags = (argv) => {
 //#region src/cli/index.ts
 process.on("SIGINT", exitGracefully);
 process.on("SIGTERM", exitGracefully);
-const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--lint", "enable linting").option("--no-lint", "skip linting").option("--dead-code", "enable dead-code analysis (default)").option("--no-dead-code", "skip dead-code analysis (unused files / exports / dependencies, circular imports)").option("--verbose", "show every rule and per-file details (default shows top 3 rules)").option("--score", "output only the score").option("--json", "output a single structured JSON report (suppresses other output)").option("--json-compact", "with --json, emit compact JSON (no indentation)").option("-y, --yes", "skip prompts, scan all workspace projects").option("--full", "force a full scan (overrides any `diff` value in config or `--diff`)").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch (pass `false` to disable; overridden by --full)").option("--no-score", "skip the score API and the share URL").option("--staged", "scan only staged (git index) files for pre-commit hooks").option("--fail-on <level>", "exit with error code on diagnostics: error, warning, none (default: error)").option("--annotations", "output diagnostics as GitHub Actions annotations").option("--pr-comment", "tune CLI output for sticky PR comments (drops weak-signal rule families like `design` from the printed list and the fail-on gate; configure via config.surfaces)").option("--explain <file:line>", "diagnose why a rule fired or why a suppression didn't apply at a specific location").option("--why <file:line>", "alias for --explain").option("--respect-inline-disables", "respect inline `// eslint-disable*` / `// oxlint-disable*` comments (default)").option("--no-respect-inline-disables", "audit mode: neutralize inline lint suppressions before scanning").addHelpText("after", `
+const program = new Command().name("react-doctor").description("Diagnose React codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--lint", "enable linting").option("--no-lint", "skip linting").option("--dead-code", "enable dead-code analysis (default)").option("--no-dead-code", "skip dead-code analysis (unused files / exports / dependencies, circular imports)").option("--verbose", "show every rule and per-file details (default shows top 3 rules)").option("--score", "output only the score").option("--json", "output a single structured JSON report (suppresses other output)").option("--json-compact", "with --json, emit compact JSON (no indentation)").option("-y, --yes", "skip prompts, scan all workspace projects").option("--full", "force a full scan (overrides any `diff` value in config or `--diff`)").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch (pass `false` to disable; overridden by --full)").option("--no-score", "skip the score API and the share URL").option("--staged", "scan only staged (git index) files for pre-commit hooks").option("--fail-on <level>", "exit with error code on diagnostics: error, warning, none (default: none)").option("--annotations", "output diagnostics as GitHub Actions annotations").option("--pr-comment", "tune CLI output for sticky PR comments (drops weak-signal rule families like `design` from the printed list and the fail-on gate; configure via config.surfaces)").option("--explain <file:line>", "diagnose why a rule fired or why a suppression didn't apply at a specific location").option("--why <file:line>", "alias for --explain").option("--respect-inline-disables", "respect inline `// eslint-disable*` / `// oxlint-disable*` comments (default)").option("--no-respect-inline-disables", "audit mode: neutralize inline lint suppressions before scanning").addHelpText("after", `
 ${highlighter.dim("Configuration:")}
   Place a ${highlighter.info("react-doctor.config.json")} (or ${highlighter.info("\"reactDoctor\"")} key in your package.json) in the project root.
   CLI flags always override config values. See the README for the full schema.

package/dist/index.d.ts

@@ -310,15 +310,17 @@ interface ProjectInfo {
   hasReactCompiler: boolean;
   hasTanStackQuery: boolean;
   /**
-   * `true` when `preact` is declared anywhere in the project's
-   * dependency manifest. Drives the `preact` capability in
-   * `buildCapabilities`, which gates every `preact-*` rule. Modeled
-   * on `hasTanStackQuery` rather than the `framework` field because
-   * the dominant Preact setup today is Preact-on-Vite — those
-   * projects classify as `framework: "vite"` for build-tool reasons
-   * but still need Preact-specific rules to fire.
+   * The declared `preact` version spec, or `null` when Preact isn't a
+   * dependency. Parallels `reactVersion` so a React-compatible runtime is
+   * modeled the same way React is. Drives the `preact` capability in
+   * `buildCapabilities` (which gates every `preact-*` rule) — keyed off
+   * this rather than `framework` because the dominant Preact setup
+   * (Preact-on-Vite) classifies as `framework: "vite"` but still needs
+   * Preact rules to fire.
    */
-  hasPreact: boolean;
+  preactVersion: string | null;
+  /** Parsed major from `preactVersion`, or `null` when absent/unparseable. Mirrors `reactMajorVersion`. */
+  preactMajorVersion: number | null;
   /**
    * `true` when the project (or any of its workspace packages) declares
    * React Native or Expo as a dependency. Enables the `react-native`
@@ -333,6 +335,13 @@ interface ProjectInfo {
    * — no `rn-*` rules load for the project at all.
    */
   hasReactNativeWorkspace: boolean;
+  /**
+   * `true` when the project (or any of its workspace packages) declares
+   * `react-native-reanimated`. Lets diagnostics surface reanimated's
+   * Compiler-compatible `.get()` / `.set()` accessors only where they
+   * apply, instead of on every React Native project.
+   */
+  hasReanimated: boolean;
   sourceFileCount: number;
 }
 //#endregion