exadev-eslint-config 2.12.3 → 2.13.0

package/README.md

@@ -170,6 +170,7 @@ Trailing arguments are arbitrary flat-config objects, appended in order after ev
 | `no-non-barrel-reexport` | ✓ | Re-exports belong only in a barrel. Catches the split form across two statements (`import { x } from './y'; export { x };` or `export default x;`) which no AST selector alone can match. Autofix deletes the export and the now-pointless import when it was the import's only use. Self-scopes away from any index file. |
 | `no-side-effects-in-index` | | A barrel file may contain only re-export statements -- nothing that could execute at import time. Self-scopes to any index file. |
 | `barrel-direct-siblings-only` | | A barrel may re-export only from a direct sibling (`./module`), never a nested path, parent, or bare package specifier (mode 3). |
+| `no-control-flow` | | Bans `if`/`switch`/`for`/`for-in`/`for-of`/`while`/`do-while`/the ternary operator outright. Not part of `recommended` or `barrel` -- ordinary code legitimately needs control flow, so this is opt-in, wired via a consumer's own `files` glob for the specific packages that want it (a composition-root package selecting an adapter/strategy by a validated key, say): a lookup table replaces a branch, a declarative array method (`map`/`filter`/`some`/`every`/...) replaces a loop. Requires no type information. |
 | `no-pointless-reassignment` | ✓ | `const foo = bar` where both sides are plain identifiers and the alias adds no transformation. Autofix rewrites every read to the original name and deletes the declaration (including its `export` keyword, when exported). Still reported but deliberately not auto-fixable where collapsing the alias would change meaning: an explicit type annotation (`const exhaustive: never = item` -- the annotation is the point), a read where the original name is shadowed, a read as a shorthand object property, more than one declarator in the statement, or a source that is written to anywhere. |
 | `no-object-assign` | ✓/suggestion | `Object.assign` does not check a source object's properties against the target's declared types, unlike object spread. A fresh object-literal target autofixes to `{ ...target, ...source }`; mutating an existing reassignable binding offers a suggestion only (changes the object's identity); a `const` binding or a non-statement call site gets a plain report with no fix. |
 | `no-mutable-union-array-param` | ✓ | A function parameter typed as an array of a union (`(string \| number)[]`) accepts a narrower caller array (`number[]`) by covariance; calling `push`/`unshift`/`splice`/`fill`/`copyWithin` on it can then insert a value the caller's own array was never declared to hold. Autofix marks the parameter `readonly`, turning the mutating call into a real compile error to resolve deliberately. Requires no type information. |

package/dist/index.cjs

@@ -159,7 +159,7 @@ function buildReactConfig(options = {}) {
 }
 //#endregion
 //#region package.json
-var version = "2.12.3";
+var version = "2.13.0";
 //#endregion
 //#region src/rules/barrel-helpers.ts
 const INDEX_BASENAME$1 = /^index\.[cm]?[tj]sx?$/;
@@ -516,6 +516,76 @@ const noArrayIsArrayMutation = createRule$7({
 	}
 });
 //#endregion
