exadev-eslint-config 2.17.0 → 2.17.2

package/dist/index.cjs

@@ -263,9 +263,6 @@ function tryRequire(specifier, requireFn = nodeRequire) {
 function isRecord$1(value) {
 	return typeof value === "object" && value !== null;
 }
-function isFlatConfig$1(value) {
-	return isRecord$1(value);
-}
 function normalizeLegacyParserOptions(record) {
 	if (!("parserOptions" in record)) return record;
 	const { parserOptions, languageOptions, ...rest } = record;
@@ -285,8 +282,7 @@ function readFlatConfig(module, path) {
 		current = current[key];
 	}
 	if (!isRecord$1(current)) return void 0;
-	const normalized = normalizeLegacyParserOptions(current);
-	return isFlatConfig$1(normalized) ? normalized : void 0;
+	return normalizeLegacyParserOptions(current);
 }
 //#endregion
 //#region src/nextjs.ts
@@ -299,7 +295,7 @@ function buildNextjsConfig(options = {}) {
 }
 //#endregion
 //#region package.json
-var version = "2.17.0";
+var version = "2.17.2";
 //#endregion
 //#region src/react.ts
 const JSX_FILE_PATTERNS = ["**/*.jsx", "**/*.tsx"];
@@ -341,12 +337,15 @@ function buildReactConfig(options = {}) {
 //#region src/rules/barrel-helpers.ts
 const INDEX_BASENAME$1 = /^index\.[cm]?[tj]sx?$/;
 function basenameOf(filename) {
-	const slash = filename.lastIndexOf("/");
-	return slash === -1 ? filename : filename.slice(slash + 1);
+	return filename.slice(filename.lastIndexOf("/") + 1);
 }
 function isIndexFile(filename) {
 	return INDEX_BASENAME$1.test(basenameOf(filename));
 }
+function moduleSpecifierValue(literal) {
+	if (typeof literal.value !== "string") throw new Error(`Unreachable: a module specifier's own grammar only ever produces a string literal, got ${typeof literal.value} instead.`);
+	return literal.value;
+}
 function isMainBarrel(filename) {
 	return filename.endsWith("/src/index.ts");
 }
@@ -381,6 +380,9 @@ function isInsideAmbientModuleDeclaration(node) {
 	}
 	return false;
 }
+function hasSource(node) {
+	return node.source !== null && node.source !== void 0;
+}
 function createSplitReexportDetector() {
 	const importsByName = /* @__PURE__ */ new Map();
 	const bareExportSpecifiers = [];
@@ -418,14 +420,15 @@ function createSplitReexportDetector() {
 				});
 			}
 			for (const declarationNode of defaultExportDeclarations) {
-				const name = declarationNode.declaration.type === "Identifier" ? declarationNode.declaration.name : void 0;
-				if (name === void 0) continue;
-				const trackedImport = importsByName.get(name);
+				if (declarationNode.declaration.type !== "Identifier") continue;
+				const identifierNode = declarationNode.declaration;
+				const trackedImport = importsByName.get(identifierNode.name);
 				if (trackedImport === void 0) continue;
 				out.push({
 					kind: "default",
 					declaration: declarationNode,
-					name,
+					identifierNode,
+					name: identifierNode.name,
 					trackedImport
 				});
 			}
