refactor(@angular/ssr): replace Map with Record in SSR manifest

Replaced `Map` with `Record` in SSR manifest to simplify structure and improve testing/setup.

packages/angular/build/src/utils/server-rendering/manifest.ts

@@ -55,7 +55,7 @@ export function generateAngularServerAppEngineManifest(
   i18nOptions: NormalizedApplicationBuildOptions['i18nOptions'],
   baseHref: string | undefined,
 ): string {
-  const entryPointsContent: string[] = [];
+  const entryPoints: Record<string, string> = {};
 
   if (i18nOptions.shouldInline) {
     for (const locale of i18nOptions.inlineLocales) {
@@ -69,16 +69,20 @@ export function generateAngularServerAppEngineManifest(
       const end = localeWithBaseHref[localeWithBaseHref.length - 1] === '/' ? -1 : undefined;
       localeWithBaseHref = localeWithBaseHref.slice(start, end);
 
-      entryPointsContent.push(`['${localeWithBaseHref}', () => import('${importPath}')]`);
+      entryPoints[localeWithBaseHref] = `() => import('${importPath}')`;
     }
   } else {
-    entryPointsContent.push(`['', () => import('./${MAIN_SERVER_OUTPUT_FILENAME}')]`);
+    entryPoints[''] = `() => import('./${MAIN_SERVER_OUTPUT_FILENAME}')`;
   }
 
   const manifestContent = `
 export default {
   basePath: '${baseHref ?? '/'}',
-  entryPoints: new Map([${entryPointsContent.join(', \n')}]),
+  entryPoints: {
+    ${Object.entries(entryPoints)
+      .map(([key, value]) => `'${key}': ${value}`)
+      .join(',\n    ')}
+  },
 };
 `;
 
@@ -122,7 +126,7 @@ export function generateAngularServerAppManifest(
   serverAssetsChunks: BuildOutputFile[];
 } {
   const serverAssetsChunks: BuildOutputFile[] = [];
-  const serverAssetsContent: string[] = [];
+  const serverAssets: Record<string, string> = {};
   for (const file of [...additionalHtmlOutputFiles.values(), ...outputFiles]) {
     const extension = extname(file.path);
     if (extension === '.html' || (inlineCriticalCss && extension === '.css')) {
@@ -135,9 +139,8 @@ export function generateAngularServerAppManifest(
         ),
       );
 
-      serverAssetsContent.push(
-        `['${file.path}', {size: ${file.size}, hash: '${file.hash}', text: () => import('./${jsChunkFilePath}').then(m => m.default)}]`,
-      );
+      serverAssets[file.path] =
+        `{size: ${file.size}, hash: '${file.hash}', text: () => import('./${jsChunkFilePath}').then(m => m.default)}`;
     }
   }
 
@@ -146,9 +149,13 @@ export default {
   bootstrap: () => import('./main.server.mjs').then(m => m.default),
   inlineCriticalCss: ${inlineCriticalCss},
   baseHref: '${baseHref}',
-  locale: ${locale !== undefined ? `'${locale}'` : undefined},
+  locale: ${JSON.stringify(locale)},
   routes: ${JSON.stringify(routes, undefined, 2)},
-  assets: new Map([\n${serverAssetsContent.join(', \n')}\n]),
+  assets: {
+    ${Object.entries(serverAssets)
+      .map(([key, value]) => `'${key}': ${value}`)
+      .join(',\n    ')}
+  },
 };
 `;
 

packages/angular/ssr/src/app-engine.ts

@@ -46,6 +46,11 @@ export class AngularAppEngine {
    */
   private readonly manifest = getAngularAppEngineManifest();
 
+  /**
+   * The number of entry points available in the server application's manifest.
+   */
+  private readonly entryPointsCount = Object.keys(this.manifest.entryPoints).length;
+
   /**
    * A cache that holds entry points, keyed by their potential locale string.
    */
@@ -113,7 +118,7 @@ export class AngularAppEngine {
     }
 
     const { entryPoints } = this.manifest;
-    const entryPoint = entryPoints.get(potentialLocale);
+    const entryPoint = entryPoints[potentialLocale];
     if (!entryPoint) {
       return undefined;
     }
@@ -136,8 +141,8 @@ export class AngularAppEngine {
    * @returns A promise that resolves to the entry point exports or `undefined` if not found.
    */
   private getEntryPointExportsForUrl(url: URL): Promise<EntryPointExports> | undefined {
-    const { entryPoints, basePath } = this.manifest;
-    if (entryPoints.size === 1) {
+    const { basePath } = this.manifest;
+    if (this.entryPointsCount === 1) {
       return this.getEntryPointExports('');
     }
 

packages/angular/ssr/src/assets.ts

@@ -27,7 +27,7 @@ export class ServerAssets {
    * @throws Error - Throws an error if the asset does not exist.
    */
   getServerAsset(path: string): ServerAsset {
-    const asset = this.manifest.assets.get(path);
+    const asset = this.manifest.assets[path];
     if (!asset) {
       throw new Error(`Server asset '${path}' does not exist.`);
     }
@@ -42,7 +42,7 @@ export class ServerAssets {
    * @returns A boolean indicating whether the asset exists.
    */
   hasServerAsset(path: string): boolean {
-    return this.manifest.assets.has(path);
+    return !!this.manifest.assets[path];
   }
 
   /**

packages/angular/ssr/src/manifest.ts

@@ -10,7 +10,7 @@ import type { SerializableRouteTreeNode } from './routes/route-tree';
 import { AngularBootstrap } from './utils/ng';
 
 /**
- * Represents of a server asset stored in the manifest.
+ * Represents a server asset stored in the manifest.
  */
 export interface ServerAsset {
   /**
@@ -53,12 +53,12 @@ export interface EntryPointExports {
  */
 export interface AngularAppEngineManifest {
   /**
-   * A map of entry points for the server application.
-   * Each entry in the map consists of:
+   * A readonly record of entry points for the server application.
+   * Each entry consists of:
    * - `key`: The base href for the entry point.
    * - `value`: A function that returns a promise resolving to an object of type `EntryPointExports`.
    */
-  readonly entryPoints: ReadonlyMap<string, () => Promise<EntryPointExports>>;
+  readonly entryPoints: Readonly<Record<string, (() => Promise<EntryPointExports>) | undefined>>;
 
   /**
    * The base path for the server application.
@@ -78,12 +78,12 @@ export interface AngularAppManifest {
   readonly baseHref: string;
 
   /**
-   * A map of assets required by the server application.
-   * Each entry in the map consists of:
+   * A readonly record of assets required by the server application.
+   * Each entry consists of:
    * - `key`: The path of the asset.
-   * - `value`: A function returning a promise that resolves to the file contents of the asset.
+   * - `value`: An object of type `ServerAsset`.
    */
-  readonly assets: ReadonlyMap<string, ServerAsset>;
+  readonly assets: Readonly<Record<string, ServerAsset | undefined>>;
 
   /**
    * The bootstrap mechanism for the server application.

packages/angular/ssr/test/app-engine_spec.ts

@@ -18,20 +18,8 @@ import { setAngularAppEngineManifest } from '../src/manifest';
 import { RenderMode } from '../src/routes/route-config';
 import { setAngularAppTestingManifest } from './testing-utils';
 
-describe('AngularAppEngine', () => {
-  let appEngine: AngularAppEngine;
-
-  describe('Localized app', () => {
-    beforeAll(() => {
-      destroyAngularServerApp();
-
-      setAngularAppEngineManifest({
-        // Note: Although we are testing only one locale, we need to configure two or more
-        // to ensure that we test a different code path.
-        entryPoints: new Map(
-          ['it', 'en'].map((locale) => [
-            locale,
-            async () => {
+function createEntryPoint(locale: string) {
+  return async () => {
     @Component({
       standalone: true,
       selector: `app-ssr-${locale}`,
@@ -78,9 +66,23 @@ describe('AngularAppEngine', () => {
       ɵgetOrCreateAngularServerApp: getOrCreateAngularServerApp,
       ɵdestroyAngularServerApp: destroyAngularServerApp,
     };
+  };
+}
+
+describe('AngularAppEngine', () => {
+  let appEngine: AngularAppEngine;
+
+  describe('Localized app', () => {
+    beforeAll(() => {
+      destroyAngularServerApp();
+
+      setAngularAppEngineManifest({
+        // Note: Although we are testing only one locale, we need to configure two or more
+        // to ensure that we test a different code path.
+        entryPoints: {
+          it: createEntryPoint('it'),
+          en: createEntryPoint('en'),
         },
-          ]),
-        ),
         basePath: '',
       });
 
@@ -143,10 +145,8 @@ describe('AngularAppEngine', () => {
       destroyAngularServerApp();
 
       setAngularAppEngineManifest({
-        entryPoints: new Map([
-          [
-            '',
-            async () => {
+        entryPoints: {
+          '': async () => {
             @Component({
               standalone: true,
               selector: 'app-home',
@@ -164,8 +164,7 @@ describe('AngularAppEngine', () => {
               ɵdestroyAngularServerApp: destroyAngularServerApp,
             };
           },
-          ],
-        ]),
+        },
         basePath: '',
       });
 

packages/angular/ssr/test/assets_spec.ts

@@ -15,8 +15,7 @@ describe('ServerAsset', () => {
     assetManager = new ServerAssets({
       baseHref: '/',
       bootstrap: undefined as never,
-      assets: new Map(
-        Object.entries({
+      assets: {
         'index.server.html': {
           text: async () => '<html>Index</html>',
           size: 18,
@@ -27,8 +26,7 @@ describe('ServerAsset', () => {
           size: 18,
           hash: '4a455a99366921d396f5d51c7253c4678764f5e9487f2c27baaa0f33553c8ce3',
         },
-        }),
-      ),
+      },
     });
   });
 

packages/angular/ssr/test/testing-utils.ts

@@ -32,8 +32,7 @@ export function setAngularAppTestingManifest(
   setAngularAppManifest({
     inlineCriticalCss: false,
     baseHref,
-    assets: new Map(
-      Object.entries({
+    assets: {
       ...additionalServerAssets,
       'index.server.html': {
         size: 25,
@@ -64,8 +63,7 @@ export function setAngularAppTestingManifest(
         </html>
       `,
       },
-      }),
-    ),
+    },
     bootstrap: async () => () => {
       @Component({
         standalone: true,