Title: `make test-only` should not depend on doc-kit · Issue #62385 · nodejs/node · GitHub
Open Graph Title: `make test-only` should not depend on doc-kit · Issue #62385 · nodejs/node
X Title: `make test-only` should not depend on doc-kit · Issue #62385 · nodejs/node
Description: I noticed this from nodejs/doc-kit#691 which was introduced by #57343 and this does not look right make test-only is for functional tests. It's not for contributing or CI testing, but for embedders or distributors to verify the Node.js b...
Open Graph Description: I noticed this from nodejs/doc-kit#691 which was introduced by #57343 and this does not look right make test-only is for functional tests. It's not for contributing or CI testing, but for embedders...
X Description: I noticed this from nodejs/doc-kit#691 which was introduced by #57343 and this does not look right make test-only is for functional tests. It's not for contributing or CI testing, but for embed...
Opengraph URL: https://github.com/nodejs/node/issues/62385
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`make test-only` should not depend on doc-kit","articleBody":"I noticed this from https://github.com/nodejs/doc-kit/pull/691 which was introduced by https://github.com/nodejs/node/pull/57343 and this does not look right\n\n1. `make test-only` is for *functional* tests. It's not for contributing or CI testing, but for embedders or distributors to verify the Node.js binary they build is functioning correctly. It should not run any code building the docs into a website or the having to support monolithic dependency as that's irrelevant for the goal.\n2. `make test-only` should not require internet installs. Again it's only for people verifying the binary, you could just be scp-ing a tarball into a build server that does not necessarily have the permission to download from npm registry - and it'd be a pain having to work around it just so that you can test the Node.js binary works correctly, and do not care about the doc building at all.\n3. As `make test-only` is functional tests, it's what should be run on experimental platforms to verify the binary works correctly on a platform that's not yet well supported. `doc-kit` depends on e.g. prebuilt addons/wasm that would make it unnecessarily difficult for no good reason, as can be seen from https://github.com/nodejs/doc-kit/pull/691\n\nFrom a glance I think we should move https://github.com/nodejs/doc-kit/blob/main/src/generators/addon-verify/index.mjs back into core and replace it with a simple script with regexes that extract the artifacts. Unless there are other non-doc tests that depend on doc-kit - as far as I can tell the addon doc building is the only dependency, but it's an overkill having to e.g. run the minifier/highlighter (??!!) to extract the addons for testing. We can simply leave explicit markers in the markdown to deterministically know how the artifacts should be extracted, having to parse markdown and guess where they are based on headings is already hacky and brittle for this.","author":{"url":"https://github.com/joyeecheung","@type":"Person","name":"joyeecheung"},"datePublished":"2026-03-22T02:25:06.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/62385/node/issues/62385"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:dc94f313-074d-dfe8-2d10-d6bd83e01d8d |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | C698:264214:25F23B:36D18A:6A4D0A59 |
| html-safe-nonce | c6a3d37a4b22e2f4f436be4234240004205794fc6a91b6575a2f61369002fe90 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNjk4OjI2NDIxNDoyNUYyM0I6MzZEMThBOjZBNEQwQTU5IiwidmlzaXRvcl9pZCI6IjY5MTY3OTI4MjQ3ODUzNDEwMTciLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | c34f7f271b3ca0b331d6ab7fd3d5adc6b37c5e8463e5226c73589e24f92ced61 |
| hovercard-subject-tag | issue:4114440490 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/nodejs/node/62385/issue_layout |
| twitter:image | https://opengraph.githubassets.com/12dfd2254da02b0bd2be4fbe97c14d8d537570b0badfb15613029f3c887bad7d/nodejs/node/issues/62385 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/12dfd2254da02b0bd2be4fbe97c14d8d537570b0badfb15613029f3c887bad7d/nodejs/node/issues/62385 |
| og:image:alt | I noticed this from nodejs/doc-kit#691 which was introduced by #57343 and this does not look right make test-only is for functional tests. It's not for contributing or CI testing, but for embedders... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | joyeecheung |
| hostname | github.com |
| expected-hostname | github.com |
| None | 31e4db13a9e20081f2fac78f441659a6576e582bbb15e153f9c7fb6473aa29f5 |
| turbo-cache-control | no-preview |
| go-import | github.com/nodejs/node git https://github.com/nodejs/node.git |
| octolytics-dimension-user_id | 9950313 |
| octolytics-dimension-user_login | nodejs |
| octolytics-dimension-repository_id | 27193779 |
| octolytics-dimension-repository_nwo | nodejs/node |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 27193779 |
| octolytics-dimension-repository_network_root_nwo | nodejs/node |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | fb70bd3c4b2bec429781b65419e912c66e2d5581 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width