From bee2ddedf3022589c317934e8722840157a79ebc Mon Sep 17 00:00:00 2001
From: Alan Agius <alanagius@google.com>
Date: Mon, 13 Mar 2023 16:37:56 +0000
Subject: [PATCH 1/2] fix(@angular-devkit/build-angular): set public class
 fields as properties

Configure Babel to use `setPublicClassFields: true`. As when shipping Angular packages without `"useDefineForClassFields": false` will increase the bundle size of an ng-new app
by ~2Kb when `useDefineForClassFields`  is not false. due to the additional `_defineProperty`, which will be added on every class property.

See: https://babeljs.io/docs/babel-plugin-proposal-class-properties
---
 .../build_angular/src/webpack/configs/common.ts            | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/packages/angular_devkit/build_angular/src/webpack/configs/common.ts b/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
index 84fba1774ead..18296e30867f 100644
--- a/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
+++ b/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
@@ -411,6 +411,13 @@ export async function getCommonConfig(wco: WebpackConfigOptions): Promise<Config
             {
               loader: require.resolve('../../babel/webpack-loader'),
               options: {
+                assumptions: {
+                  // Use `setPublicClassFields: true` to avoid decrease bundle sizes
+                  // when targetting ES2022+ with `useDefineForClassFields: true`.
+                  // This is because ervery property of the class will otherwise be wrapped in a `_defineProperty`.
+                  // This is not needed for TypeScript as TS itself will emit the right decleration of class properties.
+                  setPublicClassFields: true,
+                },
                 cacheDirectory: (cache.enabled && path.join(cache.path, 'babel-webpack')) || false,
                 aot: buildOptions.aot,
                 optimize: buildOptions.buildOptimizer,

From e91935da952d0929f270c4633a50023643038135 Mon Sep 17 00:00:00 2001
From: Alan Agius <alanagius@google.com>
Date: Mon, 13 Mar 2023 20:58:24 +0100
Subject: [PATCH 2/2] fixup! fix(@angular-devkit/build-angular): set public
 class fields as properties

Co-authored-by: Charles <19598772+clydin@users.noreply.github.com>
---
 .../angular_devkit/build_angular/src/webpack/configs/common.ts  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/angular_devkit/build_angular/src/webpack/configs/common.ts b/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
index 18296e30867f..476b0e388bcf 100644
--- a/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
+++ b/packages/angular_devkit/build_angular/src/webpack/configs/common.ts
@@ -415,7 +415,7 @@ export async function getCommonConfig(wco: WebpackConfigOptions): Promise<Config
                   // Use `setPublicClassFields: true` to avoid decrease bundle sizes
                   // when targetting ES2022+ with `useDefineForClassFields: true`.
                   // This is because ervery property of the class will otherwise be wrapped in a `_defineProperty`.
-                  // This is not needed for TypeScript as TS itself will emit the right decleration of class properties.
+                  // This is not needed for TypeScript as TS itself will emit the right declaration of class properties.
                   setPublicClassFields: true,
                 },
                 cacheDirectory: (cache.enabled && path.join(cache.path, 'babel-webpack')) || false,