FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

cli: add --require-module/--no-require-module · nodejs/node@8f6fada · GitHub

/ node Public

Commit 8f6fada

Browse files
joyeecheung
authored and
RafaelGSS
committed
cli: add --require-module/--no-require-module
To replace --experimental-require-module and --no-experimental-require-module. The experimental ones are left as legacy aliases. PR-URL: #60959 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Jordan Harband <ljharb@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
1 parent 2e39f3e commit 8f6fada

9 files changed

Lines changed: 59 additions & 38 deletions

File tree

‎doc/api/cli.md‎

Lines changed: 37 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1225,27 +1225,6 @@ added: v25.0.0
12251225
12261226
Enable experimental support for the QUIC protocol.
12271227

1228-
### `--experimental-require-module`
1229-
1230-
<!-- YAML
1231-
added:
1232-
- v22.0.0
1233-
- v20.17.0
1234-
changes:
1235-
- version:
1236-
- v23.0.0
1237-
- v22.12.0
1238-
- v20.19.0
1239-
pr-url: https://github.com/nodejs/node/pull/55085
1240-
description: This is now true by default.
1241-
-->
1242-
1243-
> Stability: 1.1 - Active Development
1244-
1245-
Supports loading a synchronous ES module graph in `require()`.
1246-
1247-
See [Loading ECMAScript modules using `require()`][].
1248-
12491228
### `--experimental-sea-config`
12501229

12511230
<!-- YAML
@@ -1920,6 +1899,11 @@ added:
19201899
- v22.0.0
19211900
- v20.17.0
19221901
changes:
1902+
- version:
1903+
- REPLACEME
1904+
pr-url: https://github.com/nodejs/node/pull/60959
1905+
description: The flag was renamed from `--no-experimental-require-module` to
1906+
`--no-require-module`, with the former marked as legacy.
19231907
- version:
19241908
- v23.0.0
19251909
- v22.12.0
@@ -1928,11 +1912,9 @@ changes:
19281912
description: This is now false by default.
19291913
-->
19301914

1931-
> Stability: 1.1 - Active Development
1932-
1933-
Disable support for loading a synchronous ES module graph in `require()`.
1915+
> Stability: 3 - Legacy: Use [`--no-require-module`][] instead.
19341916
1935-
See [Loading ECMAScript modules using `require()`][].
1917+
Legacy alias for [`--no-require-module`][].
19361918

19371919
### `--no-experimental-sqlite`
19381920

@@ -2011,6 +1993,34 @@ changes:
20111993
Disables the family autoselection algorithm unless connection options explicitly
20121994
enables it.
20131995

1996+
<a id="--experimental-require-module"></a>
1997+
1998+
### `--no-require-module`
1999+
2000+
<!-- YAML
2001+
added:
2002+
- v22.0.0
2003+
- v20.17.0
2004+
changes:
2005+
- version:
2006+
- REPLACEME
2007+
pr-url: https://github.com/nodejs/node/pull/60959
2008+
description: This flag was renamed from `--no-experimental-require-module`
2009+
to `--no-require-module`.
2010+
- version:
2011+
- v23.0.0
2012+
- v22.12.0
2013+
- v20.19.0
2014+
pr-url: https://github.com/nodejs/node/pull/55085
2015+
description: This is now false by default.
2016+
-->
2017+
2018+
> Stability: 1.2 - Release Candidate
2019+
2020+
Disable support for loading a synchronous ES module graph in `require()`.
2021+
2022+
See [Loading ECMAScript modules using `require()`][].
2023+
20142024
### `--no-strip-types`
20152025

20162026
<!-- YAML
@@ -3624,6 +3634,7 @@ one is included in the list below.
36243634
* `--report-on-signal`
36253635
* `--report-signal`
36263636
* `--report-uncaught-exception`
3637+
* `--require-module`
36273638
* `--require`, `-r`
36283639
* `--secure-heap-min`
36293640
* `--secure-heap`
@@ -4164,6 +4175,7 @@ node --stack-trace-limit=12 -p -e "Error.stackTraceLimit" # prints 12
41644175
[`--experimental-sea-config`]: single-executable-applications.md#generating-single-executable-preparation-blobs
41654176
[`--heap-prof-dir`]: #--heap-prof-dir
41664177
[`--import`]: #--importmodule
4178+
[`--no-require-module`]: #--no-require-module
41674179
[`--no-strip-types`]: #--no-strip-types
41684180
[`--openssl-config`]: #--openssl-configfile
41694181
[`--preserve-symlinks`]: #--preserve-symlinks

‎doc/api/modules.md‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -324,8 +324,8 @@ If `--experimental-print-required-tla` is enabled, instead of throwing
324324
module, try to locate the top-level awaits, and print their location to
325325
help users fix them.
326326

327-
Support for loading ES modules using `require()` is currently
328-
experimental and can be disabled using `--no-experimental-require-module`.
327+
If support for loading ES modules using `require()` results in unexpected
328+
breakage, it can be disabled using `--no-require-module`.
329329
To print where this feature is used, use [`--trace-require-module`][].
330330

