es-check 9.6.1 → 9.6.2

package/lib/constants/es-features/13.js

@@ -48,6 +48,7 @@ const ES13_FEATURES = {
       property: "at",
       excludeCallerTypes: ["ObjectExpression"],
       requireNumericArg: true,
+      requireArrayLikeCall: true,
     },
   },
   ObjectHasOwn: {

package/lib/helpers/ast.js

@@ -32,6 +32,117 @@ function hasNumericArgument(node) {
   return isNumericLiteral || isNegativeNumeric;
 }
 
+const ARRAY_LIKE_NAMES = new Set([
+  "arr",
+  "array",
+  "list",
+  "items",
+  "values",
+  "arguments",
+]);
+
+const ARRAY_CONSTRUCTOR_FUNCTIONS = new Set([
+  "Array",
+  "from",
+  "of",
+  "slice",
+  "filter",
+  "map",
+  "concat",
+]);
+
+const ARRAY_METHODS = new Set([
+  "slice",
+  "filter",
+  "map",
+  "concat",
+  "from",
+  "of",
+]);
+
+const COMMON_NON_ARRAY_NAMES = new Set([
+  "obj",
+  "object",
+  "foo",
+  "bar",
+  "baz",
+  "data",
+  "config",
+  "options",
+  "params",
+  "props",
+  "state",
+  "item",
+  "element",
+  "node",
+  "target",
+  "source",
+  "result",
+  "response",
+  "request",
+  "payload",
+  "model",
+  "view",
+  "controller",
+  "service",
+  "utils",
+  "helpers",
+  "custom",
+]);
+function isArrayExpression(calleeObject) {
+  return calleeObject.type === "ArrayExpression";
+}
+
+function isArrayLikeIdentifier(calleeObject) {
+  const isIdentifier = calleeObject.type === "Identifier";
+  if (!isIdentifier) return false;
+
+  const objectName = calleeObject.name.toLowerCase();
+
+  const isCommonNonArrayName = COMMON_NON_ARRAY_NAMES.has(objectName);
+  if (isCommonNonArrayName) return false;
+
+  return ARRAY_LIKE_NAMES.has(objectName);
+}
+
+function isArrayConstructorCall(calleeObject) {
+  const isCallExpression = calleeObject.type === "CallExpression";
+  if (!isCallExpression) return false;
+
+  const innerCallee = calleeObject.callee;
+  const isIdentifierCallee = innerCallee?.type === "Identifier";
+  const isMemberExpressionCallee = innerCallee?.type === "MemberExpression";
+
+  if (isIdentifierCallee) {
+    const functionName = innerCallee.name;
+    return ARRAY_CONSTRUCTOR_FUNCTIONS.has(functionName);
+  }
+
+  if (isMemberExpressionCallee) {
+    const innerProperty = innerCallee.property;
+    const hasPropertyName = innerProperty?.name;
+    const isArrayMethod = hasPropertyName && ARRAY_METHODS.has(innerProperty.name);
+    return isArrayMethod;
+  }
+
+  return false;
+}
+
+function isArrayLikeCall(node) {
+  const callee = node.callee;
+  const calleeObject = callee?.object;
+  const hasCalleeObject = calleeObject != null;
+
+  if (!hasCalleeObject) return false;
+
+  const isArrayExpr = isArrayExpression(calleeObject);
+  const isArrayIdent = isArrayLikeIdentifier(calleeObject);
+  const isArrayConstr = isArrayConstructorCall(calleeObject);
+  const isArray = isArrayExpr || isArrayIdent || isArrayConstr;
+
+  return isArray;
+}
+
 function checkMap(node, astInfo) {
   const callee = node.callee;
   const calleeObject = callee?.object;
@@ -82,6 +193,9 @@ function checkMap(node, astInfo) {
   const hasNumericArgMismatch =
     astInfo.requireNumericArg && !hasNumericArgument(node);
 
+  const hasArrayLikeCallMismatch =
+    astInfo.requireArrayLikeCall && !isArrayLikeCall(node);
+
   const hasMismatch =
     hasKindMismatch ||
     hasOperatorMismatch ||
@@ -93,7 +207,8 @@ function checkMap(node, astInfo) {
     hasObjectAndPropertyMismatch ||
     hasOptionsCauseMismatch ||
     hasExcludedCallerType ||
-    hasNumericArgMismatch;
+    hasNumericArgMismatch ||
+    hasArrayLikeCallMismatch;
 
   if (hasMismatch) return false;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "es-check",
-  "version": "9.6.1",
+  "version": "9.6.2",
   "description": "Checks the ECMAScript version of .js glob against a specified version of ECMAScript with a shell command",
   "main": "lib/index.js",
   "license": "MIT",