esm: rename flag --esm-extension-resolution

There are currently two supported values
"none" and "cjs"

Author: MylesBorins

default.profraw

@@ -699,7 +699,7 @@ inline Maybe<URL> ResolveIndex(const URL& search) {
 Maybe<URL> FinalizeResolution(Environment* env,
                               const URL& resolved,
                               const URL& base) {
-  if (env->options()->legacy_resolution) {
+  if (env->options()->esm_extension_resolution == "cjs") {
     Maybe<URL> file = ResolveExtensions<TRY_EXACT_NAME>(resolved);
     if (!file.IsNothing()) {
       return file;

src/module_wrap.cc

@@ -236,9 +236,9 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "(default: llhttp).",
             &EnvironmentOptions::http_parser,
             kAllowedInEnvironment);
-  AddOption("--legacy-resolution",
-            "legacy extension and folder resolution for ES Modules",
-            &EnvironmentOptions::legacy_resolution,
+  AddOption("--esm-extension-resolution",
+            "Select extension resolution algorithm for esm; either 'none' or 'cjs'",
+            &EnvironmentOptions::esm_extension_resolution,
             kAllowedInEnvironment);
   AddOption("--no-deprecation",
             "silence deprecation warnings",

src/node_options.cc

@@ -86,7 +86,7 @@ class EnvironmentOptions : public Options {
  public:
   bool abort_on_uncaught_exception = false;
   bool experimental_modules = false;
-  bool legacy_resolution = false;
+  std::string esm_extension_resolution = "none";
   std::string module_type;
   std::string experimental_policy;
   bool experimental_repl_await = false;