react-doctor 0.7.9-dev.21043e0 → 0.7.9-dev.2450582

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]="6f05eada-46fe-5b24-92d8-15fa92a38b00")}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]="c29245d6-1073-542d-91c7-90d62f3911e2")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -39228,8 +39228,10 @@ 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. Override per-surface via `config.surfaces.<surface>` to
-* promote individual rules back in by tag, category, or rule id.
+* 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.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -42060,6 +42062,7 @@ 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;
@@ -42484,6 +42487,7 @@ 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",
@@ -42530,6 +42534,7 @@ 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",
@@ -42556,6 +42561,7 @@ 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",
@@ -42573,6 +42579,7 @@ 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",
@@ -42618,6 +42625,7 @@ 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",
@@ -47215,7 +47223,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.7.9-dev.21043e0";
+const VERSION = "0.7.9-dev.2450582";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -47567,13 +47575,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.21043e0`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.7.9-dev.2450582`;
 /**
 * 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.21043e0") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.7.9-dev.2450582") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -60068,6 +60076,7 @@ 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";
@@ -60090,11 +60099,13 @@ 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,
@@ -60143,6 +60154,7 @@ const buildOutcomeAttributes = (input) => {
 			affectedFiles: summary.affectedFileCount,
 			inTestFiles: diagnosticsInTestFiles,
 			inStoryFiles: diagnosticsInStoryFiles,
+			nonProductionGateExcluded,
 			distinctRules: countByRule.size,
 			topRule,
 			sameSiteOccurrences,
@@ -61982,7 +61994,9 @@ 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 potentialScore = score ? yield* promise(() => computeProjectedScore([...printedDiagnostics], [...surfaceDiagnostics], score)) : null;
+	const scoreDiagnostics = filterDiagnosticsForSurface([...diagnostics], "score", userConfig);
+	const displayedScoreDiagnostics = filterDiagnosticsForSurface([...printedDiagnostics], "score", userConfig);
+	const potentialScore = score ? yield* promise(() => computeProjectedScore(displayedScoreDiagnostics, scoreDiagnostics, score)) : null;
 	const shouldShowShareLink = !options.noScore && options.share && !options.isCi;
 	yield* pause;
 	yield* printSummary({
@@ -63883,6 +63897,8 @@ 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) ?? "";
@@ -63906,7 +63922,7 @@ const printMultiProjectSummary = (input) => gen(function* () {
 		diagnostics: displayDiagnostics,
 		elapsedMilliseconds: totalElapsedMilliseconds,
 		scoreResult: aggregateScore,
-		potentialScore: lowestScoredScan ? yield* promise(() => computeProjectedScore(displayDiagnostics, filterScansForSurface([lowestScoredScan], "cli"), lowestScoredScan.result.score)) : null,
+		potentialScore: lowestScoredScan ? yield* promise(() => computeProjectedScore(displayedScoreDiagnostics, filterScansForSurface([lowestScoredScan], "score"), lowestScoredScan.result.score)) : null,
 		totalSourceFileCount,
 		noScoreMessage: "Score unavailable.",
 		verbose,
@@ -65851,4 +65867,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=6f05eada-46fe-5b24-92d8-15fa92a38b00
+//# debugId=c29245d6-1073-542d-91c7-90d62f3911e2

package/dist/index.d.ts

@@ -9056,8 +9056,10 @@ interface ReactDoctorIgnoreConfig {
  *
  * Defaults: design rules (tag `"design"`) are excluded from `prComment`,
  * `score`, and `ciFailure` so style cleanup doesn't dilute meaningful
- * React findings. They remain in `cli` so locally-running developers
- * still see the suggestion when they touch the file.
+ * 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.
  */
 type DiagnosticSurface = "cli" | "prComment" | "score" | "ciFailure";
 /**
@@ -9338,23 +9340,22 @@ interface ReactDoctorConfig {
    */
   adoptExistingLintConfig?: boolean;
   /**
-   * 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.
+   * 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.
    *
    * 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 (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.
+   * 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.
    */
   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]="28287c1c-b7b6-53b8-9e72-7d28d4f3ca45")}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]="a80081fb-3090-5160-8adf-e2652ffe1212")}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,8 +36028,10 @@ 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. Override per-surface via `config.surfaces.<surface>` to
-* promote individual rules back in by tag, category, or rule id.
+* 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.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -38816,6 +38818,7 @@ 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;
@@ -39240,6 +39243,7 @@ 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",
@@ -39286,6 +39290,7 @@ 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",
@@ -39312,6 +39317,7 @@ 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",
@@ -39329,6 +39335,7 @@ 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",
@@ -39374,6 +39381,7 @@ 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",
@@ -43719,4 +43727,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=28287c1c-b7b6-53b8-9e72-7d28d4f3ca45
+//# debugId=a80081fb-3090-5160-8adf-e2652ffe1212

package/dist/lsp.js

@@ -36054,8 +36054,10 @@ 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. Override per-surface via `config.surfaces.<surface>` to
-* promote individual rules back in by tag, category, or rule id.
+* 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.
 */
 const DEFAULT_SURFACE_EXCLUDED_TAGS = {
 	cli: [],
@@ -38794,6 +38796,7 @@ 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;
@@ -39218,6 +39221,7 @@ 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",
@@ -39264,6 +39268,7 @@ 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",
@@ -39290,6 +39295,7 @@ 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",
@@ -39307,6 +39313,7 @@ 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",
@@ -39352,6 +39359,7 @@ 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",
@@ -45339,5 +45347,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]="6c89dec9-5fbe-5a2e-b1fe-6b3954e0e85b")}catch(e){}}();
-//# debugId=6c89dec9-5fbe-5a2e-b1fe-6b3954e0e85b
+!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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.7.9-dev.21043e0",
+  "version": "0.7.9-dev.2450582",
   "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.21043e0"
+    "oxlint-plugin-react-doctor": "0.7.9-dev.2450582"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",
@@ -75,8 +75,8 @@
     "commander": "^14.0.3",
     "ora": "^9.4.0",
     "@react-doctor/core": "0.7.9",
-    "@react-doctor/language-server": "0.7.9",
-    "@react-doctor/api": "0.7.9"
+    "@react-doctor/api": "0.7.9",
+    "@react-doctor/language-server": "0.7.9"
   },
   "engines": {
     "node": "^20.19.0 || >=22.13.0"