feat(bazel): allow passing and rewriting an old bazel host (#31381)

ayazhafiz · alxhub · commit 11a208f55cea · 2019-07-02T11:28:41.000-07:00
Updates the decision made in #31341; this is for the Angular indexer inside Google. The indexer currently passes (and ngc-wrapped#compile accepts) a bazel host to use, but because many methods are overwritten specially for Angular compilation a better approach is to pass an old bazel compiler host and siphon methods needed off of it before creating a new host. This enables that. PR Close #31381
diff --git a/packages/bazel/src/ngc-wrapped/index.ts b/packages/bazel/src/ngc-wrapped/index.ts
@@ -145,15 +145,15 @@ export function relativeToRootDirs(filePath: string, rootDirs: string[]): string
 }
 
 export function compile({allDepsCompiledWithBazel = true, compilerOpts, tsHost, bazelOpts, files,
-                         inputs, expectedOuts, gatherDiagnostics, bazelHost}: {
+                         inputs, expectedOuts, gatherDiagnostics, oldBazelHost}: {
   allDepsCompiledWithBazel?: boolean,
   compilerOpts: ng.CompilerOptions,
   tsHost: ts.CompilerHost, inputs?: {[path: string]: string},
   bazelOpts: BazelOptions,
   files: string[],
   expectedOuts: string[],
   gatherDiagnostics?: (program: ng.Program) => ng.Diagnostics,
-  bazelHost?: CompilerHost,
+  oldBazelHost?: CompilerHost,
 }): {diagnostics: ng.Diagnostics, program: ng.Program} {
   let fileLoader: FileLoader;
 
@@ -246,9 +246,12 @@ export function compile({allDepsCompiledWithBazel = true, compilerOpts, tsHost,
         moduleName, containingFile, compilerOptions, generatedFileModuleResolverHost);
   }
 
-  if (!bazelHost) {
-    bazelHost = new CompilerHost(
-        files, compilerOpts, bazelOpts, tsHost, fileLoader, generatedFileModuleResolver);
+  const bazelHost = new CompilerHost(
+      files, compilerOpts, bazelOpts, tsHost, fileLoader, generatedFileModuleResolver);
+  if (oldBazelHost) {
+    // TODO(ayazhafiz): this kind of patching is hacky. Revisit this after the
+    // indexer consumer of this code is known to be working.
+    Object.assign(bazelHost, oldBazelHost);
   }
 
   // Also need to disable decorator downleveling in the BazelHost in Ivy mode.
