react-doctor 0.8.0-dev.c0d9d38 → 0.8.0-dev.eb2d687

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]="4433c20d-e5c5-50db-ab5b-0ff72ef6f226")}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]="cfa479fc-64e6-5081-86f2-1cbef233c1a0")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -38359,6 +38359,9 @@ const SCRIPT_EXTENSION_FRAGMENT = "[cm]?[jt]sx?";
 const STORY_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:stories|story)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:test|spec|fixture|fixtures)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_DIRECTORY_PATTERN = /(?:^|\/)(?:__tests__|__test__|tests|test|__mocks__|cypress|e2e|playwright)\//;
+const AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN = /(?:^|\/)(?:tests|test)\//;
+const NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN = /\/(?:default|error|forbidden|global-error|layout|loading|not-found|page|route|template|unauthorized)\.[cm]?[jt]sx?$/;
+const SCRIPT_FILE_SUFFIX_PATTERN = /\.[cm]?[jt]sx?$/;
 const FIXTURE_PROJECT_PATTERN = /\/(?:fixtures|__fixtures__)\//;
 const SOURCE_ROOT_PATTERN = /\/(?:src|app|lib|components|pages|features|modules|packages|apps|frontend|client)\//g;
 const stripAboveSourceRoot = (relativePath) => {
@@ -38369,6 +38372,14 @@ const stripAboveSourceRoot = (relativePath) => {
 	if (lastIdx >= 0) return relativePath.slice(lastIdx);
 	return relativePath.slice(fixtureMatch.index + fixtureMatch[0].length - 1);
 };
+const isFrameworkRouteBeforeAmbiguousTestDirectory = (relativePath) => {
+	const testDirectoryMatch = AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN.exec(relativePath);
+	if (testDirectoryMatch === null) return false;
+	let nearestRouteRoot;
+	for (const pathSegment of relativePath.slice(0, testDirectoryMatch.index).split("/")) if (pathSegment === "app" || pathSegment === "pages") nearestRouteRoot = pathSegment;
+	if (nearestRouteRoot === void 0) return false;
+	return nearestRouteRoot === "pages" ? SCRIPT_FILE_SUFFIX_PATTERN.test(relativePath) : NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN.test(relativePath);
+};
 /**
 * Classifies where a file sits relative to shipped code. A finding in a
 * `.stories.tsx` or `.spec.ts` file never runs in front of users, so
@@ -38385,6 +38396,7 @@ const classifyFileContext = (relativePath) => {
 	if (STORY_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "story";
 	if (TEST_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "test";
 	const scoped = stripAboveSourceRoot(forwardSlashed);
+	if (isFrameworkRouteBeforeAmbiguousTestDirectory(scoped)) return "production";
 	return TEST_FILE_DIRECTORY_PATTERN.test(scoped) ? "test" : "production";
 };
 /**
@@ -39244,6 +39256,7 @@ const VALID_RULE_SEVERITIES = [
 	"warn",
 	"off"
 ];
+const VALID_INCLUDED_FILE_CONTEXTS = ["test", "story"];
 const KNOWN_CATEGORY_LABEL = DIAGNOSTIC_CATEGORY_BUCKETS.join(", ");
 const isDiagnosticCategoryBucket = (value) => DIAGNOSTIC_CATEGORY_BUCKETS.includes(value);
 const filterKnownCategories = (fieldName, categories) => categories.filter((category) => {
@@ -39278,6 +39291,7 @@ const SURFACE_CONTROL_FIELD_NAMES = [
 	"includeRules",
 	"excludeRules"
 ];
+const isIncludedFileContext = (value) => VALID_INCLUDED_FILE_CONTEXTS.some((fileContext) => fileContext === value);
 const SEVERITY_FIELD_NAMES = ["rules", "categories"];
 const isPlainObject$1 = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 const formatType = (value) => typeof value === "string" ? `"${value}"` : typeof value;
@@ -39312,6 +39326,15 @@ const validateSurfaceControls = (surface, rawControls) => {
 		return;
 	}
 	const validatedSurfaceControls = {};
+	if (rawControls.includeFileContexts !== void 0) {
+		const qualifiedName = `surfaces.${surface}.includeFileContexts`;
+		const result = validateStringArrayField(qualifiedName, rawControls.includeFileContexts);
+		if (result !== void 0) validatedSurfaceControls.includeFileContexts = result.filter((fileContext) => {
+			if (isIncludedFileContext(fileContext)) return true;
+			warnConfigIssue(`config field "${qualifiedName}" lists "${fileContext}", which is not a non-production file context (expected one of: ${VALID_INCLUDED_FILE_CONTEXTS.join(", ")}); ignoring the entry.`);
+			return false;
+		});
+	}
 	for (const fieldName of SURFACE_CONTROL_FIELD_NAMES) {
 		if (rawControls[fieldName] === void 0) continue;
 		const qualifiedName = `surfaces.${surface}.${fieldName}`;
@@ -42047,6 +42070,7 @@ const buildResolvedControls = (surface, userControls) => {
 	for (const tag of includeTags) excludeTags.delete(tag);
 	for (const tag of toStringSet(userControls?.excludeTags)) excludeTags.add(tag);
 	return {
+		includeFileContexts: toStringSet(userControls?.includeFileContexts),
 		includeTags,
 		excludeTags,
 		includeCategories: toStringSet(userControls?.includeCategories),
@@ -42062,7 +42086,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 (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure") && !resolved.includeFileContexts.has(diagnostic.fileContext)) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -47223,7 +47247,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.8.0-dev.c0d9d38";
+const VERSION = "0.8.0-dev.eb2d687";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -47575,13 +47599,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.8.0-dev.c0d9d38`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.8.0-dev.eb2d687`;
 /**
 * 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.8.0-dev.c0d9d38") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.8.0-dev.eb2d687") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -65867,4 +65891,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=4433c20d-e5c5-50db-ab5b-0ff72ef6f226
+//# debugId=cfa479fc-64e6-5081-86f2-1cbef233c1a0

package/dist/index.d.ts

@@ -8980,6 +8980,83 @@ declare namespace Annotations {
 }
 //#endregion
 //#region ../core/dist/index.d.ts
+/**
+ * A secondary source location attached to a diagnostic (oxlint's
+ * non-primary labels). Editors render these as
+ * `Diagnostic.relatedInformation` so a user can jump from, say, a
+ * `no-derived-state` finding to the originating prop declaration.
+ * Positions mirror `Diagnostic` semantics: `line` / `column` are
+ * 1-indexed, and `offset` / `length` are UTF-8 byte spans from oxlint
+ * when available (preferred for precise ranges).
+ */
+interface DiagnosticRelatedLocation {
+  filePath: string;
+  line: number;
+  column: number;
+  offset?: number;
+  length?: number;
+  endLine?: number;
+  endColumn?: number;
+  message: string;
+}
+type DiagnosticFileContext = "test" | "story" | "production";
+interface Diagnostic {
+  filePath: string;
+  plugin: string;
+  rule: string;
+  severity: "error" | "warning";
+  title?: string;
+  message: string;
+  help: string;
+  url?: string;
+  line: number;
+  column: number;
+  /**
+   * UTF-8 byte offset of the diagnostic's primary span start, straight
+   * from oxlint's label span. Optional because environment / dead-code
+   * diagnostics carry no source span. Editors (LSP) convert this into a
+   * precise range via the in-memory document; non-editor consumers can
+   * ignore it.
+   */
+  offset?: number;
+  /** UTF-8 byte length of the primary span (pairs with `offset`). */
+  length?: number;
+  /** 1-indexed end line of the primary span, when derivable. */
+  endLine?: number;
+  /** 1-indexed end column of the primary span, when derivable. */
+  endColumn?: number;
+  category: string;
+  /**
+   * Set when the finding's identity is the flagged element itself (a missing
+   * attribute, a wrong element) rather than the flagged line's text — every
+   * Accessibility-category finding, plus rules that opt in via their
+   * `matchByOccurrence` metadata flag. `computeDiagnosticDelta` matches these
+   * by `(filePath, plugin/rule)` occurrence count, so reformatting the flagged
+   * line doesn't reclassify a pre-existing finding as new. Resolved at
+   * diagnostic creation, where rule metadata is available. Absent means
+   * line-text-sensitive matching (the default).
+   */
+  matchByOccurrence?: boolean;
+  /**
+   * Set when the file never ships to users (`"test"` / `"story"`), so
+   * renderers can label the site instead of implying production impact.
+   * Omitted for production files (the default).
+   */
+  fileContext?: Exclude<DiagnosticFileContext, "production">;
+  suppressionHint?: string;
+  /** Secondary source locations (oxlint's non-primary labels). */
+  relatedLocations?: DiagnosticRelatedLocation[];
+  /**
+   * Stable id shared by every finding that a single fix resolves together —
+   * e.g. four `useEffect`s that reset state on one prop change all clear with
+   * one `key` prop. Set only when ≥2 findings share a root cause; absent for
+   * standalone findings. A consumer that turns findings into work items should
+   * group by it so one fix reads as one task, not N. Presentation-only and
+   * score-neutral — the score never reads it.
+   */
+  fixGroupId?: string;
+}
+//#endregion
 //#region src/types/config.d.ts
 /**
  * Severity threshold at which a scan blocks CI (exits non-zero). Controlled
@@ -9057,9 +9134,9 @@ 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.
+ * are excluded from `score` and `ciFailure`. Both remain in `cli`. Explicit
+ * rule, category, or tag includes promote matching diagnostics, while
+ * `includeFileContexts` restores a context without bypassing other exclusions.
  */
 type DiagnosticSurface = "cli" | "prComment" | "score" | "ciFailure";
 /**
@@ -9089,6 +9166,11 @@ interface SeverityBuckets {
   "compiler-cleanup"?: RuleSeverityOverride;
 }
 interface SurfaceControls {
+  /**
+   * Non-production file contexts to retain on this surface. This restores
+   * test or story findings without overriding unrelated tag exclusions.
+   */
+  includeFileContexts?: Array<Exclude<DiagnosticFileContext, "production">>;
   /**
    * Tag names whose diagnostics should be force-included on the surface,
    * even if a default or category-level exclusion would otherwise drop
@@ -9353,9 +9435,9 @@ interface ReactDoctorConfig {
    * - `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.
+   * Your include/exclude entries layer on top. Rule, category, and tag includes
+   * win over their exclusions. Included file contexts still respect unrelated
+   * exclusions, so restoring story findings does not restore every design rule.
    */
   surfaces?: Partial<Record<DiagnosticSurface, SurfaceControls>>;
   /**
@@ -9449,84 +9531,6 @@ interface ReactDoctorConfig {
    */
   plugins?: string[];
 } //#endregion
