tweak a few comments

Author: lukesandberg

turbopack/crates/turbopack-browser/src/ecmascript/content.rs

@@ -149,6 +149,8 @@ impl VersionedContent for EcmascriptBrowserChunkContent {
             rope_builder.concat(code.source_code());
             rope_builder.push_static_bytes(b"\n");
             // Add debug ID comment if enabled
+            // There is no point in attaching the debug ID if there is no source map so we only do
+            // it conditionally.
             if let Some(debug_id) = &*self.code().debug_id().await? {
                 write!(rope_builder, "\n//# debugId={}", debug_id)?;
             }

turbopack/crates/turbopack-css/src/chunk/mod.rs

@@ -74,6 +74,7 @@ impl CssChunk {
             .reference_chunk_source_maps(Vc::upcast(self))
             .await?;
 
+        // CSS chunks never have debug IDs
         let mut code = CodeBuilder::new(source_maps, false);
         let mut body = CodeBuilder::new(source_maps, false);
         let mut external_imports = FxIndexSet::default();

turbopack/crates/turbopack-css/src/chunk/single_item_chunk/chunk.rs

@@ -53,6 +53,7 @@ impl SingleItemCssChunk {
             .chunking_context
             .reference_chunk_source_maps(Vc::upcast(self))
             .await?;
+        // CSS chunks never have debug IDs
         let mut code = CodeBuilder::new(source_maps, false);
 
         if matches!(

turbopack/crates/turbopack-ecmascript/src/minify.rs

@@ -34,6 +34,7 @@ use crate::parse::generate_js_source_map;
 
 #[instrument(level = "info", name = "minify ecmascript code", skip_all)]
 pub fn minify(code: Code, source_maps: bool, mangle: Option<MangleType>) -> Result<Code> {
+    // Pass None for the debug ID so we don't needlessly compute it for the pre-minified content.
     let source_maps = source_maps.then(|| code.generate_source_map_ref(None));
 
     let generate_debug_id = code.generate_debug_id();