refactor[ci/build]: preserve header format in artifacts (#27671)

In order to make Haste work with React's artifacts, It is important to
keep headers in this format:
```
/**
* ...
...
* ...
*/
```

For optimization purposes, Closure compiler will actually modify these
headers by removing * prefixes, which is expected.
We should pass sources to the compiler without license headers, with
these changes the current flow will be:
1. Apply top-level definitions. For UMD-bundles, for example, or
DEV-only bundles (e. g. `if (__DEV__) { ...`)
2. Apply licence headers for artifacts with sourcemaps: oss-production
and oss-profiling bundles, they don't need to preserve the header format
to comply with Haste. We need to apply these headers before passing
sources to Closure, so it can build correct mappings for sourcemaps.
3. Pass these sources to closure compiler for minification and
sourcemaps building.
4. Apply licence headers for artifacts without sourcemaps: dev bundles,
fb bundles. This way the header style will be preserved and not changed
by Closure.

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,21 @@ function getPlugins(
           );
         },
       },
+      // License and haste headers for artifacts with sourcemaps
+      // For artifacts with sourcemaps we apply these headers
+      // before passing sources to the Closure compiler, which will be building sourcemaps
+      needsSourcemaps && {
+        name: 'license-and-signature-header-for-artifacts-with-sourcemaps',
+        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 +541,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 +597,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 +606,21 @@ function getPlugins(
           };
         },
       },
+      // License and haste headers for artifacts without sourcemaps
+      // Primarily used for FB-artifacts, which should preserve specific format of the header
+      // Which potentially can be changed by Closure minification
+      !needsSourcemaps && {
+        name: 'license-and-signature-header-for-artifacts-without-sourcemaps',
+        renderChunk(source) {
+          return Wrappers.wrapWithLicenseHeader(
+            source,
+            bundleType,
+            globalName,
+            filename,
+            moduleType
+          );
+        },
+      },
       // Record bundle size.
       sizes({
         getSize: (size, gzip) => {