chore: enable eslint recommended config (#1659)

Enables the recommended lint rules from `eslint`.

Also deletes `getEnumerableProperties.js` which is no longer referenced.

eslint.config.js

@@ -1,12 +1,26 @@
 import jsdoc from "eslint-plugin-jsdoc";
+import eslintjs from "@eslint/js";
+
+const {configs: eslintConfigs} = eslintjs;
 
 export default [
   jsdoc.configs["flat/recommended"],
+  eslintConfigs["recommended"],
   {
+    languageOptions: {
+      // if we ever use more globals than this, pull in the `globals` package
+      globals: {
+        console: false
+      }
+    },
     rules: {
       "jsdoc/require-param-description": "off",
       "jsdoc/require-returns-description": "off",
       "jsdoc/tag-lines": ["error", "any", { startLines: 1 }],
+      "no-unused-vars": ["error", {
+        argsIgnorePattern: "^_",
+        caughtErrorsIgnorePattern: "^_"
+      }]
     },
   },
 ];

lib/chai/core/assertions.js

@@ -2024,11 +2024,11 @@ Assertion.addMethod('property', assertProperty);
 
 /**
  *
- * @param {unknown} name
- * @param {unknown} value
- * @param {string} msg
+ * @param {unknown} _name
+ * @param {unknown} _value
+ * @param {string} _msg
  */
-function assertOwnProperty(name, value, msg) {
+function assertOwnProperty(_name, _value, _msg) {
   flag(this, 'own', true);
   assertProperty.apply(this, arguments);
 }
@@ -4052,7 +4052,7 @@ Assertion.addProperty('frozen', function () {
  * @namespace BDD
  * @public
  */
-Assertion.addProperty('finite', function (msg) {
+Assertion.addProperty('finite', function (_msg) {
   var obj = flag(this, 'object');
 
   this.assert(

lib/chai/interface/assert.js

@@ -3164,16 +3164,22 @@ assert.isNotEmpty = function (val, msg) {
  * @param {unknown} as
  * @returns {unknown}
  */
-(function alias(name, as) {
+const aliases = [
+  ['isOk', 'ok'],
+  ['isNotOk', 'notOk'],
+  ['throws', 'throw'],
+  ['throws', 'Throw'],
+  ['isExtensible', 'extensible'],
+  ['isNotExtensible', 'notExtensible'],
+  ['isSealed', 'sealed'],
+  ['isNotSealed', 'notSealed'],
+  ['isFrozen', 'frozen'],
+  ['isNotFrozen', 'notFrozen'],
+  ['isEmpty', 'empty'],
+  ['isNotEmpty', 'notEmpty'],
+  ['isCallable', 'isFunction'],
+  ['isNotCallable', 'isNotFunction']
+];
+for (const [name, as] of aliases) {
   assert[as] = assert[name];
-  return alias;
-})('isOk', 'ok')('isNotOk', 'notOk')('throws', 'throw')('throws', 'Throw')(
-  'isExtensible',
-  'extensible'
-)('isNotExtensible', 'notExtensible')('isSealed', 'sealed')(
-  'isNotSealed',
-  'notSealed'
-)('isFrozen', 'frozen')('isNotFrozen', 'notFrozen')('isEmpty', 'empty')(
-  'isNotEmpty',
-  'notEmpty'
-)('isCallable', 'isFunction')('isNotCallable', 'isNotFunction');
+}

lib/chai/utils/index.js

@@ -107,6 +107,12 @@ export function isRegExp(obj) {
   return Object.prototype.toString.call(obj) === '[object RegExp]';
 }
 
+/**
+ * Determines if an object is numeric or not
+ *
+ * @param {unknown} obj Object to test
+ * @returns {boolean}
+ */
 export function isNumeric(obj) {
   return ['Number', 'BigInt'].includes(type(obj));
 }

lib/chai/utils/proxify.js

@@ -64,6 +64,8 @@ export function proxify(obj, nonChainableMethodName) {
         var suggestionDistance = 4;
         getProperties(target).forEach(function (prop) {
           if (
+            // we actually mean to check `Object.prototype` here
+            // eslint-disable-next-line no-prototype-builtins
             !Object.prototype.hasOwnProperty(prop) &&
             builtins.indexOf(prop) === -1
           ) {
@@ -128,17 +130,17 @@ function stringDistanceCapped(strA, strB, cap) {
   // `memo` is a two-dimensional array containing distances.
   // memo[i][j] is the distance between strA.slice(0, i) and
   // strB.slice(0, j).
-  for (var i = 0; i <= strA.length; i++) {
+  for (let i = 0; i <= strA.length; i++) {
     memo[i] = Array(strB.length + 1).fill(0);
     memo[i][0] = i;
   }
-  for (var j = 0; j < strB.length; j++) {
+  for (let j = 0; j < strB.length; j++) {
     memo[0][j] = j;
   }
 
-  for (var i = 1; i <= strA.length; i++) {
+  for (let i = 1; i <= strA.length; i++) {
     var ch = strA.charCodeAt(i - 1);
-    for (var j = 1; j <= strB.length; j++) {
+    for (let j = 1; j <= strB.length; j++) {
       if (Math.abs(i - j) >= cap) {
         memo[i][j] = cap;
         continue;

package.json

@@ -52,6 +52,7 @@
     "pathval": "^2.0.0"
   },
   "devDependencies": {
+    "@eslint/js": "^9.17.0",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@web/dev-server-rollup": "^0.6.1",
     "@web/test-runner": "^0.18.0",