fix(napi/minify): handle boolean values for compress.unused option (#14513)

fixes #14512

Fixes the `compress.unused` option implemented in PR #11796 to properly
accept boolean values from the napi binding.

Author: sushichan044

napi/minify/index.d.ts

@@ -38,11 +38,12 @@ export interface CompressOptions {
    */
   dropDebugger?: boolean
   /**
-   * Drop unreferenced functions and variables.
+   * Pass `true` to drop unreferenced functions and variables.
    *
-   * Simple direct variable assignments do not count as references unless set to "keep_assign".
+   * Simple direct variable assignments do not count as references unless set to `keep_assign`.
+   * @default true
    */
-  unused?: true | false | 'keep_assign'
+  unused?: boolean | 'keep_assign'
   /** Keep function / class names. */
   keepNames?: CompressOptionsKeepNames
   /**

napi/minify/src/options.rs

@@ -30,11 +30,12 @@ pub struct CompressOptions {
     /// @default true
     pub drop_debugger: Option<bool>,
 
-    /// Drop unreferenced functions and variables.
+    /// Pass `true` to drop unreferenced functions and variables.
     ///
-    /// Simple direct variable assignments do not count as references unless set to "keep_assign".
-    #[napi(ts_type = "true | false | 'keep_assign'")]
-    pub unused: Option<String>,
+    /// Simple direct variable assignments do not count as references unless set to `keep_assign`.
+    /// @default true
+    #[napi(ts_type = "boolean | 'keep_assign'")]
+    pub unused: Option<Either<bool, String>>,
 
     /// Keep function / class names.
     pub keep_names: Option<CompressOptionsKeepNames>,
@@ -69,8 +70,15 @@ impl TryFrom<&CompressOptions> for oxc_minifier::CompressOptions {
             drop_debugger: o.drop_debugger.unwrap_or(default.drop_debugger),
             join_vars: o.join_vars.unwrap_or(true),
             sequences: o.sequences.unwrap_or(true),
-            // TODO
-            unused: oxc_minifier::CompressOptionsUnused::Keep,
+            unused: match &o.unused {
+                Some(Either::A(true)) => oxc_minifier::CompressOptionsUnused::Remove,
+                Some(Either::A(false)) => oxc_minifier::CompressOptionsUnused::Keep,
+                Some(Either::B(s)) => match s.as_str() {
+                    "keep_assign" => oxc_minifier::CompressOptionsUnused::KeepAssign,
+                    _ => return Err(format!("Invalid unused option: `{s}`.")),
+                },
+                None => default.unused,
+            },
             keep_names: o.keep_names.as_ref().map(Into::into).unwrap_or_default(),
             treeshake: TreeShakeOptions::default(),
             max_iterations: o.passes,