@angular/build 19.1.6 → 19.2.0-next.0

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](https://github.com/angular/angular-cli/blob/main/README.md) file.
+Usage information and reference details can be found in repository [README](../../../README.md) file.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular/build",
-  "version": "19.1.6",
+  "version": "19.2.0-next.0",
   "description": "Official build system for Angular",
   "keywords": [
     "Angular CLI",
@@ -23,13 +23,13 @@
   "builders": "builders.json",
   "dependencies": {
     "@ampproject/remapping": "2.3.0",
-    "@angular-devkit/core": "19.1.6",
-    "@angular-devkit/architect": "0.1901.6",
+    "@angular-devkit/core": "19.2.0-next.0",
+    "@angular-devkit/architect": "0.1902.0-next.0",
     "@babel/core": "7.26.0",
     "@babel/helper-annotate-as-pure": "7.25.9",
     "@babel/helper-split-export-declaration": "7.24.7",
     "@babel/plugin-syntax-import-attributes": "7.26.0",
-    "@inquirer/confirm": "5.1.1",
+    "@inquirer/confirm": "5.1.3",
     "@vitejs/plugin-basic-ssl": "1.2.0",
     "beasties": "0.2.0",
     "browserslist": "^4.23.0",
@@ -43,8 +43,8 @@
     "parse5-html-rewriting-stream": "7.0.0",
     "picomatch": "4.0.2",
     "piscina": "4.8.0",
-    "rollup": "4.30.1",
-    "sass": "1.83.1",
+    "rollup": "4.31.0",
+    "sass": "1.83.4",
     "semver": "7.6.3",
     "vite": "6.0.11",
     "watchpack": "2.4.2"
@@ -53,16 +53,16 @@
     "lmdb": "3.2.2"
   },
   "peerDependencies": {
-    "@angular/compiler": "^19.0.0",
-    "@angular/compiler-cli": "^19.0.0",
-    "@angular/localize": "^19.0.0",
-    "@angular/platform-server": "^19.0.0",
-    "@angular/service-worker": "^19.0.0",
-    "@angular/ssr": "^19.1.6",
+    "@angular/compiler": "^19.0.0 || ^19.2.0-next.0",
+    "@angular/compiler-cli": "^19.0.0 || ^19.2.0-next.0",
+    "@angular/localize": "^19.0.0 || ^19.2.0-next.0",
+    "@angular/platform-server": "^19.0.0 || ^19.2.0-next.0",
+    "@angular/service-worker": "^19.0.0 || ^19.2.0-next.0",
+    "@angular/ssr": "^19.2.0-next.0",
     "less": "^4.2.0",
-    "ng-packagr": "^19.0.0",
+    "ng-packagr": "^19.0.0 || ^19.2.0-next.0",
     "postcss": "^8.4.0",
-    "tailwindcss": "^2.0.0 || ^3.0.0 || ^4.0.0",
+    "tailwindcss": "^2.0.0 || ^3.0.0",
     "typescript": ">=5.5 <5.8"
   },
   "peerDependenciesMeta": {

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

@@ -127,9 +127,6 @@ async function* runEsBuildBuildAction(action, options) {
     if (!watcher) {
         return;
     }
-    // Used to force a full result on next rebuild if there were initial errors.
-    // This ensures at least one full result is emitted.
-    let hasInitialErrors = result.errors.length > 0;
     // Wait for changes and rebuild as needed
     const currentWatchFiles = new Set(result.watchFiles);
     try {
@@ -167,11 +164,9 @@ async function* runEsBuildBuildAction(action, options) {
             if (staleWatchFiles?.size) {
                 watcher.remove([...staleWatchFiles]);
             }
-            for (const outputResult of emitOutputResults(result, outputOptions, changes, incrementalResults && !hasInitialErrors ? rebuildState : undefined)) {
+            for (const outputResult of emitOutputResults(result, outputOptions, changes, incrementalResults ? rebuildState : undefined)) {
                 yield outputResult;
             }
-            // Clear initial build errors flag if no errors are now present
-            hasInitialErrors &&= result.errors.length > 0;
         }
     }
     finally {
@@ -244,7 +239,6 @@ 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) {
@@ -260,11 +254,8 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
             incrementalResult.modified.push(file.path);
         }
         if (needFile) {
-            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
+            // Updates to non-JS files must signal an update with the dev server
+            if (!/(?:\.m?js|\.map)$/.test(file.path)) {
                 incrementalResult.background = false;
             }
             incrementalResult.files[file.path] = {
@@ -290,21 +281,12 @@ 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,
@@ -317,17 +299,6 @@ 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]) => ({
@@ -339,6 +310,3 @@ function* emitOutputResults({ outputFiles, assetFiles, errors, warnings, externa
         yield updateResult;
     }
 }
-function isCssFilePath(filePath) {
-    return /\.css(?:\.map)?$/i.test(filePath);
-}

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

@@ -38,7 +38,7 @@ async function executePostBundleSteps(metafile, options, outputFiles, assetFiles
     const allErrors = [];
     const allWarnings = [];
     const prerenderedRoutes = {};
-    const { baseHref = '/', serviceWorker, ssrOptions, indexHtmlOptions, optimizationOptions, sourcemapOptions, outputMode, serverEntryPoint, prerenderOptions, appShellOptions, publicPath, workspaceRoot, partialSSRBuild, } = options;
+    const { baseHref = '/', serviceWorker, i18nOptions, indexHtmlOptions, optimizationOptions, sourcemapOptions, outputMode, serverEntryPoint, prerenderOptions, appShellOptions, publicPath, workspaceRoot, partialSSRBuild, } = options;
     // Index HTML content without CSS inlining to be used for server rendering (AppShell, SSG and SSR).
     // NOTE: Critical CSS inlining is deliberately omitted here, as it will be handled during server rendering.
     // Additionally, when using prerendering or AppShell, the index HTML file may be regenerated.
@@ -57,7 +57,7 @@ async function executePostBundleSteps(metafile, options, outputFiles, assetFiles
     }
     // Create server manifest
     const initialFilesPaths = new Set(initialFiles.keys());
-    if (serverEntryPoint && (outputMode || prerenderOptions || appShellOptions || ssrOptions)) {
+    if (serverEntryPoint) {
         const { manifestContent, serverAssetsChunks } = (0, manifest_1.generateAngularServerAppManifest)(additionalHtmlOutputFiles, outputFiles, optimizationOptions.styles.inlineCritical ?? false, undefined, locale, baseHref, initialFilesPaths, metafile, publicPath);
         additionalOutputFiles.push(...serverAssetsChunks, (0, utils_1.createOutputFile)(manifest_1.SERVER_APP_MANIFEST_FILENAME, manifestContent, bundler_context_1.BuildOutputFileType.ServerApplication));
     }

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

@@ -44,5 +44,4 @@ 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, allowedHosts, } = options;
+    const { host, port, poll, open, verbose, watch, liveReload, hmr, headers, proxyConfig, servePath, ssl, sslCert, sslKey, prebundle, } = options;
     // Return all the normalized options
     return {
         buildTarget,
@@ -97,6 +97,5 @@ 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,12 +2,6 @@
  * 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:
@@ -85,12 +79,6 @@ 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,23 +36,6 @@
       "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

@@ -161,7 +161,6 @@ async function* serveWithVite(serverOptions, builderName, builderAction, context
                     },
                 });
             }
-            yield { baseUrl: '', success: false };
             continue;
         }
         // Clear existing error overlay on successful result
@@ -305,46 +304,6 @@ 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;
-                }
-            });
-            // Setup component HMR invalidation
-            // Invalidation occurs when the runtime cannot update a component
-            server.hot.on('angular:invalidate', (data) => {
-                if (typeof data?.id !== 'string') {
-                    context.logger.warn('Development server client sent invalid internal invalidate event.');
-                }
-                // Clear invalid template update
-                templateUpdates.delete(data.id);
-                // Some cases are expected unsupported update scenarios but some may be errors.
-                // If an error occurred, log the error in addition to the invalidation.
-                if (data.error) {
-                    context.logger.error(`Component update failed${data.message ? `: ${data.message}` : '.'}` +
-                        '\nPlease consider reporting the error at https://github.com/angular/angular-cli/issues');
-                }
-                else {
-                    context.logger.warn(`Component update unsupported${data.message ? `: ${data.message}` : '.'}`);
-                }
-                server?.ws.send({
-                    type: 'full-reload',
-                    path: '*',
-                });
-                context.logger.info('Page reload sent to client(s).');
-            });
             const urls = server.resolvedUrls;
             if (urls && (urls.local.length || urls.network.length)) {
                 serverUrl = new URL(urls.local[0] ?? urls.network[0]);
@@ -461,7 +420,7 @@ function handleUpdate(server, serverOptions, logger, componentStyles, updatedFil
                     type: 'update',
                     updates,
                 });
-                logger.info('Stylesheet update sent to client(s).');
+                logger.info('HMR update sent to client(s).');
                 return;
             }
         }
