Turbopack: refactor how EsmAssetReferences are added (#76011)

A refactor to use less `Box<dyn ModuleReference>`

The snapshots changed because this changes the order of references to the internal parts (they are now in order as opposed to arbitrary order)

The next PR will be the one leverage this to remove `CodeGenerateable` from `EsmAssetReference`

turbopack/crates/turbopack-ecmascript/src/analyzer/imports.rs

@@ -291,11 +291,11 @@ impl ImportMap {
         None
     }
 
-    pub fn references(&self) -> impl Iterator<Item = &ImportMapReference> {
+    pub fn references(&self) -> impl ExactSizeIterator<Item = &ImportMapReference> {
         self.references.iter()
     }
 
-    pub fn reexports(&self) -> impl Iterator<Item = (usize, &Reexport)> {
+    pub fn reexports(&self) -> impl ExactSizeIterator<Item = (usize, &Reexport)> {
         self.reexports.iter().map(|(i, r)| (*i, r))
     }
 

turbopack/crates/turbopack-tests/tests/snapshot/basic-tree-shake/dynamic-import/output/b1abf_turbopack-tests_tests_snapshot_basic-tree-shake_dynamic-import_input_lib_44a9b7.js

@@ -6,7 +6,7 @@ var { g: global, d: __dirname } = __turbopack_context__;
 {
 __turbopack_context__.v((parentImport) => {
     return Promise.all([
-  "output/b1abf_turbopack-tests_tests_snapshot_basic-tree-shake_dynamic-import_input_lib_4f08a3.js"
+  "output/b1abf_turbopack-tests_tests_snapshot_basic-tree-shake_dynamic-import_input_lib_8debd8.js"
 ].map((chunk) => __turbopack_context__.l(chunk))).then(() => {
         return parentImport("[project]/turbopack/crates/turbopack-tests/tests/snapshot/basic-tree-shake/dynamic-import/input/lib.js [test] (ecmascript)");
     });