From 0a4572d6a7b9cc3cb1e7383624d77acbe25808b2 Mon Sep 17 00:00:00 2001
From: RafaelGSS <rafael.nunu@hotmail.com>
Date: Thu, 6 Feb 2025 14:26:39 -0300
Subject: [PATCH] build: add skip_apidoc_files and include QUIC

PR-URL: https://github.com/nodejs/node/pull/56941
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
---
 Makefile                                                 | 4 +++-
 doc/api/cli.md                                           | 9 ---------
 doc/contributing/internal-api.md                         | 4 ++++
 src/node_options.cc                                      | 2 +-
 test/doctool/test-make-doc.mjs                           | 2 ++
 test/parallel/test-cli-node-options-docs.js              | 3 ++-
 .../test-process-env-allowed-flags-are-documented.js     | 2 ++
 7 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/Makefile b/Makefile
index 648f684d74a..fa1162787a3 100644
--- a/Makefile
+++ b/Makefile
@@ -780,7 +780,9 @@ test-v8 test-v8-intl test-v8-benchmarks test-v8-all:
 endif
 
 apidoc_dirs = out/doc out/doc/api out/doc/api/assets
-apidoc_sources = $(wildcard doc/api/*.md)
+skip_apidoc_files = doc/api/quic.md
+
+apidoc_sources = $(filter-out $(skip_apidoc_files), $(wildcard doc/api/*.md))
 apidocs_html = $(addprefix out/,$(apidoc_sources:.md=.html))
 apidocs_json = $(addprefix out/,$(apidoc_sources:.md=.json))
 
diff --git a/doc/api/cli.md b/doc/api/cli.md
index befab1aef58..91151e7cea0 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -993,14 +993,6 @@ If the ES module being `require()`'d contains top-level `await`, this flag
 allows Node.js to evaluate the module, try to locate the
 top-level awaits, and print their location to help users find them.
 
-### `--experimental-quic`
-
-<!--
-added: v23.8.0
--->
-
-Enables the experimental `node:quic` built-in module.
-
 ### `--experimental-require-module`
 
 <!-- YAML
@@ -3203,7 +3195,6 @@ one is included in the list below.
 * `--experimental-loader`
 * `--experimental-modules`
 * `--experimental-print-required-tla`
-* `--experimental-quic`
 * `--experimental-require-module`
 * `--experimental-shadow-realm`
 * `--experimental-specifier-resolution`
diff --git a/doc/contributing/internal-api.md b/doc/contributing/internal-api.md
index 47fa499da95..999a36a2cf3 100644
--- a/doc/contributing/internal-api.md
+++ b/doc/contributing/internal-api.md
@@ -14,6 +14,10 @@ application usage.
 
 Enables debugging of `ArrayBuffer` allocations.
 
+#### `--experimental-quic`
+
+Enable QUIC Protocol (under development)
+
 #### `--expose-internals`
 
 Allows the usage of `internal/*` modules, granting access to internal Node.js functionality.
diff --git a/src/node_options.cc b/src/node_options.cc
index 7bd16b7d777..614ad0735d9 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -442,7 +442,7 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             kAllowedInEnvvar,
             true);
   AddOption("--experimental-quic",
-            "experimental QUIC API",
+            "" /* undocumented until its development */,
             &EnvironmentOptions::experimental_quic,
             kAllowedInEnvvar);
   AddOption("--experimental-webstorage",
diff --git a/test/doctool/test-make-doc.mjs b/test/doctool/test-make-doc.mjs
index 33b949f06f9..490fc93d6da 100644
--- a/test/doctool/test-make-doc.mjs
+++ b/test/doctool/test-make-doc.mjs
@@ -46,9 +46,11 @@ const expectedJsons = linkedHtmls
                        .map((name) => name.replace('.html', '.json'));
 const expectedDocs = linkedHtmls.concat(expectedJsons);
 const renamedDocs = ['policy.json', 'policy.html'];
+const skipedDocs = ['quic.json', 'quic.html'];
 
 // Test that all the relative links in the TOC match to the actual documents.
 for (const expectedDoc of expectedDocs) {
+  if (skipedDocs.includes(expectedDoc)) continue;
   assert.ok(actualDocs.includes(expectedDoc), `${expectedDoc} does not exist`);
 }
 
diff --git a/test/parallel/test-cli-node-options-docs.js b/test/parallel/test-cli-node-options-docs.js
index 8bcb69cdcba..f3034ef64bc 100644
--- a/test/parallel/test-cli-node-options-docs.js
+++ b/test/parallel/test-cli-node-options-docs.js
@@ -101,7 +101,8 @@ for (const [, envVar, config] of nodeOptionsCC.matchAll(addOptionRE)) {
   }
 
   // NODE_OPTIONS
-  if (isInNodeOption && !hasTrueAsDefaultValue && new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false) {
+  if (isInNodeOption && !hasTrueAsDefaultValue &&
+    new RegExp(`\`${envVar}\``).test(nodeOptionsText) === false) {
     assert(false, `Should have option ${envVar} in NODE_OPTIONS documented`);
   }
 
diff --git a/test/parallel/test-process-env-allowed-flags-are-documented.js b/test/parallel/test-process-env-allowed-flags-are-documented.js
index 89b41da638f..0d6d06be47c 100644
--- a/test/parallel/test-process-env-allowed-flags-are-documented.js
+++ b/test/parallel/test-process-env-allowed-flags-are-documented.js
@@ -125,6 +125,8 @@ assert(undocumented.delete('--verify-base-objects'));
 assert(undocumented.delete('--no-verify-base-objects'));
 assert(undocumented.delete('--trace-promises'));
 assert(undocumented.delete('--no-trace-promises'));
+assert(undocumented.delete('--experimental-quic'));
+assert(undocumented.delete('--no-experimental-quic'));
 
 // Remove negated versions of the flags.
 for (const flag of undocumented) {