doc: add missing environment variables to manpage

PR-URL: #58963Fixes: #58894 Reviewed-By: Dario Piotrowicz <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: HumaneLogic

doc/node.1

@@ -693,12 +693,22 @@ disabled.
 .ItEvNO_COLOR
 Alias for NODE_DISABLE_COLORS
 .
+.ItEvNODE_COMPILE_CACHEArdir
+Enable the
+.Symodulecompilecache
+for the Node.js instance.
+.
 .ItEvNODE_DEBUGArmodules...
 Comma-separated list of core modules that should print debug information.
 .
 .ItEvNODE_DEBUG_NATIVEArmodules...
 Comma-separated list of C++ core modules that should print debug information.
 .
+.ItEvNODE_DISABLE_COMPILE_CACHE
+Disable the
+.Symodulecompilecache
+for the Node.js instance.
+.
 .ItEvNODE_DISABLE_COLORS
 When set to
 .Ar1 ,
@@ -754,6 +764,9 @@ When set to
 .Ar1 ,
 emit pending deprecation warnings.
 .
+.ItEvNODE_PENDING_PIPE_INSTANCES
+Set the number of pending pipe instance handles when the pipe server is waiting for connections. This setting applies to Windows only.
+.
 .ItEvNODE_PRESERVE_SYMLINKS
 When set to
 .Ar1 ,
@@ -787,11 +800,30 @@ the check for a supported platform is skipped during Node.js startup.
 Node.js might not execute correctly.
 Any issues encountered on unsupported platforms will not be fixed.
 .
+.ItEvNODE_TEST_CONTEXT
+When set to
+.Ar'child'
+, test reporter options will be overridden and test output will be sent to stdout in the TAP format.
+If any other value is provided, Node.js makes no guarantees about the reporter format used or its stability.
+.
 .ItEvNODE_TLS_REJECT_UNAUTHORIZED
 When set to
 .Ar0 ,
 TLS certificate validation is disabled.
 .
+.ItEvNODE_USE_ENV_PROXY
+When enabled, Node.js parses the
+.ArHTTP_PROXY
+,
+.ArHTTPS_PROXY
+and
+.ArNO_PROXY
+environment variables during startup, and tunnels requests over the specified proxy.
+.Pp
+This currently only affects requests sent over
+.Arfetch() .
+Support for other built-in http and https methods is under way.
+.
 .ItEvNODE_V8_COVERAGEArdir
 When set, Node.js writes JavaScript code coverage information to
 .Ardir .

test/parallel/test-cli-node-cli-manpage-env-vars.mjs

@@ -21,18 +21,8 @@ assert(manpageEnvVarNames.size > 0,
 'Unexpectedly not even a single env variable was detected when scanning the `doc/node.1` file'
 );
 
-// TODO(dario-piotrowicz): add the missing env variables to the manpage and remove this set
-// (refs: https://github.com/nodejs/node/issues/58894)
-constknownEnvVariablesMissingFromManPage=newSet([
-'NODE_COMPILE_CACHE',
-'NODE_DISABLE_COMPILE_CACHE',
-'NODE_PENDING_PIPE_INSTANCES',
-'NODE_TEST_CONTEXT',
-'NODE_USE_ENV_PROXY',
-]);
-
 for(constenvVarNameofcliMdEnvVarNames){
-if(!manpageEnvVarNames.has(envVarName)&&!knownEnvVariablesMissingFromManPage.has(envVarName)){
+if(!manpageEnvVarNames.has(envVarName)){
 assert.fail(`The "${envVarName}" environment variable (present in \`doc/api/cli.md\`) is missing from the \`doc/node.1\` file`);
 }
 manpageEnvVarNames.delete(envVarName);