Move compiler-debug into Debug namespace, which allows the compiler to be tree shaken

This debug code was added quite a while ago, constructed such that we
wouldn't have to ship this code to our users.

However, this is the sole place in the compiler project where the ts
namespace "escapes" the bundle. By moving this debug code into the
compiler itself, we no longer have any references to the ts namespace
itself for our bundles that don't export anything (tsc,
typingsInstaller). This lets bundlers tree shake the compiler, reducing
the size of our output by _5.7 MB_ (a ridiculous improvement for
_adding_ code).

Author: jakebailey

Diff for: Herebyfile.mjs

@@ -360,15 +360,6 @@ function entrypointBuildTask(options) {
     return { build, bundle, shim, main, watch };
 }
 
-const { main: compilerDebug } = entrypointBuildTask({
-    name: "compiler-debug",
-    buildDeps: [generateDiagnostics],
-    project: "src/debug",
-    srcEntrypoint: "./src/debug/compilerDebug.ts",
-    builtEntrypoint: "./built/local/debug/compilerDebug.js",
-    output: "./built/local/compiler-debug.js",
-});
-
 
 const { main: tsc, watch: watchTsc } = entrypointBuildTask({
     name: "tsc",
@@ -378,7 +369,7 @@ const { main: tsc, watch: watchTsc } = entrypointBuildTask({
     srcEntrypoint: "./src/tsc/tsc.ts",
     builtEntrypoint: "./built/local/tsc/tsc.js",
     output: "./built/local/tsc.js",
-    mainDeps: [generateLibs, compilerDebug],
+    mainDeps: [generateLibs],
 });
 export { tsc, watchTsc };
 
@@ -391,7 +382,7 @@ const { main: services, build: buildServices, watch: watchServices } = entrypoin
     srcEntrypoint: "./src/typescript/typescript.ts",
     builtEntrypoint: "./built/local/typescript/typescript.js",
     output: "./built/local/typescript.js",
-    mainDeps: [generateLibs, compilerDebug],
+    mainDeps: [generateLibs],
     bundlerOptions: { exportIsTsObject: true },
 });
 export { services, watchServices };
@@ -412,7 +403,7 @@ const { main: tsserver, watch: watchTsserver } = entrypointBuildTask({
     srcEntrypoint: "./src/tsserver/server.ts",
     builtEntrypoint: "./built/local/tsserver/server.js",
     output: "./built/local/tsserver.js",
-    mainDeps: [generateLibs, compilerDebug],
+    mainDeps: [generateLibs],
     // Even though this seems like an exectuable, so could be the default CJS,
     // this is used in the browser too. Do the same thing that we do for our
     // libraries and generate an IIFE with name `ts`, as to not pollute the global
@@ -445,7 +436,7 @@ const { main: lssl, build: buildLssl, watch: watchLssl } = entrypointBuildTask({
     srcEntrypoint: "./src/tsserverlibrary/tsserverlibrary.ts",
     builtEntrypoint: "./built/local/tsserverlibrary/tsserverlibrary.js",
     output: "./built/local/tsserverlibrary.js",
-    mainDeps: [generateLibs, compilerDebug],
+    mainDeps: [generateLibs],
     bundlerOptions: { exportIsTsObject: true },
 });
 export { lssl, watchLssl };
@@ -473,7 +464,7 @@ const { main: tests, watch: watchTests } = entrypointBuildTask({
     srcEntrypoint: "./src/testRunner/_namespaces/Harness.ts",
     builtEntrypoint: "./built/local/testRunner/runner.js",
     output: testRunner,
-    mainDeps: [generateLibs, compilerDebug],
+    mainDeps: [generateLibs],
     bundlerOptions: {
         // Ensure we never drop any dead code, which might be helpful while debugging.
         treeShaking: false,