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

package/src/tools/esbuild/utils.js

@@ -6,46 +6,21 @@
  * 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;
-};
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getSupportedNodeTargets = exports.transformSupportedBrowsersToTargets = exports.getFullOutputPath = exports.convertOutputFile = exports.createOutputFileFromData = exports.createOutputFileFromText = exports.emitFilesToDisk = exports.writeResultFiles = exports.getFeatureSupport = exports.logMessages = exports.withNoProgress = exports.withSpinner = exports.calculateEstimatedTransferSizes = exports.logBuildStats = void 0;
+exports.getSupportedNodeTargets = exports.transformSupportedBrowsersToTargets = exports.convertOutputFile = exports.createOutputFileFromData = exports.createOutputFileFromText = exports.emitFilesToDisk = exports.writeResultFiles = exports.getFeatureSupport = exports.logMessages = exports.withNoProgress = exports.withSpinner = exports.calculateEstimatedTransferSizes = exports.logBuildStats = void 0;
 const esbuild_1 = require("esbuild");
 const node_crypto_1 = require("node:crypto");
 const node_fs_1 = require("node:fs");
 const promises_1 = __importDefault(require("node:fs/promises"));
-const node_path_1 = __importStar(require("node:path"));
-const node_util_1 = require("node:util");
+const node_path_1 = __importDefault(require("node:path"));
 const node_zlib_1 = require("node:zlib");
 const semver_1 = require("semver");
 const spinner_1 = require("../../utils/spinner");
 const stats_1 = require("../webpack/utils/stats");
 const bundler_context_1 = require("./bundler-context");
