react-doctor 0.7.9-dev.2450582 → 0.7.9-dev.2979a9b

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]="6f05eada-46fe-5b24-92d8-15fa92a38b00")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -39228,10 +39228,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 +42060,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 +42484,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 +42530,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 +42556,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 +42573,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 +42618,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 +47215,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.7.9-dev.2450582";
+const VERSION = "0.7.9-dev.2979a9b";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -47575,13 +47567,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.2979a9b`;
 /**
 * 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.2979a9b") ? "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 +60068,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 +60090,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 +60143,6 @@ const buildOutcomeAttributes = (input) => {
 			affectedFiles: summary.affectedFileCount,
 			inTestFiles: diagnosticsInTestFiles,
 			inStoryFiles: diagnosticsInStoryFiles,
-			nonProductionGateExcluded,
 			distinctRules: countByRule.size,
 			topRule,
 			sameSiteOccurrences,
@@ -61994,9 +61982,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 +63883,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 +63906,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 +65851,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=c29245d6-1073-542d-91c7-90d62f3911e2
+//# debugId=6f05eada-46fe-5b24-92d8-15fa92a38b00

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]="28287c1c-b7b6-53b8-9e72-7d28d4f3ca45")}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";
@@ -36028,10 +36028,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 +38816,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 +39240,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 +39286,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 +39312,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 +39329,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 +39374,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 +43719,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=28287c1c-b7b6-53b8-9e72-7d28d4f3ca45

package/dist/lsp.js

@@ -36054,10 +36054,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 +38794,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 +39218,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 +39264,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 +39290,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 +39307,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 +39352,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 +45339,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]="6c89dec9-5fbe-5a2e-b1fe-6b3954e0e85b")}catch(e){}}();
+//# debugId=6c89dec9-5fbe-5a2e-b1fe-6b3954e0e85b

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.7.9-dev.2450582",
+  "version": "0.7.9-dev.2979a9b",
   "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.2979a9b"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",