doc: move to redesign

Makefile

@@ -876,7 +876,7 @@ VERSION=v$(RAWVER)
 
 .PHONY: doc-only
 .NOTPARALLEL: doc-only
-doc-only: $(apidoc_dirs) $(apidocs_html) $(apidocs_json) out/doc/api/all.json out/doc/llms.txt out/doc/apilinks.json  ## Builds the docs with the local or the global Node.js binary.
+doc-only: $(apidoc_dirs) $(apidocs_html) $(apidocs_json) out/doc/api/all.html out/doc/api/all.json out/doc/llms.txt out/doc/apilinks.json  ## Builds the docs with the local or the global Node.js binary.
 
 .PHONY: doc
 doc: $(NODE_EXE) doc-only ## Build Node.js, and then build the documentation with the new binary.
@@ -915,6 +915,7 @@ $(apidocs_html) $(apidocs_json) out/doc/api/all.html out/doc/api/all.json &: $(a
 			$(DOC_KIT) generate \
 			--log-level $(DOC_KIT_LOG_LEVEL) \
 			--config-file tools/doc/web.doc-kit.config.mjs \
+			-o out/doc/api \
 			-v $(VERSION) \
 			$(if $(JOBS),-p $(JOBS)) \
 		) \

test/doctool/test-make-doc.mjs

@@ -0,0 +1,60 @@
+import * as common from '../common/index.mjs';
+
+import assert from 'assert';
+import fs from 'fs';
+import path from 'path';
+
+if (common.isWindows) {
+  common.skip('`make doc` does not run on Windows');
+}
+
+// This tests that `make doc` generates the documentation properly: an HTML
+// page and a JSON file for every Markdown file under `doc/api`, and nothing
+// for anything else. Note that for this test to pass, `make doc` must be run
+// first.
+
+const apiURL = new URL('../../out/doc/api/', import.meta.url);
+const mdURL = new URL('../../doc/api/', import.meta.url);
+
+// Kept in sync with `skip_apidoc_files` in the Makefile.
+const skippedDocs = ['quic'];
+
+const sources = fs.readdirSync(mdURL)
+  .filter((name) => path.extname(name) === '.md')
+  .map((name) => path.basename(name, '.md'));
+const generated = fs.readdirSync(apiURL);
+
+// These have no Markdown source of their own.
+const expectedExtras = ['index.html', 'all.html', 'all.json'];
+for (const name of expectedExtras) {
+  assert.ok(generated.includes(name), `${name} was not generated`);
+}
+
+for (const name of sources) {
+  for (const extension of ['.html', '.json']) {
+    const file = `${name}${extension}`;
+    if (skippedDocs.includes(name)) {
+      assert.ok(!generated.includes(file), `${file} was unexpectedly generated`);
+      continue;
+    }
+    assert.ok(generated.includes(file), `${file} was not generated`);
+    assert.notStrictEqual(
+      fs.statSync(new URL(`./${file}`, apiURL)).size,
+      0,
+      `${file} is empty`,
+    );
+  }
+}
+
+// Every generated page must come from a current Markdown file, so a stale
+// `out/doc/api` (e.g. from a renamed document) is noticed.
+const expectedPages = ['404.html', 'apilinks.json', ...expectedExtras];
+for (const name of generated) {
+  const extension = path.extname(name);
+  if (extension !== '.html' && extension !== '.json') continue;
+  if (expectedPages.includes(name)) continue;
+  assert.ok(
+    sources.includes(path.basename(name, extension)),
+    `Unexpected output: out/doc/api/${name}, remove and rerun.`,
+  );
+}

tools/doc/api-links.doc-kit.config.mjs

@@ -1,17 +1,22 @@
-import { join } from 'node:path';
-import { pathToFileURL } from 'node:url';
+import { fileURLToPath } from 'node:url';
+
+const root = new URL('../../', import.meta.url);
 
 const fromRoot = (path) =>
-  pathToFileURL(join(import.meta.dirname, '..', '..', path)).href;
+  fileURLToPath(new URL(path, root)).replaceAll('\\', '/');
+
+const urlFromRoot = (path) => new URL(path, root).href;
 
 export default {
   extends: '@node-core/doc-kit/config',
 
   target: ['api-links'],
 
   global: {
-    input: ['lib/*.js'],
+    input: [fromRoot('lib/*.js')],
+    output: fromRoot('out/doc'),
 
-    changelog: fromRoot('CHANGELOG.md'),
+    // Point every loadable URL at its local file so no network request is made.
+    changelog: urlFromRoot('CHANGELOG.md'),
   },
 };