Boost: Update minify-library (#37728)

* Don't minify files that already has a .min extension

* changelog

* Swap JS minification library

* Fix

* Don't require the previously used library

* Fix expected output in test

projects/plugins/boost/app/lib/class-minify.php

@@ -9,7 +9,7 @@
 
 namespace Automattic\Jetpack_Boost\Lib;
 
-use JShrink\Minifier as JSMinifier;
+use MatthiasMullie\Minify\JS as JSMinifier;
 use tubalmartin\CssMin\Minifier as CSSMinifier;
 
 /**
@@ -30,10 +30,9 @@ class Minify {
 	 * @return string String with whitespace stripped.
 	 */
 	public static function js( $js ) {
-		require_once JETPACK_BOOST_DIR_PATH . '/vendor/tedivm/jshrink/src/JShrink/Minifier.php';
-
 		try {
-			$minified_js = JSMinifier::minify( $js );
+			$minifier    = new JSMinifier( $js );
+			$minified_js = $minifier->minify();
 		} catch ( \Exception $e ) {
 			return $js;
 		}

projects/plugins/boost/composer.json

@@ -29,7 +29,7 @@
 		"automattic/jetpack-status": "@dev",
 		"automattic/jetpack-sync": "@dev",
 		"automattic/jetpack-wp-js-data-sync": "@dev",
-		"tedivm/jshrink": "1.4.0",
+		"matthiasmullie/minify": "^1.3",
 		"tubalmartin/cssmin": "^4.1"
 	},
 	"require-dev": {

projects/plugins/boost/tests/php/lib/test-class-minify.php

@@ -15,7 +15,7 @@ public function test_js() {
 var two = "two";
 var three = "three";';
 
-		$minified_js = 'var one="one";var two="two";var three="three";';
+		$minified_js = 'var one="one";var two="two";var three="three"';
 
 		$this->assertEquals( $minified_js, Minify::js( $expanded_js ) );
 	}