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 2:44am

[cursor]

PR Summary

High Risk
Major semver bump removes core package exports and requires a new dependency; misconfigured installs will break legacy doc generation at dynamic load time, though generator behavior is largely a move with import path updates.

Overview
Legacy HTML and JSON generators are moved out of @node-core/doc-kit into a new @nodejs/doc-kit-generator-legacy package (legacy-html, legacy-html-all, legacy-json, legacy-json-all). The core package drops the ./legacy-* exports and lists the new package as an optional peer; CLI shorthand names (legacy-html, etc.) still resolve through publicGenerators to @nodejs/doc-kit-generator-legacy/*.

Implementation code and assets live under packages/legacy; generators import shared utilities from @node-core/doc-kit instead of relative core paths. legacyToJSON moves from core utils/generators.mjs into the legacy package (tests relocated accordingly). Tooling paths (c8, ESLint) point at packages/legacy/src/legacy-html/assets.

Breaking: consumers must install @nodejs/doc-kit-generator-legacy and use @nodejs/doc-kit-generator-legacy/legacy-html (etc.) instead of @node-core/doc-kit/legacy-html — marked major in the changeset for both packages.

^{Reviewed by Cursor Bugbot for commit e2f3047. 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.65%. Comparing base (eb5ee4a) to head (e2f3047).

Additional details and impacted files

@@                  Coverage Diff                   @@
##           refactor/react-kitten     #960   +/-   ##
======================================================
  Coverage                  89.64%   89.65%           
======================================================
  Files                        198      200    +2     
  Lines                      18480    18493   +13     
  Branches                    1727     1729    +2     
======================================================
+ Hits                       16566    16579   +13     
  Misses                      1907     1907           
  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.