@@ -574,14 +533,6 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
             mainFields: ['es2020', 'browser', 'module', 'main'],
             preserveSymlinks,
         },
-        dev: {
-            // This is needed when `externalDependencies` is used to prevent Vite load errors.
-            // NOTE: If Vite adds direct support for externals, this can be removed.
-            // NOTE: Vite breaks the resolution of browser modules in SSR
-            //       when accessing a url with two or more segments (e.g., 'foo/bar'),
-            //       as they are not re-based from the base href.
-            preTransformRequests: externalMetadata.explicitBrowser.length === 0 && ssrMode === plugins_1.ServerSsrMode.NoSsr,
-        },
         server: {
             warmup: {
                 ssrFiles,
@@ -590,7 +541,6 @@ 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,
@@ -619,6 +569,9 @@ async function setupServer(serverOptions, outputFiles, assets, preserveSymlinks,
                     ...[...assets.values()].map(({ source }) => source),
                 ],
             },
+            // This is needed when `externalDependencies` is used to prevent Vite load errors.
+            // NOTE: If Vite adds direct support for externals, this can be removed.
+            preTransformRequests: externalMetadata.explicitBrowser.length === 0,
         },
         ssr: {
             // Note: `true` and `/.*/` have different sematics. When true, the `external` option is ignored.

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

@@ -80,9 +80,6 @@ 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/plugins/angular-memory-plugin.js

@@ -47,13 +47,24 @@ async function createAngularMemoryPlugin(options) {
                 // `/@id/${source}` but is currently closer to a raw external than a resolved file path.
                 return source;
             }
-            if (importer && source[0] === '.') {
+            if (importer) {
                 const normalizedImporter = normalizePath(importer);
-                if (normalizedImporter.startsWith(virtualProjectRoot)) {
+                if (source[0] === '.' && normalizedImporter.startsWith(virtualProjectRoot)) {
                     // Remove query if present
                     const [importerFile] = normalizedImporter.split('?', 1);
                     source = '/' + (0, node_path_1.join)((0, node_path_1.dirname)((0, node_path_1.relative)(virtualProjectRoot, importerFile)), source);
                 }
+                else if (!ssr &&
+                    source[0] === '/' &&
+                    importer.endsWith('index.html') &&
+                    normalizedImporter.startsWith(virtualProjectRoot)) {
+                    // This is only needed when using SSR and `angularSsrMiddleware` (old style) to correctly resolve
+                    // .js files when using lazy-loading.
+                    // Remove query if present
+                    const [importerFile] = normalizedImporter.split('?', 1);
+                    source =
+                        '/' + (0, node_path_1.join)((0, node_path_1.dirname)((0, node_path_1.relative)(virtualProjectRoot, importerFile)), (0, node_path_1.basename)(source));
+                }
             }
             const [file] = source.split('?', 1);
             if (outputFiles.has(normalizePath(file))) {

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.6';
+const VERSION = '19.2.0-next.0';
 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, inputFileSystem?: typeof fsPromises, outputFileSystem?: typeof fsPromises): Promise<void>;
+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 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, inputFileSystem = node_fs_1.promises, outputFileSystem = node_fs_1.promises) {
+async function augmentAppWithServiceWorker(appRoot, workspaceRoot, outputPath, baseHref, ngswConfigPath, inputputFileSystem = 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 inputFileSystem.readFile(configPath, 'utf-8');
+        const configurationData = await inputputFileSystem.readFile(configPath, 'utf-8');
         config = JSON.parse(configurationData);
     }
     catch (error) {
@@ -158,7 +158,11 @@ 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 outputFileSystem.writeFile(resolvedDest, await inputFileSystem.readFile(src));
+        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));
     };
     await outputFileSystem.writeFile(path.join(outputPath, 'ngsw.json'), result.manifest);
     for (const { source, destination } of result.assetFiles) {