fix(terser): return promise reject on empty chunk

rollup · Nov 10, 2022 · 0d87e6b · 0d87e6b
1 parent 66da603
commit 0d87e6b
Showing 1 changed file with 33 additions and 29 deletions.
diff --git a/packages/terser/src/module.ts b/packages/terser/src/module.ts
@@ -21,43 +21,47 @@ export default function terser(options: Options = {}) {
         defaultOptions.toplevel = true;
       }
 
-      const { code: result, nameCache } = await callWorker(__filename, {
-        code,
-        options: merge({}, options || {}, defaultOptions)
-      });
-
-      if (options.nameCache && nameCache) {
-        let vars: Record<string, any> = {
-          props: {}
-        };
-
-        if (hasOwnProperty(options.nameCache, 'vars') && isObject(options.nameCache.vars)) {
-          vars = merge({}, options.nameCache.vars || {}, vars);
-        }
+      try {
+        const { code: result, nameCache } = await callWorker(__filename, {
+          code,
+          options: merge({}, options || {}, defaultOptions)
+        });
 
-        if (hasOwnProperty(nameCache, 'vars') && isObject(nameCache.vars)) {
-          vars = merge({}, nameCache.vars, vars);
-        }
+        if (options.nameCache && nameCache) {
+          let vars: Record<string, any> = {
+            props: {}
+          };
 
-        // eslint-disable-next-line no-param-reassign
-        options.nameCache.vars = vars;
+          if (hasOwnProperty(options.nameCache, 'vars') && isObject(options.nameCache.vars)) {
+            vars = merge({}, options.nameCache.vars || {}, vars);
+          }
 
-        let props: Record<string, any> = {};
+          if (hasOwnProperty(nameCache, 'vars') && isObject(nameCache.vars)) {
+            vars = merge({}, nameCache.vars, vars);
+          }
 
-        if (hasOwnProperty(options.nameCache, 'props') && isObject(options.nameCache.props)) {
-          // eslint-disable-next-line prefer-destructuring
-          props = options.nameCache.props;
-        }
+          // eslint-disable-next-line no-param-reassign
+          options.nameCache.vars = vars;
+
+          let props: Record<string, any> = {};
 
-        if (hasOwnProperty(nameCache, 'props') && isObject(nameCache.props)) {
-          props = merge({}, nameCache.props, props);
+          if (hasOwnProperty(options.nameCache, 'props') && isObject(options.nameCache.props)) {
+            // eslint-disable-next-line prefer-destructuring
+            props = options.nameCache.props;
+          }
+
+          if (hasOwnProperty(nameCache, 'props') && isObject(nameCache.props)) {
+            props = merge({}, nameCache.props, props);
+          }
+
+          // eslint-disable-next-line no-param-reassign
+          options.nameCache.props = props;
         }
 
-        // eslint-disable-next-line no-param-reassign
-        options.nameCache.props = props;
+        return result;
+      } catch (e) {
+        return Promise.reject(e);
       }
-
-      return result;
     }
   };
 }