[Testing]

Cargo.toml

@@ -403,7 +403,7 @@ owo-colors = "4.2.2"
 parcel_selectors = "0.28.2"
 parking_lot = "0.12.1"
 pathdiff = "0.2.1"
-petgraph = "0.6.3"
+petgraph = "0.8.3"
 pin-project-lite = "0.2.9"
 postcard = "1.0.4"
 proc-macro2 = "1.0.79"

packages/next-swc/package.json

@@ -8,10 +8,10 @@
   "scripts": {
     "clean": "node ../../scripts/rm.mjs native",
     "build-native": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --features plugin,image-extended --js false native",
-    "build-native-release": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --release --features plugin,image-extended,tracing/release_max_level_info --js false native",
-    "build-native-release-with-assertions": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --profile release-with-assertions --features plugin,image-extended,tracing/release_max_level_info --js false native",
+    "build-native-release": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --release --features plugin,image-extended --js false native",
+    "build-native-release-with-assertions": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --profile release-with-assertions --features plugin,image-extended --js false native",
     "build-native-no-plugin": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --features image-webp --js false native",
-    "build-native-no-plugin-release": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --release --features image-webp,tracing/release_max_level_info --js false native",
+    "build-native-no-plugin-release": "napi build --platform -p next-swc-napi --cargo-cwd ../../ --cargo-name next_swc_napi --release --features image-webp --js false native",
     "build-native-wasi": "npx --package=@napi-rs/[email protected] napi build --platform --target wasm32-wasip1-threads -p next-swc-napi --cwd ../../ --output-dir packages/next-swc/native --no-default-features",
     "build-wasm": "wasm-pack build ../../crates/wasm --scope=next",
     "cache-build-native": "[ -d native ] && echo $(ls native)",

turbopack/crates/turbo-tasks-backend/Cargo.toml

@@ -13,7 +13,7 @@ bench = false
 workspace = true
 
 [features]
-default = []
+default = ["trace_task_dirty", "verify_immutable", "verify_serialization"]
 print_cache_item_size = []
 no_fast_stale = []
 verify_serialization = []

turbopack/crates/turbo-tasks-backend/src/backend/mod.rs

@@ -958,12 +958,12 @@ impl<B: BackingStorage> TurboTasksBackendInner<B> {
             .map(|op| op.arc().clone())
             .collect::<Vec<_>>();
         drop(snapshot_request);
+        self.storage.start_snapshot();
         let mut persisted_task_cache_log = self
             .persisted_task_cache_log
             .as_ref()
             .map(|l| l.take(|i| i))
             .unwrap_or_default();
-        self.storage.start_snapshot();
         let mut snapshot_request = self.snapshot_request.lock();
         snapshot_request.snapshot_requested = false;
         self.in_progress_operations

turbopack/crates/turbo-tasks-backend/src/kv_backing_storage.rs

@@ -499,12 +499,20 @@ impl<T: KeyValueDatabase + Send + Sync + 'static> BackingStorageSealed
             tx: &D::ReadTransaction<'_>,
             task_type: &CachedTaskType,
         ) -> Result<Option<TaskId>> {
+            let span = tracing::trace_span!(
+                "forward_lookup_task_cache",
+                task_type = debug(task_type),
+                success = tracing::field::Empty
+            )
+            .entered();
             let task_type = POT_CONFIG.serialize(task_type)?;
             let Some(bytes) = database.get(tx, KeySpace::ForwardTaskCache, &task_type)? else {
+                span.record("success", false);
                 return Ok(None);
             };
             let bytes = bytes.borrow().try_into()?;
             let id = TaskId::try_from(u32::from_le_bytes(bytes)).unwrap();
+            span.record("success", *id);
             Ok(Some(id))
         }
         if inner.database.is_empty() {
@@ -659,9 +667,22 @@ fn serialize_task_type(
         let deserialize: Result<CachedTaskType, _> = serde_path_to_error::deserialize(
             &mut pot_de_symbol_list().deserializer_for_slice(&*task_type_bytes)?,
         );
-        if let Err(err) = deserialize {
-            println!("Task type would not be deserializable {task_id}: {err:?}\n{task_type:#?}");
-            panic!("Task type would not be deserializable {task_id}: {err:?}");
+        match deserialize {
+            Ok(value) => {
+                if value != **task_type {
+                    println!(
+                        "Task type would not round-trip {task_id}:\nOriginal: \
+                         {task_type:#?}\nRound-tripped: {value:#?}"
+                    );
+                    panic!("Task type would not round-trip {task_id}");
+                }
+            }
+            Err(err) => {
+                println!(
+                    "Task type would not be deserializable {task_id}: {err:?}\n{task_type:#?}"
+                );
+                panic!("Task type would not be deserializable {task_id}: {err:?}");
+            }
         }
     }
     Ok(())

turbopack/crates/turbo-tasks-backend/src/utils/chunked_vec.rs

@@ -41,7 +41,7 @@ impl<T> ChunkedVec<T> {
         }
     }
 
-    pub fn iter(&self) -> impl Iterator<Item = &T> {
+    pub fn iter(&self) -> impl ExactSizeIterator<Item = &T> {
         ExactSizeIter {
             iter: self.chunks.iter().flat_map(|chunk| chunk.iter()),
             len: self.len(),
@@ -104,3 +104,28 @@ impl<I: Iterator> ExactSizeIterator for ExactSizeIter<I> {
         self.len
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::ChunkedVec;
+
+    #[test]
+    fn test_chunked_vec() {
+        for i in 0..1000 {
+            let mut vec = ChunkedVec::new();
+            for j in 0..i {
+                vec.push(j);
+            }
+            assert_eq!(vec.len(), i);
+            assert_eq!(
+                vec.iter().copied().collect::<Vec<_>>(),
+                (0..i).collect::<Vec<_>>()
+            );
+            assert_eq!(vec.iter().len(), i);
+            assert_eq!(vec.is_empty(), i == 0);
+            let iter = vec.into_iter();
+            assert_eq!(iter.len(), i);
+            assert_eq!(iter.collect::<Vec<_>>(), (0..i).collect::<Vec<_>>());
+        }
+    }
+}

turbopack/crates/turbopack-core/src/chunk/available_modules.rs

@@ -33,6 +33,12 @@ impl AvailableModules {
         self: ResolvedVc<Self>,
         modules: ResolvedVc<AvailableModulesSet>,
     ) -> Result<Vc<Self>> {
+        let _span = tracing::trace_span!(
+            "AvailabilityInfo::with_modules",
+            this = debug(self),
+            modules = debug(&modules)
+        )
+        .entered();
         Ok(AvailableModules {
             parent: Some(self),
             modules,