Merge upstream/master into main

.github/workflows/ci.yml

@@ -1,11 +1,13 @@
 name: CI
 
+
 on:
+  merge_group:
+  pull_request:
+    types: ["opened", "reopened", "synchronize"]
   push:
     branches:
-      - master
-      - "release/**"
-  pull_request:
+      - main
 
 env:
   RUSTFLAGS: -Dwarnings

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+    "rust-analyzer.cargo.features": "all",
+}

Cargo.toml

@@ -1,11 +1,11 @@
 [package]
-name = "sourcemap"
-version = "9.3.0"
-authors = ["Sentry <hello@sentry.io>"]
+name = "swc_sourcemap"
+version = "9.3.4"
+authors = ["Sentry <hello@sentry.io>", "swc-project <https://swc.rs>"]
 keywords = ["javascript", "sourcemap", "sourcemaps"]
-description = "Basic sourcemap handling for Rust"
-homepage = "https://github.com/getsentry/rust-sourcemap"
-repository = "https://github.com/getsentry/rust-sourcemap"
+description = "Forked from https://github.com/getsentry/rust-sourcemap"
+homepage = "https://github.com/swc-project/swc-sourcemap"
+repository = "https://github.com/swc-project/swc-sourcemap"
 license = "BSD-3-Clause"
 readme = "README.md"
 edition = "2018"
@@ -24,8 +24,8 @@ all-features = true
 
 [dependencies]
 url = "2.1.1"
-serde = { version = "1.0.104", features = ["derive"] }
-serde_json = "1.0.48"
+serde = { version = "1.0.104", features = ["derive", "rc"] }
+serde_json = { version = "1.0.48", features = ["raw_value"] }
 unicode-id-start = "1"
 if_chain = "1.0.0"
 scroll = { version = "0.12.0", features = ["derive"], optional = true }
@@ -34,6 +34,7 @@ debugid = {version = "0.8.0", features = ["serde"] }
 base64-simd = { version = "0.8" }
 bitvec = "1.0.1"
 rustc-hash = "2.1.1"
+bytes-str = { version = "0.2.4", features = ["serde"] }
 
 [features]
 ram_bundle = ["scroll"]

README.md

