@angular/build 20.0.0-rc.2 → 20.0.0-rc.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular/build",
-  "version": "20.0.0-rc.2",
+  "version": "20.0.0-rc.3",
   "description": "Official build system for Angular",
   "keywords": [
     "Angular CLI",
@@ -23,7 +23,7 @@
   "builders": "builders.json",
   "dependencies": {
     "@ampproject/remapping": "2.3.0",
-    "@angular-devkit/architect": "0.2000.0-rc.2",
+    "@angular-devkit/architect": "0.2000.0-rc.3",
     "@babel/core": "7.27.1",
     "@babel/helper-annotate-as-pure": "7.27.1",
     "@babel/helper-split-export-declaration": "7.24.7",
@@ -60,7 +60,7 @@
     "@angular/platform-browser": "^20.0.0 || ^20.0.0-next.0",
     "@angular/platform-server": "^20.0.0 || ^20.0.0-next.0",
     "@angular/service-worker": "^20.0.0 || ^20.0.0-next.0",
-    "@angular/ssr": "^20.0.0-rc.2",
+    "@angular/ssr": "^20.0.0-rc.3",
     "karma": "^6.4.0",
     "less": "^4.2.0",
     "ng-packagr": "^20.0.0 || ^20.0.0-next.0",
@@ -114,7 +114,7 @@
   },
   "packageManager": "pnpm@9.15.9",
   "engines": {
-    "node": "^20.11.1 || ^22.11.0 || >=24.0.0",
+    "node": "^20.19.0 || ^22.12.0 || >=24.0.0",
     "npm": "^6.11.0 || ^7.5.6 || >=8.0.0",
     "yarn": ">= 1.13.0"
   },

package/src/builders/extract-i18n/builder.js

@@ -100,13 +100,20 @@ async function execute(options, context, extensions) {
             return node_path_1.default.relative(from, to);
         },
     };
+    const duplicateTranslationBehavior = normalizedOptions.i18nOptions.duplicateTranslationBehavior;
     const diagnostics = checkDuplicateMessages(
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    checkFileSystem, extractionResult.messages, normalizedOptions.i18nOptions.i18nDuplicateTranslation || 'warning', 
+    checkFileSystem, extractionResult.messages, duplicateTranslationBehavior, 
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     extractionResult.basePath);
-    if (diagnostics.messages.length > 0) {
-        context.logger.warn(diagnostics.formatDiagnostics(''));
+    if (diagnostics.messages.length > 0 && duplicateTranslationBehavior !== 'ignore') {
+        if (duplicateTranslationBehavior === 'error') {
+            context.logger.error(`Extraction Failed: ${diagnostics.formatDiagnostics('')}`);
+            return { success: false };
+        }
+        else {
+            context.logger.warn(diagnostics.formatDiagnostics(''));
+        }
     }
     // Serialize all extracted messages
     const serializer = await createSerializer(localizeToolsModule, normalizedOptions.format, normalizedOptions.i18nOptions.sourceLocale, extractionResult.basePath, extractionResult.useLegacyIds, diagnostics);

package/src/builders/extract-i18n/options.d.ts

@@ -5,7 +5,9 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.dev/license
  */
+import { type DiagnosticHandlingStrategy } from '@angular/localize/tools';
 import { BuilderContext } from '@angular-devkit/architect';
+import { type I18nOptions } from '../../utils/i18n-options';
 import { Schema as ExtractI18nOptions, Format } from './schema';
 export type NormalizedExtractI18nOptions = Awaited<ReturnType<typeof normalizeOptions>>;
 /**
@@ -22,7 +24,9 @@ export declare function normalizeOptions(context: BuilderContext, projectName: s
     workspaceRoot: string;
     projectRoot: string;
     buildTarget: import("@angular-devkit/architect").Target;
-    i18nOptions: import("../../utils/i18n-options").I18nOptions;
+    i18nOptions: I18nOptions & {
+        duplicateTranslationBehavior: DiagnosticHandlingStrategy;
+    };
     format: Format.Arb | Format.Json | Format.LegacyMigrate | Format.Xliff | Format.Xliff2 | Format.Xmb;
     outFile: string;
     progress: boolean;

package/src/builders/extract-i18n/options.js

@@ -33,7 +33,10 @@ async function normalizeOptions(context, projectName, options) {
     // Target specifier defaults to the current project's build target with no specified configuration
     const buildTargetSpecifier = options.buildTarget ?? ':';
     const buildTarget = (0, architect_1.targetFromTargetString)(buildTargetSpecifier, projectName, 'build');
-    const i18nOptions = (0, i18n_options_1.createI18nOptions)(projectMetadata, /** inline */ false, context.logger);
+    const i18nOptions = {
+        ...(0, i18n_options_1.createI18nOptions)(projectMetadata, /** inline */ false, context.logger),
+        duplicateTranslationBehavior: options.i18nDuplicateTranslation || 'warning',
+    };
     // Normalize xliff format extensions
     let format = options.format;
     switch (format) {

package/src/builders/extract-i18n/schema.d.ts

@@ -12,6 +12,10 @@ export type Schema = {
      * Output format for the generated file.
      */
     format?: Format;
+    /**
+     * How to handle duplicate translations.
+     */
+    i18nDuplicateTranslation?: I18NDuplicateTranslation;
     /**
      * Name of the file to output.
      */
@@ -39,3 +43,11 @@ export declare enum Format {
     Xliff2 = "xliff2",
     Xmb = "xmb"
 }
+/**
+ * How to handle duplicate translations.
+ */
+export declare enum I18NDuplicateTranslation {
+    Error = "error",
+    Ignore = "ignore",
+    Warning = "warning"
+}

package/src/builders/extract-i18n/schema.js

@@ -2,7 +2,7 @@
 // THIS FILE IS AUTOMATICALLY GENERATED. TO UPDATE THIS FILE YOU NEED TO CHANGE THE
 // CORRESPONDING JSON SCHEMA FILE, THEN RUN devkit-admin build (or bazel build ...).
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.Format = void 0;
+exports.I18NDuplicateTranslation = exports.Format = void 0;
 /**
  * Output format for the generated file.
  */
@@ -18,3 +18,12 @@ var Format;
     Format["Xliff2"] = "xliff2";
     Format["Xmb"] = "xmb";
 })(Format || (exports.Format = Format = {}));
