@exadev/eslint-config 2.10.2 → 2.10.4

package/README.md

@@ -113,6 +113,8 @@ export default tseslint.config(
    If none of these resolve, `@exadev/eslint-config`'s default export is byte-for-byte identical to the plain TypeScript ruleset -- nothing about the base package changes.
 2. **For React specifically, the file must actually be `.jsx`/`.tsx`.** The React/hooks/a11y rule block is scoped to `files: ['**/*.jsx', '**/*.tsx']`, so even if `eslint-plugin-react` is resolvable only incidentally (e.g. hoisted as a transitive dependency of something unrelated in a monorepo, with zero real JSX anywhere in the linted project), its rules are never matched against a file that isn't JSX -- ESLint's flat-config `files` matching happens per linted file, at lint time, not at config-build time. `@next/eslint-plugin-next`'s block carries no such glob: its own presence is already an unambiguous signal on its own (nothing installs it except a real Next.js project).
 
+React support pairs `eslint-plugin-react`'s `flat/recommended` with its own `flat/jsx-runtime` config, which turns `react/react-in-jsx-scope` and `react/jsx-uses-react` back off. `flat/recommended` alone assumes the classic runtime, where every file using JSX needs `import React` in scope; the automatic JSX runtime, the default since React 17 and the only mode Next.js's own compiler supports, needs no such import. Without this pairing, a consumer on the automatic runtime would see `react/react-in-jsx-scope` fire on every JSX file in the project.
+
 ### Explicit control
 
 Two ways to override the automatic behaviour, for anyone who doesn't want to rely on it:

package/dist/index.cjs

@@ -89,16 +89,23 @@ function isFlatConfig(value) {
 }
 function buildReactConfig(options = {}) {
 	if (options.enabled === false) return [];
-	const reactConfig = readFlatConfig(tryRequire("eslint-plugin-react", options.requireFn), [
+	const reactModule = tryRequire("eslint-plugin-react", options.requireFn);
+	const reactConfig = readFlatConfig(reactModule, [
 		"configs",
 		"flat",
 		"recommended"
 	]);
 	if (options.enabled === true && reactConfig === void 0) throw new Error(`@exadev/eslint-config: React support was explicitly requested but 'eslint-plugin-react' could not be resolved. Install it with: ${INSTALL_COMMAND}`);
 	if (reactConfig === void 0) return [];
+	const jsxRuntimeConfig = readFlatConfig(reactModule, [
+		"configs",
+		"flat",
+		"jsx-runtime"
+	]);
 	const hooksModule = tryRequire("eslint-plugin-react-hooks", options.requireFn);
 	return [
 		reactConfig,
+		jsxRuntimeConfig,
 		readFlatConfig(hooksModule, [
 			"configs",
 			"flat",
@@ -112,7 +119,7 @@ function buildReactConfig(options = {}) {
 }
 //#endregion
 //#region package.json
-var version = "2.10.2";
+var version = "2.10.4";
 //#endregion
 //#region src/rules/barrel-helpers.ts
 const INDEX_BASENAME$1 = /^index\.[cm]?[tj]sx?$/;
@@ -144,6 +151,19 @@ function isPermittedBarrel(filename, mode) {
 	if (mode === "single") return isMainBarrel(filename);
 	return isIndexFile(filename);
 }
+function isAncestorNode(value) {
+	if (typeof value !== "object" || value === null) return false;
+	if (!("type" in value)) return false;
+	return typeof value.type === "string";
+}
+function isInsideAmbientModuleDeclaration(node) {
+	let current = node;
+	while (isAncestorNode(current)) {
+		if (current.type === "TSModuleDeclaration") return true;
+		current = current.parent;
+	}
+	return false;
+}
 function createSplitReexportDetector() {
 	const importsByName = /* @__PURE__ */ new Map();
 	const bareExportSpecifiers = [];
@@ -305,7 +325,7 @@ const barrelPolicy = {
 			},
 			ExportNamedDeclaration(node) {
 				detector.visitExportNamed(node);
-				if (hasSource(node)) {
+				if (hasSource(node) && !isInsideAmbientModuleDeclaration(node)) {
 					const source = node.source === null || node.source === void 0 ? void 0 : node.source.value;
 					if (!isPermittedBarrel(filename, mode)) context.report({
 						node,
@@ -319,6 +339,7 @@ const barrelPolicy = {
 				}
 			},
 			ExportAllDeclaration(node) {
+				if (isInsideAmbientModuleDeclaration(node)) return;
 				const source = node.source.value;
 				if (!isPermittedBarrel(filename, mode)) context.report({
 					node,

package/dist/index.js

@@ -60,16 +60,23 @@ function isFlatConfig(value) {
 }
 function buildReactConfig(options = {}) {
 	if (options.enabled === false) return [];
-	const reactConfig = readFlatConfig(tryRequire("eslint-plugin-react", options.requireFn), [
+	const reactModule = tryRequire("eslint-plugin-react", options.requireFn);
+	const reactConfig = readFlatConfig(reactModule, [
 		"configs",
 		"flat",
 		"recommended"
 	]);
 	if (options.enabled === true && reactConfig === void 0) throw new Error(`@exadev/eslint-config: React support was explicitly requested but 'eslint-plugin-react' could not be resolved. Install it with: ${INSTALL_COMMAND}`);
 	if (reactConfig === void 0) return [];
+	const jsxRuntimeConfig = readFlatConfig(reactModule, [
+		"configs",
+		"flat",
+		"jsx-runtime"
+	]);
 	const hooksModule = tryRequire("eslint-plugin-react-hooks", options.requireFn);
 	return [
 		reactConfig,
+		jsxRuntimeConfig,
 		readFlatConfig(hooksModule, [
 			"configs",
 			"flat",
@@ -83,7 +90,7 @@ function buildReactConfig(options = {}) {
 }
 //#endregion
 //#region package.json
-var version = "2.10.2";
+var version = "2.10.4";
 //#endregion
 //#region src/rules/barrel-helpers.ts
 const INDEX_BASENAME$1 = /^index\.[cm]?[tj]sx?$/;
@@ -115,6 +122,19 @@ function isPermittedBarrel(filename, mode) {
 	if (mode === "single") return isMainBarrel(filename);
 	return isIndexFile(filename);
 }
+function isAncestorNode(value) {
+	if (typeof value !== "object" || value === null) return false;
+	if (!("type" in value)) return false;
+	return typeof value.type === "string";
+}
+function isInsideAmbientModuleDeclaration(node) {
+	let current = node;
+	while (isAncestorNode(current)) {
+		if (current.type === "TSModuleDeclaration") return true;
+		current = current.parent;
+	}
+	return false;
+}
 function createSplitReexportDetector() {
 	const importsByName = /* @__PURE__ */ new Map();
 	const bareExportSpecifiers = [];
@@ -276,7 +296,7 @@ const barrelPolicy = {
 			},
 			ExportNamedDeclaration(node) {
 				detector.visitExportNamed(node);
-				if (hasSource(node)) {
+				if (hasSource(node) && !isInsideAmbientModuleDeclaration(node)) {
 					const source = node.source === null || node.source === void 0 ? void 0 : node.source.value;
 					if (!isPermittedBarrel(filename, mode)) context.report({
 						node,
@@ -290,6 +310,7 @@ const barrelPolicy = {
 				}
 			},
 			ExportAllDeclaration(node) {
+				if (isInsideAmbientModuleDeclaration(node)) return;
 				const source = node.source.value;
 				if (!isPermittedBarrel(filename, mode)) context.report({
 					node,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exadev/eslint-config",
-  "version": "2.10.2",
+  "version": "2.10.4",
   "description": "Shared custom ESLint rules and plugin for ExaDev projects",
   "type": "module",
   "sideEffects": false,