@angular-devkit/build-angular 17.1.2 → 17.2.0-next.1

package/package.json

@@ -1,90 +1,89 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "17.1.2",
+  "version": "17.2.0-next.1",
   "description": "Angular Webpack Build Facade",
   "main": "src/index.js",
   "typings": "src/index.d.ts",
   "builders": "builders.json",
   "dependencies": {
     "@ampproject/remapping": "2.2.1",
-    "@angular-devkit/architect": "0.1701.2",
-    "@angular-devkit/build-webpack": "0.1701.2",
-    "@angular-devkit/core": "17.1.2",
-    "@babel/core": "7.23.7",
+    "@angular-devkit/architect": "0.1702.0-next.1",
+    "@angular-devkit/build-webpack": "0.1702.0-next.1",
+    "@angular-devkit/core": "17.2.0-next.1",
+    "@babel/core": "7.23.9",
     "@babel/generator": "7.23.6",
     "@babel/helper-annotate-as-pure": "7.22.5",
     "@babel/helper-split-export-declaration": "7.22.6",
-    "@babel/plugin-transform-async-generator-functions": "7.23.7",
+    "@babel/plugin-transform-async-generator-functions": "7.23.9",
     "@babel/plugin-transform-async-to-generator": "7.23.3",
-    "@babel/plugin-transform-runtime": "7.23.7",
-    "@babel/preset-env": "7.23.7",
-    "@babel/runtime": "7.23.7",
+    "@babel/plugin-transform-runtime": "7.23.9",
+    "@babel/preset-env": "7.23.9",
+    "@babel/runtime": "7.23.9",
     "@discoveryjs/json-ext": "0.5.7",
-    "@ngtools/webpack": "17.1.2",
-    "@vitejs/plugin-basic-ssl": "1.0.2",
+    "@ngtools/webpack": "17.2.0-next.1",
+    "@vitejs/plugin-basic-ssl": "1.1.0",
     "ansi-colors": "4.1.3",
-    "autoprefixer": "10.4.16",
+    "autoprefixer": "10.4.17",
     "babel-loader": "9.1.3",
     "babel-plugin-istanbul": "6.1.1",
     "browserslist": "^4.21.5",
-    "copy-webpack-plugin": "11.0.0",
+    "copy-webpack-plugin": "12.0.2",
     "critters": "0.0.20",
-    "css-loader": "6.8.1",
-    "esbuild-wasm": "0.19.11",
+    "css-loader": "6.10.0",
+    "esbuild-wasm": "0.20.0",
     "fast-glob": "3.3.2",
     "https-proxy-agent": "7.0.2",
     "http-proxy-middleware": "2.0.6",
-    "inquirer": "9.2.12",
-    "jsonc-parser": "3.2.0",
+    "inquirer": "9.2.13",
+    "jsonc-parser": "3.2.1",
     "karma-source-map-support": "1.4.0",
     "less": "4.2.0",
     "less-loader": "11.1.0",
     "license-webpack-plugin": "4.0.2",
     "loader-utils": "3.2.1",
     "magic-string": "0.30.5",
-    "mini-css-extract-plugin": "2.7.6",
+    "mini-css-extract-plugin": "2.7.7",
     "mrmime": "2.0.0",
     "open": "8.4.2",
     "ora": "5.4.1",
     "parse5-html-rewriting-stream": "7.0.0",
     "picomatch": "3.0.1",
-    "piscina": "4.2.1",
+    "piscina": "4.3.1",
     "postcss": "8.4.33",
-    "postcss-loader": "7.3.4",
+    "postcss-loader": "8.1.0",
     "resolve-url-loader": "5.0.0",
     "rxjs": "7.8.1",
-    "sass": "1.69.7",
-    "sass-loader": "13.3.3",
+    "sass": "1.70.0",
+    "sass-loader": "14.1.0",
     "semver": "7.5.4",
     "source-map-loader": "5.0.0",
     "source-map-support": "0.5.21",
-    "terser": "5.26.0",
-    "text-table": "0.2.0",
+    "terser": "5.27.0",
     "tree-kill": "1.2.2",
     "tslib": "2.6.2",
-    "undici": "6.2.1",
+    "undici": "6.5.0",
     "vite": "5.0.12",
     "watchpack": "2.4.0",
-    "webpack": "5.89.0",
+    "webpack": "5.90.0",
     "webpack-dev-middleware": "6.1.1",
     "webpack-dev-server": "4.15.1",
     "webpack-merge": "5.10.0",
     "webpack-subresource-integrity": "5.1.0"
   },
   "optionalDependencies": {
-    "esbuild": "0.19.11"
+    "esbuild": "0.20.0"
   },
   "peerDependencies": {
-    "@angular/compiler-cli": "^17.0.0",
-    "@angular/localize": "^17.0.0",
-    "@angular/platform-server": "^17.0.0",
-    "@angular/service-worker": "^17.0.0",
+    "@angular/compiler-cli": "^17.0.0 || ^17.2.0-next.0",
+    "@angular/localize": "^17.0.0 || ^17.2.0-next.0",
+    "@angular/platform-server": "^17.0.0 || ^17.2.0-next.0",
+    "@angular/service-worker": "^17.0.0 || ^17.2.0-next.0",
     "@web/test-runner": "^0.18.0",
     "browser-sync": "^3.0.2",
     "jest": "^29.5.0",
     "jest-environment-jsdom": "^29.5.0",
     "karma": "^6.3.0",
-    "ng-packagr": "^17.0.0",
+    "ng-packagr": "^17.0.0 || ^17.2.0-next.0",
     "protractor": "^7.0.0",
     "tailwindcss": "^2.0.0 || ^3.0.0",
     "typescript": ">=5.2 <5.4"

package/src/builders/application/build-action.js

@@ -54,8 +54,6 @@ async function* runEsBuildBuildAction(action, options) {
     try {
         // Perform the build action
         result = await withProgress('Building...', () => action());
-        // Log all diagnostic (error/warning) messages from the build
-        await (0, utils_1.logMessages)(logger, result);
     }
     finally {
         // Ensure Sass workers are shutdown if not watching
@@ -142,8 +140,6 @@ async function* runEsBuildBuildAction(action, options) {
                 logger.info(changes.toDebugString());
             }
             result = await withProgress('Changes detected. Rebuilding...', () => action(result.createRebuildState(changes)));
-            // Log all diagnostic (error/warning) messages from the rebuild
-            await (0, utils_1.logMessages)(logger, result);
             // Update watched locations provided by the new build result.
             // Keep watching all previous files if there are any errors; otherwise consider all
             // files stale until confirmed present in the new result's watch files.

package/src/builders/application/execute-build.js

@@ -16,14 +16,13 @@ const commonjs_checker_1 = require("../../tools/esbuild/commonjs-checker");
 const license_extractor_1 = require("../../tools/esbuild/license-extractor");
 const utils_1 = require("../../tools/esbuild/utils");
 const bundle_calculator_1 = require("../../utils/bundle-calculator");
-const color_1 = require("../../utils/color");
 const copy_assets_1 = require("../../utils/copy-assets");
 const supported_browsers_1 = require("../../utils/supported-browsers");
 const execute_post_bundle_1 = require("./execute-post-bundle");
 const i18n_1 = require("./i18n");
 const setup_bundling_1 = require("./setup-bundling");
 async function executeBuild(options, context, rebuildState) {
-    const { projectRoot, workspaceRoot, i18nOptions, optimizationOptions, assets, cacheOptions, prerenderOptions, } = options;
+    const { projectRoot, workspaceRoot, i18nOptions, optimizationOptions, assets, cacheOptions, prerenderOptions, ssrOptions, verbose, colors, jsonLogs, } = options;
     // TODO: Consider integrating into watch mode. Would require full rebuild on target changes.
     const browsers = (0, supported_browsers_1.getSupportedBrowsers)(projectRoot, context.logger);
     // Load active translations if inlining
@@ -96,12 +95,11 @@ async function executeBuild(options, context, rebuildState) {
         executionResult.addOutputFile('3rdpartylicenses.txt', await (0, license_extractor_1.extractLicenses)(metafile, workspaceRoot), bundler_context_1.BuildOutputFileType.Root);
     }
     // Perform i18n translation inlining if enabled
-    let prerenderedRoutes;
     if (i18nOptions.shouldInline) {
         const result = await (0, i18n_1.inlineI18n)(options, executionResult, initialFiles);
         executionResult.addErrors(result.errors);
         executionResult.addWarnings(result.warnings);
-        prerenderedRoutes = result.prerenderedRoutes;
+        executionResult.addPrerenderedRoutes(result.prerenderedRoutes);
     }
     else {
         const result = await (0, execute_post_bundle_1.executePostBundleSteps)(options, executionResult.outputFiles, executionResult.assetFiles, initialFiles, 
@@ -109,26 +107,21 @@ async function executeBuild(options, context, rebuildState) {
         i18nOptions.hasDefinedSourceLocale ? i18nOptions.sourceLocale : undefined);
         executionResult.addErrors(result.errors);
         executionResult.addWarnings(result.warnings);
-        prerenderedRoutes = result.prerenderedRoutes;
+        executionResult.addPrerenderedRoutes(result.prerenderedRoutes);
         executionResult.outputFiles.push(...result.additionalOutputFiles);
         executionResult.assetFiles.push(...result.additionalAssets);
     }
     if (prerenderOptions) {
-        executionResult.addOutputFile('prerendered-routes.json', JSON.stringify({ routes: prerenderedRoutes.sort((a, b) => a.localeCompare(b)) }, null, 2), bundler_context_1.BuildOutputFileType.Root);
-        let prerenderMsg = `Prerendered ${prerenderedRoutes.length} static route`;
-        if (prerenderedRoutes.length > 1) {
-            prerenderMsg += 's.';
-        }
-        else {
-            prerenderMsg += '.';
-        }
-        context.logger.info(color_1.colors.magenta(prerenderMsg) + '\n');
+        const prerenderedRoutes = executionResult.prerenderedRoutes;
+        executionResult.addOutputFile('prerendered-routes.json', JSON.stringify({ routes: prerenderedRoutes }, null, 2), bundler_context_1.BuildOutputFileType.Root);
     }
-    (0, utils_1.logBuildStats)(context.logger, metafile, initialFiles, budgetFailures, changedFiles, estimatedTransferSizes);
     // Write metafile if stats option is enabled
     if (options.stats) {
         executionResult.addOutputFile('stats.json', JSON.stringify(metafile, null, 2), bundler_context_1.BuildOutputFileType.Root);
     }
+    if (!jsonLogs) {
+        context.logger.info((0, utils_1.logBuildStats)(metafile, initialFiles, budgetFailures, colors, changedFiles, estimatedTransferSizes, !!ssrOptions, verbose));
+    }
     return executionResult;
 }
 exports.executeBuild = executeBuild;

package/src/builders/application/index.js

@@ -10,6 +10,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.buildApplication = exports.buildApplicationInternal = void 0;
 const architect_1 = require("@angular-devkit/architect");
 const bundler_context_1 = require("../../tools/esbuild/bundler-context");
+const utils_1 = require("../../tools/esbuild/utils");
+const color_1 = require("../../utils/color");
 const purge_cache_1 = require("../../utils/purge-cache");
 const version_1 = require("../../utils/version");
 const build_action_1 = require("./build-action");
@@ -57,11 +59,25 @@ context, infrastructureSettings, extensions) {
         context.addTeardown(() => controller.abort('builder-teardown'));
     }
     yield* (0, build_action_1.runEsBuildBuildAction)(async (rebuildState) => {
+        const { prerenderOptions, outputOptions, jsonLogs } = normalizedOptions;
         const startTime = process.hrtime.bigint();
         const result = await (0, execute_build_1.executeBuild)(normalizedOptions, context, rebuildState);
-        const buildTime = Number(process.hrtime.bigint() - startTime) / 10 ** 9;
-        const status = result.errors.length > 0 ? 'failed' : 'complete';
-        logger.info(`Application bundle generation ${status}. [${buildTime.toFixed(3)} seconds]`);
+        if (!jsonLogs) {
+            if (prerenderOptions) {
+                const prerenderedRoutesLength = result.prerenderedRoutes.length;
+                let prerenderMsg = `Prerendered ${prerenderedRoutesLength} static route`;
+                prerenderMsg += prerenderedRoutesLength !== 1 ? 's.' : '.';
+                logger.info(color_1.colors.magenta(prerenderMsg));
+            }
+            const buildTime = Number(process.hrtime.bigint() - startTime) / 10 ** 9;
+            const hasError = result.errors.length > 0;
+            if (writeToFileSystem && !hasError) {
+                logger.info(`Output location: ${outputOptions.base}\n`);
+            }
+            logger.info(`Application bundle generation ${hasError ? 'failed' : 'complete'}. [${buildTime.toFixed(3)} seconds]`);
+        }
+        // Log all diagnostic (error/warning) messages
+        await (0, utils_1.logMessages)(logger, result, normalizedOptions);
         return result;
     }, {
         watch: normalizedOptions.watch,

package/src/builders/application/options.d.ts

@@ -127,6 +127,7 @@ export declare function normalizeOptions(context: BuilderContext, projectName: s
         file: string;
         package: string;
     } | undefined;
+    postcssConfiguration: import("../../utils/postcss-configuration").PostcssConfiguration | undefined;
     i18nOptions: I18nOptions & {
         duplicateTranslationBehavior?: I18NTranslation | undefined;
         missingTranslationBehavior?: I18NTranslation | undefined;
@@ -136,5 +137,7 @@ export declare function normalizeOptions(context: BuilderContext, projectName: s
     publicPath: string | undefined;
     plugins: Plugin[] | undefined;
     loaderExtensions: Record<string, "binary" | "file" | "text"> | undefined;
+    jsonLogs: boolean;
+    colors: boolean;
 }>;
 export {};

package/src/builders/application/options.js

@@ -17,9 +17,12 @@ const node_module_1 = require("node:module");
 const node_path_1 = __importDefault(require("node:path"));
 const helpers_1 = require("../../tools/webpack/utils/helpers");
 const utils_1 = require("../../utils");
+const color_1 = require("../../utils/color");
+const environment_options_1 = require("../../utils/environment-options");
 const i18n_options_1 = require("../../utils/i18n-options");
 const normalize_cache_1 = require("../../utils/normalize-cache");
 const package_chunk_sort_1 = require("../../utils/package-chunk-sort");
+const postcss_configuration_1 = require("../../utils/postcss-configuration");
 const tailwind_1 = require("../../utils/tailwind");
 const webpack_browser_config_1 = require("../../utils/webpack-browser-config");
 const schema_1 = require("./schema");
@@ -108,6 +111,11 @@ async function normalizeOptions(context, projectName, options, extensions) {
             loaderExtensions[extension] = value;
         }
     }
+    const postcssConfiguration = await (0, postcss_configuration_1.loadPostcssConfiguration)(workspaceRoot, projectRoot);
+    // Skip tailwind configuration if postcss is customized
+    const tailwindConfiguration = postcssConfiguration
+        ? undefined
+        : await getTailwindConfig(workspaceRoot, projectRoot, context);
     const globalStyles = [];
     if (options.styles?.length) {
         const { entryPoints: stylesheetEntrypoints, noInjectNames } = (0, helpers_1.normalizeGlobalStyles)(options.styles || []);
@@ -212,13 +220,16 @@ async function normalizeOptions(context, projectName, options, extensions) {
         globalScripts,
         serviceWorker: typeof serviceWorker === 'string' ? node_path_1.default.join(workspaceRoot, serviceWorker) : undefined,
         indexHtmlOptions,
-        tailwindConfiguration: await getTailwindConfig(workspaceRoot, projectRoot, context),
+        tailwindConfiguration,
+        postcssConfiguration,
         i18nOptions,
         namedChunks,
         budgets: budgets?.length ? budgets : undefined,
         publicPath: deployUrl ? deployUrl : undefined,
         plugins: extensions?.codePlugins?.length ? extensions?.codePlugins : undefined,
         loaderExtensions,
+        jsonLogs: environment_options_1.useJSONBuildLogs,
+        colors: color_1.colors.enabled,
     };
 }
 exports.normalizeOptions = normalizeOptions;

package/src/builders/application/setup-bundling.js

@@ -59,11 +59,11 @@ function setupBundlerContexts(options, browsers, codeBundleCache) {
             // Disable external deps for server bundles.
             // This is because it breaks Vite 'optimizeDeps' for SSR.
             externalPackages: false,
-        }, nodeTargets, codeBundleCache), () => false));
+        }, nodeTargets, codeBundleCache)));
         // Server polyfills code
         const serverPolyfillBundleOptions = (0, application_code_bundle_1.createServerPolyfillBundleOptions)(options, nodeTargets, codeBundleCache);
         if (serverPolyfillBundleOptions) {
-            bundlerContexts.push(new bundler_context_1.BundlerContext(workspaceRoot, !!options.watch, serverPolyfillBundleOptions, () => false));
+            bundlerContexts.push(new bundler_context_1.BundlerContext(workspaceRoot, !!options.watch, serverPolyfillBundleOptions));
         }
     }
     return bundlerContexts;

