@angular/build 19.1.4 → 19.1.5

package/README.md

@@ -2,4 +2,4 @@
 
 The sources for this package are in the [Angular CLI](https://github.com/angular/angular-cli) repository. Please file issues and pull requests against that repository.
 
-Usage information and reference details can be found in repository [README](../../../README.md) file.
+Usage information and reference details can be found in repository [README](https://github.com/angular/angular-cli/blob/main/README.md) file.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular/build",
-  "version": "19.1.4",
+  "version": "19.1.5",
   "description": "Official build system for Angular",
   "keywords": [
     "Angular CLI",
@@ -23,7 +23,8 @@
   "builders": "builders.json",
   "dependencies": {
     "@ampproject/remapping": "2.3.0",
-    "@angular-devkit/architect": "0.1901.4",
+    "@angular-devkit/core": "19.1.5",
+    "@angular-devkit/architect": "0.1901.5",
     "@babel/core": "7.26.0",
     "@babel/helper-annotate-as-pure": "7.25.9",
     "@babel/helper-split-export-declaration": "7.24.7",
@@ -45,7 +46,7 @@
     "rollup": "4.30.1",
     "sass": "1.83.1",
     "semver": "7.6.3",
-    "vite": "6.0.7",
+    "vite": "6.0.11",
     "watchpack": "2.4.2"
   },
   "optionalDependencies": {
@@ -57,11 +58,11 @@
     "@angular/localize": "^19.0.0",
     "@angular/platform-server": "^19.0.0",
     "@angular/service-worker": "^19.0.0",
-    "@angular/ssr": "^19.1.4",
+    "@angular/ssr": "^19.1.5",
     "less": "^4.2.0",
     "ng-packagr": "^19.0.0",
     "postcss": "^8.4.0",
-    "tailwindcss": "^2.0.0 || ^3.0.0",
+    "tailwindcss": "^2.0.0 || ^3.0.0 || ^4.0.0",
     "typescript": ">=5.5 <5.8"
   },
   "peerDependenciesMeta": {

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

@@ -239,6 +239,7 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
             outputOptions,
         },
     };
+    let hasCssUpdates = false;
     // Initially assume all previous output files have been removed
     const removedOutputFiles = new Map(previousOutputInfo);
     for (const file of outputFiles) {
@@ -254,8 +255,11 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
             incrementalResult.modified.push(file.path);
         }
         if (needFile) {
-            // Updates to non-JS files must signal an update with the dev server
-            if (!/(?:\.m?js|\.map)$/.test(file.path)) {
+            if (file.path.endsWith('.css')) {
+                hasCssUpdates = true;
+            }
+            else if (!/(?:\.m?js|\.map)$/.test(file.path)) {
+                // Updates to non-JS files must signal an update with the dev server
                 incrementalResult.background = false;
             }
             incrementalResult.files[file.path] = {
@@ -281,12 +285,21 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
         else {
             continue;
         }
+        hasCssUpdates ||= destination.endsWith('.css');
         incrementalResult.files[destination] = {
             type: bundler_context_1.BuildOutputFileType.Browser,
             inputPath: source,
             origin: 'disk',
         };
     }
+    // Do not remove stale files yet if there are template updates.
+    // Component chunk files may still be referenced in running browser code.
+    // Module evaluation time component updates will update any of these files.
+    // This typically occurs when a lazy component is changed that has not yet
+    // been accessed at runtime.
+    if (hasTemplateUpdates && incrementalResult.background) {
+        removedOutputFiles.clear();
+    }
     // Include the removed output and asset files
     incrementalResult.removed.push(...Array.from(removedOutputFiles, ([file, { type }]) => ({
         path: file,
@@ -299,6 +312,17 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
     // If there are template updates and the incremental update was background only, a component
     // update is possible.
     if (hasTemplateUpdates && incrementalResult.background) {
+        // Template changes may be accompanied by stylesheet changes and these should also be updated hot when possible.
+        if (hasCssUpdates) {
+            const styleResult = {
+                kind: results_1.ResultKind.Incremental,
+                added: incrementalResult.added.filter(isCssFilePath),
+                removed: incrementalResult.removed.filter(({ path }) => isCssFilePath(path)),
+                modified: incrementalResult.modified.filter(isCssFilePath),
+                files: Object.fromEntries(Object.entries(incrementalResult.files).filter(([path]) => isCssFilePath(path))),
+            };
+            yield styleResult;
+        }
         const updateResult = {
             kind: results_1.ResultKind.ComponentUpdate,
             updates: Array.from(templateUpdates, ([id, content]) => ({
@@ -310,3 +334,6 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
         yield updateResult;
     }
 }
+function isCssFilePath(filePath) {
+    return /\.css(?:\.map)?$/i.test(filePath);
+}

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

@@ -44,4 +44,5 @@ export declare function normalizeOptions(context: BuilderContext, projectName: s
         host?: string;
         port?: number;
     };
+    allowedHosts: true | string[];
 }>;

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

@@ -73,7 +73,7 @@ async function normalizeOptions(context, projectName, options) {
         }
     }
     // Initial options to keep
-    const { host, port, poll, open, verbose, watch, liveReload, hmr, headers, proxyConfig, servePath, ssl, sslCert, sslKey, prebundle, } = options;
+    const { host, port, poll, open, verbose, watch, liveReload, hmr, headers, proxyConfig, servePath, ssl, sslCert, sslKey, prebundle, allowedHosts, } = options;
     // Return all the normalized options
     return {
         buildTarget,
@@ -97,5 +97,6 @@ async function normalizeOptions(context, projectName, options) {
         // Prebundling defaults to true but requires caching to function
         prebundle: cacheOptions.enabled && !optimization.scripts && prebundle,
         inspect,
+        allowedHosts: allowedHosts ? allowedHosts : [],
     };
 }

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

@@ -2,6 +2,12 @@
  * Dev Server target options for Build Facade.
  */
 export type Schema = {
+    /**
+     * The hosts that can access the development server. This option sets the Vite option of the
+     * same name. For further details:
+     * https://vite.dev/config/server-options.html#server-allowedhosts
+     */
+    allowedHosts?: AllowedHosts;
     /**
      * 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:
@@ -79,6 +85,12 @@ export type Schema = {
      */
     watch?: boolean;
 };
+/**
+ * The hosts that can access the development server. This option sets the Vite option of the
+ * same name. For further details:
+ * https://vite.dev/config/server-options.html#server-allowedhosts
+ */
+export type AllowedHosts = string[] | boolean;
 /**
  * Activate debugging inspector. This option only has an effect when 'SSR' or 'SSG' are
  * enabled.

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

@@ -36,6 +36,23 @@
       "type": "string",
       "description": "SSL certificate to use for serving HTTPS."
     },
+    "allowedHosts": {
+      "description": "The hosts that can access the development server. This option sets the Vite option of the same name. For further details: https://vite.dev/config/server-options.html#server-allowedhosts",
+      "default": [],
+      "oneOf": [
+        {
+          "type": "array",
+          "description": "List of hosts that are allowed to access the development server.",
+          "items": {
+            "type": "string"
+          }
+        },
+        {
+          "type": "boolean",
+          "description": "Indicates that all hosts are allowed. This is not recommended and a security risk."
+        }
+      ]
+    },
     "headers": {
       "type": "object",
       "description": "Custom HTTP headers to be added to all responses.",

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

@@ -304,6 +304,23 @@ async function* serveWithVite(serverOptions, builderName, builderAction, context
             const serverConfiguration = await setupServer(serverOptions, generatedFiles, assetFiles, browserOptions.preserveSymlinks, externalMetadata, ssrMode, prebundleTransformer, target, (0, internal_1.isZonelessApp)(polyfills), componentStyles, templateUpdates, browserOptions.loader, browserOptions.define, extensions?.middleware, transformers?.indexHtml, thirdPartySourcemaps);
             server = await createServer(serverConfiguration);
             await server.listen();
+            // Setup builder context logging for browser clients
+            server.hot.on('angular:log', (data) => {
+                if (typeof data?.text !== 'string') {
+                    context.logger.warn('Development server client sent invalid internal log event.');
+                }
+                switch (data.kind) {
+                    case 'error':
+                        context.logger.error(`[CLIENT ERROR]: ${data.text}`);
+                        break;
+                    case 'warning':
+                        context.logger.warn(`[CLIENT WARNING]: ${data.text}`);
+                        break;
+                    default:
+                        context.logger.info(`[CLIENT INFO]: ${data.text}`);
+                        break;
+                }
+            });
             const urls = server.resolvedUrls;
             if (urls && (urls.local.length || urls.network.length)) {
                 serverUrl = new URL(urls.local[0] ?? urls.network[0]);
@@ -353,26 +370,22 @@ async function invalidateUpdatedFiles(normalizePath, generatedFiles, assetFiles,
         updatedFiles.push(file);
     }
     // Invalidate any updated files
-    let destroyAngularServerAppCalled = false;
+    let serverApplicationChanged = false;
     for (const [file, record] of generatedFiles) {
         if (!record.updated) {
             continue;
         }
         record.updated = false;
-        if (record.type === internal_1.BuildOutputFileType.ServerApplication && !destroyAngularServerAppCalled) {
-            // Clear the server app cache
-            // This must be done before module invalidation.
-            const { ɵdestroyAngularServerApp } = (await server.ssrLoadModule('/main.server.mjs'));
-            ɵdestroyAngularServerApp();
-            destroyAngularServerAppCalled = true;
-        }
         updatedFiles.push(file);
+        serverApplicationChanged ||= record.type === internal_1.BuildOutputFileType.ServerApplication;
         const updatedModules = server.moduleGraph.getModulesByFile(normalizePath((0, node_path_1.join)(server.config.root, file)));
         updatedModules?.forEach((m) => server.moduleGraph.invalidateModule(m));
     }
-    if (destroyAngularServerAppCalled) {
-        // Trigger module evaluation before reload to initiate dependency optimization.
-        await server.ssrLoadModule('/main.server.mjs');
+    if (serverApplicationChanged) {
+        // Clear the server app cache and
+        // trigger module evaluation before reload to initiate dependency optimization.
+        const { ɵdestroyAngularServerApp } = (await server.ssrLoadModule('/main.server.mjs'));
+        ɵdestroyAngularServerApp();
     }
     return updatedFiles;
 }
@@ -424,7 +437,7 @@ function handleUpdate(server, serverOptions, logger, componentStyles, updatedFil
                     type: 'update',
                     updates,
                 });
-                logger.info('HMR update sent to client(s).');
+                logger.info('Stylesheet update sent to client(s).');
                 return;
             }
         }
@@ -545,6 +558,7 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
             strictPort: true,
             host: serverOptions.host,
             open: serverOptions.open,
+            allowedHosts: serverOptions.allowedHosts,
             headers: serverOptions.headers,
             // Disable the websocket if live reload is disabled (false/undefined are the only valid values)
             ws: serverOptions.liveReload === false && serverOptions.hmr === false ? false : undefined,

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

@@ -80,6 +80,9 @@ class AngularCompilation {
             enableResourceInlining: false,
             supportTestBed: false,
             supportJitMode: false,
+            // Disable removing of comments as TS is quite aggressive with these and can
+            // remove important annotations, such as /* @__PURE__ */ and comments like /* vite-ignore */.
+            removeComments: false,
         }));
     }
     async diagnoseFiles(modes = DiagnosticModes.All) {

package/src/tools/vite/middlewares/ssr-middleware.js

@@ -22,6 +22,10 @@ function createAngularSsrInternalMiddleware(server, indexHtmlTransformer) {
             // which must be processed by the runtime linker, even if they are not used.
             await (0, load_esm_1.loadEsmModule)('@angular/compiler');
             const { writeResponseToNodeResponse, createWebRequestFromNodeRequest } = await (0, load_esm_1.loadEsmModule)('@angular/ssr/node');
+            // The following is necessary because accessing the module after invalidation may result in an empty module,
+            // which can trigger a `TypeError: ɵgetOrCreateAngularServerApp is not a function` error.
+            // TODO: look into why.
+            await server.ssrLoadModule('/main.server.mjs');
             const { ɵgetOrCreateAngularServerApp } = (await server.ssrLoadModule('/main.server.mjs'));
             const angularServerApp = ɵgetOrCreateAngularServerApp({
                 allowStaticRouteRender: true,

package/src/utils/normalize-cache.js

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

package/src/utils/service-worker.d.ts

@@ -9,7 +9,7 @@ import type { Config, Filesystem } from '@angular/service-worker/config';
 import { promises as fsPromises } from 'node:fs';
 import { BuildOutputFile } from '../tools/esbuild/bundler-context';
 import { BuildOutputAsset } from '../tools/esbuild/bundler-execution-result';
-export declare function augmentAppWithServiceWorker(appRoot: string, workspaceRoot: string, outputPath: string, baseHref: string, ngswConfigPath?: string, inputputFileSystem?: typeof fsPromises, outputFileSystem?: typeof fsPromises): Promise<void>;
+export declare function augmentAppWithServiceWorker(appRoot: string, workspaceRoot: string, outputPath: string, baseHref: string, ngswConfigPath?: string, inputFileSystem?: typeof fsPromises, outputFileSystem?: typeof fsPromises): Promise<void>;
 export declare function augmentAppWithServiceWorkerEsbuild(workspaceRoot: string, configPath: string, baseHref: string, indexHtml: string | undefined, outputFiles: BuildOutputFile[], assetFiles: BuildOutputAsset[]): Promise<{
     manifest: string;
     assetFiles: BuildOutputAsset[];

package/src/utils/service-worker.js

@@ -133,7 +133,7 @@ class ResultFilesystem {
         throw new Error('Serviceworker manifest generator should not attempted to write.');
     }
 }
-async function augmentAppWithServiceWorker(appRoot, workspaceRoot, outputPath, baseHref, ngswConfigPath, inputputFileSystem = node_fs_1.promises, outputFileSystem = node_fs_1.promises) {
+async function augmentAppWithServiceWorker(appRoot, workspaceRoot, outputPath, baseHref, ngswConfigPath, inputFileSystem = node_fs_1.promises, outputFileSystem = node_fs_1.promises) {
     // Determine the configuration file path
     const configPath = ngswConfigPath
         ? path.join(workspaceRoot, ngswConfigPath)
@@ -141,7 +141,7 @@ async function augmentAppWithServiceWorker(appRoot, workspaceRoot, outputPath, b
     // Read the configuration file
     let config;
     try {
-        const configurationData = await inputputFileSystem.readFile(configPath, 'utf-8');
+        const configurationData = await inputFileSystem.readFile(configPath, 'utf-8');
         config = JSON.parse(configurationData);
     }
     catch (error) {
@@ -158,11 +158,7 @@ async function augmentAppWithServiceWorker(appRoot, workspaceRoot, outputPath, b
     const result = await augmentAppWithServiceWorkerCore(config, new CliFilesystem(outputFileSystem, outputPath), baseHref);
     const copy = async (src, dest) => {
         const resolvedDest = path.join(outputPath, dest);
-        return inputputFileSystem === outputFileSystem
-            ? // Native FS (Builder).
-                inputputFileSystem.copyFile(src, resolvedDest, node_fs_1.constants.COPYFILE_FICLONE)
-            : // memfs (Webpack): Read the file from the input FS (disk) and write it to the output FS (memory).
-                outputFileSystem.writeFile(resolvedDest, await inputputFileSystem.readFile(src));
+        return outputFileSystem.writeFile(resolvedDest, await inputFileSystem.readFile(src));
     };
     await outputFileSystem.writeFile(path.join(outputPath, 'ngsw.json'), result.manifest);
     for (const { source, destination } of result.assetFiles) {