BREAKING(cli/tsc): Enable isolatedModules by default (#8050)

Author: nayeemrmn

cli/module_graph2.rs

@@ -698,17 +698,14 @@ impl Graph2 {
     options: CheckOptions,
   ) -> Result<(Stats, Diagnostics, Option<IgnoredCompilerOptions>), AnyError>
   {
-    // TODO(@kitsonk) set to `true` in followup PR
-    let unstable = options.lib == TypeLib::UnstableDenoWindow
-      || options.lib == TypeLib::UnstableDenoWorker;
     let mut config = TsConfig::new(json!({
       "allowJs": true,
       // TODO(@kitsonk) is this really needed?
       "esModuleInterop": true,
       // Enabled by default to align to transpile/swc defaults
       "experimentalDecorators": true,
       "incremental": true,
-      "isolatedModules": unstable,
+      "isolatedModules": true,
       "lib": options.lib,
       "module": "esnext",
       "strict": true,

cli/tsc.rs

@@ -716,8 +716,7 @@ pub async fn runtime_compile(
     "allowNonTsExtensions": true,
     "checkJs": false,
     "esModuleInterop": true,
-    // TODO(lucacasonato): enable this by default in 1.5.0
-    "isolatedModules": unstable,
+    "isolatedModules": true,
     "jsx": "react",
     "module": "esnext",
     "sourceMap": true,

cli/tsc_config.rs

@@ -69,6 +69,7 @@ const IGNORED_COMPILER_OPTIONS: [&str; 61] = [
   "inlineSourceMap",
   "inlineSources",
   "init",
+  // TODO(nayeemrmn): Add "isolatedModules" here for 1.6.0.
   "listEmittedFiles",
   "listFiles",
   "mapRoot",