refactor(@angular/cli): avoid implicit undefined defaults for @angular/build builders

Previously, all builder options that were of type array or object were
set to a default empty value even if there was no explicit default defined
within the schema. This can be problematic for options that have differing
behavior based on their presence such as runtime calculated defaults. The
implicit defaulting behavior was also not aligned with the generated schema
types which resulted in additional type safety and initialization regardless.
As a result, the implicit behavior was effectively redundant in most cases.
Since this change could be breaking for third-party builders, the removal of
this behavior is currently limited to the `@angular/build` package.

Author: clydin

modules/testing/builder/src/builder-harness.ts

@@ -104,7 +104,9 @@ export class BuilderHarness<T> {
       ...builderInfo,
     };
 
-    this.schemaRegistry.addPostTransform(json.schema.transforms.addUndefinedDefaults);
+    if (!builderInfo?.builderName?.startsWith('@angular/build:')) {
+      this.schemaRegistry.addPostTransform(json.schema.transforms.addUndefinedDefaults);
+    }
   }
 
   private resolvePath(path: string): string {

packages/angular/build/src/builders/application/tests/setup.ts

@@ -13,7 +13,7 @@ import { Schema } from '../schema';
 export * from '../../../../../../../modules/testing/builder/src';
 
 export const APPLICATION_BUILDER_INFO = Object.freeze({
-  name: '@angular-devkit/build-angular:application',
+  name: '@angular/build:application',
   schemaPath: __dirname + '/../schema.json',
 });
 

packages/angular/cli/src/command-builder/architect-base-command-module.ts

@@ -52,10 +52,16 @@ export abstract class ArchitectBaseCommandModule<T extends object>
       return this.onMissingTarget(e.message);
     }
 
+    const isAngularBuild = builderName.startsWith('@angular/build:');
+
     const { logger } = this.context;
-    const run = await this.getArchitect().scheduleTarget(target, options as json.JsonObject, {
-      logger,
-    });
+    const run = await this.getArchitect(isAngularBuild).scheduleTarget(
+      target,
+      options as json.JsonObject,
+      {
+        logger,
+      },
+    );
 
     const analytics = isPackageNameSafeForAnalytics(builderName)
       ? await this.getAnalytics()
@@ -150,13 +156,15 @@ export abstract class ArchitectBaseCommandModule<T extends object>
   }
 
   private _architect: Architect | undefined;
-  protected getArchitect(): Architect {
+  protected getArchitect(skipUndefinedTransform: boolean): Architect {
     if (this._architect) {
       return this._architect;
     }
 
     const registry = new json.schema.CoreSchemaRegistry();
-    registry.addPostTransform(json.schema.transforms.addUndefinedDefaults);
+    if (!skipUndefinedTransform) {
+      registry.addPostTransform(json.schema.transforms.addUndefinedDefaults);
+    }
     registry.useXDeprecatedProvider((msg) => this.context.logger.warn(msg));
 
     const architectHost = this.getArchitectHost();