feat: make error format configurable

swc-project · Aug 21, 2024 · ed33d9d · ed33d9d
1 parent b0b5e36
commit ed33d9d
Showing 1 changed file with 11 additions and 12 deletions.
diff --git a/bindings/binding_core_node/src/minify.rs b/bindings/binding_core_node/src/minify.rs
@@ -1,8 +1,9 @@
 use std::sync::Arc;
 
 use napi::{
-    bindgen_prelude::{AbortSignal, AsyncTask, Buffer},
-    Task,
+    bindgen_prelude::{AbortSignal, AsyncTask, Buffer, Env, FromNapiValue},
+    sys::{napi_env, napi_value},
+    JsUnknown, NapiValue, Task,
 };
 use serde::Deserialize;
 use swc_core::{
@@ -39,7 +40,7 @@ impl MinifyTarget {
                 assert_eq!(
                     codes.len(),
                     1,
-                    "swc.minify does not support concatting multiple files yet"
+                    "swc.minify does not support concatenating multiple files yet"
                 );
 
                 let (filename, code) = codes.iter().next().unwrap();
@@ -67,7 +68,7 @@ impl Task for MinifyTask {
         .convert_err()
     }
 
-    fn resolve(&mut self, _env: napi::Env, output: Self::Output) -> napi::Result<Self::JsValue> {
+    fn resolve(&mut self, _env: Env, output: Self::Output) -> napi::Result<Self::JsValue> {
         Ok(output)
     }
 }
@@ -77,6 +78,7 @@ fn minify(code: Buffer, opts: Buffer, signal: Option<AbortSignal>) -> AsyncTask<
     crate::util::init_default_trace_subscriber();
     let code = String::from_utf8_lossy(code.as_ref()).to_string();
     let options = String::from_utf8_lossy(opts.as_ref()).to_string();
+    let options: JsMinifyOptions = deserialize_json(&opts_str).expect("Invalid options format");
 
     let c = get_compiler();
 
@@ -89,18 +91,15 @@ fn minify(code: Buffer, opts: Buffer, signal: Option<AbortSignal>) -> AsyncTask<
 pub fn minify_sync(code: Buffer, opts: Buffer) -> napi::Result<TransformOutput> {
     crate::util::init_default_trace_subscriber();
     let code: MinifyTarget = get_deserialized(code)?;
-    let opts = get_deserialized(opts)?;
+    let opts_str = String::from_utf8_lossy(opts.as_ref()).to_string();
+    let opts: JsMinifyOptions = deserialize_json(&opts_str)?;
 
     let c = get_compiler();
 
     let fm = code.to_file(c.cm.clone());
 
-    try_with(
-        c.cm.clone(),
-        false,
-        // TODO(kdy1): Maybe make this configurable?
-        ErrorFormat::Normal,
-        |handler| c.minify(fm, handler, &opts),
-    )
+    try_with(c.cm.clone(), false, ErrorFormat::Normal, |handler| {
+        c.minify(fm, handler, &opts)
+    })
     .convert_err()
 }