avoid unnecessary array operations

Author: dario-piotrowicz

lib/repl.js

@@ -1643,9 +1643,8 @@ function complete(line, callback) {
   }
 }
 
-function includesProxiesOrGetters(exprSegments, evalFn, context, callback, currentExpr = '') {
-  const currentSegment = exprSegments[0];
-  const remainingSegments = ArrayPrototypeSlice(exprSegments, 1);
+function includesProxiesOrGetters(exprSegments, evalFn, context, callback, currentExpr = '', idx = 0) {
+  const currentSegment = exprSegments[idx];
   currentExpr += `${currentExpr.length === 0 ? '' : '.'}${currentSegment}`;
   evalFn(`try { ${currentExpr} } catch { }`, context, getREPLResourceName(), (_, currentObj) => {
     if (typeof currentObj !== 'object' || currentObj === null) {
@@ -1656,17 +1655,19 @@ function includesProxiesOrGetters(exprSegments, evalFn, context, callback, curre
       return callback(true);
     }
 
-    if (remainingSegments.length === 0) {
+    const nextIdx = idx + 1;
+
+    if (nextIdx >= exprSegments.length) {
       return callback(false);
     }
 
-    const nextSegmentProp = ObjectGetOwnPropertyDescriptor(currentObj, remainingSegments[0]);
+    const nextSegmentProp = ObjectGetOwnPropertyDescriptor(currentObj, exprSegments[nextIdx]);
     const nextSegmentPropHasGetter = typeof nextSegmentProp?.get === 'function';
     if (nextSegmentPropHasGetter) {
       return callback(true);
     }
 
-    return includesProxiesOrGetters(remainingSegments, evalFn, context, callback, currentExpr);
+    return includesProxiesOrGetters(exprSegments, evalFn, context, callback, currentExpr, nextIdx);
   });
 }