Build: Reduce the slim build comment header

mgol · mgol · commit ee3e80b0cbf4 · 2020-03-28T22:54:33.000+01:00
So far, the slim build was expanded to its full exclusion list, generating the
following in the comment header on `master`, for the unminified build:
```
 * jQuery JavaScript Library v4.0.0-pre -ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-deprecated/ajax-event-alias,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready
```
and for the minified one:
```
/*! jQuery v4.0.0-pre -ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-deprecated/ajax-event-alias,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready | (c) JS Foundation and other contributors | jquery.org/license */
```

This commit changes them to, respectively, for the unminified build:
```
 * jQuery JavaScript Library v4.0.0-pre slim
```
and for the minified one:
```
/*! jQuery v4.0.0-pre slim | (c) JS Foundation and other contributors | jquery.org/license */
```

Only the pure slim build is treated this way, any modification to it goes
through the old expansion; e.g. for `custom:slim,-deprecated` we get the
following for the unminified build:
```
 * jQuery JavaScript Library v4.0.0-pre -deprecated,-deprecated/ajax-event-alias,-deprecated/event,-ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready
```
and for the minified one:
```
/*! jQuery v4.0.0-pre -deprecated,-deprecated/ajax-event-alias,-deprecated/event,-ajax,-ajax/jsonp,-ajax/load,-ajax/script,-ajax/var/location,-ajax/var/nonce,-ajax/var/rquery,-ajax/xhr,-manipulation/_evalUrl,-callbacks,-deferred,-deferred/exceptionHook,-effects,-effects/Tween,-effects/animatedSelector,-queue,-queue/delay,-core/ready | (c) JS Foundation and other contributors | jquery.org/license */
```

Also, the logic to skip including the commit hash - when provided through the
COMMIT environment variable which we do in Jenkins - in minified builds headers
has been applied to builds with exclusions as well.
diff --git a/build/tasks/build.js b/build/tasks/build.js
@@ -60,7 +60,7 @@ module.exports = function( grunt ) {
 		const done = this.async();
 
 		try {
-			let flag, index;
+			const slimFlags = [ "-ajax", "-callbacks", "-deferred", "-effects" ];
 			const flags = this.flags;
 			const optIn = flags[ "*" ];
 			let name = grunt.option( "filename" );
@@ -70,6 +70,21 @@ module.exports = function( grunt ) {
 			const included = [];
 			let version = grunt.config( "pkg.version" );
 
+			// We'll skip printing the whole big exclusions for a bare `build:*:*:slim` which
+			// usually comes from `custom:slim`.
+			const isPureSlim = !!( flags.slim && flags[ "*" ] &&
+				Object.keys( flags ).length === 2 );
+
+			delete flags[ "*" ];
+
+			if ( flags.slim ) {
+				delete flags.slim;
+				for ( const flag of slimFlags ) {
+					flags[ flag ] = true;
+				}
+			}
+
+
 			/**
 			 * Recursively calls the excluder to remove on all modules in the list
 			 * @param {Array} list
@@ -187,8 +202,7 @@ module.exports = function( grunt ) {
 			//                     trumped by explicit exclude of dependency)
 			//  *:+effects         none except effects and its dependencies
 			//                     (explicit include trumps implicit exclude of dependency)
-			delete flags[ "*" ];
-			for ( flag in flags ) {
+			for ( const flag in flags ) {
 				excluder( flag );
 			}
 
@@ -198,7 +212,8 @@ module.exports = function( grunt ) {
 				read( inputFileName ).replace( /\n*export default jQuery;\n*/, "\n" ) );
 
 			// Replace exports/global with a noop noConflict
-			if ( ( index = excluded.indexOf( "exports/global" ) ) > -1 ) {
+			if ( excluded.includes( "exports/global" ) ) {
+				const index = excluded.indexOf( "exports/global" );
 				setOverride( `${ srcFolder }/exports/global.js`,
 					"import jQuery from \"../core.js\";\n\n" +
 						"jQuery.noConflict = function() {};" );
@@ -224,20 +239,32 @@ module.exports = function( grunt ) {
 			grunt.verbose.writeflags( excluded, "Excluded" );
 			grunt.verbose.writeflags( included, "Included" );
 
-			// append excluded modules to version
-			if ( excluded.length ) {
-				version += " -" + excluded.join( ",-" );
+			// Indicate a Slim build without listing all of the exclusions
+			// to save space.
+			if ( isPureSlim ) {
+				version += " slim";
+			}
 
-				// set pkg.version to version with excludes, so minified file picks it up
-				grunt.config.set( "pkg.version", version );
-				grunt.verbose.writeln( "Version changed to " + version );
+			// Append excluded modules to version.
+			if ( !isPureSlim && excluded.length ) {
+				version += " -" + excluded.join( ",-" );
 
 				// Replace excluded modules with empty sources.
 				for ( const module of excluded ) {
 					setOverride( `${ srcFolder }/${ module }.js`, "" );
 				}
 			}
 
+			if ( isPureSlim || excluded.length ) {
+
+				// Set pkg.version to version with excludes or with the "slim" marker,
+				// so minified file picks it up but skip the commit hash the same way
+				// it's done for the full build.
+				const commitlessVersion = version.replace( " " + process.env.COMMIT, "" );
+				grunt.config.set( "pkg.version", commitlessVersion );
+				grunt.verbose.writeln( "Version changed to " + commitlessVersion );
+			}
+
 			// Turn off opt-in if necessary
 			if ( !optIn ) {
 
@@ -299,18 +326,7 @@ module.exports = function( grunt ) {
 	grunt.registerTask( "custom", function() {
 		const args = this.args;
 		const modules = args.length ?
-			args[ 0 ]
-				.split( "," )
-
-				// Replace "slim" with respective exclusions meant for
-				// the official slim build
-				.reduce( ( acc, elem ) => acc.concat(
-					elem === "slim" ?
-						[ "-ajax", "-callbacks", "-deferred", "-effects" ] :
-						[ elem ]
-				), [] )
-
-				.join( ":" ) :
+			args[ 0 ].split( "," ).join( ":" ) :
 			"";
 		const done = this.async();
 		const insight = new Insight( {