+//#region src/rules/no-control-flow.ts
+const noControlFlow = {
+	meta: {
+		type: "problem",
+		schema: [],
+		messages: {
+			ifStatement: "Use a lookup table, not an if/else branch.",
+			switchStatement: "Use a lookup table, not a switch branch.",
+			forStatement: "Use a lookup table or a declarative array method, not a for loop.",
+			forInStatement: "Use a lookup table or Object.entries, not a for-in loop.",
+			forOfStatement: "Use a declarative array method, not a for-of loop.",
+			whileStatement: "Use a lookup table or a declarative array method, not a while loop.",
+			doWhileStatement: "Use a lookup table or a declarative array method, not a do-while loop.",
+			conditionalExpression: "Use a lookup table, not a ternary."
+		}
+	},
+	create(context) {
+		return {
+			IfStatement(node) {
+				context.report({
+					node,
+					messageId: "ifStatement"
+				});
+			},
+			SwitchStatement(node) {
+				context.report({
+					node,
+					messageId: "switchStatement"
+				});
+			},
+			ForStatement(node) {
+				context.report({
+					node,
+					messageId: "forStatement"
+				});
+			},
+			ForInStatement(node) {
+				context.report({
+					node,
+					messageId: "forInStatement"
+				});
+			},
+			ForOfStatement(node) {
+				context.report({
+					node,
+					messageId: "forOfStatement"
+				});
+			},
+			WhileStatement(node) {
+				context.report({
+					node,
+					messageId: "whileStatement"
+				});
+			},
+			DoWhileStatement(node) {
+				context.report({
+					node,
+					messageId: "doWhileStatement"
+				});
+			},
+			ConditionalExpression(node) {
+				context.report({
+					node,
+					messageId: "conditionalExpression"
+				});
+			}
+		};
+	}
+};
+//#endregion
 //#region src/rules/no-enum-number-widening.ts
 const noEnumNumberWidening = _typescript_eslint_utils.ESLintUtils.RuleCreator((name) => `https://github.com/ExaDev/eslint-config/blob/main/src/rules/${name}.ts`)({
 	name: "no-enum-number-widening",
@@ -1356,6 +1426,7 @@ const plugin = {
 		"barrel-direct-siblings-only": barrelDirectSiblingsOnly,
 		"barrel-policy": barrelPolicy,
 		"no-array-isarray-mutation": noArrayIsArrayMutation,
+		"no-control-flow": noControlFlow,
 		"no-enum-number-widening": noEnumNumberWidening,
 		"no-enum-reverse-lookup-widening": noEnumReverseLookupWidening,
 		"no-index-files": noIndexFiles,

package/dist/index.js

@@ -128,7 +128,7 @@ function buildReactConfig(options = {}) {
 }
 //#endregion
 //#region package.json
-var version = "2.12.3";
+var version = "2.13.0";
 //#endregion
 //#region src/rules/barrel-helpers.ts
 const INDEX_BASENAME$1 = /^index\.[cm]?[tj]sx?$/;
@@ -485,6 +485,76 @@ const noArrayIsArrayMutation = createRule$7({
 	}
 });
 //#endregion
+//#region src/rules/no-control-flow.ts
+const noControlFlow = {
+	meta: {
+		type: "problem",
+		schema: [],
+		messages: {
+			ifStatement: "Use a lookup table, not an if/else branch.",
+			switchStatement: "Use a lookup table, not a switch branch.",
+			forStatement: "Use a lookup table or a declarative array method, not a for loop.",
+			forInStatement: "Use a lookup table or Object.entries, not a for-in loop.",
+			forOfStatement: "Use a declarative array method, not a for-of loop.",
+			whileStatement: "Use a lookup table or a declarative array method, not a while loop.",
+			doWhileStatement: "Use a lookup table or a declarative array method, not a do-while loop.",
+			conditionalExpression: "Use a lookup table, not a ternary."
+		}
+	},
+	create(context) {
+		return {
+			IfStatement(node) {
+				context.report({
+					node,
+					messageId: "ifStatement"
+				});
+			},
+			SwitchStatement(node) {
+				context.report({
+					node,
+					messageId: "switchStatement"
+				});
+			},
+			ForStatement(node) {
+				context.report({
+					node,
+					messageId: "forStatement"
+				});
+			},
+			ForInStatement(node) {
+				context.report({
+					node,
+					messageId: "forInStatement"
+				});
+			},
+			ForOfStatement(node) {
+				context.report({
+					node,
+					messageId: "forOfStatement"
+				});
+			},
+			WhileStatement(node) {
+				context.report({
+					node,
+					messageId: "whileStatement"
+				});
+			},
+			DoWhileStatement(node) {
+				context.report({
+					node,
+					messageId: "doWhileStatement"
+				});
+			},
+			ConditionalExpression(node) {
+				context.report({
+					node,
+					messageId: "conditionalExpression"
+				});
+			}
+		};
+	}
+};
+//#endregion
 //#region src/rules/no-enum-number-widening.ts
 const noEnumNumberWidening = ESLintUtils.RuleCreator((name) => `https://github.com/ExaDev/eslint-config/blob/main/src/rules/${name}.ts`)({
 	name: "no-enum-number-widening",
@@ -1325,6 +1395,7 @@ const plugin = {
 		"barrel-direct-siblings-only": barrelDirectSiblingsOnly,
 		"barrel-policy": barrelPolicy,
 		"no-array-isarray-mutation": noArrayIsArrayMutation,
+		"no-control-flow": noControlFlow,
 		"no-enum-number-widening": noEnumNumberWidening,
 		"no-enum-reverse-lookup-widening": noEnumReverseLookupWidening,
 		"no-index-files": noIndexFiles,

package/package.json

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