From d6e69fbd25dcdd41c05390f7ce363546cce37fd2 Mon Sep 17 00:00:00 2001
From: Guy Bedford <guybedford@gmail.com>
Date: Mon, 7 Oct 2019 00:09:20 -0400
Subject: [PATCH] esm: unflag --experimental-exports

PR-URL: https://github.com/nodejs/node/pull/29867
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
---
 doc/api/cli.md                     | 8 --------
 doc/api/modules.md                 | 2 +-
 lib/internal/modules/cjs/loader.js | 3 +--
 src/module_wrap.cc                 | 4 +---
 src/node_options.cc                | 5 -----
 src/node_options.h                 | 1 -
 6 files changed, 3 insertions(+), 20 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index cac1734c4d73f8..27947553d34fdb 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -161,13 +161,6 @@ the ability to import a directory that has an index file.
 
 Please see [customizing esm specifier resolution][] for example usage.
 
-### `--experimental-exports`
-<!-- YAML
-added: v12.7.0
--->
-
-Enable experimental resolution using the `exports` field in `package.json`.
-
 ### `--experimental-json-modules`
 <!-- YAML
 added: v12.9.0
@@ -1052,7 +1045,6 @@ Node.js options that are allowed are:
 * `--enable-fips`
 * `--enable-source-maps`
 * `--es-module-specifier-resolution`
-* `--experimental-exports`
 * `--experimental-json-modules`
 * `--experimental-loader`
 * `--experimental-modules`
diff --git a/doc/api/modules.md b/doc/api/modules.md
index e9d1e89b4327ea..4f51d66b1dda99 100644
--- a/doc/api/modules.md
+++ b/doc/api/modules.md
@@ -203,7 +203,7 @@ NODE_MODULES_PATHS(START)
 5. return DIRS
 ```
 
-If `--experimental-exports` is enabled, Node.js allows packages loaded via
+Node.js allows packages loaded via
 `LOAD_NODE_MODULES` to explicitly declare which file paths to expose and how
 they should be interpreted. This expands on the control packages already had
 using the `main` field.
diff --git a/lib/internal/modules/cjs/loader.js b/lib/internal/modules/cjs/loader.js
index c37105fa1ed04b..196a5b4528cadd 100644
--- a/lib/internal/modules/cjs/loader.js
+++ b/lib/internal/modules/cjs/loader.js
@@ -71,7 +71,6 @@ const {
 } = require('internal/errors').codes;
 const { validateString } = require('internal/validators');
 const pendingDeprecation = getOptionValue('--pending-deprecation');
-const experimentalExports = getOptionValue('--experimental-exports');
 
 module.exports = Module;
 
@@ -373,7 +372,7 @@ function findLongestRegisteredExtension(filename) {
 const EXPORTS_PATTERN = /^((?:@[^/\\%]+\/)?[^./\\%][^/\\%]*)(\/.*)?$/;
 function resolveExports(nmPath, request, absoluteRequest) {
   // The implementation's behavior is meant to mirror resolution in ESM.
-  if (experimentalExports && !absoluteRequest) {
+  if (!absoluteRequest) {
     const [, name, expansion = ''] =
       StringPrototype.match(request, EXPORTS_PATTERN) || [];
     if (!name) {
diff --git a/src/module_wrap.cc b/src/module_wrap.cc
index 5448cea575dc67..6a97d61d87ee3a 100644
--- a/src/module_wrap.cc
+++ b/src/module_wrap.cc
@@ -651,8 +651,7 @@ Maybe<const PackageConfig*> GetPackageConfig(Environment* env,
   }
 
   Local<Value> exports_v;
-  if (env->options()->experimental_exports &&
-      pkg_json->Get(env->context(),
+  if (pkg_json->Get(env->context(),
       env->exports_string()).ToLocal(&exports_v) &&
       !exports_v->IsNullOrUndefined()) {
     Global<Value> exports;
@@ -994,7 +993,6 @@ Maybe<URL> PackageExportsResolve(Environment* env,
                                  const std::string& pkg_subpath,
                                  const PackageConfig& pcfg,
                                  const URL& base) {
-  CHECK(env->options()->experimental_exports);
   Isolate* isolate = env->isolate();
   Local<Context> context = env->context();
   Local<Value> exports = pcfg.exports.Get(isolate);
diff --git a/src/node_options.cc b/src/node_options.cc
index d57742510ee857..a042c5073c9cc2 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -326,10 +326,6 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "experimental Source Map V3 support",
             &EnvironmentOptions::enable_source_maps,
             kAllowedInEnvironment);
-  AddOption("--experimental-exports",
-            "experimental support for exports in package.json",
-            &EnvironmentOptions::experimental_exports,
-            kAllowedInEnvironment);
   AddOption("--experimental-json-modules",
             "experimental JSON interop support for the ES Module loader",
             &EnvironmentOptions::experimental_json_modules,
@@ -344,7 +340,6 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "experimental ES Module support and caching modules",
             &EnvironmentOptions::experimental_modules,
             kAllowedInEnvironment);
-  Implies("--experimental-modules", "--experimental-exports");
   AddOption("--experimental-wasm-modules",
             "experimental ES Module support for webassembly modules",
             &EnvironmentOptions::experimental_wasm_modules,
diff --git a/src/node_options.h b/src/node_options.h
index bea7e28a33ce57..f627f2d397d05d 100644
--- a/src/node_options.h
+++ b/src/node_options.h
@@ -101,7 +101,6 @@ class EnvironmentOptions : public Options {
  public:
   bool abort_on_uncaught_exception = false;
   bool enable_source_maps = false;
-  bool experimental_exports = false;
   bool experimental_json_modules = false;
   bool experimental_modules = false;
   std::string es_module_specifier_resolution;