From 9fd356234210734ec5f44ae18f055308b7acc963 Mon Sep 17 00:00:00 2001
From: Charles Lyding <19598772+clydin@users.noreply.github.com>
Date: Mon, 12 Dec 2022 16:25:55 -0500
Subject: [PATCH] fix(@angular-devkit/build-angular): only set ngDevMode when
 script optimizations are enabled

When using the experimental esbuild-based browser application builder, the `ngDevMode` global
runtime variable was unintentionally always being set to false due to a previous bug fix that stopped
the variable from being replaced with the value of true when script optimizations were disabled.
By doing so, the fix caused the imported compiler-cli `GLOBAL_DEFS_FOR_TERSER_WITH_AOT` constant
to take precedence which contains an `ngDevMode` value of false. To prevent this situation for
development builds where a non-false `ngDevMode` is helpful to surface potential runtime problems,
`GLOBAL_DEFS_FOR_TERSER_WITH_AOT` will no longer change the value of `ngDevMode`. This fix does not
have any effect on production builds since `ngDevMode` would have been set to false regardless.

(cherry picked from commit 310144d324bd773aa6026f47b345827d5fe48332)
---
 .../src/builders/browser-esbuild/compiler-plugin.ts           | 4 ++++
 .../build_angular/src/builders/browser-esbuild/index.ts       | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/packages/angular_devkit/build_angular/src/builders/browser-esbuild/compiler-plugin.ts b/packages/angular_devkit/build_angular/src/builders/browser-esbuild/compiler-plugin.ts
index d7bb28518f92..5eac59ba54a1 100644
--- a/packages/angular_devkit/build_angular/src/builders/browser-esbuild/compiler-plugin.ts
+++ b/packages/angular_devkit/build_angular/src/builders/browser-esbuild/compiler-plugin.ts
@@ -192,6 +192,10 @@ export function createCompilerPlugin(
           // Skip keys that have been manually provided
           continue;
         }
+        if (key === 'ngDevMode') {
+          // ngDevMode is already set based on the builder's script optimization option
+          continue;
+        }
         // esbuild requires values to be a string (actual strings need to be quoted).
         // In this case, all provided values are booleans.
         build.initialOptions.define[key] = value.toString();
diff --git a/packages/angular_devkit/build_angular/src/builders/browser-esbuild/index.ts b/packages/angular_devkit/build_angular/src/builders/browser-esbuild/index.ts
index 2508c1279cff..6d15beaf1d78 100644
--- a/packages/angular_devkit/build_angular/src/builders/browser-esbuild/index.ts
+++ b/packages/angular_devkit/build_angular/src/builders/browser-esbuild/index.ts
@@ -296,7 +296,11 @@ function createCodeBundleOptions(
       ),
     ],
     define: {
+      // Only set to false when script optimizations are enabled. It should not be set to true because
+      // Angular turns `ngDevMode` into an object for development debugging purposes when not defined
+      // which a constant true value would break.
       ...(optimizationOptions.scripts ? { 'ngDevMode': 'false' } : undefined),
+      // Only AOT mode is supported currently
       'ngJitMode': 'false',
     },
   };