@@ -439,15 +442,14 @@ const barrelDirectSiblingsOnly = {
 	meta: {
 		type: "problem",
 		schema: [],
-		messages: { notADirectSibling: "A barrel may re-export only from a direct sibling file or folder ('./module' or './module.ts') -- found '{{ source }}'. Move the source closer, or import it directly at the call site rather than re-exporting it through this barrel." }
+		messages: { notADirectSibling: "A barrel may re-export only from a direct sibling file or folder ('./module' or './module.ts') — found '{{ source }}'. Move the source closer, or import it directly at the call site rather than re-exporting it through this barrel." }
 	},
 	create(context) {
 		if (!isIndexFile(context.filename)) return {};
 		return {
 			ExportNamedDeclaration(node) {
 				if (node.source === null || node.source === void 0) return;
-				const source = node.source.value;
-				if (typeof source !== "string") return;
+				const source = moduleSpecifierValue(node.source);
 				if (!isDirectSibling(source)) context.report({
 					node,
 					messageId: "notADirectSibling",
@@ -455,8 +457,7 @@ const barrelDirectSiblingsOnly = {
 				});
 			},
 			ExportAllDeclaration(node) {
-				const source = node.source.value;
-				if (typeof source !== "string") return;
+				const source = moduleSpecifierValue(node.source);
 				if (!isDirectSibling(source)) context.report({
 					node,
 					messageId: "notADirectSibling",
@@ -469,7 +470,7 @@ const barrelDirectSiblingsOnly = {
 //#endregion
 //#region src/rules/barrel-policy.ts
 function readMode(options) {
-	if (options === void 0 || typeof options !== "object" || options === null || !("mode" in options) || !isBarrelMode(options.mode)) throw new Error("exadev/barrel-policy requires options: { mode: 'banned' | 'single' | 'siblings' }.");
+	if (typeof options !== "object" || options === null || !("mode" in options) || !isBarrelMode(options.mode)) throw new Error("exadev/barrel-policy requires options: { mode: 'banned' | 'single' | 'siblings' }.");
 	return options.mode;
 }
 const barrelPolicy = {
@@ -489,20 +490,17 @@ const barrelPolicy = {
 			additionalProperties: false
 		}],
 		messages: {
-			indexFileBanned: "Index (barrel) files are banned in this project -- import directly from the module that owns the export instead. Rename this file to something descriptive.",
-			nonMainIndexFile: "Only src/index.ts may be a barrel in this project -- this index file is not it. Move its contents into the module that owns them or give the file a descriptive name.",
-			sideEffectInBarrel: "A barrel 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 }}.",
-			reexportOutsideBarrel: "Re-exports belong only in a barrel (index) file -- import this value directly in the file that uses it instead of re-exporting it through this one.",
-			notADirectSibling: "A barrel may re-export only from a direct sibling file or folder ('./module' or './module.ts') -- found '{{ source }}'. Move the source closer, or import it directly at the call site rather than re-exporting it through this barrel."
+			indexFileBanned: "Index (barrel) files are banned in this project — import directly from the module that owns the export instead. Rename this file to something descriptive.",
+			nonMainIndexFile: "Only src/index.ts may be a barrel in this project — this index file is not it. Move its contents into the module that owns them or give the file a descriptive name.",
+			sideEffectInBarrel: "A barrel 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 }}.",
+			reexportOutsideBarrel: "Re-exports belong only in a barrel (index) file — import this value directly in the file that uses it instead of re-exporting it through this one.",
+			notADirectSibling: "A barrel may re-export only from a direct sibling file or folder ('./module' or './module.ts') — found '{{ source }}'. Move the source closer, or import it directly at the call site rather than re-exporting it through this barrel."
 		}
 	},
 	create(context) {
 		const mode = readMode(context.options[0]);
 		const filename = context.filename;
 		const detector = createSplitReexportDetector();
-		function hasSource(node) {
-			return node.source !== null && node.source !== void 0;
-		}
 		return {
 			Program(node) {
 				if (mode === "banned") {
@@ -543,12 +541,12 @@ const barrelPolicy = {
 			ExportNamedDeclaration(node) {
 				detector.visitExportNamed(node);
 				if (hasSource(node) && !isInsideAmbientModuleDeclaration(node)) {
-					const source = node.source === null || node.source === void 0 ? void 0 : node.source.value;
+					const source = moduleSpecifierValue(node.source);
 					if (!isPermittedBarrel(filename, mode)) context.report({
 						node,
 						messageId: "reexportOutsideBarrel"
 					});
-					else if (mode === "siblings" && typeof source === "string" && !isDirectSibling(source)) context.report({
+					else if (mode === "siblings" && !isDirectSibling(source)) context.report({
 						node,
 						messageId: "notADirectSibling",
 						data: { source }
@@ -557,12 +555,12 @@ const barrelPolicy = {
 			},
 			ExportAllDeclaration(node) {
 				if (isInsideAmbientModuleDeclaration(node)) return;
-				const source = node.source.value;
+				const source = moduleSpecifierValue(node.source);
 				if (!isPermittedBarrel(filename, mode)) context.report({
 					node,
 					messageId: "reexportOutsideBarrel"
 				});
-				else if (mode === "siblings" && typeof source === "string" && !isDirectSibling(source)) context.report({
+				else if (mode === "siblings" && !isDirectSibling(source)) context.report({
 					node,
 					messageId: "notADirectSibling",
 					data: { source }
@@ -574,8 +572,8 @@ const barrelPolicy = {
 			"Program:exit"() {
 				for (const violation of detector.violations()) if (isPermittedBarrel(filename, mode)) {
 					if (mode === "siblings") {
-						const importSource = violation.trackedImport.declaration.source.value;
-						if (typeof importSource === "string" && !isDirectSibling(importSource)) context.report({
+						const importSource = moduleSpecifierValue(violation.trackedImport.declaration.source);
+						if (!isDirectSibling(importSource)) context.report({
 							node: violation.kind === "named" ? violation.specifier : violation.declaration,
 							messageId: "notADirectSibling",
 							data: { source: importSource }
@@ -590,6 +588,12 @@ const barrelPolicy = {
 	}
 };
 //#endregion
+//#region src/rules/scope-guards.ts
+function asIdentifierName(name) {
+	if (name.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier) throw new Error(`Unreachable: expected a Parameter/Variable Definition's own name to be an Identifier, got ${name.type} instead.`);
+	return name;
+}
+//#endregion
 //#region src/rules/no-array-isarray-mutation.ts
 const MUTATING_INSERT_METHODS$1 = /* @__PURE__ */ new Set([
 	"push",
@@ -616,7 +620,7 @@ const noArrayIsArrayMutation = createRule$7({
 		type: "problem",
 		schema: [],
 		docs: { description: "Disallow mutating-insertion calls on a parameter or local variable whose real type includes a readonly array, narrowed via Array.isArray, which silently discards the declared readonly guarantee." },
-		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by Array.isArray -- Array.isArray's own type declaration cannot preserve a readonly modifier through the guard, so a value whose real type includes a readonly array (a caller's array, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the array before inserting (e.g. a spread into a new array), or narrow with a check that preserves readonly instead of Array.isArray." }
+		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by Array.isArray — Array.isArray's own type declaration cannot preserve a readonly modifier through the guard, so a value whose real type includes a readonly array (a caller's array, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the array before inserting (e.g. a spread into a new array), or narrow with a check that preserves readonly instead of Array.isArray." }
 	},
 	defaultOptions: [],
 	create(context) {
@@ -629,14 +633,12 @@ const noArrayIsArrayMutation = createRule$7({
 		}
 		return { CallExpression(node) {
 			const { callee } = node;
-			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.object.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_INSERT_METHODS$1.has(callee.property.name)) return;
+			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_INSERT_METHODS$1.has(callee.property.name)) return;
 			const variable = context.sourceCode.getScope(node).references.find((reference) => reference.identifier === callee.object)?.resolved;
 			if (!variable) return;
 			const declarationDefinition = variable.defs.find((definition) => definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Parameter || definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Variable);
 			if (!declarationDefinition) return;
-			const declarationNode = declarationDefinition.name;
-			if (declarationNode.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier) return;
-			if (!declarationHasReadonlyArrayConstituent(declarationNode)) return;
+			if (!declarationHasReadonlyArrayConstituent(asIdentifierName(declarationDefinition.name))) return;
 			if (!isGuardedByArrayIsArray(node, variable, context)) return;
 			context.report({
 				node,
@@ -664,7 +666,7 @@ const noArrayIsArrayMutation = createRule$7({
 					if (parent.consequent === current && matchesArrayIsArrayOn(parent.test, parameterVariable, ruleContext)) return true;
 					if (parent.alternate === current && isNegatedArrayIsArrayCall(parent.test, parameterVariable, ruleContext)) return true;
 				}
-				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && parent.right === current && matchesArrayIsArrayOn(parent.left, parameterVariable, ruleContext)) return true;
+				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && matchesArrayIsArrayOn(parent.left, parameterVariable, ruleContext)) return true;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ConditionalExpression && parent.consequent === current && matchesArrayIsArrayOn(parent.test, parameterVariable, ruleContext)) return true;
 				current = parent;
 			}
@@ -676,15 +678,7 @@ const noArrayIsArrayMutation = createRule$7({
 				const { parent } = current;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.BlockStatement || parent.type === _typescript_eslint_utils.AST_NODE_TYPES.Program) {
 					const statements = parent.body;
-					let ownIndex = -1;
-					for (let i = 0; i < statements.length; i++) if (statements[i] === current) {
-						ownIndex = i;
-						break;
-					}
-					if (ownIndex > 0) for (let i = ownIndex - 1; i >= 0; i--) {
-						const sibling = statements[i];
-						if (sibling?.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedArrayIsArrayCall(sibling.test, parameterVariable, ruleContext) && definitelyExits$2(sibling.consequent)) return true;
-					}
+					if (statements.slice(0, statements.indexOf(current)).some((sibling) => sibling.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedArrayIsArrayCall(sibling.test, parameterVariable, ruleContext) && definitelyExits$2(sibling.consequent))) return true;
 				}
 				current = parent;
 			}
@@ -763,22 +757,36 @@ const noControlFlow = {
 	}
 };
 //#endregion
+//#region src/rules/ts-node-guards.ts
+function asExpression(tsNode) {
+	if (!typescript.isExpression(tsNode)) throw new Error(`Unreachable: expected a ts.Expression, got ts.SyntaxKind.${typescript.SyntaxKind[tsNode.kind]} instead.`);
+	return tsNode;
+}
+function asTypeReference(type) {
+	if (!(0, ts_api_utils.isTypeReference)(type)) throw new Error("Unreachable: expected an array/tuple type to be backed by a ts.TypeReference.");
+	return type;
+}
+function lastTokenOrThrow(sourceCode, node) {
+	const token = sourceCode.getLastToken(node);
+	if (token === null) throw new Error("Unreachable: getLastToken returned null for a node expected to always have at least one token.");
+	return token;
+}
+//#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",
 	meta: {
 		type: "problem",
-		docs: { description: "Disallow assigning a bare (non-literal) number where a numeric enum type is expected -- TypeScript accepts any number for a numeric enum slot, not just its own members, once the value is not a literal the compiler can range-check." },
+		docs: { description: "Disallow assigning a bare (non-literal) number where a numeric enum type is expected — TypeScript accepts any number for a numeric enum slot, not just its own members, once the value is not a literal the compiler can range-check." },
 		schema: [],
-		messages: { widening: "A plain 'number' value is being used where the numeric enum '{{ enumName }}' is expected. TypeScript does not verify the value is actually one of the enum's members here -- narrow it to a known member first (e.g. a lookup/guard against the enum's own values), or accept a plain 'number' parameter instead of widening it implicitly." }
+		messages: { widening: "A plain 'number' value is being used where the numeric enum '{{ enumName }}' is expected. TypeScript does not verify the value is actually one of the enum's members here — narrow it to a known member first (e.g. a lookup/guard against the enum's own values), or accept a plain 'number' parameter instead of widening it implicitly." },
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		const services = _typescript_eslint_utils.ESLintUtils.getParserServices(context);
 		const checker = services.program.getTypeChecker();
 		function checkNode(expression) {
-			const tsNode = services.esTreeNodeToTSNodeMap.get(expression);
-			if (!typescript.isExpression(tsNode)) return;
+			const tsNode = asExpression(services.esTreeNodeToTSNodeMap.get(expression));
 			const contextualType = checker.getContextualType(tsNode);
 			if (!contextualType || !(contextualType.flags & typescript.TypeFlags.EnumLike)) return;
 			const actualType = checker.getTypeAtLocation(tsNode);
@@ -814,27 +822,24 @@ const noEnumReverseLookupWidening = _typescript_eslint_utils.ESLintUtils.RuleCre
 	meta: {
 		type: "problem",
 		hasSuggestions: true,
-		docs: { description: "Disallow indexing a numeric enum's reverse mapping with a bare (non-literal) number -- TypeScript types the result as plain 'string' for any number, including one outside the enum's actual member range, where it genuinely returns 'undefined' at runtime." },
+		docs: { description: "Disallow indexing a numeric enum's reverse mapping with a bare (non-literal) number — TypeScript types the result as plain 'string' for any number, including one outside the enum's actual member range, where it genuinely returns 'undefined' at runtime." },
 		schema: [],
 		messages: {
-			widening: "Indexing the numeric enum '{{ enumName }}' with a plain 'number' relies on its reverse mapping, which TypeScript types as 'string' for any number -- including one outside the enum's actual members, where this genuinely returns 'undefined' at runtime. Narrow the index to a known member first (a runtime membership check against the enum's own values), or accept that the result may be 'undefined' and handle it.",
+			widening: "Indexing the numeric enum '{{ enumName }}' with a plain 'number' relies on its reverse mapping, which TypeScript types as 'string' for any number — including one outside the enum's actual members, where this genuinely returns 'undefined' at runtime. Narrow the index to a known member first (a runtime membership check against the enum's own values), or accept that the result may be 'undefined' and handle it.",
 			suggestWidenAnnotation: "Widen this variable's annotation to 'string | undefined' so later uses of it as a bare 'string' surface as real compile errors you can resolve."
-		}
+		},
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		const services = _typescript_eslint_utils.ESLintUtils.getParserServices(context);
 		const checker = services.program.getTypeChecker();
 		return { MemberExpression(node) {
-			if (!node.computed) return;
-			const objectTsNode = services.esTreeNodeToTSNodeMap.get(node.object);
-			if (!typescript.isExpression(objectTsNode)) return;
+			const objectTsNode = asExpression(services.esTreeNodeToTSNodeMap.get(node.object));
 			const objectType = checker.getTypeAtLocation(objectTsNode);
 			const objectSymbol = objectType.getSymbol();
 			if (!objectSymbol || !(objectSymbol.flags & typescript.SymbolFlags.Enum)) return;
 			if (!checker.getIndexInfoOfType(objectType, typescript.IndexKind.Number)) return;
-			const propertyTsNode = services.esTreeNodeToTSNodeMap.get(node.property);
-			if (!typescript.isExpression(propertyTsNode)) return;
+			const propertyTsNode = asExpression(services.esTreeNodeToTSNodeMap.get(node.property));
 			const rawPropertyType = checker.getTypeAtLocation(propertyTsNode);
 			const propertyType = checker.getBaseConstraintOfType(rawPropertyType) ?? rawPropertyType;
 			if (propertyType.flags & typescript.TypeFlags.EnumLike) {
@@ -845,7 +850,7 @@ const noEnumReverseLookupWidening = _typescript_eslint_utils.ESLintUtils.RuleCre
 			}
 			const enumName = checker.typeToString(checker.getDeclaredTypeOfSymbol(objectSymbol));
 			const parent = node.parent;
-			if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.VariableDeclarator && parent.init === node && parent.id.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && parent.id.typeAnnotation?.typeAnnotation.type === _typescript_eslint_utils.AST_NODE_TYPES.TSStringKeyword) {
+			if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.VariableDeclarator && parent.id.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && parent.id.typeAnnotation?.typeAnnotation.type === _typescript_eslint_utils.AST_NODE_TYPES.TSStringKeyword) {
 				const stringKeyword = parent.id.typeAnnotation.typeAnnotation;
 				context.report({
 					node,
@@ -874,7 +879,7 @@ const noIndexFiles = {
 	meta: {
 		type: "problem",
 		schema: [],
-		messages: { indexFileBanned: "Index (barrel) files are banned -- import directly from the module that owns the export instead. Rename this file to something descriptive." }
+		messages: { indexFileBanned: "Index (barrel) files are banned — import directly from the module that owns the export instead. Rename this file to something descriptive." }
 	},
 	create(context) {
 		if (!isIndexFile(context.filename)) return {};
@@ -911,7 +916,7 @@ const noMapInstanceofMutation = createRule$6({
 		type: "problem",
 		schema: [],
 		docs: { description: "Disallow mutating calls on a parameter or local variable whose real type includes a ReadonlyMap, narrowed via `instanceof Map`, which silently discards the declared readonly guarantee." },
-		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by 'instanceof Map' -- Map is declared as extending ReadonlyMap, so 'instanceof Map' narrows straight past the readonly guarantee to the full mutable interface, and a value whose real type includes ReadonlyMap (a caller's map, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the map before mutating (e.g. `new Map(input)`), or narrow with a check that preserves readonly instead of 'instanceof Map'." }
+		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by 'instanceof Map' — Map is declared as extending ReadonlyMap, so 'instanceof Map' narrows straight past the readonly guarantee to the full mutable interface, and a value whose real type includes ReadonlyMap (a caller's map, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the map before mutating (e.g. `new Map(input)`), or narrow with a check that preserves readonly instead of 'instanceof Map'." }
 	},
 	defaultOptions: [],
 	create(context) {
@@ -924,14 +929,12 @@ const noMapInstanceofMutation = createRule$6({
 		}
 		return { CallExpression(node) {
 			const { callee } = node;
-			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.object.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_MAP_METHODS.has(callee.property.name)) return;
+			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_MAP_METHODS.has(callee.property.name)) return;
 			const variable = context.sourceCode.getScope(node).references.find((reference) => reference.identifier === callee.object)?.resolved;
 			if (!variable) return;
 			const declarationDefinition = variable.defs.find((definition) => definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Parameter || definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Variable);
 			if (!declarationDefinition) return;
-			const declarationNode = declarationDefinition.name;
-			if (declarationNode.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier) return;
-			if (!declarationHasReadonlyMapConstituent(declarationNode)) return;
+			if (!declarationHasReadonlyMapConstituent(asIdentifierName(declarationDefinition.name))) return;
 			if (!isGuardedByInstanceofMap(node, variable, context)) return;
 			context.report({
 				node,
@@ -959,7 +962,7 @@ const noMapInstanceofMutation = createRule$6({
 					if (parent.consequent === current && matchesInstanceofMapOn(parent.test, parameterVariable, ruleContext)) return true;
 					if (parent.alternate === current && isNegatedInstanceofMapExpression(parent.test, parameterVariable, ruleContext)) return true;
 				}
-				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && parent.right === current && matchesInstanceofMapOn(parent.left, parameterVariable, ruleContext)) return true;
+				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && matchesInstanceofMapOn(parent.left, parameterVariable, ruleContext)) return true;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ConditionalExpression && parent.consequent === current && matchesInstanceofMapOn(parent.test, parameterVariable, ruleContext)) return true;
 				current = parent;
 			}
@@ -971,15 +974,7 @@ const noMapInstanceofMutation = createRule$6({
 				const { parent } = current;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.BlockStatement || parent.type === _typescript_eslint_utils.AST_NODE_TYPES.Program) {
 					const statements = parent.body;
-					let ownIndex = -1;
-					for (let i = 0; i < statements.length; i++) if (statements[i] === current) {
-						ownIndex = i;
-						break;
-					}
-					if (ownIndex > 0) for (let i = ownIndex - 1; i >= 0; i--) {
-						const sibling = statements[i];
-						if (sibling?.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedInstanceofMapExpression(sibling.test, parameterVariable, ruleContext) && definitelyExits$1(sibling.consequent)) return true;
-					}
+					if (statements.slice(0, statements.indexOf(current)).some((sibling) => sibling.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedInstanceofMapExpression(sibling.test, parameterVariable, ruleContext) && definitelyExits$1(sibling.consequent))) return true;
 				}
 				current = parent;
 			}
@@ -997,6 +992,11 @@ const MUTATING_INSERT_METHODS = /* @__PURE__ */ new Set([
 	"copyWithin"
 ]);
 const createRule$5 = _typescript_eslint_utils.ESLintUtils.RuleCreator((name) => `https://github.com/ExaDev/eslint-config/blob/main/src/rules/${name}.ts`);
+function buildReadonlyArrayFix(annotated, fixer) {
+	if (annotated.type === _typescript_eslint_utils.AST_NODE_TYPES.TSArrayType) return fixer.insertTextBefore(annotated, "readonly ");
+	if (annotated.type !== _typescript_eslint_utils.AST_NODE_TYPES.TSTypeReference) throw new Error(`Unreachable: expected a TSArrayType or TSTypeReference, got ${annotated.type} instead.`);
+	return fixer.replaceText(annotated.typeName, "ReadonlyArray");
+}
 function isUnionArrayType(typeAnnotation) {
 	if (typeAnnotation.type === _typescript_eslint_utils.AST_NODE_TYPES.TSArrayType && typeAnnotation.elementType.type === _typescript_eslint_utils.AST_NODE_TYPES.TSUnionType) return typeAnnotation.elementType;
 	if (typeAnnotation.type === _typescript_eslint_utils.AST_NODE_TYPES.TSTypeReference && typeAnnotation.typeName.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && typeAnnotation.typeName.name === "Array" && typeAnnotation.typeArguments?.params.length === 1) {
@@ -1011,30 +1011,24 @@ const noMutableUnionArrayParam = createRule$5({
 		fixable: "code",
 		docs: { description: "Disallow mutating-insertion calls on a union-element array parameter, which lets a caller pass a narrower array whose declared element type the call can silently violate." },
 		schema: [],
-		messages: { unsound: "'{{ method }}' inserts into a parameter typed as an array of a union -- a caller may have passed a narrower array (e.g. number[] where (string | number)[] is declared), and TypeScript's covariant array typing does not catch the resulting mismatch. Mark the parameter readonly to turn this into a real compile error, or narrow the parameter type." }
+		messages: { unsound: "'{{ method }}' inserts into a parameter typed as an array of a union — a caller may have passed a narrower array (e.g. number[] where (string | number)[] is declared), and TypeScript's covariant array typing does not catch the resulting mismatch. Mark the parameter readonly to turn this into a real compile error, or narrow the parameter type." },
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		return { CallExpression(node) {
 			const { callee } = node;
-			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.object.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_INSERT_METHODS.has(callee.property.name)) return;
+			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_INSERT_METHODS.has(callee.property.name)) return;
 			const parameterDefinition = (context.sourceCode.getScope(node).references.find((reference) => reference.identifier === callee.object)?.resolved)?.defs.find((definition) => definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Parameter);
 			if (!parameterDefinition) return;
-			const parameterNode = parameterDefinition.name;
-			if (parameterNode.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !parameterNode.typeAnnotation) return;
-			if (!isUnionArrayType(parameterNode.typeAnnotation.typeAnnotation)) return;
+			const parameterNode = asIdentifierName(parameterDefinition.name);
+			if (!parameterNode.typeAnnotation) return;
+			const annotated = parameterNode.typeAnnotation.typeAnnotation;
+			if (!isUnionArrayType(annotated)) return;
 			context.report({
 				node,
 				messageId: "unsound",
 				data: { method: callee.property.name },
-				fix(fixer) {
-					const typeAnnotation = parameterNode.typeAnnotation;
-					if (!typeAnnotation) return null;
-					const annotated = typeAnnotation.typeAnnotation;
-					if (annotated.type === _typescript_eslint_utils.AST_NODE_TYPES.TSArrayType) return fixer.insertTextBefore(annotated, "readonly ");
-					if (annotated.type === _typescript_eslint_utils.AST_NODE_TYPES.TSTypeReference && annotated.typeName.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier) return fixer.replaceText(annotated.typeName, "ReadonlyArray");
-					return null;
-				}
+				fix: (fixer) => buildReadonlyArrayFix(annotated, fixer)
 			});
 		} };
 	}
@@ -1050,8 +1044,7 @@ const noNonBarrelIndex = {
 	},
 	create(context) {
 		const filename = context.filename;
-		const slash = filename.lastIndexOf("/");
-		const basename = slash === -1 ? filename : filename.slice(slash + 1);
+		const basename = filename.slice(filename.lastIndexOf("/") + 1);
 		if (!INDEX_BASENAME.test(basename)) return {};
 		if (filename.endsWith("/src/index.ts")) return {};
 		return { Program(node) {
@@ -1086,8 +1079,8 @@ const noNonBarrelReexport = {
 		fixable: "code",
 		schema: [],
 		messages: {
-			splitStatementReexport: "'{{ name }}' is imported here and handed straight back out via a bare export -- the identical re-export 'export { {{ name }} } from ...' would be, just split across two statements. Re-exports belong only in the public barrel.",
-			splitStatementDefaultReexport: "'{{ name }}' is imported here and handed straight back out via `export default` -- the identical re-export 'export { {{ name }} as default } from ...' would be, just split across two statements. Re-exports belong only in the public barrel."
+			splitStatementReexport: "'{{ name }}' is imported here and handed straight back out via a bare export — the identical re-export 'export { {{ name }} } from ...' would be, just split across two statements. Re-exports belong only in the public barrel.",
+			splitStatementDefaultReexport: "'{{ name }}' is imported here and handed straight back out via `export default` — the identical re-export 'export { {{ name }} as default } from ...' would be, just split across two statements. Re-exports belong only in the public barrel."
 		}
 	},
 	create(context) {
@@ -1118,14 +1111,14 @@ const noNonBarrelReexport = {
 						}
 					});
 				} else {
-					const { declaration, name, trackedImport } = violation;
+					const { declaration, identifierNode, name, trackedImport } = violation;
 					context.report({
 						node: declaration,
 						messageId: "splitStatementDefaultReexport",
 						data: { name },
 						fix(fixer) {
 							const fixes = [fixer.remove(declaration)];
-							if (declaration.declaration.type === "Identifier" && importIsOnlyUsedByThisExport(sourceCode, trackedImport, declaration.declaration)) fixes.push(removeListMember(fixer, sourceCode, trackedImport.declaration, trackedImport.declaration.specifiers, trackedImport.specifier));
+							if (importIsOnlyUsedByThisExport(sourceCode, trackedImport, identifierNode)) fixes.push(removeListMember(fixer, sourceCode, trackedImport.declaration, trackedImport.declaration.specifiers, trackedImport.specifier));
 							return fixes;
 						}
 					});
@@ -1149,14 +1142,14 @@ const noObjectAssign = createRule$4({
 		type: "problem",
 		fixable: "code",
 		hasSuggestions: true,
-		docs: { description: "Disallow Object.assign, whose own type declarations do not check a source object's properties against the target's declared types -- object spread does." },
+		docs: { description: "Disallow Object.assign, whose own type declarations do not check a source object's properties against the target's declared types — object spread does." },
 		schema: [],
 		messages: {
 			unsound: "Object.assign does not verify that a source object's properties are assignable to the target's declared types, so a type mismatch here passes silently where a direct property assignment would be rejected. Use object spread ({ ...target, ...source }) to build a correctly type-checked replacement instead.",
-			suggestSpreadReassign: "Replace with object spread and reassignment (changes the object reference -- anything else already holding this object will not see the update)."
-		}
+			suggestSpreadReassign: "Replace with object spread and reassignment (changes the object reference — anything else already holding this object will not see the update)."
+		},
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		return { CallExpression(node) {
 			const { callee } = node;
@@ -1164,7 +1157,7 @@ const noObjectAssign = createRule$4({
 			const [target, ...sources] = node.arguments;
 			const sourcesAreSpreadable = sources.every((argument) => argument.type !== _typescript_eslint_utils.AST_NODE_TYPES.SpreadElement);
 			if (target?.type === _typescript_eslint_utils.AST_NODE_TYPES.ObjectExpression && sourcesAreSpreadable) {
-				const isBareStatement = node.parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ExpressionStatement && node.parent.expression === node;
+				const isBareStatement = node.parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ExpressionStatement;
 				context.report({
 					node,
 					messageId: "unsound",
@@ -1177,7 +1170,7 @@ const noObjectAssign = createRule$4({
 				});
 				return;
 			}
-			if (target?.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && sourcesAreSpreadable && node.parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ExpressionStatement && node.parent.expression === node) {
+			if (target?.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && sourcesAreSpreadable && node.parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ExpressionStatement) {
 				const targetName = target.name;
 				const variable = resolveFrom$1(context.sourceCode.getScope(node), targetName);
 				const isConst = (variable?.defs.find((candidate) => candidate.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Variable))?.parent.kind === "const";
@@ -1210,7 +1203,16 @@ function isIdentifierReference(reference) {
 	return reference.identifier.type === "Identifier";
 }
 function hasTypeAnnotation(id) {
-	return "typeAnnotation" in id && id.typeAnnotation !== void 0 && id.typeAnnotation !== null;
+	return "typeAnnotation" in id && id.typeAnnotation !== void 0;
+}
+function isConstDeclarator(declarator) {
+	if (declarator.parent.type !== "VariableDeclaration") throw new Error(`Unreachable: expected a VariableDeclarator's own parent to be a VariableDeclaration, got "${declarator.parent.type}" instead.`);
+	return declarator.parent.kind === "const";
+}
+function variableInScope(scope, name) {
+	const variable = scope.set.get(name);
+	if (variable === void 0) throw new Error(`Unreachable: expected scope.set to hold a variable named "${name}".`);
+	return variable;
 }
 function resolveFrom(scope, name) {
 	for (let current = scope; current; current = current.upper) {
@@ -1218,6 +1220,18 @@ function resolveFrom(scope, name) {
 		if (found) return found;
 	}
 }
+function hasEnclosingShorthandBoundary(precedingTokens) {
+	for (const token of [...precedingTokens].reverse()) {
+		if (token.value === "{" || token.value === ",") return true;
+		if (token.value === "[" || token.value === "(" || token.value === ":") return false;
+	}
+	return false;
+}
+function isShorthandPropertyRead(sourceCode, identifier) {
+	const afterToken = sourceCode.getTokenAfter(identifier);
+	if (afterToken?.value !== "}" && afterToken?.value !== ",") return false;
+	return hasEnclosingShorthandBoundary(sourceCode.getTokensBefore(identifier));
+}
 const noPointlessReassignment = {
 	meta: {
 		type: "problem",
@@ -1228,7 +1242,7 @@ const noPointlessReassignment = {
 	create(context) {
 		return { VariableDeclarator(node) {
 			if (node.id.type !== "Identifier" || node.init?.type !== "Identifier" || node.id.name.startsWith("_")) return;
-			if (node.parent.type !== "VariableDeclaration" || node.parent.kind !== "const") return;
+			if (!isConstDeclarator(node)) return;
 			const scope = context.sourceCode.getScope(node);
 			const sourceVariable = scope.references.find((reference) => reference.identifier === node.init)?.resolved;
 			if (!sourceVariable || sourceVariable.references.some((reference) => reference.isWrite() && reference.init !== true)) return;
@@ -1243,24 +1257,13 @@ const noPointlessReassignment = {
 					value: originalName
 				},
 				fix(fixer) {
-					const variable = scope.set.get(aliasName);
-					if (!variable) return null;
+					const variable = variableInScope(scope, aliasName);
 					if (aliasIsAnnotated) return null;
 					if (variable.references.filter((reference) => reference.isWrite() && reference.identifier !== node.id).length > 0) return null;
-					const readRefs = variable.references.filter((reference) => reference.isRead() && isIdentifierReference(reference));
-					if (readRefs.some((reference) => {
-						const afterToken = context.sourceCode.getTokenAfter(reference.identifier);
-						if (afterToken?.value === ":") return false;
-						if (afterToken?.value !== "}" && afterToken?.value !== ",") return false;
-						let token = context.sourceCode.getTokenBefore(reference.identifier);
-						while (token) {
-							if (token.value === "{") return true;
-							if (token.value === "[" || token.value === "(") return false;
-							if (token.value === ":") return false;
-							token = context.sourceCode.getTokenBefore(token);
-						}
-						return false;
-					})) return null;
+					const readReferences = variable.references.filter((reference) => reference.isRead());
+					const readRefs = readReferences.filter(isIdentifierReference);
+					if (readRefs.length !== readReferences.length) return null;
+					if (readRefs.some((reference) => isShorthandPropertyRead(context.sourceCode, reference.identifier))) return null;
 					if (readRefs.some((reference) => resolveFrom(reference.from, originalName) !== sourceVariable)) return null;
 					const fixes = readRefs.map((reference) => fixer.replaceText(reference.identifier, originalName));
 					const declaration = node.parent;
@@ -1297,7 +1300,7 @@ const noSetInstanceofMutation = createRule$3({
 		type: "problem",
 		schema: [],
 		docs: { description: "Disallow mutating calls on a parameter or local variable whose real type includes a ReadonlySet, narrowed via instanceof Set, which silently discards the declared read-only guarantee." },
-		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by instanceof Set -- instanceof Set's own narrowing widens straight to the mutable Set interface, so a value whose real type includes a ReadonlySet (a caller's set, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the set before mutating (e.g. new Set(input)), or narrow with a check that preserves read-only instead of instanceof Set." }
+		messages: { unsound: "'{{ method }}' mutates a parameter or local variable narrowed by instanceof Set — instanceof Set's own narrowing widens straight to the mutable Set interface, so a value whose real type includes a ReadonlySet (a caller's set, for a parameter; the value's own declared type, for a local variable) can be mutated here despite that readonly guarantee. Copy the set before mutating (e.g. new Set(input)), or narrow with a check that preserves read-only instead of instanceof Set." }
 	},
 	defaultOptions: [],
 	create(context) {
@@ -1310,14 +1313,12 @@ const noSetInstanceofMutation = createRule$3({
 		}
 		return { CallExpression(node) {
 			const { callee } = node;
-			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.object.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_SET_METHODS.has(callee.property.name)) return;
+			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed || callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !MUTATING_SET_METHODS.has(callee.property.name)) return;
 			const variable = context.sourceCode.getScope(node).references.find((reference) => reference.identifier === callee.object)?.resolved;
 			if (!variable) return;
 			const declarationDefinition = variable.defs.find((definition) => definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Parameter || definition.type === _typescript_eslint_utils.TSESLint.Scope.DefinitionType.Variable);
 			if (!declarationDefinition) return;
-			const declarationNode = declarationDefinition.name;
-			if (declarationNode.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier) return;
-			if (!declarationHasReadonlySetConstituent(declarationNode)) return;
+			if (!declarationHasReadonlySetConstituent(asIdentifierName(declarationDefinition.name))) return;
 			if (!isGuardedBySetInstanceof(node, variable, context)) return;
 			context.report({
 				node,
@@ -1345,7 +1346,7 @@ const noSetInstanceofMutation = createRule$3({
 					if (parent.consequent === current && matchesSetInstanceofOn(parent.test, parameterVariable, ruleContext)) return true;
 					if (parent.alternate === current && isNegatedSetInstanceofExpression(parent.test, parameterVariable, ruleContext)) return true;
 				}
-				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && parent.right === current && matchesSetInstanceofOn(parent.left, parameterVariable, ruleContext)) return true;
+				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression && parent.operator === "&&" && matchesSetInstanceofOn(parent.left, parameterVariable, ruleContext)) return true;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.ConditionalExpression && parent.consequent === current && matchesSetInstanceofOn(parent.test, parameterVariable, ruleContext)) return true;
 				current = parent;
 			}
@@ -1357,15 +1358,7 @@ const noSetInstanceofMutation = createRule$3({
 				const { parent } = current;
 				if (parent.type === _typescript_eslint_utils.AST_NODE_TYPES.BlockStatement || parent.type === _typescript_eslint_utils.AST_NODE_TYPES.Program) {
 					const statements = parent.body;
-					let ownIndex = -1;
-					for (let i = 0; i < statements.length; i++) if (statements[i] === current) {
-						ownIndex = i;
-						break;
-					}
-					if (ownIndex > 0) for (let i = ownIndex - 1; i >= 0; i--) {
-						const sibling = statements[i];
-						if (sibling?.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedSetInstanceofExpression(sibling.test, parameterVariable, ruleContext) && definitelyExits(sibling.consequent)) return true;
-					}
+					if (statements.slice(0, statements.indexOf(current)).some((sibling) => sibling.type === _typescript_eslint_utils.AST_NODE_TYPES.IfStatement && !sibling.alternate && isNegatedSetInstanceofExpression(sibling.test, parameterVariable, ruleContext) && definitelyExits(sibling.consequent))) return true;
 				}
 				current = parent;
 			}
@@ -1379,7 +1372,7 @@ const noSideEffectsInIndex = {
 	meta: {
 		type: "problem",
 		schema: [],
-		messages: { notAPureReexport: "A barrel (index) file 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 }}." }
+		messages: { notAPureReexport: "A barrel (index) file 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 (!isIndexFile(context.filename)) return {};
@@ -1410,6 +1403,12 @@ const DEFAULT_SORT_AZ = [
 	"resolutions",
 	"scripts"
 ];
+/** Reads `array[index]`, throwing loudly instead of silently returning `undefined` when it is genuinely out of bounds. Every call site in this file passes an index that is provably in range by construction (a loop bound, a length check, or a mathematical invariant established immediately above the call) — this exists to satisfy `noUncheckedIndexedAccess` with a real runtime guarantee instead of a silent `as` cast or a `?? fallback` that could quietly paper over a genuine bug, and none of the arrays this file indexes (keys, nodes, ranges, characters) ever legitimately store `undefined` as an element. Exported so each call site's own "this can never actually throw" claim is checked directly against a deliberately out-of-bounds input, rather than trusted on the strength of a comment alone. */
+function at(array, index) {
+	const value = array[index];
+	if (value === void 0) throw new Error(`Unreachable: index ${String(index)} is out of bounds for an array of length ${String(array.length)}.`);
+	return value;
+}
 function categorize(char) {
 	if (char >= "0" && char <= "9") return 1;
 	if (/[a-z]/iu.test(char)) return 2;
@@ -1419,14 +1418,15 @@ function categorize(char) {
 function compareSyncpackKey(a, b) {
 	const length = Math.min(a.length, b.length);
 	for (let index = 0; index < length; index += 1) {
-		const charA = a[index] ?? "";
-		const charB = b[index] ?? "";
+		const charA = a.charAt(index);
+		const charB = b.charAt(index);
 		const categoryA = categorize(charA);
 		const categoryB = categorize(charB);
 		if (categoryA !== categoryB) return categoryA - categoryB;
 		const foldedA = categoryA === 2 ? charA.toLowerCase() : charA;
 		const foldedB = categoryB === 2 ? charB.toLowerCase() : charB;
-		if (foldedA !== foldedB) return foldedA < foldedB ? -1 : 1;
+		if (foldedA < foldedB) return -1;
+		if (foldedA > foldedB) return 1;
 	}
 	return a.length - b.length;
 }
@@ -1446,16 +1446,16 @@ function isValidSortAzOrder(prev, curr) {
 	return compareSyncpackKey(prev, curr) <= 0;
 }
 /**
-* The target permutation for a whole container's own keys, given a comparator over adjacent pairs (either `isValidTopLevelOrder`-shaped or `isValidSortAzOrder`-shaped). Returns, for each output position, the ORIGINAL index of the key that belongs there -- e.g. `[2, 0, 1]` means the key currently at index 2 comes first. A stable sort (ties keep their original relative order), matching real syncpack output for keys that compare equal (see `compareSyncpackKey`'s own case-insensitive-letter test case).
+* The target permutation for a whole container's own keys, given a comparator over adjacent pairs (either `isValidTopLevelOrder`-shaped or `isValidSortAzOrder`-shaped). Returns, for each output position, the ORIGINAL index of the key that belongs there — e.g. `[2, 0, 1]` means the key currently at index 2 comes first. A stable sort (ties keep their original relative order), matching real syncpack output for keys that compare equal (see `compareSyncpackKey`'s own case-insensitive-letter test case).
 *
-* Reordering by computing this full permutation once, rather than reporting and fixing one out-of-order adjacent pair at a time (the technique `@eslint/json`'s own `sort-keys` rule uses), is deliberate: an adjacent-swap fixer only makes one pass of bubble-sort-style progress per lint pass, and a realistically scrambled `package.json` (many top-level keys all needing to move several positions, several nested objects also needing sorting) was confirmed directly to need up to 11 fix passes to fully converge -- one more than ESLint's own `Linter#verifyAndFix` will ever run (`MAX_AUTOFIX_PASSES` is 10), meaning a single real `eslint --fix` invocation could leave such a file only partially reordered. Computing the whole target order up front and replacing every affected key's position in one combined fix converges in a single pass regardless of how scrambled the input is.
+* Reordering by computing this full permutation once, rather than reporting and fixing one out-of-order adjacent pair at a time (the technique `@eslint/json`'s own `sort-keys` rule uses), is deliberate: an adjacent-swap fixer only makes one pass of bubble-sort-style progress per lint pass, and a realistically scrambled `package.json` (many top-level keys all needing to move several positions, several nested objects also needing sorting) was confirmed directly to need up to 11 fix passes to fully converge — one more than ESLint's own `Linter#verifyAndFix` will ever run (`MAX_AUTOFIX_PASSES` is 10), meaning a single real `eslint --fix` invocation could leave such a file only partially reordered. Computing the whole target order up front and replacing every affected key's position in one combined fix converges in a single pass regardless of how scrambled the input is.
 */
 function computeOrderPermutation(keys, isValidOrder) {
 	return keys.map((_, index) => index).map((index) => ({
-		key: keys[index] ?? "",
+		key: at(keys, index),
 		index
 	})).sort((a, b) => {
-		if (isValidOrder(a.key, b.key) && isValidOrder(b.key, a.key)) return a.index - b.index;
+		if (isValidOrder(a.key, b.key) && isValidOrder(b.key, a.key)) return 0;
 		return isValidOrder(a.key, b.key) ? -1 : 1;
 	}).map((entry) => entry.index);
 }
@@ -1465,17 +1465,27 @@ function isIdentityPermutation(permutation) {
 function isStringNode(node) {
 	return node.type === "String";
 }
+function isStringNamed(name) {
+	return name.type === "String";
+}
 function getMemberKeyName(member) {
-	return member.name.type === "String" ? member.name.value : void 0;
+	const { name } = member;
+	if (!isStringNamed(name)) throw new Error(`Unreachable: package-json-key-order only supports json/json and json/jsonc, where an unquoted (Identifier) member name is a parse error — got a "${name.type}" name instead.`);
+	return name.value;
+}
+function rangeOf(node) {
+	if (node.range === void 0) throw new Error("Unreachable: every node reaching this function was produced by a JSON language that always requests range tracking.");
+	return node.range;
 }
-function hasRange(node) {
-	return node.range !== void 0;
+function objectParentOrThrow(parent) {
+	if (parent === void 0) throw new Error("Unreachable: the Object visitor is never invoked for the traversal root, which is always the Document node.");
+	return parent;
 }
 const commentTypes = /* @__PURE__ */ new Set(["LineComment", "BlockComment"]);
 /**
-* Enforces the same `package.json` key order `syncpack format` would produce: `sortFirst` fields pinned to the top in that exact order, then every other top-level key alphabetically; and, inside each `sortAz`-listed field's own object or array value, its members/elements sorted the same way (no pinning there -- `sortFirst` only ever applies at the top level).
+* Enforces the same `package.json` key order `syncpack format` would produce: `sortFirst` fields pinned to the top in that exact order, then every other top-level key alphabetically; and, inside each `sortAz`-listed field's own object or array value, its members/elements sorted the same way (no pinning there — `sortFirst` only ever applies at the top level).
 *
-* This exists so a project using `@exadev/eslint-config` without syncpack still gets real `package.json` canonicalization, and so a project using both never sees them fight: `eslint --fix` and `syncpack format` converge on the identical output, confirmed directly against real `syncpack@15` output rather than assumed from its docs (see `compareSyncpackKey`'s own comment). Deliberately narrower than `@eslint/json`'s own `sort-keys` rule, which only supports a single alphabetical order for every key with no field-specific pinning -- not a fit for syncpack's own two-tier scheme.
+* This exists so a project using `@exadev/eslint-config` without syncpack still gets real `package.json` canonicalization, and so a project using both never sees them fight: `eslint --fix` and `syncpack format` converge on the identical output, confirmed directly against real `syncpack@15` output rather than assumed from its docs (see `compareSyncpackKey`'s own comment). Deliberately narrower than `@eslint/json`'s own `sort-keys` rule, which only supports a single alphabetical order for every key with no field-specific pinning — not a fit for syncpack's own two-tier scheme.
 *
 * Not part of `plugin.configs.recommended`: opt-in, the same tri-state shape as `react`/`nextjs` (see `docs.md`'s own usage example), since a project without syncpack needs to choose this deliberately, and a project with syncpack needs to confirm the two tools agree before turning it on.
 */
@@ -1513,65 +1523,42 @@ const packageJsonKeyOrder = {
 		const { sourceCode } = context;
 		function hasAdjacentComment(node) {
 			const before = sourceCode.getTokenBefore(node, { includeComments: true });
-			let after = sourceCode.getTokenAfter(node, { includeComments: true });
-			if (after?.type === "Comma") after = sourceCode.getTokenAfter(after, { includeComments: true });
+			const after = sourceCode.getTokenAfter(node, { includeComments: true });
 			return before !== null && commentTypes.has(before.type) || after !== null && commentTypes.has(after.type);
 		}
 		function buildReorderedText(nodes, permutation) {
 			const separators = nodes.slice(0, -1).map((node, index) => {
-				const next = nodes[index + 1];
-				return hasRange(node) && next !== void 0 && hasRange(next) ? sourceCode.text.slice(node.range[1], next.range[0]) : "";
+				const next = at(nodes, index + 1);
+				return sourceCode.text.slice(rangeOf(node)[1], rangeOf(next)[0]);
 			});
-			return permutation.reduce((result, originalIndex, position) => {
-				const text = sourceCode.getText(nodes[originalIndex]);
-				return position === 0 ? text : `${result}${separators[position - 1] ?? ""}${text}`;
-			}, "");
+			return permutation.map((originalIndex) => sourceCode.getText(at(nodes, originalIndex))).reduce((result, text, position) => `${result}${at(separators, position - 1)}${text}`);
 		}
 		function getNodeRanges(nodes) {
-			const ranges = [];
-			for (const node of nodes) {
-				if (!hasRange(node)) return void 0;
-				ranges.push(node.range);
-			}
-			return ranges;
+			return nodes.map((node) => rangeOf(node));
 		}
 		function reportWholeContainerReorder(nodes, keys, isValidOrder, reportLoc) {
 			const permutation = computeOrderPermutation(keys, isValidOrder);
 			if (isIdentityPermutation(permutation)) return;
-			const firstNode = nodes[0];
-			const lastNode = nodes[nodes.length - 1];
-			if (firstNode === void 0 || lastNode === void 0) return;
-			let violationIndex = 1;
-			for (let index = 1; index < keys.length; index += 1) if (!isValidOrder(keys[index - 1] ?? "", keys[index] ?? "")) {
-				violationIndex = index;
-				break;
-			}
+			const violationIndex = keys.findIndex((curr, index) => index > 0 && !isValidOrder(at(keys, index - 1), curr));
 			context.report({
-				loc: reportLoc(nodes[violationIndex] ?? firstNode),
+				loc: reportLoc(at(nodes, violationIndex)),
 				messageId: "outOfOrder",
 				data: {
-					curr: keys[violationIndex] ?? "",
-					prev: keys[violationIndex - 1] ?? ""
+					curr: at(keys, violationIndex),
+					prev: at(keys, violationIndex - 1)
 				},
 				fix(fixer) {
+					if (nodes.some((node) => hasAdjacentComment(node))) return null;
 					const ranges = getNodeRanges(nodes);
-					if (ranges === void 0 || nodes.some((node) => hasAdjacentComment(node))) return null;
-					const firstRange = ranges[0];
-					const lastRange = ranges[ranges.length - 1];
-					if (firstRange === void 0 || lastRange === void 0) return null;
+					const firstRange = at(ranges, 0);
+					const lastRange = at(ranges, ranges.length - 1);
 					const rewritten = buildReorderedText(nodes, permutation);
 					return fixer.replaceTextRange([firstRange[0], lastRange[1]], rewritten);
 				}
 			});
 		}
 		function checkMembers(members, isValidOrder) {
-			const keys = [];
-			for (const member of members) {
-				const key = getMemberKeyName(member);
-				if (key === void 0) return;
-				keys.push(key);
-			}
-			reportWholeContainerReorder(members, keys, isValidOrder, (member) => member.name.loc.start);
+			reportWholeContainerReorder(members, members.map((member) => getMemberKeyName(member)), isValidOrder, (member) => member.name.loc.start);
 		}
 		function checkElements(elements) {
 			const stringValues = [];
@@ -1582,21 +1569,19 @@ const packageJsonKeyOrder = {
 			reportWholeContainerReorder(stringValues, stringValues.map((value) => value.value), isValidSortAzOrder, (value) => value.loc.start);
 		}
 		return {
-			Object(node, parent) {
-				if (parent === void 0) return;
+			Object(node, rawParent) {
+				const parent = objectParentOrThrow(rawParent);
 				if (parent.type === "Document") {
 					checkMembers(node.members, (prev, curr) => isValidTopLevelOrder(prev, curr, sortFirst));
 					return;
 				}
 				if (parent.type === "Member") {
-					const key = getMemberKeyName(parent);
-					if (key !== void 0 && sortAz.has(key)) checkMembers(node.members, isValidSortAzOrder);
+					if (sortAz.has(getMemberKeyName(parent))) checkMembers(node.members, isValidSortAzOrder);
 				}
 			},
 			Array(node, parent) {
 				if (parent?.type !== "Member") return;
-				const key = getMemberKeyName(parent);
-				if (key !== void 0 && sortAz.has(key)) checkElements(node.elements);
+				if (sortAz.has(getMemberKeyName(parent))) checkElements(node.elements);
 			}
 		};
 	}
@@ -1614,14 +1599,14 @@ const preferNumericSortCompare = createRule$2({
 	meta: {
 		type: "suggestion",
 		hasSuggestions: true,
-		docs: { description: "Suggest an ascending numeric compare function for a bare '.sort()'/'.toSorted()' call on an array whose element type is definitively 'number' -- the default comparator sorts lexicographically, so a bare numeric sort is essentially always a bug." },
+		docs: { description: "Suggest an ascending numeric compare function for a bare '.sort()'/'.toSorted()' call on an array whose element type is definitively 'number' — the default comparator sorts lexicographically, so a bare numeric sort is essentially always a bug." },
 		schema: [],
 		messages: {
 			preferNumericCompare: "'.{{ method }}()' on a number array with no compare function sorts lexicographically (e.g. [1, 2, 10].sort() becomes [1, 10, 2]), not in ascending numeric order. Provide a compare function.",
 			addAscendingCompare: "Add an ascending numeric compare function: '(a, b) => a - b'."
-		}
+		},
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		const services = _typescript_eslint_utils.ESLintUtils.getParserServices(context);
 		const checker = services.program.getTypeChecker();
@@ -1630,12 +1615,10 @@ const preferNumericSortCompare = createRule$2({
 			const { callee } = node;
 			if (callee.type !== _typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || callee.computed) return;
 			if (callee.property.type !== _typescript_eslint_utils.AST_NODE_TYPES.Identifier || !SORT_METHOD_NAMES.has(callee.property.name)) return;
-			const receiverTsNode = services.esTreeNodeToTSNodeMap.get(callee.object);
-			if (!typescript.isExpression(receiverTsNode)) return;
+			const receiverTsNode = asExpression(services.esTreeNodeToTSNodeMap.get(callee.object));
 			const receiverType = checker.getTypeAtLocation(receiverTsNode);
 			if (!checker.isArrayType(receiverType)) return;
-			if (!(0, ts_api_utils.isTypeReference)(receiverType)) return;
-			const [elementType] = checker.getTypeArguments(receiverType);
+			const [elementType] = checker.getTypeArguments(asTypeReference(receiverType));
 			if (!elementType || !isDefinitelyNumberType(elementType)) return;
 			context.report({
 				node,
@@ -1644,8 +1627,7 @@ const preferNumericSortCompare = createRule$2({
 				suggest: [{
 					messageId: "addAscendingCompare",
 					fix(fixer) {
-						const closingParen = context.sourceCode.getLastToken(node);
-						if (!closingParen) return null;
+						const closingParen = lastTokenOrThrow(context.sourceCode, node);
 						return fixer.insertTextBefore(closingParen, "(a, b) => a - b");
 					}
 				}]
@@ -1657,7 +1639,6 @@ const preferNumericSortCompare = createRule$2({
 //#region src/rules/prefer-readonly-array-param.ts
 const createRule$1 = _typescript_eslint_utils.ESLintUtils.RuleCreator((name) => `https://github.com/ExaDev/eslint-config/blob/main/src/rules/${name}.ts`);
 function getFixableArrayOrTupleType(typeNode) {
-	if (typeNode.type === _typescript_eslint_utils.AST_NODE_TYPES.TSTypeOperator && typeNode.operator === "readonly") return void 0;
 	if (typeNode.type === _typescript_eslint_utils.AST_NODE_TYPES.TSArrayType) return typeNode;
 	if (typeNode.type === _typescript_eslint_utils.AST_NODE_TYPES.TSTupleType) return typeNode;
 	if (typeNode.type === _typescript_eslint_utils.AST_NODE_TYPES.TSTypeReference && typeNode.typeName.type === _typescript_eslint_utils.AST_NODE_TYPES.Identifier && typeNode.typeName.name === "Array") return typeNode;
@@ -1688,11 +1669,11 @@ const preferReadonlyArrayParam = createRule$1({
 	meta: {
 		type: "problem",
 		fixable: "code",
-		docs: { description: "Require array and tuple parameters to be typed readonly, regardless of whether the function body mutates them -- a narrower, safely-autofixable sibling of @typescript-eslint/prefer-readonly-parameter-types scoped to array/tuple shapes only." },
+		docs: { description: "Require array and tuple parameters to be typed readonly, regardless of whether the function body mutates them — a narrower, safely-autofixable sibling of @typescript-eslint/prefer-readonly-parameter-types scoped to array/tuple shapes only." },
 		schema: [],
-		messages: { preferReadonly: "Array and tuple parameters should be typed readonly ({{ suggestion }}) so a caller can pass a readonly or shared array with confidence, and so any mutation inside the function becomes a deliberate, visible compile error instead of a silent side effect on the caller's data." }
+		messages: { preferReadonly: "Array and tuple parameters should be typed readonly ({{ suggestion }}) so a caller can pass a readonly or shared array with confidence, and so any mutation inside the function becomes a deliberate, visible compile error instead of a silent side effect on the caller's data." },
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		function checkParam(param) {
 			const annotatedNode = getAnnotatedParamNode$1(param);
@@ -1746,15 +1727,18 @@ function isFlatPropertyType(checker, type) {
 	if ((type.flags & PRIMITIVE_LIKE_FLAGS) !== 0) return true;
 	return checker.getSignaturesOfType(type, typescript.SignatureKind.Call).length > 0 && checker.getPropertiesOfType(type).length === 0 && checker.getIndexInfosOfType(type).length === 0;
 }
+function hasSymbolFlag(type, flag) {
+	return ((type.getSymbol()?.flags ?? 0) & flag) !== 0;
+}
 function isFlatObjectType(checker, type, location) {
 	if ((type.flags & typescript.TypeFlags.Object) === 0) return false;
-	if (type.flags & (typescript.TypeFlags.Union | typescript.TypeFlags.Intersection | typescript.TypeFlags.TypeParameter)) return false;
-	if (checker.isArrayType(type) || checker.isTupleType(type)) return false;
 	if (checker.getSignaturesOfType(type, typescript.SignatureKind.Call).length > 0) return false;
 	if (checker.getSignaturesOfType(type, typescript.SignatureKind.Construct).length > 0) return false;
-	if ((type.getSymbol()?.flags ?? 0) & typescript.SymbolFlags.Class) return false;
+	if (hasSymbolFlag(type, typescript.SymbolFlags.Class)) return false;
 	const symbolName = type.getSymbol()?.name;
+	if (symbolName === "Date") return false;
 	if (symbolName === "Map" || symbolName === "ReadonlyMap" || symbolName === "Set" || symbolName === "ReadonlySet") return false;
+	if (symbolName === "WeakMap" || symbolName === "WeakSet") return false;
 	for (const property of checker.getPropertiesOfType(type)) if (!isFlatPropertyType(checker, checker.getTypeOfSymbolAtLocation(property, location))) return false;
 	for (const indexInfo of checker.getIndexInfosOfType(type)) if (!isFlatPropertyType(checker, indexInfo.type)) return false;
 	return true;
@@ -1770,11 +1754,11 @@ const preferReadonlyObjectParam = createRule({
 	meta: {
 		type: "problem",
 		fixable: "code",
-		docs: { description: "Require a 'flat' object parameter (every property is a primitive or a callback, so a shallow wrapper is provably sufficient) to be typed 'Readonly<T>' -- a narrower, safely-autofixable sibling of @typescript-eslint/prefer-readonly-parameter-types and of this package's own prefer-readonly-array-param, scoped to the object shapes where a shallow fix is genuinely complete." },
+		docs: { description: "Require a 'flat' object parameter (every property is a primitive or a callback, so a shallow wrapper is provably sufficient) to be typed 'Readonly<T>' — a narrower, safely-autofixable sibling of @typescript-eslint/prefer-readonly-parameter-types and of this package's own prefer-readonly-array-param, scoped to the object shapes where a shallow fix is genuinely complete." },
 		schema: [],
-		messages: { preferReadonlyObject: "This object parameter is 'flat' -- every property (and index-signature value, if any) is a primitive or a callback, so there is no nested mutable state a shallow wrapper could miss. Wrap it in 'Readonly<...>' so a caller can pass a readonly or shared object with confidence, and so any attempt to mutate it inside the function becomes a deliberate, visible compile error instead of a silent side effect on the caller's data." }
+		messages: { preferReadonlyObject: "This object parameter is 'flat' — every property (and index-signature value, if any) is a primitive or a callback, so there is no nested mutable state a shallow wrapper could miss. Wrap it in 'Readonly<...>' so a caller can pass a readonly or shared object with confidence, and so any attempt to mutate it inside the function becomes a deliberate, visible compile error instead of a silent side effect on the caller's data." },
+		defaultOptions: []
 	},
-	defaultOptions: [],
 	create(context) {
 		const services = _typescript_eslint_utils.ESLintUtils.getParserServices(context);
 		const checker = services.program.getTypeChecker();
@@ -1896,14 +1880,13 @@ function resolveJsonPlugin(value) {
 	if (isRecord(value) && isJsonLanguagePlugin(value["default"])) return value["default"];
 	return isJsonLanguagePlugin(value) ? value : void 0;
 }
-const JSON_PLUGIN_INSTALL_COMMAND = "pnpm add -D @eslint/json";
 function buildPackageJsonKeyOrderConfig(options = {}) {
 	const cwd = options.cwd ?? process.cwd();
 	if (options.enabled === false) return [];
 	if (options.enabled === void 0 && hasSyncpackConfig(cwd)) return [];
 	const jsonPlugin = resolveJsonPlugin(tryRequire("@eslint/json", options.requireFn));
 	if (jsonPlugin === void 0) {
-		if (options.enabled === true) throw new Error(`@exadev/eslint-config: package.json key ordering was explicitly requested but '@eslint/json' could not be resolved. Install it with: ${JSON_PLUGIN_INSTALL_COMMAND}`);
+		if (options.enabled === true) throw new Error(`@exadev/eslint-config: package.json key ordering was explicitly requested but '@eslint/json' could not be resolved. Install it with: pnpm add -D @eslint/json`);
 		return [];
 	}
 	const ruleOptions = {