Parse the same files as the compiler

Author: jablko

packages/definitions-parser/src/lib/definition-parser.ts

@@ -264,7 +264,12 @@ function getTypingDataForSingleTypesVersion(
 ): TypingDataFromIndividualTypeScriptVersion {
   const tsconfig = fs.readJson("tsconfig.json") as TsConfig;
   checkFilesFromTsConfig(packageName, tsconfig, fs.debugPath());
-  const { types, tests } = allReferencedFiles(tsconfig.files!, fs, packageName, packageDirectory);
+  const { types, tests, hasNonRelativeImports } = allReferencedFiles(
+    tsconfig.files!,
+    fs,
+    packageName,
+    packageDirectory
+  );
   const usedFiles = new Set([...types.keys(), ...tests.keys(), "tsconfig.json", "tslint.json"]);
   const otherFiles =
     ls.indexOf(unusedFilesName) > -1
@@ -289,6 +294,16 @@ function getTypingDataForSingleTypesVersion(
     filter(getTestDependencies(packageName, types, tests.keys(), dependenciesSet, fs), m => !declaredModulesSet.has(m))
   );
 
+  const { paths } = tsconfig.compilerOptions;
+  if (directoryVersion !== undefined && !(paths && `${packageName}/*` in paths) && hasNonRelativeImports) {
+    const mapping = JSON.stringify([`${packageName}/v${formatTypingVersion(directoryVersion)}/*`]);
+    throw new Error(
+      `${packageName}: Older version ${formatTypingVersion(
+        directoryVersion
+      )} must have a "paths" entry of "${packageName}/*": ${mapping}`
+    );
+  }
+
   const { dependencies, pathMappings } = calculateDependencies(
     packageName,
     tsconfig,

packages/definitions-parser/src/lib/module-info.ts

@@ -147,12 +147,13 @@ export function allReferencedFiles(
   fs: FS,
   packageName: string,
   baseDirectory: string
-): { types: Map<string, ts.SourceFile>; tests: Map<string, ts.SourceFile> } {
+): { types: Map<string, ts.SourceFile>; tests: Map<string, ts.SourceFile>; hasNonRelativeImports: boolean } {
   const seenReferences = new Set<string>();
   const types = new Map<string, ts.SourceFile>();
   const tests = new Map<string, ts.SourceFile>();
+  let hasNonRelativeImports = false;
   entryFilenames.forEach(text => recur({ text, exact: true }));
-  return { types, tests };
+  return { types, tests, hasNonRelativeImports };
 
   function recur({ text, exact }: Reference): void {
     if (seenReferences.has(text)) {
@@ -170,13 +171,14 @@ export function allReferencedFiles(
         tests.set(resolvedFilename, src);
       }
 
-      const refs = findReferencedFiles(
+      const { refs, hasNonRelativeImports: result } = findReferencedFiles(
         src,
         packageName,
         path.dirname(resolvedFilename),
         normalizeSlashes(path.relative(baseDirectory, fs.debugPath()))
       );
       refs.forEach(recur);
+      hasNonRelativeImports = hasNonRelativeImports || result;
     }
   }
 }
@@ -215,6 +217,7 @@ interface Reference {
  */
 function findReferencedFiles(src: ts.SourceFile, packageName: string, subDirectory: string, baseDirectory: string) {
   const refs: Reference[] = [];
+  let hasNonRelativeImports = false;
 
   for (const ref of src.referencedFiles) {
     // Any <reference path="foo"> is assumed to be local
@@ -235,9 +238,10 @@ function findReferencedFiles(src: ts.SourceFile, packageName: string, subDirecto
     }
     if (ref.startsWith(packageName + "/")) {
       addReference({ text: convertToRelativeReference(ref), exact: false });
+      hasNonRelativeImports = true;
     }
   }
-  return refs;
+  return { refs, hasNonRelativeImports };
 
   function addReference(ref: Reference): void {
     // `path.normalize` may add windows slashes