fix: should ignores names without columns (#117)

Author: SyMind

src/encoder.rs

@@ -70,7 +70,7 @@ impl MappingsEncoder for FullMappingsEncoder {
   fn encode(&mut self, mapping: &Mapping) {
     if self.active_mapping && self.current_line == mapping.generated_line {
       // A mapping is still active
-      if mapping.original.is_some_and(|original| {
+      if mapping.original.as_ref().is_some_and(|original| {
         original.source_index == self.current_source_index
           && original.original_line == self.current_original_line
           && original.original_column == self.current_original_column

src/helpers.rs

@@ -419,7 +419,7 @@ fn stream_chunks_of_source_map_full<'a>(
         tracking_generated_index = current_generated_index;
         tracking_generated_line = mapping.generated_line;
         tracking_generated_column = mapping.generated_column;
-        tracking_mapping_original = mapping.original;
+        tracking_mapping_original = mapping.original.clone();
 
         current_mapping = mappings_iter.next();
       }
@@ -497,7 +497,7 @@ fn stream_chunks_of_source_map_lines_final<'a>(
   let mut current_generated_line = 1;
 
   let mut on_mapping = |mut mapping: Mapping| {
-    if let Some(mut original) = mapping.original.filter(|_| {
+    if let Some(original) = mapping.original.as_mut().filter(|_| {
       current_generated_line <= mapping.generated_line
         && mapping.generated_line <= final_line
     }) {
@@ -556,8 +556,9 @@ fn stream_chunks_of_source_map_lines_full<'a>(
       }
       current_generated_line += 1;
     }
-    if let Some(mut original) = mapping
+    if let Some(original) = mapping
       .original
+      .as_mut()
       .filter(|_| mapping.generated_line as usize <= lines.len())
     {
       let chunk = lines[current_generated_line as usize - 1];

src/source.rs

@@ -480,7 +480,7 @@ pub struct Mapping {
 }
 
 /// Represent original position information of a [Mapping].
-#[derive(Debug, Clone, PartialEq, Eq, Hash, Copy)]
+#[derive(Debug, Clone, PartialEq, Eq, Hash)]
 pub struct OriginalLocation {
   /// Source index.
   pub source_index: u32,

src/source_map_source.rs

@@ -649,4 +649,48 @@ mod tests {
       .unwrap(),
     );
   }
+
+  #[test]
+  fn should_ignores_names_without_columns() {
+    let source = SourceMapSource::new(SourceMapSourceOptions {
+      value: "h",
+      name: "hello.txt",
+      source_map: SourceMap::from_json(
+        r#"{
+          "version": 3,
+          "sources": ["hello.txt"],
+          "mappings": "AAAAA",
+          "names": ["hello"]
+        }"#,
+      )
+      .unwrap(),
+      original_source: Some("hello".to_string()),
+      inner_source_map: Some(
+        SourceMap::from_json(
+          r#"{
+          "version": 3,
+          "sources": ["hello world.txt"],
+          "mappings": "AAAA",
+          "names": [],
+          "sourcesContent": ["hello, world!"]
+        }"#,
+        )
+        .unwrap(),
+      ),
+      remove_original_source: false,
+    });
+    assert_eq!(
+      source.map(&MapOptions::new(false)).unwrap(),
+      SourceMap::from_json(
+        r#"{
+          "mappings": "AAAA",
+          "names": [],
+          "sources": ["hello world.txt"],
+          "version": 3,
+          "sourcesContent": ["hello, world!"]
+        }"#
+      )
+      .unwrap()
+    );
+  }
 }