lib: add primordials.SafeArrayIterator

PR-URL: #36532 Backport-PR-URL: #39446 Reviewed-By: Rich Trott <[email protected]>

Author: aduh95

lib/internal/modules/cjs/loader.js

@@ -438,7 +438,7 @@ function trySelf(parentPath, request){
 constEXPORTS_PATTERN=/^((?:@[^/\\%]+\/)?[^./\\%][^/\\%]*)(\/.*)?$/;
 functionresolveExports(nmPath,request){
 // The implementation's behavior is meant to mirror resolution in ESM.
-const[,name,expansion='']=
+const{1: name,2: expansion=''}=
 StringPrototypeMatch(request,EXPORTS_PATTERN)||[];
 if(!name)
 return;

lib/internal/modules/esm/module_job.js

@@ -12,6 +12,7 @@ const{
  PromisePrototypeCatch,
  ReflectApply,
  RegExpPrototypeTest,
+ SafeArrayIterator,
  SafeSet,
  StringPrototypeIncludes,
  StringPrototypeMatch,
@@ -76,9 +77,9 @@ class ModuleJob{
 });
 
 if(promises!==undefined)
-awaitPromiseAll(promises);
+awaitPromiseAll(newSafeArrayIterator(promises));
 
-returnPromiseAll(dependencyJobs);
+returnPromiseAll(newSafeArrayIterator(dependencyJobs));
 };
 // Promise for the list of all dependencyJobs.
 this.linked=link();
@@ -106,8 +107,8 @@ class ModuleJob{
 }
 jobsInGraph.add(moduleJob);
 constdependencyJobs=awaitmoduleJob.linked;
-returnPromiseAll(
-ArrayPrototypeMap(dependencyJobs,addJobsToDependencyGraph));
+returnPromiseAll(newSafeArrayIterator(
+ArrayPrototypeMap(dependencyJobs,addJobsToDependencyGraph)));
 };
 awaitaddJobsToDependencyGraph(this);
 

lib/internal/per_context/primordials.js

@@ -263,6 +263,9 @@ primordials.SafeWeakSet = makeSafe(
 // Refs: https://tc39.es/ecma262/#sec-%typedarray%-intrinsic-object
 [
 {name: 'TypedArray',original: Reflect.getPrototypeOf(Uint8Array)},
+{name: 'ArrayIterator',original: {
+prototype: Reflect.getPrototypeOf(Array.prototype[Symbol.iterator]()),
+}},
 {name: 'StringIterator',original: {
 prototype: Reflect.getPrototypeOf(String.prototype[Symbol.iterator]()),
 }},
@@ -274,6 +277,10 @@ primordials.SafeWeakSet = makeSafe(
 copyPrototype(original.prototype,primordials,`${name}Prototype`);
 });
 
+primordials.SafeArrayIterator=createSafeIterator(
+primordials.ArrayPrototypeSymbolIterator,
+primordials.ArrayIteratorPrototypeNext
+);
 primordials.SafeStringIterator=createSafeIterator(
 primordials.StringPrototypeSymbolIterator,
 primordials.StringIteratorPrototypeNext

lib/internal/util/debuglog.js

@@ -6,6 +6,7 @@ const{
  ObjectDefineProperty,
  RegExp,
  RegExpPrototypeTest,
+ SafeArrayIterator,
  StringPrototypeToUpperCase
 }=primordials;
 
@@ -78,15 +79,15 @@ function debuglog(set, cb){
 debug=debuglogImpl(enabled,set);
 if(typeofcb==='function')
 cb(debug);
-debug(...args);
+debug(...newSafeArrayIterator(args));
 };
 letenabled;
 lettest=()=>{
 init();
 test=()=>enabled;
 returnenabled;
 };
-constlogger=(...args)=>debug(...args);
+constlogger=(...args)=>debug(...newSafeArrayIterator(args));
 ObjectDefineProperty(logger,'enabled',{
 get(){
 returntest();