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,
@@ -517,6 +516,19 @@ function getPlugins(
           },
           {needsSourcemaps}
         ),
+      // License and haste headers
+      {
+        name: 'license-and-signature-header',
+        renderChunk(source) {
+          return Wrappers.wrapWithLicenseHeader(
+            source,
+            bundleType,
+            globalName,
+            filename,
+            moduleType
+          );
+        },
+      },
       // Add the whitespace back if necessary.
       shouldStayReadable &&
         prettier({
@@ -561,9 +573,32 @@ function getPlugins(
           // Because it sits on disk, there's no need to have it in the `sourcesContent` array.
           // That also makes the file easier to read, and available for use by scripts.
           // This should be the only file in the array.
-          const [preMinifiedBundleSource] =
+          const [preMinifiedBundleSourceWithoutLicense] =
             sourcemapAfterClosure.sourcesContent;
 
+          // We were passing sources without license headers to the Closure compiler.
+          // The reason for it is its optimization for multiline comment:
+          // it will change the format by removing * prefixes This doesn't work well with Haste.
+          const preMinifiedBundleSource = Wrappers.wrapWithLicenseHeader(
+            preMinifiedBundleSourceWithoutLicense,
+            bundleType,
+            globalName,
+            filename,
+            moduleType
+          );
+
+          // Update the lineCount field, since we've added the license header
+          // to the minified source, but Closure didn't know about it.
+          const sourceCodeLines = codeAfterLicense.split(/\r\n|\r|\n/);
+          // There is probably an empty line at the end after the Prettier run
+          while (
+            sourceCodeLines.length > 0 &&
+            sourceCodeLines[sourceCodeLines.length - 1] === ''
+          ) {
+            sourceCodeLines.pop();
+          }
+          sourcemapAfterClosure.lineCount = sourceCodeLines.length;
+
           // Remove this entirely - we're going to write the file to disk instead.
           delete sourcemapAfterClosure.sourcesContent;