Revert "use externalTranspiler before doing diffs for status". Hot fixes #765 which takes priority over babel compile experience 🌹

This reverts commit 900a1da.

Author: basarat

dist/main/lang/modules/building.js

@@ -79,18 +79,6 @@ function getRawOutput(proj, filePath) {
     return output;
 }
 exports.getRawOutput = getRawOutput;
-function getRawOutputPostExternal(proj, filePath) {
-    var output1 = getRawOutput(proj, filePath);
-    var sourceFile = proj.languageService.getSourceFile(filePath);
-    var sourceMapContents = {};
-    return runExternalTranspiler(filePath, sourceFile.text, output1.outputFiles[0], proj, sourceMapContents).then(function () {
-        return {
-            outputFiles: output1.outputFiles,
-            emitSkipped: false
-        };
-    });
-}
-exports.getRawOutputPostExternal = getRawOutputPostExternal;
 function getBabelInstance(projectDirectory) {
     return new Promise(function (resolve) {
         if (!babels[projectDirectory]) {

dist/main/lang/projectService.js

@@ -600,27 +600,23 @@ exports.getOutputJs = getOutputJs;
 function getOutputJsStatus(query) {
     projectCache_1.consistentPath(query);
     var project = projectCache_1.getOrCreateProject(query.filePath);
-    return building_1.getRawOutputPostExternal(project, query.filePath)
-        .then(function (output) {
-        if (output.emitSkipped) {
-            if (output.outputFiles && output.outputFiles.length === 1) {
-                if (output.outputFiles[0].text === building.Not_In_Context) {
-                    return resolve({ emitDiffers: false });
-                }
-            }
-            return resolve({ emitDiffers: true });
-        }
-        else {
-            var jsFile = output.outputFiles.filter(function (x) { return path.extname(x.name) == ".js"; })[0];
-            if (!jsFile) {
+    var output = building_1.getRawOutput(project, query.filePath);
+    if (output.emitSkipped) {
+        if (output.outputFiles && output.outputFiles.length === 1) {
+            if (output.outputFiles[0].text === building.Not_In_Context) {
                 return resolve({ emitDiffers: false });
             }
-            else {
-                var emitDiffers = !fs.existsSync(jsFile.name) || fs.readFileSync(jsFile.name).toString() !== jsFile.text;
-                return resolve({ emitDiffers: emitDiffers });
-            }
         }
-    });
+        return resolve({ emitDiffers: true });
+    }
+    var jsFile = output.outputFiles.filter(function (x) { return path.extname(x.name) == ".js"; })[0];
+    if (!jsFile) {
+        return resolve({ emitDiffers: false });
+    }
+    else {
+        var emitDiffers = !fs.existsSync(jsFile.name) || fs.readFileSync(jsFile.name).toString() !== jsFile.text;
+        return resolve({ emitDiffers: emitDiffers });
+    }
 }
 exports.getOutputJsStatus = getOutputJsStatus;
 function softReset(query) {

lib/main/lang/modules/building.ts

@@ -106,20 +106,6 @@ export function getRawOutput(proj: project.Project, filePath: string): ts.EmitOu
     return output;
 }
 
-export function getRawOutputPostExternal(proj: project.Project, filePath: string): Promise<ts.EmitOutput> {
-    var output1 = getRawOutput(proj, filePath);
-    let sourceFile = proj.languageService.getSourceFile(filePath);
-    let sourceMapContents: { [index: string]: any } = {};
-    return runExternalTranspiler(
-      filePath, sourceFile.text, output1.outputFiles[0], proj, sourceMapContents
-    ).then(() => {
-      return {
-        outputFiles: output1.outputFiles,
-        emitSkipped: false
-      };
-    });
-}
-
 function getBabelInstance(projectDirectory: string) {
     return new Promise<any>(resolve => {
         if (!babels[projectDirectory]) {

lib/main/lang/projectService.ts

@@ -909,7 +909,7 @@ export function applyQuickFix(query: ApplyQuickFixQuery): Promise<ApplyQuickFixR
 interface GetOutputResponse {
     output: ts.EmitOutput;
 }
-import {getRawOutputPostExternal, getRawOutput} from "./modules/building";
+import {getRawOutput} from "./modules/building";
 export function getOutput(query: FilePathQuery): Promise<GetOutputResponse> {
     consistentPath(query);
     var project = getOrCreateProject(query.filePath);
@@ -938,26 +938,22 @@ interface GetOutputJsStatusResponse {
 export function getOutputJsStatus(query: FilePathQuery): Promise<GetOutputJsStatusResponse> {
     consistentPath(query);
     var project = getOrCreateProject(query.filePath);
-    return getRawOutputPostExternal(project, query.filePath)
-    .then((output) => {
-        if (output.emitSkipped) {
-            if (output.outputFiles && output.outputFiles.length === 1) {
-                if (output.outputFiles[0].text === building.Not_In_Context) {
-                    return resolve({ emitDiffers: false });
-                }
-            }
-            return resolve({ emitDiffers: true });
-        }
-        else {
-            var jsFile = output.outputFiles.filter(x=> path.extname(x.name) == ".js")[0];
-            if (!jsFile) {
+    var output = getRawOutput(project, query.filePath);
+    if (output.emitSkipped) {
+        if (output.outputFiles && output.outputFiles.length === 1) {
+            if (output.outputFiles[0].text === building.Not_In_Context) {
                 return resolve({ emitDiffers: false });
-            } else {
-                var emitDiffers = !fs.existsSync(jsFile.name) || fs.readFileSync(jsFile.name).toString() !== jsFile.text;
-                return resolve({ emitDiffers });
             }
         }
-    });
+        return resolve({ emitDiffers: true });
+    }
+    var jsFile = output.outputFiles.filter(x=> path.extname(x.name) == ".js")[0];
+    if (!jsFile) {
+        return resolve({ emitDiffers: false });
+    } else {
+        var emitDiffers = !fs.existsSync(jsFile.name) || fs.readFileSync(jsFile.name).toString() !== jsFile.text;
+        return resolve({ emitDiffers });
+    }
 }
 
 /**

lib/typings/atompromise.d.ts

@@ -33,7 +33,7 @@ interface PromiseConstructor {
      * and a reject callback used to reject the promise with a provided reason or error.
      */
     new <T>(init: (resolve: (value?: T | Promise<T>) => void, reject: (reason?: any) => void) => void): Promise<T>;
-
+ 
     <T>(init: (resolve: (value?: T | Promise<T>) => void, reject: (reason?: any) => void) => void): Promise<T>;
 
     /**