refactor(legacy): extract @nodejs/doc-kit-generator-legacy package

[avivkeller]

The legacy-format generators (legacy-html, legacy-html-all, legacy-json, and legacy-json-all) have moved into the new @nodejs/doc-kit-generator-legacy package (entering npm at 1.0.0), and are loaded via import specifiers such as @nodejs/doc-kit-generator-legacy/legacy-html.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
api-docs-tooling	Ready	Preview	Aug 4, 2026 3:16pm

[cursor]

PR Summary

Medium Risk
Major semver and removal of @node-core/doc-kit/legacy-* exports break direct importers until they add @nodejs/doc-kit-generator-legacy; runtime behavior should match if the peer package is installed, matching the react generator split pattern.

Overview
Moves the four legacy generators (legacy-html, legacy-html-all, legacy-json, legacy-json-all) from @node-core/doc-kit into a new @nodejs/doc-kit-generator-legacy package (packages/legacy), with exports like @nodejs/doc-kit-generator-legacy/legacy-html.

Core removes the ./legacy-* package exports and adds an optional peer dependency on the legacy package. The generator lookup table in publicGenerators now points legacy shorthand names at the new import specifiers; CLI/config shorthand names are unchanged.

legacyToJSON is removed from core utils and lives under packages/legacy (tests moved accordingly). Legacy generator code now imports shared utilities from @node-core/doc-kit instead of relative paths into core.

Tooling updates coverage/eslint paths from packages/core/src/generators/legacy-html/assets to packages/legacy/src/legacy-html/assets. Changeset marks major bumps for @node-core/doc-kit and @nodejs/doc-kit-generator-legacy.

^{Reviewed by Cursor Bugbot for commit 7471891. Bugbot is set up for automated code reviews on this repo. Configure here.}

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.64%. Comparing base (60d4518) to head (7471891).

Additional details and impacted files

@@                  Coverage Diff                   @@
##           refactor/react-kitten     #960   +/-   ##
======================================================
  Coverage                  89.63%   89.64%           
======================================================
  Files                        198      200    +2     
  Lines                      18482    18495   +13     
  Branches                    1727     1729    +2     
======================================================
+ Hits                       16566    16579   +13     
  Misses                      1909     1909           
  Partials                       7        7           

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

[ovflowd]

The legacy-format generators (legacy-html, legacy-html-all, legacy-json, and legacy-json-all) have moved into the new @doc-kittens/legacy package, and are loaded via import specifiers such as @doc-kittens/legacy/legacy-html.

nit: can we name it @doc-kittens/legacy-nodejs? (also fine w/ current)

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 0f93e3d. Configure here.}

[Copilot]

Pull request overview

Copilot reviewed 26 out of 45 changed files in this pull request and generated 1 comment.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[bmuenzenmeyer]

Approving the stack so we can test and iterate

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.