@angular-devkit/build-angular 17.1.0-next.0 → 17.1.0-next.2

package/package.json

@@ -1,38 +1,36 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "17.1.0-next.0",
+  "version": "17.1.0-next.2",
   "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.0-next.0",
-    "@angular-devkit/build-webpack": "0.1701.0-next.0",
-    "@angular-devkit/core": "17.1.0-next.0",
-    "@babel/core": "7.23.5",
-    "@babel/generator": "7.23.5",
+    "@angular-devkit/architect": "0.1701.0-next.2",
+    "@angular-devkit/build-webpack": "0.1701.0-next.2",
+    "@angular-devkit/core": "17.1.0-next.2",
+    "@babel/core": "7.23.6",
+    "@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.4",
     "@babel/plugin-transform-async-to-generator": "7.23.3",
-    "@babel/plugin-transform-runtime": "7.23.4",
-    "@babel/preset-env": "7.23.5",
-    "@babel/runtime": "7.23.5",
+    "@babel/plugin-transform-runtime": "7.23.6",
+    "@babel/preset-env": "7.23.6",
+    "@babel/runtime": "7.23.6",
     "@discoveryjs/json-ext": "0.5.7",
-    "@ngtools/webpack": "17.1.0-next.0",
+    "@ngtools/webpack": "17.1.0-next.2",
     "@vitejs/plugin-basic-ssl": "1.0.2",
     "ansi-colors": "4.1.3",
     "autoprefixer": "10.4.16",
     "babel-loader": "9.1.3",
     "babel-plugin-istanbul": "6.1.1",
     "browserslist": "^4.21.5",
-    "browser-sync": "2.29.3",
-    "chokidar": "3.5.3",
     "copy-webpack-plugin": "11.0.0",
     "critters": "0.0.20",
     "css-loader": "6.8.1",
-    "esbuild-wasm": "0.19.8",
+    "esbuild-wasm": "0.19.9",
     "fast-glob": "3.3.2",
     "https-proxy-agent": "7.0.2",
     "http-proxy-middleware": "2.0.6",
@@ -50,8 +48,8 @@
     "ora": "5.4.1",
     "parse5-html-rewriting-stream": "7.0.0",
     "picomatch": "3.0.1",
-    "piscina": "4.2.0",
-    "postcss": "8.4.31",
+    "piscina": "4.2.1",
+    "postcss": "8.4.32",
     "postcss-loader": "7.3.3",
     "resolve-url-loader": "5.0.0",
     "rxjs": "7.8.1",
@@ -60,12 +58,13 @@
     "semver": "7.5.4",
     "source-map-loader": "4.0.1",
     "source-map-support": "0.5.21",
-    "terser": "5.24.0",
+    "terser": "5.26.0",
     "text-table": "0.2.0",
     "tree-kill": "1.2.2",
     "tslib": "2.6.2",
-    "undici": "5.28.1",
-    "vite": "5.0.4",
+    "undici": "6.0.1",
+    "vite": "5.0.7",
+    "watchpack": "2.4.0",
     "webpack": "5.89.0",
     "webpack-dev-middleware": "6.1.1",
     "webpack-dev-server": "4.15.1",
