Fix transpilationDependencies transpiling for no reason

Author: CompuIves

packages/app/src/sandbox/eval/manager.js

@@ -349,7 +349,7 @@ export default class Manager {
   verifyTreeTranspiled() {
     return Promise.all(
       this.getTranspiledModules()
-        .filter(tModule => !tModule.source)
+        .filter(tModule => tModule.shouldTranspile())
         .map(tModule => tModule.transpile(this))
     );
   }
@@ -689,7 +689,13 @@ export default class Manager {
     );
 
     return Promise.all(
-      transpiledModulesToUpdate.map(tModule => tModule.transpile(this))
+      transpiledModulesToUpdate.map(tModule => {
+        if (tModule.shouldTranspile()) {
+          return tModule.transpile(this);
+        }
+
+        return Promise.resolve(tModule);
+      })
     );
   }
 

packages/app/src/sandbox/eval/transpiled-module.js

@@ -292,6 +292,17 @@ export default class TranspiledModule {
     }
   }
 
+  /**
+   * Determines if this is a module that should be transpiled if updated. If this
+   * is a transpilationDependency that's updated then it should not get transpiled, but the parent should.
+   */
+  shouldTranspile() {
+    return (
+      !this.source &&
+      !(this.initiators.size === 0 && this.transpilationInitiators.size > 0)
+    );
+  }
+
   update(module: Module): TranspiledModule {
     if (this.module.path !== module.path || this.module.code !== module.code) {
       this.module = module;