Skip to content

Content(migrations): update#8881

Open
AugustinMauroy wants to merge 6 commits intomainfrom
content-codemod-update
Open

Content(migrations): update#8881
AugustinMauroy wants to merge 6 commits intomainfrom
content-codemod-update

Conversation

@AugustinMauroy
Copy link
Copy Markdown
Member

Description

  • fix authors file
  • add recents codemod to their respective pos
  • add new post => update publish time to keep them sorted on blog/migrations page
  • use diff lang to display changes instead of before after tabs

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 build to check if the website builds without errors.

Copilot AI review requested due to automatic review settings May 7, 2026 15:07
@AugustinMauroy AugustinMauroy requested a review from a team as a code owner May 7, 2026 15:07
@vercel
Copy link
Copy Markdown

vercel Bot commented May 7, 2026

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

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview May 7, 2026 3:29pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented May 7, 2026

PR Summary

Low Risk
Low risk since changes are limited to site content/markdown and an authors.json key rename, with no runtime or backend logic impacted.

Overview
Adds new migration documentation pages for Axios → WHATWG Fetch and Node.js v16 → v18 (including the err-invalid-callback recipe).

Refreshes several existing migration posts to keep the registry current: fixes the util-print-to-console-log command in v12-to-v14, converts multiple examples to diff blocks (removing before/after tab formatting), updates publish timestamps to preserve sorting, and expands v20-to-v22/v22-to-v24 with newer codemods (notably crypto-createcipheriv-migration, http2-priority-removal, and tls-create-secure-pair-to-tls-socket).

Fixes the authors.json entry key for Richie McColl to use richiemccoll.

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

@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.23%. Comparing base (125b760) to head (c32ea58).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8881   +/-   ##
=======================================
  Coverage   73.23%   73.23%           
=======================================
  Files         102      102           
  Lines        8623     8623           
  Branches      313      313           
=======================================
  Hits         6315     6315           
  Misses       2307     2307           
  Partials        1        1           

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the “migrations” blog content to keep posts ordered, expand the list of available codemods, and standardize examples by switching from “Before/After” tabs to diff-formatted snippets. Also aligns the authors registry with the author id used by the Chalk migration post.

Changes:

  • Adjusts migration post publish timestamps to maintain intended sorting.
  • Replaces many “Before/After” code examples with ```diff blocks and adds new codemod sections (e.g., HTTP/2 priority removal, crypto createCipher migration, TLS secure pair migration, etc.).
  • Adds a new v16→v18 migrations post and updates the authors registry entry for richiemccoll.

Reviewed changes

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

Show a summary per file
File Description
apps/site/pages/en/blog/migrations/v22-to-v24.mdx Updates publish time, converts examples to diff, and adds multiple new codemod sections.
apps/site/pages/en/blog/migrations/v20-to-v22.mdx Updates publish time, converts import-assertions example to diff, and adds crypto codemod documentation.
apps/site/pages/en/blog/migrations/v16-to-v18.mdx Adds a new migrations page for v16→v18 with an initial codemod entry and examples.
apps/site/pages/en/blog/migrations/v14-to-v16.mdx Converts examples to diff formatting across multiple codemods.
apps/site/pages/en/blog/migrations/v12-to-v14.mdx Fixes the codemod command and converts the example to diff formatting.
apps/site/pages/en/blog/migrations/chalk-to-styletext.mdx Converts the Chalk→styleText example to diff and adjusts the sample transformation.
apps/site/authors.json Updates the author key for Richie McColl to match the id used in frontmatter (richiemccoll).
Comments suppressed due to low confidence (1)

apps/site/pages/en/blog/migrations/v14-to-v16.mdx:126

  • There is stray text ("reateRequireFromPath") after the final code block, which will render on the page. It looks like an accidental leftover and should be removed.
reateRequireFromPath


💡 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/v22-to-v24.mdx
Comment thread apps/site/pages/en/blog/migrations/v22-to-v24.mdx
Comment thread apps/site/pages/en/blog/migrations/v22-to-v24.mdx
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit e8f33c1. Configure here.

Comment thread apps/site/pages/en/blog/migrations/v14-to-v16.mdx Outdated
@AugustinMauroy
Copy link
Copy Markdown
Member Author

WFT copilot ???

Capture d’écran 2026-05-07 à 17 24 52

@araujogui
Copy link
Copy Markdown
Member

WFT copilot ???
Capture d’écran 2026-05-07 à 17 24 52

I guess it's because copilot is enabled by default on this repository

@AugustinMauroy
Copy link
Copy Markdown
Member Author

WFT copilot ???
Capture d’écran 2026-05-07 à 17 24 52

I guess it's because copilot is enabled by default on this repository

@ovflowd you didn't disable copilot ?

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.

3 participants