refactor(compiler): add `ModuleInfo` interface (#38320)

This introduces a new `ModuleInfo` interface to represent some of the statically analyzed data from an `NgModule`. This
gets passed into transforms to give them more context around a given `NgModule` in the compilation.

PR Close #38320

packages/compiler-cli/src/ngtsc/annotations/src/ng_module.ts

@@ -307,7 +307,10 @@ export class NgModuleDecoratorHandler implements
     });
 
     if (this.factoryTracker !== null) {
-      this.factoryTracker.track(node.getSourceFile(), analysis.factorySymbolName);
+      this.factoryTracker.track(node.getSourceFile(), {
+        name: analysis.factorySymbolName,
+        hasId: !!analysis.id,
+      });
     }
 
     this.injectableRegistry.registerInjectable(node);

packages/compiler-cli/src/ngtsc/shims/api.ts

@@ -61,10 +61,15 @@ export interface PerFileShimGenerator {
 export interface FactoryTracker {
   readonly sourceInfo: Map<string, FactoryInfo>;
 
-  track(sf: ts.SourceFile, factorySymbolName: string): void;
+  track(sf: ts.SourceFile, moduleInfo: ModuleInfo): void;
 }
 
 export interface FactoryInfo {
   sourceFilePath: string;
-  moduleSymbolNames: Set<string>;
-}
+  moduleSymbols: Map<string, ModuleInfo>;
+}
+
+export interface ModuleInfo {
+  name: string;
+  hasId: boolean;
+}

packages/compiler-cli/src/ngtsc/shims/src/factory_generator.ts

@@ -9,7 +9,7 @@ import * as ts from 'typescript';
 
 import {absoluteFromSourceFile, AbsoluteFsPath, basename} from '../../file_system';
 import {ImportRewriter} from '../../imports';
-import {FactoryInfo, FactoryTracker, PerFileShimGenerator} from '../api';
+import {FactoryInfo, FactoryTracker, ModuleInfo, PerFileShimGenerator} from '../api';
 
 import {generatedModuleName} from './util';
 
@@ -22,7 +22,7 @@ const STRIP_NG_FACTORY = /(.*)NgFactory$/;
  */
 export class FactoryGenerator implements PerFileShimGenerator, FactoryTracker {
   readonly sourceInfo = new Map<string, FactoryInfo>();
-  private sourceToFactorySymbols = new Map<string, Set<string>>();
+  private sourceToFactorySymbols = new Map<string, Map<string, ModuleInfo>>();
 
   readonly shouldEmit = true;
   readonly extensionPrefix = 'ngfactory';
@@ -85,16 +85,19 @@ export class FactoryGenerator implements PerFileShimGenerator, FactoryTracker {
       genFile.moduleName = generatedModuleName(sf.moduleName, sf.fileName, '.ngfactory');
     }
 
-    const moduleSymbolNames = new Set<string>();
-    this.sourceToFactorySymbols.set(absoluteSfPath, moduleSymbolNames);
-    this.sourceInfo.set(genFilePath, {sourceFilePath: absoluteSfPath, moduleSymbolNames});
+    const moduleSymbols = new Map<string, ModuleInfo>();
+    this.sourceToFactorySymbols.set(absoluteSfPath, moduleSymbols);
+    this.sourceInfo.set(genFilePath, {
+      sourceFilePath: absoluteSfPath,
+      moduleSymbols,
+    });
 
     return genFile;
   }
 
-  track(sf: ts.SourceFile, factorySymbolName: string): void {
+  track(sf: ts.SourceFile, moduleInfo: ModuleInfo): void {
     if (this.sourceToFactorySymbols.has(sf.fileName)) {
-      this.sourceToFactorySymbols.get(sf.fileName)!.add(factorySymbolName);
+      this.sourceToFactorySymbols.get(sf.fileName)!.set(moduleInfo.name, moduleInfo);
     }
   }
 }
@@ -123,7 +126,7 @@ function transformFactorySourceFile(
     return file;
   }
 
-  const {moduleSymbolNames, sourceFilePath} = factoryMap.get(file.fileName)!;
+  const {moduleSymbols, sourceFilePath} = factoryMap.get(file.fileName)!;
 
   file = ts.getMutableClone(file);
 
@@ -183,7 +186,7 @@ function transformFactorySourceFile(
 
         // Otherwise, check if this export is a factory for a known NgModule, and retain it if so.
         const match = STRIP_NG_FACTORY.exec(decl.name.text);
-        if (match !== null && moduleSymbolNames.has(match[1])) {
+        if (match !== null && moduleSymbols.has(match[1])) {
           transformedStatements.push(stmt);
         }
       } else {