fix: don't remove legal comments by default

src/index.js

@@ -42,7 +42,7 @@ class UglifyJsPlugin {
       exclude,
       uglifyOptions: {
         output: {
-          comments: false,
+          comments: extractComments ? false : /^\**!|@preserve|@license|@cc_on/,
         },
         ...uglifyOptions,
       },

test/__snapshots__/extract-comments-options.test.js.snap

@@ -1,107 +1,130 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`errors 1`] = `Array []`;
+exports[`"options.extractComments" is "boolean" - "true": errors 1`] = `Array []`;
 
-exports[`errors 2`] = `Array []`;
+exports[`"options.extractComments" is "boolean" - "true": test.js 1`] = `"var foo=1;"`;
 
-exports[`errors 3`] = `Array []`;
-
-exports[`errors 4`] = `Array []`;
-
-exports[`errors 5`] = `Array []`;
-
-exports[`nested/nested/test1.js 1`] = `
+exports[`"options.extractComments" is "boolean" - "true": test1.js 1`] = `
 "/*! For license information please see test1.js.LICENSE */
 var foo=1;"
 `;
 
-exports[`nested/nested/test1.js.LICENSE 1`] = `
-"/* Comment */
+exports[`"options.extractComments" is "boolean" - "true": test1.js.LICENSE 1`] = `
+"/*! Legal Comment */
 "
 `;
 
-exports[`nested/test.js 1`] = `
+exports[`"options.extractComments" is "boolean" - "true": warnings 1`] = `Array []`;
+
+exports[`"options.extractComments" is "function": errors 1`] = `Array []`;
+
+exports[`"options.extractComments" is "function": test.js 1`] = `
 "/*! For license information please see test.js.LICENSE */
 var foo=1;"
 `;
 
-exports[`nested/test1.js.LICENSE 1`] = `
+exports[`"options.extractComments" is "function": test.js.LICENSE 1`] = `
 "// Comment
 "
 `;
 
-exports[`test.js 1`] = `"var foo=1;"`;
-
-exports[`test.js 2`] = `"var foo=1;"`;
-
-exports[`test.js 3`] = `
-"/*! For license information please see test.js.LICENSE */
+exports[`"options.extractComments" is "function": test1.js 1`] = `
+"/*! For license information please see test1.js.LICENSE */
 var foo=1;"
 `;
 
-exports[`test.js 4`] = `
-"/*! For license information please see test.js.LICENSE */
-var foo=1;"
+exports[`"options.extractComments" is "function": test1.js.LICENSE 1`] = `
+"/* Comment */
+"
 `;
 
-exports[`test.js 5`] = `
-"/*! For license information please see test1.js.LICENSE */
-var foo=1;"
-`;
+exports[`"options.extractComments" is "function": warnings 1`] = `Array []`;
+
+exports[`"options.extractComments" is "object": errors 1`] = `Array []`;
 
-exports[`test.js 6`] = `
+exports[`"options.extractComments" is "object": test.js 1`] = `
 "/*! License information can be found in test.license.js */
 var foo=1;"
 `;
 
-exports[`test.js.LICENSE 1`] = `
+exports[`"options.extractComments" is "object": test.license.js 1`] = `
 "// Comment
 "
 `;
 
-exports[`test.js.LICENSE 2`] = `
-"// Comment
-"
+exports[`"options.extractComments" is "object": warnings 1`] = `Array []`;
+
+exports[`"options.extractComments" is "regex": errors 1`] = `Array []`;
+
+exports[`"options.extractComments" is "regex": test.js 1`] = `"var foo=1;"`;
+
+exports[`"options.extractComments" is "regex": test1.js 1`] = `
+"/*! For license information please see test1.js.LICENSE */
+var foo=1;"
 `;
 
-exports[`test.license.js 1`] = `
-"// Comment
+exports[`"options.extractComments" is "regex": test1.js.LICENSE 1`] = `
+"// foo
 "
 `;
 
-exports[`test1.js 1`] = `"var foo=1;"`;
+exports[`"options.extractComments" is "regex": warnings 1`] = `Array []`;
 
-exports[`test1.js 2`] = `
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: errors 1`] = `Array []`;
+
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: nested/nested/test1.js 1`] = `
 "/*! For license information please see test1.js.LICENSE */
 var foo=1;"
 `;
 
-exports[`test1.js 3`] = `
-"/*! For license information please see test1.js.LICENSE */
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: nested/nested/test1.js.LICENSE 1`] = `
+"/* Comment */
+"
+`;
+
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: nested/test.js 1`] = `
+"/*! For license information please see test.js.LICENSE */
 var foo=1;"
 `;
 
-exports[`test1.js.LICENSE 1`] = `
-"// foo
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: nested/test.js.LICENSE 1`] = `
+"// Comment
 "
 `;
 
-exports[`test1.js.LICENSE 2`] = `
-"/* Comment */
+exports[`"options.extractComments" is "string" - "all" and license file should be relative source file: warnings 1`] = `Array []`;
+
+exports[`"options.extractComments" is "string": errors 1`] = `Array []`;
+
+exports[`"options.extractComments" is "string": test.js 1`] = `
+"/*! For license information please see test.js.LICENSE */
+var foo=1;"
+`;
+
+exports[`"options.extractComments" is "string": test.js.LICENSE 1`] = `
+"// Comment
 "
 `;
 
-exports[`test1.js.LICENSE 3`] = `
+exports[`"options.extractComments" is "string": test1.js 1`] = `
+"/*! For license information please see test1.js.LICENSE */
+var foo=1;"
+`;
+
+exports[`"options.extractComments" is "string": test1.js.LICENSE 1`] = `
 "/* Comment */
 "
 `;
 
-exports[`warnings 1`] = `Array []`;
+exports[`"options.extractComments" is "string": warnings 1`] = `Array []`;
 
-exports[`warnings 2`] = `Array []`;
+exports[`"options.extractComments" is not specify: errors 1`] = `Array []`;
 
-exports[`warnings 3`] = `Array []`;
+exports[`"options.extractComments" is not specify: test.js 1`] = `"var foo=1;"`;
 
-exports[`warnings 4`] = `Array []`;
+exports[`"options.extractComments" is not specify: test1.js 1`] = `
+"/*! Legal Comment */
+var foo=1;"
+`;
 
-exports[`warnings 5`] = `Array []`;
+exports[`"options.extractComments" is not specify: warnings 1`] = `Array []`;

test/__snapshots__/invalid-options.test.js.snap

@@ -187,7 +187,7 @@ exports[`warnings 1`] = `Array []`;
 exports[`when applied with invalid options throws validation errors 1`] = `
 "UglifyJs Plugin Invalid Options
 
-options['doesntExist'] is an invalid additional property
+options['doesntExist'] should NOT have additional properties
 "
 `;