Skip to content

Doc kit docs#889

Draft
bmuenzenmeyer wants to merge 5 commits into
mainfrom
doc-kit-docs
Draft

Doc kit docs#889
bmuenzenmeyer wants to merge 5 commits into
mainfrom
doc-kit-docs

Conversation

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

Description

Introduces the doc-kit documentation site, built with doc-kit!
This also serves as a reasonable simple web example.

Validation

npm run docs:build
npx serve www/out
image

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run node --run test and all tests passed.
  • I have check code formatting with node --run format & node --run lint.
  • I've covered new added functionality with unit tests if necessary.

@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
api-docs-tooling Ready Ready Preview Jul 11, 2026 12:23am

Request Review

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 260 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.80%. Comparing base (d7bdf39) to head (09e884c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
www/theme/SideBar.jsx 0.00% 103 Missing ⚠️
scripts/build-docs-content.mjs 0.00% 82 Missing ⚠️
www/doc-kit.config.mjs 0.00% 75 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #889      +/-   ##
==========================================
- Coverage   85.04%   83.80%   -1.24%     
==========================================
  Files         179      183       +4     
  Lines       16496    16841     +345     
  Branches     1500     1512      +12     
==========================================
+ Hits        14029    14114      +85     
- Misses       2457     2717     +260     
  Partials       10       10              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

web Generator

File Base Head Diff
all.html 19.94 MB 19.94 MB -234.00 B (-0.00%)

@AugustinMauroy AugustinMauroy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice !

also we should add a "why doc-kit" section but with a "TBD" and in future have comparison with fuma-docs/docausorus ...

Comment thread www/pages/index.md

## It is a pipeline, not a Markdown converter

doc-kit is not a general-purpose Markdown-to-HTML tool. Input is parsed once

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should define how to "spell" doc-kit should it be simple text or doc-kit ?

Comment thread www/pages/index.md
Comment on lines +3 to +5
`@node-core/doc-kit` is the documentation toolchain behind the Node.js API
reference. It reads API-shaped Markdown and emits HTML, JSON, man pages,
search indexes, and `llms.txt`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
`@node-core/doc-kit` is the documentation toolchain behind the Node.js API
reference. It reads API-shaped Markdown and emits HTML, JSON, man pages,
search indexes, and `llms.txt`.
`@node-core/doc-kit` is the documentation toolchain originally design for
the Node.js API reference. It reads API-shaped Markdown and emits HTML,
JSON, man pages, search indexes, and `llms.txt`.

Comment thread www/pages/index.md
> page's identity — its sidebar label and its output filename. A file without
> one produces no page at all, and the build still exits `0`.

See [the specification](./specification.html) for the full input format.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why .html ?

Comment thread www/doc-kit.config.mjs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if I recall correctly on this file we can customize the color theme.
And IMO we should change it. Why not keep green ? this green will be understood as nodejs runtime related. so for undici, learn it's work well. But if we want to as general propose tool we should chose another colors.
Also it's good example of how can this be customized.

run: npm ci

- name: Build site
run: npm run docs:build

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
run: npm run docs:build
run: node --run docs:build

there are 0 reasons to use npm

Comment on lines +44 to +48
- name: Upload site artifact
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
with:
name: docs-site
path: www/out

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if we deploy the docs on GH page ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont have strong opinions here other that continuing our current patterns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants