eslint-plugin-remeda 1.7.0 → 1.8.0

package/dist/index.cjs

@@ -41,12 +41,13 @@ var main = "dist/index.js";
 var scripts = {
 	"build": "tsdown",
 	"typecheck": "tsc",
-	"lint": "eslint",
+	"lint": "eslint --max-warnings=0",
 	"knip": "knip",
 	"test:watch": "vitest --ui",
 	"test": "vitest run",
-	"attw": "attw --pack .",
-	"qa": "pnpm typecheck && pnpm test && pnpm knip && attw",
+	"check-sheriff-inconsistencies": "sheriff",
+	"attw": "attw --ignore-rules=false-export-default --pack .",
+	"qa": "pnpm build && pnpm typecheck && pnpm lint && pnpm test && pnpm knip && pnpm attw && pnpm check-sheriff-inconsistencies",
 	"nuke": "rm -rf node_modules pnpm-lock.yaml",
 	"update:eslint-docs": "eslint-doc-generator",
 	"lint:eslint-docs": "pnpm update:eslint-docs -- --check",
@@ -80,7 +81,7 @@ var bugs = "https://github.com/AndreaPontrandolfo/eslint-plugin-remeda/issues";
 var peerDependencies = { "eslint": ">=9.0.0" };
 var devDependencies = {
 	"@arethetypeswrong/cli": "^0.17.4",
-	"@sherifforg/cli": "^8.2.0",
+	"@sherifforg/cli": "^8.3.0",
 	"@types/lodash-es": "^4.17.12",
 	"@types/node": "^22.14.0",
 	"@typescript-eslint/utils": "^8.29.0",
@@ -202,8 +203,8 @@ function isMemberExpOf$1(node, objectName, { maxLength = Number.MAX_VALUE, allow
 	let currentNode = node;
 	let depth = maxLength;
 	while (currentNode && depth) if (allowComputed || isPropAccess(currentNode)) {
-		if (currentNode.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && "name" in currentNode.object && currentNode.object.name === objectName) return true;
-		currentNode = "object" in currentNode ? currentNode.object : void 0;
+		if (currentNode.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && currentNode.object.type === __typescript_eslint_utils.AST_NODE_TYPES.Identifier && currentNode.object.name === objectName) return true;
+		currentNode = currentNode.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression ? currentNode.object : void 0;
 		depth = depth - 1;
 	} else return false;
 	return false;
@@ -239,7 +240,7 @@ function hasOnlyOneStatement$2(func) {
 * @param node - The node to check.
 */
 function isObjectOfMethodCall(node) {
-	return (0, lodash_es.get)(node, "parent.object") === node && (0, lodash_es.get)(node, "parent.parent.type") === "CallExpression";
+	return (0, lodash_es.get)(node, "parent.object") === node && (0, lodash_es.get)(node, "parent.parent.type") === __typescript_eslint_utils.AST_NODE_TYPES.CallExpression;
 }
 /**
 * Returns whether the node is a literal.
@@ -405,7 +406,9 @@ function getExpressionComparedToInt$1(node, value$1, checkOver) {
 *
 * @param node - The node to check.
 */
-const isIndexOfCall = (node) => isMethodCall$1(node) && getMethodName$2(node) === "indexOf";
+const isIndexOfCall = (node) => {
+	return isMethodCall$1(node) && getMethodName$2(node) === "indexOf";
+};
 /**
 * Returns whether the node is a call to findIndex.
 *
@@ -549,7 +552,7 @@ __export(methodData_exports, {
 	fromPairs: () => fromPairs,
 	functions: () => functions,
 	functionsIn: () => functionsIn,
-	get: () => get$5,
+	get: () => get$4,
 	groupBy: () => groupBy,
 	gt: () => gt,
 	gte: () => gte,
@@ -593,14 +596,14 @@ __export(methodData_exports, {
 	isNative: () => isNative,
 	isNull: () => isNull,
 	isNullish: () => isNullish,
-	isNumber: () => isNumber$1,
+	isNumber: () => isNumber$2,
 	isObject: () => isObject$1,
 	isObjectLike: () => isObjectLike,
 	isPlainObject: () => isPlainObject,
 	isRegExp: () => isRegExp,
 	isSafeInteger: () => isSafeInteger,
 	isSet: () => isSet,
-	isString: () => isString,
+	isString: () => isString$1,
 	isSymbol: () => isSymbol,
 	isTypedArray: () => isTypedArray,
 	isUndefined: () => isUndefined,
@@ -1324,7 +1327,7 @@ const functionsIn = {
 	iteratee: false,
 	args: 1
 };
-const get$5 = {
+const get$4 = {
 	wrapper: false,
 	shorthand: false,
 	chainable: false,
@@ -1633,7 +1636,7 @@ const isNull = {
 	iteratee: false,
 	args: 1
 };
-const isNumber$1 = {
+const isNumber$2 = {
 	wrapper: false,
 	shorthand: false,
 	chainable: false,
@@ -1682,7 +1685,7 @@ const isSet = {
 	iteratee: false,
 	args: 1
 };
-const isString = {
+const isString$1 = {
 	wrapper: false,
 	shorthand: false,
 	chainable: false,
@@ -2889,8 +2892,7 @@ const zipWith = {
 *
 */
 function methodSupportsShorthand(method$1, shorthandType) {
-	const path = `${method$1}.shorthand`;
-	const methodShorthandData = (0, lodash_es.get)(methodData_exports, path);
+	const methodShorthandData = methodData_exports[method$1].shorthand;
 	const isShorthandObject = (0, lodash_es.isObject)(methodShorthandData);
 	return isShorthandObject ? Boolean(shorthandType && methodShorthandData[shorthandType]) : Boolean(methodShorthandData);
 }
@@ -3088,8 +3090,9 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 		let iterateeIndex;
 		if (remedaContext.isRemedaCall(node)) {
 			const method$1 = astUtil_default.getMethodName(node);
+			if (!(0, lodash_es.isString)(method$1)) return;
 			iterateeIndex = getIterateeIndex(method$1);
-			reporter(node, node.arguments[iterateeIndex], {
+			if ((0, lodash_es.isNumber)(iterateeIndex)) reporter(node, node.arguments[iterateeIndex], {
 				callType: "method",
 				method: method$1,
 				remedaContext
@@ -3098,7 +3101,7 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 			const method$1 = remedaContext.getImportedRemedaMethod(node);
 			if (method$1) {
 				iterateeIndex = getIterateeIndex(method$1);
-				reporter(node, node.arguments[iterateeIndex], {
+				if ((0, lodash_es.isNumber)(iterateeIndex)) reporter(node, node.arguments[iterateeIndex], {
 					method: method$1,
 					callType: "single",
 					remedaContext
@@ -3108,10 +3111,11 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 	};
 }
 function isRemedaCallToMethod(node, method$1, remedaContext) {
+	if (!node) return false;
 	return remedaContext.isRemedaCall(node) && isCallToMethod(node, method$1);
 }
 function isCallToRemedaMethod(node, method$1, remedaContext) {
-	if (!node || node.type !== "CallExpression") return false;
+	if (!node || node.type !== __typescript_eslint_utils.AST_NODE_TYPES.CallExpression) return false;
 	return isRemedaCallToMethod(node, method$1, remedaContext) || method$1 === remedaContext.getImportedRemedaMethod(node);
 }
 function getRemedaMethodVisitors(context, remedaCallExpVisitor) {
@@ -3141,7 +3145,7 @@ function isSideEffectIterationMethod(method$1) {
 	return (0, lodash_es.includes)(getSideEffectIterationMethods(), method$1);
 }
 function isParentCommit(node, callType) {
-	return callType === "chained" && isCallToMethod(node.parent.parent, "commit");
+	return callType === "chained" && node.parent.parent && isCallToMethod(node.parent.parent, "commit");
 }
 var collection_method_value_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$14,
@@ -3193,18 +3197,22 @@ var collection_return_default = __typescript_eslint_utils.ESLintUtils.RuleCreato
 	create(context) {
 		const remedaContext = getRemedaContext(context);
 		return {
-			"CallExpression:exit": getRemedaMethodCallExpVisitor(remedaContext, (node, iteratee$1, { method: method$1 }) => {
-				if (!isCollectionMethod(method$1)) return;
-				if (!astUtil_default.isFunctionDefinitionWithBlock(iteratee$1) || iteratee$1.async || iteratee$1.generator) return;
-				const hasReturnStatement = iteratee$1.body.type === __typescript_eslint_utils.AST_NODE_TYPES.BlockStatement && iteratee$1.body.body.some((statement) => {
-					return statement.type === __typescript_eslint_utils.AST_NODE_TYPES.ReturnStatement;
-				});
-				if (!hasReturnStatement) context.report({
-					node,
-					messageId: "no-return",
-					data: { method: method$1 }
-				});
-			}),
+			"CallExpression:exit": getRemedaMethodCallExpVisitor(
+				remedaContext,
+				// @ts-expect-error
+				(node, iteratee$1, { method: method$1 }) => {
+					if (!isCollectionMethod(method$1)) return;
+					if (!astUtil_default.isFunctionDefinitionWithBlock(iteratee$1) || iteratee$1.async || iteratee$1.generator) return;
+					const hasReturnStatement = iteratee$1.body.type === __typescript_eslint_utils.AST_NODE_TYPES.BlockStatement && iteratee$1.body.body.some((statement) => {
+						return statement.type === __typescript_eslint_utils.AST_NODE_TYPES.ReturnStatement;
+					});
+					if (!hasReturnStatement) context.report({
+						node,
+						messageId: "no-return",
+						data: { method: method$1 }
+					});
+				}
+),
 			...remedaContext.getImportVisitors()
 		};
 	}
@@ -3232,8 +3240,14 @@ var prefer_constant_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(
 			description: "enforce using R.constant over functions returning literals",
 			url: getDocsUrl(RULE_NAME$12)
 		},
-		schema: [{ type: "boolean" }, { type: "boolean" }],
-		defaultOptions: [],
+		schema: [{
+			type: "boolean",
+			description: "Whether to check arrow functions for literals in return statements. Default: true."
+		}, {
+			type: "boolean",
+			description: "Whether to check function declarations for literals in return statements. Default: false."
+		}],
+		defaultOptions: [true, false],
 		messages: { [MESSAGE_ID$1]: "Prefer R.constant over a function returning a literal" }
 	},
 	defaultOptions: [true, false],
@@ -3321,7 +3335,7 @@ var prefer_filter_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(ge
 		defaultOptions: [{ maxPropertyPathLength: 3 }],
 		messages: { "prefer-filter": MESSAGE$2 }
 	},
-	defaultOptions: [{}],
+	defaultOptions: [{ maxPropertyPathLength: 3 }],
 	create(context, [options]) {
 		const DEFAULT_MAX_PROPERTY_PATH_LENGTH = 3;
 		const maxLength = options.maxPropertyPathLength ?? DEFAULT_MAX_PROPERTY_PATH_LENGTH;
@@ -3353,7 +3367,7 @@ function isZeroIndexAccess(node) {
 	return node.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && node.property.type === __typescript_eslint_utils.AST_NODE_TYPES.Literal && node.property.value === 0;
 }
 function isChainedBeforeMethod(callType, node, method$1) {
-	return callType === "chained" && isCallToMethod(node.parent?.parent, method$1);
+	return callType === "chained" && node.parent?.parent !== void 0 && isCallToMethod(node.parent.parent, method$1);
 }
 var prefer_find_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$9,
@@ -3373,7 +3387,7 @@ var prefer_find_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getD
 	create(context) {
 		return getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1, callType, remedaContext }) => {
 			if (method$1 === "filter") {
-				if (isZeroIndexAccess(node.parent) || isCallToRemedaMethod(node.parent, "first", remedaContext) || isChainedBeforeMethod(callType, node, "first")) context.report({
+				if (node.parent && isZeroIndexAccess(node.parent) || isCallToRemedaMethod(node.parent, "first", remedaContext) || isChainedBeforeMethod(callType, node, "first")) context.report({
 					node,
 					messageId: "prefer-find"
 				});
@@ -3391,7 +3405,8 @@ var prefer_find_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getD
 const { getCaller } = astUtil_default;
 const RULE_NAME$8 = "prefer-flat-map";
 function isChainedMapFlatten(node) {
-	return isCallToMethod(getCaller(node), "map");
+	const caller = getCaller(node);
+	return caller ? isCallToMethod(caller, "map") : false;
 }
 var prefer_flat_map_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$8,
@@ -3406,12 +3421,16 @@ var prefer_flat_map_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(
 	},
 	defaultOptions: [],
 	create(context) {
-		return getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1, remedaContext }) => {
-			if (method$1 === "flat" && (isChainedMapFlatten(node) || isCallToRemedaMethod(node.arguments[0], "map", remedaContext))) context.report({
-				node,
-				messageId: "prefer-flat-map"
-			});
-		});
+		return getRemedaMethodVisitors(
+			context,
+			// @ts-expect-error
+			(node, iteratee$1, { method: method$1, remedaContext }) => {
+				if (method$1 === "flat" && (isChainedMapFlatten(node) || isCallToRemedaMethod(node.arguments[0], "map", remedaContext))) context.report({
+					node,
+					messageId: "prefer-flat-map"
+				});
+			}
+);
 	}
 });
 
@@ -3647,12 +3666,21 @@ var prefer_is_nullish_default = __typescript_eslint_utils.ESLintUtils.RuleCreato
 			return isNegationExpression(node) && checkExpression(nil, "===", node.argument) || checkExpression(nil, "!==", node);
 		}
 		function isEquivalentExistingExpression(node, leftNil, rightNil) {
+			if (node.type !== __typescript_eslint_utils.AST_NODE_TYPES.LogicalExpression) return false;
 			const leftExp = checkExpression(leftNil, "===", node.left);
-			return leftExp && isEquivalentMemberExp(leftExp, checkExpression(rightNil, "===", node.right));
+			return leftExp && isEquivalentMemberExp(
+				leftExp,
+				// @ts-expect-error
+				checkExpression(rightNil, "===", node.right)
+);
 		}
 		function isEquivalentExistingNegation(node, leftNil, rightNil) {
 			const leftExp = checkNegatedExpression(leftNil, node.left);
-			return leftExp && isEquivalentMemberExp(leftExp, checkNegatedExpression(rightNil, node.right));
+			return leftExp && isEquivalentMemberExp(
+				leftExp,
+				// @ts-expect-error
+				checkNegatedExpression(rightNil, node.right)
+);
 		}
 		const visitors = remedaContext.getImportVisitors();
 		visitors.LogicalExpression = function(node) {
@@ -3674,11 +3702,11 @@ var prefer_is_nullish_default = __typescript_eslint_utils.ESLintUtils.RuleCreato
 //#region src/rules/prefer-map.ts
 const { getFirstFunctionLine, hasOnlyOneStatement, getMethodName, isFunctionDefinitionWithBlock, collectParameterValues } = astUtil_default;
 const RULE_NAME$4 = "prefer-map";
-function onlyHasPush(func) {
-	const firstLine = getFirstFunctionLine(func);
-	const firstParam = (0, lodash_es.get)(func, "params[0]");
-	const exp = func && !isFunctionDefinitionWithBlock(func) ? firstLine : firstLine?.expression;
-	return func && hasOnlyOneStatement(func) && getMethodName(exp) === "push" && !(0, lodash_es.includes)(collectParameterValues(firstParam), (0, lodash_es.get)(exp, "callee.object.name"));
+function onlyHasPush(node) {
+	const firstLine = getFirstFunctionLine(node);
+	const firstParam = (0, lodash_es.get)(node, "params[0]");
+	const exp = node && !isFunctionDefinitionWithBlock(node) ? firstLine : firstLine?.expression;
+	return node && (node.type === __typescript_eslint_utils.AST_NODE_TYPES.ArrowFunctionExpression ? true : hasOnlyOneStatement(node)) && getMethodName(exp) === "push" && !(0, lodash_es.includes)(collectParameterValues(firstParam), (0, lodash_es.get)(exp, "callee.object.name"));
 }
 var prefer_map_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$4,
@@ -3829,7 +3857,7 @@ var prefer_some_default = __typescript_eslint_utils.ESLintUtils.RuleCreator(getD
 	defaultOptions: [],
 	create(context) {
 		const visitors = getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1 }) => {
-			if (method$1 === "findIndex" && node === getExpressionComparedToInt(node.parent, -1, true)) context.report({
+			if (method$1 === "findIndex" && node.parent?.type === __typescript_eslint_utils.AST_NODE_TYPES.BinaryExpression && node === getExpressionComparedToInt(node.parent, -1, true)) context.report({
 				node,
 				messageId: "prefer-some"
 			});
@@ -3904,7 +3932,7 @@ const configs = { recommended: {
 		[`${pluginShortName}/prefer-constant`]: 2,
 		[`${pluginShortName}/prefer-remeda-typecheck`]: 2,
 		[`${pluginShortName}/prefer-nullish-coalescing`]: 2,
-		[`${pluginShortName}/prefer-filter`]: [2, 3],
+		[`${pluginShortName}/prefer-filter`]: 2,
 		[`${pluginShortName}/prefer-has-atleast`]: 2,
 		[`${pluginShortName}/collection-method-value`]: 2,
 		[`${pluginShortName}/collection-return`]: 2,

package/dist/index.d.cts

@@ -60,7 +60,7 @@ declare const configs: {
             };
         };
         rules: {
-            [x: string]: number | number[];
+            [x: string]: number;
         };
     };
 };

package/dist/index.d.ts

@@ -60,7 +60,7 @@ declare const configs: {
             };
         };
         rules: {
-            [x: string]: number | number[];
+            [x: string]: number;
         };
     };
 };

package/dist/index.js

@@ -1,4 +1,4 @@
-import { capitalize, cond, defaults, find, flatMap, get, has, includes, isEmpty, isEqualWith, isMatch, isNumber, isObject, last, map, matches, matchesProperty, overEvery, overSome, property, some } from "lodash-es";
+import { capitalize, cond, defaults, find, flatMap, get, has, includes, isEmpty, isEqualWith, isMatch, isNumber, isObject, isString, last, map, matches, matchesProperty, overEvery, overSome, property, some } from "lodash-es";
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
 
 //#region rolldown:runtime
@@ -21,12 +21,13 @@ var main = "dist/index.js";
 var scripts = {
 	"build": "tsdown",
 	"typecheck": "tsc",
-	"lint": "eslint",
+	"lint": "eslint --max-warnings=0",
 	"knip": "knip",
 	"test:watch": "vitest --ui",
 	"test": "vitest run",
-	"attw": "attw --pack .",
-	"qa": "pnpm typecheck && pnpm test && pnpm knip && attw",
+	"check-sheriff-inconsistencies": "sheriff",
+	"attw": "attw --ignore-rules=false-export-default --pack .",
+	"qa": "pnpm build && pnpm typecheck && pnpm lint && pnpm test && pnpm knip && pnpm attw && pnpm check-sheriff-inconsistencies",
 	"nuke": "rm -rf node_modules pnpm-lock.yaml",
 	"update:eslint-docs": "eslint-doc-generator",
 	"lint:eslint-docs": "pnpm update:eslint-docs -- --check",
@@ -60,7 +61,7 @@ var bugs = "https://github.com/AndreaPontrandolfo/eslint-plugin-remeda/issues";
 var peerDependencies = { "eslint": ">=9.0.0" };
 var devDependencies = {
 	"@arethetypeswrong/cli": "^0.17.4",
-	"@sherifforg/cli": "^8.2.0",
+	"@sherifforg/cli": "^8.3.0",
 	"@types/lodash-es": "^4.17.12",
 	"@types/node": "^22.14.0",
 	"@typescript-eslint/utils": "^8.29.0",
@@ -182,8 +183,8 @@ function isMemberExpOf$1(node, objectName, { maxLength = Number.MAX_VALUE, allow
 	let currentNode = node;
 	let depth = maxLength;
 	while (currentNode && depth) if (allowComputed || isPropAccess(currentNode)) {
-		if (currentNode.type === AST_NODE_TYPES.MemberExpression && "name" in currentNode.object && currentNode.object.name === objectName) return true;
-		currentNode = "object" in currentNode ? currentNode.object : void 0;
+		if (currentNode.type === AST_NODE_TYPES.MemberExpression && currentNode.object.type === AST_NODE_TYPES.Identifier && currentNode.object.name === objectName) return true;
+		currentNode = currentNode.type === AST_NODE_TYPES.MemberExpression ? currentNode.object : void 0;
 		depth = depth - 1;
 	} else return false;
 	return false;
@@ -219,7 +220,7 @@ function hasOnlyOneStatement$2(func) {
 * @param node - The node to check.
 */
 function isObjectOfMethodCall(node) {
-	return get(node, "parent.object") === node && get(node, "parent.parent.type") === "CallExpression";
+	return get(node, "parent.object") === node && get(node, "parent.parent.type") === AST_NODE_TYPES.CallExpression;
 }
 /**
 * Returns whether the node is a literal.
@@ -385,7 +386,9 @@ function getExpressionComparedToInt$1(node, value$1, checkOver) {
 *
 * @param node - The node to check.
 */
-const isIndexOfCall = (node) => isMethodCall$1(node) && getMethodName$2(node) === "indexOf";
+const isIndexOfCall = (node) => {
+	return isMethodCall$1(node) && getMethodName$2(node) === "indexOf";
+};
 /**
 * Returns whether the node is a call to findIndex.
 *
@@ -580,7 +583,7 @@ __export(methodData_exports, {
 	isRegExp: () => isRegExp,
 	isSafeInteger: () => isSafeInteger,
 	isSet: () => isSet,
-	isString: () => isString,
+	isString: () => isString$1,
 	isSymbol: () => isSymbol,
 	isTypedArray: () => isTypedArray,
 	isUndefined: () => isUndefined,
@@ -1662,7 +1665,7 @@ const isSet = {
 	iteratee: false,
 	args: 1
 };
-const isString = {
+const isString$1 = {
 	wrapper: false,
 	shorthand: false,
 	chainable: false,
@@ -2869,8 +2872,7 @@ const zipWith = {
 *
 */
 function methodSupportsShorthand(method$1, shorthandType) {
-	const path = `${method$1}.shorthand`;
-	const methodShorthandData = get(methodData_exports, path);
+	const methodShorthandData = methodData_exports[method$1].shorthand;
 	const isShorthandObject = isObject(methodShorthandData);
 	return isShorthandObject ? Boolean(shorthandType && methodShorthandData[shorthandType]) : Boolean(methodShorthandData);
 }
@@ -3068,8 +3070,9 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 		let iterateeIndex;
 		if (remedaContext.isRemedaCall(node)) {
 			const method$1 = astUtil_default.getMethodName(node);
+			if (!isString(method$1)) return;
 			iterateeIndex = getIterateeIndex(method$1);
-			reporter(node, node.arguments[iterateeIndex], {
+			if (isNumber(iterateeIndex)) reporter(node, node.arguments[iterateeIndex], {
 				callType: "method",
 				method: method$1,
 				remedaContext
@@ -3078,7 +3081,7 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 			const method$1 = remedaContext.getImportedRemedaMethod(node);
 			if (method$1) {
 				iterateeIndex = getIterateeIndex(method$1);
-				reporter(node, node.arguments[iterateeIndex], {
+				if (isNumber(iterateeIndex)) reporter(node, node.arguments[iterateeIndex], {
 					method: method$1,
 					callType: "single",
 					remedaContext
@@ -3088,10 +3091,11 @@ function getRemedaMethodCallExpVisitor(remedaContext, reporter) {
 	};
 }
 function isRemedaCallToMethod(node, method$1, remedaContext) {
+	if (!node) return false;
 	return remedaContext.isRemedaCall(node) && isCallToMethod(node, method$1);
 }
 function isCallToRemedaMethod(node, method$1, remedaContext) {
-	if (!node || node.type !== "CallExpression") return false;
+	if (!node || node.type !== AST_NODE_TYPES.CallExpression) return false;
 	return isRemedaCallToMethod(node, method$1, remedaContext) || method$1 === remedaContext.getImportedRemedaMethod(node);
 }
 function getRemedaMethodVisitors(context, remedaCallExpVisitor) {
@@ -3121,7 +3125,7 @@ function isSideEffectIterationMethod(method$1) {
 	return includes(getSideEffectIterationMethods(), method$1);
 }
 function isParentCommit(node, callType) {
-	return callType === "chained" && isCallToMethod(node.parent.parent, "commit");
+	return callType === "chained" && node.parent.parent && isCallToMethod(node.parent.parent, "commit");
 }
 var collection_method_value_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$14,
@@ -3173,18 +3177,22 @@ var collection_return_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	create(context) {
 		const remedaContext = getRemedaContext(context);
 		return {
-			"CallExpression:exit": getRemedaMethodCallExpVisitor(remedaContext, (node, iteratee$1, { method: method$1 }) => {
-				if (!isCollectionMethod(method$1)) return;
-				if (!astUtil_default.isFunctionDefinitionWithBlock(iteratee$1) || iteratee$1.async || iteratee$1.generator) return;
-				const hasReturnStatement = iteratee$1.body.type === AST_NODE_TYPES.BlockStatement && iteratee$1.body.body.some((statement) => {
-					return statement.type === AST_NODE_TYPES.ReturnStatement;
-				});
-				if (!hasReturnStatement) context.report({
-					node,
-					messageId: "no-return",
-					data: { method: method$1 }
-				});
-			}),
+			"CallExpression:exit": getRemedaMethodCallExpVisitor(
+				remedaContext,
+				// @ts-expect-error
+				(node, iteratee$1, { method: method$1 }) => {
+					if (!isCollectionMethod(method$1)) return;
+					if (!astUtil_default.isFunctionDefinitionWithBlock(iteratee$1) || iteratee$1.async || iteratee$1.generator) return;
+					const hasReturnStatement = iteratee$1.body.type === AST_NODE_TYPES.BlockStatement && iteratee$1.body.body.some((statement) => {
+						return statement.type === AST_NODE_TYPES.ReturnStatement;
+					});
+					if (!hasReturnStatement) context.report({
+						node,
+						messageId: "no-return",
+						data: { method: method$1 }
+					});
+				}
+),
 			...remedaContext.getImportVisitors()
 		};
 	}
@@ -3212,8 +3220,14 @@ var prefer_constant_default = ESLintUtils.RuleCreator(getDocsUrl)({
 			description: "enforce using R.constant over functions returning literals",
 			url: getDocsUrl(RULE_NAME$12)
 		},
-		schema: [{ type: "boolean" }, { type: "boolean" }],
-		defaultOptions: [],
+		schema: [{
+			type: "boolean",
+			description: "Whether to check arrow functions for literals in return statements. Default: true."
+		}, {
+			type: "boolean",
+			description: "Whether to check function declarations for literals in return statements. Default: false."
+		}],
+		defaultOptions: [true, false],
 		messages: { [MESSAGE_ID$1]: "Prefer R.constant over a function returning a literal" }
 	},
 	defaultOptions: [true, false],
@@ -3301,7 +3315,7 @@ var prefer_filter_default = ESLintUtils.RuleCreator(getDocsUrl)({
 		defaultOptions: [{ maxPropertyPathLength: 3 }],
 		messages: { "prefer-filter": MESSAGE$2 }
 	},
-	defaultOptions: [{}],
+	defaultOptions: [{ maxPropertyPathLength: 3 }],
 	create(context, [options]) {
 		const DEFAULT_MAX_PROPERTY_PATH_LENGTH = 3;
 		const maxLength = options.maxPropertyPathLength ?? DEFAULT_MAX_PROPERTY_PATH_LENGTH;
@@ -3333,7 +3347,7 @@ function isZeroIndexAccess(node) {
 	return node.type === AST_NODE_TYPES.MemberExpression && node.property.type === AST_NODE_TYPES.Literal && node.property.value === 0;
 }
 function isChainedBeforeMethod(callType, node, method$1) {
-	return callType === "chained" && isCallToMethod(node.parent?.parent, method$1);
+	return callType === "chained" && node.parent?.parent !== void 0 && isCallToMethod(node.parent.parent, method$1);
 }
 var prefer_find_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$9,
@@ -3353,7 +3367,7 @@ var prefer_find_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	create(context) {
 		return getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1, callType, remedaContext }) => {
 			if (method$1 === "filter") {
-				if (isZeroIndexAccess(node.parent) || isCallToRemedaMethod(node.parent, "first", remedaContext) || isChainedBeforeMethod(callType, node, "first")) context.report({
+				if (node.parent && isZeroIndexAccess(node.parent) || isCallToRemedaMethod(node.parent, "first", remedaContext) || isChainedBeforeMethod(callType, node, "first")) context.report({
 					node,
 					messageId: "prefer-find"
 				});
@@ -3371,7 +3385,8 @@ var prefer_find_default = ESLintUtils.RuleCreator(getDocsUrl)({
 const { getCaller } = astUtil_default;
 const RULE_NAME$8 = "prefer-flat-map";
 function isChainedMapFlatten(node) {
-	return isCallToMethod(getCaller(node), "map");
+	const caller = getCaller(node);
+	return caller ? isCallToMethod(caller, "map") : false;
 }
 var prefer_flat_map_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$8,
@@ -3386,12 +3401,16 @@ var prefer_flat_map_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	},
 	defaultOptions: [],
 	create(context) {
-		return getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1, remedaContext }) => {
-			if (method$1 === "flat" && (isChainedMapFlatten(node) || isCallToRemedaMethod(node.arguments[0], "map", remedaContext))) context.report({
-				node,
-				messageId: "prefer-flat-map"
-			});
-		});
+		return getRemedaMethodVisitors(
+			context,
+			// @ts-expect-error
+			(node, iteratee$1, { method: method$1, remedaContext }) => {
+				if (method$1 === "flat" && (isChainedMapFlatten(node) || isCallToRemedaMethod(node.arguments[0], "map", remedaContext))) context.report({
+					node,
+					messageId: "prefer-flat-map"
+				});
+			}
+);
 	}
 });
 
@@ -3627,12 +3646,21 @@ var prefer_is_nullish_default = ESLintUtils.RuleCreator(getDocsUrl)({
 			return isNegationExpression(node) && checkExpression(nil, "===", node.argument) || checkExpression(nil, "!==", node);
 		}
 		function isEquivalentExistingExpression(node, leftNil, rightNil) {
+			if (node.type !== AST_NODE_TYPES.LogicalExpression) return false;
 			const leftExp = checkExpression(leftNil, "===", node.left);
-			return leftExp && isEquivalentMemberExp(leftExp, checkExpression(rightNil, "===", node.right));
+			return leftExp && isEquivalentMemberExp(
+				leftExp,
+				// @ts-expect-error
+				checkExpression(rightNil, "===", node.right)
+);
 		}
 		function isEquivalentExistingNegation(node, leftNil, rightNil) {
 			const leftExp = checkNegatedExpression(leftNil, node.left);
-			return leftExp && isEquivalentMemberExp(leftExp, checkNegatedExpression(rightNil, node.right));
+			return leftExp && isEquivalentMemberExp(
+				leftExp,
+				// @ts-expect-error
+				checkNegatedExpression(rightNil, node.right)
+);
 		}
 		const visitors = remedaContext.getImportVisitors();
 		visitors.LogicalExpression = function(node) {
@@ -3654,11 +3682,11 @@ var prefer_is_nullish_default = ESLintUtils.RuleCreator(getDocsUrl)({
 //#region src/rules/prefer-map.ts
 const { getFirstFunctionLine, hasOnlyOneStatement, getMethodName, isFunctionDefinitionWithBlock, collectParameterValues } = astUtil_default;
 const RULE_NAME$4 = "prefer-map";
-function onlyHasPush(func) {
-	const firstLine = getFirstFunctionLine(func);
-	const firstParam = get(func, "params[0]");
-	const exp = func && !isFunctionDefinitionWithBlock(func) ? firstLine : firstLine?.expression;
-	return func && hasOnlyOneStatement(func) && getMethodName(exp) === "push" && !includes(collectParameterValues(firstParam), get(exp, "callee.object.name"));
+function onlyHasPush(node) {
+	const firstLine = getFirstFunctionLine(node);
+	const firstParam = get(node, "params[0]");
+	const exp = node && !isFunctionDefinitionWithBlock(node) ? firstLine : firstLine?.expression;
+	return node && (node.type === AST_NODE_TYPES.ArrowFunctionExpression ? true : hasOnlyOneStatement(node)) && getMethodName(exp) === "push" && !includes(collectParameterValues(firstParam), get(exp, "callee.object.name"));
 }
 var prefer_map_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	name: RULE_NAME$4,
@@ -3809,7 +3837,7 @@ var prefer_some_default = ESLintUtils.RuleCreator(getDocsUrl)({
 	defaultOptions: [],
 	create(context) {
 		const visitors = getRemedaMethodVisitors(context, (node, iteratee$1, { method: method$1 }) => {
-			if (method$1 === "findIndex" && node === getExpressionComparedToInt(node.parent, -1, true)) context.report({
+			if (method$1 === "findIndex" && node.parent?.type === AST_NODE_TYPES.BinaryExpression && node === getExpressionComparedToInt(node.parent, -1, true)) context.report({
 				node,
 				messageId: "prefer-some"
 			});
@@ -3884,7 +3912,7 @@ const configs = { recommended: {
 		[`${pluginShortName}/prefer-constant`]: 2,
 		[`${pluginShortName}/prefer-remeda-typecheck`]: 2,
 		[`${pluginShortName}/prefer-nullish-coalescing`]: 2,
-		[`${pluginShortName}/prefer-filter`]: [2, 3],
+		[`${pluginShortName}/prefer-filter`]: 2,
 		[`${pluginShortName}/prefer-has-atleast`]: 2,
 		[`${pluginShortName}/collection-method-value`]: 2,
 		[`${pluginShortName}/collection-return`]: 2,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-remeda",
-  "version": "1.7.0",
+  "version": "1.8.0",
   "author": "Andrea Pontrandolfo <andrea.pontra@gmail.com>",
   "description": "ESLint plugin for Remeda library.",
   "type": "module",
@@ -8,12 +8,13 @@
   "scripts": {
     "build": "tsdown",
     "typecheck": "tsc",
-    "lint": "eslint",
+    "lint": "eslint --max-warnings=0",
     "knip": "knip",
     "test:watch": "vitest --ui",
     "test": "vitest run",
-    "attw": "attw --pack .",
-    "qa": "pnpm typecheck && pnpm test && pnpm knip && attw",
+    "check-sheriff-inconsistencies": "sheriff",
+    "attw": "attw --ignore-rules=false-export-default --pack .",
+    "qa": "pnpm build && pnpm typecheck && pnpm lint && pnpm test && pnpm knip && pnpm attw && pnpm check-sheriff-inconsistencies",
     "nuke": "rm -rf node_modules pnpm-lock.yaml",
     "update:eslint-docs": "eslint-doc-generator",
     "lint:eslint-docs": "pnpm update:eslint-docs -- --check",
@@ -49,7 +50,7 @@
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.17.4",
-    "@sherifforg/cli": "^8.2.0",
+    "@sherifforg/cli": "^8.3.0",
     "@types/lodash-es": "^4.17.12",
     "@types/node": "^22.14.0",
     "@typescript-eslint/utils": "^8.29.0",