Skip to content

feat(blog): add more codemod to migrations guides - #9086

Open
AugustinMauroy wants to merge 6 commits into
mainfrom
blog-add-codemod
Open

feat(blog): add more codemod to migrations guides#9086
AugustinMauroy wants to merge 6 commits into
mainfrom
blog-add-codemod

Conversation

@AugustinMauroy

Copy link
Copy Markdown
Member

Description

As titled

Related Issues

No related issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Copilot AI lite review requested due to automatic review settings August 5, 2026 21:09
@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Aug 5, 2026 9:18pm

Request Review

@cursor

cursor Bot commented Aug 5, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Changes are limited to MDX migration blog content with no runtime, auth, or build-logic impact.

Overview
The v16→v18 migration post now documents the @nodejs/dns-lookup-options-coercion codemod for DEP0153 (typed dns.lookup / dns.promises.lookup options), with registry link, npx command, and before/after examples for callback and promises APIs. The err-invalid-callback section heading is pluralized to Examples to match the added pattern.

The v22→v24 guide adds a one-shot npx codemod run @nodejs/v22-to-v24 command and an info AlertBox clarifying that the listed codemods are not exhaustive.

Reviewed by Cursor Bugbot for commit 119e023. Bugbot is set up for automated code reviews on this repo. Configure here.

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.06%. Comparing base (c944548) to head (119e023).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9086   +/-   ##
=======================================
  Coverage   75.06%   75.06%           
=======================================
  Files         102      102           
  Lines        9111     9111           
  Branches      377      377           
=======================================
  Hits         6839     6839           
  Misses       2268     2268           
  Partials        4        4           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

This PR updates the Node.js migration blog guides to better surface available codemods, including adding a new codemod entry to the v16→v18 guide and documenting how to run all v22→v24 codemods together.

Changes:

  • Add a “run all codemods” command and an informational note to the v22→v24 migration guide.
  • Add the dns-lookup-options-coercion codemod section (with examples) to the v16→v18 migration guide.
  • Update the err-invalid-callback section heading from “Example” to “Examples”.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
apps/site/pages/en/blog/migrations/v22-to-v24.mdx Adds guidance for running all codemods at once and clarifies list completeness via an AlertBox.
apps/site/pages/en/blog/migrations/v16-to-v18.mdx Documents an additional codemod (dns-lookup-options-coercion) and aligns example heading wording.

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

Comment thread apps/site/pages/en/blog/migrations/v22-to-v24.mdx Outdated
Comment thread apps/site/pages/en/blog/migrations/v16-to-v18.mdx Outdated
Comment thread apps/site/pages/en/blog/migrations/v22-to-v24.mdx
Comment thread apps/site/pages/en/blog/migrations/v16-to-v18.mdx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 5, 2026 21:13
AugustinMauroy and others added 3 commits August 5, 2026 23:13
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Suppressed comments (1)

apps/site/pages/en/blog/migrations/v22-to-v24.mdx:72

  • The phrase “run all of them at once” is ambiguous here given the note that the list of codemods for this migration is not complete. Clarify that the command runs the codemods listed in this guide (or the preset), not necessarily every possible codemod for the migration.
If you want to run all of them at once, you can use the following command:

Copilot AI review requested due to automatic review settings August 5, 2026 21:18

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Suppressed comments (1)

apps/site/pages/en/blog/migrations/v22-to-v24.mdx:75

  • The doc suggests running all v22→v24 codemods via npx codemod run @nodejs/v22-to-v24, but that preset/package does not appear to exist in the Codemod Registry (the registry URL for it returns 404). This would cause the command to fail for readers. Consider replacing it with a single shell command that chains the individual codemods listed below in this guide (or link to an actual preset if one exists).
If you want to run all of them at once, you can use the following command:

```bash
npx codemod run @nodejs/v22-to-v24
</details>

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📦 Build Size Comparison

Summary

Metric Value
Old Total First Load JS 7.16 MB
New Total First Load JS 7.16 MB
Delta 0 B (0.00%)

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.

2 participants