Add support for ignoreList property

Sourcemaps can now specify an `ignoreList`, a list of indexes into sources for which those sources should be shown as ignored in dev tools, for example as collapsed stack frames in program stacks.

This adds the list to the SourceMap structure but only includes it in serialized output when it’s non-zero. Open to feedback.

Added a test to exercise this through flattening indexed source maps into a regular map with updated source positions.

src/builder.rs

@@ -8,6 +8,7 @@ use std::sync::Arc;
 
 use debugid::DebugId;
 use rustc_hash::FxHashMap;
+use rustc_hash::FxHashSet;
 use url::Url;
 
 use crate::errors::Result;
@@ -28,6 +29,7 @@ pub struct SourceMapBuilder {
     sources: Vec<Arc<str>>,
     source_contents: Vec<Option<Arc<str>>>,
     sources_mapping: Vec<u32>,
+    ignore_list: FxHashSet<u32>,
     debug_id: Option<DebugId>,
 }
 
@@ -61,6 +63,7 @@ impl SourceMapBuilder {
             sources: vec![],
             source_contents: vec![],
             sources_mapping: vec![],
+            ignore_list: FxHashSet::default(),
             debug_id: None,
         }
     }
@@ -116,6 +119,10 @@ impl SourceMapBuilder {
         self.sources.get(src_id as usize).map(|x| &x[..])
     }
 