331331
This feature can be detected by checking if
@@ -422,7 +422,7 @@ LOAD_PACKAGE_IMPORTS(X, DIR)
422422
1. Find the closest package scope SCOPE to DIR.
423423
2. If no scope was found, return.
424424
3. If the SCOPE/package.json "imports" is null or undefined, return.
425-
4. If `--experimental-require-module` is enabled
425+
4. If `--no-require-module` is not enabled
426426
a. let CONDITIONS = ["node", "require", "module-sync"]
427427
b. Else, let CONDITIONS = ["node", "require"]
428428
5. let MATCH = PACKAGE_IMPORTS_RESOLVE(X, pathToFileURL(SCOPE),
@@ -436,7 +436,7 @@ LOAD_PACKAGE_EXPORTS(X, DIR)
436436
return.
437437
3. Parse DIR/NAME/package.json, and look for "exports" field.
438438
4. If "exports" is null or undefined, return.
439-
5. If `--experimental-require-module` is enabled
439+
5. If `--no-require-module` is not enabled
440440
a. let CONDITIONS = ["node", "require", "module-sync"]
441441
b. Else, let CONDITIONS = ["node", "require"]
442442
6. let MATCH = PACKAGE_EXPORTS_RESOLVE(pathToFileURL(DIR/NAME), "." + SUBPATH,

‎doc/node.1‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,9 @@ Disable top-level await keyword support in REPL.
216216
.It Fl -no-experimental-sqlite
217217
Disable the experimental node:sqlite module.
218218
.
219+
.It Fl -no-require-module
220+
Disable support for loading ECMAScript modules with require().
221+
.
219222
.It Fl -no-strip-types
220223
Disable type-stripping for TypeScript files.
221224
.

‎lib/internal/bootstrap/node.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ const features = {
283283
return binding.hasCachedBuiltins();
284284
},
285285
get require_module() {
286-
return getOptionValue('--experimental-require-module');
286+
return getOptionValue('--require-module');
287287
},
288288
get quic() {
289289
// TODO(@jasnell): When the implementation is updated to support Boring,

‎lib/internal/modules/cjs/loader.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1685,7 +1685,7 @@ function wrapSafe(filename, content, cjsModuleInstance, format) {
16851685
} else {
16861686
// For modules being loaded by `require()`, if require(esm) is disabled,
16871687
// don't try to reparse to detect format and just throw for ESM syntax.
1688-
shouldDetectModule = getOptionValue('--experimental-require-module');
1688+
shouldDetectModule = getOptionValue('--require-module');
16891689
}
16901690
}
16911691
const result = compileFunctionForCJSLoader(content, filename, false /* is_sea_main */, shouldDetectModule);
@@ -1884,7 +1884,7 @@ Module._extensions['.js'] = function(module, filename) {
18841884
const { source, format: loadedFormat } = loadSource(module, filename, format);
18851885
// Function require shouldn't be used in ES modules when require(esm) is disabled.
18861886
if ((loadedFormat === 'module' || loadedFormat === 'module-typescript') &&
1887-
!getOptionValue('--experimental-require-module')) {
1887+
!getOptionValue('--require-module')) {
18881888
const err = getRequireESMError(module, pkg, source, filename);
18891889
throw err;
18901890
}

‎lib/internal/modules/esm/loader.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@ class ModuleLoader {
299299
*/
300300
importSyncForRequire(mod, filename, source, isMain, parent) {
301301
const url = pathToFileURL(filename).href;
302-
if (!getOptionValue('--experimental-require-module')) {
302+
if (!getOptionValue('--require-module')) {
303303
throw new ERR_REQUIRE_ESM(url, true);
304304
}
305305

@@ -489,7 +489,7 @@ class ModuleLoader {
489489
}
490490

491491
if (formatFromLoad === 'module' || formatFromLoad === 'module-typescript') {
492-
if (!getOptionValue('--experimental-require-module')) {
492+
if (!getOptionValue('--require-module')) {
493493
throw new ERR_REQUIRE_ESM(url, true);
494494
}
495495
}

‎lib/internal/modules/esm/utils.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ function initializeDefaultConditions() {
7575
const userConditions = getOptionValue('--conditions');
7676
const noAddons = getOptionValue('--no-addons');
7777
const addonConditions = noAddons ? [] : ['node-addons'];
78-
const moduleConditions = getOptionValue('--experimental-require-module') ? ['module-sync'] : [];
78+
const moduleConditions = getOptionValue('--require-module') ? ['module-sync'] : [];
7979
defaultConditions = ObjectFreeze([
8080
'node',
8181
'import',

‎lib/internal/modules/helpers.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ function initializeCjsConditions() {
8484
...addonConditions,
8585
...userConditions,
8686
];
87-
if (getOptionValue('--experimental-require-module')) {
87+
if (getOptionValue('--require-module')) {
8888
cjsConditionsArray.push('module-sync');
8989
}
9090
ObjectFreeze(cjsConditionsArray);

‎src/node_options.cc‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -487,17 +487,23 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
487487
kAllowedInEnvvar,
488488
true);
489489
AddOption("--experimental-print-required-tla",
490-
"Print pending top-level await. If --experimental-require-module "
490+
"Print pending top-level await. If --require-module "
491491
"is true, evaluate asynchronous graphs loaded by `require()` but "
492492
"do not run the microtasks, in order to to find and print "
493493
"top-level await in the graph",
494494
&EnvironmentOptions::print_required_tla,
495495
kAllowedInEnvvar);
496-
AddOption("--experimental-require-module",
496+
AddOption("--require-module",
497497
"Allow loading synchronous ES Modules in require().",
498498
&EnvironmentOptions::require_module,
499499
kAllowedInEnvvar,
500500
true);
501+
AddOption("--experimental-require-module",
502+
"Legacy alias for --require-module",
503+
&EnvironmentOptions::require_module,
504+
kAllowedInEnvvar,
505+
true);
506+
Implies("--experimental-require-module", "--require-module");
501507
AddOption("--diagnostic-dir",
502508
"set dir for all output files"
503509
" (default: current working directory)",

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL