module: refactor NativeModule to avoid unsafe array iteration

PR-URL: #37656
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>

Author: aduh95

lib/internal/bootstrap/loaders.js

@@ -46,6 +46,7 @@
 const {
   ArrayPrototypeMap,
   ArrayPrototypePush,
+  ArrayPrototypeSlice,
   Error,
   ObjectCreate,
   ObjectDefineProperty,
@@ -239,8 +240,10 @@ class NativeModule {
     const { ModuleWrap } = internalBinding('module_wrap');
     const url = `node:${this.id}`;
     const nativeModule = this;
+    const exportsKeys = ArrayPrototypeSlice(this.exportKeys);
+    ArrayPrototypePush(exportsKeys, 'default');
     this.module = new ModuleWrap(
-      url, undefined, [...this.exportKeys, 'default'],
+      url, undefined, exportsKeys,
       function() {
         nativeModule.syncExports();
         this.setExport('default', nativeModule.exports);