From 005e8a55129b33f91b540f0e656d33b983342574 Mon Sep 17 00:00:00 2001
From: Mert Can Altin <mert.altin@trendyol.com>
Date: Wed, 21 Aug 2024 13:34:04 +0300
Subject: [PATCH] feat: make error format configurable

---
 bindings/binding_core_node/src/minify.rs | 25 ++++++++++++------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/bindings/binding_core_node/src/minify.rs b/bindings/binding_core_node/src/minify.rs
index 25e2a30a14b41..f7aba83fcefe0 100644
--- 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)
     }
 }
@@ -76,7 +77,8 @@ impl Task for MinifyTask {
 fn minify(code: Buffer, opts: Buffer, signal: Option<AbortSignal>) -> AsyncTask<MinifyTask> {
     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 opts_str = 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()
 }