cargdev/angular
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(bazel): only lookup amd module-name tags in .d.ts files (#25710)

Browse Source 
PR Close #25710
This commit is contained in:
Alex Eagle 2018-08-28 16:40:30 -07:00 committed by Matias Niemelä
parent 2194b5a5c3
commit 7aff3641a1
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
packages/bazel/src/ngc-wrapped/index.ts
View File

@ -222,6 +222,12 @@ export function compile({allowNonHermeticReads, allDepsCompiledWithBazel = true,
const ngHost = ng.createCompilerHost({options: compilerOpts, tsHost: bazelHost});
ngHost.fileNameToModuleName = (importedFilePath: string, containingFilePath: string) => {
// Lookup the module name specified in the TypeScript source file, if present
// it will look like ///<amd module-name="something"/>
// For performance, we only do this for .d.ts files, to avoid parsing lots of
// additional files that were not in the program.
// (We don't have the ts.Program available in this codepath)
if (importedFilePath.endsWith('.d.ts')) {
try {
const sourceFile = ngHost.getSourceFile(importedFilePath, ts.ScriptTarget.Latest);
if (sourceFile && sourceFile.moduleName) {
@ -231,6 +237,7 @@ export function compile({allowNonHermeticReads, allDepsCompiledWithBazel = true,
// File does not exist or parse error. Ignore this case and continue onto the
// other methods of resolving the module below.
}
}
if ((compilerOpts.module === ts.ModuleKind.UMD || compilerOpts.module === ts.ModuleKind.AMD) &&
ngHost.amdModuleName) {
return ngHost.amdModuleName({ fileName: importedFilePath } as ts.SourceFile);