Skip to content

feat(doc): add specification#723

Merged
avivkeller merged 2 commits into
mainfrom
specification
Apr 1, 2026
Merged

feat(doc): add specification#723
avivkeller merged 2 commits into
mainfrom
specification

Conversation

@avivkeller

Copy link
Copy Markdown
Member

Closes #402

This PR introduces a formal specification for the Markdown format consumed by doc-kit. I drafted it over the past few days while thinking through how to align webpack's documentation output (for our GSoC) with doc-kit's parseable structure, then revised and formalized it with AI.

The specification documents the Markdown we currently support. However, it defines a <!--module=node:fs/promises--> directive, as @Ethan-Arrowood proposed in #402. This allows sub-sections of a document to be associated with a specific module when needed (see §4.4.4 and §6.2.6).

@vercel

vercel Bot commented Mar 27, 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 Mar 27, 2026 9:14pm

Request Review

@avivkeller avivkeller marked this pull request as ready for review March 27, 2026 17:49
@avivkeller avivkeller requested a review from a team as a code owner March 27, 2026 17:49
Copilot AI review requested due to automatic review settings March 27, 2026 17:49
@codecov

codecov Bot commented Mar 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.15%. Comparing base (84ee128) to head (da789c7).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #723   +/-   ##
=======================================
  Coverage   76.15%   76.15%           
=======================================
  Files         151      151           
  Lines       13641    13641           
  Branches     1066     1066           
=======================================
  Hits        10388    10388           
  Misses       3248     3248           
  Partials        5        5           

☔ View full report in Codecov by Sentry.
📢 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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a formal Markdown specification for the documentation format parsed by @node-core/doc-kit, intended to codify the supported structure and metadata conventions (and to address #402’s request to define a ruleset).

Changes:

  • Introduces docs/specification.md describing document structure, heading semantics, YAML-in-HTML-comment metadata, stability indicators, typed parameter lists, and cross-references.
  • Defines (in spec) a <!--module=...--> directive for scoping sub-sections to a module.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md
Comment thread docs/specification.md Outdated
Comment thread docs/specification.md
Comment thread docs/specification.md

@ovflowd ovflowd 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.

SGTM! Really good stuff, this is phenomenal work. I've left a few comments!

@github-actions

Copy link
Copy Markdown
Contributor

orama-db Generator

File Base Head Diff
orama-db.json 8.08 MB 8.08 MB +1.00 B (+0.00%)

@ovflowd

ovflowd commented Mar 29, 2026

Copy link
Copy Markdown
Member

cc @nodejs/web-infra would be good to have more reviews before we land this!

@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.

that seem good

@Ethan-Arrowood Ethan-Arrowood left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a great start! Excellent work.

@avivkeller avivkeller added this pull request to the merge queue Mar 31, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Mar 31, 2026
@avivkeller avivkeller merged commit b4a3262 into main Apr 1, 2026
20 checks passed
@avivkeller avivkeller deleted the specification branch April 1, 2026 15:04
@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

I'm late to this - but this is really stellar!

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.

Document structure of parseable documentation markdown

6 participants