Skip to content

Commit 190b972

Browse files
committed
doc: test for cli options
1 parent c975384 commit 190b972

File tree

4 files changed

+178
-3
lines changed

4 files changed

+178
-3
lines changed

doc/api/cli.md

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2618,11 +2618,15 @@ V8 options that are allowed are:
26182618

26192619
<!-- node-options-v8 end -->
26202620

2621+
<!-- node-options-others start -->
2622+
26212623
`--perf-basic-prof-only-functions`, `--perf-basic-prof`,
26222624
`--perf-prof-unwinding-info`, and `--perf-prof` are only available on Linux.
26232625

26242626
`--enable-etw-stack-walking` is only available on Windows.
26252627

2628+
<!-- node-options-others end -->
2629+
26262630
### `NODE_PATH=path[:…]`
26272631

26282632
<!-- YAML
@@ -2898,6 +2902,24 @@ options are of interest only to V8 developers. Despite this, there is a small
28982902
set of V8 options that are widely applicable to Node.js, and they are
28992903
documented here:
29002904

2905+
<!-- v8-options start -->
2906+
2907+
### `--abort-on-uncaught-exception`
2908+
2909+
### `--disallow-code-generation-from-strings`
2910+
### `--enable-etw-stack-walking`
2911+
### `--harmony-shadow-realm`
2912+
### `--huge-max-old-generation-size`
2913+
### `--jitless`
2914+
2915+
### `--interpreted-frames-native-stack`
2916+
2917+
### `--prof`
2918+
### `--perf-basic-prof`
2919+
### `--perf-basic-prof-only-functions`
2920+
### `--perf-prof`
2921+
### `--perf-prof-unwinding-info`
2922+
29012923
### `--max-old-space-size=SIZE` (in megabytes)
29022924

