fix(@angular-devkit/build-angular): export @angular/platform-server symbols in server bundle

This commit adds an internal file to export needed symbols from `@angular/platform-server` when building a server bundle. This is needed.  This is needed so that DI tokens can be referenced and set at runtime outside of the bundle.

Also, it adds a migration to remove these exports from the users files as otherwise an export collition would occur due to the same symbol being exported multiple times.

Author: alan-agius4

packages/angular_devkit/build_angular/src/builders/server/index.ts

@@ -68,6 +68,8 @@ export function execute(
 
   return from(initialize(options, context, transforms.webpackConfiguration)).pipe(
     concatMap(({ config, i18n, target }) => {
+      addPlatformServerExportsAsEntry(config, root);
+
       return runWebpack(config, context, {
         webpackFactory: require('webpack') as typeof webpack,
         logging: (stats, config) => {
@@ -164,3 +166,27 @@ async function initialize(
 
   return { config: transformedConfig || config, i18n, target };
 }
+/**
+ * Add `@angular/platform-server` exports as an entry-point.
+ * This is needed so that DI tokens can be referenced and set at runtime outside of the bundle.
+ */
+function addPlatformServerExportsAsEntry(config: webpack.Configuration, root: string): void {
+  try {
+    // Only add `@angular/platform-server` exports when it is installed.
+    // In some cases this builder is used when `@angular/platform-server` is not installed.
+    // Example: when using `@nguniversal/common/clover` which does not need `@angular/platform-server`.
+    require.resolve('@angular/platform-server', { paths: [root] });
+  } catch {
+    return;
+  }
+
+  const platformServerExportsFile = path.join(__dirname, 'platform-server-exports.js');
+
+  if (typeof config.entry === 'object' && !Array.isArray(config.entry) && config.entry['main']) {
+    if (Array.isArray(config.entry['main'])) {
+      config.entry['main'].push(platformServerExportsFile);
+    } else {
+      config.entry['main'] = [platformServerExportsFile, config.entry['main'] as string];
+    }
+  }
+}

packages/angular_devkit/build_angular/src/builders/server/platform-server-exports.js

@@ -0,0 +1,9 @@
+/**
+ * @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
+ */
+
+export { renderModule, renderApplication } from '@angular/platform-server';

packages/angular_devkit/build_angular/test/hello-world-app/src/main.server.ts

@@ -16,4 +16,3 @@ if (environment.production) {
 }
 
 export { AppServerModule } from './app/app.server.module';
-export { renderModule } from '@angular/platform-server';

packages/schematics/angular/migrations/migration-collection.json

@@ -4,6 +4,11 @@
       "version": "15.0.0",
       "factory": "./update-15/remove-browserslist-config",
       "description": "Remove Browserslist configuration files that matches the Angular CLI default configuration."
+    },
+    "remove-platform-server-exports": {
+      "version": "15.0.0",
+      "factory": "./update-15/remove-platform-server-exports",
+      "description": "Remove `@angular/platform-server` exported rendering methods. The `renderModule` and `renderApplication` methods are now exported by the Angular CLI."
     }
   }
 }

packages/schematics/angular/migrations/update-15/remove-platform-server-exports.ts

@@ -0,0 +1,104 @@
+/**
+ * @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 { DirEntry, Rule, UpdateRecorder } from '@angular-devkit/schematics';
+import * as ts from '../../third_party/github.com/Microsoft/TypeScript/lib/typescript';
+
+const PLATFORM_SERVER_EXPORTS_TO_DELETE = new Set(['renderModule', 'renderApplication']);
+
+function* visit(directory: DirEntry): IterableIterator<ts.SourceFile> {
+  for (const path of directory.subfiles) {
+    if (path.endsWith('.ts') && !path.endsWith('.d.ts')) {
+      const entry = directory.file(path);
+      if (entry) {
+        const content = entry.content;
+        if (
+          content.includes('@angular/platform-server') &&
+          [...PLATFORM_SERVER_EXPORTS_TO_DELETE].some((exportName) => content.includes(exportName))
+        ) {
+          const source = ts.createSourceFile(
+            entry.path,
+            content.toString().replace(/^\uFEFF/, ''),
+            ts.ScriptTarget.Latest,
+            true,
+          );
+
+          yield source;
+        }
+      }
+    }
+  }
+
+  for (const path of directory.subdirs) {
+    if (path === 'node_modules' || path.startsWith('.')) {
+      continue;
+    }
+
+    yield* visit(directory.dir(path));
+  }
+}
+
+export default function (): Rule {
+  return (tree) => {
+    for (const sourceFile of visit(tree.root)) {
+      let recorder: UpdateRecorder | undefined;
+      let printer: ts.Printer | undefined;
+
+      ts.forEachChild(sourceFile, function analyze(node) {
+        if (
+          !(
+            ts.isExportDeclaration(node) &&
+            node.moduleSpecifier &&
+            ts.isStringLiteral(node.moduleSpecifier) &&
+            node.moduleSpecifier.text === '@angular/platform-server' &&
+            node.exportClause &&
+            ts.isNamedExports(node.exportClause)
+          )
+        ) {
+          // Not a @angular/platform-server named export.
+          return;
+        }
+
+        const exportClause = node.exportClause;
+        const newElements: ts.ExportSpecifier[] = [];
+        for (const element of exportClause.elements) {
+          if (!PLATFORM_SERVER_EXPORTS_TO_DELETE.has(element.name.text)) {
+            newElements.push(element);
+          }
+        }
+
+        if (newElements.length === exportClause.elements.length) {
+          // No changes
+          return;
+        }
+
+        recorder ??= tree.beginUpdate(sourceFile.fileName);
+
+        if (newElements.length) {
+          // Update named exports and there are leftovers.
+          const newExportClause = ts.factory.updateNamedExports(exportClause, newElements);
+          printer ??= ts.createPrinter();
+          const fix = printer.printNode(ts.EmitHint.Unspecified, newExportClause, sourceFile);
+
+          const index = exportClause.getStart();
+          const length = exportClause.getWidth();
+          recorder.remove(index, length).insertLeft(index, fix);
+        } else {
+          // Delete export as no exports remain.
+          recorder.remove(node.getStart(), node.getWidth());
+        }
+
+        ts.forEachChild(node, analyze);
+      });
+
+      if (recorder) {
+        tree.commitUpdate(recorder);
+      }
+    }
+  };
+}

packages/schematics/angular/migrations/update-15/remove-platform-server-exports_spec.ts

@@ -0,0 +1,90 @@
+/**
+ * @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 { EmptyTree } from '@angular-devkit/schematics';
+import { SchematicTestRunner, UnitTestTree } from '@angular-devkit/schematics/testing';
+
+describe('Migration to delete platform-server exports', () => {
+  const schematicName = 'remove-platform-server-exports';
+
+  const schematicRunner = new SchematicTestRunner(
+    'migrations',
+    require.resolve('../migration-collection.json'),
+  );
+
+  let tree: UnitTestTree;
+
+  beforeEach(() => {
+    tree = new UnitTestTree(new EmptyTree());
+  });
+
+  const testTypeScriptFilePath = './test.ts';
+
+  describe('Migration to import() style lazy routes', () => {
+    beforeEach(async () => {
+      tree = new UnitTestTree(new EmptyTree());
+      tree.create('/package.json', JSON.stringify({}));
+    });
+
+    it(`should delete '@angular/platform-server' export when 'renderModule' is the only exported symbol`, async () => {
+      tree.create(
+        testTypeScriptFilePath,
+        `
+          import { Path, join } from '@angular-devkit/core';
+          export { renderModule } from '@angular/platform-server';
+        `,
+      );
+
+      const newTree = await schematicRunner.runSchematicAsync(schematicName, {}, tree).toPromise();
+      const content = newTree.readContent(testTypeScriptFilePath);
+      expect(content).not.toContain('@angular/platform-server');
+      expect(content).toContain(`import { Path, join } from '@angular-devkit/core';`);
+    });
+
+    it(`should delete only 'renderModule' and 'renderApplication' when there are additional exports`, async () => {
+      tree.create(
+        testTypeScriptFilePath,
+        `
+          import { Path, join } from '@angular-devkit/core';
+          export { renderModule, ServerModule, renderApplication } from '@angular/platform-server';
+        `,
+      );
+
+      const newTree = await schematicRunner.runSchematicAsync(schematicName, {}, tree).toPromise();
+      const content = newTree.readContent(testTypeScriptFilePath);
+      expect(content).toContain(`import { Path, join } from '@angular-devkit/core';`);
+      expect(content).toContain(`export { ServerModule } from '@angular/platform-server';`);
+    });
+
+    it(`should not delete 'renderModule' when it's exported from another module`, async () => {
+      tree.create(
+        testTypeScriptFilePath,
+        `
+          export { renderModule } from '@angular/core';
+        `,
+      );
+
+      const newTree = await schematicRunner.runSchematicAsync(schematicName, {}, tree).toPromise();
+      const content = newTree.readContent(testTypeScriptFilePath);
+      expect(content).toContain(`export { renderModule } from '@angular/core';`);
+    });
+
+    it(`should not delete 'renderModule' when it's imported from '@angular/platform-server'`, async () => {
+      tree.create(
+        testTypeScriptFilePath,
+        `
+          import { renderModule } from '@angular/platform-server';
+        `,
+      );
+
+      const newTree = await schematicRunner.runSchematicAsync(schematicName, {}, tree).toPromise();
+      const content = newTree.readContent(testTypeScriptFilePath);
+      expect(content).toContain(`import { renderModule } from '@angular/platform-server'`);
+    });
+  });
+});

packages/schematics/angular/universal/files/src/__main@stripTsExtension__.ts.template

@@ -21,4 +21,3 @@ if (environment.production) {
 }
 
 export { <%= rootModuleClassName %> } from './app/<%= stripTsExtension(rootModuleFileName) %>';
-export { renderModule } from '@angular/platform-server';

tests/legacy-cli/e2e/tests/build/platform-server.ts

@@ -32,7 +32,8 @@ export default async function () {
     './server.ts',
     ` import 'zone.js/dist/zone-node';
         import * as fs from 'fs';
-        import { AppServerModule, renderModule } from './src/main.server';
+        import { renderModule } from '@angular/platform-server';
+        import { AppServerModule } from './src/main.server';
 
         renderModule(AppServerModule, {
           url: '/',