fix(@angular-devkit/build-angular): avoid attempting to optimize copi…

…ed JavaScript assets

When in watch mode (`ng serve`/`ng build --watch`), Webpack's `copy-webpack-plugin` is currently used to implement the project `assets` option within `angular.json`. Files specified by the `assets` option are intended to be copied to the output unmodified and in their original form.  However, if any JavaScript assets were present they previously would have been unintentionally subject to their respective optimization plugins which would result in modified asset outputs and the potential for breakage of the assets. `ng build` outside of watch mode uses a direct file copy with copy-on-write (where supported) to process assets and is therefore not affected by this situation.
When the `copy-webpack-plugin` is used, it adds a `copied` flag to an asset's info metadata. This flag is now used to exclude all such copied JavaScript assets from optimization.

(cherry picked from commit f5d019f)

packages/angular_devkit/build_angular/src/webpack/plugins/javascript-optimizer-plugin.ts

@@ -96,34 +96,36 @@ export class JavaScriptOptimizerPlugin {
             }
 
             const scriptAsset = compilation.getAsset(assetName);
+            // Skip assets that have already been optimized or are verbatim copies (project assets)
+            if (!scriptAsset || scriptAsset.info.minimized || scriptAsset.info.copied) {
+              continue;
+            }
 
-            if (scriptAsset && !scriptAsset.info.minimized) {
-              const { source: scriptAssetSource, name } = scriptAsset;
-              let cacheItem;
-
-              if (cache) {
-                const eTag = cache.getLazyHashedEtag(scriptAssetSource);
-                cacheItem = cache.getItemCache(name, eTag);
-                const cachedOutput = await cacheItem.getPromise<
-                  { source: sources.Source } | undefined
-                >();
-
-                if (cachedOutput) {
-                  compilation.updateAsset(name, cachedOutput.source, {
-                    minimized: true,
-                  });
-                  continue;
-                }
+            const { source: scriptAssetSource, name } = scriptAsset;
+            let cacheItem;
+
+            if (cache) {
+              const eTag = cache.getLazyHashedEtag(scriptAssetSource);
+              cacheItem = cache.getItemCache(name, eTag);
+              const cachedOutput = await cacheItem.getPromise<
+                { source: sources.Source } | undefined
+              >();
+
+              if (cachedOutput) {
+                compilation.updateAsset(name, cachedOutput.source, {
+                  minimized: true,
+                });
+                continue;
               }
-
-              const { source, map } = scriptAssetSource.sourceAndMap();
-              scriptsToOptimize.push({
-                name: scriptAsset.name,
-                code: typeof source === 'string' ? source : source.toString(),
-                map,
-                cacheItem,
-              });
             }
+
+            const { source, map } = scriptAssetSource.sourceAndMap();
+            scriptsToOptimize.push({
+              name: scriptAsset.name,
+              code: typeof source === 'string' ? source : source.toString(),
+              map,
+              cacheItem,
+            });
           }
 
           if (scriptsToOptimize.length === 0) {