docs: add Concurrency section comparing child_process, worker_threads, and cluster#110
Conversation
…, and cluster Adds a new Learn article that compares Node.js's three built-in concurrency mechanisms side by side, with examples and guidance on when to reach for each one.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Navigation and contributor docs are updated so the section is discoverable: a Concurrency sidebar group in Reviewed by Cursor Bugbot for commit 5aa3e63. Bugbot is set up for automated code reviews on this repo. Configure here. |
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website Please review the changes when you have a chance. Thank you! 🙏 |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5695b86. Configure here.
Addresses review feedback: the exit handler previously restarted a worker unconditionally, which would also respawn workers during a graceful shutdown (worker.disconnect()/cluster.disconnect()).
|
Good catch — fixed in 863218b: the exit handler now checks |
The three comparison tables used long full-sentence cells across 4-5 columns, making rows wider than the rest of the site's content and forcing horizontal scrolling. Shortened cell content so tables fit within the normal content width.
Tables without alignment markers in the separator row (plain --- vs :---) were rendered by doc-kit without a text-align inline style on th/td, unlike every other table in this repo. That missing style appears to interact badly with the site's layout, causing the whole content column (not just the table) to overflow horizontally instead of wrapping. Matching the :--- convention used elsewhere fixes it.
The H1 was 82 characters, tied for the longest title in the repo. Headings on this site use white-space:nowrap (for the anchor-link icon layout), and a heading this long appears to force the whole content column wider than the viewport instead of wrapping, which explains why paragraphs and tables on this page overflowed horizontally while every other page wraps normally. Shortened to match the site's typical title length; the fuller description now lives in the opening paragraph instead.
Co-authored-by: Aviv Keller <me@aviv.sh> Signed-off-by: Omprakash Selvaraj <cse.omprakash@gmail.com>
Co-authored-by: Aviv Keller <me@aviv.sh> Signed-off-by: Omprakash Selvaraj <cse.omprakash@gmail.com>
Co-authored-by: Aviv Keller <me@aviv.sh> Signed-off-by: Omprakash Selvaraj <cse.omprakash@gmail.com>
3 participants
Summary
child_process,worker_threads, andclusterside by side — comparison tables, runnable examples, and a decision guide for choosing between them.site.jsonsidebar navigation and thepages/index.mdhomepage list to include the new section.CONTRIBUTING.md's section table.Closes #109
Test plan
npm run lint— passesnpm run format:check— passesnpm run build— succeeds; verified generated HTML contains all expected headings and the homepage links to the new page