exadev-eslint-config 1.2.2 → 1.2.3

package/README.md

@@ -68,8 +68,9 @@ export default tseslint.config(
 );
 ```
 
-**`recommended`/`barrel` carry no `files`/`ignores` of their own, and applying either unscoped will misfire.** `no-side-effects-in-index` flags any top-level statement that isn't a bare re-export -- applied to an ordinary file with real `export function`/`export const`/`export interface` declarations, every one of them trips it (confirmed directly: 88 false-positive errors on a single real source file in a repo that tried this). Scope the `extends` block itself to just `src/index.ts` if you want `no-side-effects-in-index`'s coverage from `recommended`/`barrel`, or -- what every current consumer of this plugin actually does -- skip `recommended`/`barrel` and wire the four `exadev/*` rules individually, each with your own project's real `files`/`ignores`, exactly as shown in the first example above. That's not a workaround; it's the intended shape once a project's re-export-ban exceptions and barrel location are genuinely project-specific.
-```
+**`recommended`/`barrel` carry no `files`/`ignores` of their own, and are safe to apply unscoped anyway -- the two rules that care which file they're looking at (`no-side-effects-in-index`, `no-non-barrel-reexport`) each check `context.filename` themselves, the same self-scoping pattern `no-non-barrel-index` already used.** `no-side-effects-in-index` no-ops on every file except `src/index.ts`, since it has no legitimate target anywhere else; `no-non-barrel-reexport` no-ops specifically on `src/index.ts`, since a real single-statement re-export there is the intended, normal shape. An earlier version of this package lacked that self-scoping and genuinely misfired when `recommended`/`barrel` were applied without an external `files: ['src/index.ts']`/`ignores: ['src/index.ts']` wrapper -- 88 false-positive errors on a single real source file in a repo that tried it, since `no-side-effects-in-index` flagged every ordinary `export function`/`export const`/`export interface` declaration it saw. That's fixed at the rule level now, not documented around.
+
+The one thing self-scoping can't know on your behalf is a barrel that lives somewhere other than `src/index.ts`, or a project-specific exception beyond the barrel (an extra file you want exempt from the re-export ban). For either of those, layer an additional override on top of `recommended`/`barrel` -- e.g. `{ files: ['lib/other-legacy-reexport.ts'], rules: { 'exadev/no-non-barrel-reexport': 'off' } }` -- rather than falling back to wiring all four rules individually, which is still fine but no longer required for the common case.
 
 Both `recommended` and `barrel` are usable in a plain JavaScript project with no TypeScript and no `typescript-eslint` installed: this plugin's own four rules operate on plain ESTree import/export/declaration nodes, nothing TypeScript-specific, and neither config references `typescript-eslint` at all.
 
@@ -136,7 +137,7 @@ The `lint`/`typecheck`/`test`/`build` npm scripts are thin wrappers around turbo
 
 ## Conventions
 
-`eslint.config.ts` dogfoods this package's own rules on itself, importing `./src/index` by relative path rather than as an installed dependency. All four rules apply to this repo's own source: `no-non-barrel-reexport` is scoped to `src/**/*.ts` excluding `src/index.ts` (the barrel is where re-exports are meant to live), and `no-side-effects-in-index` is scoped to `src/index.ts` alone -- the plugin-construction logic lives in `src/plugin.ts` specifically so `src/index.ts` can stay a pure re-export point both rules assume.
+`eslint.config.ts` dogfoods this package's own rules on itself, importing `./src/index` by relative path rather than as an installed dependency. All four rules are wired in one block with no `files`/`ignores` of its own -- `no-side-effects-in-index` and `no-non-barrel-reexport` each self-scope to `src/index.ts` internally, so applying them repo-wide here is both the simplest wiring and the live proof that doing so works. The plugin-construction logic lives in `src/plugin.ts` specifically so `src/index.ts` can stay a pure re-export point both rules assume.
 
 `tsconfig.json` enables `verbatimModuleSyntax` (type-only imports/exports must use `import type`/`export type` explicitly -- enforced too by the `consistent-type-imports` eslint rule) and `noUncheckedIndexedAccess` (indexed access returns `T | undefined`, narrow before use rather than asserting).
 

package/dist/index.cjs

@@ -1,2 +1,2 @@
-const require_plugin = require("./plugin-DZKXx2me.cjs");
+const require_plugin = require("./plugin-BSeiZs_j.cjs");
 module.exports = require_plugin.plugin;

package/dist/index.js

@@ -1,2 +1,2 @@
-import { t as plugin } from "./plugin-PQMeiTvg.js";
+import { t as plugin } from "./plugin-Bx8s9ONl.js";
 export { plugin as default };

package/dist/{plugin-DZKXx2me.cjs → plugin-BSeiZs_j.cjs}

@@ -3,7 +3,7 @@
 const plugin = {
 	meta: {
 		name: "@exadev/eslint-config",
-		version: "1.2.2",
+		version: "1.2.3",
 		namespace: "exadev"
 	},
 	rules: {

package/dist/{plugin-PQMeiTvg.js → plugin-Bx8s9ONl.js}

@@ -7,7 +7,7 @@ import noSideEffectsInIndex from "./rules/no-side-effects-in-index.js";
 const plugin = {
 	meta: {
 		name: "@exadev/eslint-config",
-		version: "1.2.2",
+		version: "1.2.3",
 		namespace: "exadev"
 	},
 	rules: {

package/dist/plugin.cjs

@@ -1,2 +1,2 @@
-const require_plugin = require("./plugin-DZKXx2me.cjs");
+const require_plugin = require("./plugin-BSeiZs_j.cjs");
 module.exports = require_plugin.plugin;

package/dist/plugin.js

@@ -1,2 +1,2 @@
-import { t as plugin } from "./plugin-PQMeiTvg.js";
+import { t as plugin } from "./plugin-Bx8s9ONl.js";
 export { plugin as default };

package/dist/recommended-type-checked.cjs

@@ -20,7 +20,7 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 	enumerable: true
 }) : target, mod));
 //#endregion
-const require_plugin = require("./plugin-DZKXx2me.cjs");
+const require_plugin = require("./plugin-BSeiZs_j.cjs");
 let typescript_eslint = require("typescript-eslint");
 typescript_eslint = __toESM(typescript_eslint, 1);
 //#region src/recommended-type-checked.ts

package/dist/recommended-type-checked.js

@@ -1,4 +1,4 @@
-import { t as plugin } from "./plugin-PQMeiTvg.js";
+import { t as plugin } from "./plugin-Bx8s9ONl.js";
 import tseslint from "typescript-eslint";
 //#region src/recommended-type-checked.ts
 const recommendedTypeChecked = [

package/dist/rules/no-non-barrel-reexport.cjs

@@ -26,6 +26,7 @@ const noNonBarrelReexport = {
 		}
 	},
 	create(context) {
+		if (context.filename.endsWith("/src/index.ts")) return {};
 		const importsByName = /* @__PURE__ */ new Map();
 		const bareExportSpecifiers = [];
 		const defaultExportDeclarations = [];

package/dist/rules/no-non-barrel-reexport.js

@@ -26,6 +26,7 @@ const noNonBarrelReexport = {
 		}
 	},
 	create(context) {
+		if (context.filename.endsWith("/src/index.ts")) return {};
 		const importsByName = /* @__PURE__ */ new Map();
 		const bareExportSpecifiers = [];
 		const defaultExportDeclarations = [];

package/dist/rules/no-side-effects-in-index.cjs

@@ -10,6 +10,7 @@ const noSideEffectsInIndex = {
 		messages: { notAPureReexport: "The public barrel (src/index.ts) may contain only re-export statements ('export * from ...' / 'export { x } from ...' / 'export type { x } from ...') -- nothing else, so it can never have a side effect at import time by construction. Found: {{ description }}." }
 	},
 	create(context) {
+		if (!context.filename.endsWith("/src/index.ts")) return {};
 		return { Program(node) {
 			for (const statement of node.body) if (!isPureReexport(statement)) context.report({
 				node: statement,

package/dist/rules/no-side-effects-in-index.js

@@ -10,6 +10,7 @@ const noSideEffectsInIndex = {
 		messages: { notAPureReexport: "The public barrel (src/index.ts) may contain only re-export statements ('export * from ...' / 'export { x } from ...' / 'export type { x } from ...') -- nothing else, so it can never have a side effect at import time by construction. Found: {{ description }}." }
 	},
 	create(context) {
+		if (!context.filename.endsWith("/src/index.ts")) return {};
 		return { Program(node) {
 			for (const statement of node.body) if (!isPureReexport(statement)) context.report({
 				node: statement,

package/package.json

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