src: set default config as node.config.json

Author: marco-ippolito

Makefile

@@ -809,7 +809,7 @@ doc: $(NODE_EXE) doc-only ## Build Node.js, and then build the documentation wit
 
 out/doc:
 	mkdir -p $@
-	cp doc/node_config_json_schema.json $@
+	cp doc/node-config-schema.json $@
 
 # If it's a source tarball, doc/api already contains the generated docs.
 # Just copy everything under doc/api over.

doc/api/cli.md

@@ -919,8 +919,13 @@ added: REPLACEME
 
 > Stability: 1.0 - Early development
 
-Use this flag to specify a configuration file that will be loaded and parsed
-before the application starts.
+If the `--experimental-config-file` flag is present, Node.js will look for a
+`node.config.json` file in the current working directory and load it as a
+as configuration file.
+
+If [`--experimental-config-file-path`][] is present, Node.js will look for a
+configuration file at the specified path.
+
 Node.js will read the configuration file and apply the settings.
 The configuration file should be a JSON file
 with the following structure:
@@ -929,11 +934,9 @@ with the following structure:
 {
   "$schema": "https://nodejs.org/dist/REPLACEME/docs/node_config_json_schema.json",
   "nodeOptions": {
-    "experimental-transform-types": true,
     "import": [
       "amaro/transform"
     ],
-    "disable-warning": "ExperimentalWarning",
     "watch-path": "src",
     "watch-preserve-output": true
   }
@@ -944,7 +947,7 @@ In the `nodeOptions` field, only flags that are allowed in [`NODE_OPTIONS`][] ar
 No-op flags are not supported.
 Not all V8 flags are currently supported.
 
-It is possible to use the [official JSON schema](../node_config_json_schema.json)
+It is possible to use the [official JSON schema](../node-config-schema.json)
 to validate the configuration file, which may vary depending on the Node.js version.
 Each key in the configuration file corresponds to a flag that can be passed
 as a command-line argument. The value of the key is the value that would be
@@ -954,7 +957,7 @@ For example, the configuration file above is equivalent to
 the following command-line arguments:
 
 ```bash
-node --experimental-transform-types --import amaro/transform --disable-warning=ExperimentalWarning --watch-path=src --watch-preserve-output
+node --import amaro/transform --disable-warning=ExperimentalWarning --watch-path=src --watch-preserve-output
 ```
 
 The priority in configuration is as follows:
@@ -976,6 +979,22 @@ unknown keys or keys that cannot used in `NODE_OPTIONS`.
 Node.js will not sanitize or perform validation on the user-provided configuration,
 so **NEVER** use untrusted configuration files.
 
+### `--experimental-config-file-path`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1.0 - Early development
+
+Use this flag to specify the path to the configuration file that will be loaded
+and parsed before the application starts.
+Example:
+
+```bash
+node --experimental-config-file-path=/path/to/config.json index.js
+```
+
 ### `--experimental-eventsource`
 
 <!-- YAML
@@ -3833,6 +3852,7 @@ node --stack-trace-limit=12 -p -e "Error.stackTraceLimit" # prints 12
 [`--env-file-if-exists`]: #--env-file-if-existsconfig
 [`--env-file`]: #--env-fileconfig
 [`--experimental-addon-modules`]: #--experimental-addon-modules
+[`--experimental-config-file-path`]: #--experimental-config-file-path
 [`--experimental-sea-config`]: single-executable-applications.md#generating-single-executable-preparation-blobs
 [`--experimental-wasm-modules`]: #--experimental-wasm-modules
 [`--heap-prof-dir`]: #--heap-prof-dir

doc/node_config_json_schema.json doc/node-config-schema.jsondoc/node_config_json_schema.json renamed to doc/node-config-schema.json

@@ -169,6 +169,9 @@ Enable experimental addon module support.
 .It Fl -experimental-config-file
 Enable support for experimental config file
 .
+.It Fl -experimental-config-file-path
+Specifies the experimental config file path
+.
 .It Fl -experimental-import-meta-resolve
 Enable experimental ES modules support for import.meta.resolve().
 .

doc/node.1

@@ -8,22 +8,31 @@ namespace node {
 
 std::optional<std::string_view> ConfigReader::GetDataFromArgs(
     const std::vector<std::string>& args) {
-  constexpr std::string_view flag = "--experimental-config-file";
+  constexpr std::string_view flag_path = "--experimental-config-file-path";
+  constexpr std::string_view flag_file = "--experimental-config-file";
+
+  bool has_experimental_config_file = false;
 
   for (auto it = args.begin(); it != args.end(); ++it) {
-    if (*it == flag) {
-      // Case: "--experimental-config-file foo"
+    if (*it == flag_path) {
+      // Case: "--experimental-config-file-path foo"
       if (auto next = std::next(it); next != args.end()) {
         return *next;
       }
-    } else if (it->starts_with(flag)) {
-      // Case: "--experimental-config-file=foo"
-      if (it->size() > flag.size() && (*it)[flag.size()] == '=') {
-        return it->substr(flag.size() + 1);
+    } else if (it->starts_with(flag_path)) {
+      // Case: "--experimental-config-file-path=foo"
+      if (it->size() > flag_path.size() && (*it)[flag_path.size()] == '=') {
+        return it->substr(flag_path.size() + 1);
       }
+    } else if (*it == flag_file || it->starts_with(flag_file)) {
+      has_experimental_config_file = true;
     }
   }
 
+  if (has_experimental_config_file) {
+    return "node.config.json";
+  }
+
   return std::nullopt;
 }
 

src/node_config_file.cc

@@ -684,8 +684,12 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             &EnvironmentOptions::optional_env_file);
   Implies("--env-file-if-exists", "[has_env_file_string]");
   AddOption("--experimental-config-file",
-            "set config file from supplied file",
+            "set config file from node.config.json file",
             &EnvironmentOptions::experimental_config_file);
+  AddOption("--experimental-config-file-path",
+            "set config file from supplied file",
+            &EnvironmentOptions::experimental_config_file_path);
+  Implies("--experimental-config-file-path", "--experimental-config-file");
   AddOption("--test",
             "launch test runner on startup",
             &EnvironmentOptions::test_runner);

src/node_options.cc

@@ -258,7 +258,8 @@ class EnvironmentOptions : public Options {
 
   bool report_exclude_env = false;
   bool report_exclude_network = false;
-  std::string experimental_config_file;
+  bool experimental_config_file = false;
+  std::string experimental_config_file_path;
 
   inline DebugOptions* get_debug_options() { return &debug_options_; }
   inline const DebugOptions& debug_options() const { return debug_options_; }

src/node_options.h

@@ -0,0 +1,5 @@
+{
+  "nodeOptions": {
+    "max-http-header-size": 10
+  }
+}

test/fixtures/rc/default/node.config.json

@@ -0,0 +1,5 @@
+{
+  "nodeOptions": {
+    "max-http-header-size": 20
+  }
+}

test/fixtures/rc/default/override.json

@@ -0,0 +1,5 @@
+{
+    "nodeOptions": {
+        "max-http-header-size": 10
+    }
+}