fix(@angular-devkit/build-angular): emit error when using prerender a…

…nd app-shell builders with application builder The application builder has built-in prerendering and app-shell generation capabilities which makes using these builders redundant.
angular · Dec 26, 2023 · 49ed9a2 · 49ed9a2
1 parent 140a5f5
commit 49ed9a2
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/packages/angular_devkit/build_angular/src/builders/app-shell/index.ts b/packages/angular_devkit/build_angular/src/builders/app-shell/index.ts
@@ -178,6 +178,16 @@ async function _appShellBuilder(
     serviceWorker: false,
     optimization: optimization as unknown as JsonObject,
   });
+
+  if (browserTargetRun.info.builderName === '@angular-devkit/build-angular:application') {
+    return {
+      success: false,
+      error:
+        '"@angular-devkit/build-angular:application" has built-in app-shell generation capabilities. ' +
+        'The "appShell" option should be used instead.',
+    };
+  }
+
   const serverTargetRun = await context.scheduleTarget(serverTarget, {
     watch: false,
   });

diff --git a/packages/angular_devkit/build_angular/src/builders/prerender/index.ts b/packages/angular_devkit/build_angular/src/builders/prerender/index.ts
@@ -106,6 +106,16 @@ async function _scheduleBuilds(
     serviceWorker: false,
     // todo: handle service worker augmentation
   });
+
+  if (browserTargetRun.info.builderName === '@angular-devkit/build-angular:application') {
+    return {
+      success: false,
+      error:
+        '"@angular-devkit/build-angular:application" has built-in prerendering capabilities. ' +
+        'The "prerender" option should be used instead.',
+    };
+  }
+
   const serverTargetRun = await context.scheduleTarget(serverTarget, {
     watch: false,
   });