-const compressAsync = (0, node_util_1.promisify)(node_zlib_1.brotliCompress);
 function logBuildStats(context, metafile, initial, budgetFailures, changedFiles, estimatedTransferSizes) {
     const stats = [];
     let unchangedCount = 0;
@@ -90,22 +65,41 @@ function logBuildStats(context, metafile, initial, budgetFailures, changedFiles,
 exports.logBuildStats = logBuildStats;
 async function calculateEstimatedTransferSizes(outputFiles) {
     const sizes = new Map();
-    const pendingCompression = [];
-    for (const outputFile of outputFiles) {
-        // Only calculate JavaScript and CSS files
-        if (!outputFile.path.endsWith('.js') && !outputFile.path.endsWith('.css')) {
-            continue;
+    if (outputFiles.length <= 0) {
+        return sizes;
+    }
+    return new Promise((resolve, reject) => {
+        let completeCount = 0;
+        for (const outputFile of outputFiles) {
+            // Only calculate JavaScript and CSS files
+            if (!outputFile.path.endsWith('.js') && !outputFile.path.endsWith('.css')) {
+                ++completeCount;
+                continue;
+            }
+            // Skip compressing small files which may end being larger once compressed and will most likely not be
+            // compressed in actual transit.
+            if (outputFile.contents.byteLength < 1024) {
+                sizes.set(outputFile.path, outputFile.contents.byteLength);
+                ++completeCount;
+                continue;
+            }
+            // Directly use the async callback function to minimize the number of Promises that need to be created.
+            (0, node_zlib_1.brotliCompress)(outputFile.contents, (error, result) => {
+                if (error) {
+                    reject(error);
+                    return;
+                }
+                sizes.set(outputFile.path, result.byteLength);
+                if (++completeCount >= outputFiles.length) {
+                    resolve(sizes);
+                }
+            });
         }
-        // Skip compressing small files which may end being larger once compressed and will most likely not be
-        // compressed in actual transit.
-        if (outputFile.contents.byteLength < 1024) {
-            sizes.set(outputFile.path, outputFile.contents.byteLength);
-            continue;
+        // Covers the case where no files need to be compressed
+        if (completeCount >= outputFiles.length) {
+            resolve(sizes);
         }
-        pendingCompression.push(compressAsync(outputFile.contents).then((result) => sizes.set(outputFile.path, result.byteLength)));
-    }
-    await Promise.all(pendingCompression);
-    return sizes;
+    });
 }
 exports.calculateEstimatedTransferSizes = calculateEstimatedTransferSizes;
 async function withSpinner(text, action) {
@@ -182,31 +176,45 @@ function getFeatureSupport(target) {
     return supported;
 }
 exports.getFeatureSupport = getFeatureSupport;
-async function writeResultFiles(outputFiles, assetFiles, outputPath) {
+async function writeResultFiles(outputFiles, assetFiles, { base, browser, media, server }) {
     const directoryExists = new Set();
-    const ensureDirectoryExists = async (basePath) => {
-        if (basePath && !directoryExists.has(basePath)) {
-            await promises_1.default.mkdir(node_path_1.default.join(outputPath, basePath), { recursive: true });
+    const ensureDirectoryExists = async (destPath) => {
+        const basePath = node_path_1.default.dirname(destPath);
+        if (!directoryExists.has(basePath)) {
+            await promises_1.default.mkdir(node_path_1.default.join(base, basePath), { recursive: true });
             directoryExists.add(basePath);
         }
     };
     // Writes the output file to disk and ensures the containing directories are present
     await emitFilesToDisk(outputFiles, async (file) => {
-        const fullOutputPath = file.fullOutputPath;
+        let outputDir;
+        switch (file.type) {
+            case bundler_context_1.BuildOutputFileType.Browser:
+            case bundler_context_1.BuildOutputFileType.Media:
+                outputDir = browser;
+                break;
+            case bundler_context_1.BuildOutputFileType.Server:
+                outputDir = server;
+                break;
+            case bundler_context_1.BuildOutputFileType.Root:
+                outputDir = '';
+                break;
+            default:
+                throw new Error(`Unhandled write for file "${file.path}" with type "${bundler_context_1.BuildOutputFileType[file.type]}".`);
+        }
+        const destPath = node_path_1.default.join(outputDir, file.path);
         // Ensure output subdirectories exist
-        const basePath = node_path_1.default.dirname(fullOutputPath);
-        await ensureDirectoryExists(basePath);
+        await ensureDirectoryExists(destPath);
         // Write file contents
-        await promises_1.default.writeFile(node_path_1.default.join(outputPath, fullOutputPath), file.contents);
+        await promises_1.default.writeFile(node_path_1.default.join(base, destPath), file.contents);
     });
     if (assetFiles?.length) {
         await emitFilesToDisk(assetFiles, async ({ source, destination }) => {
+            const destPath = node_path_1.default.join(browser, destination);
             // Ensure output subdirectories exist
-            const destPath = (0, node_path_1.join)('browser', destination);
-            const basePath = node_path_1.default.dirname(destPath);
-            await ensureDirectoryExists(basePath);
+            await ensureDirectoryExists(destPath);
             // Copy file contents
-            await promises_1.default.copyFile(source, node_path_1.default.join(outputPath, destPath), node_fs_1.constants.COPYFILE_FICLONE);
+            await promises_1.default.copyFile(source, node_path_1.default.join(base, destPath), node_fs_1.constants.COPYFILE_FICLONE);
         });
     }
 }
@@ -235,9 +243,6 @@ function createOutputFileFromText(path, text, type) {
         get contents() {
             return Buffer.from(this.text, 'utf-8');
         },
-        get fullOutputPath() {
-            return getFullOutputPath(this);
-        },
         clone() {
             return createOutputFileFromText(this.path, this.text, this.type);
         },
@@ -257,9 +262,6 @@ function createOutputFileFromData(path, data, type) {
         get contents() {
             return data;
         },
-        get fullOutputPath() {
-            return getFullOutputPath(this);
-        },
         clone() {
             return createOutputFileFromData(this.path, this.contents, this.type);
         },
@@ -276,27 +278,12 @@ function convertOutputFile(file, type) {
         get text() {
             return Buffer.from(this.contents.buffer, this.contents.byteOffset, this.contents.byteLength).toString('utf-8');
         },
-        get fullOutputPath() {
-            return getFullOutputPath(this);
-        },
         clone() {
             return convertOutputFile(this, this.type);
         },
     };
 }
 exports.convertOutputFile = convertOutputFile;
-function getFullOutputPath(file) {
-    switch (file.type) {
-        case bundler_context_1.BuildOutputFileType.Browser:
-        case bundler_context_1.BuildOutputFileType.Media:
-            return (0, node_path_1.join)('browser', file.path);
-        case bundler_context_1.BuildOutputFileType.Server:
-            return (0, node_path_1.join)('server', file.path);
-        default:
-            return file.path;
-    }
-}
-exports.getFullOutputPath = getFullOutputPath;
 /**
  * Transform browserlists result to esbuild target.
  * @see https://esbuild.github.io/api/#target

package/src/utils/index-file/index-html-generator.js

@@ -6,33 +6,10 @@
  * 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.IndexHtmlGenerator = 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");
 const strip_bom_1 = require("../strip-bom");
 const augment_index_html_1 = require("./augment-index-html");
 const inline_critical_css_1 = require("./inline-critical-css");
@@ -85,10 +62,20 @@ class IndexHtmlGenerator {
         };
     }
     async readAsset(path) {
-        return fs.promises.readFile(path, 'utf-8');
+        try {
+            return await (0, promises_1.readFile)(path, 'utf-8');
+        }
+        catch {
+            throw new Error(`Failed to read asset "${path}".`);
+        }
     }
     async readIndex(path) {
-        return fs.promises.readFile(path, 'utf-8');
+        try {
+            return await (0, promises_1.readFile)(path, 'utf-8');
+        }
+        catch {
+            throw new Error(`Failed to read index HTML file "${path}".`);
+        }
     }
 }
 exports.IndexHtmlGenerator = IndexHtmlGenerator;
@@ -104,7 +91,7 @@ function augmentIndexHtmlPlugin(generator) {
             sri,
             lang,
             entrypoints,
-            loadOutputFile: (filePath) => generator.readAsset((0, path_1.join)(outputPath, filePath)),
+            loadOutputFile: (filePath) => generator.readAsset((0, node_path_1.join)(outputPath, filePath)),
             imageDomains,
             files,
             hints,

package/src/utils/index.d.ts

@@ -8,6 +8,7 @@
 export * from './default-progress';
 export * from './delete-output-dir';
 export * from './run-module-as-observable-fork';
+export * from './load-proxy-config';
 export * from './normalize-file-replacements';
 export * from './normalize-asset-patterns';
 export * from './normalize-source-maps';

package/src/utils/index.js

@@ -24,6 +24,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("./default-progress"), exports);
 __exportStar(require("./delete-output-dir"), exports);
 __exportStar(require("./run-module-as-observable-fork"), exports);
+__exportStar(require("./load-proxy-config"), exports);
 __exportStar(require("./normalize-file-replacements"), exports);
 __exportStar(require("./normalize-asset-patterns"), exports);
 __exportStar(require("./normalize-source-maps"), exports);

package/src/{builders/dev-server → utils}/load-proxy-config.js

@@ -37,8 +37,8 @@ const promises_1 = require("node:fs/promises");
 const node_path_1 = require("node:path");
 const node_url_1 = require("node:url");
 const picomatch_1 = require("picomatch");
-const error_1 = require("../../utils/error");
-const load_esm_1 = require("../../utils/load-esm");
+const error_1 = require("./error");
+const load_esm_1 = require("./load-esm");
 async function loadProxyConfiguration(root, proxyConfig, normalize = false) {
     if (!proxyConfig) {
         return undefined;