react-doctor 0.7.9-dev.2450582 → 0.7.9-dev.2ba83c3

package/dist/cli.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="c29245d6-1073-542d-91c7-90d62f3911e2")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="5c4df375-a5c4-57d2-9076-4e54cbb8f3bd")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -36252,53 +36252,6 @@ const SUPPLY_CHAIN_OVERLAP_TIMEOUT_MS = 9e4;
 const SUPPLY_CHAIN_CACHE_TTL_MS = 864e5;
 const SUPPLY_CHAIN_CACHE_SUBDIR = "supply-chain";
 const SUPPLY_CHAIN_IGNORED_PACKAGES = new Set(["next"]);
-const lastSourceMappingUrl = (fileContent) => {
-	const lastReference = [...fileContent.matchAll(/\/\/# sourceMappingURL=(\S+)/g)].at(-1);
-	return lastReference ? lastReference[1] : null;
-};
-const mapTargetsSource = (mapContent, input) => {
-	const parsedMap = JSON.parse(mapContent);
-	if (typeof parsedMap !== "object" || parsedMap === null) return false;
-	const { file, sources } = parsedMap;
-	if (file !== input.expectedFile) return false;
-	if (!Array.isArray(sources) || sources.length !== 1 || typeof sources[0] !== "string") return false;
-	return Path.posix.normalize(Path.posix.join(input.mapDirectory, sources[0])) === input.sourcePath;
-};
-const quartetProvesEmit = (stem, sourcePath, readFileText) => {
-	try {
-		const emitBaseName = Path.posix.basename(stem);
-		const mapDirectory = Path.posix.dirname(stem);
-		return lastSourceMappingUrl(readFileText(`${stem}.js`)) === `${emitBaseName}.js.map` && lastSourceMappingUrl(readFileText(`${stem}.d.ts`)) === `${emitBaseName}.d.ts.map` && mapTargetsSource(readFileText(`${stem}.js.map`), {
-			expectedFile: `${emitBaseName}.js`,
-			mapDirectory,
-			sourcePath
-		}) && mapTargetsSource(readFileText(`${stem}.d.ts.map`), {
-			expectedFile: `${emitBaseName}.d.ts`,
-			mapDirectory,
-			sourcePath
-		});
-	} catch {
-		return false;
-	}
-};
-const collectTypeScriptEmitDuplicateJsPaths = (input) => {
-	const untracked = new Set(input.untrackedPaths);
-	const duplicateJsPaths = /* @__PURE__ */ new Set();
-	for (const jsPath of untracked) {
-		if (!jsPath.endsWith(".js")) continue;
-		const stem = jsPath.slice(0, -3);
-		if (![
-			jsPath,
-			`${stem}.js.map`,
-			`${stem}.d.ts`,
-			`${stem}.d.ts.map`
-		].every((emitPath) => untracked.has(emitPath) && !input.trackedPaths.has(emitPath))) continue;
-		const sourcePath = [`${stem}.ts`, `${stem}.tsx`].find((candidatePath) => input.trackedPaths.has(candidatePath));
-		if (!sourcePath) continue;
-		if (quartetProvesEmit(stem, sourcePath, input.readFileText)) duplicateJsPaths.add(jsPath);
-	}
-	return duplicateJsPaths;
-};
 const isIgnoredDirectoryName = (directoryName) => IGNORED_DIRECTORIES.has(directoryName) || directoryName.startsWith(".") && !SCANNED_DOT_DIRECTORIES.has(directoryName);
 const hasIgnoredPathSegment = (relativePath) => relativePath.split(/[/\\]/).slice(0, -1).some((segment) => isIgnoredDirectoryName(segment));
 const isLintableSourceFile = (filePath) => SOURCE_FILE_PATTERN.test(filePath) && !GENERATED_BUNDLE_FILE_PATTERN.test(filePath) && !GENERATED_SOURCE_DIRECTORY_PATTERN.test(filePath);
@@ -36366,29 +36319,20 @@ const collectSizedSourceFiles = (rootDirectory, relativePaths) => {
 	}
 	return entries;
 };
-const gitLsFiles = (rootDirectory, flags) => {
+const listSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
 		"-z",
-		...flags
+		"--cached",
+		"--others",
+		"--exclude-standard"
 	], {
 		cwd: rootDirectory,
 		encoding: "utf-8",
 		maxBuffer: GIT_LS_FILES_MAX_BUFFER_BYTES
 	});
 	if (result.error || result.status !== 0) return null;
