@exadev/eslint-config 1.2.1 → 1.2.3

package/README.md

@@ -52,6 +52,26 @@ export default defineConfig([
 ]);
 ```
 
+`typescript-eslint`'s own `tseslint.config()` helper (rather than ESLint's `defineConfig()`) does **not** accept the string form of `extends` at all -- it throws `has an 'extends' array that contains a string ... This is a feature of eslint's defineConfig() helper and is not supported by typescript-eslint`. A `tseslint.config()`-based project passes the config value directly instead:
+
+```ts
+import exadev from '@exadev/eslint-config';
+import tseslint from 'typescript-eslint';
+
+export default tseslint.config(
+  // ...your own config...
+  {
+    files: ['**/*.ts'],
+    plugins: { exadev },
+    extends: [exadev.configs.recommended], // or exadev.configs.barrel
+  },
+);
+```
+
+**`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.
 
 ### The typed-linting bundle: `@exadev/eslint-config/recommended-type-checked`
@@ -117,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-B_lOdZQD.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-CpzdmIrK.js";
+import { t as plugin } from "./plugin-Bx8s9ONl.js";
 export { plugin as default };

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

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

package/dist/{plugin-CpzdmIrK.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.1",
+		version: "1.2.3",
 		namespace: "exadev"
 	},
 	rules: {

package/dist/plugin.cjs

@@ -1,2 +1,2 @@
-const require_plugin = require("./plugin-B_lOdZQD.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-CpzdmIrK.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-B_lOdZQD.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-CpzdmIrK.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.1",
+  "version": "1.2.3",
   "description": "Shared custom ESLint rules and plugin for ExaDev projects",
   "type": "module",
   "sideEffects": false,