fix(ext/event_worker): don't use tracing as the log backend if cli/tracing feature flag is not enabled (#469)

* fix(ext/event_worker): add a feature flag

* chore(cli): update `Cargo.toml`

* chore: update `Cargo.lock`

Author: nyannyacha

Cargo.lock

@@ -15,6 +15,8 @@ base.workspace = true
 deno_manifest.workspace = true
 graph.workspace = true
 
+sb_event_worker = { workspace = true, optional = true, features = ["tracing"] }
+
 anyhow.workspace = true
 log.workspace = true
 tokio.workspace = true
@@ -26,4 +28,4 @@ tracing-subscriber = { workspace = true, optional = true }
 env_logger = "0.10.0"
 
 [features]
-tracing = ["dep:tracing-subscriber"]
+tracing = ["dep:tracing-subscriber", "dep:sb_event_worker"]

cli/Cargo.toml

@@ -19,4 +19,7 @@ anyhow.workspace = true
 tokio.workspace = true
 log.workspace = true
 tracing.workspace = true
-enum-as-inner.workspace = true
+enum-as-inner.workspace = true
+
+[features]
+tracing = []

ext/event_worker/Cargo.toml

@@ -4,7 +4,7 @@ use deno_core::error::AnyError;
 use deno_core::op2;
 use deno_core::OpState;
 use tokio::sync::mpsc;
-use tracing::{event, trace};
+use tracing::trace;
 
 #[op2(fast)]
 fn op_user_worker_log(
@@ -35,11 +35,19 @@ fn op_user_worker_log(
         trace!(?metadata);
         tx.send(metadata)?;
     } else {
-        match level {
-            LogLevel::Debug => event!(tracing::Level::DEBUG, "{msg}"),
-            LogLevel::Info => event!(tracing::Level::INFO, "{msg}"),
-            LogLevel::Warning => event!(tracing::Level::WARN, "{msg}"),
-            LogLevel::Error => event!(tracing::Level::ERROR, "{msg}"),
+        #[cfg(feature = "tracing")]
+        {
+            match level {
+                LogLevel::Debug => tracing::event!(tracing::Level::DEBUG, "{msg}"),
+                LogLevel::Info => tracing::event!(tracing::Level::INFO, "{msg}"),
+                LogLevel::Warning => tracing::event!(tracing::Level::WARN, "{msg}"),
+                LogLevel::Error => tracing::event!(tracing::Level::ERROR, "{msg}"),
+            }
+        }
+
+        #[cfg(not(feature = "tracing"))]
+        {
+            log::error!("[{:?}] {}", level, msg.to_string());
         }
     }