@@ -73,13 +72,14 @@
     "webpack-subresource-integrity": "5.1.0"
   },
   "optionalDependencies": {
-    "esbuild": "0.19.8"
+    "esbuild": "0.19.9"
   },
   "peerDependencies": {
     "@angular/compiler-cli": "^17.0.0 || ^17.1.0-next.0",
     "@angular/localize": "^17.0.0 || ^17.1.0-next.0",
     "@angular/platform-server": "^17.0.0 || ^17.1.0-next.0",
     "@angular/service-worker": "^17.0.0 || ^17.1.0-next.0",
+    "browser-sync": "^2.29.3",
     "jest": "^29.5.0",
     "jest-environment-jsdom": "^29.5.0",
     "karma": "^6.3.0",
@@ -98,6 +98,9 @@
     "@angular/service-worker": {
       "optional": true
     },
+    "browser-sync": {
+      "optional": true
+    },
     "jest": {
       "optional": true
     },

package/src/builders/application/build-action.d.ts

@@ -24,4 +24,5 @@ export declare function runEsBuildBuildAction(action: (rebuildState?: RebuildSta
     deleteOutputPath?: boolean;
     poll?: number;
     signal?: AbortSignal;
+    preserveSymlinks?: boolean;
 }): AsyncIterable<(ExecutionResult['outputWithFiles'] | ExecutionResult['output']) & BuilderOutput>;

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

@@ -34,15 +34,16 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.runEsBuildBuildAction = void 0;
+const node_fs_1 = require("node:fs");
 const node_path_1 = __importDefault(require("node:path"));
 const sass_language_1 = require("../../tools/esbuild/stylesheets/sass-language");
 const utils_1 = require("../../tools/esbuild/utils");
 const delete_output_dir_1 = require("../../utils/delete-output-dir");
 const environment_options_1 = require("../../utils/environment-options");
 async function* runEsBuildBuildAction(action, options) {
-    const { writeToFileSystemFilter, writeToFileSystem = true, watch, poll, logger, deleteOutputPath, cacheOptions, outputPath, verbose, projectRoot, workspaceRoot, progress, } = options;
+    const { writeToFileSystemFilter, writeToFileSystem = true, watch, poll, logger, deleteOutputPath, cacheOptions, outputPath, verbose, projectRoot, workspaceRoot, progress, preserveSymlinks, } = options;
     if (deleteOutputPath && writeToFileSystem) {
-        await (0, delete_output_dir_1.deleteOutputDir)(workspaceRoot, outputPath);
+        await (0, delete_output_dir_1.deleteOutputDir)(workspaceRoot, outputPath, ['browser', 'server']);
     }
     const withProgress = progress ? utils_1.withSpinner : utils_1.withNoProgress;
     // Initial build
@@ -62,20 +63,25 @@ async function* runEsBuildBuildAction(action, options) {
         if (progress) {
             logger.info('Watch mode enabled. Watching for file changes...');
         }
+        const ignored = [
+            // Ignore the output and cache paths to avoid infinite rebuild cycles
+            outputPath,
+            cacheOptions.basePath,
+            `${workspaceRoot.replace(/\\/g, '/')}/**/.*/**`,
+        ];
+        if (!preserveSymlinks) {
+            // Ignore all node modules directories to avoid excessive file watchers.
+            // Package changes are handled below by watching manifest and lock files.
+            // NOTE: this is not enable when preserveSymlinks is true as this would break `npm link` usages.
+            ignored.push('**/node_modules/**');
+        }
         // Setup a watcher
         const { createWatcher } = await Promise.resolve().then(() => __importStar(require('../../tools/esbuild/watcher')));
         watcher = createWatcher({
             polling: typeof poll === 'number',
             interval: poll,
-            ignored: [
-                // Ignore the output and cache paths to avoid infinite rebuild cycles
-                outputPath,
-                cacheOptions.basePath,
-                // Ignore all node modules directories to avoid excessive file watchers.
-                // Package changes are handled below by watching manifest and lock files.
-                '**/node_modules/**',
-                `${workspaceRoot.replace(/\\/g, '/')}/**/.*/**`,
-            ],
+            followSymlinks: preserveSymlinks,
+            ignored,
         });
         // Setup abort support
         options.signal?.addEventListener('abort', () => void watcher?.close());
@@ -96,7 +102,9 @@ async function* runEsBuildBuildAction(action, options) {
             '.pnp.cjs',
             '.pnp.data.json',
         ];
-        watcher.add(packageWatchFiles.map((file) => node_path_1.default.join(workspaceRoot, file)));
+        watcher.add(packageWatchFiles
+            .map((file) => node_path_1.default.join(workspaceRoot, file))
+            .filter((file) => (0, node_fs_1.existsSync)(file)));
         // Watch locations provided by the initial build result
         watcher.add(result.watchFiles);
     }

package/src/builders/application/execute-post-bundle.js

@@ -39,9 +39,11 @@ async function executePostBundleSteps(options, outputFiles, assetFiles, initialF
      * NOTE: we don't perform critical CSS inlining as this will be done during server rendering.
      */
     let indexContentOutputNoCssInlining;
+    // When using prerender/app-shell the index HTML file can be regenerated.
+    // Thus, we use a Map so that we do not generate 2 files with the same filename.
+    const additionalHtmlOutputFiles = new Map();
     // Generate index HTML file
     // If localization is enabled, index generation is handled in the inlining process.
-    // NOTE: Localization with SSR is not currently supported.
     if (indexHtmlOptions) {
         const { content, contentWithoutCriticalCssInlined, errors, warnings } = await (0, index_html_generator_1.generateIndexHtml)(initialFiles, outputFiles, {
             ...options,
@@ -50,9 +52,10 @@ async function executePostBundleSteps(options, outputFiles, assetFiles, initialF
         indexContentOutputNoCssInlining = contentWithoutCriticalCssInlined;
         allErrors.push(...errors);
         allWarnings.push(...warnings);
-        additionalOutputFiles.push((0, utils_1.createOutputFileFromText)(indexHtmlOptions.output, content, bundler_context_1.BuildOutputFileType.Browser));
+        additionalHtmlOutputFiles.set(indexHtmlOptions.output, (0, utils_1.createOutputFileFromText)(indexHtmlOptions.output, content, bundler_context_1.BuildOutputFileType.Browser));
         if (ssrOptions) {
-            additionalOutputFiles.push((0, utils_1.createOutputFileFromText)('index.server.html', contentWithoutCriticalCssInlined, bundler_context_1.BuildOutputFileType.Server));
+            const serverIndexHtmlFilename = 'index.server.html';
+            additionalHtmlOutputFiles.set(serverIndexHtmlFilename, (0, utils_1.createOutputFileFromText)(serverIndexHtmlFilename, contentWithoutCriticalCssInlined, bundler_context_1.BuildOutputFileType.Server));
         }
     }
     // Pre-render (SSG) and App-shell
@@ -64,9 +67,10 @@ async function executePostBundleSteps(options, outputFiles, assetFiles, initialF
         allWarnings.push(...warnings);
         prerenderedRoutes.push(...Array.from(generatedRoutes));
         for (const [path, content] of Object.entries(output)) {
-            additionalOutputFiles.push((0, utils_1.createOutputFileFromText)(path, content, bundler_context_1.BuildOutputFileType.Browser));
+            additionalHtmlOutputFiles.set(path, (0, utils_1.createOutputFileFromText)(path, content, bundler_context_1.BuildOutputFileType.Browser));
         }
     }
+    additionalOutputFiles.push(...additionalHtmlOutputFiles.values());
     // Augment the application with service worker support
     // If localization is enabled, service worker is handled in the inlining process.
     if (serviceWorker) {

package/src/builders/application/index.js

@@ -29,6 +29,13 @@ context, infrastructureSettings, plugins) {
         return;
     }
     const normalizedOptions = await (0, options_1.normalizeOptions)(context, projectName, options, plugins);
+    // Setup an abort controller with a builder teardown if no signal is present
+    let signal = context.signal;
+    if (!signal) {
+        const controller = new AbortController();
+        signal = controller.signal;
+        context.addTeardown(() => controller.abort('builder-teardown'));
+    }
     yield* (0, build_action_1.runEsBuildBuildAction)(async (rebuildState) => {
         const startTime = process.hrtime.bigint();
         const result = await (0, execute_build_1.executeBuild)(normalizedOptions, context, rebuildState);
@@ -38,6 +45,7 @@ context, infrastructureSettings, plugins) {
         return result;
     }, {
         watch: normalizedOptions.watch,
+        preserveSymlinks: normalizedOptions.preserveSymlinks,
         poll: normalizedOptions.poll,
         deleteOutputPath: normalizedOptions.deleteOutputPath,
         cacheOptions: normalizedOptions.cacheOptions,
@@ -53,7 +61,7 @@ context, infrastructureSettings, plugins) {
             ? undefined
             : (file) => file.type !== bundler_context_1.BuildOutputFileType.Server,
         logger: context.logger,
-        signal: context.signal,
+        signal,
     });
 }
 exports.buildApplicationInternal = buildApplicationInternal;

package/src/builders/application/options.js

@@ -181,7 +181,7 @@ async function normalizeOptions(context, projectName, options, plugins) {
     const { allowedCommonJsDependencies, aot, baseHref, crossOrigin, externalDependencies, extractLicenses, inlineStyleLanguage = 'css', outExtension, serviceWorker, poll, polyfills, statsJson, stylePreprocessorOptions, subresourceIntegrity, verbose, watch, progress = true, externalPackages, deleteOutputPath, namedChunks, budgets, deployUrl, } = options;
     // Return all the normalized options
     return {
-        advancedOptimizations: !!aot,
+        advancedOptimizations: !!aot && optimizationOptions.scripts,
         allowedCommonJsDependencies,
         baseHref,
         cacheOptions,

package/src/builders/browser-esbuild/builder-status-warnings.js

@@ -30,9 +30,7 @@ function logBuilderStatusWarnings(options, { logger }) {
         if (typeof value === 'object' && Object.keys(value).length === 0) {
             continue;
         }
-        if (unsupportedOption === 'vendorChunk' ||
-            unsupportedOption === 'resourcesOutputPath' ||
-            unsupportedOption === 'deployUrl') {
+        if (unsupportedOption === 'vendorChunk' || unsupportedOption === 'resourcesOutputPath') {
             logger.warn(`The '${unsupportedOption}' option is not used by this builder and will be ignored.`);
             continue;
         }

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

@@ -65,6 +65,11 @@ function execute(options, context, transforms = {}, extensions) {
             if (transforms?.logging || transforms?.webpackConfiguration) {
                 throw new Error('The `application` and `browser-esbuild` builders do not support Webpack transforms.');
             }
+            if (normalizedOptions.forceEsbuild &&
+                builderName === '@angular-devkit/build-angular:browser') {
+                // The compatibility builder should be used if esbuild is force enabled with the official Webpack-based builder.
+                builderName = '@angular-devkit/build-angular:browser-esbuild';
+            }
             return (0, rxjs_1.defer)(() => Promise.resolve().then(() => __importStar(require('./vite-server')))).pipe((0, rxjs_1.switchMap)(({ serveWithVite }) => serveWithVite(normalizedOptions, builderName, context, transforms, extensions)));
         }
         if (extensions?.buildPlugins?.length) {

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

@@ -74,8 +74,11 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
     }
     // Set all packages as external to support Vite's prebundle caching
     browserOptions.externalPackages = serverOptions.cacheOptions.enabled;
-    if (serverOptions.servePath === undefined && browserOptions.baseHref !== undefined) {
-        serverOptions.servePath = browserOptions.baseHref;
+    const baseHref = browserOptions.baseHref;
+    if (serverOptions.servePath === undefined && baseHref !== undefined) {
+        // Remove trailing slash
+        serverOptions.servePath =
+            baseHref[baseHref.length - 1] === '/' ? baseHref.slice(0, -1) : baseHref;
     }
     // The development server currently only supports a single locale when localizing.
     // This matches the behavior of the Webpack-based development server but could be expanded in the future.
@@ -157,7 +160,7 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
         assetFiles.clear();
         if (result.assetFiles) {
             for (const asset of result.assetFiles) {
-                assetFiles.set('/' + normalizePath(asset.destination), asset.source);
+                assetFiles.set('/' + normalizePath(asset.destination), normalizePath(asset.source));
             }
         }
         // To avoid disconnecting the array objects from the option, these arrays need to be mutated instead of replaced.
@@ -179,7 +182,7 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
         if (server) {
             // Update fs allow list to include any new assets from the build option.
             server.config.server.fs.allow = [
-                ...new Set(...server.config.server.fs.allow, ...assetFiles.values()),
+                ...new Set([...server.config.server.fs.allow, ...assetFiles.values()]),
             ];
             handleUpdate(normalizePath, generatedFiles, server, serverOptions, context.logger);
         }
@@ -350,7 +353,7 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
         css: {
             devSourcemap: true,
         },
-        // Vite will normalize the `base` option by adding a leading and trailing forward slash.
+        // Vite will normalize the `base` option by adding a leading slash.
         base: serverOptions.servePath,
         resolve: {
             mainFields: ['es2020', 'browser', 'module', 'main'],
@@ -476,6 +479,10 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
                         // Rewrite all build assets to a vite raw fs URL
                         const assetSourcePath = assets.get(pathname);
                         if (assetSourcePath !== undefined) {
+                            // Workaround to disable Vite transformer middleware.
+                            // See: https://github.com/vitejs/vite/blob/746a1daab0395f98f0afbdee8f364cb6cf2f3b3f/packages/vite/src/node/server/middlewares/transform.ts#L201 and
+                            // https://github.com/vitejs/vite/blob/746a1daab0395f98f0afbdee8f364cb6cf2f3b3f/packages/vite/src/node/server/transformRequest.ts#L204-L206
+                            req.headers.accept = 'text/html';
                             // The encoding needs to match what happens in the vite static middleware.
                             // ref: https://github.com/vitejs/vite/blob/d4f13bd81468961c8c926438e815ab6b1c82735e/packages/vite/src/node/server/middlewares/static.ts#L163
                             req.url = `${server.config.base}@fs/${encodeURI(assetSourcePath)}`;

package/src/builders/karma/index.js

@@ -90,9 +90,12 @@ function execute(options, context, transforms = {}) {
             : getBuiltInKarmaConfig(context.workspaceRoot, projectName);
         karmaOptions.singleRun = singleRun;
         // Convert browsers from a string to an array
-        if (options.browsers) {
+        if (typeof options.browsers === 'string' && options.browsers) {
             karmaOptions.browsers = options.browsers.split(',');
         }
+        else if (options.browsers === false) {
+            karmaOptions.browsers = [];
+        }
         if (options.reporters) {
             // Split along commas to make it more natural, and remove empty strings.
             const reporters = options.reporters

package/src/builders/karma/schema.d.ts

@@ -7,9 +7,9 @@ export interface Schema {
      */
     assets?: AssetPattern[];
     /**
-     * Override which browsers tests are run against.
+     * Override which browsers tests are run against. Set to `false` to not use any browser.
      */
-    browsers?: string;
+    browsers?: Browsers;
     /**
      * Output a code coverage report.
      */
@@ -117,6 +117,10 @@ export interface AssetPatternClass {
      */
     output: string;
 }
+/**
+ * Override which browsers tests are run against. Set to `false` to not use any browser.
+ */
+export type Browsers = boolean | string;
 export interface FileReplacement {
     replace?: string;
     replaceWith?: string;

package/src/builders/karma/schema.json

@@ -199,8 +199,18 @@
       "description": "Do not use the real path when resolving modules. If unset then will default to `true` if NodeJS option --preserve-symlinks is set."
     },
     "browsers": {
-      "type": "string",
-      "description": "Override which browsers tests are run against."
+      "description": "Override which browsers tests are run against. Set to `false` to not use any browser.",
+      "oneOf": [
+        {
+          "type": "string",
+          "description": "A comma seperate list of browsers to run tests against."
+        },
+        {
+          "const": false,
+          "type": "boolean",
+          "description": "Does use run tests against a browser."
+        }
+      ]
     },
     "codeCoverage": {
       "type": "boolean",

package/src/builders/ssr-dev-server/index.js

@@ -35,7 +35,6 @@ const architect_1 = require("@angular-devkit/architect");
 const core_1 = require("@angular-devkit/core");
 const path_1 = require("path");
 const rxjs_1 = require("rxjs");
-const operators_1 = require("rxjs/operators");
 const url = __importStar(require("url"));
 const error_1 = require("../../utils/error");
 const utils_1 = require("./utils");
@@ -60,7 +59,17 @@ function execute(options, context) {
         progress: options.progress,
         verbose: options.verbose,
     });
-    const bsInstance = require('browser-sync').create();
+    let browserSync;
+    try {
+        browserSync = require('browser-sync');
+    }
+    catch {
+        return (0, rxjs_1.of)({
+            success: false,
+            error: '"browser-sync" is not installed, most likely you need to run `npm install browser-sync --save-dev` in your project.',
+        });
+    }
+    const bsInstance = browserSync.create();
     context.logger.error(core_1.tags.stripIndents `
   ****************************************************************************************
   This is a simple server for use in testing or debugging Angular applications locally.
@@ -69,35 +78,35 @@ function execute(options, context) {
   DON'T USE IT FOR PRODUCTION!
   ****************************************************************************************
  `);
-    return (0, rxjs_1.zip)(browserTargetRun, serverTargetRun, (0, utils_1.getAvailablePort)()).pipe((0, operators_1.switchMap)(([br, sr, nodeServerPort]) => {
+    return (0, rxjs_1.zip)(browserTargetRun, serverTargetRun, (0, utils_1.getAvailablePort)()).pipe((0, rxjs_1.switchMap)(([br, sr, nodeServerPort]) => {
         return (0, rxjs_1.combineLatest)([br.output, sr.output]).pipe(
         // This is needed so that if both server and browser emit close to each other
         // we only emit once. This typically happens on the first build.
-        (0, operators_1.debounceTime)(120), (0, operators_1.switchMap)(([b, s]) => {
+        (0, rxjs_1.debounceTime)(120), (0, rxjs_1.switchMap)(([b, s]) => {
             if (!s.success || !b.success) {
                 return (0, rxjs_1.of)([b, s]);
             }
-            return startNodeServer(s, nodeServerPort, context.logger, !!options.inspect).pipe((0, operators_1.mapTo)([b, s]), (0, operators_1.catchError)((err) => {
+            return startNodeServer(s, nodeServerPort, context.logger, !!options.inspect).pipe((0, rxjs_1.map)(() => [b, s]), (0, rxjs_1.catchError)((err) => {
                 context.logger.error(`A server error has occurred.\n${mapErrorToMessage(err)}`);
                 return rxjs_1.EMPTY;
             }));
-        }), (0, operators_1.map)(([b, s]) => [
+        }), (0, rxjs_1.map)(([b, s]) => [
             {
                 success: b.success && s.success,
                 error: b.error || s.error,
             },
             nodeServerPort,
-        ]), (0, operators_1.tap)(([builderOutput]) => {
+        ]), (0, rxjs_1.tap)(([builderOutput]) => {
             if (builderOutput.success) {
                 context.logger.info('\nCompiled successfully.');
             }
-        }), (0, operators_1.debounce)(([builderOutput]) => builderOutput.success && !options.inspect
+        }), (0, rxjs_1.debounce)(([builderOutput]) => builderOutput.success && !options.inspect
             ? (0, utils_1.waitUntilServerIsListening)(nodeServerPort)
-            : rxjs_1.EMPTY), (0, operators_1.finalize)(() => {
+            : rxjs_1.EMPTY), (0, rxjs_1.finalize)(() => {
             void br.stop();
             void sr.stop();
         }));
-    }), (0, operators_1.concatMap)(([builderOutput, nodeServerPort]) => {
+    }), (0, rxjs_1.concatMap)(([builderOutput, nodeServerPort]) => {
         if (!builderOutput.success) {
             return (0, rxjs_1.of)(builderOutput);
         }
@@ -106,7 +115,7 @@ function execute(options, context) {
             return (0, rxjs_1.of)(builderOutput);
         }
         else {
-            return (0, rxjs_1.from)(initBrowserSync(bsInstance, nodeServerPort, options, context)).pipe((0, operators_1.tap)((bs) => {
+            return (0, rxjs_1.from)(initBrowserSync(bsInstance, nodeServerPort, options, context)).pipe((0, rxjs_1.tap)((bs) => {
                 const baseUrl = getBaseUrl(bs);
                 context.logger.info(core_1.tags.oneLine `
                 **
@@ -114,19 +123,19 @@ function execute(options, context) {
                 open your browser on ${baseUrl}
                 **
               `);
-            }), (0, operators_1.mapTo)(builderOutput));
+            }), (0, rxjs_1.map)(() => builderOutput));
         }
-    }), (0, operators_1.map)((builderOutput) => ({
+    }), (0, rxjs_1.map)((builderOutput) => ({
         success: builderOutput.success,
         error: builderOutput.error,
         baseUrl: getBaseUrl(bsInstance),
         port: bsInstance.getOption('port'),
-    })), (0, operators_1.finalize)(() => {
+    })), (0, rxjs_1.finalize)(() => {
         if (bsInstance) {
             bsInstance.exit();
             bsInstance.cleanup();
         }
-    }), (0, operators_1.catchError)((error) => (0, rxjs_1.of)({
+    }), (0, rxjs_1.catchError)((error) => (0, rxjs_1.of)({
         success: false,
         error: mapErrorToMessage(error),
     })));
@@ -152,10 +161,10 @@ function startNodeServer(serverOutput, port, logger, inspectMode = false) {
     if (inspectMode) {
         args.unshift('--inspect-brk');
     }
-    return (0, rxjs_1.of)(null).pipe((0, operators_1.delay)(0), // Avoid EADDRINUSE error since it will cause the kill event to be finish.
-    (0, operators_1.switchMap)(() => (0, utils_1.spawnAsObservable)('node', args, { env, shell: true })), (0, operators_1.tap)((res) => log({ stderr: res.stderr, stdout: res.stdout }, logger)), (0, operators_1.ignoreElements)(), 
+    return (0, rxjs_1.of)(null).pipe((0, rxjs_1.delay)(0), // Avoid EADDRINUSE error since it will cause the kill event to be finish.
+    (0, rxjs_1.switchMap)(() => (0, utils_1.spawnAsObservable)('node', args, { env, shell: true })), (0, rxjs_1.tap)((res) => log({ stderr: res.stderr, stdout: res.stdout }, logger)), (0, rxjs_1.ignoreElements)(), 
     // Emit a signal after the process has been started
-    (0, operators_1.startWith)(undefined));
+    (0, rxjs_1.startWith)(undefined));
 }
 async function initBrowserSync(browserSyncInstance, nodeServerPort, options, context) {
     if (browserSyncInstance.active) {

package/src/tools/esbuild/angular/angular-host.js

@@ -28,6 +28,12 @@ exports.ensureSourceFileVersions = ensureSourceFileVersions;
 function createAngularCompilerHost(compilerOptions, hostOptions) {
     // Create TypeScript compiler host
     const host = typescript_1.default.createIncrementalCompilerHost(compilerOptions);
+    // Set the parsing mode to the same as TS 5.3 default for tsc. This provides a parse
+    // performance improvement by skipping non-type related JSDoc parsing.
+    // NOTE: The check for this enum can be removed when TS 5.3 support is the minimum.
+    if (typescript_1.default.JSDocParsingMode) {
+        host.jsDocParsingMode = typescript_1.default.JSDocParsingMode.ParseForTypeErrors;
+    }
     // The AOT compiler currently requires this hook to allow for a transformResource hook.
     // Once the AOT compiler allows only a transformResource hook, this can be reevaluated.
     host.readResource = async function (filename) {

package/src/tools/esbuild/angular/compiler-plugin.js

@@ -308,7 +308,7 @@ function createCompilerPlugin(pluginOptions, styleOptions) {
             }));
             // Setup bundling of component templates and stylesheets when in JIT mode
             if (pluginOptions.jit) {
-                (0, jit_plugin_callbacks_1.setupJitPluginCallbacks)(build, stylesheetBundler, additionalResults, styleOptions.inlineStyleLanguage);
+                (0, jit_plugin_callbacks_1.setupJitPluginCallbacks)(build, stylesheetBundler, additionalResults, styleOptions.inlineStyleLanguage, pluginOptions.loadResultCache);
             }
             build.onEnd((result) => {
                 // Ensure other compilations are unblocked if the main compilation throws during start

package/src/tools/esbuild/angular/component-stylesheets.js

@@ -79,7 +79,7 @@ class ComponentStylesheetBundler {
                                 namespace,
                             };
                         });
-                        build.onLoad({ filter: /^css;/, namespace }, async () => {
+                        build.onLoad({ filter: /^css;/, namespace }, () => {
                             return {
                                 contents: data,
                                 loader: 'css',

package/src/tools/esbuild/angular/jit-plugin-callbacks.d.ts

@@ -6,6 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 import type { Metafile, OutputFile, PluginBuild } from 'esbuild';
+import { LoadResultCache } from '../load-result-cache';
 import { ComponentStylesheetBundler } from './component-stylesheets';
 /**
  * Sets up esbuild resolve and load callbacks to support Angular JIT mode processing
@@ -19,4 +20,4 @@ import { ComponentStylesheetBundler } from './component-stylesheets';
 export declare function setupJitPluginCallbacks(build: PluginBuild, stylesheetBundler: ComponentStylesheetBundler, additionalResultFiles: Map<string, {
     outputFiles?: OutputFile[];
     metafile?: Metafile;
-}>, inlineStyleLanguage: string): void;
+}>, inlineStyleLanguage: string, loadCache?: LoadResultCache): void;