fix(loader): side effect of manifest item (#239)

artusjs · Apr 21, 2023 · 454edf5 · 454edf5
1 parent 40290d8
commit 454edf5
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/loader/factory.ts b/src/loader/factory.ts
@@ -69,13 +69,15 @@ export class LoaderFactory {
 
     // group by loader names
     for (const item of itemList) {
-      item.path = root ? path.join(root, item.path) : item.path;
-      item.loader = item.loader ?? DEFAULT_LOADER;
       if (!itemMap.has(item.loader)) {
         // compatible for custom loader
         itemMap.set(item.loader, []);
       }
-      itemMap.get(item.loader)!.push(item);
+      itemMap.get(item.loader)!.push({
+        ...item,
+        path: root ? path.join(root, item.path) : item.path,
+        loader: item.loader ?? DEFAULT_LOADER,
+      });
     }
 
     // trigger loader