+    pub fn add_to_ignore_list(&mut self, src_id: u32) {
+        self.ignore_list.insert(src_id);
+    }
+
     /// Sets the source contents for an already existing source.
     pub fn set_source_contents(&mut self, src_id: u32, contents: Option<&str>) {
         assert!(src_id != !0, "Cannot set sources for tombstone source id");
@@ -301,7 +308,14 @@ impl SourceMapBuilder {
             None
         };
 
-        let mut sm = SourceMap::new(self.file, self.tokens, self.names, self.sources, contents);
+        let mut sm = SourceMap::new(
+            self.file,
+            self.tokens,
+            self.names,
+            self.sources,
+            contents,
+            Some(self.ignore_list.iter().cloned().collect()),
+        );
         sm.set_source_root(self.source_root);
         sm.set_debug_id(self.debug_id);
 

src/decoder.rs

@@ -251,7 +251,14 @@ pub fn decode_regular(rsm: RawSourceMap) -> Result<SourceMap> {
         .sources_content
         .map(|x| x.into_iter().map(|v| v.map(Into::into)).collect::<Vec<_>>());
 
-    let mut sm = SourceMap::new(file, tokens, names, sources, source_content);
+    let mut sm = SourceMap::new(
+        file,
+        tokens,
+        names,
+        sources,
+        source_content,
+        rsm.ignore_list,
+    );
     sm.set_source_root(rsm.source_root);
     sm.set_debug_id(rsm.debug_id);
 

src/encoder.rs

@@ -170,6 +170,11 @@ impl Encodable for SourceMap {
             names: Some(self.names().map(|x| Value::String(x.to_string())).collect()),
             range_mappings: serialize_range_mappings(self),
             mappings: Some(serialize_mappings(self)),
+            ignore_list: if self.ignore_list.is_empty() {
+                None
+            } else {
+                Some(self.ignore_list.iter().cloned().collect())
+            },
             x_facebook_offsets: None,
             x_metro_module_paths: None,
             x_facebook_sources: None,
@@ -203,6 +208,7 @@ impl Encodable for SourceMapIndex {
             names: None,
             range_mappings: None,
             mappings: None,
+            ignore_list: None,
             x_facebook_offsets: None,
             x_metro_module_paths: None,
             x_facebook_sources: None,

src/jsontypes.rs

@@ -46,6 +46,8 @@ pub struct RawSourceMap {
     pub range_mappings: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub mappings: Option<String>,
+    #[serde(rename = "ignoreList", skip_serializing_if = "Option::is_none")]
+    pub ignore_list: Option<Vec<u32>>,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub x_facebook_offsets: Option<Vec<Option<u32>>>,
     #[serde(skip_serializing_if = "Option::is_none")]

src/types.rs

@@ -1,5 +1,6 @@
 use std::borrow::Cow;
 use std::cmp::Ordering;
+use std::collections::hash_set::Iter;
 use std::fmt;
 use std::io::{Read, Write};
 use std::path::Path;
@@ -14,6 +15,7 @@ use crate::sourceview::SourceView;
 use crate::utils::{find_common_prefix, greatest_lower_bound};
 
 use debugid::DebugId;
+use rustc_hash::FxHashSet;
 
 /// Controls the `SourceMap::rewrite` behavior
 ///
@@ -468,6 +470,7 @@ pub struct SourceMap {
     pub(crate) sources: Vec<Arc<str>>,
     pub(crate) sources_prefixed: Option<Vec<Arc<str>>>,
     pub(crate) sources_content: Vec<Option<SourceView>>,
+    pub(crate) ignore_list: FxHashSet<u32>,
     pub(crate) debug_id: Option<DebugId>,
 }
 
@@ -575,12 +578,14 @@ impl SourceMap {
     /// - `names`: a vector of names
     /// - `sources` a vector of source filenames
     /// - `sources_content` optional source contents
+    /// - `ignore_list` optional list of source indexes for devtools to ignore
     pub fn new(
         file: Option<Arc<str>>,
         mut tokens: Vec<RawToken>,
         names: Vec<Arc<str>>,
         sources: Vec<Arc<str>>,
         sources_content: Option<Vec<Option<Arc<str>>>>,
+        ignore_list: Option<Vec<u32>>,
     ) -> SourceMap {
         tokens.sort_unstable_by_key(|t| (t.dst_line, t.dst_col));
         SourceMap {
@@ -595,6 +600,10 @@ impl SourceMap {
                 .into_iter()
                 .map(|opt| opt.map(SourceView::new))
                 .collect(),
+            ignore_list: match ignore_list {
+                Some(ignore_list) => ignore_list.into_iter().collect(),
+                None => FxHashSet::default(),
+            },
             debug_id: None,
         }
     }
@@ -655,6 +664,10 @@ impl SourceMap {
         }
     }
 
+    pub fn ignore_list(&self) -> Iter<'_, u32> {
+        self.ignore_list.iter()
+    }
+
     /// Looks up a token by its index.
     pub fn get_token(&self, idx: usize) -> Option<Token<'_>> {
         self.tokens.get(idx).map(|raw| Token {
@@ -1205,6 +1218,9 @@ impl SourceMapIndex {
                         map.get_source_contents(token.get_src_id()),
                     );
                 }
+                if map.ignore_list.contains(&token.get_src_id()) {
+                    builder.add_to_ignore_list(raw.src_id);
+                }
             }
         }
 

tests/test_index.rs

@@ -158,3 +158,50 @@ fn test_indexed_sourcemap_for_react_native() {
     let ism = SourceMapIndex::from_reader(input).unwrap();
     assert!(ism.is_for_ram_bundle());
 }
+
+#[test]
+fn test_flatten_indexed_sourcemap_with_ignore_list() {
+    let input: &[_] = br#"{
+        "version": 3,
+        "file": "bla",
+        "sections": [
+            {
+                "offset": {
+                    "line": 0,
+                    "column": 0
+                },
+                "map": {
+                    "version":3,
+                    "sources":["file1.js"],
+                    "names":["add","a","b"],
+                    "mappings":"AAAA,QAASA,KAAIC,EAAGC,GACf,YACA,OAAOD,GAAIC",
+                    "file":"file1.min.js"
+                }
+            },
+            {
+                "offset": {
+                    "line": 1,
+                    "column": 0
+                },
+                "map": {
+                    "version":3,
+                    "sources":["file2.js"],
+                    "names":["multiply","a","b","divide","add","c","e","Raven","captureException"],
+                    "mappings":"AAAA,QAASA,UAASC,EAAGC,GACpB,YACA,OAAOD,GAAIC,EAEZ,QAASC,QAAOF,EAAGC,GAClB,YACA,KACC,MAAOF,UAASI,IAAIH,EAAGC,GAAID,EAAGC,GAAKG,EAClC,MAAOC,GACRC,MAAMC,iBAAiBF",
+                    "file":"file2.min.js",
+                    "ignoreList": [0]
+                }
+            }
+        ]
+    }"#;
+
+    let ism = SourceMapIndex::from_reader(input).unwrap();
+    assert_eq!(
+        ism.flatten()
+            .unwrap()
+            .ignore_list()
+            .cloned()
+            .collect::<Vec<u32>>(),
+        vec![1]
+    );
+}