refactor[ci/build]: preserve header format in artifacts

Author: hoxyq

scripts/rollup/build.js

@@ -470,11 +470,10 @@ function getPlugins(
       // I'm going to port "art" to ES modules to avoid this problem.
       // Please don't enable this for anything else!
       isUMDBundle && entry === 'react-art' && commonjs(),
-      // License and haste headers, top-level `if` blocks.
       {
-        name: 'license-and-headers',
+        name: 'top-level-definitions',
         renderChunk(source) {
-          return Wrappers.wrapBundle(
+          return Wrappers.wrapWithTopLevelDefinitions(
             source,
             bundleType,
             globalName,
@@ -484,6 +483,19 @@ function getPlugins(
           );
         },
       },
+      // License and haste headers
+      {
+        name: 'license-and-signature-header',
+        renderChunk(source) {
+          return Wrappers.wrapWithLicenseHeader(
+            source,
+            bundleType,
+            globalName,
+            filename,
+            moduleType
+          );
+        },
+      },
       // Apply dead code elimination and/or minification.
       // closure doesn't yet support leaving ESM imports intact
       needsMinifiedByClosure &&
@@ -527,7 +539,7 @@ function getPlugins(
         }),
       needsSourcemaps && {
         name: 'generate-prod-bundle-sourcemaps',
-        async renderChunk(codeAfterLicense, chunk, options, meta) {
+        async renderChunk(minifiedCodeWithChangedHeader, chunk, options, meta) {
           // We want to generate a sourcemap that shows the production bundle source
           // as it existed before Closure Compiler minified that chunk, rather than
           // showing the "original" individual source files. This better shows
@@ -583,7 +595,7 @@ function getPlugins(
 
           // Add the sourcemap URL to the actual bundle, so that tools pick it up
           const sourceWithMappingUrl =
-            codeAfterLicense +
+            minifiedCodeWithChangedHeader +
             `\n//# sourceMappingURL=${finalSourcemapFilename}`;
 
           return {
@@ -592,6 +604,42 @@ function getPlugins(
           };
         },
       },
+      // After minification with Closure, the header can be changed.
+      needsMinifiedByClosure && {
+        name: 'format-header',
+        renderChunk(source) {
+          // Don't change if its already correct or doesn't start with the header
+          if (source.startsWith('/**') || !source.startsWith('/*')) {
+            return source;
+          }
+
+          // Update the header to follow this format:
+          // /**
+          //  *
+          //  * ...
+          //  */
+          // We can shift the lines to the right, but don't add / remove lines
+          // It can break mappings in sourcemaps
+          const lines = source.split(/\r\n|\r|\n/);
+          for (let lineIndex = 0; lineIndex < lines.length; ++lineIndex) {
+            const currentLine = lines[lineIndex];
+
+            if (currentLine === '*/') {
+              lines[lineIndex] = ' */';
+              break;
+            }
+
+            if (currentLine === '/*') {
+              lines[lineIndex] = '/**';
+              continue;
+            }
+
+            lines[lineIndex] = ` *${currentLine}`;
+          }
+
+          return lines.join('\r\n');
+        },
+      },
       // Record bundle size.
       sizes({
         getSize: (size, gzip) => {