module: disable require(esm) for policy and network import

joyeecheung · marco-ippolito · commit ed2d373e5a18 · 2025-02-11T11:58:20.000+01:00
These features are already removed since v22. There is no point supporting them to work with require(esm) in v20. PR-URL: #56927 Refs: #52697 Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
diff --git a/src/node_options.cc b/src/node_options.cc
@@ -121,6 +121,10 @@ void EnvironmentOptions::CheckOptions(std::vector<std::string>* errors,
     }
   }
 
+  if (!experimental_policy.empty() || experimental_https_modules) {
+    require_module = false;
+  }
+
   if (!type.empty()) {
     if (type != "commonjs" && type != "module") {
       errors->push_back("--experimental-default-type must be "

Original file line number	Diff line number	Diff line change
`@@ -121,6 +121,10 @@ void EnvironmentOptions::CheckOptions(std::vector<std::string>* errors,`
`121`	`121`	`}`
`122`	`122`	`}`
`123`	`123`
	`124`	`+if (!experimental_policy.empty() \|\| experimental_https_modules) {`
	`125`	`+require_module = false;`
	`126`	`+}`
	`127`	`+`
`124`	`128`	`if (!type.empty()) {`
`125`	`129`	`if (type != "commonjs" && type != "module") {`
`126`	`130`	`errors->push_back("--experimental-default-type must be "`