Revert "WIP DEBUG"

This reverts commit f34b6e0cd49fbc4653694f24c29d5bf1676d5bba.

Author: mischnic

turbopack/crates/turbopack-core/src/module_graph/mod.rs

@@ -364,7 +364,7 @@ impl SingleModuleGraph {
                     }
                 }
                 if !duplicates.is_empty() {
-                    println!("Duplicate module idents in graph: {duplicates:#?}");
+                    panic!("Duplicate module idents in graph: {duplicates:#?}");
                 }
             }
         }

turbopack/crates/turbopack-css/src/asset.rs

@@ -1,6 +1,6 @@
 use anyhow::Result;
 use turbo_rcstr::rcstr;
-use turbo_tasks::{IntoTraitRef, ResolvedVc, TryJoinIterExt, ValueToString, Vc, task::TaskOutput};
+use turbo_tasks::{IntoTraitRef, ResolvedVc, TryJoinIterExt, ValueToString, Vc};
 use turbo_tasks_fs::FileSystemPath;
 use turbopack_core::{
     asset::{Asset, AssetContent},
@@ -46,26 +46,20 @@ pub struct CssModuleAsset {
 impl CssModuleAsset {
     /// Creates a new CSS asset.
     #[turbo_tasks::function]
-    pub async fn new(
+    pub fn new(
         source: ResolvedVc<Box<dyn Source>>,
         asset_context: ResolvedVc<Box<dyn AssetContext>>,
         ty: CssModuleAssetType,
         import_context: Option<ResolvedVc<ImportContext>>,
         environment: Option<ResolvedVc<Environment>>,
-    ) -> Result<Vc<Self>> {
-        println!(
-            "Css {:?} {:?} {:?}",
-            source.ident().to_string().await?,
-            source.try_into_raw_vc()?,
-            asset_context.into_trait_ref().await?.layer().name()
-        );
-        Ok(Self::cell(CssModuleAsset {
+    ) -> Vc<Self> {
+        Self::cell(CssModuleAsset {
             source,
             asset_context,
             import_context,
             ty,
             environment,
-        }))
+        })
     }
 
     /// Returns the asset ident of the source without the "css" modifier

turbopack/crates/turbopack-css/src/module_asset.rs

@@ -5,7 +5,7 @@ use indoc::formatdoc;
 use lightningcss::css_modules::CssModuleReference;
 use swc_core::common::{BytePos, FileName, LineCol, SourceMap};
 use turbo_rcstr::{RcStr, rcstr};
-use turbo_tasks::{FxIndexMap, IntoTraitRef, ResolvedVc, ValueToString, Vc, task::TaskOutput};
+use turbo_tasks::{FxIndexMap, IntoTraitRef, ResolvedVc, ValueToString, Vc};
 use turbo_tasks_fs::{FileSystemPath, rope::Rope};
 use turbopack_core::{
     asset::{Asset, AssetContent},
@@ -49,20 +49,14 @@ pub struct ModuleCssAsset {
 #[turbo_tasks::value_impl]
 impl ModuleCssAsset {
     #[turbo_tasks::function]
-    pub async fn new(
+    pub fn new(
         source: ResolvedVc<Box<dyn Source>>,
         asset_context: ResolvedVc<Box<dyn AssetContext>>,
-    ) -> Result<Vc<Self>> {
-        println!(
-            "ModuleCss {:?} {:?} {:?}",
-            source.ident().to_string().await?,
-            source.try_into_raw_vc()?,
-            asset_context.into_trait_ref().await?.layer().name()
-        );
-        Ok(Self::cell(ModuleCssAsset {
+    ) -> Vc<Self> {
+        Self::cell(ModuleCssAsset {
             source,
             asset_context,
-        }))
+        })
     }
 }
 

turbopack/crates/turbopack-node/src/transforms/postcss.rs

@@ -3,8 +3,8 @@ use indoc::formatdoc;
 use serde::{Deserialize, Serialize};
 use turbo_rcstr::{RcStr, rcstr};
 use turbo_tasks::{
-    Completion, Completions, NonLocalValue, ResolvedVc, TaskInput, TryFlatJoinIterExt,
-    ValueToString, Vc, fxindexmap, trace::TraceRawVcs,
+    Completion, Completions, NonLocalValue, ResolvedVc, TaskInput, TryFlatJoinIterExt, Vc,
+    fxindexmap, trace::TraceRawVcs,
 };
 use turbo_tasks_bytes::stream::SingleValue;
 use turbo_tasks_fs::{
@@ -131,8 +131,8 @@ impl PostCssTransform {
 #[turbo_tasks::value_impl]
 impl SourceTransform for PostCssTransform {
     #[turbo_tasks::function]
-    async fn transform(&self, source: ResolvedVc<Box<dyn Source>>) -> Result<Vc<Box<dyn Source>>> {
-        let x = Vc::upcast(
+    fn transform(&self, source: ResolvedVc<Box<dyn Source>>) -> Vc<Box<dyn Source>> {
+        Vc::upcast(
             PostCssTransformedAsset {
                 evaluate_context: self.evaluate_context,
                 execution_context: self.execution_context,
@@ -141,14 +141,7 @@ impl SourceTransform for PostCssTransform {
                 source_map: self.source_maps,
             }
             .cell(),
-        );
-        println!(
-            "PostCssTransformedAsset {:?} {:?}",
-            source,
-            // source.ident().to_string().await?,
-            x
-        );
-        Ok(x)
+        )
     }
 }
 

turbopack/crates/turbopack-node/src/transforms/webpack.rs

@@ -128,25 +128,17 @@ impl WebpackLoaders {
 #[turbo_tasks::value_impl]
 impl SourceTransform for WebpackLoaders {
     #[turbo_tasks::function]
-    async fn transform(
+    fn transform(
         self: ResolvedVc<Self>,
         source: ResolvedVc<Box<dyn Source>>,
-    ) -> Result<Vc<Box<dyn Source>>> {
-        let x = Vc::upcast(
+    ) -> Vc<Box<dyn Source>> {
+        Vc::upcast(
             WebpackLoadersProcessedAsset {
                 transform: self,
                 source,
             }
             .cell(),
-        );
-        println!(
-            "WebpackLoadersProcessedAsset {:?} {:?} {:?}",
-            source,
-            // source.ident().to_string().await?,
-            self.await?.resolve_options_context.await?.browser,
-            x
-        );
-        Ok(x)
+        )
     }
 }
 

turbopack/crates/turbopack/src/transition/full_context_transition.rs

@@ -1,8 +1,4 @@
-use anyhow::Result;
-use turbo_tasks::{ResolvedVc, ValueToString, Vc};
-use turbopack_core::{
-    context::ProcessResult, module::Module, reference_type::ReferenceType, source::Source,
-};
+use turbo_tasks::{ResolvedVc, Vc};
 
 use crate::{ModuleAssetContext, transition::Transition};
 
@@ -29,37 +25,4 @@ impl Transition for FullContextTransition {
     ) -> Vc<ModuleAssetContext> {
         *self.module_context
     }
-
-    #[turbo_tasks::function]
-    async fn process(
-        self: Vc<Self>,
-        asset: Vc<Box<dyn Source>>,
-        module_asset_context: Vc<ModuleAssetContext>,
-        reference_type: ReferenceType,
-    ) -> Result<Vc<ProcessResult>> {
-        let asset = self.process_source(asset);
-        let module_asset_context = self.process_context(module_asset_context);
-        let asset = asset.to_resolved().await?;
-
-        Ok(match &*module_asset_context
-            .process_default(asset, reference_type)
-            .await?
-            .await?
-        {
-            ProcessResult::Module(m) => {
-                let x = self
-                    .process_module(**m, module_asset_context)
-                    .to_resolved()
-                    .await?;
-                let ident = x.ident().to_string().await?;
-                if ident.contains("styles.module") {
-                    println!("FullContextTransition: {:?} {:?} {:?}", asset, x, ident);
-                }
-                ProcessResult::Module(x)
-            }
-            ProcessResult::Unknown(source) => ProcessResult::Unknown(*source),
-            ProcessResult::Ignore => ProcessResult::Ignore,
-        }
-        .cell())
-    }
 }