29032925
Sets the max memory size of V8's old memory section. As memory
@@ -2936,6 +2958,19 @@ for MiB in 16 32 64 128; do
29362958
done
29372959
```
29382960

2961+
### `--security-revert`
2962+
2963+
### `--stack-trace-limit=limit`
2964+
2965+
The maximum number of stack frames to collect in an error's stack trace.
2966+
Setting it to 0 disables stack trace collection. The default value is 10.
2967+
2968+
```bash
2969+
node --stack-trace-limit=12 -p -e "Error.stackTraceLimit" # prints 12
2970+
```
2971+
2972+
<!-- v8-options end -->
2973+
29392974
[#42511]: https://github.com/nodejs/node/issues/42511
29402975
[Chrome DevTools Protocol]: https://chromedevtools.github.io/devtools-protocol/
29412976
[CommonJS]: modules.md

doc/contributing/internal-api.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,12 @@ rules. The core developers may remove these flags in any version of Node.js.
88

99
### Flags
1010

11+
#### `--debug-arraybuffer-allocations`
12+
13+
#### `--expose-internals`
14+
15+
Allows to require the `internal/*` modules.
16+
1117
#### `--inspect-brk-node[=[host:]port]`
1218

1319
<!-- YAML
@@ -17,3 +23,11 @@ added: v7.6.0
1723
Activate inspector on `host:port` and break at start of the first internal
1824
JavaScript script executed when the inspector is available.
1925
Default `host:port` is `127.0.0.1:9229`.
26+
27+
#### `--node-snapshot`
28+
29+
#### `--test-udp-no-try-send`
30+
31+
#### `--trace-promises`
32+
33+
#### `--verify-base-objects`

doc/node.1

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -160,9 +160,6 @@ Interpret as either ES modules or CommonJS modules input via --eval or STDIN, wh
160160
.js or extensionless files with no sibling or parent package.json;
161161
.js or extensionless files whose nearest parent package.json lacks a "type" field, unless under node_modules.
162162
.
163-
.It Fl -experimental-global-webcrypto
164-
Expose the Web Crypto API on the global scope.
165-
.
166163
.It Fl -experimental-import-meta-resolve
167164
Enable experimental ES modules support for import.meta.resolve().
168165
.
Lines changed: 129 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
1+
'use strict';
2+
const common = require('../common');
3+
if (process.config.variables.node_without_node_options)
4+
common.skip('missing NODE_OPTIONS support');
5+
6+
// Test options specified by env variable.
7+
8+
const assert = require('assert');
9+
const fs = require('fs');
10+
const path = require('path');
11+
12+
const rootDir = path.resolve(__dirname, '..', '..');
13+
const cliMd = path.join(rootDir, 'doc', 'api', 'cli.md');
14+
const cliText = fs.readFileSync(cliMd, { encoding: 'utf8' });
15+
16+
const internalApiMd = path.join(rootDir, 'doc', 'contributing', 'internal-api.md');
17+
const internalApiText = fs.readFileSync(internalApiMd, { encoding: 'utf8' });
18+
19+
const nodeOptionsCC = fs.readFileSync(path.resolve(rootDir, 'src', 'node_options.cc'), 'utf8');
20+
const addOptionRE = /AddOption[\s\n\r]*\([\s\n\r]*"([^"]+)"(.*?)\);/gs;
21+
22+
const nodeOptionsText = cliText.match(/<!-- node-options-node start -->(.*)<!-- node-options-others end -->/s)[1];
23+
const v8OptionsText = cliText.match(/<!-- v8-options start -->(.*)<!-- v8-options end -->/s)[1];
24+
25+
const manPage = path.join(rootDir, 'doc', 'node.1');
26+
const manPageText = fs.readFileSync(manPage, { encoding: 'utf8' });
27+
28+
// Documented in /doc/api/deprecations.md
29+
const deprecated = [
30+
'--debug',
31+
'--debug-brk',
32+
];
33+
34+
35+
const manPagesOptions = new Set();
36+
37+
for (const [, envVar] of manPageText.matchAll(/\.It Fl (-[a-zA-Z0-9._-]+)/g)) {
38+
manPagesOptions.add(envVar);
39+
}
40+
41+
for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
42+
let hasTrueAsDefaultValue = false;
43+
let isInNodeOption = false;
44+
let isV8Option = false;
45+
let isNoOp = false;
46+
47+
if (config.includes('NoOp{}')) {
48+
isNoOp = true;
49+
}
50+
51+
if (config.includes('kAllowedInEnvvar')) {
52+
isInNodeOption = true;
53+
}
54+
if (config.includes('kDisallowedInEnvvar')) {
55+
isInNodeOption = false;
56+
}
57+
58+
if (config.includes('V8Option{}')) {
59+
isV8Option = true;
60+
}
61+
62+
if (/^\s*true\s*$/.test(config.split(',').pop())) {
63+
hasTrueAsDefaultValue = true;
64+
}
65+
66+
if (
67+
envVar.startsWith('[') ||
68+
deprecated.includes(envVar) ||
69+
isNoOp
70+
) {
71+
// assert(!manPagesOptions.has(envVar.slice(1)), `Option ${envVar} should not be documented`)
72+
manPagesOptions.delete(envVar.slice(1));
73+
continue;
74+
}
75+
76+
// Internal API options are documented in /doc/contributing/internal-api.md
77+
if (new RegExp(`####.*\`${envVar}[[=\\s\\b\`]`).test(internalApiText) === true) {
78+
manPagesOptions.delete(envVar.slice(1));
79+
continue;
80+
}
81+
82+
// CLI options
83+
if (!isV8Option && !hasTrueAsDefaultValue) {
84+
if (new RegExp(`###.*\`${envVar}[[=\\s\\b\`]`).test(cliText) === false) {
85+
assert(false, `Should have option ${envVar} documented`);
86+
} else {
87+
manPagesOptions.delete(envVar.slice(1));
88+
}
89+
}
90+
91+
if (!hasTrueAsDefaultValue && new RegExp(`###.*\`--no${envVar.slice(1)}[[=\\s\\b\`]`).test(cliText) === true) {
92+
assert(false, `Should not have option --no${envVar.slice(1)} documented`);
93+
}
94+
95+
if (!isV8Option && hasTrueAsDefaultValue) {
96+
if (new RegExp(`###.*\`--no${envVar.slice(1)}[[=\\s\\b\`]`).test(cliText) === false) {
97+
assert(false, `Should have option --no${envVar.slice(1)} documented`);
98+
} else {
99+
manPagesOptions.delete(`-no${envVar.slice(1)}`);
100+
}
101+
}
102+
103+
// NODE_OPTIONS
104+
if (isInNodeOption && !hasTrueAsDefaultValue && new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false) {
105+
assert(false, `Should have option ${envVar} in NODE_OPTIONS documented`);
106+
}
107+
108+
if (isInNodeOption && hasTrueAsDefaultValue && new RegExp(`\`--no${envVar.slice(1)}`).test(cliText) === false) {
109+
assert(false, `Should have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
110+
}
111+
112+
if (!hasTrueAsDefaultValue && new RegExp(`\`--no${envVar.slice(1)}`).test(cliText) === true) {
113+
assert(false, `Should not have option --no${envVar.slice(1)} in NODE_OPTIONS documented`);
114+
}
115+
116+
// V8 options
117+
if (isV8Option) {
118+
if (new RegExp(`###.*\`${envVar}[[=\\s\\b\`]`).test(v8OptionsText) === false) {
119+
assert(false, `Should have option ${envVar} in V8 options documented`);
120+
} else {
121+
manPagesOptions.delete(envVar.slice(1));
122+
}
123+
}
124+
}
125+
126+
// add alias handling
127+
manPagesOptions.delete('-trace-events-enabled');
128+
129+
assert.strictEqual(manPagesOptions.size, 0, `Man page options not documented: ${[...manPagesOptions]}`);

0 commit comments

Comments
 (0)