-	return result.stdout.split("\0").filter((filePath) => filePath.length > 0);
-};
-const listSourceFilesViaGit = (rootDirectory) => {
-	const trackedPaths = gitLsFiles(rootDirectory, ["--cached"]);
-	const untrackedPaths = gitLsFiles(rootDirectory, ["--others", "--exclude-standard"]);
-	if (trackedPaths === null || untrackedPaths === null) return null;
-	const emitDuplicateJsPaths = collectTypeScriptEmitDuplicateJsPaths({
-		trackedPaths: new Set(trackedPaths),
-		untrackedPaths,
-		readFileText: (relativePath) => NFS.readFileSync(Path.resolve(rootDirectory, relativePath), "utf-8")
-	});
-	return [...trackedPaths, ...untrackedPaths].filter((filePath) => isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath) && !emitDuplicateJsPaths.has(filePath));
+	return result.stdout.split("\0").filter((filePath) => filePath.length > 0 && isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath));
 };
 const listSourceFilesViaFilesystem = (rootDirectory) => {
 	const filePaths = [];
@@ -39228,10 +39172,8 @@ const isDiagnosticSurface = (value) => typeof value === "string" && DIAGNOSTIC_S
 * to the local CLI so developers see them while editing, but they're
 * removed from the PR comment surface, the score, and the CI gate so
 * they can't bury real React findings or fail a build over a Tailwind
-* shorthand. Diagnostics from test and story files likewise remain in
-* raw output while staying out of the production-health score and CI
-* gate. Override per-surface via `config.surfaces.<surface>` to promote
-* individual rules back in by tag, category, or rule id.
+* shorthand. Override per-surface via `config.surfaces.<surface>` to
+* promote individual rules back in by tag, category, or rule id.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -42062,7 +42004,6 @@ const isDiagnosticOnSurface = (diagnostic, surface, config) => {
 	if (resolved.includeRuleKeys.has(ruleKey)) return true;
 	if (resolved.includeCategories.has(category)) return true;
 	if (intersects(tags, resolved.includeTags)) return true;
-	if (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure")) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -42487,7 +42428,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}
 			const diff = yield* runGit(input.directory, [
 				"diff",
-				"--no-ext-diff",
 				"-z",
 				"--name-only",
 				"--diff-filter=ACMR",
@@ -42534,7 +42474,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (resolvedCurrentBranch !== null && resolvedCurrentBranch === baseBranch) {
 					const uncommitted = yield* runGit(directory, [
 						"diff",
-						"--no-ext-diff",
 						"-z",
 						"--name-only",
 						"--diff-filter=ACMR",
@@ -42561,7 +42500,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (mergeBaseRef === null) return null;
 				const diff = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"-z",
 					"--name-only",
 					"--diff-filter=ACMR",
@@ -42579,7 +42517,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}).pipe(withSpan("Git.diffSelection")),
 			stagedFilePaths: (directory) => runGit(directory, [
 				"diff",
-				"--no-ext-diff",
 				"--cached",
 				"-z",
 				"--name-only",
@@ -42625,7 +42562,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (baseRef !== void 0 && !isSafeGitRevision(baseRef)) return null;
 				const result = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"--unified=0",
 					"--diff-filter=ACMR",
 					"--relative",
@@ -47223,7 +47159,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.7.9-dev.2450582";
+const VERSION = "0.7.9-dev.2ba83c3";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -47575,13 +47511,13 @@ const isDevVersion = (version) => version === "0.0.0" || version.includes("-");
 * uploads source-map artifacts under, so stack frames symbolicate. Honors the
 * standard `SENTRY_RELEASE` override.
 */
-const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.7.9-dev.2450582`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.7.9-dev.2ba83c3`;
 /**
 * Deployment environment shown in Sentry's environment filter. Defaults to
 * `production` for tagged releases and `development` for dev/unbuilt versions,
 * overridable via the standard `SENTRY_ENVIRONMENT` env var.
 */
-const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.7.9-dev.2450582") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.7.9-dev.2ba83c3") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -60076,7 +60012,6 @@ const buildOutcomeAttributes = (input) => {
 	const summary = summarizeDiagnostics(result.diagnostics);
 	const blockingLevel = resolveTelemetryBlocking(input.userConfig);
 	const gateDiagnostics = filterDiagnosticsForSurface(result.diagnostics, "ciFailure", input.userConfig);
-	const gateDiagnosticSet = new Set(gateDiagnostics);
 	const complete = isInspectResultComplete(result);
 	const didLintHardFail = hasLintHardFailure(result);
 	const failsOnHardFailure = didLintHardFail && blockingLevel !== "none";
@@ -60099,13 +60034,11 @@ const buildOutcomeAttributes = (input) => {
 	const largestRuleBucket = buildRuleBlastRadii(result.diagnostics)[0] ?? null;
 	let diagnosticsInTestFiles = 0;
 	let diagnosticsInStoryFiles = 0;
-	let nonProductionGateExcluded = 0;
 	const diagnosticSiteCounts = /* @__PURE__ */ new Map();
 	const findingsPerFixGroup = /* @__PURE__ */ new Map();
 	for (const diagnostic of result.diagnostics) {
 		if (diagnostic.fileContext === "test") diagnosticsInTestFiles += 1;
 		if (diagnostic.fileContext === "story") diagnosticsInStoryFiles += 1;
-		if (diagnostic.fileContext !== void 0 && !gateDiagnosticSet.has(diagnostic)) nonProductionGateExcluded += 1;
 		const diagnosticSite = JSON.stringify([
 			diagnostic.filePath,
 			diagnostic.line,
@@ -60154,7 +60087,6 @@ const buildOutcomeAttributes = (input) => {
 			affectedFiles: summary.affectedFileCount,
 			inTestFiles: diagnosticsInTestFiles,
 			inStoryFiles: diagnosticsInStoryFiles,
-			nonProductionGateExcluded,
 			distinctRules: countByRule.size,
 			topRule,
 			sameSiteOccurrences,
@@ -61994,9 +61926,7 @@ const finalizeAndRender = (input) => gen(function* () {
 		yield* log(highlighter.gray(`  ${demotedDiagnosticCount} demoted from the ${options.outputSurface} surface (e.g. design cleanup) — run \`npx react-doctor@latest .\` locally for the full list.`));
 		yield* log("");
 	}
-	const scoreDiagnostics = filterDiagnosticsForSurface([...diagnostics], "score", userConfig);
-	const displayedScoreDiagnostics = filterDiagnosticsForSurface([...printedDiagnostics], "score", userConfig);
-	const potentialScore = score ? yield* promise(() => computeProjectedScore(displayedScoreDiagnostics, scoreDiagnostics, score)) : null;
+	const potentialScore = score ? yield* promise(() => computeProjectedScore([...printedDiagnostics], [...surfaceDiagnostics], score)) : null;
 	const shouldShowShareLink = !options.noScore && options.share && !options.isCi;
 	yield* pause;
 	yield* printSummary({
@@ -63897,8 +63827,6 @@ const printMultiProjectSummary = (input) => gen(function* () {
 	const categoryFilters = input.categoryFilters ?? /* @__PURE__ */ new Set();
 	const animateRender = !verbose && canAnimateOnboarding(process.stdout);
 	const displayDiagnostics = filterDiagnosticsByCategories(filterScansForSurface(completedScans, "cli"), categoryFilters);
-	const scoreDiagnostics = new Set(filterScansForSurface(completedScans, "score"));
-	const displayedScoreDiagnostics = displayDiagnostics.filter((diagnostic) => scoreDiagnostics.has(diagnostic));
 	const projectRootByDiagnostic = /* @__PURE__ */ new WeakMap();
 	for (const scan of completedScans) for (const diagnostic of scan.result.diagnostics) projectRootByDiagnostic.set(diagnostic, scan.result.project.rootDirectory);
 	const resolveDiagnosticSourceRoot = (diagnostic) => projectRootByDiagnostic.get(diagnostic) ?? "";
@@ -63922,7 +63850,7 @@ const printMultiProjectSummary = (input) => gen(function* () {
 		diagnostics: displayDiagnostics,
 		elapsedMilliseconds: totalElapsedMilliseconds,
 		scoreResult: aggregateScore,
-		potentialScore: lowestScoredScan ? yield* promise(() => computeProjectedScore(displayedScoreDiagnostics, filterScansForSurface([lowestScoredScan], "score"), lowestScoredScan.result.score)) : null,
+		potentialScore: lowestScoredScan ? yield* promise(() => computeProjectedScore(displayDiagnostics, filterScansForSurface([lowestScoredScan], "cli"), lowestScoredScan.result.score)) : null,
 		totalSourceFileCount,
 		noScoreMessage: "Score unavailable.",
 		verbose,
@@ -65867,4 +65795,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=c29245d6-1073-542d-91c7-90d62f3911e2
+//# debugId=5c4df375-a5c4-57d2-9076-4e54cbb8f3bd

package/dist/index.d.ts

@@ -9056,10 +9056,8 @@ interface ReactDoctorIgnoreConfig {
  *
  * Defaults: design rules (tag `"design"`) are excluded from `prComment`,
  * `score`, and `ciFailure` so style cleanup doesn't dilute meaningful
- * React findings. Diagnostics stamped with `fileContext: "test" | "story"`
- * are excluded from `score` and `ciFailure`. Both remain in `cli`, and an
- * explicit rule, category, or tag include promotes a diagnostic back onto
- * the selected surface.
+ * React findings. They remain in `cli` so locally-running developers
+ * still see the suggestion when they touch the file.
  */
 type DiagnosticSurface = "cli" | "prComment" | "score" | "ciFailure";
 /**
@@ -9340,22 +9338,23 @@ interface ReactDoctorConfig {
    */
   adoptExistingLintConfig?: boolean;
   /**
-   * Per-surface include/exclude controls. Each `DiagnosticSurface` resolves
-   * rule tags, categories, and identifiers independently. This lets you keep a rule
-   * visible locally while hiding it from PR comments, the score, or the CI
-   * gate. These controls don't change rule severity or activation.
+   * Per-surface include/exclude controls. Each `DiagnosticSurface` is
+   * resolved independently against rule tags, category, and id so a
+   * single rule can be visible locally yet hidden from PR comments,
+   * neutralized from the score, and excluded from the CI gate — all
+   * without touching the rule's severity or activation.
    *
    * Defaults (applied before user overrides):
    *
    * - `prComment` excludes tag `"design"`
    * - `score` excludes tag `"design"`
    * - `ciFailure` excludes tag `"design"`
-   * - `score` and `ciFailure` exclude test/story file contexts unless explicitly included
    *
-   * Pass any controls block, including an empty `{}`, to keep the defaults.
-   * Your include/exclude entries layer on top. Includes always win over
-   * excludes. For example, you can promote one high-signal `design-*` rule
-   * back into the score or PR-comment surface.
+   * Pass any controls block (even an empty `{}`) to keep the default
+   * exclusions; the user's include/exclude entries layer on top.
+   * Include entries always win over exclude entries — handy for
+   * promoting a single high-signal `design-*` rule back into the
+   * score or PR-comment surface.
    */
   surfaces?: Partial<Record<DiagnosticSurface, SurfaceControls>>;
   /**

package/dist/index.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="a80081fb-3090-5160-8adf-e2652ffe1212")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="526cf949-6544-5cb8-bf6d-3f4d854b6678")}catch(e){}}();
 import { r as __toESM$1, t as __commonJSMin$1 } from "./chunk-N93fKeF6.js";
 import { createRequire } from "node:module";
 import reactDoctorPlugin, { ALL_REACT_DOCTOR_RULE_KEYS, CROSS_FILE_DEPENDENCY_COLLECTORS, CROSS_FILE_RULE_IDS, FRAMEWORK_SPECIFIC_RULE_KEYS, FRAMEWORK_TOKENS, MOTION_LIBRARY_PACKAGES, REACT_COMPILER_RULES, REACT_DOCTOR_RULES, classifySecurityScanFile, collectCrossFileDependencyProbes, resetManifestCaches, shouldReadSecurityScanContent } from "oxlint-plugin-react-doctor";
@@ -33050,53 +33050,6 @@ const SUPPLY_CHAIN_OVERLAP_TIMEOUT_MS = 9e4;
 const SUPPLY_CHAIN_CACHE_TTL_MS = 864e5;
 const SUPPLY_CHAIN_CACHE_SUBDIR = "supply-chain";
 const SUPPLY_CHAIN_IGNORED_PACKAGES = new Set(["next"]);
-const lastSourceMappingUrl = (fileContent) => {
-	const lastReference = [...fileContent.matchAll(/\/\/# sourceMappingURL=(\S+)/g)].at(-1);
-	return lastReference ? lastReference[1] : null;
-};
-const mapTargetsSource = (mapContent, input) => {
-	const parsedMap = JSON.parse(mapContent);
-	if (typeof parsedMap !== "object" || parsedMap === null) return false;
-	const { file, sources } = parsedMap;
-	if (file !== input.expectedFile) return false;
-	if (!Array.isArray(sources) || sources.length !== 1 || typeof sources[0] !== "string") return false;
-	return Path.posix.normalize(Path.posix.join(input.mapDirectory, sources[0])) === input.sourcePath;
-};
-const quartetProvesEmit = (stem, sourcePath, readFileText) => {
-	try {
-		const emitBaseName = Path.posix.basename(stem);
-		const mapDirectory = Path.posix.dirname(stem);
-		return lastSourceMappingUrl(readFileText(`${stem}.js`)) === `${emitBaseName}.js.map` && lastSourceMappingUrl(readFileText(`${stem}.d.ts`)) === `${emitBaseName}.d.ts.map` && mapTargetsSource(readFileText(`${stem}.js.map`), {
-			expectedFile: `${emitBaseName}.js`,
-			mapDirectory,
-			sourcePath
-		}) && mapTargetsSource(readFileText(`${stem}.d.ts.map`), {
-			expectedFile: `${emitBaseName}.d.ts`,
-			mapDirectory,
-			sourcePath
-		});
-	} catch {
-		return false;
-	}
-};
-const collectTypeScriptEmitDuplicateJsPaths = (input) => {
-	const untracked = new Set(input.untrackedPaths);
-	const duplicateJsPaths = /* @__PURE__ */ new Set();
-	for (const jsPath of untracked) {
-		if (!jsPath.endsWith(".js")) continue;
-		const stem = jsPath.slice(0, -3);
-		if (![
-			jsPath,
-			`${stem}.js.map`,
-			`${stem}.d.ts`,
-			`${stem}.d.ts.map`
-		].every((emitPath) => untracked.has(emitPath) && !input.trackedPaths.has(emitPath))) continue;
-		const sourcePath = [`${stem}.ts`, `${stem}.tsx`].find((candidatePath) => input.trackedPaths.has(candidatePath));
-		if (!sourcePath) continue;
-		if (quartetProvesEmit(stem, sourcePath, input.readFileText)) duplicateJsPaths.add(jsPath);
-	}
-	return duplicateJsPaths;
-};
 const isIgnoredDirectoryName = (directoryName) => IGNORED_DIRECTORIES.has(directoryName) || directoryName.startsWith(".") && !SCANNED_DOT_DIRECTORIES.has(directoryName);
 const hasIgnoredPathSegment = (relativePath) => relativePath.split(/[/\\]/).slice(0, -1).some((segment) => isIgnoredDirectoryName(segment));
 const isLintableSourceFile = (filePath) => SOURCE_FILE_PATTERN.test(filePath) && !GENERATED_BUNDLE_FILE_PATTERN.test(filePath) && !GENERATED_SOURCE_DIRECTORY_PATTERN.test(filePath);
@@ -33167,29 +33120,20 @@ const collectSizedSourceFiles = (rootDirectory, relativePaths) => {
 	}
 	return entries;
 };
-const gitLsFiles = (rootDirectory, flags) => {
+const listSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
 		"-z",
-		...flags
+		"--cached",
+		"--others",
+		"--exclude-standard"
 	], {
 		cwd: rootDirectory,
 		encoding: "utf-8",
 		maxBuffer: GIT_LS_FILES_MAX_BUFFER_BYTES
 	});
 	if (result.error || result.status !== 0) return null;
-	return result.stdout.split("\0").filter((filePath) => filePath.length > 0);
-};
-const listSourceFilesViaGit = (rootDirectory) => {
-	const trackedPaths = gitLsFiles(rootDirectory, ["--cached"]);
-	const untrackedPaths = gitLsFiles(rootDirectory, ["--others", "--exclude-standard"]);
-	if (trackedPaths === null || untrackedPaths === null) return null;
-	const emitDuplicateJsPaths = collectTypeScriptEmitDuplicateJsPaths({
-		trackedPaths: new Set(trackedPaths),
-		untrackedPaths,
-		readFileText: (relativePath) => NFS.readFileSync(Path.resolve(rootDirectory, relativePath), "utf-8")
-	});
-	return [...trackedPaths, ...untrackedPaths].filter((filePath) => isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath) && !emitDuplicateJsPaths.has(filePath));
+	return result.stdout.split("\0").filter((filePath) => filePath.length > 0 && isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath));
 };
 const listSourceFilesViaFilesystem = (rootDirectory) => {
 	const filePaths = [];
@@ -36028,10 +35972,8 @@ const isDiagnosticSurface = (value) => typeof value === "string" && DIAGNOSTIC_S
 * to the local CLI so developers see them while editing, but they're
 * removed from the PR comment surface, the score, and the CI gate so
 * they can't bury real React findings or fail a build over a Tailwind
-* shorthand. Diagnostics from test and story files likewise remain in
-* raw output while staying out of the production-health score and CI
-* gate. Override per-surface via `config.surfaces.<surface>` to promote
-* individual rules back in by tag, category, or rule id.
+* shorthand. Override per-surface via `config.surfaces.<surface>` to
+* promote individual rules back in by tag, category, or rule id.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -38818,7 +38760,6 @@ const isDiagnosticOnSurface = (diagnostic, surface, config) => {
 	if (resolved.includeRuleKeys.has(ruleKey)) return true;
 	if (resolved.includeCategories.has(category)) return true;
 	if (intersects(tags, resolved.includeTags)) return true;
-	if (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure")) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -39243,7 +39184,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}
 			const diff = yield* runGit(input.directory, [
 				"diff",
-				"--no-ext-diff",
 				"-z",
 				"--name-only",
 				"--diff-filter=ACMR",
@@ -39290,7 +39230,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (resolvedCurrentBranch !== null && resolvedCurrentBranch === baseBranch) {
 					const uncommitted = yield* runGit(directory, [
 						"diff",
-						"--no-ext-diff",
 						"-z",
 						"--name-only",
 						"--diff-filter=ACMR",
@@ -39317,7 +39256,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (mergeBaseRef === null) return null;
 				const diff = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"-z",
 					"--name-only",
 					"--diff-filter=ACMR",
@@ -39335,7 +39273,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}).pipe(withSpan("Git.diffSelection")),
 			stagedFilePaths: (directory) => runGit(directory, [
 				"diff",
-				"--no-ext-diff",
 				"--cached",
 				"-z",
 				"--name-only",
@@ -39381,7 +39318,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (baseRef !== void 0 && !isSafeGitRevision(baseRef)) return null;
 				const result = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"--unified=0",
 					"--diff-filter=ACMR",
 					"--relative",
@@ -43727,4 +43663,4 @@ const toJsonReport = (result, options) => buildJsonReport({
 export { AmbiguousProjectError, NoReactDependencyError, NotADirectoryError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, defineConfig, diagnose, filterSourceFiles, getDiffInfo, hasReactRuntime, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 
 //# sourceMappingURL=index.js.map
-//# debugId=a80081fb-3090-5160-8adf-e2652ffe1212
+//# debugId=526cf949-6544-5cb8-bf6d-3f4d854b6678

package/dist/lsp.js

@@ -33102,53 +33102,6 @@ const SUPPLY_CHAIN_OVERLAP_TIMEOUT_MS = 9e4;
 const SUPPLY_CHAIN_CACHE_TTL_MS = 864e5;
 const SUPPLY_CHAIN_CACHE_SUBDIR = "supply-chain";
 const SUPPLY_CHAIN_IGNORED_PACKAGES = new Set(["next"]);
-const lastSourceMappingUrl = (fileContent) => {
-	const lastReference = [...fileContent.matchAll(/\/\/# sourceMappingURL=(\S+)/g)].at(-1);
-	return lastReference ? lastReference[1] : null;
-};
-const mapTargetsSource = (mapContent, input) => {
-	const parsedMap = JSON.parse(mapContent);
-	if (typeof parsedMap !== "object" || parsedMap === null) return false;
-	const { file, sources } = parsedMap;
-	if (file !== input.expectedFile) return false;
-	if (!Array.isArray(sources) || sources.length !== 1 || typeof sources[0] !== "string") return false;
-	return Path.posix.normalize(Path.posix.join(input.mapDirectory, sources[0])) === input.sourcePath;
-};
-const quartetProvesEmit = (stem, sourcePath, readFileText) => {
-	try {
-		const emitBaseName = Path.posix.basename(stem);
-		const mapDirectory = Path.posix.dirname(stem);
-		return lastSourceMappingUrl(readFileText(`${stem}.js`)) === `${emitBaseName}.js.map` && lastSourceMappingUrl(readFileText(`${stem}.d.ts`)) === `${emitBaseName}.d.ts.map` && mapTargetsSource(readFileText(`${stem}.js.map`), {
-			expectedFile: `${emitBaseName}.js`,
-			mapDirectory,
-			sourcePath
-		}) && mapTargetsSource(readFileText(`${stem}.d.ts.map`), {
-			expectedFile: `${emitBaseName}.d.ts`,
-			mapDirectory,
-			sourcePath
-		});
-	} catch {
-		return false;
-	}
-};
-const collectTypeScriptEmitDuplicateJsPaths = (input) => {
-	const untracked = new Set(input.untrackedPaths);
-	const duplicateJsPaths = /* @__PURE__ */ new Set();
-	for (const jsPath of untracked) {
-		if (!jsPath.endsWith(".js")) continue;
-		const stem = jsPath.slice(0, -3);
-		if (![
-			jsPath,
-			`${stem}.js.map`,
-			`${stem}.d.ts`,
-			`${stem}.d.ts.map`
-		].every((emitPath) => untracked.has(emitPath) && !input.trackedPaths.has(emitPath))) continue;
-		const sourcePath = [`${stem}.ts`, `${stem}.tsx`].find((candidatePath) => input.trackedPaths.has(candidatePath));
-		if (!sourcePath) continue;
-		if (quartetProvesEmit(stem, sourcePath, input.readFileText)) duplicateJsPaths.add(jsPath);
-	}
-	return duplicateJsPaths;
-};
 const isIgnoredDirectoryName = (directoryName) => IGNORED_DIRECTORIES.has(directoryName) || directoryName.startsWith(".") && !SCANNED_DOT_DIRECTORIES.has(directoryName);
 const hasIgnoredPathSegment = (relativePath) => relativePath.split(/[/\\]/).slice(0, -1).some((segment) => isIgnoredDirectoryName(segment));
 const isLintableSourceFile = (filePath) => SOURCE_FILE_PATTERN.test(filePath) && !GENERATED_BUNDLE_FILE_PATTERN.test(filePath) && !GENERATED_SOURCE_DIRECTORY_PATTERN.test(filePath);
@@ -33219,29 +33172,20 @@ const collectSizedSourceFiles = (rootDirectory, relativePaths) => {
 	}
 	return entries;
 };
-const gitLsFiles = (rootDirectory, flags) => {
+const listSourceFilesViaGit = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
 		"-z",
-		...flags
+		"--cached",
+		"--others",
+		"--exclude-standard"
 	], {
 		cwd: rootDirectory,
 		encoding: "utf-8",
 		maxBuffer: GIT_LS_FILES_MAX_BUFFER_BYTES
 	});
 	if (result.error || result.status !== 0) return null;
-	return result.stdout.split("\0").filter((filePath) => filePath.length > 0);
-};
-const listSourceFilesViaGit = (rootDirectory) => {
-	const trackedPaths = gitLsFiles(rootDirectory, ["--cached"]);
-	const untrackedPaths = gitLsFiles(rootDirectory, ["--others", "--exclude-standard"]);
-	if (trackedPaths === null || untrackedPaths === null) return null;
-	const emitDuplicateJsPaths = collectTypeScriptEmitDuplicateJsPaths({
-		trackedPaths: new Set(trackedPaths),
-		untrackedPaths,
-		readFileText: (relativePath) => NFS.readFileSync(Path.resolve(rootDirectory, relativePath), "utf-8")
-	});
-	return [...trackedPaths, ...untrackedPaths].filter((filePath) => isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath) && !emitDuplicateJsPaths.has(filePath));
+	return result.stdout.split("\0").filter((filePath) => filePath.length > 0 && isLintableSourceFile(filePath) && !hasIgnoredPathSegment(filePath));
 };
 const listSourceFilesViaFilesystem = (rootDirectory) => {
 	const filePaths = [];
@@ -36054,10 +35998,8 @@ const isDiagnosticSurface = (value) => typeof value === "string" && DIAGNOSTIC_S
 * to the local CLI so developers see them while editing, but they're
 * removed from the PR comment surface, the score, and the CI gate so
 * they can't bury real React findings or fail a build over a Tailwind
-* shorthand. Diagnostics from test and story files likewise remain in
-* raw output while staying out of the production-health score and CI
-* gate. Override per-surface via `config.surfaces.<surface>` to promote
-* individual rules back in by tag, category, or rule id.
+* shorthand. Override per-surface via `config.surfaces.<surface>` to
+* promote individual rules back in by tag, category, or rule id.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -38796,7 +38738,6 @@ const isDiagnosticOnSurface = (diagnostic, surface, config) => {
 	if (resolved.includeRuleKeys.has(ruleKey)) return true;
 	if (resolved.includeCategories.has(category)) return true;
 	if (intersects(tags, resolved.includeTags)) return true;
-	if (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure")) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -39221,7 +39162,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}
 			const diff = yield* runGit(input.directory, [
 				"diff",
-				"--no-ext-diff",
 				"-z",
 				"--name-only",
 				"--diff-filter=ACMR",
@@ -39268,7 +39208,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (resolvedCurrentBranch !== null && resolvedCurrentBranch === baseBranch) {
 					const uncommitted = yield* runGit(directory, [
 						"diff",
-						"--no-ext-diff",
 						"-z",
 						"--name-only",
 						"--diff-filter=ACMR",
@@ -39295,7 +39234,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (mergeBaseRef === null) return null;
 				const diff = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"-z",
 					"--name-only",
 					"--diff-filter=ACMR",
@@ -39313,7 +39251,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 			}).pipe(withSpan("Git.diffSelection")),
 			stagedFilePaths: (directory) => runGit(directory, [
 				"diff",
-				"--no-ext-diff",
 				"--cached",
 				"-z",
 				"--name-only",
@@ -39359,7 +39296,6 @@ var Git = class Git extends Service()("react-doctor/Git") {
 				if (baseRef !== void 0 && !isSafeGitRevision(baseRef)) return null;
 				const result = yield* runGit(directory, [
 					"diff",
-					"--no-ext-diff",
 					"--unified=0",
 					"--diff-filter=ACMR",
 					"--relative",
@@ -45347,5 +45283,5 @@ const startLanguageServer = () => {
 };
 //#endregion
 export { startLanguageServer };
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="c9d2ed05-185e-5e8c-bab7-8555349d9999")}catch(e){}}();
-//# debugId=c9d2ed05-185e-5e8c-bab7-8555349d9999
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="3eb1e331-d4d9-517d-a23d-3170e3d3203f")}catch(e){}}();
+//# debugId=3eb1e331-d4d9-517d-a23d-3170e3d3203f

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.7.9-dev.2450582",
+  "version": "0.7.9-dev.2ba83c3",
   "description": "Your agent writes bad React. This catches it",
   "keywords": [
     "accessibility",
@@ -66,7 +66,7 @@
     "vscode-uri": "^3.1.0",
     "yaml": "^2.9.0",
     "deslop-js": "0.7.9",
-    "oxlint-plugin-react-doctor": "0.7.9-dev.2450582"
+    "oxlint-plugin-react-doctor": "0.7.9-dev.2ba83c3"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",
@@ -74,8 +74,8 @@
     "@xterm/headless": "^6.0.0",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@react-doctor/core": "0.7.9",
     "@react-doctor/api": "0.7.9",
+    "@react-doctor/core": "0.7.9",
     "@react-doctor/language-server": "0.7.9"
   },
   "engines": {