Make getFilesByName buildinfo ready

Author: sheetalkamat

src/compiler/program.ts

@@ -824,7 +824,7 @@ namespace ts {
          * - false if sourceFile missing for source of project reference redirect
          * - undefined otherwise
          */
-        const filesByName = new Map<string, SourceFile | false | undefined>();
+        const filesByName = new Map<string, SourceFile | false | 0>();
         let missingFilePaths: readonly Path[] | undefined;
         // stores 'filename -> file association' ignoring case
         // used to track cases when two file names differ only in casing
@@ -921,7 +921,7 @@ namespace ts {
                 }
             }
 
-            missingFilePaths = arrayFrom(mapDefinedIterator(filesByName.entries(), ([path, file]) => file === undefined ? path as Path : undefined));
+            missingFilePaths = arrayFrom(mapDefinedIterator(filesByName.entries(), ([path, file]) => file === 0 ? path as Path : undefined));
             files = stableSort(processingDefaultLibFiles, compareDefaultLibFiles).concat(processingOtherFiles);
             processingDefaultLibFiles = undefined;
             processingOtherFiles = undefined;
@@ -1568,7 +1568,7 @@ namespace ts {
                     }
                     return;
                 }
-                filesByName.set(path, filesByName.get(oldFile.path));
+                filesByName.set(path, filesByName.get(oldFile.path)!);
             });
 
             files = newSourceFiles;
@@ -1658,8 +1658,8 @@ namespace ts {
                 (_ref, index) => resolvedProjectReferences![index]?.commandLine,
                 fileName => {
                     const path = toPath(fileName);
-                    const sourceFile = getSourceFileByPath(path);
-                    return sourceFile ? sourceFile.text : filesByName.has(path) ? undefined : host.readFile(path);
+                    const sourceFile = filesByName.get(path);
+                    return sourceFile ? sourceFile.text : sourceFile !== undefined ? undefined : host.readFile(path);
                 }
             );
         }
@@ -2484,13 +2484,13 @@ namespace ts {
                 }
             }
             const originalFileName = fileName;
-            if (filesByName.has(path)) {
-                const file = filesByName.get(path);
-                addFileToRefFileMap(fileName, file || undefined, refFile);
+            const cachedFile = filesByName.get(path);
+            if (cachedFile !== undefined) {
+                addFileToRefFileMap(fileName, cachedFile || undefined, refFile);
                 // try to check if we've already seen this file but with a different casing in path
                 // NOTE: this only makes sense for case-insensitive file systems, and only on files which are not redirected
-                if (file && options.forceConsistentCasingInFileNames) {
-                    const checkedName = file.fileName;
+                if (cachedFile && options.forceConsistentCasingInFileNames) {
+                    const checkedName = cachedFile.fileName;
                     const isRedirect = toPath(checkedName) !== toPath(fileName);
                     if (isRedirect) {
                         fileName = getProjectReferenceRedirect(fileName) || fileName;
@@ -2499,34 +2499,34 @@ namespace ts {
                     const checkedAbsolutePath = getNormalizedAbsolutePathWithoutRoot(checkedName, currentDirectory);
                     const inputAbsolutePath = getNormalizedAbsolutePathWithoutRoot(fileName, currentDirectory);
                     if (checkedAbsolutePath !== inputAbsolutePath) {
-                        reportFileNamesDifferOnlyInCasingError(fileName, file, refFile);
+                        reportFileNamesDifferOnlyInCasingError(fileName, cachedFile, refFile);
                     }
                 }
 
                 // If the file was previously found via a node_modules search, but is now being processed as a root file,
                 // then everything it sucks in may also be marked incorrectly, and needs to be checked again.
-                if (file && sourceFilesFoundSearchingNodeModules.get(file.path) && currentNodeModulesDepth === 0) {
-                    sourceFilesFoundSearchingNodeModules.set(file.path, false);
+                if (cachedFile && sourceFilesFoundSearchingNodeModules.get(cachedFile.path) && currentNodeModulesDepth === 0) {
+                    sourceFilesFoundSearchingNodeModules.set(cachedFile.path, false);
                     if (!options.noResolve) {
-                        processReferencedFiles(file, isDefaultLib);
-                        processTypeReferenceDirectives(file);
+                        processReferencedFiles(cachedFile, isDefaultLib);
+                        processTypeReferenceDirectives(cachedFile);
                     }
                     if (!options.noLib) {
-                        processLibReferenceDirectives(file);
+                        processLibReferenceDirectives(cachedFile);
                     }
 
-                    modulesWithElidedImports.set(file.path, false);
-                    processImportedModules(file);
+                    modulesWithElidedImports.set(cachedFile.path, false);
+                    processImportedModules(cachedFile);
                 }
                 // See if we need to reprocess the imports due to prior skipped imports
-                else if (file && modulesWithElidedImports.get(file.path)) {
+                else if (cachedFile && modulesWithElidedImports.get(cachedFile.path)) {
                     if (currentNodeModulesDepth < maxNodeModuleJsDepth) {
-                        modulesWithElidedImports.set(file.path, false);
-                        processImportedModules(file);
+                        modulesWithElidedImports.set(cachedFile.path, false);
+                        processImportedModules(cachedFile);
                     }
                 }
 
-                return file || undefined;
+                return cachedFile || undefined;
             }
 
             let redirectedPath: Path | undefined;
@@ -2639,11 +2639,11 @@ namespace ts {
 
         function addFileToFilesByName(file: SourceFile | undefined, path: Path, redirectedPath: Path | undefined) {
             if (redirectedPath) {
-                filesByName.set(redirectedPath, file);
+                filesByName.set(redirectedPath, file || 0);
                 filesByName.set(path, file || false);
             }
             else {
-                filesByName.set(path, file);
+                filesByName.set(path, file || 0);
             }
         }
 

src/compiler/types.ts

@@ -3717,7 +3717,7 @@ namespace ts {
         /* @internal */
         getRefFileMap(): MultiMap<Path, RefFile> | undefined;
         /* @internal */
-        getFilesByNameMap(): ESMap<string, SourceFile | false | undefined>;
+        getFilesByNameMap(): ESMap<string, SourceFile | false | 0>;
 
         /**
          * Emits the JavaScript and declaration files.  If targetSourceFile is not specified, then