-//#region src/types/diagnostic.d.ts
-/**
- * A secondary source location attached to a diagnostic (oxlint's
- * non-primary labels). Editors render these as
- * `Diagnostic.relatedInformation` so a user can jump from, say, a
- * `no-derived-state` finding to the originating prop declaration.
- * Positions mirror `Diagnostic` semantics: `line` / `column` are
- * 1-indexed, and `offset` / `length` are UTF-8 byte spans from oxlint
- * when available (preferred for precise ranges).
- */
-interface DiagnosticRelatedLocation {
-  filePath: string;
-  line: number;
-  column: number;
-  offset?: number;
-  length?: number;
-  endLine?: number;
-  endColumn?: number;
-  message: string;
-}
-type DiagnosticFileContext = "test" | "story" | "production";
-interface Diagnostic {
-  filePath: string;
-  plugin: string;
-  rule: string;
-  severity: "error" | "warning";
-  title?: string;
-  message: string;
-  help: string;
-  url?: string;
-  line: number;
-  column: number;
-  /**
-   * UTF-8 byte offset of the diagnostic's primary span start, straight
-   * from oxlint's label span. Optional because environment / dead-code
-   * diagnostics carry no source span. Editors (LSP) convert this into a
-   * precise range via the in-memory document; non-editor consumers can
-   * ignore it.
-   */
-  offset?: number;
-  /** UTF-8 byte length of the primary span (pairs with `offset`). */
-  length?: number;
-  /** 1-indexed end line of the primary span, when derivable. */
-  endLine?: number;
-  /** 1-indexed end column of the primary span, when derivable. */
-  endColumn?: number;
-  category: string;
-  /**
-   * Set when the finding's identity is the flagged element itself (a missing
-   * attribute, a wrong element) rather than the flagged line's text — every
-   * Accessibility-category finding, plus rules that opt in via their
-   * `matchByOccurrence` metadata flag. `computeDiagnosticDelta` matches these
-   * by `(filePath, plugin/rule)` occurrence count, so reformatting the flagged
-   * line doesn't reclassify a pre-existing finding as new. Resolved at
-   * diagnostic creation, where rule metadata is available. Absent means
-   * line-text-sensitive matching (the default).
-   */
-  matchByOccurrence?: boolean;
-  /**
-   * Set when the file never ships to users (`"test"` / `"story"`), so
-   * renderers can label the site instead of implying production impact.
-   * Omitted for production files (the default).
-   */
-  fileContext?: Exclude<DiagnosticFileContext, "production">;
-  suppressionHint?: string;
-  /** Secondary source locations (oxlint's non-primary labels). */
-  relatedLocations?: DiagnosticRelatedLocation[];
-  /**
-   * Stable id shared by every finding that a single fix resolves together —
-   * e.g. four `useEffect`s that reset state on one prop change all clear with
-   * one `key` prop. Set only when ≥2 findings share a root cause; absent for
-   * standalone findings. A consumer that turns findings into work items should
-   * group by it so one fix reads as one task, not N. Presentation-only and
-   * score-neutral — the score never reads it.
-   */
-  fixGroupId?: string;
-}
-//#endregion
 //#region src/types/project-info.d.ts
 type Framework = FrameworkToken;
 interface ProjectInfo {

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]="81e065f1-a43b-51c2-ac53-2f38390032ed")}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]="4a473a3e-0567-5c2a-8eea-947cdfc7b9ee")}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";
@@ -35155,6 +35155,9 @@ const SCRIPT_EXTENSION_FRAGMENT = "[cm]?[jt]sx?";
 const STORY_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:stories|story)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:test|spec|fixture|fixtures)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_DIRECTORY_PATTERN = /(?:^|\/)(?:__tests__|__test__|tests|test|__mocks__|cypress|e2e|playwright)\//;
