refactor: cleanup compiler runtimes (#4230)

- Cleanup "tsCompilerOnMessage" by factoring out separate methods for each 
  request type:
    * "compile"
    * "runtimeCompile"
    * "runtimeTranspile"
- Simplify control flow of compiler workers by a) no longer calling "close()" in worker runtime after a
single message; b) explicitly shutting down worker from host after a single message

Co-authored-by: Ryan Dahl <ry@tinyclouds.org>

1 files changed, 1 insertions, 5 deletions

diff --git a/cli/compilers/ts.rs b/cli/compilers/ts.rs
index b08a12beb..f2c8e41fe 100644
--- a/cli/compilers/ts.rs
+++ b/cli/compilers/ts.rs
@@ -620,11 +620,7 @@ async fn execute_in_thread(
     WorkerEvent::Message(buf) => Ok(buf),
     WorkerEvent::Error(error) => Err(error),
   }?;
-  // Compiler worker finishes after one request
-  // so we should receive signal that channel was closed.
-  // Then close worker's channel and join the thread.
-  let event = handle.get_event().await;
-  assert!(event.is_none());
+  // Shutdown worker and wait for thread to finish
   handle.sender.close_channel();
   join_handle.join().unwrap();
   Ok(buf)