@angular-devkit/build-angular 17.0.0 → 17.0.1

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "@angular-devkit/build-angular",
-  "version": "17.0.0",
+  "version": "17.0.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.1700.0",
-    "@angular-devkit/build-webpack": "0.1700.0",
-    "@angular-devkit/core": "17.0.0",
+    "@angular-devkit/architect": "0.1700.1",
+    "@angular-devkit/build-webpack": "0.1700.1",
+    "@angular-devkit/core": "17.0.1",
     "@babel/core": "7.23.2",
     "@babel/generator": "7.23.0",
     "@babel/helper-annotate-as-pure": "7.22.5",
@@ -20,7 +20,7 @@
     "@babel/preset-env": "7.23.2",
     "@babel/runtime": "7.23.2",
     "@discoveryjs/json-ext": "0.5.7",
-    "@ngtools/webpack": "17.0.0",
+    "@ngtools/webpack": "17.0.1",
     "@vitejs/plugin-basic-ssl": "1.0.1",
     "ansi-colors": "4.1.3",
     "autoprefixer": "10.4.16",

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

@@ -34,32 +34,15 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.runEsBuildBuildAction = void 0;
-const promises_1 = __importDefault(require("node:fs/promises"));
 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");
-const error_1 = require("../../utils/error");
 async function* runEsBuildBuildAction(action, options) {
     const { writeToFileSystemFilter, writeToFileSystem = true, watch, poll, logger, deleteOutputPath, cacheOptions, outputPath, verbose, projectRoot, workspaceRoot, progress, } = options;
-    if (writeToFileSystem) {
-        // Clean output path if enabled
-        if (deleteOutputPath) {
-            if (outputPath === workspaceRoot) {
-                logger.error('Output path MUST not be workspace root directory!');
-                return;
-            }
-            await promises_1.default.rm(outputPath, { force: true, recursive: true, maxRetries: 3 });
-        }
-        // Create output directory if needed
-        try {
-            await promises_1.default.mkdir(outputPath, { recursive: true });
-        }
-        catch (e) {
-            (0, error_1.assertIsError)(e);
-            logger.error('Unable to create output directory: ' + e.message);
-            return;
-        }
+    if (deleteOutputPath && writeToFileSystem) {
+        await (0, delete_output_dir_1.deleteOutputDir)(workspaceRoot, outputPath);
     }
     const withProgress = progress ? utils_1.withSpinner : utils_1.withNoProgress;
     // Initial build
@@ -135,7 +118,7 @@ async function* runEsBuildBuildAction(action, options) {
         return;
     }
     // Wait for changes and rebuild as needed
-    let previousWatchFiles = new Set(result.watchFiles);
+    const currentWatchFiles = new Set(result.watchFiles);
     try {
         for await (const changes of watcher) {
             if (options.signal?.aborted) {
@@ -146,12 +129,22 @@ async function* runEsBuildBuildAction(action, options) {
             }
             result = await withProgress('Changes detected. Rebuilding...', () => action(result.createRebuildState(changes)));
             // Update watched locations provided by the new build result.
-            // Add any new locations
-            watcher.add(result.watchFiles.filter((watchFile) => !previousWatchFiles.has(watchFile)));
-            const newWatchFiles = new Set(result.watchFiles);
-            // Remove any old locations
-            watcher.remove([...previousWatchFiles].filter((watchFile) => !newWatchFiles.has(watchFile)));
-            previousWatchFiles = newWatchFiles;
+            // 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.
+            const staleWatchFiles = result.errors.length > 0 ? undefined : new Set(currentWatchFiles);
+            for (const watchFile of result.watchFiles) {
+                if (!currentWatchFiles.has(watchFile)) {
+                    // Add new watch location
+                    watcher.add(watchFile);
+                    currentWatchFiles.add(watchFile);
+                }
+                // Present so remove from stale locations
+                staleWatchFiles?.delete(watchFile);
+            }
+            // Remove any stale locations if the build was successful
+            if (staleWatchFiles?.size) {
+                watcher.remove([...staleWatchFiles]);
+            }
             if (writeToFileSystem) {
                 // Write output files
                 const filesToWrite = writeToFileSystemFilter

package/src/builders/application/options.js

@@ -11,6 +11,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizeOptions = void 0;
+const promises_1 = require("node:fs/promises");
 const node_module_1 = require("node:module");
 const node_path_1 = __importDefault(require("node:path"));
 const helpers_1 = require("../../tools/webpack/utils/helpers");
@@ -67,8 +68,15 @@ async function normalizeOptions(context, projectName, options, plugins) {
     let fileReplacements;
     if (options.fileReplacements) {
         for (const replacement of options.fileReplacements) {
+            const fileReplaceWith = node_path_1.default.join(workspaceRoot, replacement.with);
+            try {
+                await (0, promises_1.access)(fileReplaceWith, promises_1.constants.F_OK);
+            }
+            catch {
+                throw new Error(`The ${fileReplaceWith} path in file replacements does not exist.`);
+            }
             fileReplacements ??= {};
-            fileReplacements[node_path_1.default.join(workspaceRoot, replacement.replace)] = node_path_1.default.join(workspaceRoot, replacement.with);
+            fileReplacements[node_path_1.default.join(workspaceRoot, replacement.replace)] = fileReplaceWith;
         }
     }
     const globalStyles = [];

package/src/builders/browser/index.js

@@ -74,7 +74,7 @@ async function initialize(options, context, webpackConfigurationTransform) {
         transformedConfig = await webpackConfigurationTransform(config);
     }
     if (options.deleteOutputPath) {
-        (0, utils_1.deleteOutputDir)(context.workspaceRoot, originalOutputPath);
+        await (0, utils_1.deleteOutputDir)(context.workspaceRoot, originalOutputPath);
     }
     return { config: transformedConfig || config, projectRoot, projectSourceRoot, i18n };
 }

package/src/builders/browser-esbuild/index.js

@@ -16,6 +16,7 @@ const node_fs_1 = require("node:fs");
 const promises_1 = __importDefault(require("node:fs/promises"));
 const node_path_1 = __importDefault(require("node:path"));
 const utils_1 = require("../../tools/esbuild/utils");
+const utils_2 = require("../../utils");
 const application_1 = require("../application");
 const builder_status_warnings_1 = require("./builder-status-warnings");
 /**
@@ -29,7 +30,11 @@ async function* buildEsbuildBrowser(userOptions, context, infrastructureSettings
     // Inform user of status of builder and options
     (0, builder_status_warnings_1.logBuilderStatusWarnings)(userOptions, context);
     const normalizedOptions = normalizeOptions(userOptions);
-    const fullOutputPath = node_path_1.default.join(context.workspaceRoot, normalizedOptions.outputPath);
+    const { deleteOutputPath, outputPath } = normalizedOptions;
+    const fullOutputPath = node_path_1.default.join(context.workspaceRoot, outputPath);
+    if (deleteOutputPath && infrastructureSettings?.write !== false) {
+        await (0, utils_2.deleteOutputDir)(context.workspaceRoot, outputPath);
+    }
     for await (const result of (0, application_1.buildApplicationInternal)(normalizedOptions, context, {
         write: false,
     }, plugins)) {

package/src/builders/dev-server/index.d.ts

@@ -11,4 +11,4 @@ import { DevServerBuilderOutput } from './webpack-server';
 export { DevServerBuilderOptions, DevServerBuilderOutput, execute as executeDevServerBuilder };
 declare const _default: import("../../../../architect/src/internal").Builder<DevServerBuilderOptions & import("../../../../core/src").JsonObject>;
 export default _default;
-export { execute as serveWebpackBrowser };
+export { execute as executeDevServer };

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

@@ -7,9 +7,9 @@
  * found in the LICENSE file at https://angular.io/license
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.serveWebpackBrowser = exports.executeDevServerBuilder = void 0;
+exports.executeDevServer = exports.executeDevServerBuilder = void 0;
 const architect_1 = require("@angular-devkit/architect");
 const builder_1 = require("./builder");
 Object.defineProperty(exports, "executeDevServerBuilder", { enumerable: true, get: function () { return builder_1.execute; } });
-Object.defineProperty(exports, "serveWebpackBrowser", { enumerable: true, get: function () { return builder_1.execute; } });
+Object.defineProperty(exports, "executeDevServer", { enumerable: true, get: function () { return builder_1.execute; } });
 exports.default = (0, architect_1.createBuilder)(builder_1.execute);

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

@@ -60,11 +60,12 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
         poll: serverOptions.poll,
         verbose: serverOptions.verbose,
     }, builderName));
-    if (browserOptions.prerender) {
+    if (browserOptions.prerender || browserOptions.ssr) {
         // Disable prerendering if enabled and force SSR.
         // This is so instead of prerendering all the routes for every change, the page is "prerendered" when it is requested.
-        browserOptions.ssr = true;
         browserOptions.prerender = false;
+        // Avoid bundling and processing the ssr entry-point as this is not used by the dev-server.
+        browserOptions.ssr = true;
         // https://nodejs.org/api/process.html#processsetsourcemapsenabledval
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         process.setSourceMapsEnabled(true);
@@ -90,7 +91,7 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
     // Always enable JIT linking to support applications built with and without AOT.
     // In a development environment the additional scope information does not
     // have a negative effect unlike production where final output size is relevant.
-    { sourcemap: true, jit: true }, 1, true);
+    { sourcemap: true, jit: true, thirdPartySourcemaps: true }, 1, true);
     // Extract output index from options
     // TODO: Provide this info from the build results
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -107,6 +108,13 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
         implicitServer: [],
         explicit: [],
     };
+    // Add cleanup logic via a builder teardown.
+    let deferred;
+    context.addTeardown(async () => {
+        await server?.close();
+        await prebundleTransformer.close();
+        deferred?.();
+    });
     const build = builderName === '@angular-devkit/build-angular:application'
         ? application_1.buildApplicationInternal
         : browser_esbuild_1.buildEsbuildBrowser;
@@ -206,13 +214,6 @@ async function* serveWithVite(serverOptions, builderName, context, transformers,
             baseUrl: serverUrl?.href,
         };
     }
-    // Add cleanup logic via a builder teardown
-    let deferred;
-    context.addTeardown(async () => {
-        await server?.close();
-        await prebundleTransformer.close();
-        deferred?.();
-    });
     await new Promise((resolve) => (deferred = resolve));
 }
 exports.serveWithVite = serveWithVite;
@@ -421,19 +422,15 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
                     const originalssrTransform = server.ssrTransform;
                     server.ssrTransform = async (code, map, url, originalCode) => {
                         const result = await originalssrTransform(code, null, url, originalCode);
-                        if (!result) {
-                            return null;
-                        }
-                        let transformedCode = result.code;
-                        if (result.map && map) {
-                            transformedCode +=
-                                `\n//# sourceMappingURL=` +
-                                    `data:application/json;base64,${Buffer.from(JSON.stringify((0, remapping_1.default)([result.map, map], () => null))).toString('base64')}`;
+                        if (!result || !result.map || !map) {
+                            return result;
                         }
+                        const remappedMap = (0, remapping_1.default)([result.map, map], () => null);
+                        // Set the sourcemap root to the workspace root. This is needed since we set a virtual path as root.
+                        remappedMap.sourceRoot = serverOptions.workspaceRoot + '/';
                         return {
                             ...result,
-                            map: null,
-                            code: transformedCode,
+                            map: remappedMap,
                         };
                     };
                     // Assets and resources get handled first

package/src/builders/server/index.js

@@ -150,7 +150,7 @@ async function initialize(options, context, webpackConfigurationTransform) {
         return [getPlatformServerExportsConfig(wco), (0, configs_1.getCommonConfig)(wco), (0, configs_1.getStylesConfig)(wco)];
     });
     if (options.deleteOutputPath) {
-        (0, utils_1.deleteOutputDir)(context.workspaceRoot, originalOutputPath);
+        await (0, utils_1.deleteOutputDir)(context.workspaceRoot, originalOutputPath);
     }
     const transformedConfig = (await webpackConfigurationTransform?.(config)) ?? config;
     return { config: transformedConfig, i18n, projectRoot, projectSourceRoot };

package/src/tools/esbuild/angular/application-compiler.d.ts

@@ -0,0 +1,7 @@
+/**
+ * @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
+ */

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

@@ -0,0 +1,41 @@
+"use strict";
+/**
+ * @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
+ */
+// import { AngularCompilation, createAngularCompilation } from './compilation';
+// import { ComponentStylesheetBundler } from './component-stylesheets';
+// export interface InvalidationResult {
+//   code: boolean;
+//   styles: string[];
+// }
+// export class ApplicationAngularCompiler {
+//   #compilation: AngularCompilation | undefined;
+//   #stylesheetBundler: ComponentStylesheetBundler;
+//   readonly #emitFileCache = new Map<string, string>();
+//   constructor(options: { persistentCachePath?: string; jitMode?: boolean, incremental?: boolean }) {
+//     const {
+//       jitMode = false,
+//       incremental = false,
+//       persistentCachePath,
+//     } = options;
+//     this.#stylesheetBundler = new ComponentStylesheetBundler({}, incremental);
+//   }
+//   get isJit(): boolean {
+//     return false;
+//   }
+//   get usesAllowJs(): boolean {
+//     return false;
+//   }
+//   async initialize() {
+//   }
+//   invalidate(files: Iterable<string>): InvalidationResult {
+//     return {
+//       code: true,
+//       styles: [],
+//     };
+//   }
+// }

package/src/tools/esbuild/angular/compilation/parallel-compilation.js

@@ -35,6 +35,9 @@ class ParallelCompilation extends angular_compilation_1.AngularCompilation {
             minThreads: 1,
             maxThreads: 1,
             idleTimeout: Infinity,
+            // Web containers do not support transferable objects with receiveOnMessagePort which
+            // is used when the Atomics based wait loop is enable.
+            useAtomics: !process.versions.webcontainer,
             filename: localRequire.resolve('./parallel-worker'),
         });
     }

package/src/tools/esbuild/angular/compilation-state.d.ts

@@ -7,8 +7,8 @@
  */
 export declare class SharedTSCompilationState {
     #private;
-    get waitUntilReady(): Promise<void>;
-    markAsReady(): void;
+    get waitUntilReady(): Promise<boolean>;
+    markAsReady(hasErrors: boolean): void;
     markAsInProgress(): void;
     dispose(): void;
 }

package/src/tools/esbuild/angular/compilation-state.js

@@ -12,17 +12,19 @@ class SharedTSCompilationState {
     #pendingCompilation = true;
     #resolveCompilationReady;
     #compilationReadyPromise;
+    #hasErrors = true;
     get waitUntilReady() {
         if (!this.#pendingCompilation) {
-            return Promise.resolve();
+            return Promise.resolve(this.#hasErrors);
         }
         this.#compilationReadyPromise ??= new Promise((resolve) => {
             this.#resolveCompilationReady = resolve;
         });
         return this.#compilationReadyPromise;
     }
-    markAsReady() {
-        this.#resolveCompilationReady?.();
+    markAsReady(hasErrors) {
+        this.#hasErrors = hasErrors;
+        this.#resolveCompilationReady?.(hasErrors);
         this.#compilationReadyPromise = undefined;
         this.#pendingCompilation = false;
     }
@@ -30,7 +32,7 @@ class SharedTSCompilationState {
         this.#pendingCompilation = true;
     }
     dispose() {
-        this.markAsReady();
+        this.markAsReady(true);
         globalSharedCompilationState = undefined;
     }
 }

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

@@ -202,8 +202,7 @@ function createCompilerPlugin(pluginOptions, styleOptions) {
                     return result;
                 }
                 if (compilation instanceof compilation_1.NoopCompilation) {
-                    await sharedTSCompilationState.waitUntilReady;
-                    hasCompilationErrors = false;
+                    hasCompilationErrors = await sharedTSCompilationState.waitUntilReady;
                     return result;
                 }
                 const diagnostics = await compilation.diagnoseFiles();
@@ -250,7 +249,7 @@ function createCompilerPlugin(pluginOptions, styleOptions) {
                 hasCompilationErrors = !!result.errors?.length;
                 // Reset the setup warnings so that they are only shown during the first build.
                 setupWarnings = undefined;
-                sharedTSCompilationState.markAsReady();
+                sharedTSCompilationState.markAsReady(hasCompilationErrors);
                 return result;
             });
             build.onLoad({ filter: /\.[cm]?[jt]sx?$/ }, async (args) => {
@@ -314,7 +313,7 @@ function createCompilerPlugin(pluginOptions, styleOptions) {
             }
             build.onEnd((result) => {
                 // Ensure other compilations are unblocked if the main compilation throws during start
-                sharedTSCompilationState?.markAsReady();
+                sharedTSCompilationState?.markAsReady(hasCompilationErrors);
                 for (const { outputFiles, metafile } of additionalResults.values()) {
                     // Add any additional output files to the main output files
                     if (outputFiles?.length) {

package/src/tools/esbuild/application-code-bundle.js

@@ -194,7 +194,6 @@ function createServerPolyfillBundleOptions(options, target, sourceFileCache) {
     if (!polyfillBundleOptions) {
         return;
     }
-    const { workspaceRoot } = options;
     const buildOptions = {
         ...polyfillBundleOptions,
         platform: 'node',
@@ -218,8 +217,6 @@ function createServerPolyfillBundleOptions(options, target, sourceFileCache) {
             'polyfills.server': namespace,
         },
     };
-    buildOptions.plugins ??= [];
-    buildOptions.plugins.push((0, rxjs_esm_resolution_plugin_1.createRxjsEsmResolutionPlugin)());
     return () => buildOptions;
 }
 exports.createServerPolyfillBundleOptions = createServerPolyfillBundleOptions;

package/src/tools/esbuild/bundler-context.js

@@ -161,22 +161,26 @@ class BundlerContext {
                 throw failure;
             }
         }
+        finally {
+            if (this.incremental) {
+                // When incremental always add any files from the load result cache
+                if (this.#loadCache) {
+                    this.#loadCache.watchFiles
+                        .filter((file) => !isInternalAngularFile(file))
+                        // watch files are fully resolved paths
+                        .forEach((file) => this.watchFiles.add(file));
+                }
+            }
+        }
         // Update files that should be watched.
         // While this should technically not be linked to incremental mode, incremental is only
         // currently enabled with watch mode where watch files are needed.
         if (this.incremental) {
             // Add input files except virtual angular files which do not exist on disk
             Object.keys(result.metafile.inputs)
-                .filter((input) => !input.startsWith('angular:'))
+                .filter((input) => !isInternalAngularFile(input))
                 // input file paths are always relative to the workspace root
                 .forEach((input) => this.watchFiles.add((0, node_path_1.join)(this.workspaceRoot, input)));
-            // Also add any files from the load result cache
-            if (this.#loadCache) {
-                this.#loadCache.watchFiles
-                    .filter((file) => !file.startsWith('angular:'))
-                    // watch files are fully resolved paths
-                    .forEach((file) => this.watchFiles.add(file));
-            }
         }
         // Return if the build encountered any errors
         if (result.errors.length) {
@@ -271,12 +275,12 @@ class BundlerContext {
     #addErrorsToWatch(result) {
         for (const error of result.errors) {
             let file = error.location?.file;
-            if (file) {
+            if (file && !isInternalAngularFile(file)) {
                 this.watchFiles.add((0, node_path_1.join)(this.workspaceRoot, file));
             }
             for (const note of error.notes) {
                 file = note.location?.file;
-                if (file) {
+                if (file && !isInternalAngularFile(file)) {
                     this.watchFiles.add((0, node_path_1.join)(this.workspaceRoot, file));
                 }
             }
@@ -324,3 +328,6 @@ class BundlerContext {
     }
 }
 exports.BundlerContext = BundlerContext;
+function isInternalAngularFile(file) {
+    return file.startsWith('angular:');
+}

package/src/tools/esbuild/bundler-execution-result.d.ts

@@ -53,7 +53,7 @@ export declare class ExecutionResult {
         success: boolean;
         outputFiles: BuildOutputFile[];
         assetFiles: BuildOutputAsset[];
-        errors: (PartialMessage | Message)[];
+        errors: (Message | PartialMessage)[];
         externalMetadata: ExternalResultMetadata | undefined;
     };
     get watchFiles(): string[];

package/src/tools/esbuild/load-result-cache.js

@@ -18,8 +18,8 @@ function createCachedLoad(cache, callback) {
         let result = cache.get(loadCacheKey);
         if (result === undefined) {
             result = await callback(args);
-            // Do not cache null or undefined or results with errors
-            if (result && result.errors === undefined) {
+            // Do not cache null or undefined
+            if (result) {
                 await cache.put(loadCacheKey, result);
             }
         }

package/src/tools/esbuild/stylesheets/less-language.js

@@ -107,6 +107,7 @@ async function compileString(data, filename, options, resolver, unsafeInlineJava
     }
     catch (error) {
         if (isLessException(error)) {
+            const location = convertExceptionLocation(error);
             // Retry with a warning for less files requiring the deprecated inline JavaScript option
             if (error.message.includes('Inline JavaScript is not enabled.')) {
                 const withJsResult = await compileString(data, filename, options, resolver, 
@@ -114,7 +115,7 @@ async function compileString(data, filename, options, resolver, unsafeInlineJava
                 withJsResult.warnings = [
                     {
                         text: 'Deprecated inline execution of JavaScript has been enabled ("javascriptEnabled")',
-                        location: convertExceptionLocation(error),
+                        location,
                         notes: [
                             {
                                 location: null,
@@ -133,10 +134,11 @@ async function compileString(data, filename, options, resolver, unsafeInlineJava
                 errors: [
                     {
                         text: error.message,
-                        location: convertExceptionLocation(error),
+                        location,
                     },
                 ],
                 loader: 'css',
+                watchFiles: location.file ? [filename, location.file] : [filename],
             };
         }
         throw error;

package/src/tools/esbuild/stylesheets/sass-language.js

@@ -170,7 +170,7 @@ async function compileString(data, filePath, syntax, options, resolveUrl) {
     }
     catch (error) {
         if (isSassException(error)) {
-            const file = error.span.url ? (0, node_url_1.fileURLToPath)(error.span.url) : undefined;
+            const fileWithError = error.span.url ? (0, node_url_1.fileURLToPath)(error.span.url) : undefined;
             return {
                 loader: 'css',
                 errors: [
@@ -179,7 +179,7 @@ async function compileString(data, filePath, syntax, options, resolveUrl) {
                     },
                 ],
                 warnings,
-                watchFiles: file ? [file] : undefined,
+                watchFiles: fileWithError ? [filePath, fileWithError] : [filePath],
             };
         }
         throw error;

package/src/utils/delete-output-dir.d.ts

@@ -8,4 +8,4 @@
 /**
  * Delete an output directory, but error out if it's the root of the project.
  */
-export declare function deleteOutputDir(root: string, outputPath: string): void;
+export declare function deleteOutputDir(root: string, outputPath: string): Promise<void>;

package/src/utils/delete-output-dir.js

@@ -6,41 +6,32 @@
  * 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
  */
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.deleteOutputDir = void 0;
-const fs = __importStar(require("fs"));
-const path_1 = require("path");
+const promises_1 = require("node:fs/promises");
+const node_path_1 = require("node:path");
 /**
  * Delete an output directory, but error out if it's the root of the project.
  */
-function deleteOutputDir(root, outputPath) {
-    const resolvedOutputPath = (0, path_1.resolve)(root, outputPath);
+async function deleteOutputDir(root, outputPath) {
+    const resolvedOutputPath = (0, node_path_1.resolve)(root, outputPath);
     if (resolvedOutputPath === root) {
         throw new Error('Output path MUST not be project root directory!');
     }
-    fs.rmSync(resolvedOutputPath, { force: true, recursive: true, maxRetries: 3 });
+    // Avoid removing the actual directory to avoid errors in cases where the output
+    // directory is mounted or symlinked. Instead the contents are removed.
+    let entries;
+    try {
+        entries = await (0, promises_1.readdir)(resolvedOutputPath);
+    }
+    catch (error) {
+        if (error instanceof Error && 'code' in error && error.code === 'ENOENT') {
+            return;
+        }
+        throw error;
+    }
+    for (const entry of entries) {
+        await (0, promises_1.rm)((0, node_path_1.join)(resolvedOutputPath, entry), { force: true, recursive: true, maxRetries: 3 });
+    }
 }
 exports.deleteOutputDir = deleteOutputDir;

package/src/utils/environment-options.js

@@ -68,11 +68,8 @@ exports.allowMinify = debugOptimize.minify;
  */
 const maxWorkersVariable = process.env['NG_BUILD_MAX_WORKERS'];
 exports.maxWorkers = isPresent(maxWorkersVariable) ? +maxWorkersVariable : 4;
-// Default to enabled unless inside a Web Container which currently fails when transferring MessagePort objects
 const parallelTsVariable = process.env['NG_BUILD_PARALLEL_TS'];
-exports.useParallelTs = isPresent(parallelTsVariable)
-    ? !isDisabled(parallelTsVariable)
-    : !process.versions.webcontainer;
+exports.useParallelTs = !isPresent(parallelTsVariable) || !isDisabled(parallelTsVariable);
 const legacySassVariable = process.env['NG_BUILD_LEGACY_SASS'];
 exports.useLegacySass = (() => {
     if (!isPresent(legacySassVariable)) {

package/src/utils/routes-extractor/extractor.js

@@ -74,9 +74,15 @@ export async function* extractRoutes(bootstrapAppFnOrModule, document) {
         await whenStable(applicationRef);
         const injector = applicationRef.injector;
         const router = injector.get(Router);
-        const compiler = injector.get(Compiler);
-        // Extract all the routes from the config.
-        yield* getRoutesFromRouterConfig(router.config, compiler, injector);
+        if (router.config.length === 0) {
+            // In case there are no routes available
+            yield { route: '', success: true, redirect: false };
+        }
+        else {
+            const compiler = injector.get(Compiler);
+            // Extract all the routes from the config.
+            yield* getRoutesFromRouterConfig(router.config, compiler, injector);
+        }
     }
     finally {
         platformRef.destroy();