+const AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN = /(?:^|\/)(?:tests|test)\//;
+const NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN = /\/(?:default|error|forbidden|global-error|layout|loading|not-found|page|route|template|unauthorized)\.[cm]?[jt]sx?$/;
+const SCRIPT_FILE_SUFFIX_PATTERN = /\.[cm]?[jt]sx?$/;
 const FIXTURE_PROJECT_PATTERN = /\/(?:fixtures|__fixtures__)\//;
 const SOURCE_ROOT_PATTERN = /\/(?:src|app|lib|components|pages|features|modules|packages|apps|frontend|client)\//g;
 const stripAboveSourceRoot = (relativePath) => {
@@ -35165,6 +35168,14 @@ const stripAboveSourceRoot = (relativePath) => {
 	if (lastIdx >= 0) return relativePath.slice(lastIdx);
 	return relativePath.slice(fixtureMatch.index + fixtureMatch[0].length - 1);
 };
+const isFrameworkRouteBeforeAmbiguousTestDirectory = (relativePath) => {
+	const testDirectoryMatch = AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN.exec(relativePath);
+	if (testDirectoryMatch === null) return false;
+	let nearestRouteRoot;
+	for (const pathSegment of relativePath.slice(0, testDirectoryMatch.index).split("/")) if (pathSegment === "app" || pathSegment === "pages") nearestRouteRoot = pathSegment;
+	if (nearestRouteRoot === void 0) return false;
+	return nearestRouteRoot === "pages" ? SCRIPT_FILE_SUFFIX_PATTERN.test(relativePath) : NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN.test(relativePath);
+};
 /**
 * Classifies where a file sits relative to shipped code. A finding in a
 * `.stories.tsx` or `.spec.ts` file never runs in front of users, so
@@ -35181,6 +35192,7 @@ const classifyFileContext = (relativePath) => {
 	if (STORY_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "story";
 	if (TEST_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "test";
 	const scoped = stripAboveSourceRoot(forwardSlashed);
+	if (isFrameworkRouteBeforeAmbiguousTestDirectory(scoped)) return "production";
 	return TEST_FILE_DIRECTORY_PATTERN.test(scoped) ? "test" : "production";
 };
 /**
@@ -36044,6 +36056,7 @@ const VALID_RULE_SEVERITIES = [
 	"warn",
 	"off"
 ];
+const VALID_INCLUDED_FILE_CONTEXTS = ["test", "story"];
 const KNOWN_CATEGORY_LABEL = DIAGNOSTIC_CATEGORY_BUCKETS.join(", ");
 const isDiagnosticCategoryBucket = (value) => DIAGNOSTIC_CATEGORY_BUCKETS.includes(value);
 const filterKnownCategories = (fieldName, categories) => categories.filter((category) => {
@@ -36078,6 +36091,7 @@ const SURFACE_CONTROL_FIELD_NAMES = [
 	"includeRules",
 	"excludeRules"
 ];
+const isIncludedFileContext = (value) => VALID_INCLUDED_FILE_CONTEXTS.some((fileContext) => fileContext === value);
 const SEVERITY_FIELD_NAMES = ["rules", "categories"];
 const isPlainObject$1 = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 const formatType = (value) => typeof value === "string" ? `"${value}"` : typeof value;
@@ -36112,6 +36126,15 @@ const validateSurfaceControls = (surface, rawControls) => {
 		return;
 	}
 	const validatedSurfaceControls = {};
+	if (rawControls.includeFileContexts !== void 0) {
+		const qualifiedName = `surfaces.${surface}.includeFileContexts`;
+		const result = validateStringArrayField(qualifiedName, rawControls.includeFileContexts);
+		if (result !== void 0) validatedSurfaceControls.includeFileContexts = result.filter((fileContext) => {
+			if (isIncludedFileContext(fileContext)) return true;
+			warnConfigIssue(`config field "${qualifiedName}" lists "${fileContext}", which is not a non-production file context (expected one of: ${VALID_INCLUDED_FILE_CONTEXTS.join(", ")}); ignoring the entry.`);
+			return false;
+		});
+	}
 	for (const fieldName of SURFACE_CONTROL_FIELD_NAMES) {
 		if (rawControls[fieldName] === void 0) continue;
 		const qualifiedName = `surfaces.${surface}.${fieldName}`;
@@ -38803,6 +38826,7 @@ const buildResolvedControls = (surface, userControls) => {
 	for (const tag of includeTags) excludeTags.delete(tag);
 	for (const tag of toStringSet(userControls?.excludeTags)) excludeTags.add(tag);
 	return {
+		includeFileContexts: toStringSet(userControls?.includeFileContexts),
 		includeTags,
 		excludeTags,
 		includeCategories: toStringSet(userControls?.includeCategories),
@@ -38818,7 +38842,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 (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure") && !resolved.includeFileContexts.has(diagnostic.fileContext)) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -43727,4 +43751,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=81e065f1-a43b-51c2-ac53-2f38390032ed
+//# debugId=4a473a3e-0567-5c2a-8eea-947cdfc7b9ee

package/dist/lsp.js

@@ -35207,6 +35207,9 @@ const SCRIPT_EXTENSION_FRAGMENT = "[cm]?[jt]sx?";
 const STORY_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:stories|story)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_SUFFIX_PATTERN = new RegExp(`\\.(?:test|spec|fixture|fixtures)\\.(?:${SCRIPT_EXTENSION_FRAGMENT})$`);
 const TEST_FILE_DIRECTORY_PATTERN = /(?:^|\/)(?:__tests__|__test__|tests|test|__mocks__|cypress|e2e|playwright)\//;
+const AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN = /(?:^|\/)(?:tests|test)\//;
+const NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN = /\/(?:default|error|forbidden|global-error|layout|loading|not-found|page|route|template|unauthorized)\.[cm]?[jt]sx?$/;
+const SCRIPT_FILE_SUFFIX_PATTERN = /\.[cm]?[jt]sx?$/;
 const FIXTURE_PROJECT_PATTERN = /\/(?:fixtures|__fixtures__)\//;
 const SOURCE_ROOT_PATTERN = /\/(?:src|app|lib|components|pages|features|modules|packages|apps|frontend|client)\//g;
 const stripAboveSourceRoot = (relativePath) => {
@@ -35217,6 +35220,14 @@ const stripAboveSourceRoot = (relativePath) => {
 	if (lastIdx >= 0) return relativePath.slice(lastIdx);
 	return relativePath.slice(fixtureMatch.index + fixtureMatch[0].length - 1);
 };
+const isFrameworkRouteBeforeAmbiguousTestDirectory = (relativePath) => {
+	const testDirectoryMatch = AMBIGUOUS_TEST_ROUTE_DIRECTORY_PATTERN.exec(relativePath);
+	if (testDirectoryMatch === null) return false;
+	let nearestRouteRoot;
+	for (const pathSegment of relativePath.slice(0, testDirectoryMatch.index).split("/")) if (pathSegment === "app" || pathSegment === "pages") nearestRouteRoot = pathSegment;
+	if (nearestRouteRoot === void 0) return false;
+	return nearestRouteRoot === "pages" ? SCRIPT_FILE_SUFFIX_PATTERN.test(relativePath) : NEXT_APP_ROUTE_FILE_SUFFIX_PATTERN.test(relativePath);
+};
 /**
 * Classifies where a file sits relative to shipped code. A finding in a
 * `.stories.tsx` or `.spec.ts` file never runs in front of users, so
@@ -35233,6 +35244,7 @@ const classifyFileContext = (relativePath) => {
 	if (STORY_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "story";
 	if (TEST_FILE_SUFFIX_PATTERN.test(forwardSlashed)) return "test";
 	const scoped = stripAboveSourceRoot(forwardSlashed);
+	if (isFrameworkRouteBeforeAmbiguousTestDirectory(scoped)) return "production";
 	return TEST_FILE_DIRECTORY_PATTERN.test(scoped) ? "test" : "production";
 };
 /**
@@ -36070,6 +36082,7 @@ const VALID_RULE_SEVERITIES = [
 	"warn",
 	"off"
 ];
+const VALID_INCLUDED_FILE_CONTEXTS = ["test", "story"];
 const KNOWN_CATEGORY_LABEL = DIAGNOSTIC_CATEGORY_BUCKETS.join(", ");
 const isDiagnosticCategoryBucket = (value) => DIAGNOSTIC_CATEGORY_BUCKETS.includes(value);
 const filterKnownCategories = (fieldName, categories) => categories.filter((category) => {
@@ -36104,6 +36117,7 @@ const SURFACE_CONTROL_FIELD_NAMES = [
 	"includeRules",
 	"excludeRules"
 ];
+const isIncludedFileContext = (value) => VALID_INCLUDED_FILE_CONTEXTS.some((fileContext) => fileContext === value);
 const SEVERITY_FIELD_NAMES = ["rules", "categories"];
 const isPlainObject$1 = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 const formatType = (value) => typeof value === "string" ? `"${value}"` : typeof value;
@@ -36138,6 +36152,15 @@ const validateSurfaceControls = (surface, rawControls) => {
 		return;
 	}
 	const validatedSurfaceControls = {};
+	if (rawControls.includeFileContexts !== void 0) {
+		const qualifiedName = `surfaces.${surface}.includeFileContexts`;
+		const result = validateStringArrayField(qualifiedName, rawControls.includeFileContexts);
+		if (result !== void 0) validatedSurfaceControls.includeFileContexts = result.filter((fileContext) => {
+			if (isIncludedFileContext(fileContext)) return true;
+			warnConfigIssue(`config field "${qualifiedName}" lists "${fileContext}", which is not a non-production file context (expected one of: ${VALID_INCLUDED_FILE_CONTEXTS.join(", ")}); ignoring the entry.`);
+			return false;
+		});
+	}
 	for (const fieldName of SURFACE_CONTROL_FIELD_NAMES) {
 		if (rawControls[fieldName] === void 0) continue;
 		const qualifiedName = `surfaces.${surface}.${fieldName}`;
@@ -38781,6 +38804,7 @@ const buildResolvedControls = (surface, userControls) => {
 	for (const tag of includeTags) excludeTags.delete(tag);
 	for (const tag of toStringSet(userControls?.excludeTags)) excludeTags.add(tag);
 	return {
+		includeFileContexts: toStringSet(userControls?.includeFileContexts),
 		includeTags,
 		excludeTags,
 		includeCategories: toStringSet(userControls?.includeCategories),
@@ -38796,7 +38820,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 (diagnostic.fileContext !== void 0 && (surface === "score" || surface === "ciFailure") && !resolved.includeFileContexts.has(diagnostic.fileContext)) return false;
 	if (resolved.excludeRuleKeys.has(ruleKey)) return false;
 	if (resolved.excludeCategories.has(category)) return false;
 	if (intersects(tags, resolved.excludeTags)) return false;
@@ -45347,5 +45371,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]="9eca6678-56ba-5032-b04c-b1ed965dffb4")}catch(e){}}();
-//# debugId=9eca6678-56ba-5032-b04c-b1ed965dffb4
+!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]="2dde2cd1-f390-5e9f-8370-937973c22f36")}catch(e){}}();
+//# debugId=2dde2cd1-f390-5e9f-8370-937973c22f36

package/dist/skills/react-doctor/references/explain.md

@@ -51,6 +51,7 @@ Match the control to the intent — prefer the narrowest one:
 - **A whole area is unwanted** (e.g. all React Native rules) → `rules category "<Category>" off`.
 - **A behavioral family is noisy** (`design`, `test-noise`, `migration-hint`) → `rules ignore-tag <tag>`.
 - **Keep it locally but hide from PR comment / score / CI gate only** → do NOT disable. Edit `surfaces` in your config (`surfaces.prComment.excludeRules`, `surfaces.score.excludeTags`, `surfaces.ciFailure.excludeCategories`). The rule still shows in local `cli` output.
+- **Restore test or story findings to production health** → set `surfaces.score.includeFileContexts` or `surfaces.ciFailure.includeFileContexts` to `["test"]`, `["story"]`, or both. Other surface exclusions still apply.
 
 How the layers combine: `ignore.tags` disables every rule carrying that tag **before** linting, so a tagged rule stays off even if `rules`/`categories` set it to `warn`/`error` (a rule-level override cannot re-enable a tag-ignored rule). For rules that aren't tag-disabled, `rules` overrides `categories` overrides the rule's default. `surfaces` is visibility-only and never changes whether a rule runs.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.8.0-dev.c0d9d38",
+  "version": "0.8.0-dev.eb2d687",
   "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.8.0",
-    "oxlint-plugin-react-doctor": "0.8.0-dev.c0d9d38"
+    "oxlint-plugin-react-doctor": "0.8.0-dev.eb2d687"
   },
   "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.8.0",
     "@react-doctor/api": "0.8.0",
+    "@react-doctor/core": "0.8.0",
     "@react-doctor/language-server": "0.8.0"
   },
   "engines": {