feat(napi): add tracing-subscriber feature; turned on by default (#546)

closes #545

Author: Boshen

README.md

@@ -224,12 +224,6 @@ OXC_LOG=DEBUG your_program
 RD_LOG='oxc_resolver' rolldown build
 ```
 
-### Rspack
-
-```bash
-RSPACK_PROFILE='TRACE=filter=oxc_resolver=trace&layer=logger' rspack build
-```
-
 ## Test
 
 Tests are ported from

napi/Cargo.toml

@@ -24,7 +24,7 @@ oxc_resolver = { workspace = true }
 
 napi = { version = "3.0.0-beta", default-features = false, features = ["napi3", "serde-json"] }
 napi-derive = { version = "3.0.0-beta" }
-tracing-subscriber = { version = "0.3.18", default-features = false, features = ["std", "fmt"] } # Omit the `regex` feature
+tracing-subscriber = { version = "0.3.18", optional = true, default-features = false, features = ["std", "fmt"] } # Omit the `regex` feature
 
 [target.'cfg(not(any(target_os = "linux", target_os = "freebsd", target_arch = "arm", target_family = "wasm")))'.dependencies]
 mimalloc-safe = { version = "0.1.51", optional = true, features = ["skip_collect_on_exit"] }
@@ -39,5 +39,6 @@ mimalloc-safe = { version = "0.1.51", optional = true, features = ["skip_collect
 napi-build = "2.2.0"
 
 [features]
-default = []
+default = ["tracing-subscriber"]
 allocator = ["dep:mimalloc-safe"]
+tracing-subscriber = ["dep:tracing-subscriber"]

napi/src/lib.rs

@@ -14,12 +14,10 @@ use napi::{Task, bindgen_prelude::AsyncTask};
 use napi_derive::napi;
 use oxc_resolver::{PackageJson, ResolveOptions, Resolver};
 
-use self::{
-    options::{NapiResolveOptions, StrOrStrList},
-    tracing::init_tracing,
-};
+use self::options::{NapiResolveOptions, StrOrStrList};
 
 mod options;
+#[cfg(feature = "tracing-subscriber")]
 mod tracing;
 
 #[napi(object)]
@@ -117,7 +115,10 @@ pub struct ResolverFactory {
 impl ResolverFactory {
     #[napi(constructor)]
     pub fn new(options: Option<NapiResolveOptions>) -> Self {
-        init_tracing();
+        #[cfg(feature = "tracing-subscriber")]
+        {
+            tracing::init_tracing();
+        }
         let options = options.map_or_else(ResolveOptions::default, Self::normalize_options);
         Self { resolver: Arc::new(Resolver::new(options)) }
     }