oxlint-plugin-react-doctor 0.6.2-dev.ac71a3b → 0.6.2-dev.d8628d7

package/dist/index.js

@@ -378,13 +378,18 @@ const isBrowserArtifactPath = (relativePath, isGeneratedBundle) => {
 const isConfigOrCiPath = (relativePath) => /(?:^|\/)(?:package\.json|Dockerfile|docker-compose\.ya?ml|\.github\/workflows\/[^/]+\.ya?ml|vercel\.json|next\.config\.[cm]?[jt]s|netlify\.toml)$/i.test(relativePath);
 //#endregion
 //#region src/plugin/rules/security-scan/utils/is-production-file-path.ts
+const classificationCacheByPattern = /* @__PURE__ */ new Map();
 const isProductionFilePath = (relativePath, sourceFilePattern) => {
-	if (!sourceFilePattern.test(relativePath)) return false;
-	if (TEST_CONTEXT_PATTERN.test(relativePath)) return false;
-	if (BUILD_SCRIPT_CONTEXT_PATTERN.test(relativePath)) return false;
-	if (DOCUMENTATION_CONTEXT_PATTERN.test(relativePath)) return false;
-	if (GENERATED_SOURCE_CONTEXT_PATTERN.test(relativePath)) return false;
-	return true;
+	let classificationByPath = classificationCacheByPattern.get(sourceFilePattern);
+	if (!classificationByPath) {
+		classificationByPath = /* @__PURE__ */ new Map();
+		classificationCacheByPattern.set(sourceFilePattern, classificationByPath);
+	}
+	const cached = classificationByPath.get(relativePath);
+	if (cached !== void 0) return cached;
+	const isProduction = sourceFilePattern.test(relativePath) && !TEST_CONTEXT_PATTERN.test(relativePath) && !BUILD_SCRIPT_CONTEXT_PATTERN.test(relativePath) && !DOCUMENTATION_CONTEXT_PATTERN.test(relativePath) && !GENERATED_SOURCE_CONTEXT_PATTERN.test(relativePath);
+	classificationByPath.set(relativePath, isProduction);
+	return isProduction;
 };
 //#endregion
 //#region src/plugin/rules/security-scan/utils/is-production-source-path.ts
@@ -1677,7 +1682,7 @@ const isNextjsMetadataImageRouteFilename = (rawFilename) => {
 };
 //#endregion
 //#region src/plugin/utils/normalize-filename.ts
-const normalizeFilename = (filename) => filename.replaceAll("\\", "/");
+const normalizeFilename = (filename) => filename.includes("\\") ? filename.replaceAll("\\", "/") : filename;
 //#endregion
 //#region src/plugin/utils/is-generated-image-render-context.ts
 const IMAGE_RESPONSE_MODULES = ["next/og", "@vercel/og"];
@@ -15521,16 +15526,30 @@ const stripJsComments = (sourceText) => sourceText.replace(BLOCK_COMMENT_PATTERN
 const DEFAULT_EXPORT_DECLARATION_PATTERN = /^\s*export\s+default\b/m;
 const NAMED_EXPORT_DECLARATION_PATTERN = /^\s*export\s+(?:declare\s+)?(?:(?:async\s+)?function|(?:abstract\s+)?class|const|let|var|enum|interface|type)\s+([\w$]+)/gm;
 const LOCAL_EXPORT_SPECIFIER_DECLARATION_PATTERN$1 = /^\s*export\s+(?:type\s+)?\{([\s\S]*?)\}(?:\s+from\s+["'][^"']+["'])?\s*;?\s*(?:(?:\/\/[^\n]*)?\s*)/gm;
-const doesSourceTextExportName = (sourceText, exportedName) => {
+const collectSourceTextExportNames = (sourceText) => {
 	const strippedSource = stripJsComments(sourceText);
-	if (exportedName === "default" && DEFAULT_EXPORT_DECLARATION_PATTERN.test(strippedSource)) return true;
-	for (const match of strippedSource.matchAll(NAMED_EXPORT_DECLARATION_PATTERN)) if (match[1] === exportedName) return true;
-	for (const match of strippedSource.matchAll(LOCAL_EXPORT_SPECIFIER_DECLARATION_PATTERN$1)) if (parseExportSpecifiers(match[1] ?? "", false).map((specifier) => specifier.exportedName).includes(exportedName)) return true;
-	return false;
+	const exportedNames = /* @__PURE__ */ new Set();
+	if (DEFAULT_EXPORT_DECLARATION_PATTERN.test(strippedSource)) exportedNames.add("default");
+	for (const match of strippedSource.matchAll(NAMED_EXPORT_DECLARATION_PATTERN)) if (match[1]) exportedNames.add(match[1]);
+	for (const match of strippedSource.matchAll(LOCAL_EXPORT_SPECIFIER_DECLARATION_PATTERN$1)) {
+		const specifiersText = match[1] ?? "";
+		for (const specifier of parseExportSpecifiers(specifiersText, false)) exportedNames.add(specifier.exportedName);
+	}
+	return exportedNames;
 };
+const exportNamesCache = /* @__PURE__ */ new Map();
 const doesModuleExportName = (filePath, exportedName) => {
 	try {
-		return doesSourceTextExportName(fs.readFileSync(filePath, "utf8"), exportedName);
+		const fileStat = fs.statSync(filePath);
+		const cached = exportNamesCache.get(filePath);
+		if (cached && cached.mtimeMs === fileStat.mtimeMs && cached.size === fileStat.size) return cached.exportedNames.has(exportedName);
+		const exportedNames = collectSourceTextExportNames(fs.readFileSync(filePath, "utf8"));
+		exportNamesCache.set(filePath, {
+			mtimeMs: fileStat.mtimeMs,
+			size: fileStat.size,
+			exportedNames
+		});
+		return exportedNames.has(exportedName);
 	} catch {
 		return false;
 	}
@@ -17413,8 +17432,15 @@ const getProgramAnalysis = (anyNode) => {
 	programToAnalysis.set(programNode, analysis);
 	return analysis;
 };
+const scopeByNodeCache = /* @__PURE__ */ new WeakMap();
 const getScopeForNode = (node, manager) => {
 	if (!node.range) return null;
+	let scopeByNode = scopeByNodeCache.get(manager);
+	if (!scopeByNode) {
+		scopeByNode = /* @__PURE__ */ new WeakMap();
+		scopeByNodeCache.set(manager, scopeByNode);
+	}
+	if (scopeByNode.has(node)) return scopeByNode.get(node) ?? null;
 	let bestScope = null;
 	let bestSize = Infinity;
 	for (const scope of manager.scopes) {
@@ -17427,6 +17453,7 @@ const getScopeForNode = (node, manager) => {
 			bestScope = scope;
 		}
 	}
+	scopeByNode.set(node, bestScope);
 	return bestScope;
 };
 //#endregion
@@ -17461,11 +17488,20 @@ const descend = (node, visit, visited = /* @__PURE__ */ new Set()) => {
 		} else if (isAstNode(child)) descend(child, visit, visited);
 	}
 };
+const upstreamRefsCache = /* @__PURE__ */ new WeakMap();
 const getUpstreamRefs = (analysis, ref) => {
+	let upstreamByRef = upstreamRefsCache.get(analysis);
+	if (!upstreamByRef) {
+		upstreamByRef = /* @__PURE__ */ new WeakMap();
+		upstreamRefsCache.set(analysis, upstreamByRef);
+	}
+	const cached = upstreamByRef.get(ref);
+	if (cached) return cached;
 	const refs = [];
 	ascend(analysis, ref, (upRef) => {
 		refs.push(upRef);
 	});
+	upstreamByRef.set(ref, refs);
 	return refs;
 };
 const findDownstreamNodes = (topNode, type) => {
@@ -17475,11 +17511,24 @@ const findDownstreamNodes = (topNode, type) => {
 	});
 	return nodes;
 };
+const refByIdentifierCache = /* @__PURE__ */ new WeakMap();
 const getRef = (analysis, identifier) => {
+	let refByIdentifier = refByIdentifierCache.get(analysis);
+	if (!refByIdentifier) {
+		refByIdentifier = /* @__PURE__ */ new WeakMap();
+		refByIdentifierCache.set(analysis, refByIdentifier);
+	}
+	if (refByIdentifier.has(identifier)) return refByIdentifier.get(identifier) ?? null;
+	let resolvedReference = null;
 	const scope = getScopeForNode(identifier, analysis.scopeManager);
-	if (!scope) return null;
-	for (const reference of scope.references) if (reference.identifier === identifier) return reference;
-	return null;
+	if (scope) {
+		for (const reference of scope.references) if (reference.identifier === identifier) {
+			resolvedReference = reference;
+			break;
+		}
+	}
+	refByIdentifier.set(identifier, resolvedReference);
+	return resolvedReference;
 };
 const downstreamRefsCache = /* @__PURE__ */ new WeakMap();
 const getDownstreamRefs = (analysis, node) => {
@@ -17554,22 +17603,6 @@ const isEventualCallTo = (analysis, ref, predicate) => {
 };
 //#endregion
 //#region src/plugin/rules/state-and-effects/utils/effect/react.ts
-const getOuterScopeContaining = (analysis, node) => {
-	if (!node.range) return null;
-	let best = null;
-	let bestSize = Infinity;
-	for (const scope of analysis.scopeManager.scopes) {
-		const block = scope.block;
-		if (!block?.range) continue;
-		if (node.range[0] < block.range[0] || node.range[1] > block.range[1]) continue;
-		const size = block.range[1] - block.range[0];
-		if (size <= bestSize) {
-			bestSize = size;
-			best = scope;
-		}
-	}
-	return best;
-};
 const KNOWN_PURE_HOC_NAMES = new Set(["memo", "forwardRef"]);
 const startsWithUppercase = (name) => Boolean(name && name.length > 0 && name[0] >= "A" && name[0] <= "Z");
 const isReactFunctionalComponent = (node) => {
@@ -17600,7 +17633,7 @@ const isReactFunctionalHOC = (analysis, node) => {
 	const isWrappedSeparately = () => {
 		if (!isNodeOfType(node.id, "Identifier")) return false;
 		const bindingName = node.id.name;
-		const containingScope = getOuterScopeContaining(analysis, node);
+		const containingScope = getScopeForNode(node, analysis.scopeManager);
 		if (!containingScope) return false;
 		const variable = containingScope.variables.find((v) => v.name === bindingName);
 		if (!variable) return false;
@@ -41598,7 +41631,14 @@ const getStaticPropertyName = (member) => {
 	if (member.computed && isNodeOfType(property, "Literal") && typeof property.value === "string") return property.value;
 	return null;
 };
+const importInfoCache = /* @__PURE__ */ new WeakMap();
 const getImportInfoForIdentifier = (identifier) => {
+	if (importInfoCache.has(identifier)) return importInfoCache.get(identifier) ?? null;
+	const importInfo = computeImportInfoForIdentifier(identifier);
+	importInfoCache.set(identifier, importInfo);
+	return importInfo;
+};
+const computeImportInfoForIdentifier = (identifier) => {
 	const specifier = findVariableInitializer(identifier, identifier.name)?.initializer;
 	if (!specifier) return null;
 	const declaration = specifier.parent;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-react-doctor",
-  "version": "0.6.2-dev.ac71a3b",
+  "version": "0.6.2-dev.d8628d7",
   "description": "React Doctor rules for oxlint.",
   "keywords": [
     "accessibility",