@@ -28,7 +28,7 @@ memory intensive.
 Usage:
 
 ```rust
-use sourcemap::SourceMap;
+use swc_sourcemap::SourceMap;
 let input: &[_] = b"{
     \"version\":3,
     \"sources\":[\"coolstuff.js\"],

cli/src/main.rs

@@ -3,7 +3,7 @@ use std::path::PathBuf;
 use std::process;
 
 use argh::FromArgs;
-use sourcemap::{DecodedMap, SourceView, Token};
+use swc_sourcemap::{DecodedMap, SourceView, Token};
 
 /// Utility for working with source maps.
 #[derive(FromArgs, Debug)]

examples/read.rs

@@ -2,7 +2,7 @@ use std::env;
 use std::fs;
 use std::io::Read;
 
-use sourcemap::{decode, DecodedMap, RewriteOptions, SourceMap};
+use swc_sourcemap::{decode, DecodedMap, RewriteOptions, SourceMap};
 
 fn load_from_reader<R: Read>(mut rdr: R) -> SourceMap {
     match decode(&mut rdr).unwrap() {

examples/rewrite.rs

@@ -3,7 +3,7 @@ use std::fs;
 use std::io::Read;
 use std::path::Path;
 
-use sourcemap::{decode, DecodedMap, RewriteOptions, SourceMap};
+use swc_sourcemap::{decode, DecodedMap, RewriteOptions, SourceMap};
 
 fn test(sm: &SourceMap) {
     for (src_id, source) in sm.sources().enumerate() {
@@ -14,7 +14,7 @@ fn test(sm: &SourceMap) {
             if f.read_to_string(&mut contents).ok().is_none() {
                 continue;
             }
-            if Some(contents.as_str()) != sm.get_source_contents(src_id as u32) {
+            if Some(contents.as_str()) != sm.get_source_contents(src_id as u32).map(|v| &**v) {
                 println!("  !!! {source}");
             }
         }

examples/split_ram_bundle.rs

@@ -3,8 +3,8 @@ use std::fs;
 use std::fs::File;
 use std::path::Path;
 
-use sourcemap::ram_bundle::{split_ram_bundle, RamBundle, RamBundleType};
-use sourcemap::SourceMapIndex;
+use swc_sourcemap::ram_bundle::{split_ram_bundle, RamBundle, RamBundleType};
+use swc_sourcemap::SourceMapIndex;
 
 const USAGE: &str = "
 Usage:

src/builder.rs

@@ -5,8 +5,8 @@ use std::env;
 use std::fs;
 use std::io::Read;
 use std::path::{Path, PathBuf};
-use std::sync::Arc;
 
+use bytes_str::BytesStr;
 use debugid::DebugId;
 use rustc_hash::FxHashMap;
 use url::Url;
@@ -20,14 +20,14 @@ use crate::types::{RawToken, SourceMap, Token};
 /// objects is generally not very comfortable.  As a general aid this
 /// type can help.
 pub struct SourceMapBuilder {
-    file: Option<Arc<str>>,
-    name_map: FxHashMap<Arc<str>, u32>,
-    names: Vec<Arc<str>>,
+    file: Option<BytesStr>,
+    name_map: FxHashMap<BytesStr, u32>,
+    names: Vec<BytesStr>,
     tokens: Vec<RawToken>,
-    source_map: FxHashMap<Arc<str>, u32>,
-    source_root: Option<Arc<str>>,
-    sources: Vec<Arc<str>>,
-    source_contents: Vec<Option<Arc<str>>>,
+    source_map: FxHashMap<BytesStr, u32>,
+    source_root: Option<BytesStr>,
+    sources: Vec<BytesStr>,
+    source_contents: Vec<Option<BytesStr>>,
     sources_mapping: Vec<u32>,
     ignore_list: BTreeSet<u32>,
     debug_id: Option<DebugId>,
@@ -52,9 +52,9 @@ fn resolve_local_reference(base: &Url, reference: &str) -> Option<PathBuf> {
 
 impl SourceMapBuilder {
     /// Creates a new source map builder and sets the file.
-    pub fn new(file: Option<&str>) -> SourceMapBuilder {
+    pub fn new(file: Option<BytesStr>) -> SourceMapBuilder {
         SourceMapBuilder {
-            file: file.map(Into::into),
+            file,
             name_map: FxHashMap::default(),
             names: vec![],
             tokens: vec![],
@@ -74,7 +74,7 @@ impl SourceMapBuilder {
     }
 
     /// Sets the file for the sourcemap (optional)
-    pub fn set_file<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
+    pub fn set_file<T: Into<BytesStr>>(&mut self, value: Option<T>) {
         self.file = value.map(Into::into);
     }
 
@@ -84,7 +84,7 @@ impl SourceMapBuilder {
     }
 
     /// Sets a new value for the source_root.
-    pub fn set_source_root<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
+    pub fn set_source_root<T: Into<BytesStr>>(&mut self, value: Option<T>) {
         self.source_root = value.map(Into::into);
     }
 
@@ -94,24 +94,24 @@ impl SourceMapBuilder {
     }
 
     /// Registers a new source with the builder and returns the source ID.
-    pub fn add_source(&mut self, src: &str) -> u32 {
+    pub fn add_source(&mut self, src: BytesStr) -> u32 {
         self.add_source_with_id(src, !0)
     }
 
-    fn add_source_with_id(&mut self, src: &str, old_id: u32) -> u32 {
+    fn add_source_with_id(&mut self, src: BytesStr, old_id: u32) -> u32 {
         let count = self.sources.len() as u32;
-        let id = *self.source_map.entry(src.into()).or_insert(count);
+        let id = *self.source_map.entry(src.clone()).or_insert(count);
         if id == count {
-            self.sources.push(src.into());
+            self.sources.push(src);
             self.sources_mapping.push(old_id);
         }
         id
     }
 
     /// Changes the source name for an already set source.
-    pub fn set_source(&mut self, src_id: u32, src: &str) {
+    pub fn set_source(&mut self, src_id: u32, src: BytesStr) {
         assert!(src_id != !0, "Cannot set sources for tombstone source id");
-        self.sources[src_id as usize] = src.into();
+        self.sources[src_id as usize] = src;
     }
 
     /// Looks up a source name for an ID.
@@ -124,12 +124,12 @@ impl SourceMapBuilder {
     }
 
     /// Sets the source contents for an already existing source.
-    pub fn set_source_contents(&mut self, src_id: u32, contents: Option<&str>) {
+    pub fn set_source_contents(&mut self, src_id: u32, contents: Option<BytesStr>) {
         assert!(src_id != !0, "Cannot set sources for tombstone source id");
         if self.sources.len() > self.source_contents.len() {
             self.source_contents.resize(self.sources.len(), None);
         }
-        self.source_contents[src_id as usize] = contents.map(Into::into);
+        self.source_contents[src_id as usize] = contents;
     }
 
     /// Returns the current source contents for a source.
@@ -169,7 +169,7 @@ impl SourceMapBuilder {
             if let Ok(mut f) = fs::File::open(path) {
                 let mut contents = String::new();
                 if f.read_to_string(&mut contents).is_ok() {
-                    self.set_source_contents(src_id, Some(&contents));
+                    self.set_source_contents(src_id, Some(contents.into()));
                 }
             }
         }
@@ -178,11 +178,11 @@ impl SourceMapBuilder {
     }
 
     /// Registers a name with the builder and returns the name ID.
-    pub fn add_name(&mut self, name: &str) -> u32 {
+    pub fn add_name(&mut self, name: BytesStr) -> u32 {
         let count = self.names.len() as u32;
-        let id = *self.name_map.entry(name.into()).or_insert(count);
+        let id = *self.name_map.entry(name.clone()).or_insert(count);
         if id == count {
-            self.names.push(name.into());
+            self.names.push(name);
         }
         id
     }
@@ -195,8 +195,8 @@ impl SourceMapBuilder {
         dst_col: u32,
         src_line: u32,
         src_col: u32,
-        source: Option<&str>,
-        name: Option<&str>,
+        source: Option<BytesStr>,
+        name: Option<BytesStr>,
         is_range: bool,
     ) -> RawToken {
         self.add_with_id(
@@ -211,9 +211,9 @@ impl SourceMapBuilder {
         dst_col: u32,
         src_line: u32,
         src_col: u32,
-        source: Option<&str>,
+        source: Option<BytesStr>,
         source_id: u32,
-        name: Option<&str>,
+        name: Option<BytesStr>,
         is_range: bool,
     ) -> RawToken {
         let src_id = match source {
@@ -273,9 +273,9 @@ impl SourceMapBuilder {
             token.get_dst_col(),
             token.get_src_line(),
             token.get_src_col(),
-            token.get_source(),
+            token.get_source().cloned(),
             token.get_src_id(),
-            name,
+            name.cloned(),
             token.is_range(),
         )
     }
@@ -289,7 +289,7 @@ impl SourceMapBuilder {
                     prefix.push('/');
                 }
                 if source.starts_with(&prefix) {
-                    *source = source[prefix.len()..].into();
+                    source.advance(prefix.len());
                     break;
                 }
             }

src/decoder.rs

@@ -124,7 +124,7 @@ pub fn strip_junk_header(slice: &[u8]) -> io::Result<&[u8]> {
 }
 
 /// Decodes range mappping bitfield string into index
-fn decode_rmi(rmi_str: &str, val: &mut BitVec<u8, Lsb0>) -> Result<()> {
+pub(crate) fn decode_rmi(rmi_str: &str, val: &mut BitVec<u8, Lsb0>) -> Result<()> {
     val.clear();
     val.resize(rmi_str.len() * 6, false);
 
@@ -295,7 +295,7 @@ fn decode_index(rsm: RawSourceMap) -> Result<SourceMapIndex> {
 
     // file sometimes is not a string for unexplicable reasons
     let file = rsm.file.map(|val| match val {
-        Value::String(s) => s,
+        Value::String(s) => s.into(),
         _ => "<invalid>".into(),
     });
 

src/encoder.rs

@@ -20,11 +20,11 @@ pub fn encode<M: Encodable, W: Write>(sm: &M, mut w: W) -> Result<()> {
     Ok(())
 }
 
-fn encode_vlq_diff(out: &mut String, a: u32, b: u32) {
+pub(crate) fn encode_vlq_diff(out: &mut String, a: u32, b: u32) {
     encode_vlq(out, i64::from(a) - i64::from(b))
 }
 
-fn encode_rmi(out: &mut Vec<u8>, data: &[u8]) {
+pub(crate) fn encode_rmi(out: &mut Vec<u8>, data: &[u8]) {
     fn encode_byte(b: u8) -> u8 {
         match b {
             0..=25 => b + b'A',

src/hermes.rs

@@ -1,3 +1,5 @@
+use bytes_str::BytesStr;
+
 use crate::decoder::{decode, decode_regular, decode_slice};
 use crate::encoder::{encode, Encodable};
 use crate::errors::{Error, Result};
@@ -21,7 +23,7 @@ pub struct HermesScopeOffset {
 
 #[derive(Debug, Clone, PartialEq)]
 pub struct HermesFunctionMap {
-    names: Vec<String>,
+    names: Vec<BytesStr>,
     mappings: Vec<HermesScopeOffset>,
 }
 
@@ -93,14 +95,14 @@ impl SourceMapHermes {
 
     /// Given a bytecode offset, this will find the enclosing scopes function
     /// name.
-    pub fn get_original_function_name(&self, bytecode_offset: u32) -> Option<&str> {
+    pub fn get_original_function_name(&self, bytecode_offset: u32) -> Option<&BytesStr> {
         let token = self.sm.lookup_token(0, bytecode_offset)?;
 
         self.get_scope_for_token(token)
     }
 
     /// Resolves the name of the enclosing function for the given [`Token`].
-    pub fn get_scope_for_token(&self, token: Token) -> Option<&str> {
+    pub fn get_scope_for_token(&self, token: Token) -> Option<&BytesStr> {
         let function_map = self
             .function_maps
             .get(token.get_src_id() as usize)?
@@ -115,10 +117,7 @@ impl SourceMapHermes {
             &(token.get_src_line() + 1, token.get_src_col()),
             |o| (o.line, o.column),
         )?;
-        function_map
-            .names
-            .get(mapping.name_index as usize)
-            .map(|n| n.as_str())
+        function_map.names.get(mapping.name_index as usize)
     }
 
     /// This rewrites the sourcemap according to the provided rewrite

src/jsontypes.rs

@@ -1,3 +1,4 @@
+use bytes_str::BytesStr;
 use debugid::DebugId;
 use serde::de::IgnoredAny;
 use serde::{Deserialize, Deserializer, Serialize};
@@ -19,7 +20,7 @@ pub struct RawSection {
 
 #[derive(Serialize, Deserialize, Clone, PartialEq, Debug)]
 pub struct FacebookScopeMapping {
-    pub names: Vec<String>,
+    pub names: Vec<BytesStr>,
     pub mappings: String,
 }