react-doctor 0.7.0-dev.6b21b70 → 0.7.0-dev.c0c3fc1

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]="e920d892-391e-5c2c-85c7-689bd4fe9775")}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]="5d16dd1c-3fc1-567b-943f-887174c145c1")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -22405,6 +22405,7 @@ var Diagnostic = class extends Class("Diagnostic")({
 	endLine: optional(Number$1),
 	endColumn: optional(Number$1),
 	category: String$1,
+	matchByOccurrence: optional(Boolean$1),
 	fileContext: optional(Literals(["test", "story"])),
 	suppressionHint: optional(String$1),
 	relatedLocations: optional(ArraySchema(DiagnosticRelatedLocation)),
@@ -36120,6 +36121,7 @@ const DIAGNOSTIC_CATEGORY_BUCKETS = [
 	"Accessibility",
 	"Maintainability"
 ];
+const OCCURRENCE_MATCHED_CATEGORIES = new Set(["Accessibility"]);
 const APP_ONLY_RULE_KEYS = new Set([
 	"react-hooks-js/static-components",
 	"react-doctor/no-render-prop-children",
@@ -43111,6 +43113,7 @@ const parseRuleCode = (code) => {
 	};
 };
 const resolveDiagnosticCategory = (plugin, rule) => getRuleCategory(rule) ?? lookupOwnString(PLUGIN_CATEGORY_MAP, plugin) ?? "Bugs";
+const resolveMatchByOccurrence = (rule, category) => OCCURRENCE_MATCHED_CATEGORIES.has(category) || Boolean(reactDoctorPlugin.rules[rule]?.matchByOccurrence);
 /**
 * Maps oxlint's non-primary labels (`labels[1..]`) into related source
 * locations. Editors surface these as a diagnostic's
@@ -43174,6 +43177,7 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 		const normalizedFilePath = diagnostic.filename.replaceAll("\\", "/");
 		const primarySpan = primaryLabel?.span;
 		const relatedLocations = buildRelatedLocations(diagnostic.labels, normalizedFilePath);
+		const category = resolveDiagnosticCategory(plugin, rule);
 		return {
 			filePath: normalizedFilePath,
 			plugin,
@@ -43189,7 +43193,8 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 				offset: primarySpan.offset,
 				length: primarySpan.length
 			} : {},
-			category: resolveDiagnosticCategory(plugin, rule),
+			category,
+			...resolveMatchByOccurrence(rule, category) ? { matchByOccurrence: true } : {},
 			...relatedLocations.length > 0 ? { relatedLocations } : {}
 		};
 	});
@@ -45425,7 +45430,7 @@ const buildSkippedChecks = (input) => {
 };
 const fingerprintDiagnostic = (diagnostic, lineText) => {
 	const ruleKey = `${diagnostic.plugin}/${diagnostic.rule}`;
-	const snippet = lineText === null ? "" : createHash("sha1").update(lineText.trim()).digest("hex");
+	const snippet = lineText === null || diagnostic.matchByOccurrence ? "" : createHash("sha1").update(lineText.trim()).digest("hex");
 	return `${diagnostic.filePath}\u0000${ruleKey}\u0000${snippet}`;
 };
 /**
@@ -45436,6 +45441,17 @@ const fingerprintDiagnostic = (diagnostic, lineText) => {
 * look new, while a genuinely new occurrence (new line text, or one more of
 * the same) surfaces. Identical repeated findings are matched by count.
 *
+* Diagnostics carrying `matchByOccurrence` (resolved at diagnostic creation:
+* every Accessibility-category finding, plus rules opting in via their
+* `matchByOccurrence` metadata flag) drop the line-text snippet and match by
+* `(filePath, plugin/rule)` occurrence count alone. Their identity is the
+* flagged element, not the line's text, so editing the line (reindentation,
+* prettier reflow, collapsing a multi-line JSX element) doesn't reclassify a
+* pre-existing finding as new — while one MORE occurrence of the same rule in
+* the file still surfaces. Expression-level rules keep the line-text snippet:
+* there the flagged expression IS the finding, so changed text means new +
+* fixed.
+*
 * v1 limitation: the fingerprint keys on the head-relative `filePath`, and base
 * content is read at that same path. A file renamed by the change therefore has
 * no base match, so its pre-existing findings are reported as new. This
@@ -45908,7 +45924,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.7.0-dev.6b21b70";
+const VERSION = "0.7.0-dev.c0c3fc1";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -46277,13 +46293,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.0-dev.6b21b70`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.7.0-dev.c0c3fc1`;
 /**
 * 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.0-dev.6b21b70") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.7.0-dev.c0c3fc1") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -64438,4 +64454,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=e920d892-391e-5c2c-85c7-689bd4fe9775
+//# debugId=5d16dd1c-3fc1-567b-943f-887174c145c1

package/dist/index.d.ts

@@ -9493,6 +9493,17 @@ interface Diagnostic {
   /** 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.

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]="07deeb70-f47f-5572-9930-e743f970f84f")}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]="cf5e895a-deab-574a-b55b-a973e310d0e0")}catch(e){}}();
 import { r as __toESM$1, t as __commonJSMin$1 } from "./chunk-N93fKeF6.js";
 import { createRequire } from "node:module";
 import * as NFS from "node:fs";
@@ -19304,6 +19304,7 @@ var Diagnostic = class extends Class("Diagnostic")({
 	endLine: optional(Number$1),
 	endColumn: optional(Number$1),
 	category: String$1,
+	matchByOccurrence: optional(Boolean$1),
 	fileContext: optional(Literals(["test", "story"])),
 	suppressionHint: optional(String$1),
 	relatedLocations: optional(ArraySchema(DiagnosticRelatedLocation)),
@@ -32919,6 +32920,7 @@ const DIAGNOSTIC_CATEGORY_BUCKETS = [
 	"Accessibility",
 	"Maintainability"
 ];
+const OCCURRENCE_MATCHED_CATEGORIES = new Set(["Accessibility"]);
 const APP_ONLY_RULE_KEYS = new Set([
 	"react-hooks-js/static-components",
 	"react-doctor/no-render-prop-children",
@@ -39868,6 +39870,7 @@ const parseRuleCode = (code) => {
 	};
 };
 const resolveDiagnosticCategory = (plugin, rule) => getRuleCategory(rule) ?? lookupOwnString(PLUGIN_CATEGORY_MAP, plugin) ?? "Bugs";
+const resolveMatchByOccurrence = (rule, category) => OCCURRENCE_MATCHED_CATEGORIES.has(category) || Boolean(reactDoctorPlugin.rules[rule]?.matchByOccurrence);
 /**
 * Maps oxlint's non-primary labels (`labels[1..]`) into related source
 * locations. Editors surface these as a diagnostic's
@@ -39931,6 +39934,7 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 		const normalizedFilePath = diagnostic.filename.replaceAll("\\", "/");
 		const primarySpan = primaryLabel?.span;
 		const relatedLocations = buildRelatedLocations(diagnostic.labels, normalizedFilePath);
+		const category = resolveDiagnosticCategory(plugin, rule);
 		return {
 			filePath: normalizedFilePath,
 			plugin,
@@ -39946,7 +39950,8 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 				offset: primarySpan.offset,
 				length: primarySpan.length
 			} : {},
-			category: resolveDiagnosticCategory(plugin, rule),
+			category,
+			...resolveMatchByOccurrence(rule, category) ? { matchByOccurrence: true } : {},
 			...relatedLocations.length > 0 ? { relatedLocations } : {}
 		};
 	});
@@ -42436,4 +42441,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=07deeb70-f47f-5572-9930-e743f970f84f
+//# debugId=cf5e895a-deab-574a-b55b-a973e310d0e0

package/dist/lsp.js

@@ -19333,6 +19333,7 @@ var Diagnostic = class extends Class("Diagnostic")({
 	endLine: optional(Number$1),
 	endColumn: optional(Number$1),
 	category: String$1,
+	matchByOccurrence: optional(Boolean$1),
 	fileContext: optional(Literals(["test", "story"])),
 	suppressionHint: optional(String$1),
 	relatedLocations: optional(ArraySchema(DiagnosticRelatedLocation)),
@@ -32978,6 +32979,7 @@ const DIAGNOSTIC_CATEGORY_BUCKETS = [
 	"Accessibility",
 	"Maintainability"
 ];
+const OCCURRENCE_MATCHED_CATEGORIES = new Set(["Accessibility"]);
 const APP_ONLY_RULE_KEYS = new Set([
 	"react-hooks-js/static-components",
 	"react-doctor/no-render-prop-children",
@@ -39853,6 +39855,7 @@ const parseRuleCode = (code) => {
 	};
 };
 const resolveDiagnosticCategory = (plugin, rule) => getRuleCategory(rule) ?? lookupOwnString(PLUGIN_CATEGORY_MAP, plugin) ?? "Bugs";
+const resolveMatchByOccurrence = (rule, category) => OCCURRENCE_MATCHED_CATEGORIES.has(category) || Boolean(reactDoctorPlugin.rules[rule]?.matchByOccurrence);
 /**
 * Maps oxlint's non-primary labels (`labels[1..]`) into related source
 * locations. Editors surface these as a diagnostic's
@@ -39916,6 +39919,7 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 		const normalizedFilePath = diagnostic.filename.replaceAll("\\", "/");
 		const primarySpan = primaryLabel?.span;
 		const relatedLocations = buildRelatedLocations(diagnostic.labels, normalizedFilePath);
+		const category = resolveDiagnosticCategory(plugin, rule);
 		return {
 			filePath: normalizedFilePath,
 			plugin,
@@ -39931,7 +39935,8 @@ const parseOxlintOutput = (stdout, project, rootDirectory) => {
 				offset: primarySpan.offset,
 				length: primarySpan.length
 			} : {},
-			category: resolveDiagnosticCategory(plugin, rule),
+			category,
+			...resolveMatchByOccurrence(rule, category) ? { matchByOccurrence: true } : {},
 			...relatedLocations.length > 0 ? { relatedLocations } : {}
 		};
 	});
@@ -44155,5 +44160,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]="1c7dd926-433e-5edb-a6d7-2bb0a9f6df24")}catch(e){}}();
-//# debugId=1c7dd926-433e-5edb-a6d7-2bb0a9f6df24
+!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]="07f4ef15-cbc2-553c-ac41-b73f1d3b7bf8")}catch(e){}}();
+//# debugId=07f4ef15-cbc2-553c-ac41-b73f1d3b7bf8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.7.0-dev.6b21b70",
+  "version": "0.7.0-dev.c0c3fc1",
   "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.0",
-    "oxlint-plugin-react-doctor": "0.7.0-dev.6b21b70"
+    "oxlint-plugin-react-doctor": "0.7.0-dev.c0c3fc1"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",