chore: update devDeps.; linting

Author: brettz9

badges/coverage-badge.svg

@@ -86,7 +86,7 @@ const updateResults = () => {
             path: jsonpathEl.value,
             json,
             eval: /** @type {'safe'|'native'|boolean} */ (
-                $i('#eval').value === 'false' ? false : $i('#eval').value
+                ($i('#eval').value !== 'false') && $i('#eval').value
             ),
             ignoreEvalErrors: $i('#ignoreEvalErrors').value === 'true'
         });

badges/licenses-badge-dev.svg

@@ -1372,10 +1372,7 @@ const SafeEval = {
    * @returns {UnknownResult}
    */
   evalConditionalExpression(ast, subs) {
-    if (SafeEval.evalAst(ast.test, subs)) {
-      return SafeEval.evalAst(ast.consequent, subs);
-    }
-    return SafeEval.evalAst(ast.alternate, subs);
+    return SafeEval.evalAst(ast.test, subs) ? SafeEval.evalAst(ast.consequent, subs) : SafeEval.evalAst(ast.alternate, subs);
   },
   /**
    * @param {jsep.Identifier} ast
@@ -1810,11 +1807,11 @@ class JSONPathClass {
     this.json = opts.json || obj;
     this.path = opts.path || expr;
     this.resultType = opts.resultType || 'value';
-    this.flatten = Object.hasOwn(opts, 'flatten') ? opts.flatten : false;
-    this.wrap = Object.hasOwn(opts, 'wrap') ? opts.wrap : true;
+    this.flatten = Object.hasOwn(opts, 'flatten') && opts.flatten;
+    this.wrap = !Object.hasOwn(opts, 'wrap') || opts.wrap;
     this.sandbox = opts.sandbox || {};
     this.eval = opts.eval === undefined ? 'safe' : opts.eval;
-    this.ignoreEvalErrors = typeof opts.ignoreEvalErrors === 'undefined' ? false : opts.ignoreEvalErrors;
+    this.ignoreEvalErrors = typeof opts.ignoreEvalErrors !== 'undefined' && opts.ignoreEvalErrors;
     this.parent = Object.hasOwn(opts, 'parent') ? opts.parent : null;
     this.parentProperty = Object.hasOwn(opts, 'parentProperty') ? opts.parentProperty : null;
     this.callback = opts.callback || (/** @type {JSONPathCallback} */
@@ -1823,28 +1820,29 @@ class JSONPathClass {
       throw new TypeError('You must supply an otherTypeCallback callback option ' + 'with the @other() operator.');
     };
     this.customTypes = opts.customTypes || {};
-    if (opts.autostart !== false) {
-      const args = /** @type {JSONPathOptions} */{
-        path: optObj ? opts.path : expr
-      };
-      if (!optObj && obj !== undefined) {
-        args.json = obj;
-      } else if ('json' in opts) {
-        args.json = opts.json;
-      }
-      const ret = this.evaluate(args);
-      if (!ret || typeof ret !== 'object') {
-        const err = /** @type {Error & {value: UnknownResult}} */
-        new Error('JSONPath should not be called with "new" (it ' + 'prevents return of (unwrapped) scalar values)');
-        err.value = ret;
-        throw err;
-      }
-
-      // eslint-disable-next-line @stylistic/max-len -- Long
-      // @ts-expect-error - Constructor returns evaluate result for legacy API
-      // eslint-disable-next-line no-constructor-return -- Legacy API
-      return ret;
+    if (opts.autostart === false) {
+      return;
+    }
+    const args = /** @type {JSONPathOptions} */{
+      path: optObj ? opts.path : expr
+    };
+    if (!optObj && obj !== undefined) {
+      args.json = obj;
+    } else if ('json' in opts) {
+      args.json = opts.json;
     }
+    const ret = this.evaluate(args);
+    if (!ret || typeof ret !== 'object') {
+      const err = /** @type {Error & {value: UnknownResult}} */
+      new Error('JSONPath should not be called with "new" (it ' + 'prevents return of (unwrapped) scalar values)');
+      err.value = ret;
+      throw err;
+    }
+
+    // @ts-expect-error - Constructor returns evaluate result for legacy API
+    // eslint-disable-next-line @stylistic/max-len -- Long
+    // eslint-disable-next-line consistent-return, no-constructor-return -- Legacy API
+    return ret;
   }
 
   // PUBLIC METHODS
@@ -2211,7 +2209,6 @@ class JSONPathClass {
             addType = true;
           }
           break;
-        /* c8 ignore next 2 */
         default:
           if (this.currCustomTypes && Object.hasOwn(this.currCustomTypes, valueType)) {
             addType = this.currCustomTypes[valueType](val, path, parent, parentPropName) || false;
@@ -2252,22 +2249,23 @@ class JSONPathClass {
     if (this._hasParentSelector) {
       for (let t = 0; t < ret.length; t++) {
         const rett = ret[t];
-        if (rett && rett.isParentSelector) {
-          const exprToUse = /** @type {ExpressionArray} */
-          rett.expr;
-          const pathToUse = /** @type {ExpressionArray} */
-          rett.path;
-          const tmp = this._trace(exprToUse, val, pathToUse, parent, parentPropName, callback, hasArrExpr);
-          if (Array.isArray(tmp)) {
-            ret[t] = tmp[0];
-            const tl = tmp.length;
-            for (let tt = 1; tt < tl; tt++) {
-              t++;
-              ret.splice(t, 0, tmp[tt]);
-            }
-          } else {
-            ret[t] = tmp;
+        if (!rett || !rett.isParentSelector) {
+          continue;
+        }
+        const exprToUse = /** @type {ExpressionArray} */
+        rett.expr;
+        const pathToUse = /** @type {ExpressionArray} */
+        rett.path;
+        const tmp = this._trace(exprToUse, val, pathToUse, parent, parentPropName, callback, hasArrExpr);
+        if (Array.isArray(tmp)) {
+          ret[t] = tmp[0];
+          const tl = tmp.length;
+          for (let tt = 1; tt < tl; tt++) {
+            t++;
+            ret.splice(t, 0, tmp[tt]);
           }
+        } else {
+          ret[t] = tmp;
         }
       }
     }