+/**
+ * How to handle duplicate translations.
+ */
+var I18NDuplicateTranslation;
+(function (I18NDuplicateTranslation) {
+    I18NDuplicateTranslation["Error"] = "error";
+    I18NDuplicateTranslation["Ignore"] = "ignore";
+    I18NDuplicateTranslation["Warning"] = "warning";
+})(I18NDuplicateTranslation || (exports.I18NDuplicateTranslation = I18NDuplicateTranslation = {}));

package/src/builders/extract-i18n/schema.json

@@ -27,6 +27,11 @@
     "outFile": {
       "type": "string",
       "description": "Name of the file to output."
+    },
+    "i18nDuplicateTranslation": {
+      "type": "string",
+      "description": "How to handle duplicate translations.",
+      "enum": ["error", "warning", "ignore"]
     }
   },
   "additionalProperties": false

package/src/builders/unit-test/builder.js

@@ -107,7 +107,6 @@ async function* execute(options, context, extensions = {}) {
                 `import { NgModule } from '@angular/core';`,
                 `import { getTestBed, ɵgetCleanupHook as getCleanupHook } from '@angular/core/testing';`,
                 `import { BrowserTestingModule, platformBrowserTesting } from '@angular/platform-browser/testing';`,
-                `import { beforeEach, afterEach } from 'vitest';`,
                 '',
                 normalizedOptions.providersFile
                     ? `import providers from './${node_path_1.default
@@ -140,7 +139,7 @@ async function* execute(options, context, extensions = {}) {
     extensions.codePlugins.unshift(virtualTestBedInit);
     let instance;
     // Setup vitest browser options if configured
-    const { browser, errors } = setupBrowserConfiguration(normalizedOptions.browsers, projectSourceRoot);
+    const { browser, errors } = setupBrowserConfiguration(normalizedOptions.browsers, normalizedOptions.debug, projectSourceRoot);
     if (errors?.length) {
         errors.forEach((error) => context.logger.error(error));
         return { success: false };
@@ -150,6 +149,13 @@ async function* execute(options, context, extensions = {}) {
     if (buildTargetOptions?.polyfills?.length) {
         setupFiles.push('polyfills.js');
     }
+    const debugOptions = normalizedOptions.debug
+        ? {
+            inspectBrk: true,
+            isolate: false,
+            fileParallelism: false,
+        }
+        : {};
     try {
         for await (const result of (0, application_1.buildApplicationInternal)(buildOptions, context, extensions)) {
             if (result.kind === results_1.ResultKind.Failure) {
@@ -160,7 +166,10 @@ async function* execute(options, context, extensions = {}) {
             }
             (0, node_assert_1.default)(result.files, 'Builder did not provide result files.');
             await (0, application_builder_1.writeTestFiles)(result.files, outputPath);
-            instance ??= await startVitest('test', undefined /* cliFilters */, undefined /* options */, {
+            instance ??= await startVitest('test', undefined /* cliFilters */, {
+                // Disable configuration file resolution/loading
+                config: false,
+            }, {
                 test: {
                     root: outputPath,
                     globals: true,
@@ -173,10 +182,25 @@ async function* execute(options, context, extensions = {}) {
                     reporters: normalizedOptions.reporters ?? ['default'],
                     coverage: {
                         enabled: normalizedOptions.codeCoverage,
-                        exclude: normalizedOptions.codeCoverageExclude,
                         excludeAfterRemap: true,
                     },
+                    ...debugOptions,
                 },
+                plugins: [
+                    {
+                        name: 'angular-coverage-exclude',
+                        configureVitest(context) {
+                            // Adjust coverage excludes to not include the otherwise automatically inserted included unit tests.
+                            // Vite does this as a convenience but is problematic for the bundling strategy employed by the
+                            // builder's test setup. To workaround this, the excludes are adjusted here to only automatically
+                            // exclude the TypeScript source test files.
+                            context.project.config.coverage.exclude = [
+                                ...(normalizedOptions.codeCoverageExclude ?? []),
+                                '**/*.{test,spec}.?(c|m)ts',
+                            ];
+                        },
+                    },
+                ],
             });
             // Check if all the tests pass to calculate the result
             const testModules = instance.state.getTestModules();
@@ -201,7 +225,7 @@ function findBrowserProvider(projectResolver) {
         catch { }
     }
 }
-function setupBrowserConfiguration(browsers, projectSourceRoot) {
+function setupBrowserConfiguration(browsers, debug, projectSourceRoot) {
     if (browsers === undefined) {
         return {};
     }
@@ -221,6 +245,12 @@ function setupBrowserConfiguration(browsers, projectSourceRoot) {
         errors.push('The "browsers" option requires either "playwright" or "webdriverio" to be installed within the project.' +
             ' Please install one of these packages and rerun the test command.');
     }
+    // Vitest current requires the playwright browser provider to use the inspect-brk option used by "debug"
+    if (debug && provider !== 'playwright') {
+        errors ??= [];
+        errors.push('Debugging browser mode tests currently requires the use of "playwright".' +
+            ' Please install this package and rerun the test command.');
+    }
     if (errors) {
         return { errors };
     }

package/src/builders/unit-test/karma-bridge.js

@@ -42,6 +42,9 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.useKarmaBuilder = useKarmaBuilder;
 async function useKarmaBuilder(context, unitTestOptions) {
+    if (unitTestOptions.debug) {
+        context.logger.warn('The "karma" test runner does not support the "debug" option. The option will be ignored.');
+    }
     const buildTargetOptions = (await context.validateOptions(await context.getTargetOptions(unitTestOptions.buildTarget), await context.getBuilderNameForTarget(unitTestOptions.buildTarget)));
     const options = {
         tsConfig: unitTestOptions.tsConfig,

package/src/builders/unit-test/options.d.ts

@@ -22,6 +22,7 @@ export declare function normalizeOptions(context: BuilderContext, projectName: s
     tsConfig: string;
     reporters: string[] | undefined;
     browsers: string[] | undefined;
-    watch: boolean | undefined;
+    watch: boolean;
+    debug: boolean;
     providersFile: string | undefined;
 }>;

package/src/builders/unit-test/options.js

@@ -15,6 +15,7 @@ const architect_1 = require("@angular-devkit/architect");
 const node_path_1 = __importDefault(require("node:path"));
 const normalize_cache_1 = require("../../utils/normalize-cache");
 const project_metadata_1 = require("../../utils/project-metadata");
+const tty_1 = require("../../utils/tty");
 async function normalizeOptions(context, projectName, options) {
     // Setup base paths based on workspace root and project information
     const workspaceRoot = context.workspaceRoot;
@@ -26,7 +27,7 @@ async function normalizeOptions(context, projectName, options) {
     // Target specifier defaults to the current project's build target using a development configuration
     const buildTargetSpecifier = options.buildTarget ?? `::development`;
     const buildTarget = (0, architect_1.targetFromTargetString)(buildTargetSpecifier, projectName, 'build');
-    const { codeCoverage, codeCoverageExclude, tsConfig, runner, reporters, browsers, watch } = options;
+    const { codeCoverage, codeCoverageExclude, tsConfig, runner, reporters, browsers } = options;
     return {
         // Project/workspace information
         workspaceRoot,
@@ -43,20 +44,8 @@ async function normalizeOptions(context, projectName, options) {
         tsConfig,
         reporters,
         browsers,
-        watch,
+        watch: options.watch ?? (0, tty_1.isTTY)(),
+        debug: options.debug ?? false,
         providersFile: options.providersFile && node_path_1.default.join(workspaceRoot, options.providersFile),
     };
 }
-/**
- * Normalize a directory path string.
- * Currently only removes a trailing slash if present.
- * @param path A path string.
- * @returns A normalized path string.
- */
-function normalizeDirectoryPath(path) {
-    const last = path[path.length - 1];
-    if (last === '/' || last === '\\') {
-        return path.slice(0, -1);
-    }
-    return path;
-}

package/src/builders/unit-test/schema.d.ts

@@ -21,6 +21,10 @@ export type Schema = {
      * Globs to exclude from code coverage.
      */
     codeCoverageExclude?: string[];
+    /**
+     * Initialize the test runner to support using the Node Inspector for test debugging.
+     */
+    debug?: boolean;
     /**
      * Globs of files to exclude, relative to the project root.
      */

package/src/builders/unit-test/schema.json

@@ -46,6 +46,11 @@
       "type": "boolean",
       "description": "Run build when files change."
     },
+    "debug": {
+      "type": "boolean",
+      "description": "Initialize the test runner to support using the Node Inspector for test debugging.",
+      "default": false
+    },
     "codeCoverage": {
       "type": "boolean",
       "description": "Output a code coverage report.",

package/src/tools/esbuild/utils.js

@@ -315,7 +315,7 @@ function transformSupportedBrowsersToTargets(supportedBrowsers) {
     }
     return transformed;
 }
-const SUPPORTED_NODE_VERSIONS = '^20.11.1 || ^22.11.0 || >=24.0.0';
+const SUPPORTED_NODE_VERSIONS = '^20.19.0 || ^22.12.0 || >=24.0.0';
 /**
  * Transform supported Node.js versions to esbuild target.
  * @see https://esbuild.github.io/api/#target

package/src/utils/i18n-options.d.ts

@@ -5,7 +5,6 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.dev/license
  */
-import { DiagnosticHandlingStrategy } from '@angular/localize/tools';
 import type { TranslationLoader } from './load-translations';
 export interface LocaleDescription {
     files: {
@@ -25,7 +24,6 @@ export interface I18nOptions {
     flatOutput?: boolean;
     readonly shouldInline: boolean;
     hasDefinedSourceLocale?: boolean;
-    i18nDuplicateTranslation?: DiagnosticHandlingStrategy;
 }
 export declare function createI18nOptions(projectMetadata: {
     i18n?: unknown;

package/src/utils/load-proxy-config.js

@@ -78,8 +78,7 @@ async function loadProxyConfiguration(root, proxyConfig) {
             // Load the ESM configuration file using the TypeScript dynamic import workaround.
             // Once TypeScript provides support for keeping the dynamic import this workaround can be
             // changed to a direct dynamic import.
-            proxyConfiguration = (await (0, load_esm_1.loadEsmModule)((0, node_url_1.pathToFileURL)(proxyPath)))
-                .default;
+            proxyConfiguration = await (0, load_esm_1.loadEsmModule)((0, node_url_1.pathToFileURL)(proxyPath));
             break;
         case '.cjs':
             proxyConfiguration = require(proxyPath);
@@ -97,13 +96,15 @@ async function loadProxyConfiguration(root, proxyConfig) {
                     // Load the ESM configuration file using the TypeScript dynamic import workaround.
                     // Once TypeScript provides support for keeping the dynamic import this workaround can be
                     // changed to a direct dynamic import.
-                    proxyConfiguration = (await (0, load_esm_1.loadEsmModule)((0, node_url_1.pathToFileURL)(proxyPath)))
-                        .default;
+                    proxyConfiguration = await (0, load_esm_1.loadEsmModule)((0, node_url_1.pathToFileURL)(proxyPath));
                     break;
                 }
                 throw e;
             }
     }
+    if ('default' in proxyConfiguration) {
+        proxyConfiguration = proxyConfiguration.default;
+    }
     return normalizeProxyConfiguration(proxyConfiguration);
 }
 /**

package/src/utils/normalize-cache.js

@@ -10,7 +10,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizeCacheOptions = normalizeCacheOptions;
 const node_path_1 = require("node:path");
 /** Version placeholder is replaced during the build process with actual package version */
-const VERSION = '20.0.0-rc.2';
+const VERSION = '20.0.0-rc.3';
 function hasCacheMetadata(value) {
     return (!!value &&
         typeof value === 'object' &&