fix(metadata): emit metadata rooted at 'angular2'

fixes #8144

closes #8147

tools/broccoli/broccoli-typescript.ts

@@ -88,7 +88,16 @@ class DiffingTSCompiler implements DiffingBroccoliPlugin {
     this.tsServiceHost = new CustomLanguageServiceHost(this.tsOpts, this.rootFilePaths,
                                                        this.fileRegistry, this.inputPath);
     this.tsService = ts.createLanguageService(this.tsServiceHost, ts.createDocumentRegistry());
-    this.metadataCollector = new MetadataCollector();
+    this.metadataCollector = new MetadataCollector({
+      // Since our code isn't under a node_modules directory, we need to reverse the module
+      // resolution to get metadata rooted at 'angular2'.
+      // see https://github.com/angular/angular/issues/8144
+      reverseModuleResolution(fileName: string) {
+        if (/\.tmp\/angular2/.test(fileName)) {
+          return fileName.substr(fileName.lastIndexOf('.tmp/angular2/') + 5).replace(/\.ts$/, '');
+        }
+      }
+    });
   }
 
 

tools/metadata/src/collector.ts

@@ -27,10 +27,15 @@ function pathTo(from: string, to: string): string {
   return result;
 }
 
-function moduleNameFromBaseName(moduleFileName: string, baseFileName: string): string {
-  // Remove the extension
-  moduleFileName = moduleFileName.replace(EXT_REGEX, '');
+export interface MetadataCollectorHost {
+  reverseModuleResolution: (moduleFileName: string) => string;
+}
 
+const nodeModuleResolutionHost: MetadataCollectorHost = {
+  // Reverse moduleResolution=node for packages resolved in node_modules
+  reverseModuleResolution(fileName: string) {
+    // Remove the extension
+    const moduleFileName = fileName.replace(EXT_REGEX, '');
     // Check for node_modules
     const nodeModulesIndex = moduleFileName.lastIndexOf(NODE_MODULES);
     if (nodeModulesIndex >= 0) {
@@ -39,16 +44,15 @@ function moduleNameFromBaseName(moduleFileName: string, baseFileName: string): s
     if (moduleFileName.lastIndexOf(NODE_MODULES_PREFIX, NODE_MODULES_PREFIX.length) !== -1) {
       return moduleFileName.substr(NODE_MODULES_PREFIX.length);
     }
-
-  // Construct a simplified path from the file to the module
-  return pathTo(baseFileName, moduleFileName);
-}
+    return null;
+  }
+};
 
 /**
  * Collect decorator metadata from a TypeScript module.
  */
 export class MetadataCollector {
-  constructor() {}
+  constructor(private host: MetadataCollectorHost = nodeModuleResolutionHost) {}
 
   /**
    * Returns a JSON.stringify friendly form describing the decorators of the exported classes from
@@ -56,8 +60,16 @@ export class MetadataCollector {
    */
   public getMetadata(sourceFile: ts.SourceFile, typeChecker: ts.TypeChecker): ModuleMetadata {
     const locals = new Symbols();
-    const moduleNameOf = (fileName: string) =>
-        moduleNameFromBaseName(fileName, sourceFile.fileName);
+    const moduleNameOf = (fileName: string) => {
+      // If the module was resolved with TS moduleResolution, reverse that mapping
+      const hostResolved = this.host.reverseModuleResolution(fileName);
+      if (hostResolved) {
+        return hostResolved;
+      }
+      // Construct a simplified path from the file to the module
+      return pathTo(sourceFile.fileName, fileName).replace(EXT_REGEX, '');
+    };
+
     const evaluator = new Evaluator(typeChecker, locals, moduleNameOf);
 
     function objFromDecorator(decoratorNode: ts.Decorator): MetadataSymbolicExpression {
@@ -85,11 +97,9 @@ export class MetadataCollector {
     }
 
     function classMetadataOf(classDeclaration: ts.ClassDeclaration): ClassMetadata {
-      let result: ClassMetadata =
-      { __symbolic: "class" }
+      let result: ClassMetadata = {__symbolic: "class"};
 
-      function getDecorators(decorators: ts.Decorator[]):
-          MetadataSymbolicExpression[] {
+      function getDecorators(decorators: ts.Decorator[]): MetadataSymbolicExpression[] {
         if (decorators && decorators.length)
           return decorators.map(decorator => objFromDecorator(decorator));
         return undefined;
@@ -175,7 +185,7 @@ export class MetadataCollector {
             const classDeclaration = <ts.ClassDeclaration>declaration;
             if (classDeclaration.decorators) {
               if (!metadata) metadata = {};
-              metadata[classDeclaration.name.text] = classMetadataOf(classDeclaration)
+              metadata[classDeclaration.name.text] = classMetadataOf(classDeclaration);
             }
             break;
           case ts.SyntaxKind.VariableDeclaration: