src: conditionally disable source phase imports by default

Author: codebytere

src/node.cc

@@ -780,7 +780,9 @@ static ExitCode ProcessGlobalArgsInternal(std::vector<std::string>* args,
     env_opts->abort_on_uncaught_exception = true;
   }
 
-  v8_args.emplace_back("--js-source-phase-imports");
+  if (per_process::cli_options->js_source_phase_imports) {
+    v8_args.emplace_back("--js-source-phase-imports");
+  }
 
 #ifdef __POSIX__
   // Block SIGPROF signals when sleeping in epoll_wait/kevent/etc.  Avoids the

src/node_options.cc

@@ -1419,6 +1419,10 @@ PerProcessOptionsParser::PerProcessOptionsParser(
       "performance.",
       &PerProcessOptions::disable_wasm_trap_handler,
       kAllowedInEnvvar);
+  AddOption("--js_source_phase_imports",
+            "Enable JavaScript source phase imports.",
+            &PerProcessOptions::js_source_phase_imports,
+            kAllowedInEnvvar);
 }
 
 inline std::string RemoveBrackets(const std::string& host) {

src/node_options.h

@@ -368,6 +368,7 @@ class PerProcessOptions : public Options {
 #endif
 
   bool disable_wasm_trap_handler = false;
+  bool js_source_phase_imports = false;
 
   // Per-process because reports can be triggered outside a known V8 context.
   bool report_on_fatalerror = false;