Fix compile on save issues (#10615)

* only return .ts and .tsx files for affected document list
* Return all files exclude mixed-content files for CoS affected list

src/server/builder.ts

@@ -15,6 +15,10 @@ namespace ts.server {
         createHash(algorithm: string): Hash
     } = require("crypto");
 
+    export function shouldEmitFile(scriptInfo: ScriptInfo) {
+        return !scriptInfo.hasMixedContent;
+    }
+
     /**
      * An abstract file info that maintains a shape signature.
      */
@@ -160,16 +164,17 @@ namespace ts.server {
          */
         getFilesAffectedBy(scriptInfo: ScriptInfo): string[] {
             const info = this.getOrCreateFileInfo(scriptInfo.path);
+            const singleFileResult = scriptInfo.hasMixedContent ? [] : [scriptInfo.fileName];
             if (info.updateShapeSignature()) {
                 const options = this.project.getCompilerOptions();
                 // If `--out` or `--outFile` is specified, any new emit will result in re-emitting the entire project,
                 // so returning the file itself is good enough.
                 if (options && (options.out || options.outFile)) {
-                    return [scriptInfo.fileName];
+                    return singleFileResult;
                 }
-                return this.project.getFileNamesWithoutDefaultLib();
+                return this.project.getAllEmittableFiles();
             }
-            return [scriptInfo.fileName];
+            return singleFileResult;
         }
     }
 
@@ -319,18 +324,19 @@ namespace ts.server {
         getFilesAffectedBy(scriptInfo: ScriptInfo): string[] {
             this.ensureProjectDependencyGraphUpToDate();
 
+            const singleFileResult = scriptInfo.hasMixedContent ? [] : [scriptInfo.fileName];
             const fileInfo = this.getFileInfo(scriptInfo.path);
             if (!fileInfo || !fileInfo.updateShapeSignature()) {
-                return [scriptInfo.fileName];
+                return singleFileResult;
             }
 
             if (!fileInfo.isExternalModuleOrHasOnlyAmbientExternalModules()) {
-                return this.project.getFileNamesWithoutDefaultLib();
+                return this.project.getAllEmittableFiles();
             }
 
             const options = this.project.getCompilerOptions();
             if (options && (options.isolatedModules || options.out || options.outFile)) {
-                return [scriptInfo.fileName];
+                return singleFileResult;
             }
 
             // Now we need to if each file in the referencedBy list has a shape change as well.
@@ -339,8 +345,8 @@ namespace ts.server {
 
             // Use slice to clone the array to avoid manipulating in place
             const queue = fileInfo.referencedBy.slice(0);
-            const fileNameSet = createMap<boolean>();
-            fileNameSet[scriptInfo.fileName] = true;
+            const fileNameSet = createMap<ScriptInfo>();
+            fileNameSet[scriptInfo.fileName] = scriptInfo;
             while (queue.length > 0) {
                 const processingFileInfo = queue.pop();
                 if (processingFileInfo.updateShapeSignature() && processingFileInfo.referencedBy.length > 0) {
@@ -350,9 +356,15 @@ namespace ts.server {
                         }
                     }
                 }
-                fileNameSet[processingFileInfo.scriptInfo.fileName] = true;
+                fileNameSet[processingFileInfo.scriptInfo.fileName] = processingFileInfo.scriptInfo;
             }
-            return Object.keys(fileNameSet);
+            const result: string[] = [];
+            for (const fileName in fileNameSet) {
+                if (shouldEmitFile(fileNameSet[fileName])) {
+                    result.push(fileName);
+                }
+            }
+            return result;
         }
     }