package/src/builders/dev-server/options.js

@@ -29,8 +29,9 @@ async function normalizeOptions(context, projectName, options) {
     const projectMetadata = await context.getProjectMetadata(projectName);
     const projectRoot = node_path_1.default.join(workspaceRoot, projectMetadata.root ?? '');
     const cacheOptions = (0, normalize_cache_1.normalizeCacheOptions)(projectMetadata, workspaceRoot);
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-    const buildTarget = (0, architect_1.targetFromTargetString)(options.buildTarget ?? options.browserTarget);
+    // Target specifier defaults to the current project's build target using a development configuration
+    const buildTargetSpecifier = options.buildTarget ?? options.browserTarget ?? `::development`;
+    const buildTarget = (0, architect_1.targetFromTargetString)(buildTargetSpecifier, projectName, 'build');
     // Initial options to keep
     const { host, port, poll, open, verbose, watch, allowedHosts, disableHostCheck, liveReload, hmr, headers, proxyConfig, servePath, publicHost, ssl, sslCert, sslKey, forceEsbuild, } = options;
     // Return all the normalized options

package/src/builders/dev-server/schema.json

@@ -13,7 +13,7 @@
     "buildTarget": {
       "type": "string",
       "description": "A build builder target to serve in the format of `project:target[:configuration]`. You can also pass in more than one configuration name as a comma-separated list. Example: `project:target:production,staging`.",
-      "pattern": "^[^:\\s]+:[^:\\s]+(:[^\\s]+)?$"
+      "pattern": "^[^:\\s]*:[^:\\s]*(:[^\\s]+)?$"
     },
     "port": {
       "type": "number",

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

@@ -30,8 +30,9 @@ async function normalizeOptions(context, projectName, options) {
     const workspaceRoot = context.workspaceRoot;
     const projectMetadata = await context.getProjectMetadata(projectName);
     const projectRoot = node_path_1.default.join(workspaceRoot, projectMetadata.root ?? '');
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-    const buildTarget = (0, architect_1.targetFromTargetString)(options.buildTarget ?? options.browserTarget);
+    // Target specifier defaults to the current project's build target with no specified configuration
+    const buildTargetSpecifier = options.buildTarget ?? options.browserTarget ?? ':';
+    const buildTarget = (0, architect_1.targetFromTargetString)(buildTargetSpecifier, projectName, 'build');
     const i18nOptions = (0, i18n_options_1.createI18nOptions)(projectMetadata);
     // Normalize xliff format extensions
     let format = options.format;

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

@@ -13,7 +13,7 @@
     "buildTarget": {
       "type": "string",
       "description": "A builder target to extract i18n messages in the format of `project:target[:configuration]`. You can also pass in more than one configuration name as a comma-separated list. Example: `project:target:production,staging`.",
-      "pattern": "^[^:\\s]+:[^:\\s]+(:[^\\s]+)?$"
+      "pattern": "^[^:\\s]*:[^:\\s]*(:[^\\s]+)?$"
     },
     "format": {
       "type": "string",

package/src/builders/jest/index.js

@@ -31,15 +31,16 @@ var __importStar = (this && this.__importStar) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const architect_1 = require("@angular-devkit/architect");
-const child_process_1 = require("child_process");
-const path = __importStar(require("path"));
-const util_1 = require("util");
+const node_child_process_1 = require("node:child_process");
+const fs = __importStar(require("node:fs/promises"));
+const path = __importStar(require("node:path"));
+const node_util_1 = require("node:util");
 const color_1 = require("../../utils/color");
 const test_files_1 = require("../../utils/test-files");
 const application_1 = require("../application");
 const schema_1 = require("../browser-esbuild/schema");
 const options_1 = require("./options");
-const execFile = (0, util_1.promisify)(child_process_1.execFile);
+const execFile = (0, node_util_1.promisify)(node_child_process_1.execFile);
 /** Main execution function for the Jest builder. */
 exports.default = (0, architect_1.createBuilder)(async (schema, context) => {
     context.logger.warn('NOTE: The Jest builder is currently EXPERIMENTAL and not ready for production use.');
@@ -65,8 +66,21 @@ exports.default = (0, architect_1.createBuilder)(async (schema, context) => {
             error: '`jest-environment-jsdom` is not installed. Install it with `npm install jest-environment-jsdom --save-dev`.',
         };
     }
+    const [testFiles, customConfig] = await Promise.all([
+        (0, test_files_1.findTestFiles)(options.include, options.exclude, context.workspaceRoot),
+        findCustomJestConfig(context.workspaceRoot),
+    ]);
+    // Warn if a custom Jest configuration is found. We won't use it, so if a developer is trying to use a custom config, this hopefully
+    // makes a better experience than silently ignoring the configuration.
+    // Ideally, this would be a hard error. However a Jest config could exist for testing other files in the workspace outside of Angular
+    // CLI, so we likely can't produce a hard error in this situation without an opt-out.
+    if (customConfig) {
+        context.logger.warn('A custom Jest config was found, but this is not supported by `@angular-devkit/build-angular:jest` and will be' +
+            ` ignored: ${customConfig}. This is an experiment to see if completely abstracting away Jest's configuration is viable. Please` +
+            ` consider if your use case can be met without directly modifying the Jest config. If this is a major obstacle for your use` +
+            ` case, please post it in this issue so we can collect feedback and evaluate: https://github.com/angular/angular-cli/issues/25434.`);
+    }
     // Build all the test files.
-    const testFiles = await (0, test_files_1.findTestFiles)(options.include, options.exclude, context.workspaceRoot);
     const jestGlobal = path.join(__dirname, 'jest-global.mjs');
     const initTestBed = path.join(__dirname, 'init-test-bed.mjs');
     const buildResult = await build(context, {
@@ -94,6 +108,7 @@ exports.default = (0, architect_1.createBuilder)(async (schema, context) => {
         '--experimental-vm-modules',
         jest,
         `--rootDir="${path.join(testOut, 'browser')}"`,
+        `--config=${path.join(__dirname, 'jest.config.mjs')}`,
         '--testEnvironment=jsdom',
         // TODO(dgp1130): Enable cache once we have a mechanism for properly clearing / disabling it.
         '--no-cache',
@@ -158,3 +173,27 @@ function resolveModule(module) {
         return undefined;
     }
 }
+/** Returns whether or not the provided directory includes a Jest configuration file. */
+async function findCustomJestConfig(dir) {
+    const entries = await fs.readdir(dir, { withFileTypes: true });
+    // Jest supports many file extensions (`js`, `ts`, `cjs`, `cts`, `json`, etc.) Just look
+    // for anything with that prefix.
+    const config = entries.find((entry) => entry.isFile() && entry.name.startsWith('jest.config.'));
+    if (config) {
+        return path.join(dir, config.name);
+    }
+    // Jest also supports a `jest` key in `package.json`, look for a config there.
+    const packageJsonPath = path.join(dir, 'package.json');
+    let packageJson;
+    try {
+        packageJson = await fs.readFile(packageJsonPath, 'utf8');
+    }
+    catch {
+        return undefined; // No package.json, therefore no Jest configuration in it.
+    }
+    const json = JSON.parse(packageJson);
+    if ('jest' in json) {
+        return packageJsonPath;
+    }
+    return undefined;
+}

package/src/builders/jest/jest.config.mjs

@@ -0,0 +1,11 @@
+/**
+ * @license
+ * Copyright Google LLC All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+// Empty config file, everything is specified via CLI options right now.
+// This file is used just so Jest doesn't accidentally inherit a custom user-specified Jest config.
+export default {};

package/src/tools/babel/plugins/elide-angular-metadata.d.ts

@@ -7,7 +7,7 @@
  */
 /// <reference path="../../../../../../../../../../packages/angular_devkit/build_angular/src/babel-bazel.d.ts" />
 /// <reference types="@angular/compiler-cli/private/babel" />
-import { PluginObj } from '@babel/core';
+import type { PluginObj } from '@babel/core';
 /**
  * Provides one or more keywords that if found within the content of a source file indicate
  * that this plugin should be used with a source file.

package/src/tools/babel/plugins/elide-angular-metadata.js

@@ -8,7 +8,6 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getKeywords = void 0;
-const core_1 = require("@babel/core");
 /**
  * The name of the Angular class metadata function created by the Angular compiler.
  */
@@ -28,9 +27,18 @@ const SET_CLASS_DEBUG_INFO_NAME = 'ɵsetClassDebugInfo';
  * @returns An a string iterable containing one or more keywords.
  */
 function getKeywords() {
-    return [SET_CLASS_METADATA_NAME, SET_CLASS_METADATA_ASYNC_NAME, SET_CLASS_DEBUG_INFO_NAME];
+    return Object.keys(angularMetadataFunctions);
 }
 exports.getKeywords = getKeywords;
+/**
+ * An object map of function names and related value checks for discovery of Angular generated
+ * metadata calls.
+ */
+const angularMetadataFunctions = {
+    [SET_CLASS_METADATA_NAME]: isSetClassMetadataCall,
+    [SET_CLASS_METADATA_ASYNC_NAME]: isSetClassMetadataAsyncCall,
+    [SET_CLASS_DEBUG_INFO_NAME]: isSetClassDebugInfoCall,
+};
 /**
  * A babel plugin factory function for eliding the Angular class metadata function (`ɵsetClassMetadata`).
  *
@@ -40,20 +48,23 @@ function default_1() {
     return {
         visitor: {
             CallExpression(path) {
-                const callee = path.node.callee;
-                const callArguments = path.node.arguments;
+                const callee = path.get('callee');
                 // The function being called must be the metadata function name
                 let calleeName;
-                if (core_1.types.isMemberExpression(callee) && core_1.types.isIdentifier(callee.property)) {
-                    calleeName = callee.property.name;
+                if (callee.isMemberExpression()) {
+                    const calleeProperty = callee.get('property');
+                    if (calleeProperty.isIdentifier()) {
+                        calleeName = calleeProperty.node.name;
+                    }
+                }
+                else if (callee.isIdentifier()) {
+                    calleeName = callee.node.name;
                 }
-                else if (core_1.types.isIdentifier(callee)) {
-                    calleeName = callee.name;
+                if (!calleeName) {
+                    return;
                 }
-                if (calleeName !== undefined &&
-                    (isRemoveClassMetadataCall(calleeName, callArguments) ||
-                        isRemoveClassmetadataAsyncCall(calleeName, callArguments) ||
-                        isSetClassDebugInfoCall(calleeName, callArguments))) {
+                if (Object.hasOwn(angularMetadataFunctions, calleeName) &&
+                    angularMetadataFunctions[calleeName](path.get('arguments'))) {
                     // The metadata function is always emitted inside a function expression
                     const parent = path.getFunctionParent();
                     if (parent && (parent.isFunctionExpression() || parent.isArrowFunctionExpression())) {
@@ -68,35 +79,32 @@ function default_1() {
 }
 exports.default = default_1;
 /** Determines if a function call is a call to `setClassMetadata`. */
-function isRemoveClassMetadataCall(name, args) {
+function isSetClassMetadataCall(callArguments) {
     // `setClassMetadata` calls have to meet the following criteria:
     // * First must be an identifier
     // * Second must be an array literal
-    return (name === SET_CLASS_METADATA_NAME &&
-        args.length === 4 &&
-        core_1.types.isIdentifier(args[0]) &&
-        core_1.types.isArrayExpression(args[1]));
+    return (callArguments.length === 4 &&
+        callArguments[0].isIdentifier() &&
+        callArguments[1].isArrayExpression());
 }
 /** Determines if a function call is a call to `setClassMetadataAsync`. */
-function isRemoveClassmetadataAsyncCall(name, args) {
+function isSetClassMetadataAsyncCall(callArguments) {
     // `setClassMetadataAsync` calls have to meet the following criteria:
     // * First argument must be an identifier.
     // * Second argument must be an inline function.
     // * Third argument must be an inline function.
-    return (name === SET_CLASS_METADATA_ASYNC_NAME &&
-        args.length === 3 &&
-        core_1.types.isIdentifier(args[0]) &&
-        isInlineFunction(args[1]) &&
-        isInlineFunction(args[2]));
+    return (callArguments.length === 3 &&
+        callArguments[0].isIdentifier() &&
+        isInlineFunction(callArguments[1]) &&
+        isInlineFunction(callArguments[2]));
 }
 /** Determines if a function call is a call to `setClassDebugInfo`. */
-function isSetClassDebugInfoCall(name, args) {
-    return (name === SET_CLASS_DEBUG_INFO_NAME &&
-        args.length === 2 &&
-        core_1.types.isIdentifier(args[0]) &&
-        core_1.types.isObjectExpression(args[1]));
+function isSetClassDebugInfoCall(callArguments) {
+    return (callArguments.length === 2 &&
+        callArguments[0].isIdentifier() &&
+        callArguments[1].isObjectExpression());
 }
 /** Determines if a node is an inline function expression. */
-function isInlineFunction(node) {
-    return core_1.types.isFunctionExpression(node) || core_1.types.isArrowFunctionExpression(node);
+function isInlineFunction(path) {
+    return path.isFunctionExpression() || path.isArrowFunctionExpression();
 }

package/src/tools/babel/plugins/pure-toplevel-functions.d.ts

@@ -7,7 +7,7 @@
  */
 /// <reference path="../../../../../../../../../../packages/angular_devkit/build_angular/src/babel-bazel.d.ts" />
 /// <reference types="@angular/compiler-cli/private/babel" />
-import { PluginObj } from '@babel/core';
+import type { PluginObj } from '@babel/core';
 /**
  * A babel plugin factory function for adding the PURE annotation to top-level new and call expressions.
  *

package/src/tools/babel/plugins/pure-toplevel-functions.js

@@ -33,7 +33,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const core_1 = require("@babel/core");
 const helper_annotate_as_pure_1 = __importDefault(require("@babel/helper-annotate-as-pure"));
 const tslib = __importStar(require("tslib"));
 /**
@@ -67,14 +66,14 @@ function default_1() {
                 if (path.getFunctionParent()) {
                     return;
                 }
-                const callee = path.node.callee;
-                if ((core_1.types.isFunctionExpression(callee) || core_1.types.isArrowFunctionExpression(callee)) &&
+                const callee = path.get('callee');
+                if ((callee.isFunctionExpression() || callee.isArrowFunctionExpression()) &&
                     path.node.arguments.length !== 0) {
                     return;
                 }
                 // Do not annotate TypeScript helpers emitted by the TypeScript compiler.
                 // TypeScript helpers are intended to cause side effects.
-                if (core_1.types.isIdentifier(callee) && isTslibHelperName(callee.name)) {
+                if (callee.isIdentifier() && isTslibHelperName(callee.node.name)) {
                     return;
                 }
                 (0, helper_annotate_as_pure_1.default)(path);