fix(@angular-devkit/build-angular): change service worker errors to compilation errors

Previously, when there was an error during a build that had service workers enabled, the dev-server crashed as the promise was rejected instead of emitting a compilation error.

With this change we update the logic so that any errors during the SW augmentation phase are changed to a compilation error and also update the logic so that when there are compilation errors we don't try to generate a SW.

Author: alan-agius4

packages/angular_devkit/build_angular/src/webpack/plugins/service-worker-plugin.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import type { Compiler } from 'webpack';
+import { Compiler, WebpackError } from 'webpack';
 import { augmentAppWithServiceWorker } from '../../utils/service-worker';
 
 export interface ServiceWorkerPluginOptions {
@@ -21,6 +21,12 @@ export class ServiceWorkerPlugin {
 
   apply(compiler: Compiler) {
     compiler.hooks.done.tapPromise('angular-service-worker', async ({ compilation }) => {
+      if (compilation.errors.length) {
+        // Don't generate a service worker if the compilation has errors.
+        // When there are errors some files will not be emitted which would cause other errors down the line such as readdir failures.
+        return;
+      }
+
       const { projectRoot, root, baseHref = '', ngswConfigPath } = this.options;
       // We use the output path from the compilation instead of build options since during
       // localization the output path is modified to a temp directory.
@@ -31,17 +37,25 @@ export class ServiceWorkerPlugin {
         throw new Error('Compilation output path cannot be empty.');
       }
 
-      await augmentAppWithServiceWorker(
-        projectRoot,
-        root,
-        outputPath,
-        baseHref,
-        ngswConfigPath,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        (compiler.inputFileSystem as any).promises,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        (compiler.outputFileSystem as any).promises,
-      );
+      try {
+        await augmentAppWithServiceWorker(
+          projectRoot,
+          root,
+          outputPath,
+          baseHref,
+          ngswConfigPath,
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+          (compiler.inputFileSystem as any).promises,
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+          (compiler.outputFileSystem as any).promises,
+        );
+      } catch (error) {
+        compilation.errors.push(
+          new WebpackError(
+            `Failed to generate service worker - ${error instanceof Error ? error.message : error}`,
+          ),
+        );
+      }
     });
   }
 }