Respect mangle: false & sourceMap: false in minifierConfig (fac…

…ebook#749) Summary: Fixes facebook#301 Pull Request resolved: facebook#749 Reviewed By: motiz88 Differential Revision: D33398597 Pulled By: rh389 fbshipit-source-id: c6fff45e20df500977215bebe6707d0bb84e84d2
nevilm-lt · Mar 14, 2022 · de1872e · de1872e
1 parent 0ebc844
commit de1872e
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 16 deletions.
diff --git a/packages/metro-minify-terser/src/minifier.js b/packages/metro-minify-terser/src/minifier.js
@@ -33,15 +33,20 @@ function minify({code, map, reserved, config}: MinifierOptions): {
 } {
   const options = {
     ...config,
-    mangle: {
-      ...config.mangle,
-      reserved,
-    },
+    mangle:
+      config.mangle === false
+        ? false
+        : {
+            ...config.mangle,
+            reserved,
+          },
     sourceMap: map
-      ? {
-          ...config.sourceMap,
-          content: map,
-        }
+      ? config.sourceMap === false
+        ? false
+        : {
+            ...config.sourceMap,
+            content: map,
+          }
       : false,
   };
 

diff --git a/packages/metro-minify-uglify/src/minifier.js b/packages/metro-minify-uglify/src/minifier.js
@@ -34,14 +34,20 @@ function minify({code, map, reserved, config}: MinifierOptions): {
 } {
   const options = {
     ...config,
-    mangle: {
-      ...config.mangle,
-      reserved,
-    },
-    sourceMap: {
-      ...config.sourceMap,
-      content: map,
-    },
+    mangle:
+      config.mangle === false
+        ? false
+        : {
+            ...config.mangle,
+            reserved,
+          },
+    sourceMap:
+      config.sourceMap === false
+        ? false
+        : {
+            ...config.sourceMap,
+            content: map,
+          },
   };
 
   /* $FlowFixMe(>=0.111.0 site=react_native_fb) This comment suppresses an